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

CWL parser fails for numeric characters in cwl name #7

Open
erikjohnson24 opened this issue Jun 18, 2019 · 1 comment
Open

CWL parser fails for numeric characters in cwl name #7

erikjohnson24 opened this issue Jun 18, 2019 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@erikjohnson24
Copy link
Contributor

The conduit build command fails to properly parse cwl containing numerical variable names
example
is3D: int?
produces the following error related to the creation of datajoint variables

File "/conduit/utils/cwlparser.py", line 81, in init
self.job_param_db = schema(create_dj_schema(self.cwl['inputs'], self.workflow_name))()
File "/usr/local/lib/python3.6/site-packages/datajoint/schema.py", line 208, in call
self.process_relation_class(cls, context=dict(context, self=cls, **{cls.name: cls}))
File "/usr/local/lib/python3.6/site-packages/datajoint/schema.py", line 186, in process_relation_class
instance.declare(context)
File "/usr/local/lib/python3.6/site-packages/datajoint/table.py", line 62, in declare
sql, uses_external = declare(self.full_table_name, self.definition, context)
File "/usr/local/lib/python3.6/site-packages/datajoint/declare.py", line 223, in declare
name, sql, is_external = compile_attribute(line, in_key, foreign_key_sql)
File "/usr/local/lib/python3.6/site-packages/datajoint/declare.py", line 260, in compile_attribute
line=err.args[0], pos=err.args[1], msg=err.args[2]))
datajoint.errors.DataJointError: Declaration error in position 3 in line:

@erikjohnson24 erikjohnson24 added the bug Something isn't working label Jun 18, 2019
@Raphtor
Copy link
Contributor

Raphtor commented Oct 1, 2019

Ah, the problem is that it cannot have any upper case letters in it. https://docs.datajoint.io/python/definition/04-Definition-Syntax.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Conduit Development
Awaiting triage
Development

No branches or pull requests

2 participants