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

Memspec type attribute is not required #3

Closed
Sv3n opened this issue Jul 22, 2014 · 0 comments
Closed

Memspec type attribute is not required #3

Sv3n opened this issue Jul 22, 2014 · 0 comments
Assignees
@Sv3n
Labels
enhancement

Comments

@Sv3n
Copy link
Collaborator

Sv3n commented Jul 22, 2014

The type attribute (i.e. int, double, etc) assigned to parameters in memspec files is not used. Locally I have made its parsing optional but kept the dtd file compatible. I think we should remove it completely from the parser.
@Sv3n Sv3n self-assigned this Jul 22, 2014
Sv3n added a commit that referenced this issue Jul 25, 2014
@Sv3n
[#3] Made 'type' attribute in memspec optional.
c76ade0 
It is only used in the streaming output function of Parameter, which we don't call anyway. It only clutters the memspecs.
@Sv3n Sv3n closed this as completed Jul 25, 2014
myzinsky pushed a commit that referenced this issue Jan 6, 2020
Merge pull request #3 from kannoth/lpddr4_and_hbm2
970cfb0 
Fixes on libdrampower Make and Multirank power calculation
IESE-T3 added a commit that referenced this issue Oct 26, 2022
@IESE-T3
CI Test #3
f67f198
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sv3n Sv3n

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Sv3n