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

Update requirements file for dev env #241

Merged
merged 2 commits into from
Mar 6, 2023
Merged

Update requirements file for dev env #241

merged 2 commits into from
Mar 6, 2023

Conversation

vpchung
Copy link
Member

@vpchung vpchung commented Mar 6, 2023

Bug

When doing local development, the following libraries/packages are missing from the requirements file:

  • jellyfish
  • tabulate
  • scipy

Proposed fix

Add missing libraries/packages to requirements-dev.txt. Also update synapseclient version to the latest stable version (2.7.0).

@vpchung vpchung requested review from a team and rrchai and removed request for a team March 6, 2023 18:29
@vpchung vpchung merged commit 1fa68d8 into develop Mar 6, 2023
@vpchung vpchung deleted the dev-reqs branch March 6, 2023 18:47
vpchung added a commit that referenced this pull request Mar 17, 2023
* fix UnboundLocalError when calling `valid` in command_validate_docker (#224)

* Add code owners (#227)

* Add code owners

* Add extra line

* Make code of conduct changes (#228)

* Add setup.cfg (#229)

* Add setup.cfg

* lint and fix versioning

* Add HACK comment

* Use black (#230)

* Add black

* Lint

* Lint with black

* Give Organizers team "Moderate"-level access to live project (#233)

* add new MODERATE permission level

* give organizers team moderate permissions to live site

* add CLI function for adding Moderator flair

* update test

* fix lint

* Add CodeQL workflow for GitHub code scanning (#235)

Co-authored-by: LGTM Migrator <lgtm-migrator@users.noreply.github.com>

* Add support for py10 (#237)

* Add support for py10

* Fix requirements

* use v3 (#236)

* Update cmd name in setup config (#239)

* update cmd name

* fix lint (#240)

* Update requirements file for dev env (#241)

* update requirements file for dev env

* update synapseclient version

* Cheat detection (#242)

* Adding cheat detection functions

* add additional comments and documentation

* added in rst module description

* adding in cheat detection tests

* add in tests and evaluation round limit handlers

* make testing adjustments

* added package installation and command line fixes

* adding in exception handling

* additional exception handling

* documentation and descriptions

* adjust the scipy install requirements

* making lint happy

* Update challengeutils/__main__.py

Co-authored-by: Verena Chung <9377970+vpchung@users.noreply.github.com>

* lint changes

* making the linter happy

* making the linter happyy and adding in additional exception handling

* lint changes

* making the linter happy

* linting

* linting

* linting

* Exception handling

---------

Co-authored-by: Timothy Bergquist <tbergquist@Timothys-MacBook-Air.local>
Co-authored-by: Timothy Bergquist <tbergquist@Timothys-Air.localdomain>
Co-authored-by: Verena Chung <9377970+vpchung@users.noreply.github.com>

* add pre-commit

* run pre-commit on all files

* update readme

* update readme

* Release v4.2 (#244)

* update version

* add changelog for this release

---------

Co-authored-by: Thomas Yu <thomas.yu@sagebase.org>
Co-authored-by: lgtm-com[bot] <43144390+lgtm-com[bot]@users.noreply.github.com>
Co-authored-by: LGTM Migrator <lgtm-migrator@users.noreply.github.com>
Co-authored-by: Timothy Bergquist <timothy.bergquist@sagebase.org>
Co-authored-by: Timothy Bergquist <tbergquist@Timothys-MacBook-Air.local>
Co-authored-by: Timothy Bergquist <tbergquist@Timothys-Air.localdomain>
Co-authored-by: mdsage1 <122999770+mdsage1@users.noreply.github.com>
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.

3 participants