mscross / pysplit

A package for HYSPLIT air parcel trajectory analysis.
BSD 3-Clause "New" or "Revised" License
149 stars 80 forks source link

ValueError: LineStrings must have at least 2 coordinate tuples #21

Closed shyamrnjn closed 7 years ago

shyamrnjn commented 7 years ago

Hi..

Thanks a lot for your informationa but after updating pysplit i have got another issues using this code (trajgroup = pysplit.make_trajectorygroup('C:/trajectories/jnushyam/aug') .

AttributeError Traceback (most recent call last) C:\Users\Shyam\Anaconda3\lib\site-packages\shapely\geometry\linestring.py in geos_linestring_from_py(ob, update_geom, update_ndim) 199 # From array protocol --> 200 array = ob.array_interface 201 assert len(array['shape']) == 2

AttributeError: 'list' object has no attribute 'array_interface'

During handling of the above exception, another exception occurred:

ValueError Traceback (most recent call last) in () ----> 1 trajgroup = pysplit.make_trajectorygroup('C:/trajectories/jnushyam/aug')

C:\Users\Shyam\Anaconda3\lib\site-packages\pysplit\hy_processor.py in make_trajectorygroup(signature) 74 trajectories.append(Trajectory(data, path, datetime, head, 75 folder, hyfile, clipdir, ---> 76 multitraj)) 77 78 # initialize trajectory group

C:\Users\Shyam\Anaconda3\lib\site-packages\pysplit\traj.py in init(self, trajdata, pathdata, datetime, trajheader, folder, filename, cfolder, multitraj) 52 If True, is from a file containing multiple trajectories 53 """ ---> 54 HyPath.init(self, trajdata, pathdata, datetime, trajheader) 55 56 self.data.rename(columns={'AIR_TEMP': 'Temperature',

C:\Users\Shyam\Anaconda3\lib\site-packages\pysplit\hypath.py in init(self, alongpath, pathdata, datetime, header) 37 columns=header[1:], geometry=pts) 38 ---> 39 self.path = LineString(pts) 40 41 self.data['DateTime'] = datetime

C:\Users\Shyam\Anaconda3\lib\site-packages\shapely\geometry\linestring.py in init(self, coordinates) 47 BaseGeometry.init(self) 48 if coordinates is not None: ---> 49 self._set_coords(coordinates) 50 51 @property

C:\Users\Shyam\Anaconda3\lib\site-packages\shapely\geometry\linestring.py in _set_coords(self, coordinates) 92 def _set_coords(self, coordinates): 93 self.empty() ---> 94 self._geom, self._ndim = geos_linestring_from_py(coordinates) 95 96 coords = property(BaseGeometry._get_coords, _set_coords)

C:\Users\Shyam\Anaconda3\lib\site-packages\shapely\geometry\linestring.py in geos_linestring_from_py(ob, update_geom, update_ndim) 256 if m < 2: 257 raise ValueError( --> 258 "LineStrings must have at least 2 coordinate tuples") 259 260 def _coords(o):

ValueError: LineStrings must have at least 2 coordinate tuples

Interestingly, if I use to remove the fromaug (i.e aug) (trajgroup = pysplit.make_trajectorygroup('C:/trajectories/jnushyam/aug*') then thiis command shows no error but executing the next commands don't display the trajectory map. To generate the traj map , I used the follwing commands to display the map with trajectory at jupyter notebook on window 10.

years = [2016] months = [2,6,8,11] hours = [11, 17, 23] altitudes = [500, 1000, 1500] location = (28.70, 77.14) runtime = -120 monthslice=slice(0, 32, 2)

pysplit.generate_bulktraj(basename, working_dir, storage_dir, meteo_dir, years, months, hours, altitudes, location, runtime, monthslice)

trajgroup = pysplit.make_trajectorygroup(r'C:/trajectories/jnushyam/aug*')

`mapcorners = [40, -15, 170, 60]

standard_pm = None`

bmap_params = pysplit.MapDesign(mapcorners, standard_pm)

bmap = bmap_params.make_basemap()

`color_dict = {500.0 : 'blue',

          1000.0 : 'orange',

          1500.0 : 'black'}`

`for traj in trajgroup:

altitude0 = traj.data.geometry.apply(lambda p: p.z)[0]

traj.trajcolor = color_dict[altitude0]`

`for traj in trajgroup[::5]:

bmap.plot(*traj.path.xy, c=traj.trajcolor, latlon=True, zorder=20)`

Thanks shyam-aug-2016.zip

mscross commented 7 years ago

This LineString error is very strange, I have never seen it before. Can you give me a little more information:

In part 1:

Thanks!

shyamrnjn commented 7 years ago

Hi, In part 1:

basename = 'jnushyam'

In part 2:

standard_pm = None`

bmap_params = pysplit.MapDesign(mapcorners, standard_pm) bmap = bmap_params.make_basemap() `color_dict = {500.0 : 'blue',

          1000.0 : 'orange',

          1500.0 : 'black'}`

`for traj in trajgroup:

altitude0 = traj.data.geometry.apply(lambda p: p.z)[0]

traj.trajcolor = color_dict[altitude0]`

`for traj in trajgroup[::5]:

bmap.plot(*traj.path.xy, c=traj.trajcolor, latlon=True, zorder=20)`

print(trajgroup.trajcount) bmap.ax.get_figure()

Its realy great that you have created this wondeerful plugins to help open source supporters like me. It would be great if you have some spare time solve this issues.

With Best

shyam

mscross commented 7 years ago

I opened up a few of your trajectory files and finally found the problem! During trajectory generation, PySPLIT did not find all of the meteorology files it needed. This is the header in your files:

 5     1
GDAS    16     7    22     0     0
GDAS    16     7    29     0     0
GDAS    16     8     1     0     0
GDAS    16     8    29     0     0
GDAS    16     9     1     0     0
 1 BACKWARD OMEGA   
16     8     7    23   28.700   77.140   500.0
 1 PRESSURE

Notice it goes from GDAS 16 8 1 0 0 to GDAS 16 8 29 0 0- in August, you're missing weeks 2, 3, and 4 of your meteorology (which if found would be listed in your header as GDAS 16 8 8 0 0, GDAS 16 8 15 0 0, and GDAS 16 8 22 0 0). In trajectory files belonging to those weeks, there is absolutely nothing beyond this header- no data to read in means no Trajectory. Your trajectory files that do have path information that can be read properly, but the data-less trajectory files cutoff the TrajectoryGroup formation before those are reached.

So, check that you have those meteorology files and that their names follow the same pattern as the others, then try generating your trajectories again. I'll try to find a way to alert users of missing meteorology and hopefully prevent others from experiencing this issue.

Good luck! Let me know if you have any other issues! -Mellissa