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

Add parallax correction via new ParallaxCorrectionModifier #1904

Merged
merged 129 commits into from
Jul 28, 2022

Commits on May 5, 2021

  1. Parallax correction

    Jussi Leinonen committed May 5, 2021
    Configuration menu
    Copy the full SHA
    8f1257f View commit details
    Browse the repository at this point in the history

Commits on May 6, 2021

  1. Added projection

    Jussi Leinonen committed May 6, 2021
    Configuration menu
    Copy the full SHA
    58a960a View commit details
    Browse the repository at this point in the history
  2. Added projection

    Jussi Leinonen committed May 6, 2021
    Configuration menu
    Copy the full SHA
    342030f View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2021

  1. better error message

    uhamann committed Aug 23, 2021
    Configuration menu
    Copy the full SHA
    a49855c View commit details
    Browse the repository at this point in the history

Commits on Aug 24, 2021

  1. parallax example script

    uhamann committed Aug 24, 2021
    Configuration menu
    Copy the full SHA
    2a796de View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    77f5e50 View commit details
    Browse the repository at this point in the history

Commits on Nov 29, 2021

  1. Started unit test for parallax correction

    Started working on unit tests for the parallax correction.  Not much
    content yet.  Everything is likely to change completely.
    
    Added a dummy implementation, failing, just so that the unit test gets
    beyond the initial import.
    gerritholl committed Nov 29, 2021
    Configuration menu
    Copy the full SHA
    daf4400 View commit details
    Browse the repository at this point in the history
  2. Merge remote-tracking branch 'remotes/tmp/parallax' into parallax-cor…

    …rection
    
    Merge history of monti pytroll parallax branch into satpy to preserve
    history of parallax.py
    gerritholl committed Nov 29, 2021
    Configuration menu
    Copy the full SHA
    eebe910 View commit details
    Browse the repository at this point in the history
  3. Merge parallax.py into satpy

    Merge first version of parallax.py into satpy, adding (some) docstrings
    and fixing pep8 issues.  From here we can try to make a composite out of
    it and match it up with the unit test already added.
    gerritholl committed Nov 29, 2021
    Configuration menu
    Copy the full SHA
    db6ec84 View commit details
    Browse the repository at this point in the history

Commits on Nov 30, 2021

  1. Configuration menu
    Copy the full SHA
    ae72e98 View commit details
    Browse the repository at this point in the history
  2. Adapt parallax dependency from meteoswiss

    Adapt a dependency from the parallax module adopted from meteoswiss.  I
    expect adding this module is temporary.  I don't really know what it's
    for.
    gerritholl committed Nov 30, 2021
    Configuration menu
    Copy the full SHA
    a9d5668 View commit details
    Browse the repository at this point in the history
  3. Complement AUTHORS.md

    To AUTHORS.md, add Jussi Leinonen due to his parallax contributions.
    Also add affiliation for DWD authors (request from DWD).
    gerritholl committed Nov 30, 2021
    Configuration menu
    Copy the full SHA
    c3918bf View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    1b38fb8 View commit details
    Browse the repository at this point in the history
  5. Extend unit tests for parallax correction.

    Add unit tests for the low-level parallax_correct function, according to
    an interface I believe it should have, but that it doesn't have.  Most
    of the unit tests are failing.
    
    The higher level class is not unit tested yet because I don't understand
    what it's supposed to do exactly.  Probably much of it needs to be
    deleted.
    gerritholl committed Nov 30, 2021
    Configuration menu
    Copy the full SHA
    c321b18 View commit details
    Browse the repository at this point in the history
  6. More unit tests for parallax correction

    Add more unit tests for the parallax correction.  Started with unit
    tests for the class.  They're failing, but I don't know why.
    gerritholl committed Nov 30, 2021
    Configuration menu
    Copy the full SHA
    5dae1e6 View commit details
    Browse the repository at this point in the history

Commits on Dec 1, 2021

  1. Remove some unused code

    Remove some unused code.  Will remove more later.
    gerritholl committed Dec 1, 2021
    Configuration menu
    Copy the full SHA
    70292ae View commit details
    Browse the repository at this point in the history

Commits on Dec 2, 2021

  1. Extend parallax unit tests

    Add another parallax correction unit test that uses a much tighter area.
    gerritholl committed Dec 2, 2021
    Configuration menu
    Copy the full SHA
    2cf131a View commit details
    Browse the repository at this point in the history
  2. Cleaner parallax correction implementation

    Jussi Leinonen committed Dec 2, 2021
    Configuration menu
    Copy the full SHA
    a9aebd0 View commit details
    Browse the repository at this point in the history

Commits on Dec 3, 2021

  1. Merge Jussis simplification back in parallax.py

    Merge the simplification and improvement from the previous commit by
    Jussi back into parallax.py, and delete the unused and unneedded
    projection.py as well as an unused convolve import.
    gerritholl committed Dec 3, 2021
    Configuration menu
    Copy the full SHA
    913adaf View commit details
    Browse the repository at this point in the history
  2. Improve parameterised unit tests

    Parameterise some of the fixtures used in the parallax unit tests.
    Still needs some more testing but we're making progress.
    gerritholl committed Dec 3, 2021
    Configuration menu
    Copy the full SHA
    5ffc618 View commit details
    Browse the repository at this point in the history
  3. Add more unit tests for parallax correction

    Add more unit tests for the parallax correction, or rather more
    parameters to use in the unit tests.  The parallax correction fails at
    the antimeridian.
    gerritholl committed Dec 3, 2021
    Configuration menu
    Copy the full SHA
    1077d7a View commit details
    Browse the repository at this point in the history
  4. Extend unit tests

    Started a partly cloudy unit test.  Needs more tests for actual content.
    gerritholl committed Dec 3, 2021
    Configuration menu
    Copy the full SHA
    eed47ec View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    b0a2c68 View commit details
    Browse the repository at this point in the history

Commits on Dec 6, 2021

  1. Switch to nearest neighbour

    Switch to nearest neighbour resampling and add some tolerance to
    floating point comparisons.  Now only 3 rather than 8 tests fail.
    gerritholl committed Dec 6, 2021
    Configuration menu
    Copy the full SHA
    0850114 View commit details
    Browse the repository at this point in the history
  2. Add more tests for parallax correction

    In order to try to understand where the serious loss of precision
    occurs, add more tests.  Haven't found it yet.
    gerritholl committed Dec 6, 2021
    Configuration menu
    Copy the full SHA
    7960f1b View commit details
    Browse the repository at this point in the history

Commits on Dec 7, 2021

  1. Restore slant cloudy test

    gerritholl committed Dec 7, 2021
    Configuration menu
    Copy the full SHA
    e017866 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    714c110 View commit details
    Browse the repository at this point in the history

Commits on Dec 14, 2021

  1. Configuration menu
    Copy the full SHA
    b30f8bc View commit details
    Browse the repository at this point in the history
  2. Fix swapped lon/lat in unit test

    Make unit test pass by fixing a swapping of lon/lat.
    gerritholl committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    0c45d39 View commit details
    Browse the repository at this point in the history
  3. Add parameterised fixture for generating a cloud.

    Add a parameterised fixture for generating a cloud with certain
    parameters.  Will be used for to-be-written unit tests.
    gerritholl committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    06a5716 View commit details
    Browse the repository at this point in the history

