refactor: perform dimensional analysis and numerical evaluation in same pass (WIP) #1556
mgremingerAnnotations
10 errors, 1 warning, and 1 notice
|
[chromium] › test_custom_base_units.spec.mjs:187:1 › Test custom base units with code generation:
tests/test_custom_base_units.spec.mjs#L205
1) [chromium] › test_custom_base_units.spec.mjs:187:1 › Test custom base units with code generation
Error: expect(received).toBe(expected) // Object.is equality
- Expected - 4
+ Received + 4
@@ -1,8 +1,8 @@
- def y(x):
- """
- Function 'y' automatically generated by EngineeringPaper.xyz
+ def _function_id_wrapper(172,y)(x):
+ """
+ Function '_function_id_wrapper(172,y)' automatically generated by EngineeringPaper.xyz
Parameters
----------
x : float
'x' has units of [m].
@@ -11,9 +11,9 @@
-------
float
Return value has units of [km].
"""
- result = x
+ result = y
return result / 1000
↵
203 | let content = await page.locator('code').textContent();
204 |
> 205 | expect(content).toBe(`def y(x):
| ^
206 | """
207 | Function 'y' automatically generated by EngineeringPaper.xyz
208 |
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_custom_base_units.spec.mjs:205:19
|
|
[chromium] › test_custom_base_units.spec.mjs:187:1 › Test custom base units with code generation:
tests/test_custom_base_units.spec.mjs#L205
1) [chromium] › test_custom_base_units.spec.mjs:187:1 › Test custom base units with code generation
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toBe(expected) // Object.is equality
- Expected - 3
+ Received + 3
@@ -1,8 +1,8 @@
- def y(x):
+ def _function_id_wrapper(7,y)(x):
"""
- Function 'y' automatically generated by EngineeringPaper.xyz
+ Function '_function_id_wrapper(7,y)' automatically generated by EngineeringPaper.xyz
Parameters
----------
x : float
'x' has units of [m].
@@ -11,9 +11,9 @@
-------
float
Return value has units of [km].
"""
- result = x
+ result = y
return result / 1000
↵
203 | let content = await page.locator('code').textContent();
204 |
> 205 | expect(content).toBe(`def y(x):
| ^
206 | """
207 | Function 'y' automatically generated by EngineeringPaper.xyz
208 |
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_custom_base_units.spec.mjs:205:19
|
|
[chromium] › test_custom_base_units.spec.mjs:187:1 › Test custom base units with code generation:
tests/test_custom_base_units.spec.mjs#L205
1) [chromium] › test_custom_base_units.spec.mjs:187:1 › Test custom base units with code generation
Retry #2 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toBe(expected) // Object.is equality
- Expected - 3
+ Received + 3
@@ -1,8 +1,8 @@
- def y(x):
+ def _function_id_wrapper(7,y)(x):
"""
- Function 'y' automatically generated by EngineeringPaper.xyz
+ Function '_function_id_wrapper(7,y)' automatically generated by EngineeringPaper.xyz
Parameters
----------
x : float
'x' has units of [m].
@@ -11,9 +11,9 @@
-------
float
Return value has units of [km].
"""
- result = x
+ result = y
return result / 1000
↵
203 | let content = await page.locator('code').textContent();
204 |
> 205 | expect(content).toBe(`def y(x):
| ^
206 | """
207 | Function 'y' automatically generated by EngineeringPaper.xyz
208 |
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_custom_base_units.spec.mjs:205:19
|
|
[chromium] › test_custom_base_units.spec.mjs:228:1 › Test cell units supersede with code generation:
tests/test_custom_base_units.spec.mjs#L246
2) [chromium] › test_custom_base_units.spec.mjs:228:1 › Test cell units supersede with code generation
Error: expect(received).toBe(expected) // Object.is equality
- Expected - 3
+ Received + 3
@@ -1,8 +1,8 @@
- def y(x):
+ def _function_id_wrapper(7,y)(x):
"""
- Function 'y' automatically generated by EngineeringPaper.xyz
+ Function '_function_id_wrapper(7,y)' automatically generated by EngineeringPaper.xyz
Parameters
----------
x : float
'x' has units of [m].
@@ -11,9 +11,9 @@
-------
float
Return value has units of [mm].
"""
- result = x
+ result = y
return result / 0.001
↵
244 | let content = await page.locator('code').textContent();
245 |
> 246 | expect(content).toBe(`def y(x):
| ^
247 | """
248 | Function 'y' automatically generated by EngineeringPaper.xyz
249 |
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_custom_base_units.spec.mjs:246:19
|
|
[chromium] › test_custom_base_units.spec.mjs:228:1 › Test cell units supersede with code generation:
tests/test_custom_base_units.spec.mjs#L246
2) [chromium] › test_custom_base_units.spec.mjs:228:1 › Test cell units supersede with code generation
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toBe(expected) // Object.is equality
- Expected - 3
+ Received + 3
@@ -1,8 +1,8 @@
- def y(x):
+ def _function_id_wrapper(7,y)(x):
"""
- Function 'y' automatically generated by EngineeringPaper.xyz
+ Function '_function_id_wrapper(7,y)' automatically generated by EngineeringPaper.xyz
Parameters
----------
x : float
'x' has units of [m].
@@ -11,9 +11,9 @@
-------
float
Return value has units of [mm].
"""
- result = x
+ result = y
return result / 0.001
↵
244 | let content = await page.locator('code').textContent();
245 |
> 246 | expect(content).toBe(`def y(x):
| ^
247 | """
248 | Function 'y' automatically generated by EngineeringPaper.xyz
249 |
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_custom_base_units.spec.mjs:246:19
|
|
[chromium] › test_custom_base_units.spec.mjs:228:1 › Test cell units supersede with code generation:
tests/test_custom_base_units.spec.mjs#L246
2) [chromium] › test_custom_base_units.spec.mjs:228:1 › Test cell units supersede with code generation
Retry #2 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toBe(expected) // Object.is equality
- Expected - 3
+ Received + 3
@@ -1,8 +1,8 @@
- def y(x):
+ def _function_id_wrapper(7,y)(x):
"""
- Function 'y' automatically generated by EngineeringPaper.xyz
+ Function '_function_id_wrapper(7,y)' automatically generated by EngineeringPaper.xyz
Parameters
----------
x : float
'x' has units of [m].
@@ -11,9 +11,9 @@
-------
float
Return value has units of [mm].
"""
- result = x
+ result = y
return result / 0.001
↵
244 | let content = await page.locator('code').textContent();
245 |
> 246 | expect(content).toBe(`def y(x):
| ^
247 | """
248 | Function 'y' automatically generated by EngineeringPaper.xyz
249 |
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_custom_base_units.spec.mjs:246:19
|
|
[chromium] › test_database.spec.mjs:176:1 › Test database consistency:
tests/test_database.spec.mjs#L194
3) [chromium] › test_database.spec.mjs:176:1 › Test database consistency ─────────────────────────
Error: expect(received).toEqual(expected) // deep equality
Expected: 0
Received: 5139
192 | await page.screenshot({ path: `${screenshotDir}/${browserName}_screenshot_reference_check.png`, fullPage: true });
193 |
> 194 | expect(compareImages(`${browserName}_reference.png`, `${browserName}_screenshot_reference_check.png`)).toEqual(0);
| ^
195 | });
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_database.spec.mjs:194:106
|
|
[chromium] › test_database.spec.mjs:176:1 › Test database consistency:
tests/test_database.spec.mjs#L194
3) [chromium] › test_database.spec.mjs:176:1 › Test database consistency ─────────────────────────
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toEqual(expected) // deep equality
Expected: 0
Received: 5139
192 | await page.screenshot({ path: `${screenshotDir}/${browserName}_screenshot_reference_check.png`, fullPage: true });
193 |
> 194 | expect(compareImages(`${browserName}_reference.png`, `${browserName}_screenshot_reference_check.png`)).toEqual(0);
| ^
195 | });
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_database.spec.mjs:194:106
|
|
[chromium] › test_database.spec.mjs:176:1 › Test database consistency:
tests/test_database.spec.mjs#L186
3) [chromium] › test_database.spec.mjs:176:1 › Test database consistency ─────────────────────────
Retry #2 ───────────────────────────────────────────────────────────────────────────────────────
TimeoutError: locator.waitFor: Timeout 5000ms exceeded.
Call log:
- waiting for locator('h3').locator('text=Retrieving Sheet') to be detached
184 | await page.setViewportSize({ width: width, height: height });
185 |
> 186 | await page.locator('h3 >> text=Retrieving Sheet').waitFor({state: 'detached', timeout: 5000});
| ^
187 | await page.locator('text=Accept').click();
188 | await page.waitForSelector('.status-footer', { state: 'detached', timeout: pyodideLoadTimeout });
189 | await page.keyboard.press('Escape'); // unselect all cells
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_database.spec.mjs:186:53
|
|
[chromium] › test_generated_code.spec.mjs:15:1 › Unitless function:
tests/test_generated_code.spec.mjs#L25
4) [chromium] › test_generated_code.spec.mjs:15:1 › Unitless function ────────────────────────────
Error: expect(received).toBe(expected) // Object.is equality
- Expected - 3
+ Received + 3
@@ -1,8 +1,8 @@
- def y(x):
+ def _function_id_wrapper(7,y)(x):
"""
- Function 'y' automatically generated by EngineeringPaper.xyz
+ Function '_function_id_wrapper(7,y)' automatically generated by EngineeringPaper.xyz
Parameters
----------
x : float
'x' is unitless.
@@ -11,9 +11,9 @@
-------
float
Return value is unitless.
"""
- result = x
+ result = y
return result
↵
23 | let content = await page.locator('code').textContent();
24 |
> 25 | expect(content).toBe(`def y(x):
| ^
26 | """
27 | Function 'y' automatically generated by EngineeringPaper.xyz
28 |
at /home/runner/work/EngineeringPaper.xyz/EngineeringPaper.xyz/tests/test_generated_code.spec.mjs:25:19
|
|
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "screenshots", "test-results".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
|
|
🎭 Playwright Run Summary
32 failed
[chromium] › test_custom_base_units.spec.mjs:187:1 › Test custom base units with code generation
[chromium] › test_custom_base_units.spec.mjs:228:1 › Test cell units supersede with code generation
[chromium] › test_database.spec.mjs:176:1 › Test database consistency ──────────────────────────
[chromium] › test_generated_code.spec.mjs:15:1 › Unitless function ─────────────────────────────
[chromium] › test_generated_code.spec.mjs:48:1 › Function with input and output units ──────────
[chromium] › test_generated_code.spec.mjs:83:1 › Function with temperature input and output ────
[chromium] › test_generated_code.spec.mjs:118:1 › Function with temperature input and output, offset only
[chromium] › test_generated_code.spec.mjs:153:1 › Funciton with two inputs ─────────────────────
[chromium] › test_generated_code.spec.mjs:188:1 › Function with unitless matrix output ─────────
[chromium] › test_generated_code.spec.mjs:229:1 › Function with matrix output with units ───────
[chromium] › test_generated_code.spec.mjs:275:1 › Function with matrix output with units and user units
[chromium] › test_generated_code.spec.mjs:351:1 › Numpy function rewrite fix, sympy #25514 ─────
[chromium] › test_generated_code.spec.mjs:386:1 › Variable name rewriting ──────────────────────
[chromium] › test_number_format.spec.mjs:551:1 › Test intermediate results rendering edge cases
[firefox] › test_custom_base_units.spec.mjs:187:1 › Test custom base units with code generation
[firefox] › test_custom_base_units.spec.mjs:228:1 › Test cell units supersede with code generation
[firefox] › test_database.spec.mjs:176:1 › Test database consistency ───────────────────────────
[firefox] › test_generated_code.spec.mjs:15:1 › Unitless function ──────────────────────────────
[firefox] › test_generated_code.spec.mjs:48:1 › Function with input and output units ───────────
[firefox] › test_generated_code.spec.mjs:83:1 › Function with temperature input and output ─────
[firefox] › test_generated_code.spec.mjs:118:1 › Function with temperature input and output, offset only
[firefox] › test_generated_code.spec.mjs:153:1 › Funciton with two inputs ──────────────────────
[firefox] › test_generated_code.spec.mjs:188:1 › Function with unitless matrix output ──────────
[firefox] › test_generated_code.spec.mjs:229:1 › Function with matrix output with units ────────
[firefox] › test_generated_code.spec.mjs:275:1 › Function with matrix output with units and user units
[firefox] › test_generated_code.spec.mjs:351:1 › Numpy function rewrite fix, sympy #25514 ──────
[firefox] › test_generated_code.spec.mjs:386:1 › Variable name rewriting ───────────────────────
[firefox] › test_number_format.spec.mjs:551:1 › Test intermediate results rendering edge cases ─
[firefox] › test_plotting.spec.mjs:354:1 › Test copy plot data ─────────────────────────────────
[firefox] › test_plotting.spec.mjs:639:1 › Test visual comparison of function plot with identical scatter line plot
[firefox] › test_plotting.spec.mjs:764:1 › Test visual comparison of function plot with identical parametric plot
[firefox] › test_symbolic_expression_error_handling.spec.mjs:5:1 › Test handling of symbolic expression error
6 flaky
[chromium] › test_plotting.spec.mjs:764:1 › Test visual comparison of function plot with identical parametric plot
[firefox] › test_file_save_open.spec.mjs:138:1 › Test opening file with results and syntax error
[firefox] › test_matrix_inverse.spec.mjs:53:1 › Matrix exponent inverse with variable matrix ───
[firefox] › test_number_format.spec.mjs:92:1 › Test disabling automatic expressions simplification
[firefox] › test_plotting.spec.mjs:601:1 › test scatter plot y has inconsistent units ──────────
[firefox] › test_system_solve.spec.mjs:382:1 › Test restarting pyodide on a calculation that has caused sympy to hang
9 skipped
731 passed (33.2m)
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
screenshots
Expired
|
16 MB |
|
|
test-results
Expired
|
830 MB |
|