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

Parameterize all config fields in odin module #194

Open
zduey opened this issue Apr 11, 2018 · 0 comments
Open

Parameterize all config fields in odin module #194

zduey opened this issue Apr 11, 2018 · 0 comments
Assignees
@zduey

Comments

@zduey
Copy link
Contributor

zduey commented Apr 11, 2018

Most of the values for ramulator configs are currently hard-coded in odin.py. Most of these are already available in the parameter classes that vary by experiment. It would be better to just pass around these classes instead of having the hard-coding
@zduey zduey added this to the 2.1.10 milestone Apr 11, 2018
@zduey zduey self-assigned this Apr 11, 2018
@mivade mivade removed this from the 2.1.10 milestone Jun 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zduey zduey

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mivade @zduey