Mscoco transformations to BigML-COCO and documentation changes #261
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jaor
approved these changes
Jun 12, 2024
README.rst
Outdated
| c:\Python27\python.exe distribute_setup.py | ||
|
|
||
| After that, you'll be able to install ``pip`` by typing the following command | ||
| To install BigMLer on Windows environments, you'll Python installed. |
README.rst
Outdated
|
|
||
| After that, you'll be able to install ``pip`` by typing the following command | ||
| To install BigMLer on Windows environments, you'll Python installed. | ||
| The code has been tested with Python 3.10 and you can create a conda |
There was a problem hiding this comment.
nit: maybe make "conda" italics if it's usually spelled lowercase? or maybe not, if everybody knows wthat that is..
README.rst
Outdated
| To install BigMLer on Windows environments, you'll Python installed. | ||
| The code has been tested with Python 3.10 and you can create a conda | ||
| environment with that Python version or download it from `Python for Windows | ||
| <http://www.python.org/download/>`_ and install it. In the last case, you'll |
README.rst
Outdated
| .. code-block:: bash | ||
|
|
||
| c:\Python27\Scripts\easy_install.exe pip | ||
| To install ``pip``, first you need to open your command line window |
There was a problem hiding this comment.
a command line terminal window (?)
| @@ -180,6 +164,11 @@ issuing | |||
|
|
|||
| should show BigMLer version information. | |||
|
|
|||
| Extensions of BigMLer to use images are not supported in Windows by default. | |||
| The libraries needed for those models are not available usually for that | |||
bigmler/options/source.py
Outdated
| " the corresponding annotations, unless some " | ||
| " folder attribute is provided in each" | ||
| " annotation. In that case it should point to" | ||
| " the folder parent directory and --anotations-dir" |
bigmler/options/source.py
Outdated
| " the corresponding annotations, unless some " | ||
| " folder attribute is provided in each" | ||
| " annotation. In that case it should point to" | ||
| " the folder parent directory and --anotations-dir" |
| filenames = voc_to_cocojson(args.annotations_dir, args, session_file) | ||
| elif args.annotations_language == "YOLO": | ||
| filenames = yolo_to_cocojson(args.annotations_dir, args, session_file) | ||
| elif args.annotations_language == "COCO": |
There was a problem hiding this comment.
i am neutral about it, but have you considered naming it MSCOCO to be explicit about its not being pure COCO? or is it the case that the only COCO is MSCOCO?
There was a problem hiding this comment.
I though so too, but COCO is shorter and from what I saw is most common.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Includes the #258 changes and adds checks and tests.