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

Change values from 0.001 to 0.1 / use config from btrack-examples #374

Merged
merged 6 commits into from
Aug 7, 2023

Conversation

paddyroddy
Copy link
Collaborator

Fixes #372

Copy link
Owner

@quantumjot quantumjot left a comment

Choose a reason for hiding this comment

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

👌

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Apparently this should never have been here

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@p-j-smith 👋

@deprecated-napari-hub-preview-bot
Copy link

deprecated-napari-hub-preview-bot bot commented Aug 7, 2023

Preview page for your plugin is ready here:
https://preview.napari-hub.org/quantumjot/btrack/374
Updated: 2023-08-07T13:42:31.707330

@p-j-smith
Copy link
Collaborator

I think the changes will need to be made in the example configs (https://github.com/lowe-lab-ucl/btrack-examples) rather than here, as the defaults are essentially overridden and set using the example cell_config.json

Copy link
Collaborator

@p-j-smith p-j-smith left a comment

Choose a reason for hiding this comment

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

nice! I think this will clear up some confusion

@@ -55,8 +55,7 @@ instance
lib
lib64
local_settings.py
models/MDCK_*
models/test_config.json
models
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We don't need this folder at all now

filename = CONFIG_FILE
assert isinstance(filename, Path)
assert isinstance(filename, str)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This test seems a bit stupid. But I'll leave it here for now

@paddyroddy paddyroddy changed the title Change values from 0.001 to 0.1 Change values from 0.001 to 0.1 / use config from btrack-examples Aug 7, 2023
@paddyroddy
Copy link
Collaborator Author

paddyroddy commented Aug 7, 2023

PR changed purpose a bit. Now also fixes #375.

@paddyroddy paddyroddy merged commit 009622b into main Aug 7, 2023
5 checks passed
@paddyroddy paddyroddy deleted the fix-issue-372 branch August 7, 2023 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST] Set default value of p(not track)
3 participants