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

`mergeStates` error #199

Closed
brodieG opened this issue Dec 17, 2016 · 2 comments
Closed

`mergeStates` error #199

brodieG opened this issue Dec 17, 2016 · 2 comments
Labels
Milestone

Comments

@brodieG
Copy link
Owner

@brodieG brodieG commented Dec 17, 2016

> library(unitizer)
> unitize_dir()

Warning: namespace ‘ggplot2’ is not available and has been replaced             
by .GlobalEnv when processing object ‘mustWork’
| Summary of files in common directory 'unitizer':                              
| 
|                  Pass Fail
| *1.  abstract.R    20    9
| *2.     alike.R   160    1
|  3.   classes.R    23    -
| *4. datatable.R     -    7
|  5.  internal.R   134    -
|  6.  language.R    45    -
|  7.      type.R    38    -
| ..........................
|                   420   17
| Legend:
| * `unitizer` requires review

| Type number of unitizer to review, 'A' to review all that require review

unitizer> 2
+------------------------------------------------------------------------------+
| unitizer for: unitizer/alike.R                                               |
+------------------------------------------------------------------------------+

                           Pass Fail
                   Atomic    12    -
                    lists    10    -
 NULL values as wildcards     5    -
          Matrix / Arrays    10    1
              Data Frames    15    -
              Time Series     6    -
                  Factors     5    -
 Environments / Pairlists    15    -
         Calls / Formulas    37    -
                Functions     3    -
                   .alike    12    -
                 Examples    30    -
....................................
                            160    1

= Matrix / Arrays ==============================================================

- Failed -----------------------------------------------------------------------

| The following test failed because the new evaluation does not match the 
| reference value from the store. Overwrite with new result ([Y]es, [N]o, 
| [P]rev, [B]rowse, [R]erun, [Q]uit, [H]elp)?

> alike(matrix(integer(), nrow = 3, dimnames = list(c("R", "G", 
+     "B"), NULL)), matrix(1:9, nrow = 3, dimnames = list(NULL, 
+     c("R", "G", "B"))))
[1] "row.names(matrix(1:9, nrow = 3, dimnames = list(NULL, c(\"R\", \n    \"G\",
\"B\"))))\nshould be type \"character\" (is \"NULL\")"

| Value mismatch: 

< .ref                                   > .new                                 
@@ 1 @@                                  @@ 1 @@                                
< [1] "  row.names(matrix(1:9, nrow = 3  > [1] "row.names(matrix(1:9, nrow = 3, 
: , dimnames = list(NULL, c(\"R\", \n    : dimnames = list(NULL, c(\"R\", \n    
:     \"G\", \"B\"))))\nshould be type   : \"G\", \"B\"))))\nshould be type \"ch
: \"character\" (is \"NULL\")"           : aracter\" (is \"NULL\")"             

| State mismatch; see `.DIFF$state` for details.

unitizer> cat(.new, "\n")
row.names(matrix(1:9, nrow = 3, dimnames = list(NULL, c("R", 
    "G", "B"))))
should be type "character" (is "NULL") 
unitizer> Q

| No changes recorded.
Error in mergeStates(items.ref, x@state.new, x@state.ref) : 
  Logic Error: global index mapping matrix malformed; contact maintainer.
| Unexpectedly exited before storing unitizer; if you were reviewing a unitizer 
| changes to that unitizer were not saved.  Note that any unitizers you *
| completed* review of should have been saved.

> packageVersion('unitizer')
[1] ‘1.3.0.9000’
@brodieG brodieG added the bug label Dec 17, 2016
@brodieG brodieG added this to the 1.4.0 milestone Dec 17, 2016
@brodieG
Copy link
Owner Author

@brodieG brodieG commented Feb 24, 2017

See also the now closed issue #212 (closed because duplicate of this one)

@brodieG
Copy link
Owner Author

@brodieG brodieG commented Feb 25, 2017

Currently can't reproduce this problem. Tried using the alike data.table tests by adding a call to Sys.time() to force a failure in case having a failure was a requisite condition, but no dice. Tried both with unitize and unitize_dir.

@brodieG brodieG closed this in 9d89738 Apr 11, 2017
brodieG added a commit that referenced this issue Apr 11, 2017
Fix #215, Fix #214, Fix #213, Fix #208, Fix #203,
Fix #201, Fix #199, Fix #197
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.