Commits on Jan 3, 2022

  1. Configuration menu
    Copy the full SHA
    b54a431 View commit details
    Browse the repository at this point in the history
  2. Add more detailed tests

    Add more detailed tests to understand why (corrdist < uncorrdist).all()
    succeeds for me but fails on GitHub CI::
    
        (Pdb) p corr_dist
        array([[7847.61343724, 6029.28528333, 4715.82906281, 4386.94953032,
                5231.74248224],
               [6029.28528332, 3923.80691175, 2357.91453878, 2615.87129834,
                4386.94953031],
               [4715.8290628 , 2357.91453878,    0.        , 2357.91453878,
                4715.8290628 ],
               [4386.94953031, 2615.87129834, 2357.91453878, 3923.80691175,
                6029.28528332],
               [5231.74248224, 4386.94953032, 4715.82906281, 6029.28528333,
                7847.61343724]])
        (Pdb) p uncorr_dist
        array([[7862.66446501, 6040.84891332, 4724.87359889, 4395.36330457,
                5241.77649481],
               [6040.84891332, 3931.33241962, 2362.43680545, 2620.88830285,
                4395.36330457],
               [4724.87359889, 2362.43680545,    0.        , 2362.43680545,
                4724.87359889],
               [4395.36330457, 2620.88830285, 2362.43680545, 3931.33241962,
                6040.84891332],
               [5241.77649481, 4395.36330457, 4724.87359889, 6040.84891332,
                7862.66446501]])
    gerritholl committed Jan 3, 2022
    Configuration menu
    Copy the full SHA
    e713e89 View commit details
    Browse the repository at this point in the history
  3. Add tolerance to distance comparison

    For the uncorrected / corrected distance comparison, add a tolerance.
    Although tests pass for me, they fail on GitHub CI.  Maybe adding a
    tolerance will make them pass.
    gerritholl committed Jan 3, 2022
    Configuration menu
    Copy the full SHA
    9c1404b View commit details
    Browse the repository at this point in the history

Commits on Jan 6, 2022

  1. Add tests for no or partly overlapping CTH

    Add tests that CTH that fails to overlap with the base area raise an
    error, and if it overlays only partly raises a warning.  Implementation
    of this functionality is awaiting fixes in pyresample.
    gerritholl committed Jan 6, 2022
    Configuration menu
    Copy the full SHA
    6518d19 View commit details
    Browse the repository at this point in the history
  2. Allow costumisation of resampling method in parallax correction

    Allow user to customise the method used for the parallax correction.
    Test for nearest and bilinear.  Mark bilinear tests as failing since
    they are.
    gerritholl committed Jan 6, 2022
    Configuration menu
    Copy the full SHA
    c2de3da View commit details
    Browse the repository at this point in the history

Commits on Jan 7, 2022

  1. Add tests with dask and same areas

    Add parallax correction tests where the areas are the same.  Also enable
    dask for one version of the partly cloudy tests.
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    4ed3aa9 View commit details
    Browse the repository at this point in the history
  2. Point out that CompositeBase is also for modifiers

    In the docstring of CompositeBase, point out that all modifier classes
    /also/ derive from CompositeBase.
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    7854191 View commit details
    Browse the repository at this point in the history
  3. Started work on ParallaxCorrectionModifier class

    Started work on the ParallaxCorrectionModifier class integrating the new
    parallax correction into the modifier interface.
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    90d57e1 View commit details
    Browse the repository at this point in the history
  4. Add modifier interface for parallax correction

    Add a modifier interface for the parallax correction, as well as a unit
    test testing this implementation.
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    f149eba View commit details
    Browse the repository at this point in the history
  5. Mark partly/no overlap as xfail

    Mark the partly/no overlap tests as xfail, awaiting fixes in pyresample.
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    ee97200 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    f7d692f View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    07e7731 View commit details
    Browse the repository at this point in the history
  8. Adapt to pre-3.9 syntax

    Changed dictionary merging syntax to support Python 3.8
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    d987fc1 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    25da335 View commit details
    Browse the repository at this point in the history
  10. Add documentation on parallax correction modifier

    Add documentation on the parallax correction modifier.  NB: this commit
    relies that pytroll#1969 is already merged.
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    33df7c1 View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    cc01a86 View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    d6a9cc0 View commit details
    Browse the repository at this point in the history
  13. Remove unused function and TODO

    Remove unused empty function from geometry.py.
    Replace a TODO by a catch for the unlikely scenario that the satellite
    elevation is exactly zero, and add a unit test for this case.
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    7df874b View commit details
    Browse the repository at this point in the history
  14. Repair forward parallax after elevation change

    Repair the forward parallax function after introducing a bug with the
    elevation change.  Rename X to x.
    gerritholl committed Jan 7, 2022
    Configuration menu
    Copy the full SHA
    6e94b77 View commit details
    Browse the repository at this point in the history

Commits on Jan 10, 2022

  1. More specific parameter failure marking

    In the parallax correction unit tests, mark only particular combinations
    of parameters as failing.  Until now the pytest.mark.xfail was marked
    too widely.
    gerritholl committed Jan 10, 2022
    Configuration menu
    Copy the full SHA
    a9022e0 View commit details
    Browse the repository at this point in the history
  2. Mark one more case as xfail

    On different architecture another variant of bilinear fails, such as on
    GitHub.  Mark as fail because bilinear fails anyway.
    gerritholl committed Jan 10, 2022
    Configuration menu
    Copy the full SHA
    bf56335 View commit details
    Browse the repository at this point in the history

Commits on Jan 11, 2022

  1. Write log message when doing parallax correction

    Write a log message when performing the parallax correction.
    Add a unit test to test that this log message is written.
    gerritholl committed Jan 11, 2022
    Configuration menu
    Copy the full SHA
    b5340da View commit details
    Browse the repository at this point in the history

Commits on Jan 12, 2022

  1. Add backup calculation of orbital parameters

    Add a backup calculation of orbital parameters if those are missing from
    the metadata.
    
    Introduces soft dependencies on skyfield and astropy.
    gerritholl committed Jan 12, 2022
    Configuration menu
    Copy the full SHA
    688e2c7 View commit details
    Browse the repository at this point in the history
  2. Adapt to Python 3.7 syntax

    Downgrade syntax to Python 3.7 as this is still used for GitHub CI.
    gerritholl committed Jan 12, 2022
    Configuration menu
    Copy the full SHA
    17479d9 View commit details
    Browse the repository at this point in the history
  3. Add missing dependencies for continuous integration

    The latest functionality needs skyfield and astropy.  Install those via
    continuous_integration/environmentm.yaml.
    gerritholl committed Jan 12, 2022
    Configuration menu
    Copy the full SHA
    0a8e515 View commit details
    Browse the repository at this point in the history
  4. Try unstable test with latest astropy

    The previous commit added an astropy dependency.  The experimental test
    fails as the latest released astropy doesn't work with the latest main
    numpy.  Try latest main astropy to see if this resolves the problem.
    gerritholl committed Jan 12, 2022
    Configuration menu
    Copy the full SHA
    75f2e2d View commit details
    Browse the repository at this point in the history
  5. Improve example and documentation for parallax composite

    Improve the documentation including the example on how to create a
    composite that applies a parallax correction modifier.  Also point out
    that composites can themselves not be modified, so that RGBs with
    parallax correction applied will have to apply the correction to each
    channel seperately.
    gerritholl committed Jan 12, 2022
    Configuration menu
    Copy the full SHA
    0ea5bbb View commit details
    Browse the repository at this point in the history

Commits on Jan 17, 2022

  1. Configuration menu
    Copy the full SHA
    fc3c94e View commit details
    Browse the repository at this point in the history

Commits on Jan 20, 2022

  1. Add failing test for NWCSAF confetti bug

    Add a (failing) test corresponding to the confetti bug obtained when
    using CTH from NWCSAF-GEO data.
    gerritholl committed Jan 20, 2022
    Configuration menu
    Copy the full SHA
    e43c499 View commit details
    Browse the repository at this point in the history

Commits on Jan 26, 2022

  1. Add debug mode to parallax correction

    Add an optional debug mode to parallax correction.  Among other things,
    this will retain the lat/lons so that users can measure the distance
    objects have been moved due to parallax correction.
    gerritholl committed Jan 26, 2022
    Configuration menu
    Copy the full SHA
    6236340 View commit details
    Browse the repository at this point in the history

