You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for DAISIEmainland and its thorough testing! One of my favorite functions in that regard is create_test_island_spec, with examples for 43 scenarios! That seems like a solid foundation for a solid R package!
What is unexpected, however, is that it is never actually used, e.g. in plotting functions. Also, the tests for some scenarios is exactly the same:
In other words, the different scenarios are untested to be different.
From this line of reasoning, the create_test_island_spec appears to have underused potential. I do predict this function had their use for e.g. a function plotting the ontology (#11), but as that plotting functionality did not get in, create_test_island_spec was left close to empty handed.
I do see that some scenarios of create_test_island_spec are used (see below), yet nowhere I see a test with e.g. scenario 42 and 43 give different results.
As an observant user, I hope I can see all create_test_island_spec scenario's shine, it would definitely be helpful to be sure DAISIEmainland works awesomely!
The text was updated successfully, but these errors were encountered:
Dear DAISIEmainland maintainer,
Thanks for DAISIEmainland and its thorough testing! One of my favorite functions in that regard is
create_test_island_spec
, with examples for 43 scenarios! That seems like a solid foundation for a solid R package!What is unexpected, however, is that it is never actually used, e.g. in plotting functions. Also, the tests for some scenarios is exactly the same:
In other words, the different scenarios are untested to be different.
From this line of reasoning, the
create_test_island_spec
appears to have underused potential. I do predict this function had their use for e.g. a function plotting the ontology (#11), but as that plotting functionality did not get in,create_test_island_spec
was left close to empty handed.I do see that some scenarios of
create_test_island_spec
are used (see below), yet nowhere I see a test with e.g. scenario 42 and 43 give different results.As an observant user, I hope I can see all
create_test_island_spec
scenario's shine, it would definitely be helpful to be sure DAISIEmainland works awesomely!The text was updated successfully, but these errors were encountered: