objectionary / speco

Object Specialization for EO Programs
MIT License
6 stars 4 forks source link

fix bug with tuples #102

Closed advasileva closed 1 year ago

advasileva commented 1 year ago

Fixes bug, fixes old tests, add a new test with "fake int counter" (afaik AOI cannot work with nested objects or atoms now, so I add a wrapper for int to test replacement with specialized version based on type). Closes #101.

codecov[bot] commented 1 year ago

Codecov Report

Merging #102 (79f91a9) into master (58ba62b) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #102   +/-   ##
=========================================
  Coverage     89.36%   89.36%           
  Complexity        8        8           
=========================================
  Files             2        2           
  Lines            47       47           
  Branches          3        3           
=========================================
  Hits             42       42           
  Misses            5        5           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

advasileva commented 1 year ago

@yegor256 This PR is ready for review

yegor256 commented 1 year ago

@rultor merge

rultor commented 1 year ago

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 1 year ago

@rultor merge

@advasileva @yegor256 Oops, I failed. You can see the full log here (spent 2hr)

Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.547 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[1]/o[1]
Matches both "node()" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "node()" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.547 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[1]/o[1]/@base
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[1]/o[1]/@line
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[1]/o[1]/@pos
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[2]
Matches both "node()" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "node()" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[2]/@base
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[2]/@data
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[2]/@line
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[2]/@pos
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[1]/o[2]/o[2]/text()[1]
Matches both "node()" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "node()" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.548 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[2]
Matches both "node()" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "node()" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.549 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[2]/@base
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.549 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[2]/@line
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.549 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[2]/@pos
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.549 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[2]/o[1]
Matches both "node()" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "node()" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.549 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[2]/o[1]/@base
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.549 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[2]/o[1]/@line
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.549 [main] WARN  com.jcabi.xml.ConsoleErrorListener MDC= - #warning(): Ambiguous rule match for /program/speco[1]/version[3]/o[1]/o[3]/o[1]/o[2]/o[1]/@pos
Matches both "@*" on line 109 of file:///org/eolang/speco/1-2-specialization.xsl
and "@*" on line 96 of file:///org/eolang/speco/1-2-specialization.xsl
11:51:33.863 [main] DEBUG com.yegor256.xsline.Xsline MDC= - Transformed XML through 9 shift(s) in 463ms
11:51:34.117 [main] DEBUG com.yegor256.xsline.Xsline MDC= - Transformed XML through 1 shift(s) in 247ms
0073002c19f9549d866b9d379d1913038ecfbc05c693b9e727256a5d5f0e6c5b
'pid' file is absent on the server after the end of operation; it seems that we didn't manage to start Docker container correctly
yegor256 commented 1 year ago

@rultor merge

rultor commented 1 year ago

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 1 year ago

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 15min)

advasileva commented 1 year ago

@yegor256 Repo become more elegant, thank you!