Comments (4)
Wanted to reference OpenDroneMap/ODM#1074 here, since this may be the same issue.
The sanitizing in NodeODM seems unnecessary since the rest of ODM works without it. In addition to the above issue, I found it causes errors when using exif files sent from a master odm instance running with --sm-cluster, and any use of the feature and match data being transferred back will also have sanitized filenames that won't match filenames in the master node.
I would think this has been missed since the two most common use cases with WebODM and CloudODM both transfer data to NodeODM at the start of processing, ensuring all filenames are mangled from the start (except GCP entries)
from nodeodm.
Sorry for the late reply, mm, I think you're right, special characters should work OK in ODM (?) and I think this was implemented to originally fix the common issue of spaces in filenames, but since ODM handles those now, this is probably no longer needed.
from nodeodm.
Commit referencing sanitization: cbf5e40
from nodeodm.
Removed sanitization in latest master. This can be closed.
from nodeodm.
Related Issues (20)
- "rerun" argument does not work on nodeODM when running from pyODM HOT 2
- Was looking at this issue and I find it the other-way around- every time I start a task, it automatically adds the --pc-ept and --cog options, even if I didn't use them (working with pyodm) HOT 1
- node-odm:gpu don't start on Ubuntu 22.04 due of lacking cgroups v2 support HOT 3
- Separate webhooks for processing success & failure HOT 1
- Unable to use gpu acceleration as docker can't identify nvidia gpu driver HOT 2
- nodeodm-windows-x64.zip Missing libraries? HOT 1
- Add apptainer def file to allow HPC optimized containers HOT 2
- Adding the porting of Dockerfile to Apptainer def file to the workflow HOT 5
- Feature request: Return meshes from split merge HOT 3
- Cannot use external hard drive with NodeODM
- There are no usable processing nodes. Make sure that at least one processing node is reachable and that you have granted the current user sufficient permissions to view the processing node HOT 1
- Callback URL when the task is finish HOT 1
- Could you support local path for run HOT 1
- want to set options and changed 3d-tiles to true HOT 2
- nodeodm refuses to launch with undefined minimist module HOT 7
- Unable to See Releases Tags in the github HOT 1
- matcher-order setting doesn't have any effect HOT 1
- BFMatcher_GPU HOT 1
- DensifyPointCloud: error while loading shared libraries: libcuda.so.1 HOT 1
- libtbb2 is missing in nodeodm:gpu docker container HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from nodeodm.