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

use getfield instead of eval in tagging_schemes.jl #247

Closed
wants to merge 1 commit into from

Conversation

rushabh-v
Copy link

resolves: #242

On testing, it raises UndefVarError: BIO1 not defined. I tried figuring out the reason but couldn't. Should I use something else instead of Main?

I am new to Julia, please guide me.

@rushabh-v
Copy link
Author

rushabh-v commented Jan 8, 2021

@aviks Could you help me, please?

@aviks
Copy link
Member

aviks commented Jan 8, 2021

You have to use the module in which these symbols are defined. Not Main. Main is the top level module (eg. in the REPL). These symbols are not defined in Main.

@rushabh-v
Copy link
Author

rushabh-v commented Jan 8, 2021

they are defined in the same module (i.e., tagging_schemes.jl).

@rssdev10
Copy link
Collaborator

implemented in 5778129

@rssdev10 rssdev10 closed this Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Eval can be replaced by getfield in tag_scheme!
3 participants