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

check_*_domain should fail when the same domain is given as a source and a target #16

Open
kachayev opened this issue Nov 21, 2023 · 4 comments
Labels

Comments

@kachayev
Copy link
Collaborator

This check is currently not implemented.

tgnassou added a commit that referenced this issue Nov 22, 2023
@BuenoRuben
Copy link
Contributor

hey, sorry but I'm not sure I understood what the function is exactly supposed to do:
is it supposed to check if two domains have the same name (1 and -1 as an exemple),
or is it supposed to check if two domains, one from target and another from source, have the same elements

@kachayev
Copy link
Collaborator Author

@BuenoRuben The idea here is that a specific domain should be included either as source or as a target. Which means that the domain id is either consistently positive or consistently negative. Let's say the id of the domain is 3. It should either be 3 (source) or -3 (target). There shouldn't be a situation where we have both 3 and -3 in the sample_domain.

@kachayev
Copy link
Collaborator Author

The one place where this happens (and is necessary) is Leave-One-Domain-Out splitter for model selection.

@BuenoRuben
Copy link
Contributor

Thanks you for your answer, I will take a look at those functions

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

2 participants