Skip to content

Conversation

@ljwolf
Copy link
Member

@ljwolf ljwolf commented Dec 9, 2018

this starts to move mgwr from using pickles to using dataframes. This is broken as it stands, though, since the results are different from those I get when I just run the tests.

Not sure if this is seed setting, platform dependence, or what, but the machinery is now there to read in the expected results from a csv and compare mgwr against that known csv. I just don't know how to construct the csv to pass the tests, like what seed to use or what numpy the tests expect.

@weikang9009
Copy link
Member

@ljwolf Could you let me know how you convert mgwr_example.p to georgia_mgwr_model_frame.csv? I am worried that the values in the csv file is not the same as those in mgwr_example.p once loaded?

@jGaboardi jGaboardi requested a review from weikang9009 January 5, 2019 03:07
@ljwolf
Copy link
Member Author

ljwolf commented Jan 5, 2019

@weikang9009 I just did it manually, and was also concerned that the numbers were different. I didn't retain the generation code, though, as it's in the active session on my computer in the UK.

Thanks for sorting this!

@ljwolf ljwolf merged commit f77e334 into pysal:master Jan 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants