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

feat(back): Merge Pixano apps in backend and docs #27

Merged
merged 9 commits into from
Jan 10, 2024

Conversation

cpvannier
Copy link
Member

@cpvannier cpvannier commented Jan 9, 2024

Description

Following the merge of the Pixano Annotator and Explorer apps in the frontend (#23), we need to merge these apps in the backend and in the documentation as well.

Warning: breaking change for local run

To run the backend locally you should now call pixano.app instead of pixano.apps:

DATA_DIR=your_datasets_directory/ uvicorn pixano.app:create_app --factory --reload

Changes

Changed

  • Refactor Pixano Explorer and Annotator apps into a single Pixano app in the backend
  • Update notebooks and documentation accordingly

Fixed

  • Fix GitHub CI/CD actions running twice on pull requests
    • Actions have been disabled on branches other than develop and main
    • To run the actions on your feature branch as soon as you start working on it, create a draft pull request

Change to run CI/CD actions (format, lint, test) only on develop, main, and PRs to develop.

To run those actions on a feature branch that is not yet ready for an actual PR, creating a draft PR works.
@cpvannier cpvannier self-assigned this Jan 9, 2024
@cpvannier cpvannier added scope: docs Related to Pixano documentation enhancement New feature or request scope: back Related to Pixano backend labels Jan 9, 2024
@cpvannier cpvannier changed the title chore: Merge apps in backend and docs chore(back): Merge apps in backend and docs Jan 9, 2024
@cpvannier cpvannier changed the title chore(back): Merge apps in backend and docs feat(back): Merge apps in backend and docs Jan 9, 2024
@cpvannier cpvannier changed the title feat(back): Merge apps in backend and docs feat(back): Merge Pixano apps in backend and docs Jan 9, 2024
Reformat Image get_uri method into complete_uri property
Use complete_uri property in COCOExporter
Reformat URI in ItemView
@cpvannier cpvannier marked this pull request as ready for review January 9, 2024 16:16
docs/user/app.md Outdated Show resolved Hide resolved
@cpvannier cpvannier merged commit 51460fe into develop Jan 10, 2024
5 checks passed
@cpvannier cpvannier added this to the Pixano 0.5.0 milestone Jan 16, 2024
@cpvannier cpvannier deleted the chore/backend_cleanup branch January 31, 2024 10:54
@cpvannier cpvannier restored the chore/backend_cleanup branch January 31, 2024 10:54
@cpvannier cpvannier deleted the chore/backend_cleanup branch January 31, 2024 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request scope: back Related to Pixano backend scope: docs Related to Pixano documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants