scipion-em / scipion-em-tomo

Base tomography Scipion plugin with data model definition
GNU General Public License v3.0
8 stars 4 forks source link

Import TS, TSM and tomos: remove double underscore from both the link… #491

Closed JorMaister closed 4 weeks ago

JorMaister commented 1 month ago

…ed filenames and the tsIds.

pconesa commented 1 month ago

What about subtomograms imports.... should we do the same?

Not sure if we should fo this this way. Many thanks anyway @JorMaister for taking care of this this fast.

Anothe option would be to:

    • do this cleanning in eman, I think is the right way to go, but Jorge tried and undo it because it was affecting many parts of the code (risky)

2.- Do nothing, maybe want about this?

3.- GO ahead?

@azazellochg , @JorMaister ?

azazellochg commented 1 month ago

I agree that handling this inside eman plugin would be nice, but for the reasons you cite above I vote to go ahead with this.

JorMaister commented 1 month ago

ok, I think we should go ahead, but I get the point of how it looks to make a specific program condition our import...

sonarcloud[bot] commented 4 weeks ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud