Skip to content
This repository has been archived by the owner on Jun 30, 2023. It is now read-only.

p1_NLCD_data_zipped gets outdated after first run through pipeline #61

Closed
jds485 opened this issue Jan 21, 2022 · 4 comments · Fixed by #163
Closed

p1_NLCD_data_zipped gets outdated after first run through pipeline #61

jds485 opened this issue Jan 21, 2022 · 4 comments · Fixed by #163
Assignees
Labels
pipeline issues related to targets pipeline or tar_meta

Comments

@jds485
Copy link
Member

jds485 commented Jan 21, 2022

After a 1st tar_make(), three NLCD targets are outdated (image below). When a 2nd tar_make() is called, the first two (zipped and unzipped) are rebuilt, and the last is skipped. After that, tar_make() skips all targets. So, I'm expecting that something in the p1_NLCD_data function is causing the previous two NLCD targets to become outdated. I did not see anything that would cause it, though.
image

@jds485
Copy link
Member Author

jds485 commented Mar 16, 2022

Noting that this should be addressed by the suggested ScienceBase date last modified check in #98.

@lekoenig lekoenig added documentation Improvements or additions to documentation pipeline issues related to targets pipeline or tar_meta and removed documentation Improvements or additions to documentation labels Mar 17, 2022
@msleckman
Copy link
Collaborator

is this still happening btw?

@jds485
Copy link
Member Author

jds485 commented Mar 17, 2022

yes. it happened yesterday when I tested building the full pipeline in the container

@jds485
Copy link
Member Author

jds485 commented May 25, 2022

I figured out why this is happening: The p1_NLCD2011_data_unzipped target writes an unzipped directory within the p1_NLCD2011_data_zipped output directory, so that causes p1_NLCD2011_data_zipped to become outdated.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pipeline issues related to targets pipeline or tar_meta
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants