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

GEOPY-607 #575

Merged
merged 70 commits into from
Jun 20, 2023
Merged

GEOPY-607 #575

merged 70 commits into from
Jun 20, 2023

Conversation

jamiefb
Copy link
Contributor

@jamiefb jamiefb commented Jun 14, 2023 •
edited by github-actions bot
Loading

**GEOPY-607 - Use Base inversion for Mag/Grav application **

@jamiefb
initial commit for potential fields dash layout
9274db7
@jamiefb
add all visibility callbacks
61706fb
@jamiefb
clean up layout
6d244ba
@jamiefb
start moving functions over from notebook app
56370c5
@jamiefb
update component list, displayed inversion params
d6c6744
@jamiefb
Merge branch 'GEOPY-592' into GEOPY-607
34040d7
@jamiefb
first pass updating input data components
65e874e
@jamiefb
plotting with matplotlib image
5faf338
@jamiefb
begin converting matplotlib plot to plotly
7038f05
@jamiefb
plotting working
09158eb
@jamiefb
initialize inversion param starting/reference model values
95c72b5
@jamiefb
remove topography relative to sensor option
d97d113
@jamiefb
select to display inversion parameters
fbfbf7f
@jamiefb
replace matplotlib with plotly graph
9d06fb2
@jamiefb
add plot labels, ticks, aspect ratio checkbox
ab7ed89
@jamiefb
split gravity inversion app
60d78a1
@jamiefb
add button to link to mesh creation
4bd350d
@jamiefb
Merge branch 'GEOPY-592' into GEOPY-607
0ec4e62 
# Conflicts:
#	geoapps/base/dash_application.py
@jamiefb
resolution on plot
0affebb
@jamiefb
add plotting rotation
3fc77bc
@jamiefb
add azimuth back
d94bfcf
@jamiefb
some changes to plotting, starting writing out params
194e5c2
@jamiefb
working on writing params
3760c0c
@jamiefb
downsample with downsample_grid, downsample_xy
1d46011
@jamiefb
saving params ui.json
cdf3bc1
@jamiefb
types for input boxes
5abcb1f
@jamiefb
clear plots after object change, and empty channel
6fd545a
@jamiefb
initialize plot window
e13b587
@jamiefb
move input data and topography in layout
76a3aa9
@jamiefb
generalize write_trigger
2b86ae7
@jamiefb
fix pylint issues
e49ad7e
@jamiefb
fix issues with copying data/objects to new workspace
149d807
@jamiefb
plot initialization issue
eebe4d2
@jamiefb
reading in forward only
e1e5591
@jamiefb
fix model names for using magnetics
a27fd04
@jamiefb
fix plot initialization; saving window params
ccf7bbc
@jamiefb
Merge branch 'develop' into GEOPY-607
af48b31 
# Conflicts:
#	geoapps/base/dash_application.py
#	geoapps/inversion/potential_fields/gravity/constants.py
#	geoapps/inversion/potential_fields/magnetic_scalar/constants.py
#	geoapps/inversion/potential_fields/magnetic_vector/constants.py
@jamiefb
dash inversion apps initializing correctly
ddb5c45
@jamiefb
writing to ui.json
e148d93
@jamiefb
make mesh object for bounds, models
9dfbc90
@jamiefb
topography only data; rename variables
a46ec71
@jamiefb
make run test for dash inversion; remove original notebook
fd3db60
@jamiefb
Merge branch 'develop' into GEOPY-607
cd446d9 
# Conflicts:
#	geoapps/base/dash_application.py
#	geoapps/inversion/potential_fields/application.py
#	geoapps/scatter_plot/application.py
@jamiefb
fix merging issues
8ca81c7
@jamiefb
update copyright; rebuild lock files
26e349e
@jamiefb
update variable names
577a471
@jamiefb
fix some failing tests
b7e3a3c
@jamiefb
Merge branch 'develop' into GEOPY-607
2ee1576 
# Conflicts:
#	conda-py-3.10-lock.yml
#	conda-py-3.9-lock.yml
@jamiefb
more renaming
24a9b4d
@jamiefb
clear notebooks; more renaming
a9e663d
@jamiefb
Merge branch 'develop' into GEOPY-607
7cecdf1
domfournier
domfournier reviewed Jun 15, 2023
View reviewed changes
Copy link
Contributor

@domfournier domfournier left a comment

Choose a reason for hiding this comment

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

See comments below. Also, any chance we can re-scale the colors based on the color stretch in GA? The colormap (if any) is stored under the data.entity_type.color_map as an array of [value, r, g, b]

image

geoapps/inversion/potential_fields/magnetic_scalar/application.py Show resolved Hide resolved
geoapps/inversion/potential_fields/magnetic_vector/constants.py Outdated Show resolved Hide resolved
geoapps/inversion/potential_fields/magnetic_vector/constants.py Outdated Show resolved Hide resolved
@domfournier domfournier self-requested a review June 20, 2023 18:59
@domfournier domfournier merged commit 280c0de into develop Jun 20, 2023
@domfournier domfournier deleted the GEOPY-607 branch June 20, 2023 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@domfournier domfournier domfournier approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@jamiefb @domfournier