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

Fix JSONLoader _rootObjectBatches removed but not mentioned in upgrade guide #2558

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Jul 18, 2023

Closes #2556

PR updates upgrade documentation with changes to json module for 3.0.0 release.

@nreese
Copy link
Contributor Author

nreese commented Jul 18, 2023

cc @ibgreen

I ran into problems upgrading from v2.3 to v3.4. Changes from #1570 where not documented in upgrade documentation.

Copy link
Collaborator

@ibgreen ibgreen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nreese Many thanks. I guess this slipped between the cracks because it was an experimental underscore option and not considered part of the official API. But it doesn't hurt to mention it.

Current docs are deployed from 3.4 branch so this won't show up for a while unless we backport.

@ibgreen ibgreen merged commit 729619b into visgl:master Jul 19, 2023
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.

JSONLoader _rootObjectBatches removed but not mentioned in upgrade guide
2 participants