Skip to content

Commit 936b0d8

Browse files
committed
Update number of points to uint64
1 parent e1bac5f commit 936b0d8

4 files changed

Lines changed: 10 additions & 10 deletions

File tree

src/MappingUtils.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
#include <utility>
1515
#include <vector>
1616

17-
std::pair<const mv::LinkedData*, unsigned int> getSelectionMapping(const mv::Dataset<Points>& source, const mv::Dataset<Points>& target, LinkedDataCondition checkMapping) {
17+
std::pair<const mv::LinkedData*, std::uint64_t> getSelectionMapping(const mv::Dataset<Points>& source, const mv::Dataset<Points>& target, LinkedDataCondition checkMapping) {
1818
const std::vector<mv::LinkedData>& linkedDatas = source->getLinkedData();
1919

2020
if (linkedDatas.empty())
@@ -34,7 +34,7 @@ std::pair<const mv::LinkedData*, unsigned int> getSelectionMapping(const mv::Dat
3434
return { nullptr, 0 };
3535
}
3636

37-
std::pair<const mv::LinkedData*, unsigned int> getSelectionMappingColorsToPositions(const mv::Dataset<Points>& colors, const mv::Dataset<Points>& positions) {
37+
std::pair<const mv::LinkedData*, std::uint64_t> getSelectionMappingColorsToPositions(const mv::Dataset<Points>& colors, const mv::Dataset<Points>& positions) {
3838
auto testTargetAndParent = [](const mv::LinkedData& linkedData, const mv::Dataset<Points>& positions) -> bool {
3939
const mv::Dataset<mv::DatasetImpl> mapTargetData = linkedData.getTargetDataset();
4040
return mapTargetData == positions || parentHasSameNumPoints(mapTargetData, positions);
@@ -43,7 +43,7 @@ std::pair<const mv::LinkedData*, unsigned int> getSelectionMappingColorsToPositi
4343
return getSelectionMapping(colors, positions, testTargetAndParent);
4444
}
4545

46-
std::pair<const mv::LinkedData*, unsigned int> getSelectionMappingPositionsToColors(const mv::Dataset<Points>& positions, const mv::Dataset<Points>& colors) {
46+
std::pair<const mv::LinkedData*, std::uint64_t> getSelectionMappingPositionsToColors(const mv::Dataset<Points>& positions, const mv::Dataset<Points>& colors) {
4747
auto testTarget = [](const mv::LinkedData& linkedData, const mv::Dataset<Points>& colors) -> bool {
4848
return linkedData.getTargetDataset() == colors;
4949
};
@@ -58,7 +58,7 @@ std::pair<const mv::LinkedData*, unsigned int> getSelectionMappingPositionsToCol
5858
return { mapping, numTargetPoints };
5959
}
6060

61-
std::pair<const mv::LinkedData*, unsigned int> getSelectionMappingPositionSourceToColors(const mv::Dataset<Points>& positions, const mv::Dataset<Points>& colors) {
61+
std::pair<const mv::LinkedData*, std::uint64_t> getSelectionMappingPositionSourceToColors(const mv::Dataset<Points>& positions, const mv::Dataset<Points>& colors) {
6262
if (!positions->isDerivedData())
6363
return { nullptr, 0 };
6464

src/MappingUtils.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,17 +39,17 @@ using LinkedDataCondition = std::function<bool(const mv::LinkedData& linkedData,
3939
};
4040
This function will return the first match of the condition
4141
*/
42-
std::pair<const mv::LinkedData*, unsigned int> getSelectionMapping(const mv::Dataset<Points>& source, const mv::Dataset<Points>& target, LinkedDataCondition checkMapping);
42+
std::pair<const mv::LinkedData*, std::uint64_t> getSelectionMapping(const mv::Dataset<Points>& source, const mv::Dataset<Points>& target, LinkedDataCondition checkMapping);
4343

4444
// Returns a mapping (linked data) from colors whose target is positions or whose target's parent has the same number of points as positions
45-
std::pair<const mv::LinkedData*, unsigned int> getSelectionMappingColorsToPositions(const mv::Dataset<Points>& colors, const mv::Dataset<Points>& positions);
45+
std::pair<const mv::LinkedData*, std::uint64_t> getSelectionMappingColorsToPositions(const mv::Dataset<Points>& colors, const mv::Dataset<Points>& positions);
4646

4747
// Returns a mapping (linked data) from positions whose target is colors or
4848
// a mapping from positions' parent whose target is colors if the number of data points match
49-
std::pair<const mv::LinkedData*, unsigned int> getSelectionMappingPositionsToColors(const mv::Dataset<Points>& positions, const mv::Dataset<Points>& colors);
49+
std::pair<const mv::LinkedData*, std::uint64_t> getSelectionMappingPositionsToColors(const mv::Dataset<Points>& positions, const mv::Dataset<Points>& colors);
5050

5151
// Returns a mapping (linked data) from positions' source data whose target is colors
52-
std::pair<const mv::LinkedData*, unsigned int> getSelectionMappingPositionSourceToColors(const mv::Dataset<Points>& positions, const mv::Dataset<Points>& colors);
52+
std::pair<const mv::LinkedData*, std::uint64_t> getSelectionMappingPositionSourceToColors(const mv::Dataset<Points>& positions, const mv::Dataset<Points>& colors);
5353

5454
// Check if the mapping is surjective, i.e. hits all elements in the target
5555
bool checkSurjectiveMapping(const mv::LinkedData& linkedData, const std::uint32_t numPointsInTarget);

src/ScatterplotPlugin.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -846,7 +846,7 @@ void ScatterplotPlugin::loadColors(const Dataset<Clusters>& clusters)
846846
return;
847847

848848
// Get global indices from the position dataset
849-
int totalNumPoints = 0;
849+
std::uint64_t totalNumPoints = 0;
850850
if (_positionDataset->isDerivedData())
851851
totalNumPoints = _positionSourceDataset->getFullDataset<Points>()->getNumPoints();
852852
else

src/ScatterplotPlugin.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ class ScatterplotPlugin : public ViewPlugin
119119
Dataset<Points> _positionDataset; /** Smart pointer to points dataset for point position */
120120
Dataset<Points> _positionSourceDataset; /** Smart pointer to source of the points dataset for point position (if any) */
121121
std::vector<mv::Vector2f> _positions; /** Point positions */
122-
unsigned int _numPoints; /** Number of point positions */
122+
std::uint64_t _numPoints; /** Number of point positions */
123123
QPointer<SettingsAction> _settingsAction; /** Group action for all settings */
124124
QPointer<HorizontalToolbarAction> _primaryToolbarAction; /** Horizontal toolbar for primary content */
125125
QRectF _selectionBoundaries; /** Boundaries of the selection */

0 commit comments

Comments
 (0)