Activity for Dave Kuhlman
-
Dave Kuhlman committed [7d5296] on Code
v. 2.44.1 black formatting; fix regexprs for Py 3.12
-
I don't think this can be fixed. LibreOffice Writer does not seem to support the ability to create a second footnote reference that points at the same (an existing) footnote body. At least, I have not figured out how the create a second reference to an existing footnote body in LibreOffice Writer. If someone knows how to do that, please give me a hint. Dave
-
v. 2.43.3 Added warning union inside simpleType and type_substitution unit test
-
v. 2.43.2 Several fixes for GraphQL support
-
v.. 2.43.1 Added split/join modules capability
-
v. 2.42.2 Added graphql unit test. Added graphql demo.
-
I have added support for the generation of modules that implement a GraphQL server that can be run with the Strawberry GraphQL Python package (https://strawberry.rocks/docs). You can learn about it here -- http://www.davekuhlman.org/generateDS.html#graphql-support There are likely bugs in this, but I did generate and run it with two reasonably complex examples: (1) the Demos/People schema and XML instance doc in the generateDS repository and (2) the neuroml schema (https://github.com/NeuroML/NeuroML2.git...
-
I have added support for the generation of modules that implement a GraphQL server that can be run with the Strawberry GraphQL Python package (https://strawberry.rocks/docs). You can learn about it here -- http://www.davekuhlman.org/generateDS.html#graphql-support There are likely bugs in this, but I did generate and run it with two reasonably complex examples: (1) the Demos/People schema and XML instance doc in the generateDS repository and (2) the neuroml schema (https://github.com/NeuroML/NeuroML2.git...
-
v. 2.42.1 Added support for Strawberry and GraphQL
-
v. 2.41.5 Changed name of generated method "_hasContent" --> "has__content"
-
v. 2.41.4 Fix to formatting of floats.
-
Thank you for your report. However, I do not know what to do about this. You may be right in suggesting that the behavior should be different. Do you want to submit a patch? If so I'll consider it. But, I do not want to make a change that I do not understand. Maybe you could explain in detail how you think it should function. Or maybe tell me how the current behavior is a problem for you. Is there some behavior that you need and cannot get in main.py? I'll welcome suggestions.
-
v. 2.41.3 Recognize mixed=true on complexContent
-
v. 2.41.2 Fix to export of mixed and simpleContent
-
v. 2.41.1 Added command line option --use-regex-module
-
Mikhail, Thanks for your detailed report. Yes, this seems like a reasonable request to me. But ... In response to your ticket and your questions, I have searched a bit at the code in generateDS.py looking for support for namespaces on attributes. I cannot find any. And, adding/implementing that support for namespace prefixes on attribute names would require significant changes at a number of locations in the code. That is not a task that I'm willing to take on. In addition, I'm a bit confused about...
-
One more question: What command line flags did you use when you ran generateDS.py (or generateds)? What were the commands that you ran for your test? Perhaps you could copy and paste them in a reply. In general, I'm groping for any information that would enable me to reproduce the error. Dave
-
Pablo, I'm trying to reproduce the exception that you've reported. Here's what I've done: I copied people.xsd and people.xml from generateds/tests in the source code distribution. I generated a module with this: $ ./generateDS.py -o people.py people.xsd I ran that module against people.xml: $ python people.py people.xml The above ran successfully and produced output. There are also people.xsd and people.xml in the generateds/Demos/People subdirectory. In order to run a test with those, you need to...
-
v. 2.40.13 Fix for default value and/or nillable
-
Great, That patch, including your refinement, has now been uploaded as v. 2.40.12. Thanks again. Dave
-
Great, That patch has now been uploaded as v. 2.40.12. Thanks again. Dave
-
v. 2.40.12 Strip white space boolean values and backslash-n to
-
Pete, Thanks for reporting this. Attached is a patch -- actually, it's just the change that you suggest above. If it looks good to you, let me know, and it will be in the next release. Dave
-
OK. So, attached is another patch. With this patch, generateDS.py produces the following line in the generated module: s1 = s1.replace('\n', ' ') I'm actually unsure about what it is supposed to do. So, I'll wait for you to tell me that it works and solves your problem before I upload another release. Thanks for your very helpful and explicit description of a solution. Hope I got it right this time. Dave
-
Concerning your question about what is the difference between the following two lines? s1 = s1.replace(r'\\n', ' ') s1 = s1.replace('\n', ' ') Try running the following: a = r'\\n' b = '\n' print(len(a)) print(len(b)) print(a[0]) print(a[1]) print(a[2]) print(ord(b)) Thanks for the link to the StackOverflow page. Yes, that is informative and is a good explanation. So, did that latest version (v. 2.40.11) fix things for you? Dave
-
Gabriel, I've created a new release: v. 2.40.11. It has been uploaded to sourceforge.net and to pypi.org. Thanks for your help with this. Dave
-
v. 2.40.11 Added transform of backslash-n to
-
David, Thanks for the report and the link to the schemas. I've looked into this a bit. I don't think I'm going to be able to fix it. It looks like fixes would be needed at multiple places in the code. There are several other places in the code where similar recursion-limit-exceeded occurs. And, I don't even know the correct way to fix any of those places. I can patch those to skip the recursion error, but then the module that we generate is incomplete. If you have a suggestion or a patch, please...
-
Gabriel, Thanks for the note. Your suggestion to transformed "\n" to " " sounds good to me. Although, I actually have no idea what the correct policy should be on this. If you have a rational, I'd be interested. I added a line to generateDS.py that replaces "\n" with " ". Hope I got it right. Attached is a patch. Please let me know whether this is what you intended and whether it works for you. If it works, I'll create a new release. Thanks for help. Dave
-
v. 2.40.10 Force conversion to string for anytypeobjs
-
v. 2.40.9 Fix to method gds_validate_simple_patterns
-
v. 2.40.9 Fix to method gds_validate_simple_patterns and update README
-
v. 2.40.8 Fix to generated __str__ method
-
v. 2.40.7 Fixes for -p (class name prefix) command line option
-
Ankur, Your fix looks good to me. I've applied that patch, did some testing, created a new version, and uploaded it to PyPI and SourceForge.net. Thanks for your help with this. Please let me know if/when there is another issue. Dave
-
v. 2.40.6 Fix for Python 2.7 and StringIO
-
OK, I've found a number of examples (*.nml) here: https://github.com/NeuroML/NeuroML2/tree/master/examples. But, the ones I have tried so far do not cause an exception, either with Python 3.9.7 nor with Python 2.7.18. The ones I've tried are: NML2_AnalogSynapsesHH.nml NML2_FullNeuroML.nml NML2_PyNNCells.nml For example, it displays exported output when I run this: $ python nml.py NML2_FullNeuroML.nml By the way, I'm running nml.py with Python 2.7.18, but I generated nml.py with Python 3.9.7. Dav...
-
Ankur, Thanks for your help with this. What I've done so far: Downloaded NeuroML_v2.2.xsd, helper_methods.py, and regenerate-nml.sh. Modified regenerate-nml.sh for my environment: path to generateDS.py, name of XML schema file. Ran regenerate-nml.sh -- generated nml.py. Build a virtual environment with Python 2.7.18, then installed lxml, and six required by generateDS. Next -- I need to run the new nml.py against an XML instance document so that it generates the reported exception. I looked in the...
-
f'' strings break pre-3.6 Python releases
-
Closed
-
Assertion Error while create the DS
-
Closed
-
Samantha, Thank you for reporting this. I was able to reproduce this error using the POCD_MT000040.xsd schema that I found at https://github.com/srdc/ontmalizer. I'm guessing that you were using the same. Here are notes from the README.rst: Version 2.40.5 (10/19/2021) - Fix to test for targetNamespace agreement between including and included schemas. Thanks to Samantha R. for this report. See ticket #27 "Assertion Error while create the DS" at SourceForge. Comment: The assertion exception was caused...
-
v. 2.40.5 Fix to test for targetNamespace agreement
-
Mustafa, Thanks for this fix. I've merged it. I have uploaded it and one more change as a new version: V. 2.40..4. By the way, I believe that your fix is a good one. Still, you might be interested to look at the tempfile module. It might make this kind of task easier in the future. See -- https://docs.python.org/3/library/tempfile.html Thanks again. Dave
-
Removed formatted string literals, not supported until Py 3.6
-
v. 2.40.4 Formatted string literals and base classes not found
-
Fix base classes not found
-
Sam, Thanks for the report on this. I did not realize that it was not supported by earlier Python 3 versions. I've made your patches and also a couple more that I found by doing a search. I've attached a diff of my changes. Let me know if you spot something wrong with them. I will create a new version containing these changes soon. Dave
-
Help with tag that can appear anywhere
-
XS:Imports Namespace lost
-
annotations on enumerations
-
Using `--use-getter-setter=none` does not add `add_anytypeobjs_`, but this method is still used in places in the generated python
-
Alexey, Trying to diagnose this ... I used this XML schema: <?xml version="1.0" encoding="utf-8"?> <xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema" > <xs:element name="container" type="containerType"/> <xs:complexType name="containerType"> <xs:sequence> <xs:element name="time01" type="ID_String"/> </xs:sequence> </xs:complexType> <xs:simpleType name="ID_String"> <xs:restriction base="xs:string"> <xs:pattern value="[A-Z0-9-]{16}-(ELXP|NGET)-RMT-[0-9]{8}"/> <xs:maxLength value="34" /> </xs:restriction>...
-
Alexey, Trying to diagnose this ... I used this XML schema: <?xml version="1.0" encoding="utf-8"?> <xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema" > <xs:element name="container" type="containerType"/> <xs:complexType name="containerType"> <xs:sequence> <xs:element name="time01" type="ID_String"/> </xs:sequence> </xs:complexType> <xs:simpleType name="ID_String"> <xs:restriction base="xs:string"> <xs:pattern value="[A-Z0-9-]{16}-(ELXP|NGET)-RMT-[0-9]{8}"/> <xs:maxLength value="34" /> </xs:restriction>...
-
Alexey, Trying to diagnose this ... I used this XML schema: <?xml version="1.0" encoding="utf-8"?> <xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema" > <xs:element name="container" type="containerType"/> <xs:complexType name="containerType"> <xs:sequence> <xs:element name="time01" type="ID_String"/> </xs:sequence> </xs:complexType> <xs:simpleType name="ID_String"> <xs:restriction base="xs:string"> <xs:pattern value="[A-Z0-9-]{16}-(ELXP|NGET)-RMT-[0-9]{8}"/> <xs:maxLength value="34" /> </xs:restriction>...
-
Mustafa, I introduced a bug. I got the indentation of __str__ wrong. It is fixed in version 2.40.3. I apologize. Hope I got it right this time. I'll look into your suggestion for the improved __str__ method, next week. Dave
-
v. 2.40.3 Another fix to __str__
-
Mustafa, Uploaded the fix for __str__ and export with --disable-xml. It's version 2.40.2. Thanks for the help with this. Dave
-
v. 2.40.2 Fix to __str__ used with --disable-xml
-
Mustafa, Oops. OK, so I've "disabled" generation of that __str__ method. Actually, it's generated, but is commented out. That method is intended as a convenience for a programmer who wants to display an object while debugging. Am I right that you can do without it? With the new code and when using --disable-xml, when you ask for a string representation, you will see: [ins] In [21]: import tmp04sup as lib4 [ins] In [22]: b = lib4.containerType() [ins] In [23]: print(str(b)) <tmp04sup.containerType...
-
Mustafa, I've merged your changes, created a new release, and pushed/uploaded it as v. 2.40.1. Thank you much for your work on this. I appreciate it, and, hopefully, other users will, too.
-
Added utils/xml_to_json01.py
-
v. 2.40.1 Merged PR from Mustafa Şenol Coşar for episodic compilation
-
Basic episodic compilation
-
Thinking about this a bit more. You may not have found this, because it's buried quite deep in the code. It may interest you to take a look at function create_mapping in file generateds/gds_collect_namespace_mappings.py. It's called from function parseAndGenerate in file generateds/generateDS.py. It does not do what you are trying to do, as far as I understand it. But, it might be of interest. It might provide ideas about how to solve your namespace-to-module mapping problem. Dave
-
I agree. This is likely an area and feature that need work. When you have something that you believe is working and that I can look at, please send me a pull request or a diff, whichever is easiest.
-
Ankur, I tried this with NeuroML_v2.2.xsd. See comments from generated module, below. I cannot find any references to add_anytypeobjs_. This was fixed in generateds v. 2.39.10, I believe. Let me know if generateds is still causing this problem. Dave 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25#!/usr/bin/env python # -*- coding: utf-8 -*- # # Generated Wed Sep 15 11:06:05 2021 by generateDS.py version 2.39.11. # Python 3.9.5 (default, May 11 2021, 08:20:37) [GCC 10.3.0] # # Command...
-
v. 2.39.11 Added __str__ and utils/post_process.py
-
v. 2.39.10 Fixed add_anytypeobjs_, cast in gds_format_integer, gds_format_base64
-
I've made a fix to this. This patch will prevent the generation of the call to add_anytypeobjs_. diff -r 29824eaf0796 generateDS.py --- a/generateDS.py Wed Aug 25 14:51:37 2021 -0700 +++ b/generateDS.py Fri Aug 27 16:22:15 2021 -0700 @@ -4838,7 +4838,7 @@ wrt(" else:\n") wrt(" %scontent_ = self.gds_build_any(child_, '%s')\n" % ( fill, type_name, )) - wrt(' %sself.add_anytypeobjs_(content_)\n' % (fill, )) + wrt(' %sself.anytypeobjs_.append(content_)\n' % (fill, )) else: if keyword == 'if': fill =...
-
John, I've modified these links to the following: https://sourceforge.net/p/generateds/mailman/generateds-discuss/ This change and the change that you reported for attribute groups plus one additional change (added a unit test) have been uploaded as v. 2.39.9. Thank you for reporting this. Dave
-
v. 2.39.9 New unit test for recursive generator and fixes to generateDS.txt
-
John, Thanks for your report. With respect to the attributeGroup, I've modified example so that it looks like the following: ``generateDS.py`` now handles definition and use of attribute groups. For example: the use of something like the following:: <xs:attributeGroup name="favorites"> <xs:attribute name="fruit" /> <xs:attribute name="vegetable" /> </xs:attributeGroup> And, a reference or use like the following:: <xs:element name="person" type="personType"/> <xs:complexType name="personType" mixed="0">...
-
v. 2.39.8 Applied patch for generating recursive generator method
-
v. 2.39.7 Fix to generation of MemberSpec_ for attribute names
-
v. 2.39.6 Additional patches for reverse_mapping in to_etree
-
Opps. I forgot to copy my temporary changes to the repo. That was dumb of me to do it this way. Now the changes are there.: v. 2.39.5 And, they've been uploaded. Dave
-
Copied changes in django_etl into the repo.
-
v. 2.39.5 Copied changes in django_etl into the repo.
-
Mark, I've removed the try/except from around the import of lxml. I agree with you. That was a trap waiting to bite someone. I apologize that it was you. I worry that there might be someone out there who has distributed a module generated by gDS and who does not want to require all their users to install lxml. But, requiring lxml makes their code safer and they should do it for their own good and their users good, and tough love, etc. The new version (v. 2.39.4) is at SourceForge and PyPI. Thanks...
-
Drew, The exception had a different cause than I expected. Anyway, I fixed that error, I believe. And, I've also updated the boilerplate in django_etl/generatedssuper.py. Now I can generate models.py, admin.py, and forms.py. If you get a chance to take a look, please let me know if it now does what you expect it to do. The new version (v. 2.39.4) is at SourceForge and PyPI. Thanks for help. Dave
-
Mark, I've removed the try/except from around the import of lxml. I agree with you. That was a trap waiting to bite someone. I apologize that it was you. I worry that there might be someone out there who has distributed a module generated by gDS and who does not want to require all their users to install lxml. But, requiring lxml makes their code safer and they should do it for their own good and their users good, and tough love, etc. Thanks for the nudge on this. Dave
-
v. 2.39.4 Always import lxml. Fix in django_etl. Reverse mapping in to_etree.
-
Drew, Thanks for the report. I've been able to reproduce the exception. I believe that the superclass used by the Django support can get out of sync with the latest version generated by gDS. I don't normally try to keep the file django_etl/gends_generate_django.py up-to-date. Anyway, that's the first thing I'll check. By the way, comparing the lines and line numbers in the traceback, it seems that you are using the files in ./django_etl/ and not the ones under ./django/. Please let me know if I'm...
-
v. 2.39.3 Generate code to convert DataTime to str.
-
Mark, Making Lxml a requirement sounds like a good idea to me. In generateds/setup.py, I already find the following, but I can't find any documentation on what it does. install_requires=[ "six", "lxml", "requests>=2.21.0" ], Is that what you were suggesting? Or is there something more that I should do? Dave
-
v. 2.39.2 Renamed generated methods that are non-public (leading underscore)
-
Robert, I've done some work on this. The latest version of generateDS now creates doc strings from the annotation/documentation on a complexType. When there is documentation on a child element or an attribute in the complexType, generateDS attempts to attach that to the class formatted as reStructuredText unordered lists (so that they can be formatted by Sphinx. I suspect that there are still some cases that it does not handle, for example attributeGroup and model group (reusable groups of child...
-
v. 2.39.1 Fixes and formatting of doc strings. generatedssuper
-
Robert, Thanks for this suggestion. I'm starting work on another possible enhancement that also concerns generation of doc strings. I'll look into support for attributes at the same time. Dave
-
Thanks for advice. I've uploaded the wheel for the latest version (v. 2.38.6). I won't try to delete any egg files at PyPI (don't know if that can be done). But I've remove the egg file from the script that I use to do the upload to PyPI (generateds/upload_pypi). So, the egg will not be at PyPI for future versions. If someone complains, I can always put it back. Thanks again for the suggestion. And, please let me know whether the wheel (from PyPI) does what you expect it to.
-
John, Thank you for this suggestion and your guidance. OK. I've added it to my build script and have built a *.whl file. I have not yet uploaded it to PyPI. Am I supposed to upload both the wheel and the egg files? Or, are egg files no longer in use? Is there any downside to uploading both wheel and egg files o PyPI? I'm curious -- Given the way we currently have it (an egg but not a wheel at PyPI), does PyPI automatically convert the egg to a wheel when I install generateds using pip? Thanks in...
-
v. 2.38.6 Added validation of some simpleType lists
-
v. 2.38.5 Code formating and cleanup (for flake8).
-
v. 2.38.4 Fixes for (no suitable) namespace prefix warning
-
Omar, I agree with you that it would be good if gDS generated some support for validating lists of xs:simpleType. But, I wanted to get something to you that enabled you to use it. OK, yes I was lazy, too. So, maybe that validation is an enhancement that I can look into sometime. In the meantime, I suppose you will have to do your own checking when setting the value programmatically. I'll try to set aside some time for this validation. But, I've got to clear up a few other things first. Thank you...
-
Omar, I've implemented a fix. Now, gDS successfully determines that it's a list type even when it's the xs:simpleType that is being restricted where it's define as a list. I've also implemented this fix for other types in addition to the xs:double in your case. I've also added a unit test that tries to tests these cases. I've uploaded a new version (v. 2.38.3) containing these fixes to sourceforge.net and PyPI. When you try this, could you please let me know whether it fixes things for you. And,...
1 >
