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

Raster extent issue 77 #89

Merged
merged 5 commits into from
Oct 14, 2024
Merged

Raster extent issue 77 #89

merged 5 commits into from
Oct 14, 2024

Conversation

misgna3
Copy link
Contributor

@misgna3 misgna3 commented Oct 11, 2024

Closes #77
This fixes issue #77 related to the issue of introducing vector objects in the raster introduction episode on raster extent negatively affecting the flow of the lesson.
-Previous image depicting vector objects for raster extent replaced with appropriate image depicting raster extent.
-Challenge question and answer about spatial extent under this section moved to introduction to vector section.

Delete the figure depicting vector spatial extent. Will be replaced with appropriate figure depicting raster spatial extent.
A figure depicting raster spatial extent.
Remove the challenge question and answer that was deemed irrelevant after changing the vector spatial extent figure with appropriate figure depicting raster spatial extent. Will insert this challenge question and answer in the intro to vector data episode where it is relevant.
The challenge and answer question about spatial extent that has been deleted from the intro to raster episode is inserted in the appropriate intro to vector episode where it belongs.
Figure depicting vector spatial extent is inserted. This figure and the challenge and answer associated with it were previously in the intro to raster episode. Introducing vector extent in the raster episode before discussing vector data concept was disorienting to a reader.
Copy link

🆗 Pre-flight checks passed 😃

This pull request has been checked and contains no modified workflow files, spoofing, or invalid commits.

It should be safe to Approve and Run the workflows that need maintainer approval.

@misgna3 misgna3 merged commit 2a12b04 into datacarpentry:main Oct 14, 2024
2 of 3 checks passed
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 this pull request may close these issues.

Introduction of raster extent
1 participant