Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Added histogram values to the bindings
  • Loading branch information
elpaso committed Nov 29, 2016
1 parent 0ee93ee commit 61b599e
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions python/core/raster/qgsrasterhistogram.sip
Expand Up @@ -4,6 +4,9 @@ class QgsRasterHistogram
#include <qgsrasterhistogram.h>
%End
public:

typedef QVector<int> HistogramVector;

QgsRasterHistogram();

/** Compares region, size etc. not histogram itself */
Expand All @@ -22,9 +25,9 @@ class QgsRasterHistogram
bool includeOutOfRange;

/** \brief Store the histogram for a given layer
* @note not available via python binding
*/
// HistogramVector histogramVector;
* @note added in version 3.0
*/
HistogramVector histogramVector;

/** \brief The maximum histogram value. */
double maximum;
Expand Down

3 comments on commit 61b599e

@3nids
Copy link
Member

@3nids 3nids commented on 61b599e Aug 2, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@elpaso have you been able to use this? @alexbruy seems to have issues with it: https://lists.osgeo.org/pipermail/qgis-developer/2017-August/049498.html

@elpaso
Copy link
Contributor Author

@elpaso elpaso commented on 61b599e Aug 3, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC I tested I and it was working correctly when I exposed the histogramVector to the bindings. I stepped into that while I was testing the IPython integration and I wanted to display a chart inside the console, the vector with the data was not accessible from Python, that's why I re-enabled it.

@elpaso
Copy link
Contributor Author

@elpaso elpaso commented on 61b599e Sep 30, 2017 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.