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

Experimental #6

Draft
wants to merge 223 commits into
base: main
Choose a base branch
from
Draft

Experimental #6

wants to merge 223 commits into from

Conversation

jedman
Copy link

@jedman jedman commented Dec 4, 2023

Summary

PR Checklist

  • If this is a work in progress PR, set as a Draft PR
  • Linted my code according to the style guides.
  • Added tests to verify changes to the code.
  • Added necessary documentation to any new functions/classes following the
    expect style.
  • Marked as ready for review (if this is was a draft PR), and converted
    to a Pull Request
  • Tagged @simpeg/simpeg-developers when ready for review.

Reference issue

What does this implement/fix?

Additional information

jcapriot and others added 30 commits November 2, 2022 08:27
Also adds more documentation with examples
replace with n_cells
thibaut-kobold and others added 30 commits January 3, 2024 10:34
* ta/weighting_strategy:
  fix new depreciation
  Ditch deprecated functions in utils.model_builder (simpeg#1311)
  more warnings
  move warning
  try fix `| None` test error
  stacklevel
  fix tests
  address comments
  One more randn
  switch tutorials rand to randn
* ta/weighting_strategy:
  fix test
  replace numpy distance calculation by `scipy.spatial.distance.cdist`, with possibility to pass optional args.
…e_experimental2

* commit '5231446ee858fe677cdcbdcfe65be081b61ebd20':
  Triaxial magnetic gradient forward modelling (simpeg#1288)

# Conflicts:
#	SimPEG/potential_fields/magnetics/simulation.py
#	tests/pf/test_forward_Mag_Linear.py
Run tests with coterminal angles in each quadrant and for right angles
(multiples of np.pi / 2).
This change assumes that the coterminal function returns the coterminal
angles in the [-pi, pi) interval.
* main:
  Fix typo in error messages (simpeg#1324)
  Update test_regularization.py to test vector derivatives. Update AmplitudeSmoothnessFirstOrder in vector.py so that the call matches the derivative.
  Move `__init__` in `BaseSimulation` to the top of the class (simpeg#1323)
  Bump versions of flake8 and black and pin flake plugins (simpeg#1330)
  Update test_regularization.py to test vector derivatives. Update AmplitudeSmoothnessFirstOrder in vector.py so that the call matches the derivative.
  Update test_regularization.py to test vector derivatives. Update AmplitudeSmoothnessFirstOrder in vector.py so that the call matches the derivative.
  Add `weights_keys` method to `BaseRegularization` (simpeg#1320)
  Fix description of source_field in gravity survey (simpeg#1322)
  Documentation improvements for classes in Objective Function Pieces (simpeg#1292)

# Conflicts:
#	tests/base/regularizations/test_regularization.py
…ngles

* commit '95c80fa8357c6b4245c8ae55bc5f33a32126bdf8':
  Revert changes to coterminal, to be addressed on ISSUE-1333
  Apply sign to second SmoothnessSecondOrder
  Switch to gradient.sign()
  Change coterminal expression again
  CHange to coterminal
  Remove negative sign
  Extend test to the other orientations

# Conflicts:
#	tests/base/regularizations/test_regularization.py
* ISSUE-1333:
  Remove failing test on sign
  Change bounds on pi in docs
  Go for joe's modulus
  Fix angle in tests: use -pi instead of pi
  Add more tests for the coterminal function
  Add unittest
  Re-apply modulo for coterminal comps
…ngles

* commit '04ed16e113d9a19f38d72c419bf9ca136bcb9f7b':

# Conflicts:
#	tests/base/regularizations/test_regularization.py
Co-authored-by: Jacob Edman <jacob.edman@koboldmetals.com>
…e-experimental-20240129

* commit '6edc293dacbd1b873a7fcea15ae49cbdb3364836':
  Apply joe's suggestion to other components.
  Update SimPEG/regularization/regularization_mesh.py
  Simplify check for invalid multipliers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

7 participants