Remove total_sites from SiteCollection.expand

Bug #1282958 reported by Michele Simionato
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenQuake Hazard Library
Fix Released
High
Michele Simionato

Bug Description

Each SiteCollection object has a method site_coll.expand(data, total_sites, placeholder) expanding the array data on the sites in site_coll and returning an array of size total_sites. The typical use case is the expansion of values relative to a filtered site collection. If we give to the filtered site collection the knowledge of total_sites before the filtering, then the parameter does not need to be passed and the API can be simplified. Damiano agrees that it is a good idea.

Revision history for this message
Michele Simionato (michele-simionato) wrote :
Changed in oq-hazardlib:
importance: Undecided → High
assignee: nobody → Michele Simionato (michele-simionato)
status: New → Fix Committed
Changed in oq-hazardlib:
milestone: none → 0.12
Changed in oq-hazardlib:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.