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

predicted labels should read "Bird" and not "Tree" by default when use_bird_release() #490

Merged
merged 4 commits into from
Oct 3, 2023

Conversation

bw4sz
Copy link
Collaborator

@bw4sz bw4sz commented Oct 2, 2023

I added a test and updated main.use_bird_release() to make sure that the bird label is automatically assigned instead of "Tree" default.

@bw4sz bw4sz linked an issue Oct 2, 2023 that may be closed by this pull request
@bw4sz
Copy link
Collaborator Author

bw4sz commented Oct 2, 2023

@henrykironde I cleaned up the change in the docs. This should be ready to go.

@henrykironde
Copy link
Contributor

Looks good to me, Can you run yapf -i --recursive deepforest/ --style=.style.yapf and commit amend to clean up this code

deepforest/main.py	(reformatted)
117 
         self.config["score_thresh"] = 0.3
 
         # Set label dictionary to Bird
-        self.label_dict = {"Bird":0}
+        self.label_dict = {"Bird": 0}

@bw4sz
Copy link
Collaborator Author

bw4sz commented Oct 2, 2023

I added another commit since it gets squashed when merged into main.

@bw4sz bw4sz merged commit 5482b13 into main Oct 3, 2023
1 of 3 checks passed
@bw4sz bw4sz deleted the issue_488 branch October 7, 2023 04:17
janjatovic pushed a commit to Treeconomy/DeepForest_new that referenced this pull request Mar 26, 2024
…e_bird_release() (weecology#490)

* updated tests pass

* matching index.rst with main

* style updates

* style changes
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.

Bird detector should update label_dict automatically.
2 participants