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

Add hot_encoding parameter to nplet_measures #14

Open
Laouen opened this issue Oct 20, 2024 · 0 comments
Open

Add hot_encoding parameter to nplet_measures #14

Laouen opened this issue Oct 20, 2024 · 0 comments
Labels

Comments

@Laouen
Copy link
Owner

Laouen commented Oct 20, 2024

nplet_measures works with nplets of the same order, if the nplets have not the same order, then it converts the nplets to hot encoded nplets and return de result of nplets_measures_hot_encoded.

This return a small accuracy error because the hot encoded measures has some extra steps that introduce a small numerical error. Then, we want to give the option by default to not use the hot encoded version and if the user prefers the hot encoded version, then set the parameter hot_encoding in True and that would perform the current behaviour of converting the nplets into hot encoded nplets and use the nplets_measures_hot_encoded function to obtain the result.

In de default hot_encoding not in True, the nplets must first be grouped by orders, then batch processed by order and then the result is reordered in the original order (this was implemented in the simulated_annealing_multi_order and can be retrieved from there 3c455224d05c3c1137175e724c7ceaf932ab0f64)

@Laouen Laouen added the feature label Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant