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.

Sign up for GitHub

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

Jump to bottom

Square grid in Self-Organizing Map not working #6424

Closed
PrimozGodec opened this issue Apr 24, 2023 · 2 comments · Fixed by #6438
Closed

Square grid in Self-Organizing Map not working #6424

PrimozGodec opened this issue Apr 24, 2023 · 2 comments · Fixed by #6438
Assignees
@PrimozGodec
Labels
snack This will take an hour or two

Comments

@PrimozGodec
Copy link
Contributor

PrimozGodec commented Apr 24, 2023 •
edited
Loading

What's wrong?

When one changes the type of grid, a Square grid, nothing happens (the grid is still hexagonal).

Screenshot 2023-04-24 at 15 39 42

How can we reproduce the problem?

  1. File widget->SOM
  2. Change the grid to Square.
@PrimozGodec PrimozGodec added the bug report Bug is reported by user, not yet confirmed by the core team label Apr 24, 2023
@PrimozGodec PrimozGodec changed the title Self-Organizing Map Square grid in Self-Organizing Map not working Apr 24, 2023
@PrimozGodec PrimozGodec added bug A bug confirmed by the core team snack This will take an hour or two and removed bug report Bug is reported by user, not yet confirmed by the core team labels Apr 24, 2023
@ales-erjavec
Copy link
Contributor

Press Start to rerun the optimization. Grid topology is parameter of the algorithm not just a visualization option.

@PrimozGodec PrimozGodec removed the bug A bug confirmed by the core team label Apr 25, 2023
@PrimozGodec
Copy link
Contributor Author

Thanks, @ales-erjavec. I was not careful enough. Then I suggest implementing the info that tells the user to press start, such as it is done in t-SNE. I can volunteer to implement that.

@PrimozGodec PrimozGodec self-assigned this Apr 25, 2023
@PrimozGodec PrimozGodec mentioned this issue Apr 27, 2023
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PrimozGodec PrimozGodec

Labels
snack This will take an hour or two
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ENH] SOM - Warn user to restart optimization after parameter change PrimozGodec/orange3
2 participants
@ales-erjavec @PrimozGodec