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.

Sign up for GitHub

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

Jump to bottom

Processes classes and variable attributes as attr extension, stores, and way too much!! #33

Merged
merged 97 commits into from
May 7, 2018
Merged

Processes classes and variable attributes as attr extension, stores, and way too much!! #33

merged 97 commits into from
May 7, 2018

Conversation

benbovy
Copy link
Member

@benbovy benbovy commented Mar 6, 2018 •
edited
Loading

Closes issues #19 #20 #26 #28 #29 #31.

@benbovy benbovy changed the title WIP: big refactoring of internals and interface WIP: processes classes and variable arttributes as attr extension Mar 8, 2018
@benbovy benbovy changed the title WIP: processes classes and variable arttributes as attr extension WIP: processes classes and variable attributes as attr extension Mar 8, 2018
benbovy added 22 commits March 8, 2018 20:01
@benbovy
allow intent to in, out or inout and maybe set read-only properties
418b35a 
TODO: fix write access to foreign variables declared as input in their own
process
@benbovy
better variable properties + rename AttrType to VarType
705babb
@benbovy
rename get_variables to filter_variables + add func argument
b28b58f
@benbovy
minor tweaks
7643840
@benbovy
start writing what's new entries
34c13ed
@benbovy
detect cycles when looking for target variable (maybe remove later)
1b361c2
@benbovy
perform more sanity checks when creating variable properties
b668248
@benbovy
rename __xsimlab_keys__ to __xsimlab_store_keys__ + fix group keys
92bae94
@benbovy
docstrings tweaks
35801e5
@benbovy
change default intent from 'inout' to 'in' for variable and foreign
1dc85a9
@benbovy
WIP refactor Model (add _ModelBuilder class)
df7bae9
@benbovy
fix handling of on-demand variables in get_input_variables
72653e8
@benbovy
use Enum for variable intent
421cb4e
@benbovy
WIP model builder (process dependencies and sorting)
3fd166f
@benbovy
change intent rules for foreign variables
fbd606e
@benbovy
complete ModelBuilder implementation
335b6a0
@benbovy
fix variable group
e461452
@benbovy
update Model input variables API
f15ef9b
@benbovy
some docstring tweaks
55ce033
@benbovy
fix process deps when a variable involves more than 2 processes
4b554f3
@benbovy
identify which processes have to be executed at each simulation stage
5840463
@benbovy
update whats new with some enhancements
f9521cb
benbovy added 20 commits May 2, 2018 16:51
@benbovy
wip update tests
ab5202b
@benbovy
update tests
0c8ea35
@benbovy
fix tests and increase coverage
cf93f49
@benbovy
fix tests for python < 3.6 (dict order)
b7488ab
@benbovy
drop py34 support and require xarray >= 0.10.0
ef92931 
Use some features added in xarray 0.10.0.

Conda packages are not been built anymore (both conda-forge and
default channels) for a while (older versions of some packages not
available in conda for py34).
@benbovy
update travis conf (add xarray and attrs dev to test matrix)
a9267db
@benbovy
fix travis conf
f83d2d0
@benbovy
fix travis take 2 (fix conda env name in dev tests)
3886cbd
@benbovy
update doc (framework and api)
536f083
@benbovy
add line return at the end of all repr
2c5ca50
@benbovy
fix xarray simulation driver (avoid xarray objects for time steps)
cb5e2d6
@benbovy
wip update doc (update all user guide sections)
7143027
@benbovy
doc add some content
8a6ae4e
@benbovy
try build docs in test matrix (travis)
c454c87
@benbovy
doc build in travis: build docs instead of running tests
86aa886
@benbovy
try fix doc build in travis 2
b54b316
@benbovy
disable push build on PRs (push build only on master)
1510d4f
@benbovy
simplify doc conf (rtd theme)
366fc36
@benbovy
require attrs >= 18.1.0 (use fields_dict internally)
f70da01
@benbovy
doc update examples
6f0f8be
@benbovy benbovy changed the title WIP: processes classes and variable attributes as attr extension, stores, and way too much!! Processes classes and variable attributes as attr extension, stores, and way too much!! May 7, 2018
@benbovy benbovy merged commit 65f4f01 into master May 7, 2018
This was referenced May 7, 2018
Closed
Closed
Closed
Closed
Closed
Closed
Closed
@benbovy benbovy deleted the big-refactor-process branch December 11, 2019 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@benbovy