{"action":"created","issue":{"url":"https://api.github.com/repos/Pyomo/pyomo/issues/607","repository_url":"https://api.github.com/repos/Pyomo/pyomo","labels_url":"https://api.github.com/repos/Pyomo/pyomo/issues/607/labels{/name}","comments_url":"https://api.github.com/repos/Pyomo/pyomo/issues/607/comments","events_url":"https://api.github.com/repos/Pyomo/pyomo/issues/607/events","html_url":"https://github.com/Pyomo/pyomo/pull/607","id":340797970,"node_id":"MDExOlB1bGxSZXF1ZXN0MjAxMTI1NjQ4","number":607,"title":"Add pyomo.dataportal subpackage","user":{"login":"whart222","id":728056,"node_id":"MDQ6VXNlcjcyODA1Ng==","avatar_url":"https://avatars2.githubusercontent.com/u/728056?v=4","gravatar_id":"","url":"https://api.github.com/users/whart222","html_url":"https://github.com/whart222","followers_url":"https://api.github.com/users/whart222/followers","following_url":"https://api.github.com/users/whart222/following{/other_user}","gists_url":"https://api.github.com/users/whart222/gists{/gist_id}","starred_url":"https://api.github.com/users/whart222/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whart222/subscriptions","organizations_url":"https://api.github.com/users/whart222/orgs","repos_url":"https://api.github.com/users/whart222/repos","events_url":"https://api.github.com/users/whart222/events{/privacy}","received_events_url":"https://api.github.com/users/whart222/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-07-12T21:05:09Z","updated_at":"2018-07-12T23:33:19Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/Pyomo/pyomo/pulls/607","html_url":"https://github.com/Pyomo/pyomo/pull/607","diff_url":"https://github.com/Pyomo/pyomo/pull/607.diff","patch_url":"https://github.com/Pyomo/pyomo/pull/607.patch"},"body":"## Fixes NA.\r\n\r\n## Summary/Motivation:\r\nMove DataPortal logic out of pyomo.core to better modularize the Pyomo code base.\r\n\r\n## Changes proposed in this PR:\r\n- Moved code from of pyomo.core.data into pyomo.dataportal\r\n- Moved plugins in pyomo.core.plugins into pyomo.dataporta.plugins\r\n- Moved some tests in pyomo.core.tests into pyomo.dataportal.tests\r\n\r\n### Legal Acknowledgement\r\n\r\nBy contributing to this software project, I agree to the following terms and conditions for my contribution:\r\n\r\n1. I agree my contributions are submitted under the BSD license.\r\n2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.\r\n"},"comment":{"url":"https://api.github.com/repos/Pyomo/pyomo/issues/comments/404682164","html_url":"https://github.com/Pyomo/pyomo/pull/607#issuecomment-404682164","issue_url":"https://api.github.com/repos/Pyomo/pyomo/issues/607","id":404682164,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNDY4MjE2NA==","user":{"login":"codecov-io","id":8655789,"node_id":"MDQ6VXNlcjg2NTU3ODk=","avatar_url":"https://avatars0.githubusercontent.com/u/8655789?v=4","gravatar_id":"","url":"https://api.github.com/users/codecov-io","html_url":"https://github.com/codecov-io","followers_url":"https://api.github.com/users/codecov-io/followers","following_url":"https://api.github.com/users/codecov-io/following{/other_user}","gists_url":"https://api.github.com/users/codecov-io/gists{/gist_id}","starred_url":"https://api.github.com/users/codecov-io/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codecov-io/subscriptions","organizations_url":"https://api.github.com/users/codecov-io/orgs","repos_url":"https://api.github.com/users/codecov-io/repos","events_url":"https://api.github.com/users/codecov-io/events{/privacy}","received_events_url":"https://api.github.com/users/codecov-io/received_events","type":"User","site_admin":false},"created_at":"2018-07-12T23:33:19Z","updated_at":"2018-07-12T23:33:19Z","author_association":"NONE","body":"# [Codecov](https://codecov.io/gh/Pyomo/pyomo/pull/607?src=pr&el=h1) Report\n> Merging [#607](https://codecov.io/gh/Pyomo/pyomo/pull/607?src=pr&el=desc) into [master](https://codecov.io/gh/Pyomo/pyomo/commit/31062954088241f5f086bb67bfdd9ee7f04140c5?src=pr&el=desc) will **decrease** coverage by `0.1%`.\n> The diff coverage is `87.91%`.\n\n[![Impacted file tree graph](https://codecov.io/gh/Pyomo/pyomo/pull/607/graphs/tree.svg?token=p0AaEU41yH&src=pr&height=150&width=650)](https://codecov.io/gh/Pyomo/pyomo/pull/607?src=pr&el=tree)\n\n```diff\n@@            Coverage Diff             @@\n##           master     #607      +/-   ##\n==========================================\n- Coverage   66.04%   65.93%   -0.11%     \n==========================================\n  Files         374      376       +2     \n  Lines       60310    60331      +21     \n==========================================\n- Hits        39829    39779      -50     \n- Misses      20481    20552      +71\n```\n\n\n| [Impacted Files](https://codecov.io/gh/Pyomo/pyomo/pull/607?src=pr&el=tree) | Coverage Δ | |\n|---|---|---|\n| [pyomo/dataportal/parse\\_datacmds.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vZGF0YXBvcnRhbC9wYXJzZV9kYXRhY21kcy5weQ==) | `84.73% <ø> (ø)` | |\n| [pyomo/core/base/plugin.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vY29yZS9iYXNlL3BsdWdpbi5weQ==) | `77.86% <ø> (+0.93%)` | :arrow_up: |\n| [pyomo/scripting/driver\\_help.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vc2NyaXB0aW5nL2RyaXZlcl9oZWxwLnB5) | `31.69% <0%> (ø)` | :arrow_up: |\n| [pyomo/core/base/PyomoModel.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vY29yZS9iYXNlL1B5b21vTW9kZWwucHk=) | `76.55% <100%> (ø)` | :arrow_up: |\n| [pyomo/dataportal/TableData.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vZGF0YXBvcnRhbC9UYWJsZURhdGEucHk=) | `90.68% <100%> (ø)` | |\n| [pyomo/dataportal/plugins/csv\\_table.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vZGF0YXBvcnRhbC9wbHVnaW5zL2Nzdl90YWJsZS5weQ==) | `85.71% <100%> (ø)` | |\n| [pyomo/dataportal/plugins/text.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vZGF0YXBvcnRhbC9wbHVnaW5zL3RleHQucHk=) | `86.2% <100%> (ø)` | |\n| [examples/pyomo/tutorials/param.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-ZXhhbXBsZXMvcHlvbW8vdHV0b3JpYWxzL3BhcmFtLnB5) | `100% <100%> (ø)` | :arrow_up: |\n| [pyomo/dataportal/plugins/json\\_dict.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vZGF0YXBvcnRhbC9wbHVnaW5zL2pzb25fZGljdC5weQ==) | `84.5% <100%> (ø)` | |\n| [pyomo/dataportal/plugins/xml\\_table.py](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree#diff-cHlvbW8vZGF0YXBvcnRhbC9wbHVnaW5zL3htbF90YWJsZS5weQ==) | `75.36% <100%> (ø)` | |\n| ... and [19 more](https://codecov.io/gh/Pyomo/pyomo/pull/607/diff?src=pr&el=tree-more) | |\n\n------\n\n[Continue to review full report at Codecov](https://codecov.io/gh/Pyomo/pyomo/pull/607?src=pr&el=continue).\n> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)\n> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`\n> Powered by [Codecov](https://codecov.io/gh/Pyomo/pyomo/pull/607?src=pr&el=footer). Last update [3106295...b6141d6](https://codecov.io/gh/Pyomo/pyomo/pull/607?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).\n"}}
{"actor":{"display_login":"codecov-io"}}