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

CPU frequency/temperature tests fail on data parsing for v1210 #207

Closed
Psotas opened this issue Feb 8, 2024 · 2 comments
Closed

CPU frequency/temperature tests fail on data parsing for v1210 #207

Psotas opened this issue Feb 8, 2024 · 2 comments
Assignees
Labels
bug Something isn't working needs feedback vault_jsl

Comments

@Psotas
Copy link
Contributor

Psotas commented Feb 8, 2024

Device

V1210

RTE version

Dasharo v0.9.2

Affected component(s) or functionality

No response

Brief summary

Improve CPU freq/temp tests to work properly for v1210

How reproducible

No response

How to reproduce

Run cpu-frequency.robot and cpu-temperature.robot tests.

Expected behavior

Test pass

Actual behavior

Tests fail on data parsing

Link to screenshots or logs

CPU-freq-temp_log.zip

CPF002.001, CPF002.001 and CPF004.001:

Traceback (most recent call last):
  None
RuntimeError: Evaluating expression 're.sub(r'(?s)[^0-9]*([1-9][0-9]*)[,.][0-9]+MHz', r'\1', $frequency)' failed: SyntaxError: invalid syntax (<string>, line 1)

CPT001.001 and CPT002.001:

Traceback (most recent call last):
  None
RuntimeError: '105.0°C)
' cannot be converted to a floating point number: ValueError: could not convert string to float: '105.0°C)\n'

Additional context

No response

Solutions you've tried

No response

@Psotas Psotas added the bug Something isn't working label Feb 8, 2024
@macpijan
Copy link
Contributor

@pkubaj please update the state here after retesting V1210 next week

@macpijan
Copy link
Contributor

This is still valid on revision: 04e4100

macpijan added a commit that referenced this issue Jul 12, 2024
Fixes: #207

The regexp seems redundant, convert can already handle this.

Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
macpijan added a commit that referenced this issue Jul 22, 2024
Fixes: #207

The regexp seems redundant, convert can already handle this.

Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
JanPrusinowski pushed a commit that referenced this issue Jul 24, 2024
Fixes: #207

The regexp seems redundant, convert can already handle this.

Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
wiktormowinski pushed a commit that referenced this issue Sep 2, 2024
Fixes: #207

The regexp seems redundant, convert can already handle this.

Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
wiktormowinski pushed a commit that referenced this issue Sep 23, 2024
Fixes: #207

The regexp seems redundant, convert can already handle this.

Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs feedback vault_jsl
Projects
None yet
Development

No branches or pull requests

3 participants