psidev-pi-dev

[Psidev-pi-dev] [HUPO-PSI/mzIdentML] Fix error that occurs during cs class generation (#15)

[stefanks <not...@gi...>]

This error and solution:
http://stackoverflow.com/questions/6678934/unable-to-generate-a-temporary-class-result-1-error-cs0030-cannot-convert-ty
https://developer.salesforce.com/forums/?id=906F0000000AiPEIA0
http://stackoverflow.com/questions/5595779/xmlserializer-invalidoperationexc-known-issue-converting-types
You can view, comment on, or merge this pull request online at:

  https://github.com/HUPO-PSI/mzIdentML/pull/15

-- Commit Summary --

  * Fix error that occurs during cs class generation

-- File Changes --

    M schema/mzIdentML1.2.0-candidate.xsd (3)

-- Patch Links --

https://github.com/HUPO-PSI/mzIdentML/pull/15.patch
https://github.com/HUPO-PSI/mzIdentML/pull/15.diff

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/HUPO-PSI/mzIdentML/pull/15

Re: [Psidev-pi-dev] [HUPO-PSI/mzIdentML] Fix error that occurs during cs class generation (#15)

[andrewrobertjones <not...@gi...>]

Hi @stefanks can you give me the plain English description of the issue and solution? Thanks Andy

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/HUPO-PSI/mzIdentML/pull/15#issuecomment-215742630

Re: [Psidev-pi-dev] [HUPO-PSI/mzIdentML] Fix error that occurs during cs class generation (#15)

[stefanks <not...@gi...>]

I'm trying to automatically generate a usable class to use in C# on windows. Usually it works by running the command xsd.exe theSchema.xsd /classes in the visual studio developer console. Because of a bug in xsd.exe, it generates a class with a double array instead of a single array anytime it encounters maxOccurs="unbounded" in the xsd schema. 

Since microsoft has not fixed the error, adding this harmless line streamlines the workflow for anyone trying to automatically generate a class from the schema. 

Does it break any use/test cases?

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/HUPO-PSI/mzIdentML/pull/15#issuecomment-215745359

Re: [Psidev-pi-dev] [HUPO-PSI/mzIdentML] Fix error that occurs during cs class generation (#15)

[andrewrobertjones <not...@gi...>]

It appears that you would like to add an attribute "tmp" to one of the elements, to fix a downstream bug in a piece of Microsoft software? The addition of the attribute wouldn't be harmless, since people could put data in the attribute - or do I misunderstand? We wouldn't normally change the standard to fix downstream bugs in software

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/HUPO-PSI/mzIdentML/pull/15#issuecomment-215747290

Re: [Psidev-pi-dev] [HUPO-PSI/mzIdentML] Fix error that occurs during cs class generation (#15)

[stefanks <not...@gi...>]

Yeah, I agree that it's not a good idea to change the schema to fix downstream bugs. I just wish standard tools were working without any hacks...

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/HUPO-PSI/mzIdentML/pull/15#issuecomment-215749987

Re: [Psidev-pi-dev] [HUPO-PSI/mzIdentML] Fix error that occurs during cs class generation (#15)

[stefanks <not...@gi...>]

Closed #15.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/HUPO-PSI/mzIdentML/pull/15#event-645724058