-
Notifications
You must be signed in to change notification settings - Fork 6
replace (local/global_)arrays functions by (locals/globals_)to_session functions? #609
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
Comments
You kind of open Pandora's box if you do that, but that is bound to happen one day or another, so let's start:
|
You know how LArray is used in some in-house models. If we modify Well, the best is maybe to not fix this issue as long as a user doesn't ask. |
@gdementen I think this issue is no longer a question. We now have Session objects that contain axes, groups and arrays. Still having |
Indeed, but there are open questions anyway: how to name the new functions and the other questions I raise above. Concerning the name, I still don't really like locals_to_session and globals_to_session. |
@gdementen Once #578 implemented, Sessions should include Axis and Group objects by default, so... Do you agree with this issue?
The text was updated successfully, but these errors were encountered: