scially / Cesium3DTilesConverter

Convert OSGB,Shp,GDAL To Cesium 3DTiles
GNU Lesser General Public License v2.1
369 stars 85 forks source link

一点改进的小建议 #5

Closed cbtogu closed 9 months ago

cbtogu commented 2 years ago

首先很感谢您的工作,在fanvanzh的基础上做了很多改进,这些对我来说帮助很大。不过有几点小问题可以改进一下

  1. 需要设置PROJ_LIB环境变量,不然程序会提示找不到proj.db。另外如果电脑里之前已经安装过PostgreSQL的话,运行Converter.exe可能也会报错。这个可以在README说明。
  2. 如果metadata.xml内容不符合要求的话,没有出错提示,会让人无法定位错误。这个可以稍微改进一下哈。
  3. 还有一点,如果osgb储存的是点云,而不是三角面片的话,无法完成转换,这是故意这么设计的吗?
scially commented 2 years ago
  1. 您好,关于第一点,已经在代码上解决,https://github.com/scially/Cesium3DTilesConverter/blob/master/include/GDALWrapper.h#L18-L20
  2. 第三点是因为解析的时候就是按照三角面片解析的,后面我优化下
  3. 第二点是什么错误提示呢?
cbtogu commented 2 years ago

就是没有任何提示,转换结果就是不对

scially commented 2 years ago

有用最新的版本试下么。