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

[ENH] SOM: output columns with coordinates and errors #6542

Merged
merged 3 commits into from
Sep 1, 2023

Conversation

janezd
Copy link
Contributor

@janezd janezd commented Aug 20, 2023

Issue

Resolves #6540.

Description of changes

Added columns, with lazy values and everything.

To discuss:

  • som_cell column type.
  • split into fitter and model or leave as is? (I'd keep it until (if ever) we really need separate classes)
Includes
  • Code changes
  • Tests
  • Documentation

@janezd janezd force-pushed the som-output-columns branch 3 times, most recently from 6a82c47 to 8f4140f Compare August 22, 2023 21:14
@codecov
Copy link

codecov bot commented Aug 22, 2023

Codecov Report

Merging #6542 (f0fb80b) into master (0906348) will increase coverage by 0.00%.
Report is 29 commits behind head on master.
The diff coverage is 96.07%.

❗ Current head f0fb80b differs from pull request most recent head a340d0a. Consider uploading reports for the commit a340d0a to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #6542    +/-   ##
========================================
  Coverage   87.69%   87.70%            
========================================
  Files         321      321            
  Lines       69423    69534   +111     
========================================
+ Hits        60879    60982   +103     
- Misses       8544     8552     +8     

@janezd janezd changed the title Som output columns SOM: output columns with coordinates and errors Aug 24, 2023
@janezd janezd added the needs discussion Core developers need to discuss the issue label Aug 24, 2023
@janezd janezd force-pushed the som-output-columns branch 2 times, most recently from e2e1a51 to df17cdb Compare August 24, 2023 10:27
Copy link
Member

@markotoplak markotoplak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

__eq__ and __hash__

Orange/widgets/unsupervised/owsom.py Outdated Show resolved Hide resolved
Orange/widgets/unsupervised/owsom.py Show resolved Hide resolved
@janezd janezd force-pushed the som-output-columns branch 2 times, most recently from 2111175 to f0fb80b Compare August 24, 2023 12:50
@janezd janezd marked this pull request as ready for review August 24, 2023 18:01
@janezd janezd force-pushed the som-output-columns branch 2 times, most recently from 9e885ae to ac0609e Compare August 25, 2023 12:28
@markotoplak markotoplak removed the needs discussion Core developers need to discuss the issue label Aug 25, 2023
@janezd janezd added this to the 3.36.0 milestone Sep 1, 2023
janezd and others added 2 commits September 1, 2023 15:02
Added result invaliation into the widget so that tests can get the
outputs.
@markotoplak markotoplak changed the title SOM: output columns with coordinates and errors [ENH] SOM: output columns with coordinates and errors Sep 1, 2023
@markotoplak markotoplak merged commit 3778aee into biolab:master Sep 1, 2023
20 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add output coordinates to SOM
2 participants