Commits on Jan 27, 2022

  1. Consistent use metres above Earth's surface

    In parallax correction, consistent use metres above the Earth's surface.
    No km, and no distances above the centre of the Earth.
    gerritholl committed Jan 27, 2022
    Configuration menu
    Copy the full SHA
    c75681f View commit details
    Browse the repository at this point in the history
  2. Remember that satellite altitude reported in km

    Remember that satellite altitude is actually reported in km.  Test it
    accordingly.
    gerritholl committed Jan 27, 2022
    Configuration menu
    Copy the full SHA
    eeb52a4 View commit details
    Browse the repository at this point in the history
  3. Remove destructive preprocess method

    The preprocess method was making things kaputt.
    gerritholl committed Jan 27, 2022
    Configuration menu
    Copy the full SHA
    5c4089c View commit details
    Browse the repository at this point in the history

Commits on Jan 28, 2022

  1. Verify cloud moves toward observer

    Add a unit test that a cloud moves toward the satellite after applying
    parallax correction using the modifier interface.
    
    This test is currently failing, it's not moving at all ☹
    gerritholl committed Jan 28, 2022
    Configuration menu
    Copy the full SHA
    e20ad8f View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    c96dd78 View commit details
    Browse the repository at this point in the history
  3. Improve test confirm cloud moves south

    Improve the test that confirms the cloud moves south.  Now captures two
    failure modes: one where it actually moves north, and one where it moves
    so far out of the field of view that the original cloud isn't touched at
    all.  Add some more diagnostics that I may remove later.
    gerritholl committed Jan 28, 2022
    Configuration menu
    Copy the full SHA
    c4ae9d1 View commit details
    Browse the repository at this point in the history
  4. Fix cloud direction movement test

    Fix the test for the cloud direction movement.  The origin of satpy
    areas is in the northwest, so higher y actually means lower latitude.
    To be sure, and for easier interpretation, compare the actual latitudes.
    gerritholl committed Jan 28, 2022
    Configuration menu
    Copy the full SHA
    65e2533 View commit details
    Browse the repository at this point in the history
  5. Perform test with different radii of influence

    The test with the cloud movement in the modifier interface fails either
    way, but it fails differently if the radius of influence is 10 km rather
    than 25 km.  But for nearest neighbour it shouldn't matter?
    gerritholl committed Jan 28, 2022
    Configuration menu
    Copy the full SHA
    8047bc5 View commit details
    Browse the repository at this point in the history

Commits on Feb 2, 2022

  1. Clarify comments

    gerritholl committed Feb 2, 2022
    Configuration menu
    Copy the full SHA
    ff40e6a View commit details
    Browse the repository at this point in the history

Commits on Feb 3, 2022

  1. Configuration menu
    Copy the full SHA
    6e4e3d1 View commit details
    Browse the repository at this point in the history
  2. Try with bucket resampler

    Try to use bucket resampler in parallax correction.  Still a draft that
    can't possibly work in the general case.
    gerritholl committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    a73b31a View commit details
    Browse the repository at this point in the history

Commits on Feb 4, 2022

  1. Change parallax resampling approach

    Change the parallax resampling approach.  Rather than assigning an area,
    use resample_nearest with source and destination area directly.  By
    subtracting rather than adding inv_lon_diff and inv_lat_diff, I can
    resample to the resulting SwathDefinition directly.
    
    This is the first version where the unit test confirming that parallax
    correction moves the cloud equatorward and that the old cloud now has
    fill values, passes.  However, some problems do remain.  Some cloud
    pixels disappear entirely, and when the cloud moves out of the
    destination area, the unit test still fails.  A later commit will fix
    these problems and add corresponding unit tests.  My fixes have also
    caused failure in other (weaker) unit tests.
    gerritholl committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    22ae499 View commit details
    Browse the repository at this point in the history
  2. Add test confirming new cloud area fully cloudy

    Add a test confirming that the new destination cloud area is fully
    cloudy.
    This test is currently failing.
    gerritholl committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    bd8b5a9 View commit details
    Browse the repository at this point in the history
  3. Fix get_abs_max implementation

    Fix bug in get_abs_max that was getting the wrong longitudes.  Uses the
    new method in the bucket resampler implemented at
    pytroll/pyresample#418
    gerritholl committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    a67a206 View commit details
    Browse the repository at this point in the history
  4. Resample CTH to target area first

    Resample CTH to target area first and clean up resampling in parallax
    correction.  This improves the functionality when CTH is on a different
    resolution.  It does mean yet more resampling steps.
    gerritholl committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    cffb1d2 View commit details
    Browse the repository at this point in the history
  5. Ensure test case has valid polar area

    Test case was having an invalid area when centred at the pole.  Correct
    the EPSG code so that it's valid.
    gerritholl committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    e539415 View commit details
    Browse the repository at this point in the history
  6. Also use 3575 at antimeridian

    Also use EPSG 3575 at the antimeridian.  Not ideal but will do for now.
    gerritholl committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    568b551 View commit details
    Browse the repository at this point in the history

