Skip to content

Blog for Import loops#207

Merged
tawsifkamal merged 12 commits into
developfrom
blog-for-import-loops
Feb 5, 2025
Merged

Blog for Import loops#207
tawsifkamal merged 12 commits into
developfrom
blog-for-import-loops

Conversation

@tawsifkamal

Copy link
Copy Markdown
Contributor

Adding a pytorch specific usecase blog for import loops

@tawsifkamal tawsifkamal requested review from a team and codegen-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

CLAassistant commented Jan 30, 2025

Copy link
Copy Markdown

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.

@codecov

codecov Bot commented Jan 30, 2025

Copy link
Copy Markdown

❌ 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!

Comment thread docs/blog/fixing-import-loops.mdx Outdated
@tawsifkamal tawsifkamal merged commit bcf53f3 into develop Feb 5, 2025
@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