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

Provide an informative repr for TiledDataset #401

Closed
Tracked by #398
SolarDrew opened this issue Jun 24, 2024 · 0 comments · Fixed by #402
Closed
Tracked by #398

Provide an informative repr for TiledDataset #401

SolarDrew opened this issue Jun 24, 2024 · 0 comments · Fixed by #402

Comments

@SolarDrew
Copy link
Contributor

No description provided.

@SolarDrew SolarDrew changed the title Provide an informative repr Provide an informative repr for TiledDataset Jun 24, 2024
SolarDrew added a commit to SolarDrew/dkist that referenced this issue Jun 24, 2024
Just uses ds[0, 0] for now for ease of putting something together.

Closes DKISTDC#401
Cadair pushed a commit to SolarDrew/dkist that referenced this issue Jul 11, 2024
Just uses ds[0, 0] for now for ease of putting something together.

Closes DKISTDC#401
Cadair pushed a commit to SolarDrew/dkist that referenced this issue Jul 18, 2024
Just uses ds[0, 0] for now for ease of putting something together.

Closes DKISTDC#401
SolarDrew added a commit to SolarDrew/dkist that referenced this issue Jul 26, 2024
Just uses ds[0, 0] for now for ease of putting something together.

Closes DKISTDC#401
SolarDrew added a commit to SolarDrew/dkist that referenced this issue Aug 7, 2024
Just uses ds[0, 0] for now for ease of putting something together.

Closes DKISTDC#401
@Cadair Cadair closed this as completed Aug 21, 2024
Cadair added a commit that referenced this issue Aug 26, 2024
* Add basic repr for TiledDataset

Just uses ds[0, 0] for now for ease of putting something together.

Closes #401

* Add tiles_shape property

* Add changelog

* Add method to slice all datasets

* Add new TiledDataset fixture because the old one wasn't behaving itself

* Add test for TiledDataset tile slicing

* Add a basic test for TiledDataset.__repr__

Ripped off from the one for Dataset.__repr__

* Apparently ndcube 2.1.2 doesn't allow `Dataset.shape`

* Test `tiles_shape`

* Add external object for slicing TiledDataset tiles

* Fix a thing

* Minor tweak

* Tweak

* Update changelog

---------

Co-authored-by: Stuart Mumford <stuart@cadair.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants