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

Added SetFit example test #46

Merged
merged 13 commits into from
Jun 7, 2023
Merged

Added SetFit example test #46

merged 13 commits into from
Jun 7, 2023

Conversation

franz101
Copy link
Contributor

@franz101 franz101 commented Jun 6, 2023

Refactoring:

  • move populator script to scripts
  • add script for replacing text
  • automatically replace the dataset with a test dataset
  • in a notebook: add # skip ci: true to cell to skip it during testing

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@franz101 franz101 marked this pull request as ready for review June 7, 2023 16:35
cell.source = cell.source.replace(f'{key}', f'{value}')
# If the cell has the following comment: (# skip ci: true)
# replace it with an empty cell instead
if "# skip ci: true" in cell.source:
Copy link
Contributor

Choose a reason for hiding this comment

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

😱

@franz101 franz101 merged commit 1c0f1f6 into main Jun 7, 2023
@franz101 franz101 deleted the chore/cleanup branch June 7, 2023 19:20
franz101 added a commit that referenced this pull request Jun 22, 2023
* Semseg example (#44)

Semseg example notebook with test

* Added SetFit example test (#46)

* refactor directory

* updated workflow

* fix glob

* formatting and small fix

* fix test path

* remove set console

* fix version

* retry with version bump

* added inference uuid

* update version

* remove space

* added skip ci comment

* small: cleanup special character

---------

Co-authored-by: Anthony Corletti <anthony@rungalileo.io>

* Bucket url bug (#47)

* fix: install ultralytics modal (#48)

* Update workflows to include workflow_dispatch

* modal populator_gpu: docker install libgl1

* Changed params (#49)

* Debug/od example (#50)

* debug: od example

* updates

* updates

* docstr updates

* Chore/bug (#51)

* Changed params

* Remove comment

* Rename OD cloud run (#52)

* fix run name

* fix CI

* Remove output (#53)

---------

Co-authored-by: dcaustin33 <dcaustin33@gmail.com>
Co-authored-by: Anthony Corletti <anthony@rungalileo.io>
Co-authored-by: Bogdan Gheorghe <112427971+bogdan-galileo@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