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

Remove mention of Sorensen's index from table C3 caption #26

Closed
2 tasks
egouldo opened this issue Jun 3, 2024 · 4 comments
Closed
2 tasks

Remove mention of Sorensen's index from table C3 caption #26

egouldo opened this issue Jun 3, 2024 · 4 comments
Assignees
Labels
bug an unexpected problem or unintended behavior

Comments

@egouldo
Copy link
Owner

egouldo commented Jun 3, 2024

  • Model not included in that parameter table, so remove from caption.
  • But check that model parameter estimates for Sorensen's models are included somewhere in that document.
@egouldo egouldo added the bug an unexpected problem or unintended behavior label Jun 3, 2024
@egouldo egouldo self-assigned this Jun 3, 2024
@egouldo egouldo added this to the Respond Peer Review Comments milestone Jul 29, 2024
@egouldo egouldo assigned parkerth and unassigned egouldo Jul 31, 2024
@parkerth
Copy link
Collaborator

Unfortunately, I cannot solve this issue myself at this time.

current table caption:
"Table C.3: Parameter estimates for univariate models of Box-Cox transformed deviation from the mean yi estimate as a function of categorical peer-review rating, continuous peer-review rating, and Sorensen’s index for blue tit and Eucalyptus analyses, and also for the inclusion of random effects for Eucalyptus analyses. "

So, there are two things mentioned in the caption that are not in the table.

  1. Sorensen's index for both Euc and BT for yi
  2. Inclusion of random effects for Euc for yi

Neither of these things are present in any other table. So we have two choices. We could either
a. add these things to this table
or
b. create one or two new tables.

I am leaving the table caption as is right now until we decide which path to take.

@egouldo egouldo assigned hannahsfraser and unassigned parkerth Aug 1, 2024
@egouldo
Copy link
Owner Author

egouldo commented Aug 1, 2024

Thanks Tim, I couldn't find them either, but wasn't sure if I had overlooked them. I reckon we:

  • include these analyses in this table rather than making a new table. Either myself or @hannahsfraser could probably do that part.

hannahsfraser added a commit that referenced this issue Aug 8, 2024
As per issue #26 I have added sorensen and random effects results to this table - note that I haven't been able to check this because I cant seem to run any many analysts stuff on my computer anymore.
@hannahsfraser
Copy link
Collaborator

I have attempted a fix of this but cant test it at the moment. See commit #66

@egouldo
Copy link
Owner Author

egouldo commented Aug 10, 2024

Closing, fixed in #66

@egouldo egouldo closed this as completed Aug 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

3 participants