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

feat: Explorers #237

Open
wants to merge 75 commits into
base: develop
Choose a base branch
from
Open

feat: Explorers #237

wants to merge 75 commits into from

Conversation

NiciXitri
Copy link
Collaborator

Summary

  • Added Exploration Module to frontend code
  • Added Explorations backend code (jobs and components)
  • Added documentation for changing explorers accepted types and values (for future/current changes to DashAI types)
  • New backend Dependencies:
    • wordcloud
    • pathvalidate

Type of change

  • Front end new feature.
  • Back end new feature.
  • Bug fix.
  • Documentation.

Changes

  • Project merge
    • Feature: Exploration Module

How to Test

The code was tested manually and could be replicated by using the frontend UI.

@NiciXitri
added config for Explorer results
0a798b5
@NiciXitri
added explorer model and endpoints
3e2c340
@NiciXitri
added explorer Job
bfd070f
@NiciXitri
added validations and retrieval of results
7f2ef23
@NiciXitri
Merge remote-tracking branch 'origin/develop' into project/explorer
185e442
@NiciXitri
validation changes to explorers
14c2a37
@NiciXitri
added explorer creation, launch and selection to frontend
d04dcbb
@NiciXitri
fix typing error
ee1ea1c
@NiciXitri
fix typing errors
f8fdc8d
@NiciXitri
added visualization for explorations results
13161c9
@NiciXitri
modifications to metadata and results getter
c4fe4c7
@NiciXitri
added explorations > explorers on backend
8d25e82
@NiciXitri
added explorations > explorers on frontend
414c000
@NiciXitri
(explorers) removed deprecated files
8569469
@NiciXitri
fix pre-commit errors
9911785
@NiciXitri
added shared components
4b7bf63
@NiciXitri
added prepare dataset to explorer
74fd36c
@NiciXitri
modified schema and parameter preparation
bea941b
@NiciXitri
added scatter plot explorer
f3951f6
@NiciXitri
take split in explorer
1d7ebb3
@NiciXitri
modified actions to use TooltipedCellItem
d1f8113
@NiciXitri
explorer module context modification
47807ca
@NiciXitri
small fix to get_sample
03aace9
@NiciXitri
added visualizers
d5e7a1c
@NiciXitri
optimization, documentation and details
e615cea
@NiciXitri
feat: exploration runner & results visulization
795285c
@NiciXitri
precommit prettier fix
b7e9d43
@NiciXitri
fix: Fixed typing issues
d219a9d
@NiciXitri
added wordcloud dependency
1f2849c
@NiciXitri
front: added image visualizer
04db4b8
@NiciXitri
disabled filter and sorting model
23d7a8e
@NiciXitri
added dataset columns refresh to context
079436c
@NiciXitri
added snackbar messages and modified buttons
f0487e0
@NiciXitri
added tooltip to values on index column
f7b9beb
@NiciXitri
centered and added icons to detail tabs
0271b17
@NiciXitri
modified default viewMode and order
76c2d1c
@NiciXitri
added optional closeOnBackdropClick property
08fcc37
@NiciXitri
skip column validation on patch methods
0ed7695 
This lets the user save an explorer successfully after transforming the dataset column spec
@NiciXitri
Merge branch 'develop' into project/explorer
58726ad
@NiciXitri
tooltip does not follow cursor
3e54783
@NiciXitri
sort explorer options
3d16455
@NiciXitri
added correlation and covariance matrix explorers
7874f6b
@NiciXitri
small optimization
d43d178
@NiciXitri
display exploration type on result
13b835d
@NiciXitri
small fixes and used plotly io module instead of pickle
b4bf240
@NiciXitri
added more explorers
9690a10
@IsaiasVenegas IsaiasVenegas mentioned this pull request Jan 10, 2025
Open
@NiciXitri
Merge branch 'develop' into project/explorer
5b7d805
@NiciXitri
added typeguard to explorerJob
7d58d51
@NiciXitri
Merge remote-tracking branch 'origin/develop' into project/explorer
53a58ea
@NiciXitri
Merge remote-tracking branch 'origin/develop' into project/explorer
4f14bc5
@NiciXitri
Merge remote-tracking branch 'origin/feat/predict-models-datasets' in…
18b2d37 
…to project/explorer
@NiciXitri
fix: updated explorers to use DashAIDataset
ffe12a2 
This are changes following the new datasets-splits logic, centered around splitting only on experiments
@NiciXitri
fix: pre-commit issues
19e0992 
Just some imports accidentaly left-over
@NiciXitri
Merge remote-tracking branch 'origin/develop' into project/explorer
83b80ed 
Also removed pathvalidate dependency (not available on 3.8), and with it setting exploration/explorer names as folder/filenames
@NiciXitri
compiled requirements with wordcloud
5f9d59e
@NiciXitri
fix: fix endpoint format for jobs on frontend
1674ec0
@NiciXitri
fix: pip-compile adding pywin32 to requirements.txt
d2072cc 
sorry, I compiled from windows 😔
@NiciXitri
Merge remote-tracking branch 'origin/develop' into project/explorer
6134c8b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cristian-tamblay cristian-tamblay Awaiting requested review from cristian-tamblay

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@NiciXitri