search

biglimp / qgis-fusion

QGIS3 plugin for FUSION/LDV
GNU General Public License v3.0
7 stars 0 forks source link

Fixes to CloudMetrics #4

Closed spono closed 4 years ago

spono commented 4 years ago

I fixed a couple of things in CloudMetrics and now it works. I might try to pull some new tools more in the next days

biglimp commented 4 years ago

Great! I will merge. Please wait until I have pushed my changes on including the laz fileformat

spono commented 4 years ago

oh, ok...I'm quite a newbie with the PR process: shall I make another one later or what? Thanks for your help

biglimp commented 4 years ago

I merged. Now you can go ahead to add new tools. Also, if you want to add 64-bit capabilities and keep the option active you just change the last input variable from False to True as the example below from line 81 in cloudmetrix.py.

self.addParameter(QgsProcessingParameterBoolean( self.VERSION64, self.tr('Use 64-bit version'), True))

biglimp commented 4 years ago

Just make sure that you have the latest commit from the repo and then just do PR and I will merge

spono commented 4 years ago

shall I delete my fork and re-fork yours or there's another way? I'm working from github online

biglimp commented 4 years ago

hm, I am not sure but if you are working with the online github you should probably refork.