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

Update psevents.dvc (2nd attempt) #8675

Merged
merged 4 commits into from
Jan 10, 2025
Merged

Update psevents.dvc (2nd attempt) #8675

merged 4 commits into from
Jan 10, 2025

Conversation

Esteban82
Copy link
Member

Second attemp to #8656. Related to #8611

@Esteban82 Esteban82 self-assigned this Jan 9, 2025
@Esteban82 Esteban82 requested a review from seisman January 9, 2025 18:18
@Esteban82 Esteban82 requested a review from remkos January 9, 2025 18:18
Copy link
Contributor

github-actions bot commented Jan 9, 2025

Summary of changed images

This is an auto-generated report of images that have changed on the DVC remote

Status Path
modified test/baseline/psevents/
modified test/baseline/psevents/events.ps

Image diff(s)

Added images

Modified images

Path Old New
test/baseline/psevents/events.ps

Report last updated at commit 57f7f4d

@Esteban82
Copy link
Member Author

This is an auto-generated report of images that have changed on the DVC remote

Status Path
deleted test/baseline/psevents/event_lines.ps
deleted test/baseline/psevents/eventlabelbox.ps
deleted test/baseline/psevents/events.ps
deleted test/baseline/psevents/lines.ps
deleted test/baseline/psevents/polygons.ps
deleted test/baseline/psevents/psevents_functions.ps
deleted test/baseline/psevents/varline.ps
modified test/baseline/psevents/

Again the same message.

@seisman
Copy link
Member

seisman commented Jan 10, 2025

@Esteban82

Could you try run dvc add test/baselines/psevents/ instead, rather than dvc add test/baselines/psevents/events.ps.

@seisman
Copy link
Member

seisman commented Jan 10, 2025

Are you sure you have the new psevents/events.ps before running dvc add test/baselines/psevents/?

@Esteban82
Copy link
Member Author

Are you sure you have the new psevents/events.ps before running dvc add test/baselines/psevents/?

Mm, not in this PC. Thanks

@seisman
Copy link
Member

seisman commented Jan 10, 2025

That's weird. Could you please upload the new events.ps file then I can give it a try.

@Esteban82
Copy link
Member Author

Here is the file. It's actually a ps but I changed the extension to pdf so I could upload it.

events.pdf

@seisman
Copy link
Member

seisman commented Jan 10, 2025

Now it works. Maybe you forgot to run dvc push?

@Esteban82
Copy link
Member Author

Probably yes. Thanks again!

@seisman seisman merged commit d317a5d into master Jan 10, 2025
4 of 6 checks passed
@seisman seisman deleted the Update-psevents.dvc-2nd branch January 10, 2025 15:25
@Esteban82 Esteban82 added the documentation Improve documentation label Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improve documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants