{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":2577269,"defaultBranch":"main","name":"specutils","ownerLogin":"astropy","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2011-10-14T16:11:38.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/847984?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1714581212.0","currentOid":""},"activityList":{"items":[{"before":"1a1164fa649dc20c160b647d0f0810b25a0227fd","after":"f36fe1b24f09a22cbd1344ce3b13003e70bcc848","ref":"refs/heads/main","pushedAt":"2024-05-14T15:33:47.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Separate out the codecov upload to a separate job in CI (#1138)","shortMessageHtmlLink":"Separate out the codecov upload to a separate job in CI (#1138)"}},{"before":"855a710659c26c16265df9836fa747e578305f93","after":"1a1164fa649dc20c160b647d0f0810b25a0227fd","ref":"refs/heads/main","pushedAt":"2024-05-08T18:07:32.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"pllim","name":"P. L. Lim","path":"/pllim","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/2090236?s=80&v=4"},"commit":{"message":"Merge pull request #1136 from pllim/sdss-remote-fix\n\nTST: Change SDSS URLs to HTTPS","shortMessageHtmlLink":"Merge pull request #1136 from pllim/sdss-remote-fix"}},{"before":"4e8412a46f7f527aafb5ca324dc20fafd7501776","after":"855a710659c26c16265df9836fa747e578305f93","ref":"refs/heads/main","pushedAt":"2024-05-01T16:34:50.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Changelog back to development","shortMessageHtmlLink":"Changelog back to development"}},{"before":"bf2355b3093e49adea5175f454e7e63992c9e2a6","after":"4e8412a46f7f527aafb5ca324dc20fafd7501776","ref":"refs/heads/main","pushedAt":"2024-04-30T15:51:07.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Add read, write, as_table, and from_qtable to SpectralRegion (#1133)\n\n* Add read, write, as_table, and from_qtable to SpectralRegion\r\n\r\n* Clarify docstrings\r\n\r\n* Add skips to automodapi\r\n\r\n* Add section in docs for new functionality\r\n\r\n* Fix doc build\r\n\r\n* Add test\r\n\r\n* Codestyle\r\n\r\n* Use plain old property instead of cached_property","shortMessageHtmlLink":"Add read, write, as_table, and from_qtable to SpectralRegion (#1133)"}},{"before":"a702a2c2f8b4245c63f334bc06e725191a16f981","after":"bf2355b3093e49adea5175f454e7e63992c9e2a6","ref":"refs/heads/main","pushedAt":"2024-04-30T14:08:48.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Minor changes to DC loaders for future surveys (#1122)\n\n* Include all remote data in coverage run\r\n\r\nThis should reduce the number of tests skipped, and resolve some of the\r\nweirdness with coverage seen in #1122.\r\n\r\n* Minor changes to DC loaders for future surveys\r\n\r\nThis makes it easier to use a fallback header and access the last\r\nspectra from a wrapper loader. These changes are to support loading some\r\nof the IFS surveys (such as SAMI) that Data Central hosts.\r\n\r\n* Add SAMI loaders for testing updated DC loaders\r\n\r\nThis adds an initial version of the future SAMI loaders, in order to\r\ntest the changes to the lower level DC loaders.","shortMessageHtmlLink":"Minor changes to DC loaders for future surveys (#1122)"}},{"before":"0524eb5110f6add133d9b7a9d564a589213a4ad4","after":"a702a2c2f8b4245c63f334bc06e725191a16f981","ref":"refs/heads/main","pushedAt":"2024-04-30T14:05:25.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Include all remote data in coverage run (#1132)\n\nThis should reduce the number of tests skipped, and resolve some of the\r\nweirdness with coverage seen in #1122.","shortMessageHtmlLink":"Include all remote data in coverage run (#1132)"}},{"before":"75d94462bf21f470c813f9b10fddec9ebcf28763","after":"0524eb5110f6add133d9b7a9d564a589213a4ad4","ref":"refs/heads/main","pushedAt":"2024-04-24T21:05:08.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Update github actions versions in CI (#1134)\n\n* Update actions versions\r\n\r\n* Use Codecov token repository secret\r\n\r\n* Try pull_request_target\r\n\r\n* Back to just on pull_request","shortMessageHtmlLink":"Update github actions versions in CI (#1134)"}},{"before":"5ec4931ff66454b545ccf45013b03b1265b97709","after":"75d94462bf21f470c813f9b10fddec9ebcf28763","ref":"refs/heads/main","pushedAt":"2024-04-16T15:43:24.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Added changelog section for next release","shortMessageHtmlLink":"Added changelog section for next release"}},{"before":"dac04cafe03079e487c9b5351a3713f1da09f6a7","after":"5ec4931ff66454b545ccf45013b03b1265b97709","ref":"refs/heads/main","pushedAt":"2024-04-05T20:13:58.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"MNT: Compat with numpy 2 and astropy 6.1 (#1130)","shortMessageHtmlLink":"MNT: Compat with numpy 2 and astropy 6.1 (#1130)"}},{"before":"fcf45cece9a25fcdc6bae62c966b2295f2f16906","after":null,"ref":"refs/heads/kelle-dev-docs","pushedAt":"2024-04-05T18:56:56.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"pllim","name":"P. L. Lim","path":"/pllim","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/2090236?s=80&v=4"}},{"before":"a3a32b47bc2cb3959118ae99e21c329a77c347f6","after":"bbaa7431296fa70b58031ac939a0e464129213e0","ref":"refs/heads/v2.0-dev","pushedAt":"2024-03-28T17:27:39.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Update spectrum class name from Spectrum1D to Spectrum (#1126)\n\n* Rename Spectrum1D to Spectrum\r\n\r\n* Update name in analysis\r\n\r\n* Update name in fitting and manipulation\r\n\r\n* Update name in tests\r\n\r\n* Update name in io\r\n\r\n* Update name in io tests\r\n\r\n* Add changelog entry\r\n\r\n* Update class name in docs\r\n\r\n* Update schema ref in spectrum_list asdf\r\n\r\n* Replace instances of 'spectrum1d'\r\n\r\n* Properly (I think) version change to asdf schema\r\n\r\n* Add PR number to changelog entry\r\n\r\n* Remove unused import\r\n\r\n* Fix doc example output\r\n\r\n* Fix failing remote data test\r\n\r\n* Update url to MANGA data for tests\r\n\r\n* Fix RSS filename","shortMessageHtmlLink":"Update spectrum class name from Spectrum1D to Spectrum (#1126)"}},{"before":"98dfcfd1906df4160d1597fa877da1b2b513bc9b","after":"dac04cafe03079e487c9b5351a3713f1da09f6a7","ref":"refs/heads/main","pushedAt":"2024-03-28T14:27:28.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Update MANGA urls in tests (#1128)","shortMessageHtmlLink":"Update MANGA urls in tests (#1128)"}},{"before":"20deaacf635d77475611aa97361bccaa4af72d44","after":"a3a32b47bc2cb3959118ae99e21c329a77c347f6","ref":"refs/heads/v2.0-dev","pushedAt":"2024-03-26T20:05:06.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Improved GWCS handling in Spectrum1D (#1074)\n\n* Allow spectral axis to be anywhere, instead of forcing it to be last (#1033)\n\n* Starting to work on flexible spectral axis location\n\nDebugging initial spectrum creation\n\nSet private attribute here\n\nWorking on debugging failing tests\n\nMore things are temporarily broken, but I don't want to lose this work so I'm committing here\n\nSet spectral axis index to 0 if flux is None\n\nWorking through test failures\n\nFix codestyle\n\nAllow passing spectral_axis_index to wcs_fits loader\n\nRequire specification of spectral_axis_index if WCS is 1D and flux is multi-D\n\nDecrement spectral_axis_index when slicing with integers\n\nPropagate spectral_axis_index through resampling\n\nFix last test to account for spectral axis staying first\n\nFix codestyle\n\nSpecify spectral_axis_index in SDSS plate loader\n\nGreatly simply extract_bounding_spectral_region\n\nAccount for variable spectral axis location in moment calculation, fix doc example\n\nWorking on SpectrumCollection moment handling...not sure this is the way\n\nNeed to add one to the axis index here\n\nUpdate narrative docs to reflect updates\n\n* Add back in the option to move the spectral axis to last, for back-compatibility\n\nWork around pixel unit slicing failure\n\nChange order on crop example\n\nFix spectral slice handling in tuple input case (e.g. crop)\n\nUpdate output of crop example\n\n* Apply suggestions from code review\n\nCo-authored-by: Adam Ginsburg \n\nApply suggestion from code review\n\nAdd helpful comment\n\n* Address review comment about move_spectral_axis, more docs\n\n* Add suggested line to docstring\n\nCo-authored-by: Erik Tollerud \n\n* Add convenience method\n\nMake this a docstring\n\n* Add v2.0.0 changelog section\n\n---------\n\nCo-authored-by: Erik Tollerud \n\n* Prepare changelog for 1.10.0 release\n\n* Fix Changelog\n\n* Fixed issues with ndcube 2.1 docs\n\n* Fix incorrect fluxes and uncertainties returned by FluxConservingResampler, increase computation speed (#1060)\n\n* new implementation of flux conserving resample\n\n* removed unused method\n\n* handle multi dimensional flux inputs\n\n* .\n\n* Update CHANGES.rst\n\nCo-authored-by: Erik Tollerud \n\n* omit removing units\n\n* added test to compare output to output from running SpectRes\n\n---------\n\nCo-authored-by: Erik Tollerud \n\n* Update changelog for 1.11.0 release\n\n* Changelog back to unreleased\n\n* Working on retaining full GWCS information in Spectrum1D rather than just spectral coords\n\n* Handle getting the spectral axis out of a GWCS\n\nAdd changelog heading\n\nRemove debugging prints\n\nFix changelog\n\nFix codestyle\n\n* Add changelog entry\n\n* Delete the commented-out old wavelength parsing code\n\n* More accurate changelog\n\n---------\n\nCo-authored-by: Erik Tollerud \nCo-authored-by: Nabil Freij \nCo-authored-by: Clare Shanahan ","shortMessageHtmlLink":"Improved GWCS handling in Spectrum1D (#1074)"}},{"before":"2fe4fa25b629e256cdc31367ed1dac428e34dc6a","after":"98dfcfd1906df4160d1597fa877da1b2b513bc9b","ref":"refs/heads/main","pushedAt":"2024-02-20T21:40:39.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Update function name to with_flux unit and update docstring+tests (#1124)","shortMessageHtmlLink":"Update function name to with_flux unit and update docstring+tests (#1124"}},{"before":"5aea420c1566dafb987624c5d47a4b3139af9537","after":"20deaacf635d77475611aa97361bccaa4af72d44","ref":"refs/heads/v2.0-dev","pushedAt":"2024-02-20T19:14:20.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Improved GWCS handling in Spectrum1D (#1074)\n\n* Allow spectral axis to be anywhere, instead of forcing it to be last (#1033)\n\n* Starting to work on flexible spectral axis location\n\nDebugging initial spectrum creation\n\nSet private attribute here\n\nWorking on debugging failing tests\n\nMore things are temporarily broken, but I don't want to lose this work so I'm committing here\n\nSet spectral axis index to 0 if flux is None\n\nWorking through test failures\n\nFix codestyle\n\nAllow passing spectral_axis_index to wcs_fits loader\n\nRequire specification of spectral_axis_index if WCS is 1D and flux is multi-D\n\nDecrement spectral_axis_index when slicing with integers\n\nPropagate spectral_axis_index through resampling\n\nFix last test to account for spectral axis staying first\n\nFix codestyle\n\nSpecify spectral_axis_index in SDSS plate loader\n\nGreatly simply extract_bounding_spectral_region\n\nAccount for variable spectral axis location in moment calculation, fix doc example\n\nWorking on SpectrumCollection moment handling...not sure this is the way\n\nNeed to add one to the axis index here\n\nUpdate narrative docs to reflect updates\n\n* Add back in the option to move the spectral axis to last, for back-compatibility\n\nWork around pixel unit slicing failure\n\nChange order on crop example\n\nFix spectral slice handling in tuple input case (e.g. crop)\n\nUpdate output of crop example\n\n* Apply suggestions from code review\n\nCo-authored-by: Adam Ginsburg \n\nApply suggestion from code review\n\nAdd helpful comment\n\n* Address review comment about move_spectral_axis, more docs\n\n* Add suggested line to docstring\n\nCo-authored-by: Erik Tollerud \n\n* Add convenience method\n\nMake this a docstring\n\n* Add v2.0.0 changelog section\n\n---------\n\nCo-authored-by: Erik Tollerud \n\n* Prepare changelog for 1.10.0 release\n\n* Fix Changelog\n\n* Fixed issues with ndcube 2.1 docs\n\n* Fix incorrect fluxes and uncertainties returned by FluxConservingResampler, increase computation speed (#1060)\n\n* new implementation of flux conserving resample\n\n* removed unused method\n\n* handle multi dimensional flux inputs\n\n* .\n\n* Update CHANGES.rst\n\nCo-authored-by: Erik Tollerud \n\n* omit removing units\n\n* added test to compare output to output from running SpectRes\n\n---------\n\nCo-authored-by: Erik Tollerud \n\n* Update changelog for 1.11.0 release\n\n* Changelog back to unreleased\n\n* Working on retaining full GWCS information in Spectrum1D rather than just spectral coords\n\n* Handle getting the spectral axis out of a GWCS\n\nAdd changelog heading\n\nRemove debugging prints\n\nFix changelog\n\nFix codestyle\n\n* Add changelog entry\n\n* Delete the commented-out old wavelength parsing code\n\n* More accurate changelog\n\n---------\n\nCo-authored-by: Erik Tollerud \nCo-authored-by: Nabil Freij \nCo-authored-by: Clare Shanahan ","shortMessageHtmlLink":"Improved GWCS handling in Spectrum1D (#1074)"}},{"before":"0b7098d83a3bb8bf1a1ed446ff2d040af0bd6ffe","after":"2fe4fa25b629e256cdc31367ed1dac428e34dc6a","ref":"refs/heads/main","pushedAt":"2024-02-19T21:23:38.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Changelog back to unreleased state","shortMessageHtmlLink":"Changelog back to unreleased state"}},{"before":"c646007fba23883bb796dd4975b0e4d1c1f60c2d","after":"0b7098d83a3bb8bf1a1ed446ff2d040af0bd6ffe","ref":"refs/heads/main","pushedAt":"2024-02-13T18:25:06.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"add: new SDSS V datatype loaders (#1107)\n\n* add: sdss_v.py\r\n\r\nworking on loaders\r\n\r\n* add: test.py for testing things, fix ordered method for apVisit\r\n\r\n* feat: apStar/apVisit functionality, new helper funcs\r\n\r\nnew helper funcs:\r\n - _fetch_metadata to perform grab of common metadata\r\n - _fetch_flux_unit to get flux unit from the given HDU and convert it to an astropy unit object\r\n\r\n* add: astra_readers.py\r\n\r\npulled from https://raw.githubusercontent.com/sdss/astra/298a73ce600db428cf0a2ed8a707a56c2182ae57/python/astra/tools/spectrum/readers.py\r\n\r\n* feat: BOSS specFull loaders\r\n\r\ncan fully load boss files now\r\n\r\nother changes:\r\n- commented out abstract @ things to make it so ipython autoreloads\r\n\r\nnotes:\r\n- flux unit where for BOSS files?\r\n- what the heck are spAll, zAll, and zList HDUs?\r\n- does the InverseVariance need a unit?\r\n\r\n* feat: mwmVisit, mwmStar loaders\r\n\r\nadded mwmVisit and mwmStar loaders.\r\n\r\nupdated demonstration notebook accordingly and output to PDF\r\n\r\ndel: test.pdf and test.ipynb\r\n\r\ndel: secret SDSS-V data\r\n\r\n* feat: specLite and other BOSS REDUX loader functionality\r\n\r\nable to now load all BOSS spec directly with the same underlying code.\r\n\r\nrequired refactoring methods into BOSS_spec loaders\r\n\r\n* chore: identifier + documentation\r\n\r\n* add: test_implementation.py\r\n\r\ngoing to now write implementation test\r\n\r\nadd\r\n\r\n* feat: partial implementation of loaders\r\n\r\nmwm confirmed working\r\n\r\nstill todo:\r\n- add HDU not-specified message\r\n- merge the mwm types into a single 2 loaders\r\n- confirm all other loaders work and add to __all__\r\n\r\n* feat: functioning loaders (+refactoring & chore: docs)\r\n\r\n- refactored BOSS spec methods and mwm spec methods into single functions for simplicity\r\n- all loaders WORKING!! (except apStar multi)\r\n- all the documentation + type hinting (excluding outputs)\r\n- changed variable names to standard types used in specutils\r\n- TODO: the apStar multi-loader is confusing, so it remains unimplemented for now.\r\n- CHECK: do I need to clean the files of zero vals?\r\n- TODO: BUNIT pulls for spec and mwm files\r\n- TODO: check with data team what mwm files are needed\r\n\r\n* add: test_implementation jupyter notebook\r\n\r\n- currently non-functional because of zero values in x-axis\r\n\r\n- deleted test_implementation.ipynb for policy reasons\r\n\r\n* fix: jdaviz nan and zero flux bug\r\n\r\n- jdaviz hates nan and zero flux, so they have to be removed\r\n- TODO: open issue on jdaviz repo about nan and zero flux bug\r\n\r\nthe bug originates in the x_min x_max values used for the redshift slider for line_lists (somehow) on nan and zero flux values in the Spectrum object.\r\n\r\n* feat: all multiloaders functional\r\n\r\napStar loader not yet tested because file is of length 1 (no visits)\r\nmwm loaders will SKIP any DATASUM=0 because Spectrum1D cannot be instantiated with zero data\r\n\r\n* fix: Astropy units warning + warning format -> print\r\n\r\n* ignore: demonstration and test files\r\n\r\n* fix: jdaviz specviz 1D in 2D array handling\r\n\r\nfixes a jdaviz issue regarding a 1D flux in a 2D object, where it gets confused and explodes\r\n\r\ni will put an issue in for it\r\n\r\nthis fix is different from the previous as it keeps all zero and NaN flux points\r\n\r\n* fix: header method -> .get() + other minor fixes\r\n\r\n* feat: unit tests on dummy data (excl. MWM)\r\n\r\nneed someone to help me write a BinTableHDU for mwm files...\r\n\r\n* feat: unit tests with assertions\r\n\r\nstill need to write mwm dummy file for the tests\r\n\r\nthere's also a foobar variable check for the metadata\r\n\r\n* chore: docustrings\r\n\r\n* fix: header fetch method -> specutils standard\r\n\r\nnow obtains header from PrimaryHDU in the HDUList, any data that was previously accessed through it has been removed too\r\n\r\n* del: mpl_preamble.py\r\n\r\n* feat: individual identifiers + unit tests update\r\n\r\n* fix: .gitignore list\r\n\r\nkeeping .jukit incase anyone else uses vim-jukit during dev\r\n\r\n* add: bitmasks to Spectrum object outputs\r\n\r\nSpectrum1D intializer converts any 0 to valid values. I'm assuming that zeroes in the bitmask means that its valid, as per manga.py\r\n\r\n* fix: spec mask, AND_MASK -> OR_MASK\r\n\r\nfix as per @Sean-Morrison 's suggestion in astropy pull req [#1107](https://github.com/astropy/specutils/pull/1107)\r\n\r\ncould be reverted in future, in which case this commit can just be deleted\r\n\r\n* fix: spec file identify OBSERVAT column\r\n\r\nOBSERVAT column not in everything so i changed it, also adding another LOGLAM check to the coadd HDU check.\r\n\r\n* fix: hdu spec -> find 1st hdu with data\r\n\r\ninstead of specifying a hdu on Spectrum1D loaders for spec and mwm types, it will not find the first HDU with data, or in the case of spec, just use the coadd.\r\n\r\nthis means that it works directly with jdaviz for those two datatypes correctly now.\r\n\r\nthere are no user facing methods, and I don't want to break anything, but it should be noted that these datafiles can contain several spectra, which inherently limits this.\r\n\r\nin theory, I could put everything as a Spectrum1D nD flux object, but I'm pretty sure that breaks sometimes for jdaviz.\r\n\r\n* add: mwm dummy file tests + mask fixes\r\n\r\n- force masks to be boolean prior to entering initializer\r\n- add mwm file tests based on dummy file (credit to @andycasey for those dummy file generators)\r\n- add more mwm file tests for failures\r\n- added checks to see if file is empty for mwm files based on datasum (failsafe)\r\n\r\n* Fix codestyle errors\r\n\r\n---------\r\n\r\nCo-authored-by: Ricky O'Steen ","shortMessageHtmlLink":"add: new SDSS V datatype loaders (#1107)"}},{"before":"425eab33ab5967a98706d8352a03cdb660c852af","after":"c646007fba23883bb796dd4975b0e4d1c1f60c2d","ref":"refs/heads/main","pushedAt":"2024-02-13T14:58:40.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Update __str__ and __repr__ for Spectrum1D (#1123)\n\n* Fixing numpy format error when printing Spectrum1D\r\n\r\n* Also shorten __repr__ to something more reasonable\r\n\r\n* Show actual spectral axis and flux if it's small\r\n\r\n* str test updated\r\n\r\n* Spectrum1D tests passing, more tweaks to format\r\n\r\n* Update docstrings\r\n\r\n* Fixing doc outputs\r\n\r\n* Fix typo\r\n\r\n* More docs fixes\r\n\r\n* Surprised this changed, testing...\r\n\r\n* Finish changing docs\r\n\r\n* Fix remote data doc tests\r\n\r\n* Went back to = over :, other improvements\r\n\r\n* Reconcile resample docstring updates\r\n\r\n* Add changelog","shortMessageHtmlLink":"Update __str__ and __repr__ for Spectrum1D (#1123)"}},{"before":"17a6f6b763af5cba87fb0f50b1ea32bdf22da1c5","after":"425eab33ab5967a98706d8352a03cdb660c852af","ref":"refs/heads/main","pushedAt":"2024-02-09T19:32:15.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Truncate in template_comparison by default rather than nan_fill (#1121)\n\n* Working on adding truncate option as default for template comparison, need diff strategy than None fill\r\n\r\n* Trying different strategy for truncate, just drop all NaNs\r\n\r\n* Working on resolving test failures, added warning for no overlap and return None instead of blank Spectrum1D\r\n\r\n* Warn in redshifted case if there's no overlap with any redshift\r\n\r\n* Resolve test failures, changed returned template to resampled version.\r\n\r\n* Resolve remaining test failures\r\n\r\n* Fix typos\r\n\r\n* Fix bug, improving test coverage\r\n\r\n* Return full normalized+redshifted template","shortMessageHtmlLink":"Truncate in template_comparison by default rather than nan_fill (#1121)"}},{"before":"30575fc5cec977181f4af07caf00c63baff24dbc","after":"17a6f6b763af5cba87fb0f50b1ea32bdf22da1c5","ref":"refs/heads/main","pushedAt":"2024-02-01T21:04:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Fix `with_spectral_unit` (#1119)\n\n* Fix with_spectral_axis when Spectrum1D instantiated with a spectral_axis\r\n\r\n* Deprecate old method name, new one is more specific\r\n\r\n* Works for WCS-initialized spectra as well\r\n\r\n* Remove unneeded imports\r\n\r\n* Add kwarg default\r\n\r\n* Add copy and deepcopy methods\r\n\r\nCodestyle\r\n\r\n* Update specutils/spectra/spectrum_mixin.py\r\n\r\nCo-authored-by: Derek Homeier <709020+dhomeier@users.noreply.github.com>\r\n\r\n* Add changelog entry\r\n\r\n* Test to make sure we kept the old WCS in meta\r\n\r\n* Better name for meta attribute, add test for it\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Derek Homeier <709020+dhomeier@users.noreply.github.com>\r\n\r\n* Fix codestyle\r\n\r\n* Update docstring\r\n\r\nCo-authored-by: Derek Homeier <709020+dhomeier@users.noreply.github.com>\r\n\r\n* Change docstring to fix docs build failure. Also fixed some incorrect references to SpectralRegion\r\n\r\n* Fix one more SpectralRegion reference\r\n\r\n---------\r\n\r\nCo-authored-by: Derek Homeier <709020+dhomeier@users.noreply.github.com>","shortMessageHtmlLink":"Fix with_spectral_unit (#1119)"}},{"before":"08e77ce4456fe7575ea2f3c2620277c33067c525","after":"30575fc5cec977181f4af07caf00c63baff24dbc","ref":"refs/heads/main","pushedAt":"2024-01-30T22:22:51.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Some updates to specutils.analysis functions (#937)\n\n* Updated template_redshift to return final redshifted spectrum and _chi_square_for_templates and _normalize_for_template_matching to accept NaNs\r\n\r\n* Update tests, only sum template where observed is not NaN\r\n\r\n---------\r\n\r\nCo-authored-by: Ricky O'Steen ","shortMessageHtmlLink":"Some updates to specutils.analysis functions (#937)"}},{"before":"47232064d511319fd8fc64f3e0b67863471712bd","after":"08e77ce4456fe7575ea2f3c2620277c33067c525","ref":"refs/heads/main","pushedAt":"2024-01-23T22:46:24.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Add a DESI file reader (#1116)\n\n* Initial dump of reader code\r\n\r\n* working on reader and tests.\r\n\r\n* working on identifiers\r\n\r\n* remove unused import\r\n\r\n* exclude deeper nested tests\r\n\r\n* add DESI test mini-files\r\n\r\n* fix style; work around missing importlib\r\n\r\n* additional parameters\r\n\r\n* set format keyword\r\n\r\n* add tests for full coverage\r\n\r\n* remove remote tests and rename files.\r\n\r\n* update test configuration","shortMessageHtmlLink":"Add a DESI file reader (#1116)"}},{"before":"fb5bd083f0ab6f2572e989a1c7c0b8f04219904a","after":"47232064d511319fd8fc64f3e0b67863471712bd","ref":"refs/heads/main","pushedAt":"2024-01-16T17:59:21.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Allow template correlation for spectrum with no uncertainty (#1118)\n\n* Allow template correlation for spectrum with no uncertainty\r\n\r\n* Add changelog","shortMessageHtmlLink":"Allow template correlation for spectrum with no uncertainty (#1118)"}},{"before":"4041081f221f488d9d55b19b6e347da8bdaa2225","after":"5aea420c1566dafb987624c5d47a4b3139af9537","ref":"refs/heads/v2.0-dev","pushedAt":"2024-01-09T18:44:22.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Improved GWCS handling in Spectrum1D (#1074)\n\n* Allow spectral axis to be anywhere, instead of forcing it to be last (#1033)\n\n* Starting to work on flexible spectral axis location\n\nDebugging initial spectrum creation\n\nSet private attribute here\n\nWorking on debugging failing tests\n\nMore things are temporarily broken, but I don't want to lose this work so I'm committing here\n\nSet spectral axis index to 0 if flux is None\n\nWorking through test failures\n\nFix codestyle\n\nAllow passing spectral_axis_index to wcs_fits loader\n\nRequire specification of spectral_axis_index if WCS is 1D and flux is multi-D\n\nDecrement spectral_axis_index when slicing with integers\n\nPropagate spectral_axis_index through resampling\n\nFix last test to account for spectral axis staying first\n\nFix codestyle\n\nSpecify spectral_axis_index in SDSS plate loader\n\nGreatly simply extract_bounding_spectral_region\n\nAccount for variable spectral axis location in moment calculation, fix doc example\n\nWorking on SpectrumCollection moment handling...not sure this is the way\n\nNeed to add one to the axis index here\n\nUpdate narrative docs to reflect updates\n\n* Add back in the option to move the spectral axis to last, for back-compatibility\n\nWork around pixel unit slicing failure\n\nChange order on crop example\n\nFix spectral slice handling in tuple input case (e.g. crop)\n\nUpdate output of crop example\n\n* Apply suggestions from code review\n\nCo-authored-by: Adam Ginsburg \n\nApply suggestion from code review\n\nAdd helpful comment\n\n* Address review comment about move_spectral_axis, more docs\n\n* Add suggested line to docstring\n\nCo-authored-by: Erik Tollerud \n\n* Add convenience method\n\nMake this a docstring\n\n* Add v2.0.0 changelog section\n\n---------\n\nCo-authored-by: Erik Tollerud \n\n* Prepare changelog for 1.10.0 release\n\n* Fix Changelog\n\n* Fixed issues with ndcube 2.1 docs\n\n* Fix incorrect fluxes and uncertainties returned by FluxConservingResampler, increase computation speed (#1060)\n\n* new implementation of flux conserving resample\n\n* removed unused method\n\n* handle multi dimensional flux inputs\n\n* .\n\n* Update CHANGES.rst\n\nCo-authored-by: Erik Tollerud \n\n* omit removing units\n\n* added test to compare output to output from running SpectRes\n\n---------\n\nCo-authored-by: Erik Tollerud \n\n* Update changelog for 1.11.0 release\n\n* Changelog back to unreleased\n\n* Working on retaining full GWCS information in Spectrum1D rather than just spectral coords\n\n* Handle getting the spectral axis out of a GWCS\n\nAdd changelog heading\n\nRemove debugging prints\n\nFix changelog\n\nFix codestyle\n\n* Add changelog entry\n\n* Delete the commented-out old wavelength parsing code\n\n* More accurate changelog\n\n---------\n\nCo-authored-by: Erik Tollerud \nCo-authored-by: Nabil Freij \nCo-authored-by: Clare Shanahan ","shortMessageHtmlLink":"Improved GWCS handling in Spectrum1D (#1074)"}},{"before":"f678dbd48fdf40654a5bc95ede65f80fae02d73e","after":"fb5bd083f0ab6f2572e989a1c7c0b8f04219904a","ref":"refs/heads/main","pushedAt":"2023-12-08T21:43:22.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"feat: add correlation mode to template_correlation (#1114)\n\n* feat: add correlation mode to template_correlation\r\n\r\n* Apply suggestions from code review\r\n\r\n* Clean up trailing whitespace\r\n\r\n---------\r\n\r\nCo-authored-by: Ricky O'Steen <39831871+rosteen@users.noreply.github.com>","shortMessageHtmlLink":"feat: add correlation mode to template_correlation (#1114)"}},{"before":"f24e19ccf61a8c55105ad40f56668c45eccc4c14","after":"f678dbd48fdf40654a5bc95ede65f80fae02d73e","ref":"refs/heads/main","pushedAt":"2023-12-04T17:43:28.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"add tabular-fits support for Spectrum1D mask (#1104)\n\n* add tabular-fits support for Spectrum1D mask\r\n\r\n* Fix codestyle\r\n\r\n---------\r\n\r\nCo-authored-by: Ricky O'Steen ","shortMessageHtmlLink":"add tabular-fits support for Spectrum1D mask (#1104)"}},{"before":"53af673246c33dec1a56f663e08d121981962279","after":"f24e19ccf61a8c55105ad40f56668c45eccc4c14","ref":"refs/heads/main","pushedAt":"2023-11-27T17:11:13.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"Update Weekly CI (#1115)\n\n* Update ci_cron_weekly.yml\r\n\r\n* Update action version","shortMessageHtmlLink":"Update Weekly CI (#1115)"}},{"before":"029ddfd43f00d6680941b72dc6aad0fe472d84ab","after":"53af673246c33dec1a56f663e08d121981962279","ref":"refs/heads/main","pushedAt":"2023-11-22T19:07:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"TST: Fix devdeps and clean up matrix (#1111)\n\n* TST: Pin casa-formats-io in oldest-deps\r\njob because it is upgrading numpy but we do not want that to happen.\r\n\r\n* Replace tmpdir with tmp_path (#1095)\r\n\r\n* Replace tmpdir with tmp_path\r\n\r\n* Fix path construction\r\n\r\n* Removed unused tmp_path\r\n\r\nCo-authored-by: P. L. Lim <2090236+pllim@users.noreply.github.com>\r\n\r\n---------\r\n\r\nCo-authored-by: P. L. Lim <2090236+pllim@users.noreply.github.com>\r\n\r\n* Fix devdeps and clean up matrix\r\nand stuff\r\n\r\n* Use astropy>=5.3 for doctest\r\n\r\n* Add tox back in\r\n\r\n---------\r\n\r\nCo-authored-by: Ricky O'Steen <39831871+rosteen@users.noreply.github.com>\r\nCo-authored-by: Ricky O'Steen ","shortMessageHtmlLink":"TST: Fix devdeps and clean up matrix (#1111)"}},{"before":"fd486302630e2b9b6df499208f21fdb99533de51","after":"029ddfd43f00d6680941b72dc6aad0fe472d84ab","ref":"refs/heads/main","pushedAt":"2023-11-22T18:54:04.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rosteen","name":"Ricky O'Steen","path":"/rosteen","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/39831871?s=80&v=4"},"commit":{"message":"One more fix to dev docs (#1112)\n\n* Update contributing.rst\r\n\r\nNeeded single quotes in order for pip install command to work.\r\n\r\n* Update contributing.rst\r\n\r\npin python version to 3.11","shortMessageHtmlLink":"One more fix to dev docs (#1112)"}},{"before":"374f14d8215ff1702f32065e75e2cc069831c2e9","after":"fcf45cece9a25fcdc6bae62c966b2295f2f16906","ref":"refs/heads/kelle-dev-docs","pushedAt":"2023-11-22T18:49:59.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kelle","name":"Kelle Cruz","path":"/kelle","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1444249?s=80&v=4"},"commit":{"message":"Update contributing.rst\n\npin python version to 3.11","shortMessageHtmlLink":"Update contributing.rst"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAESaRMGwA","startCursor":null,"endCursor":null}},"title":"Activity ยท astropy/specutils"}