Skip to content

Learning updated #398

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

Open
wants to merge 7 commits into
base: scalability
Choose a base branch
from
Open

Learning updated #398

wants to merge 7 commits into from

Conversation

Blain354
Copy link
Contributor

added group_models :
from single models, you can now create a list and analyze on that list instead of individually in the results pane
image

@MahdiAll99
Copy link
Contributor

Tested on windows 10, works perfectly. Some minor comments:

  • The tooltip indicating possible links at the end of the node is not correct (see screenshot below)
  • Maybe remove the options for this node since there are none (see screenshot below).
  • Node orders: Place group models node in the list of nodes according to its logic order in the pipeline (see screenshot below).

GroupModels
image
image

@Blain354
Copy link
Contributor Author

Blain354 commented Feb 5, 2025

About your first point, it is normal to output a "model" type IO because its basically a list of models so its the same kind of output as compare_models for example.

@MahdiAll99
Copy link
Contributor

I added regression settings for group models node. Everything runs perfectly no errors but i noticed one issue:

  • (Classification & Regression) In the results section, in train model results, only one model's information is shown (see picture below)
    image
  • The group models node does not work for regression, and only one plot is shown:
    image

@Blain354
Copy link
Contributor Author

Blain354 commented Apr 7, 2025

  • Now, all pipelines are showed but for nodes after group_models, it only shows results for the one that computed. When checking compare mode, you can see all combined results in only one pipeline.
  • Also, when a node has been run and is selected, the orange message saying "node has not been run yet" has been removed

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.

2 participants