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.

Sign up for GitHub

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

Jump to bottom

Parallelise postage stamps #3

Open
ajstewart opened this issue Dec 14, 2018 · 2 comments
Open

Parallelise postage stamps #3

ajstewart opened this issue Dec 14, 2018 · 2 comments
Assignees
@ajstewart
Labels
enhancement New feature or request help wanted Extra attention is needed pipeline Issue related to the pipeline.

Comments

@ajstewart
Copy link
Owner

May be possible to produce these faster by parallelising the code, but may run into issues of an image attempting to be accessed at the same time. In addition to multiple versions of the same plot being loaded into memory.
@ajstewart ajstewart added the enhancement New feature or request label Dec 14, 2018
@ajstewart ajstewart self-assigned this Dec 14, 2018
@ajstewart
Copy link
Owner Author

This has been attempted but I could not get matplotlib to play nice when generating the plots in parallel. Text would sometimes appear misplaced or stretched, including the legend. I think the issue is rooted at savefig or something to do with how matplotlib adds text to the figure.

See examples below. For now on hold. Probably best to run multiple instances of the process at once to get through images.

Separate plotting script?

sumss_21h08m43 01s_-48d55m02 2001s_sidebyside
sumss_20h26m46 47s_-50d18m43 8998s_sidebyside

@ajstewart ajstewart added help wanted Extra attention is needed on hold labels Jan 30, 2019
ajstewart added a commit that referenced this issue Feb 7, 2019
@ajstewart
First iteration of transient functionality
663aca6 
Added the first iteration of the transient functionality of the script. Also added the framework for multicore simultaneous postage stamp plotting, but this deactivated for now due to issues reported in #3. Addresses #1, #6 and #7.
@ajstewart ajstewart mentioned this issue Feb 7, 2019
Merged
@ajstewart
Copy link
Owner Author

This will be possible when dropping aplpy and switching to matplotlib/astropy. A new postage stamp routine will probably be needed.

@ajstewart ajstewart added pipeline Issue related to the pipeline. and removed on hold labels Nov 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ajstewart ajstewart

Labels
enhancement New feature or request help wanted Extra attention is needed pipeline Issue related to the pipeline.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ajstewart