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

Devel #23

Merged
merged 28 commits into from
Jul 13, 2022
Merged

Devel #23

merged 28 commits into from
Jul 13, 2022

Conversation

ZLI-afk
Copy link

@ZLI-afk ZLI-afk commented Jul 12, 2022

fix bugs for fcc gamma line calculations

njzjz and others added 28 commits June 9, 2022 12:51
* refactor simplify

1. Used `dp model-devi` to calculate model deviation, instead of local calculation. Supported by deepmodeling/deepmd-kit#1618, released in v2.1.1. So the version earlier than 2.1.1 is not supported any more.
2. Assumed all systems are MultiSystems.
3. Removed energy model deviation support

* expand path when getting multisystems

* let `make_train` and `run_train` expand paths

* load numpy array instead

* use dpdata to get nframes

* fix tests

* update README
* supports non-list mdata

The mdata of a task is a list of a single dict. This "list" looks useless and not clear enough. So this commit supports using the dict without a list.

Note that old parameters are still supported, so no breaking changes are made. The "list" is just removed from all examples. Both list and non-list parameters are in the unittest.

* fix typo
* upgrade all tasks to dpdispatcher

This commit upgrades init_reaction and init_surf to use dpdispatcher

* fix method args

* fix typo

* change the variable name from `work_dir` to `work_path`
…/EOS.py (deepmodeling#741)

* update: add a new option of absolute volume in ./dpgen/auto_test/EOS.py

* update: add doc in /dpgen/doc/toymodels/

* update: change the description for eos, change the doc in /dpgen/doc/toymodels/

* update: change the notice of absolute volume from print into dlog.info
)

fix compatibility as the key was changed in the dpdispatcher
* generate machine parameter page for simplify and init

* switching to new dargs directive
* add auto cli docs

* fix typo

* fix package name...

* forgot to return parser

* add the blank line
* fix ‘post_fp_cp2k’, add param rfailed

* Update run.py
* refactor simplify

1. Used `dp model-devi` to calculate model deviation, instead of local calculation. Supported by deepmodeling/deepmd-kit#1618, released in v2.1.1. So the version earlier than 2.1.1 is not supported any more.
2. Assumed all systems are MultiSystems.
3. Removed energy model deviation support

* expand path when getting multisystems

* let `make_train` and `run_train` expand paths

* load numpy array instead

* use dpdata to get nframes

* fix tests

* update README
* supports non-list mdata

The mdata of a task is a list of a single dict. This "list" looks useless and not clear enough. So this commit supports using the dict without a list.

Note that old parameters are still supported, so no breaking changes are made. The "list" is just removed from all examples. Both list and non-list parameters are in the unittest.

* fix typo
* upgrade all tasks to dpdispatcher

This commit upgrades init_reaction and init_surf to use dpdispatcher

* fix method args

* fix typo

* change the variable name from `work_dir` to `work_path`
…/EOS.py (deepmodeling#741)

* update: add a new option of absolute volume in ./dpgen/auto_test/EOS.py

* update: add doc in /dpgen/doc/toymodels/

* update: change the description for eos, change the doc in /dpgen/doc/toymodels/

* update: change the notice of absolute volume from print into dlog.info
)

fix compatibility as the key was changed in the dpdispatcher
* generate machine parameter page for simplify and init

* switching to new dargs directive
* add auto cli docs

* fix typo

* fix package name...

* forgot to return parser

* add the blank line
* fix ‘post_fp_cp2k’, add param rfailed

* Update run.py
@kevinwenminion kevinwenminion merged commit 3f9add2 into kevinwenminion:devel Jul 13, 2022
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.

5 participants