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

doc: continue renaming location to project in documentation #3570

Merged
merged 6 commits into from
Apr 17, 2024

Conversation

petrasovaa
Copy link
Contributor

@petrasovaa petrasovaa commented Apr 8, 2024

Some cases are simply replaced (location to project), in some cases 'coordinate reference system' or 'dataset' are better replacements.

@petrasovaa petrasovaa self-assigned this Apr 8, 2024
@petrasovaa petrasovaa added this to the 8.4.0 milestone Apr 8, 2024
@github-actions github-actions bot added GUI wxGUI related raster Related to raster data processing HTML Related code is in HTML libraries module docs general display imagery misc labels Apr 8, 2024
@petrasovaa petrasovaa requested a review from veroandreo April 8, 2024 21:12
@github-actions github-actions bot added vector Related to vector data processing temporal Related to temporal data processing raster3d labels Apr 9, 2024
Copy link
Contributor

@veroandreo veroandreo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reached r.in.lidar, will continue tomorrow. As commented in the other PRs, I was also trying to fix places where both project and projection appeared together in the same sentence or message. I believe, where possible, CRS is a good replacement for projection.

display/d.where/d.where.html Outdated Show resolved Hide resolved
general/g.gisenv/g.gisenv.html Show resolved Hide resolved
general/g.proj/g.proj.html Outdated Show resolved Hide resolved
general/g.proj/g.proj.html Outdated Show resolved Hide resolved
general/g.proj/g.proj.html Outdated Show resolved Hide resolved
raster/r.in.gdal/r.in.gdal.html Outdated Show resolved Hide resolved
raster/r.in.gdal/r.in.gdal.html Outdated Show resolved Hide resolved
raster/r.in.gdal/r.in.gdal.html Outdated Show resolved Hide resolved
raster/r.in.gdal/r.in.gdal.html Outdated Show resolved Hide resolved
raster/r.in.gdal/r.in.gdal.html Outdated Show resolved Hide resolved
Copy link
Contributor

@veroandreo veroandreo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I went through all files changed now. Some error messages I left untouched as they should be modified in r.in.gdal, v.in.ogr, r.proj and v.proj, IIRC. For a different PR perhaps :)

raster/r.in.lidar/r.in.lidar.html Outdated Show resolved Hide resolved
raster/r.in.lidar/r.in.lidar.html Outdated Show resolved Hide resolved
raster/r.in.pdal/r.in.pdal.html Outdated Show resolved Hide resolved
raster/r.in.pdal/r.in.pdal.html Outdated Show resolved Hide resolved
raster/r.proj/r.proj.html Outdated Show resolved Hide resolved
vector/v.in.ogr/v.in.ogr.html Outdated Show resolved Hide resolved
vector/v.in.ogr/v.in.ogr.html Outdated Show resolved Hide resolved
vector/v.in.ogr/v.in.ogr.html Outdated Show resolved Hide resolved
vector/v.in.ogr/v.in.ogr.html Outdated Show resolved Hide resolved
vector/v.in.pdal/v.in.pdal.html Outdated Show resolved Hide resolved
@github-actions github-actions bot added the C Related code is in C label Apr 11, 2024
@petrasovaa petrasovaa requested a review from veroandreo April 15, 2024 18:11
@petrasovaa petrasovaa enabled auto-merge (squash) April 17, 2024 15:38
@petrasovaa petrasovaa merged commit af7aa69 into OSGeo:main Apr 17, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Related code is in C display docs general GUI wxGUI related HTML Related code is in HTML imagery libraries misc module raster Related to raster data processing raster3d temporal Related to temporal data processing vector Related to vector data processing
Development

Successfully merging this pull request may close these issues.

2 participants