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 initial draft of a Roadmap page #373

Merged
merged 4 commits into from
Jul 28, 2021
Merged

Conversation

djhoese
Copy link
Member

@djhoese djhoese commented Jul 21, 2021

A couple months ago a couple of us have been having "design meetings" to work towards a Pyresample 2.0 design. As part of these meetings we conducted a user survey and got a lot of good responses. After a lot of back and forth we ended up with some important changes and what we think are good interfaces. In order to get the word out there and get more feedback on this I've put together a Roadmap document for the website that people (maintainers and users) can refer to when talking about Pyresample 2.0.

Let me know what you think. I'm not sure how much of this I just assumed from our meetings and how much is what we actually agreed upon. You won't hurt my feelings if you completely disagree. That's the point of this, let's discuss.

@djhoese djhoese added this to the v2.0 milestone Jul 21, 2021
@djhoese djhoese self-assigned this Jul 21, 2021
@codecov
Copy link

codecov bot commented Jul 21, 2021

Codecov Report

Merging #373 (fe247db) into main (e49adbd) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #373   +/-   ##
=======================================
  Coverage   93.52%   93.52%           
=======================================
  Files          50       50           
  Lines       10406    10406           
=======================================
  Hits         9732     9732           
  Misses        674      674           
Flag Coverage Δ
unittests 93.52% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e49adbd...fe247db. Read the comment docs.

@coveralls
Copy link

coveralls commented Jul 21, 2021

Coverage Status

Coverage remained the same at 93.512% when pulling fe247db on djhoese:doc-roadmap into e49adbd on pytroll:main.

Copy link
Member

@mraspaud mraspaud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for putting this together! It looks really good. I just have a small comment about the last paragraph.

docs/source/roadmap.rst Outdated Show resolved Hide resolved
@mraspaud
Copy link
Member

The rtd build is failing, is that expected?

@djhoese
Copy link
Member Author

djhoese commented Jul 22, 2021

No, but I think this was fixed in @adybbroe's PR. I may have made this branch without updating to include his changes (I had this roadmap sitting locally for a while).

Edit: I'll update in a bit after a meeting.

@djhoese
Copy link
Member Author

djhoese commented Jul 22, 2021

Or I'll do it now 😉

@djhoese
Copy link
Member Author

djhoese commented Jul 22, 2021

Coveralls upload failed it seems so I'm restarting the jobs.

Copy link
Member

@sfinkens sfinkens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job, thank you very much! Just one little typo.

docs/source/roadmap.rst Outdated Show resolved Hide resolved
@djhoese
Copy link
Member Author

djhoese commented Jul 23, 2021

I'm realizing I should add something about xarray and geoxarray. I'll try to do that later today.

@djhoese
Copy link
Member Author

djhoese commented Jul 28, 2021

You two approved this before going on vacation. I need to merge this so that I can link to it from a progress report due next week. I can fix any issues you have with it later. Thanks for understanding.

@djhoese djhoese merged commit d7567b3 into pytroll:main Jul 28, 2021
@djhoese djhoese deleted the doc-roadmap branch July 28, 2021 19:16
@sfinkens
Copy link
Member

sfinkens commented Aug 9, 2021

Now it's even better :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants