Skip to content

Conversation

@mottosso
Copy link
Contributor

Continuing from #388, for some reason I wasn't able to push directly to that branch.

As this PR merely deletes unused code and that it's been proposed for over two months, I'm merging this now.

BigRoy and others added 5 commits June 4, 2019 09:44
- Remove old manager and loader, replace with cbloader and cbsceneinventory
- Refactor cbloader -> loader
- Refactor cbsceneinventory -> sceneinventory
# Conflicts:
#	avalon/tools/cbloader/app.py
@BigRoy
Copy link
Collaborator

BigRoy commented Aug 20, 2019

Just a heads up.

This was still remaining as an open PR as it would introduce backwards compatibility issues for studio tools that relied on the widgets/models that were in these tools.

Another reason #414 made some sensible choices as we rely a lot on the AssetWidget internally too.

Any way, I'll refactor internal code when I'm back from vacation to what it needs to be by then. ;)

@mottosso
Copy link
Contributor Author

Thanks Roy

@mottosso
Copy link
Contributor Author

For future reference, this is why it's important that external users only reference api.py and host-API's. Anything outside of those modules are implementation detail. That will allow us to safely make these kinds of changes, without breaking anything.

If there's something you need that isn't exposed via an API, submit an issue.

For us internally, we can avoid this becoming an issue as well by gracefully deprecating parts of our code. For example, we could reverse the AVALON_EARLY_ADOPTER flag, such that it defaults to cbloader but can be restored by a temporary variable, e.g. AVALON_LATE_ADOPTER=true. That way, we can deploy, people can test, and if it's a problem they won't have to wait for a PR to retain compatibility.

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