<html><head/><body><html><head><meta http-equiv="Content-Type" content="text/html; charset=UTF-8" /></head><body>Why not releasing a beta before merging?<br>
All the best.<br><br><div class="gmail_quote">Marco Hugentobler <marco.hugentobler@sourcepole.ch> ha scritto:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><br /></div><div>Hi Martin</div><div><br /></div><div>Cool, great news! Can't wait to try out the new vector classes. I'm away from the computer until 4th of January. Probably other core devs are out of contact too, so please make the review period long enough.</div><div><br /></div><div>Regards, Marco</div><div><br /></div><div><br /></div><div><br /></div><div><div style="font-size:100%">Von Samsung Galaxy Note gesendet</div></div> <br />Martin Dobias <wonder.sk@gmail.com> hat geschrieben:<br />Hi everyone,<br /><br />it seems that now it is a good time to ask for a broader review of the<br />work I have been doing during recent months: essentially making QGIS<br />vector API more flexible and more ready for introduction of threading<br />for rendering. That resulted in a greater refactoring of some parts of<br />QgsVectorLayer class and QgsVectorDataProvider implementations.<br />Everything is in new_vector_api branch in QGIS repository on GitHub<br />[1].<br />
<br
/>There are few things that are not finished, but should not take too<br />much work to sort out:<br />- disabled providers - mssql, osm, sqlanywhere, wfs - not yet updated to new API<br />- disabled mapserver - there are few things to update<br /><br />If no serious problems will be found, I would like to merge the branch<br />to master and continue working on that on master branch to avoid the<br />possibility to drift (again) too much from the quick development that<br />happens on master. In short term I plan to do some polishing and<br />fixing bugs, then eventually start looking at the threading again.<br /><br />For more details about what happened in the branch, see the text below<br />(long read!). A great help from early testers would be to compile the<br />branch, try to do some "usual" stuff with vectors and see if things<br />break apart (crashes? data corruption?).<br /><br />Looking forward to your comments!<br /><br />Regards<br />Martin<br /><br />[1]
https://github.com/qgis/Quantum-GIS/tree/new_vector_api<br /><br /><br /><br />QGIS VECTOR API CHANGES<br /><br />1. QgsFeature (changes)<br /><br />a) access to attributes by name. Vector data providers and vector<br />layer assign pointer to the fields in QgsFeature, so it is possible to<br />set/get attributes by their name. This is not as efficient as with<br />direct access by index, but often this convenience is useful for<br />users.<br /><br />b) attributes are stored in a QVector container rather than a QMap.<br />The major advantage is simplification of logic related to handling of<br />attributes: it's not possible to have "holes" in the array of<br />fields/attributes. Currently it is possible that for example layer<br />with three fields returns indexes 0,1,3 - but it is not so common nor<br />obvious, so it's a common source of errors. After refactoring there<br />must not be any holes, so a layer with three fields always returns<br />indexes 0,1,2. When iterati
ng over
layer's features, QgsFeature always<br />contains a vector of all layer's attributes. In case the client has<br />not requested attributes to be fetched, the attributes contain invalid<br />values.<br /><br /><br />2. QgsFields (new class)<br /><br />Just like attributes within a feature are stored in a vector instead<br />of map, also layer's fields are now stored in a vector. QgsFields is a<br />new class that mimics QVector API and adds two more pieces of<br />functionality:<br /><br />a) fast lookup of field index from name. When QgsFields is populated<br />with fields, it creates a map of fields that facilitates fast lookups<br /><br />b) detection of field origin. When working with a vector layer, it is<br />sometimes useful to find out origin of the field - whether it comes<br />from provider, from a join or whether it is a newly added field (not<br />committed). In the future we could add also expression-based fields,<br />creating a field calculator that calculates t
he
values on the fly.<br /><br /><br />3. QgsFeatureRequest (new class)<br /><br />Class that encapsulates requests for features to a vector layer or<br />vector data provider. Right now in master branch, the request is<br />defined by arguments to select() method. That's not very flexible nor<br />simple to use. Feature request class allows easier extensibility in<br />future (support generic expression filter, native provider's SQL<br />filter...).<br /><br />Without any customization, the request will ask for all features with<br />geometries and attributes - somehow better default that the current<br />one that does not fetch attributes if their list is not explicitly<br />given.<br /><br />(I'm not yet completely happy with the API of this class, so it may be<br />changed to some degree. Suggestions are welcome.)<br /><br />Examples:<br />- fetch all features:<br /> QgsFeatureRequest()<br />- fetch all features, only one attribute<br />
QgsFeatureRequest().setSubsetOfAttributes(QStringList("myfield"),<br />provider->fields())<br />- fetch all features, without geometries<br /> QgsFeatureRequest().setFlags(QgsFeatureRequest::NoGeometry)<br />- fetch only features from particular extent<br /> QgsFeatureRequest().setFilterRect(QgsRectangle(0,0,1,1))<br />- fetch only one feature<br /> QgsFeatureRequest().setFilterFid(45)<br /><br /><br /><br />4. QgsFeatureIterator (new class)<br /><br />The iterator class allows iteration over features of a vector layer or<br />a provider. It contains the usual nextFeature() method that fills<br />given QgsFeature class with data.<br /><br />Internally, this class is a wrapper around QgsAbstractFeatureIterator<br />interface that is implemented by each vector data provider and by<br />vector layer. In theory we could use directly<br />QgsAbstractFeatureIterator pointers, but this wrapper allows us to use<br />it as a loc
al
variable, so it gets destroyed automatically when it<br />gets out of scope, not requiring an explicit "delete" call that would<br />be necessary with a pointer (easy to forget, right?)<br /><br /><br />5. Access to features<br /><br />Vector layer and providers implement one important method call:<br />getFeatures(). It takes single argument (QgsFeatureRequest) and<br />returns QgsFeatureIterator. This replaces select(), nextFeature(),<br />rewind() and featureAtId() methods.<br /><br />This approach allows users to create multiple iterators over a vector<br />layer (or a vector data provider). But currently that's not supported<br />because this needs support in the providers. At some point, we could<br />add support for simultaneous iterators - something that old API does<br />not allow at all.<br /><br />When QgsFeatureIterator instance of a particular layer is closed (by<br />calling close() or destructed), then a new getFeatures() call may be<br />issued.<br /><br /><br
/>6.
QgsVectorLayerEditBuffer (new class)<br /><br />All editing functionality of vector layer has been moved out of<br />QgsVectorLayer into a new class that stores everything related to<br />editing: added features, removed features, changed geometries, changed<br />attribute values, added attributes, removed attributes. This class is<br />accessible from QgsVectorLayer, but it exists only when the layer is<br />in editing mode - it is deleted once features are rolled back or<br />committed.<br /><br />The undo/redo support has been completely rewritten and greatly<br />simplified: instead of one undo command that gathers changes within<br />the edit buffer, there is now one undo command implementation for each<br />action. The undo commands can be grouped together with QUndoStack<br />macros. The new implementation ensures that the undo stack does not<br />get out of sync, because all editing changes are stored onto undo<br />stack (unlike current implementation in master branc
h
where<br />begin/end-UndoCommand() methods must be called before/after edit<br />operations to ensure correct behavior of undo).<br /><br /><br />7. QgsVectorLayerEditUtils (new class)<br /><br />Advanced editing operations that operate on vector layers (e.g. add<br />part) that can be composed from one or more simple edit operations<br />have been moved out of QgsVectorLayer and its edit buffer, so that the<br />vector layer API is not polluted by methods that do not need access to<br />its private members.<br /><br /><br />8. QgsVectorLayerCache (new class)<br /><br />To speed up geometry editing, QGIS keeps a cache of geometries<br />currently shown in map canvas. This cache has been decoupled from<br />editing operations and moved to a separate class to keep things tidy.<br />In the future we should be able to add more general caching scheme to<br />allow faster rendering of vectors. Also, this might be a good place to<br />store index of feature ID -> row number that
is
calculated every time<br />when attribute table is opened.<br /><br /><br />9. Python<br /><br />Python API has been updated to support functionality from the points<br />above, that is access by attribute name and support for python<br />iterators, e.g.:<br /><br />for feature in layer.getFeatures(request):<br /> print feature["road_id"]<br />_______________________________________________<br />Qgis-developer mailing list<br />Qgis-developer@lists.osgeo.org<br />http://lists.osgeo.org/mailman/listinfo/qgis-developer<br /><p style="margin-top: 2.5em; margin-bottom: 1em; border-bottom: 1px solid #000"></p><pre style="white-space: pre-wrap; word-wrap:break-word; font-family: sans-serif; margin-top: 0px"><hr /><br />Qgis-developer mailing list<br />Qgis-developer@lists.osgeo.org<br /><a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br /></pre></blockquote></div><br>
-- <br>
<a href="http://faunalia.it/pc">http://faunalia.it/pc</a></body></html></body></html>