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

init release commit #10

Merged
merged 25 commits into from
Dec 16, 2023
Merged

init release commit #10

merged 25 commits into from
Dec 16, 2023

Conversation

yeshaokai
Copy link
Collaborator

@yeshaokai yeshaokai commented Dec 11, 2023

  • full code release of amadeusGPT source code and streamlit app

@yeshaokai
Copy link
Collaborator Author

The pytest failure is caused by API key not in the secret variable. I probably do not have access to setting the secret variable in this repo.

@yeshaokai yeshaokai self-assigned this Dec 11, 2023
@MMathisLab
Copy link
Member

MMathisLab commented Dec 14, 2023

@yeshaokai can you confirm the errors are still due to a missing openAI key; it, to me, seems a code error: missing amadeusgpt.modules -- but I might have updated something inadvertently in my testing...

Edit to add: fixed with removing the import * in main.py

@MMathisLab MMathisLab mentioned this pull request Dec 14, 2023
@MMathisLab
Copy link
Member

failing due to

=========================== short test summary info ============================
ERROR tests/test_EPM.py - FileNotFoundError: [Errno 2] No such file or directory: '/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/amadeusgpt/interface.txt'
ERROR tests/test_MABE.py - FileNotFoundError: [Errno 2] No such file or directory: '/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/amadeusgpt/interface.txt'
ERROR tests/test_MausHaus.py - FileNotFoundError: [Errno 2] No such file or directory: '/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/amadeusgpt/interface.txt'
!!!!!!!!!!!!!!!!!!! Interrupted: 3 errors during collection !!!!!!!!!!!!!!!!!!!!
========================= 1 warning, 3 errors in 5.56s =========================

- this is needed to fun cebra, but fails in github testing...
Copy link
Member

@MMathisLab MMathisLab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see convo to resolve failing test in another PR;

  • The test is failing due to poor import *
  • and then interface.txt not being read properly.
  • also need to decide if pinning torch, cebra, etc is needed for some OS.

@MMathisLab MMathisLab merged commit ec944f1 into main Dec 16, 2023
4 of 5 checks passed
@MMathisLab MMathisLab deleted the release_branch branch December 16, 2023 19:23
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.

2 participants