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

Create a Candidate class, possibly a Cluster class also #23

Open
sk8forether opened this issue Jul 24, 2018 · 3 comments
Open

Create a Candidate class, possibly a Cluster class also #23

sk8forether opened this issue Jul 24, 2018 · 3 comments
Assignees
Labels
low priority Nice to have or possible improvement

Comments

@sk8forether
Copy link
Collaborator

No description provided.

@sk8forether sk8forether added the medium priority Necessary to project, but not urgent label Jul 24, 2018
@sk8forether sk8forether added this to the Complete New Pipeline milestone Jul 24, 2018
@platodeliyannis
Copy link
Contributor

After thinking about it, this isn't as practical as I thought it would be. Might come back to the idea later

@sk8forether
Copy link
Collaborator Author

Did you run into specific issues that make it impractical?

@platodeliyannis
Copy link
Contributor

Comparing the candidates output of Spitler pipeline's single pulse search with my FOF, the only common statistics were start/end in frequency and time of the candidates. It doesn't seem immediately necessary to create a hierarchical class structure. When I get to thinking about candidate scoring methods, this might be a good tool, but I'd rather flesh out more of the pipeline first

@sk8forether sk8forether added low priority Nice to have or possible improvement and removed medium priority Necessary to project, but not urgent labels Aug 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low priority Nice to have or possible improvement
Projects
None yet
Development

No branches or pull requests

2 participants