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

Measure rwc per object #256

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

emiglietta
Copy link
Collaborator

No description provided.

Now, I have to create the for loop function.
I deleted of commented all parts having to do with other measurements (including per image measurements) and also removed if statements, assuming here wants_obects is always True and perform_RWC is always True
in order to escape import errors in CP (name of file must be all lower case) and debugger cache (Nodar had to change the actual name bc the previous capitalized one remained cached)
if len(self.objects_list.value) == 0:
raise ValidationError("No object sets selected", self.objects_list)

def upgrade_settings(self, setting_values, variable_revision_number, module_name):
Copy link
Member

Choose a reason for hiding this comment

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

You don't need this for this plugin (yet)

M_PER_OBJECT = "Within each object individually"

"""Feature name format for the RWC Coefficient measurement"""
F_RWC_FORMAT = "Correlation_RWC_%s_%s"
Copy link
Member

Choose a reason for hiding this comment

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

This may cause a collision if the person has also measured RWC regularly, can we name it something different?

return True


def get_scale(scale_1, scale_2):
Copy link
Member

Choose a reason for hiding this comment

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

Is this needed?

#################################

import centrosome.cpmorphology
import centrosome.zernike
Copy link
Member

Choose a reason for hiding this comment

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

Make sure to remove the ones you don't. need

@@ -0,0 +1,28 @@
{
Copy link
Member

Choose a reason for hiding this comment

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

Can you remove this file from the commit, please?

Copy link
Member

@bethac07 bethac07 left a comment

Choose a reason for hiding this comment

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

Awesome job! Just some cleanup to do, plus add to the documentation

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.

2 participants