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

Do we now have our own setMisc and getMisc that intefere with the AlphaSimR's? #584

Open
janaobsteter opened this issue Aug 29, 2024 · 3 comments
Open

Do we now have our own setMisc and getMisc that intefere with the AlphaSimR's? #584

janaobsteter opened this issue Aug 29, 2024 · 3 comments

Comments

@janaobsteter
Copy link
Collaborator

Not sure about this, just writing it down so that we check. The package is now on CRAN, so the current main will reflect that. We can correct everything in the next push
@gregorgorjanc
Copy link
Member

Are you getting any errors @janaobsteter ?

@janaobsteter
Copy link
Collaborator Author

It doesn't throw an error but it could be that one masks the other. Tbh, I forgot to check before submitting to CRAN

@gregorgorjanc
Copy link
Member

I would imagine that we would get some tests failing!? I think that NAMESPACEs should help us here, but maybe we will have to make it a generic method with different versions for Pop vs Colony classes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gregorgorjanc @janaobsteter