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

[BUG] The calypso interface is not available for the current devel branch #1065

Closed
Chengqian-Zhang opened this issue Dec 3, 2022 · 3 comments · Fixed by #1068
Closed

[BUG] The calypso interface is not available for the current devel branch #1065

Chengqian-Zhang opened this issue Dec 3, 2022 · 3 comments · Fixed by #1068
Labels
bug Something isn't working

Comments

@Chengqian-Zhang
Copy link
Collaborator

Summary

The calypso interface is not available for the current devel branch. “ArgumentValueError: [at root location] get invalid choice calypso for flag key model_devi_engine”

DPGEN Version and Platform

The newest devel branch.

Expected Behavior

Job successfully submitted.

Actual Behavior

image

Steps to Reproduce

image
@Chengqian-Zhang Chengqian-Zhang added the bug Something isn't working label Dec 3, 2022
@njzjz
Copy link
Member

njzjz commented Dec 3, 2022 •
edited
Loading

@y1xiaoc Is it expected when strict=False? I thought loose checking will allow this situation but it looks not.

@y1xiaoc
Copy link
Contributor

y1xiaoc commented Dec 3, 2022

flag has to be in one if those choices. I don't think non strict mode would release this requirement, since it will not be able to do any checking in that case.

njzjz added a commit to njzjz/dpgen that referenced this issue Dec 8, 2022
@njzjz
add placeholders for variants
491f430 
Fix deepmodeling#1065. Fix deepmodeling#1067.
@njzjz njzjz mentioned this issue Dec 8, 2022
Merged
@njzjz njzjz linked a pull request Dec 8, 2022 that will close this issue
add placeholders for variants #1068
Merged
wanghan-iapcm pushed a commit that referenced this issue Dec 19, 2022
@njzjz
add placeholders for variants (#1068)
6ea1812 
Fix #1065. Fix #1067.

Needs someone to finish the doc...
@njzjz
Copy link
Member

njzjz commented Dec 20, 2022

Fixed in #1068.

@njzjz njzjz closed this as completed Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add placeholders for variants njzjz/dpgen
3 participants
@njzjz @y1xiaoc @Chengqian-Zhang