Difference between revisions of "Developer Meetings/20131126"

From Slicer Wiki
Jump to: navigation, search
(Created page with '== To discuss == * Refactoring of RemoteIO. See below * Extension and dependency on ITKDeprecated and ITKV3Compatibility library == News & Update == * Support for Slicer_USE_SYS…')
 
 
(7 intermediate revisions by 2 users not shown)
Line 1: Line 1:
 
== To discuss ==
 
== To discuss ==
 +
* SimpleITK v0.7.1
 +
** a couple bug fixes
 
* Refactoring of RemoteIO. See below
 
* Refactoring of RemoteIO. See below
 
* Extension and dependency on ITKDeprecated and ITKV3Compatibility library
 
* Extension and dependency on ITKDeprecated and ITKV3Compatibility library
Line 14: Line 16:
  
 
<pre>
 
<pre>
I notice that the only reason we depend on curl is for existence of RemoteIO library that has been "replaced" by SampleData and DataStore.
 
  
I really like the concept of registering URI that could be downloaded that is why we should keep the infrastructure in place.
+
(1) Jc - Nov 26, 2013
 +
 
 +
I notice that the only reason we depend on curl is for existence
 +
of RemoteIO library that has been "replaced" by SampleData
 +
and DataStore.
 +
 
 +
I really like the concept of registering URI that could be
 +
downloaded that is why we should keep the infrastructure in place.
  
 
That said, I was thinking to get remove the following classes:
 
That said, I was thinking to get remove the following classes:
Line 27: Line 35:
 
vtkXNDTagTable.h
 
vtkXNDTagTable.h
  
Then, we could implement a Qt based  vtkHTTPHandler handler, that way we could simply remove the curl dependency.
+
Then, we could implement a Qt based  vtkHTTPHandler handler, that  
 +
way we could simply remove the curl dependency.
  
In the mean time, I would also remove the FetchMI logic which is not maintained or used at that point.
+
In the mean time, I would also remove the FetchMI logic which is  
 +
not maintained or used at that point.
  
 
Let me know what you think,
 
Let me know what you think,
Jc
+
 
 +
// --------------------
 +
(2) Andras - Nov 26, 2013
 +
 
 +
It’s always a good idea to make the core smaller and simpler, so
 +
I support the idea of getting rid of unused classes and unnecessary
 +
dependencies.
 +
 
 +
We planned to use curl for uploading training results in a .zip
 +
file to Moodle, but I guess we can add curl to our module then.
 +
 
 +
// --------------------
 +
(3) Jc - Nov 26, 2013
 +
 
 +
Thanks for commenting.
 +
 
 +
If your module is a loadable or a scripted module, all "curl"
 +
functionality can be provided by Qt. Would that work for you ?
 +
 
 +
// --------------------
 +
(4) Andras - Nov 26, 2013
 +
 
 +
We would need quite complex curl functionalities for uploading data,
 +
see the scripts below:
 +
 
 +
https://gist.github.com/Joshkunz/4516316
 +
 
 +
https://github.com/proyectoeduhub/moodle-cli/blob/master/moodle-file-upload-curl.sh
 +
 
 +
Would it be feasible to implement these?
 +
 
 +
// --------------------
 +
(5) Jc - Nov 26, 2013
 +
 
 +
Good point. The plan would then be to:
 +
 
 +
  (1) remove dependency of Slicer core against curl
 +
 
 +
  (2) Only, build both the Curl library and the curl executable, keeping
 +
them in the package.
 +
 
 +
That way, it would provide great flexibility with minimum overhead.
 +
 
 +
// --------------------
 +
(6) Steve - Nov 26, 2013
 +
 
 +
Sounds like an excellent plan.
 +
 
 +
Yes, I like the design of the remote IO support in MRML, but it's not
 +
being used and is great to simplify wherever possible.  FetchMI can
 +
go away for now, with the hope that we'll reproduce and improve on
 +
it in the future (someday).
 
</pre>
 
</pre>
 +
 +
== Conclusion ==
 +
 +
* Bump:
 +
** ITK tag to <code>4.5rc01</code>
 +
** SimpleITK to release branch <code>a2b29110771a</code>
 +
 +
* Steve: Slicer AHM Tutorial related to wrapping of custom class using SimpleITK ?
 +
** Brad: The underlying infrastructure is not yet ready to support this. We should provide some requirement / use case.

Latest revision as of 23:15, 26 November 2013

Home < Developer Meetings < 20131126

To discuss

  • SimpleITK v0.7.1
    • a couple bug fixes
  • Refactoring of RemoteIO. See below
  • Extension and dependency on ITKDeprecated and ITKV3Compatibility library

News & Update

  • Support for Slicer_USE_SYSTEM_* options

In progress


Usage of RemoteIO in Slicer


(1) Jc - Nov 26, 2013

I notice that the only reason we depend on curl is for existence 
of RemoteIO library that has been "replaced" by SampleData 
and DataStore.

I really like the concept of registering URI that could be
downloaded that is why we should keep the infrastructure in place.

That said, I was thinking to get remove the following classes:

vtkHIDHandler.h
vtkHIDTagTable.h
vtkSRBHandler.h
vtkXNATHandler.h
vtkXNDHandler.h
vtkXNDTagTable.h

Then, we could implement a Qt based  vtkHTTPHandler handler, that 
way we could simply remove the curl dependency.

In the mean time, I would also remove the FetchMI logic which is 
not maintained or used at that point.

Let me know what you think,

// --------------------
(2) Andras - Nov 26, 2013

It’s always a good idea to make the core smaller and simpler, so 
I support the idea of getting rid of unused classes and unnecessary 
dependencies.

We planned to use curl for uploading training results in a .zip 
file to Moodle, but I guess we can add curl to our module then.

// --------------------
(3) Jc - Nov 26, 2013

Thanks for commenting.

If your module is a loadable or a scripted module, all "curl" 
functionality can be provided by Qt. Would that work for you ?

// --------------------
(4) Andras - Nov 26, 2013

We would need quite complex curl functionalities for uploading data, 
see the scripts below:

https://gist.github.com/Joshkunz/4516316

https://github.com/proyectoeduhub/moodle-cli/blob/master/moodle-file-upload-curl.sh

Would it be feasible to implement these?

// --------------------
(5) Jc - Nov 26, 2013

Good point. The plan would then be to:

  (1) remove dependency of Slicer core against curl

  (2) Only, build both the Curl library and the curl executable, keeping
 them in the package.

That way, it would provide great flexibility with minimum overhead.

// --------------------
(6) Steve - Nov 26, 2013

Sounds like an excellent plan.

Yes, I like the design of the remote IO support in MRML, but it's not
 being used and is great to simplify wherever possible.  FetchMI can 
go away for now, with the hope that we'll reproduce and improve on 
it in the future (someday).

Conclusion

  • Bump:
    • ITK tag to 4.5rc01
    • SimpleITK to release branch a2b29110771a
  • Steve: Slicer AHM Tutorial related to wrapping of custom class using SimpleITK ?
    • Brad: The underlying infrastructure is not yet ready to support this. We should provide some requirement / use case.