-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implement GridComparator #113
base: main
Are you sure you want to change the base?
Conversation
|
||
self.comparison_array[row, col, channel] = pair_result | ||
|
||
return self.comparison_array |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there any kind of way we can give this a meaningful order?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or, phrased differently, what is the use case that you have in mind?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or, phrased differently, what is the use case that you have in mind?
It is essentially a multi-channel adjacency matrix. A set of adjacency matrices that store information on different types of relationships.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there any kind of way we can give this a meaningful order?
I am working on that. Presently, I have implemented a mapping between the factors of interest and some numerical indices. If I had to improve on this, I'd probably need something like xarray
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are your ideas for approaches for giving this a consistent shape? At least for most conventional ML approaches, you will need your inputs to always have the same shape (or you only limit yourself to producing sequences)
This PR should implement a GridComparator. This would allow the generation of similarity grids for sets of molecules.