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

It should not be, but is, possible to reclass from CCAP to NLCD and vice-versa #84

Open
2 tasks done
NSPECT opened this issue Mar 23, 2022 · 3 comments
Open
2 tasks done
Labels
enhancement New feature or request

Comments

@NSPECT
Copy link
Collaborator

NSPECT commented Mar 23, 2022

Latest QNSPECT version

  • I am running the latest QNSPECT version

Similar requests do not exist

  • I have searched existing issues to make sure a similar request does not exist

Feature description

Given our current setup, it is possible to reclass from CCAP to NLCD and vice-versa. That doesn't make sense and should not be permitted.

I suggest adding a check on range of values in the incoming raster to determine type and then ask.

Additional context

No response

@NSPECT NSPECT added the enhancement New feature or request label Mar 23, 2022
@ar-siddiqui
Copy link
Collaborator

Can you provide more details on where is this happening which tool and process? Is this for the modify land use by name tool?

@NSPECT NSPECT changed the title It is possible to reclass from CCAP to NLCD and vice-versa It should not be, but is, possible to reclass from CCAP to NLCD and vice-versa Mar 23, 2022
@NSPECT
Copy link
Collaborator Author

NSPECT commented Mar 23, 2022

Yes, sorry. It is in the Modify_Land_Use tool. Since we display the combined CCAP+NLCD list of land covers, it is possible to choose the wrong one.

The different classification schemes have different arrays of values. CCAP is 0 through 25 and uses all those values. NLCD's values range from 11 to 95, with just a few of the intermediate values used (19 for the 2001-2006 classification). They use a hierarchical numbering scheme.

I wonder if we can easily & quickly query the uploaded raster for its min and max values. Then use that to determine the classification and to limit subsequent choices. The only confusion in the min-max range is in the unlikely event that someone uses NLCD with only water, glaciers, and developed areas in their raster. I'm not worried about that use case.

Classification schemes attached.

landcover_ccap2001.txt
Landcover_nlcd2001_2006.txt

@ar-siddiqui
Copy link
Collaborator

ar-siddiqui commented Mar 23, 2022

I understand. This makes sense but it would require creating a custom widget that can alter choices based on user input. Out of the box functionality of QGIS doesn't allow this. A good one for the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants