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

[Refactor] VEDA Core Features - Time-Series Analysis Tool (E&A) Discovery #902

Closed
sandrahoang686 opened this issue Mar 21, 2024 · 3 comments
Closed

[Refactor] VEDA Core Features - Time-Series Analysis Tool (E&A) Discovery #902

sandrahoang686 opened this issue Mar 21, 2024 · 3 comments
Assignees
@sandrahoang686
Labels
veda v2 VEDA Refactor Epic Work

Comments

@sandrahoang686
Copy link
Collaborator

sandrahoang686 commented Mar 21, 2024 •
edited
Loading

Expose E&A component and document all tech-debt we run into into separate issues to break this out. Try to get it working with NextJs instance if possible.

Link tech-debt issues here
@sandrahoang686 sandrahoang686 added tech debt veda v2 VEDA Refactor Epic Work labels Mar 21, 2024
@sandrahoang686 sandrahoang686 mentioned this issue Mar 21, 2024
Closed
4 tasks
@aboydnw
Copy link
Contributor

aboydnw commented Aug 28, 2024

@sandrahoang686 I wonder if this should be a component/feature. My main doubt is that people will use this independent of the rest of the E&A page. Maybe we should just have E&A as a "composite feature" based on the different categories in #1076

But maybe I'm splitting hairs and, even if we tell stakeholders the E&A page is a composite feature, this ticket still makes sense defined as is. What do you think?

cc @hanbyul-here @dzole0311 @faustoperez

@sandrahoang686
Copy link
Collaborator Author

This is the composite E&A component as we discussed during checkin 👍🏼

@aboydnw aboydnw changed the title [VEDA2 Refactor] - Exploration Page Panel/Widget Component Core Feature Breakout [Refactor] Composite Features - Time-Series Analysis Tool (E&A) Aug 29, 2024
This was referenced Aug 29, 2024
Closed
Open
@sandrahoang686 sandrahoang686 self-assigned this Sep 3, 2024
@aboydnw aboydnw changed the title [Refactor] Composite Features - Time-Series Analysis Tool (E&A) [Refactor] VEDA Feature - Time-Series Analysis Tool (E&A) Sep 12, 2024
@aboydnw aboydnw changed the title [Refactor] VEDA Feature - Time-Series Analysis Tool (E&A) [Refactor] VEDA Feature - Time-Series Analysis Tool (E&A) Discovery Sep 12, 2024
@aboydnw aboydnw changed the title [Refactor] VEDA Feature - Time-Series Analysis Tool (E&A) Discovery [Refactor] VEDA Core Features - Time-Series Analysis Tool (E&A) Discovery Sep 12, 2024
@sandrahoang686 sandrahoang686 mentioned this issue Sep 17, 2024
Merged
@sandrahoang686
Copy link
Collaborator Author

sandrahoang686 commented Sep 17, 2024 •
edited
Loading

Summary: Feature Branches created and tech-debt identified which will require more time to work on to bring E&A to next instance
PRs/Feature Branches:
veda-ui: #1154
next-veda-ui: developmentseed/next-veda-ui#4

Following tech-debt tickets created from discovery:

@sandrahoang686 sandrahoang686 mentioned this issue Sep 17, 2024
Closed
@sandrahoang686 sandrahoang686 mentioned this issue Sep 17, 2024
Closed
@hanbyul-here hanbyul-here mentioned this issue Oct 30, 2024
Merged
sandrahoang686 added a commit that referenced this issue Nov 14, 2024
@sandrahoang686
[FEATURE BRANCH] 902 E&A breakout (#1154)
a8311c2 
**Related Ticket:** #902

### Description of Changes
**Related NextJs PR/Branch:**
developmentseed/next-veda-ui#4

**GHG Preview** with this branch pulled in generated here =>
US-GHG-Center/veda-config-ghg#659

**Iteration 1** - Issue
#902, changes by
@sandrahoang686
* Exposed `ExplorationAndAnalysis` component
* Exposed `timelineDatasetsAtom` to be used NextJs side
* Exposed & Moved/decoupled `DatasetSelectorModal` out of
`ExplorationAndAnalysis`. ExplorationAndAnalysis only needs to know the
dataset layers to be displayed. DatasetSelectorModal or whatever layer
selection method should be controlled at the parent level above
`ExplorationAndAnalysis`.
* Commented out certain controls that were breaking
`ExplorationAndAnalysis` in NextJs instance
   * `AnalysisMessageControl`
   * `CustomAoIControl`

### Notes & Questions About Changes
_{Add additonal notes and outstanding questions here related to changes
in this pull request}_

### Validation / Testing
* Validate that Datalayer Selection Modal works as expected
* Validate that E&A features work as expected - aois, analyzation,
adding/removing cards
* Validate that all card linking works as expected
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sandrahoang686 sandrahoang686

Labels
veda v2 VEDA Refactor Epic Work
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sandrahoang686 @aboydnw