Skip to content
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

[components][fix] Fix issue with additional scope determination #27043

Conversation

OwenKephart
Copy link
Contributor

Summary & Motivation

Fixes a weird issue I ran into. In essence, in pydantic 3.8.x, the generated json schema will have the json_schema_extras stored on dereferenced json schemas of any subtype, whereas in 3.10.x, the extras will be stored on the reference itself. No idea why this is different but it is what it is.

How I Tested These Changes

Changelog

NOCHANGELOG

@OwenKephart OwenKephart force-pushed the 01-08-_components_docs_schema_viewer branch from 7488996 to 1fe20a5 Compare January 11, 2025 01:46
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from 34d6a92 to b85c7e5 Compare January 11, 2025 01:47
@OwenKephart OwenKephart force-pushed the 01-08-_components_docs_schema_viewer branch from 1fe20a5 to 200b269 Compare January 13, 2025 16:57
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from b85c7e5 to b741d37 Compare January 13, 2025 16:57
@OwenKephart OwenKephart force-pushed the 01-08-_components_docs_schema_viewer branch 2 times, most recently from 983c653 to 3bd5528 Compare January 13, 2025 17:26
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from b741d37 to ecdf2a8 Compare January 13, 2025 17:26
@OwenKephart OwenKephart force-pushed the 01-08-_components_docs_schema_viewer branch from 3bd5528 to 392db15 Compare January 14, 2025 19:12
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from ecdf2a8 to 78418ae Compare January 14, 2025 19:13
@OwenKephart OwenKephart force-pushed the 01-08-_components_docs_schema_viewer branch from 392db15 to b1feabe Compare January 14, 2025 22:36
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from 78418ae to 217a183 Compare January 14, 2025 22:36
@OwenKephart OwenKephart force-pushed the 01-08-_components_docs_schema_viewer branch from b1feabe to 91025ae Compare January 14, 2025 22:42
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from 217a183 to a92b720 Compare January 14, 2025 22:42
@OwenKephart OwenKephart force-pushed the 01-08-_components_docs_schema_viewer branch from 91025ae to 21fe4a1 Compare January 14, 2025 22:44
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from a92b720 to 023cb87 Compare January 14, 2025 22:44
@OwenKephart OwenKephart force-pushed the 01-08-_components_docs_schema_viewer branch from 21fe4a1 to 1f53636 Compare January 14, 2025 23:06
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from 023cb87 to f5bae12 Compare January 14, 2025 23:06
Copy link
Contributor Author

OwenKephart commented Jan 14, 2025

Merge activity

  • Jan 14, 6:21 PM EST: A user started a stack merge that includes this pull request via Graphite.
  • Jan 14, 6:26 PM EST: Graphite rebased this pull request as part of a merge.
  • Jan 14, 6:27 PM EST: A user merged this pull request with Graphite.

@OwenKephart OwenKephart changed the base branch from 01-08-_components_docs_schema_viewer to graphite-base/27043 January 14, 2025 23:22
@OwenKephart OwenKephart changed the base branch from graphite-base/27043 to master January 14, 2025 23:23
@OwenKephart OwenKephart force-pushed the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch from f5bae12 to 24ec7c9 Compare January 14, 2025 23:25
@OwenKephart OwenKephart merged commit ef7725f into master Jan 14, 2025
1 check was pending
@OwenKephart OwenKephart deleted the 01-10-_components_fix_fix_issue_with_additional_scope_determination branch January 14, 2025 23:27
marijncv pushed a commit to marijncv/dagster that referenced this pull request Jan 21, 2025
…ter-io#27043)

## Summary & Motivation

Fixes a weird issue I ran into. In essence, in pydantic 3.8.x, the generated json schema will have the `json_schema_extras` stored on dereferenced json schemas of any subtype, whereas in 3.10.x, the extras will be stored on the reference itself. No idea why this is different but it is what it is.


## How I Tested These Changes

## Changelog

NOCHANGELOG
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.

2 participants