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.

Sign up for GitHub

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

Jump to bottom

i.vi: fix PVI formula #1985

Merged
merged 2 commits into from
Nov 28, 2021
Merged

i.vi: fix PVI formula #1985

merged 2 commits into from
Nov 28, 2021

Conversation

pesekon2
Copy link
Contributor

fix #1984

@pesekon2 pesekon2 added bug Something isn't working C Related code is in C labels Nov 22, 2021
@pesekon2 pesekon2 self-assigned this Nov 22, 2021
@pesekon2
i.vi: change a value to the original one
35613b9 
change of the parameter will be done in a separate PR
@pesekon2
Copy link
Contributor Author

pesekon2 commented Nov 22, 2021 •
edited
Loading

@YannChemin: As you are the original author of the module, could I ask you to be a reviewer? (for some reason, I couldn't choose you in the reviewer combobox)

EDIT: I can tag you after mentioning you here in the comment. So I have taken the privilege to tag you as a reviewer.

@YannChemin
Copy link
Contributor

YannChemin commented Nov 23, 2021 via email •
edited by neteler
Loading

@neteler
Copy link
Member

neteler commented Nov 23, 2021

@YannChemin: As you are the original author of the module, could I ask you to be a reviewer? (for some reason, I couldn't choose you in the reviewer combobox)

(FYI: Yann was missing from the GitHub committers team, I have invited him yesterday)

@neteler neteler modified the milestones: 7.8.7, 8.0.0 Nov 24, 2021
@pesekon2 pesekon2 merged commit 98f11bf into OSGeo:main Nov 28, 2021
@pesekon2 pesekon2 deleted the fix_pvi_formula branch November 28, 2021 06:41
pesekon2 added a commit to pesekon2/grass that referenced this pull request Nov 28, 2021
@pesekon2
i.vi: fix PVI formula (OSGeo#1985)
951d8a8 
* Fix wrong mathematical formula in the PVI index computation in i.vi

* fix OSGeo#1984
pesekon2 added a commit to pesekon2/grass that referenced this pull request Nov 28, 2021
@pesekon2
i.vi: fix PVI formula (OSGeo#1985)
0378bd8 
* Fix wrong mathematical formula in the PVI index computation in i.vi

* fix OSGeo#1984
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Oct 26, 2022
@pesekon2 @ninsbl
i.vi: fix PVI formula (OSGeo#1985)
de304b0 
* Fix wrong mathematical formula in the PVI index computation in i.vi

* fix OSGeo#1984
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Feb 17, 2023
@pesekon2 @ninsbl
i.vi: fix PVI formula (OSGeo#1985)
b30f502 
* Fix wrong mathematical formula in the PVI index computation in i.vi

* fix OSGeo#1984
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YannChemin YannChemin Awaiting requested review from YannChemin

Assignees

@pesekon2 pesekon2

Labels
bug Something isn't working C Related code is in C
Projects
None yet
Milestone
8.0.0
Development

Successfully merging this pull request may close these issues.

[Bug] i.vi: The formula for the PVI computation is wrong
3 participants
@pesekon2 @YannChemin @neteler