balglave / FishMap

https://balglave.github.io/FishMap/
Other
0 stars 1 forks source link

As user, I'd like to have all asked parameters as entry of the `fm_load_data()` correctly documented #43

Closed statnmap closed 1 year ago

statnmap commented 1 year ago

As user, I'd like to have all parameters possible as entry of the functions correctly documented, so that I can adapt the model to my datasets

Client - Validation

image

=> https://balglave.github.io/FishMap/reference/fm_load_data.html

image

=> https://balglave.github.io/FishMap/reference/fm_load_data.html => https://github.com/balglave/FishMap/blob/main/man/fm_load_data.Rd

Screenshot 2023-03-21 at 10 45 37 image image

=> https://balglave.github.io/FishMap/reference/fm_load_data.html

We specify possible values with match.atg() : image

=> https://github.com/balglave/FishMap/blob/main/R/fm_load_data.R

match.arg() returns an error for incorrect values : image

The error behavior is integrated in the test of the function : image

=> https://github.com/balglave/FishMap/blob/main/tests/testthat/test-fm_load_data.R

In test :

image

=> https://github.com/balglave/FishMap/blob/main/tests/testthat/test-fm_load_data.R

In examples :

image

=> https://github.com/balglave/FishMap/blob/main/vignettes/user-running-fishmap.Rmd

Dev - Tech

Client inputs

To be able to write the correct documentation, could you please list all parameters that you would like to see in the parameters of the functions, separated by family and by function

To do so, we will ask you to list the parameters as a 'roxygen' documentation for each function.
_If you think it would be better to move some parameters in the fm_fit_model(), it is still time to ask._

You can look at the current documentation of each function to see what already exists: https://balglave.github.io/FishMap/reference/index.html

We would need this information:

You can give this information as a csv/Excel file if you want to, or any other mean that suits you

JCasemajor commented 1 year ago

Hi, here are the parameters we would like to see in the functions.

functions_parameters.xlsx

Note that we removed anisotropy and const_spphab from fm_fit_model() and added fitted_data to fm_load_data().

JCasemajor commented 1 year ago

Je valide.

balglave commented 1 year ago

Je valide