Commits on Feb 7, 2022

  1. Fix tests for 15 km cloud

    Fix tests for 15 km high cloud, where the could gets deformed.  Also
    point out in documentation and comments that this does happen.
    gerritholl committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    2f18c60 View commit details
    Browse the repository at this point in the history
  2. Add test with dask arrays

    Add test to check that parallax correction works with dask arrays.
    
    This test is currently failing.
    gerritholl committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    7b67560 View commit details
    Browse the repository at this point in the history
  3. make parallax correction more dask-friendly

    Make parallax correction more dask-friendly by using resample_dataset
    rather than resample_nearest.
    gerritholl committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    6766f90 View commit details
    Browse the repository at this point in the history
  4. Re-arrange and extend parallax unit tests

    Rearrange and extend unit tests for the parallax correction.  The unit
    tests pass, but a real life test produces huge holes, which I haven't
    reproduced in any unit test yet.
    gerritholl committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    17d7468 View commit details
    Browse the repository at this point in the history

Commits on Feb 8, 2022

  1. Rename variable

    gerritholl committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    fd3a2ac View commit details
    Browse the repository at this point in the history
  2. In debug mode, store counts as dataset not ndarray

    In the debug mode, store the counts as a dataset that can be resampled,
    not as a basic ndarray without metadata.
    gerritholl committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    4e4ce01 View commit details
    Browse the repository at this point in the history
  3. Fix radius of influence in final resampling step

    The radiuf of influence in the final resampling step was too low.
    Increase it!
    gerritholl committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    bea71a3 View commit details
    Browse the repository at this point in the history

Commits on Mar 18, 2022

  1. Merge branch 'main' into parallax-correction

    Merge branch main while fixing one merge conflict
    gerritholl committed Mar 18, 2022
    Configuration menu
    Copy the full SHA
    e9e8b20 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a3765c6 View commit details
    Browse the repository at this point in the history

Commits on Mar 22, 2022

  1. Refactor parallax correction tests

    Refactor the tests in test_parallax.py into three classes, and remove
    some dead code.  Hopefully this will satisfy the worries by CodeScene.
    gerritholl committed Mar 22, 2022
    Configuration menu
    Copy the full SHA
    8a23932 View commit details
    Browse the repository at this point in the history

Commits on Mar 23, 2022

  1. Configuration menu
    Copy the full SHA
    c452e7d View commit details
    Browse the repository at this point in the history

Commits on Apr 6, 2022

  1. Configuration menu
    Copy the full SHA
    1297774 View commit details
    Browse the repository at this point in the history

Commits on Apr 7, 2022

  1. Configuration menu
    Copy the full SHA
    e4fd0c9 View commit details
    Browse the repository at this point in the history

Commits on Apr 11, 2022

  1. Configuration menu
    Copy the full SHA
    fe4f938 View commit details
    Browse the repository at this point in the history
  2. Make sure to call apply_modifier_info

    It is important to call apply_modifier_info in the modifier, including
    the parallax correction modifier.  Otherwise Satpy may get confused
    between the original and the corrected dataset.
    gerritholl committed Apr 11, 2022
    Configuration menu
    Copy the full SHA
    173ed44 View commit details
    Browse the repository at this point in the history

