Skip to content

Ckan provider (WIP "kinda")#39

Open
AaronV77 wants to merge 13 commits intoerdc:masterfrom
AaronV77:ckan_provider
Open

Ckan provider (WIP "kinda")#39
AaronV77 wants to merge 13 commits intoerdc:masterfrom
AaronV77:ckan_provider

Conversation

@AaronV77
Copy link
Collaborator

This is the provider plugin for Ckan. This plugin is working correctly, but there is no Ckan live site to test the upload correctly. Until we find a Ckan site that allows us to add resources or someone gets back to me on how to upload files correctly, then I think we should wait on this merge.

@AaronV77 AaronV77 self-assigned this Apr 12, 2018
counter += 1000
list_of_datasets.extend(results['results'])

return list_of_datasets

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E271 multiple spaces after keyword


return list_of_datasets

class CKANGeoService(CKANServiceBase):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E302 expected 2 blank lines, found 1

dataset_name = param.String(default="", doc="", precedence=2)
author = param.String(default="", doc="", precedence=3)
author_email = param.String(default="", doc="", precedence=4)
availability = param.ObjectSelector(default=None, doc="", objects=[True,False], precedence=5)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E231 missing whitespace after ','

type = param.String(default="", doc="Data type", precedence=7)
dataset = param_util.DatasetListSelector(default=(), filters={'status': 'downloaded'}, precedence=8,
doc="dataset to publish to ckan")
@property

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E301 expected 1 blank line, found 0

to loop. I then grab the next group of datasets, increment my counter, and then add it to
the list of other datasets.
"""
results = self.demo.action.package_search(**kwargs, start=0, rows=1000)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E999 SyntaxError: invalid syntax

@AaronV77 AaronV77 added the enhancement New feature or request label Apr 12, 2018
@AaronV77 AaronV77 added this to the RD WorkShop milestone Apr 12, 2018
AaronV77 and others added 8 commits April 16, 2018 11:45
* Added the cuahsi coverage fix. The change was not getting all the resources and just getting all resources that had a coverage from the world.

* Made kwargs the standard over using options, params, and kwargs.

* Updated the load_providers to only update if update-cache is True or if the global is None.
… and there is not good working demo site inorder to test effectively.

providers = {name: driver.DriverManager('quest.services', name, invoke_on_load=True, invoke_kwds={'name': name}).driver for name in web_services}
if update_cache or the_providers is None:
providers = {name: driver.DriverManager('quest.services', name, invoke_on_load=True, invoke_kwds={'name': name}).driver for name in web_services}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E501 line too long (153 > 120 characters)

if len(settings.get('USER_SERVICES', [])) > 0:
for uri in settings.get('USER_SERVICES', []):
try:
drv = driver.DriverManager('quest.services', 'user', invoke_on_load=True, invoke_kwds={'uri': uri}).driver

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E501 line too long (126 > 120 characters)

@AaronV77 AaronV77 removed this from the RD WorkShop milestone May 7, 2018
@AaronV77
Copy link
Collaborator Author

So I tired working on this plugin again, but there seems to be a lot of issues still going on with the Demo Site that they have here: https://demo.ckan.org/. If anyone knows of a CKAN site that you are able to grab data and push data too let me know. I haven't been able to find one that works correctly, or one that has the rest api turned on. Maybe for now we just focus on getting data from CKAN rather than trying to store data.

@AaronV77 AaronV77 added the help wanted Extra attention is needed label May 30, 2018
@sdc50
Copy link
Member

sdc50 commented May 30, 2018

Maybe we just need to stand up our own instance of CKAN.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants