Skip to content

Commit b021e14

Browse files
authored
Apply suggestions from code review
1 parent 01d3849 commit b021e14

File tree

1 file changed

+13
-4
lines changed

1 file changed

+13
-4
lines changed

finance/proposal-calls/cycle4/moritz_ascii.md

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@ Addressing issues in `astropy.io.ascii`
88

99
### Project Description
1010

11-
Address (by fixing or closing) open issues in `astropy.io.ascii`
11+
Address (by fixing or closing) open issues in `astropy.io.ascii`, astropy affiliate editor, organization of dev-telecons, development and review of other astropy and specutils code.
1212

1313

1414
### Project / Work
15-
15+
Part 1: `astropy.io.ascii`
1616
At the time of writing there are [66 open issues in astropy tagged `io.ascii`](https://github.com/astropy/astropy/issues?q=is%3Aopen+is%3Aissue+label%3Aio.ascii), 57 of which are older than 18 months.
1717

1818
While astropy clearly thrives with those bugs and issues, each of them represents at least one user
@@ -27,6 +27,15 @@ require upstream fixes or
2727
[would require fairly invasive changes that would break backwards compatibility in a way that does not seem warrented](https://github.com/astropy/astropy/issues/5440#issuecomment-880703756));
2828
however most can be addressed given some developer time.
2929

30+
Part 2:
31+
Organizational work for Astropy:
32+
- Astropy affiliate package editor with pyOpenSci
33+
- organization of dev-telecons
34+
- strategic planning and proposal writing
35+
36+
Part 3:
37+
38+
Bring X-ray spectroscopy to `specutils` and `astropy.modelling`. Moritz is one of the core developers of [sherpa](https://github.com/sherpa/sherpa/). Sherpa is a complete package for X-ray spectral modelling with an infrastructure very similar to `astropy.modelling`, providing (a) some models and fitters that astropy misses and (b) ways to handle [spectra and responses in the OGIP format](https://heasarc.gsfc.nasa.gov/docs/heasarc/ofwg/docs/spectra/ogip_92_007/node5.html). Indeed, we have made a [bridge between sherpa and astropy a few years ago](https://saba.readthedocs.io/en/latest/), but maintaining compatibility between three separate packages (astropy, saba, sherpa) through versions turned out to be cumbersome. Also, Sherpa has a lot of infrastructure to read OGIP compliant X-ray data (PHA, ARF, RMF) that would fit well into a `specutils.OGIP` package to make those formats available to other parts of the astropy ecosystem. With Sherpa's continued developent in question given NASA's budget proposal to wind down Chandra support, I suggest to move or re-implement (depending on package license) those routines. Several (non affiliated) Python packages have done bits and parts of this, but it seems the time has come to bring the OGIP formats to a well-used astropy ecosystem package like `spectuils`.
3039
Moritz proposes to pay a sub-grant to MIT to pay for some of his time to
3140
work on those issues. Based on previous PRs, I estimate that I can address
3241
on average one issue in one to two work days, so paying for 10% of my work time for one year should allow me to address the majority of the currently open issues.
@@ -37,11 +46,11 @@ on average one issue in one to two work days, so paying for 10% of my work time
3746

3847
Budget breakdown (nominal):
3948

40-
- $25,000 for 10% of my time for 1 year
49+
- $25,000 for 10% of my time per year
4150

4251
(Since this will be a sub-award to MIT, my salary is fixed by MIT and the overhead rate is fixed by agreement between MIT and the US government. We can change the length of the contract, but not the hourly rate.)
4352

4453
The minimum useful time is determined by how much effort it is to set up that sub-award. I can work on one issue in 8 hours of work, but neither MIT nor NumFOCUS would want to write and process a subaward contract for that little money. Realistically, it's probably not worth the effort if less than $5-10k are allocated to this project.
4554

4655
### Period of Performance
47-
1 year
56+
3 years

0 commit comments

Comments
 (0)