Repository navigation
[ImgBot] Optimize images - #1
Closed
imgbot[bot] wants to merge 358 commits into
Closed
imgbot[bot] wants to merge 358 commits into
imgbot[bot] wants to merge 358 commits into
Conversation
It covers the currently-public API. There is still discussion over whether TAB_HEADER and/or _add_tab_header should be public.
Most of the coverage comes from the main test suite, but this covers the case where the lextab/parsetab does not yet exist and has to be generated.
Make classproperty and lazyproperty thread-safe
Fix string formatting for minimum_python_version
PLY has some restrictions about only defining one parser per module (no doubt related to whatever deep magic it's using), so generate a unique module name each time the test is run.
TST: Set CI in Azure Pipelines
Make Unit and Angle string parsing thread-safe
Make leap-second initialisation thread-safe
It currently fails, because I haven't fixed the issue yet.
- Remove `modules` property that lazily creates the underlying `_modules` attribute, in favour of an `__init__` that creates `modules` as an attribute. - Add a lock that is held for all mutations of the object. Fixes astropy#11220.
It should be possible to test, but added the pragma to avoid codecov failure for code that I've merely re-indented. Co-authored-by: P. L. Lim <[email protected]>
This makes it a bit easier to test UfuncHelper without affecting the main singleton instance.
Make UfuncHelpers thread-safe
What's New entry for threadsafety improvements
… z2>z1 Updating documentation to reflect discussion in astropy issue astropy#11197 Added warning if z2<z1 in angular_diameter_distance_z1z2. Split out test
Minor stylistic change.
[ci skip] Co-authored-by: Derek Homeier <[email protected]>
Allow documentation tests in FITS verification page
Check input to cosmo.angular_diameter_distance_z1z2, that z2>z1
Revert "Merge pull request astropy#11143 from maxnoe/unix_time_ptp"
Remove recommendation to use intersphinx (ref astropy#10286).
[ci skip]
Co-authored-by: Stuart Mumford <[email protected]>
Switch to Towncrier
- Temporary fix to matplotlib/matplotlib#19839, - from astropy#11118, - closes (?) astropy#11464 Co-authored-by: @pllim Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
…_error Avoid deprecation warning in wcsaxes doc
[ci skip]
TST: Preload UT1 in test_precision.py
- E.g. allow for column “dec” and “pm_dec” - clarify error message for multiple matches in Table Co-authored-by: Erik Tollerud <[email protected]> Co-authored-by: @mhvk Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Expand SkyCoord.guess_from_table
Previously, the distance was ignored, which meant the units of the
proper motions would be wrong:
c = CartesianRepresentation([1, 0, 0]*u.kpc)
cd = CartesianDifferential([-10, 0, 4.74]*u.km/u.s)
cd.represent_as(UnitSphericalDifferential, base=c)
<UnitSphericalDifferential (d_lon, d_lat) in km rad / s
(0., 4.74)>
Previously, the following gave the wrong unit for the proper motion:
c = SkyCoord(x=1*u.kpc, y=0*u.kpc, z=0*u.kpc,
v_x=10*u.km/u.s, v_y=0*u.km/u.s, v_z=4.74*u.km/u.s,
representation_type='cartesian')
c.representation_type = 'unitspherical'
repr(c)
<SkyCoord (ICRS): (ra, dec) in deg
(0., 0.)
(pm_ra, pm_dec) in km rad / s
(0., 4.74)>
…ential-scale Fix conversion of cartesian to unit-spherical coordinates with velocities
Fix indexing of fits.Header with Numpy integers
*Total -- 399.47kb -> 270.19kb (32.36%) /docs/development/workflow/forking_button.png -- 9.27kb -> 4.26kb (54.04%) /docs/development/workflow/milestone.png -- 22.53kb -> 11.74kb (47.88%) /docs/development/workflow/pull_button.png -- 15.16kb -> 8.17kb (46.12%) /docs/table/table_architecture.png -- 28.10kb -> 15.49kb (44.88%) /docs/table/table_row.png -- 40.39kb -> 23.38kb (42.13%) /docs/table/table_repr_html.png -- 41.49kb -> 26.37kb (36.43%) /docs/table/table_show_in_nb.png -- 64.85kb -> 41.32kb (36.29%) /docs/development/workflow/worked_example_switch_branch.png -- 33.63kb -> 21.59kb (35.79%) /docs/development/workflow/branch_dropdown.png -- 39.03kb -> 25.14kb (35.58%) /docs/_static/timer_prediction_pow10.png -- 27.79kb -> 21.70kb (21.91%) /docs/_static/astropy_banner_96.png -- 26.21kb -> 23.03kb (12.13%) /docs/_static/astropy_banner.svg -- 21.47kb -> 19.23kb (10.43%) /docs/time/time_scale_conversion.png -- 6.61kb -> 6.33kb (4.36%) /examples/io/Hs-2009-14-a-web.jpg -- 22.94kb -> 22.44kb (2.18%) Signed-off-by: ImgBotApp <[email protected]>
nstarman
pushed a commit
that referenced
this pull request
Apr 4, 2022
break out into test_constellation_edge_cases
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Beep boop. Your images are optimized!
Your image file size has been reduced by 32% 🎉
Details
Black Lives Matter | 💰 donate | 🎓 learn | ✍🏾 sign
📝 docs |
repo | 🙋🏾 issues | 🏅 swag | 🏪 marketplace