Opened 11 years ago
Closed 11 years ago
#496 closed defect (fixed)
Surrect the WHIZARD C Interface
Reported by: | Juergen Reuter | Owned by: | Juergen Reuter |
---|---|---|---|
Priority: | P5 | Milestone: | v2.2.0 |
Component: | core | Version: | 2.1.1 |
Severity: | normal | Keywords: | |
Cc: |
Description
At the moment, in 2.2.0 it's lost.
Change History (10)
comment:1 Changed 11 years ago by
Priority: | P3 → P1 |
---|
comment:2 Changed 11 years ago by
Priority: | P1 → P4 |
---|
comment:4 Changed 11 years ago by
Owner: | changed from kilian to Juergen Reuter |
---|---|
Priority: | P4 → P1 |
Status: | new → assigned |
Mostly done. New structure: there is a whizard instance that is handed around as a C pointer. The only global structure now are the syntax lists. The C example is working again, have to transfer the C++ and Python examples.
comment:5 Changed 11 years ago by
Owner: | changed from Juergen Reuter to kilian |
---|---|
Priority: | P1 → P0 |
Status: | assigned → new |
... has to be fixed first ... broken trunk.
comment:6 Changed 11 years ago by
Status: | new → assigned |
---|
comment:7 Changed 11 years ago by
Owner: | changed from kilian to Juergen Reuter |
---|---|
Status: | assigned → new |
r5287: this version compiles (both C and C++) and appears to do something. Process compilation (in the C example, not yet in C++) is still broken, but the test is not an active part of the test suite yet.
In the C++ wrapper, the compiler doesn't like the conversion from string to char*, but accepts it. Somebody should do this in a PC way.
I'll leave this for now and return the ticket to JR, maybe I can pick it up later. There is lots of room for improvement ...
comment:8 Changed 11 years ago by
Priority: | P0 → P2 |
---|
comment:9 Changed 11 years ago by
Priority: | P2 → P5 |
---|
Ok, the C and C++ examples are working (I disabled the HepMC example for now, it's compiling but not doing much). The Python example is not yet working, and I also did not introduce checks for Python in the configure framework. As WK said, there is much room for improvement, but I tend to close the ticket and leave the rest for #60. Opinions?
comment:10 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
After confirmation, closing. All the necessary things (adapting to new refactored WHIZARD structure and trivial C and C++ prime examples) are done.
My view: important to have it in the release but not essential for the beta.