Commits on Apr 12, 2022

  1. Add test for double-load bug

    Added a unit test that reproduces the bug fixed in the previous commit,
    in which the original and the parallax corrected dataset would share the
    same dask array and thus be both parallax corrected.
    gerritholl committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    ddfebcb View commit details
    Browse the repository at this point in the history
  2. Add test confirming no compute occurs.

    Add test confirming that ``sc.load(...)`` with the parallax correction
    does not trigger any computation.
    
    This test currently fails awaiting fixes in pyresample, and is therefore
    marked with xfail.
    gerritholl committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    d4edc4c View commit details
    Browse the repository at this point in the history
  3. Fix example and test YAML config

    Fix the examlpe and test YAML config, by not passing a standard_name and
    using the SingleBandCompositor, so that the standard_name is inherited
    correctly and the same enhancement is configured as for the single
    channel data.
    gerritholl committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    933dac6 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    f60b0ad View commit details
    Browse the repository at this point in the history
  5. refactor unit test test_modifier_interface_cloud_moves_to_observer

    refactor unit test test_modifier_interface_cloud_moves_to_observer.
    hopefully this satisfies the CodeScene PR check.
    gerritholl committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    cb15600 View commit details
    Browse the repository at this point in the history
  6. refactor test_correct_area_clearsky

    Refactor test_correct_area_clearsky to have less arguments.
    gerritholl committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    11fee22 View commit details
    Browse the repository at this point in the history

Commits on Apr 14, 2022

  1. Configuration menu
    Copy the full SHA
    d585dbd View commit details
    Browse the repository at this point in the history
  2. Change versionadded to 0.37

    For the parallax correction, update "version added" to 0.37.
    gerritholl committed Apr 14, 2022
    Configuration menu
    Copy the full SHA
    58f55b6 View commit details
    Browse the repository at this point in the history

Commits on May 4, 2022

  1. Configuration menu
    Copy the full SHA
    c3e393d View commit details
    Browse the repository at this point in the history

Commits on May 5, 2022

  1. Refactoring and docstring improvements

    Various refactoring and docstring improvements in the parallax
    correction module.  Thanks @sfinkens for useful review comments.
    gerritholl committed May 5, 2022
    Configuration menu
    Copy the full SHA
    5c3a4f0 View commit details
    Browse the repository at this point in the history
  2. Add more prominent "experimental" warning

    For the parallax correction, add a more prominent warning that this
    correction is currently experimental and subject to change.
    gerritholl committed May 5, 2022
    Configuration menu
    Copy the full SHA
    5323363 View commit details
    Browse the repository at this point in the history

Commits on May 6, 2022

  1. Fix filename for pc docstring example

    Fix the filename referred to in the parallax correction docstring
    example.  Thanks @adybbroe for spotting this.
    gerritholl committed May 6, 2022
    Configuration menu
    Copy the full SHA
    259a0a2 View commit details
    Browse the repository at this point in the history
  2. Point out pc only supported for GEO

    Point out in the API documentation that parallax correction is currently
    only supported for CTH coming from geostationary satellites.
    gerritholl committed May 6, 2022
    Configuration menu
    Copy the full SHA
    ed38cda View commit details
    Browse the repository at this point in the history
  3. Move get_satpos_from_name to satpy.utils

    Move the functionality to get a satellite position when we only have the
    name from the parallax module to the utils module.  Add a corresponding
    unit test there.
    gerritholl committed May 6, 2022
    Configuration menu
    Copy the full SHA
    26564f5 View commit details
    Browse the repository at this point in the history
  4. Needs pyresample 1.24

    Due to the updates in the parallax correction relying on the latest
    bucket resampler, satpy now needs pyresample 1.24 (which has not been
    released yet).
    gerritholl committed May 6, 2022
    Configuration menu
    Copy the full SHA
    48690ca View commit details
    Browse the repository at this point in the history

Commits on Jun 17, 2022

  1. Change variable name for bucket resampler instance

    Change the variable name for the bucket resampler instance from br to
    bur.  One of the online linters dislikes the two-letter names?
    gerritholl committed Jun 17, 2022
    Configuration menu
    Copy the full SHA
    bb2040a View commit details
    Browse the repository at this point in the history

Commits on Jul 11, 2022

  1. Configuration menu
    Copy the full SHA
    afd5951 View commit details
    Browse the repository at this point in the history
  2. Mock TLE downloading in parallax correction unit tests

    One unit tests in the parallax correction was actually downloading a
    TLE, which was failing in GitHub CI.  Mock this consistently so no
    socket access is performed.
    gerritholl committed Jul 11, 2022
    Configuration menu
    Copy the full SHA
    0b3ea1c View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    1230822 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    02bbda3 View commit details
    Browse the repository at this point in the history
  5. Minor change to satisfy codebeat

    Minor change in the GPM Imerg reader to fix a codebeat complaint about
    block depth.
    gerritholl committed Jul 11, 2022
    Configuration menu
    Copy the full SHA
    e78a9f8 View commit details
    Browse the repository at this point in the history

Commits on Jul 25, 2022

  1. Configuration menu
    Copy the full SHA
    2aac451 View commit details
    Browse the repository at this point in the history
  2. Describe version, not only year

    In the parallax correction documentation, describe the version when it
    was added in the text.
    gerritholl committed Jul 25, 2022
    Configuration menu
    Copy the full SHA
    1d953a8 View commit details
    Browse the repository at this point in the history
  3. Move units note from comment to docstring

    Put the note on units in the docstring rather than in a comment.
    gerritholl committed Jul 25, 2022
    Configuration menu
    Copy the full SHA
    41e6ec5 View commit details
    Browse the repository at this point in the history
  4. Write "degrees" rather than "°"

    In docstrings, write out the word "degrees" rather than using the symbol
    "°".
    gerritholl committed Jul 25, 2022
    Configuration menu
    Copy the full SHA
    7d8ff7b View commit details
    Browse the repository at this point in the history
  5. typo: or → on

    gerritholl committed Jul 25, 2022
    Configuration menu
    Copy the full SHA
    d874994 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    5e37359 View commit details
    Browse the repository at this point in the history
  7. Add some more flexibility in parallax correction

    Add some more flexibility in the ParallaxCorrection class.
    gerritholl committed Jul 25, 2022
    Configuration menu
    Copy the full SHA
    9509274 View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2022

  1. Improve configurability in parallax correction

    Allow user to set the radii of influence in the two nearest neighbour
    resampling steps in the parallax correction.
    gerritholl committed Jul 26, 2022
    Configuration menu
    Copy the full SHA
    4e97767 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    567d27a View commit details
    Browse the repository at this point in the history

Commits on Jul 27, 2022

  1. Fix duplicate doc and rename parameter

    Remove a duplicate line from the modifiers documentation.  Change the
    maybe_attempt_tle parameter in ``utils.get_satpos`` to ``use_tle``.
    gerritholl committed Jul 27, 2022
    Configuration menu
    Copy the full SHA
    2c3a5d7 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    5ca49c1 View commit details
    Browse the repository at this point in the history
  3. Adapt unit test in test_utils

    In test_utils, adapt unit test for the changed parameter name in
    get_satpos.
    gerritholl committed Jul 27, 2022
    Configuration menu
    Copy the full SHA
    66f02f1 View commit details
    Browse the repository at this point in the history
  4. Simplify docstring for forward_parallax

    Simplify the documentation for the forward_parallax function based on a
    suggestion by @ghiggi.
    gerritholl committed Jul 27, 2022
    Configuration menu
    Copy the full SHA
    a8fce07 View commit details
    Browse the repository at this point in the history

Commits on Jul 28, 2022

  1. Utility function for parallax displacement + docs

    Add utility function for parallax displacement, as suggested by @ghiggi.
    Rename the function `forward_parallax` to be called
    `get_parallax_corrected_lonlats`.
    Various smaller edits/changes in documentation and variable names.
    gerritholl committed Jul 28, 2022
    4 Configuration menu
    Copy the full SHA
    5130cdb View commit details
    Browse the repository at this point in the history
  2. Small docstring fixes

    gerritholl committed Jul 28, 2022
    Configuration menu
    Copy the full SHA
    dc352e3 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    9b5e980 View commit details
    Browse the repository at this point in the history