-
Notifications
You must be signed in to change notification settings - Fork 66
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
Pass data as an option #218
Conversation
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.
clang-tidy made some suggestions
f36d73b
to
c8ef7b3
Compare
dc2923d
to
5721d0a
Compare
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.
Теперь Algorithm::LoadData
не принимает параметров, но все еще должен быть обязательно вызван перед Execute
, что не всегда нужно. Cтоит подумать о том, чтобы разрешить вызывать Execute
без LoadData
перед этим.
This brings the tuple and the enum closer together to avoid confusion, like in Desbordante#209
1c0b878
to
6571177
Compare
`pybind11::object` manages reference counting, which is sometimes unneeded. Also changes some `std::string` to `std::string_view`.
Allows to denote when multiple types can be accepted.
The files are structured this way because an algorithm may use several tables (HyMD) or even a list of tables (Spider). Those algorithms still need the `InputTable` type, but they will register different options, which may become common later and will thus be put in this directory as well.
Also includes some minor test rewrites. Makes it easier to implement algorithms that take something other than a single table.
24ba80c
to
b054296
Compare
Uses a more robust way of getting the number of columns. Avoids the reliance on `input_table_` being set to the table being processed now, indirectly inherited from the old implementation. Previously, there was a possibility that `input_table_` would not be set to the table being processed in `PliBasedFdAlgorithm` because of its `LoadData` overload, but the only scenario which used it was using `TypoMiner`, and that class would always set the option correctly.
This makes it easier to implement algorithms that accept anything other than a table.