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

Update citation info #68

Merged
merged 2 commits into from
Feb 3, 2024
Merged

Update citation info #68

merged 2 commits into from
Feb 3, 2024

Conversation

jinningwang
Copy link
Member

  • Update citation info
  • Update copyright

@jinningwang
Copy link
Member Author

@zmalkmus
Zack, I noticed that in main.py, the method selftest remains unchanged. Maybe you can integrate the tests you developed there? Thus to enable the cli agvis st.
Corresponding ANDES practice is here, https://github.com/CURENT/andes/blob/00e2675af32a709dc7913450df42c81301344c8b/andes/main.py#L788

@zmalkmus
Copy link
Collaborator

zmalkmus commented Feb 3, 2024

@zmalkmus Zack, I noticed that in main.py, the method selftest remains unchanged. Maybe you can integrate the tests you developed there? Thus to enable the cli agvis st. Corresponding ANDES practice is here, https://github.com/CURENT/andes/blob/00e2675af32a709dc7913450df42c81301344c8b/andes/main.py#L788

Whoops, I forgot to add that part when I was changing the ./go.sh script to automate the testing. I will do this.

@zmalkmus zmalkmus merged commit bd5488c into CURENT:develop Feb 3, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zmalkmus zmalkmus zmalkmus approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jinningwang @zmalkmus