Skip to content

Blog for Import loops #207

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Feb 5, 2025
Merged

Blog for Import loops #207

merged 12 commits into from
Feb 5, 2025

Conversation

tawsifkamal
Copy link
Contributor

Adding a pytorch specific usecase blog for import loops

@tawsifkamal tawsifkamal requested review from codegen-team and a team as code owners January 30, 2025 05:36
@tawsifkamal tawsifkamal requested review from bagel897 and removed request for a team January 30, 2025 05:37
@CLAassistant
Copy link

CLAassistant commented Jan 30, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ tawsifkamal
❌ codegen-bot


codegen-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

codecov bot commented Jan 30, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
1882 1 1881 57
View the full list of 1 ❄️ flaky tests
tests.unit.codegen.sdk.code_generation.test_api_doc_generation test_mdx_api_doc_generation_sanity@codegen

Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)

Stack Traces | 83.5s run time
codebase = <Codebase(name=codegen-sdk, language=PYTHON, path=/codegen-sdk)>

    @pytest.mark.timeout(120)
    @pytest.mark.xdist_group("codegen")
    def test_mdx_api_doc_generation_sanity(codebase) -> None:
>       docs_json = generate_docs_json(codebase, "HEAD")

.../sdk/code_generation/test_api_doc_generation.py:53: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
.../code_generation/doc_utils/generate_docs_json.py:156: in generate_docs_json
    method_doc = process_method(method, cls, cls_doc, seen_methods)
.../code_generation/doc_utils/generate_docs_json.py:84: in process_method
    return_type = replace_multiple_types(
.../code_generation/doc_utils/utils.py:337: in replace_multiple_types
    return process_parts(input_str)
.../code_generation/doc_utils/utils.py:320: in process_parts
    processed_bracket = process_parts(bracket_content)
.../code_generation/doc_utils/utils.py:324: in process_parts
    replacement = find_symbol(codebase=codebase, symbol_name=part, resolved_types=resolved_types, parent_class=parent_class, parent_symbol=parent_symbol, types_cache=types_cache)
.../code_generation/doc_utils/utils.py:224: in find_symbol
    cls_obj = safe_get_class(codebase, symbol_name)
.../code_generation/doc_utils/utils.py:188: in safe_get_class
    symbols = codebase.get_symbols(class_name)
.../sdk/core/codebase.py:599: in get_symbols
    return sort_editables(x for x in self.symbols if x.name == symbol_name)
.../sdk/core/codebase.py:325: in symbols
    return self._symbols()
.../sdk/core/codebase.py:217: in _symbols
    matches: list[Symbol] = self.G.get_nodes(NodeType.SYMBOL)
.../sdk/codebase/codebase_graph.py:526: in get_nodes
    return [self.get_node(node_id) for node_id in self._graph.filter_nodes(lambda node: node.node_type == node_type)]
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = CodebaseGraph, node_id = 5581

>   def get_node(self, node_id: int) -> Any:
E   Failed: Timeout >120.0s

.../sdk/codebase/codebase_graph.py:518: Failed

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

@tawsifkamal tawsifkamal merged commit bcf53f3 into develop Feb 5, 2025
26 of 33 checks passed
@tawsifkamal tawsifkamal deleted the blog-for-import-loops branch February 5, 2025 01:30
jayhack pushed a commit that referenced this pull request Feb 9, 2025
Adding a pytorch specific usecase blog for import loops

---------

Co-authored-by: codegen-bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants