Skip to content

Commit 8939c6b

Browse files
authored
rm dynamic scaler option, it will be only in brainflow (#1082)
* rm dynamic scaler option, it will be only in brainflow Signed-off-by: Andrey Parfenov <a1994ndrey@gmail.com>
1 parent 4db29e9 commit 8939c6b

4 files changed

Lines changed: 1 addition & 39 deletions

File tree

OpenBCI_GUI/ADS1299SettingsBoard.pde

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -345,6 +345,4 @@ interface ADS1299SettingsBoard {
345345
public ADS1299Settings getADS1299Settings();
346346
public char getChannelSelector(int channel);
347347
public double getGain(int channel);
348-
public void setUseDynamicScaler(boolean val);
349-
public boolean getUseDynamicScaler();
350348
};

OpenBCI_GUI/BoardCyton.pde

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -252,16 +252,12 @@ implements ImpedanceSettingsBoard, AccelerometerCapableBoard, AnalogCapableBoard
252252
protected boolean[] isCheckingImpedanceN;
253253
protected boolean[] isCheckingImpedanceP;
254254

255-
// same for all channels
256-
private final double brainflowGain = 24.0;
257-
258255
private int[] accelChannelsCache = null;
259256
private int[] analogChannelsCache = null;
260257

261258
protected String serialPort = "";
262259
protected String ipAddress = "";
263260
private CytonBoardMode currentBoardMode = CytonBoardMode.DEFAULT;
264-
private boolean useDynamicScaler;
265261

266262
public BoardCyton() {
267263
super();
@@ -276,7 +272,6 @@ implements ImpedanceSettingsBoard, AccelerometerCapableBoard, AnalogCapableBoard
276272

277273
// The command 'd' is automatically sent by brainflow on prepare_session
278274
currentADS1299Settings = new CytonDefaultSettings(this);
279-
useDynamicScaler = true;
280275
}
281276

282277
// implement mandatory abstract functions
@@ -554,24 +549,6 @@ implements ImpedanceSettingsBoard, AccelerometerCapableBoard, AnalogCapableBoard
554549
}
555550
}
556551

557-
@Override
558-
protected double[][] getNewDataInternal() {
559-
double[][] data = super.getNewDataInternal();
560-
int[] exgChannels = getEXGChannels();
561-
for (int i = 0; i < exgChannels.length; i++) {
562-
for (int j = 0; j < data[exgChannels[i]].length; j++) {
563-
// brainflow assumes a fixed gain of 24. Undo brainflow's scaling and apply new scale.
564-
double currentGain = 1.0;
565-
if (useDynamicScaler) {
566-
currentGain = currentADS1299Settings.values.gain[i].getScalar();
567-
}
568-
double scalar = brainflowGain / currentGain;
569-
data[exgChannels[i]][j] *= scalar;
570-
}
571-
}
572-
return data;
573-
}
574-
575552
@Override
576553
public ADS1299Settings getADS1299Settings() {
577554
return currentADS1299Settings;
@@ -638,16 +615,6 @@ implements ImpedanceSettingsBoard, AccelerometerCapableBoard, AnalogCapableBoard
638615
return getADS1299Settings().values.gain[channel].getScalar();
639616
}
640617

641-
@Override
642-
public boolean getUseDynamicScaler() {
643-
return useDynamicScaler;
644-
}
645-
646-
@Override
647-
public void setUseDynamicScaler(boolean val) {
648-
useDynamicScaler = val;
649-
}
650-
651618
@Override
652619
public List<double[]> getDataWithAccel(int maxSamples) {
653620
return getData(maxSamples);

OpenBCI_GUI/Extras.pde

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -469,10 +469,7 @@ class DataStatus {
469469
}
470470

471471
if (currentBoard instanceof ADS1299SettingsBoard) {
472-
double scaler = (4.5 / (pow (2, 23) - 1) / 1.0 * 1000000.);
473-
if (((ADS1299SettingsBoard)currentBoard).getUseDynamicScaler()) {
474-
scaler = (4.5 / (pow (2, 23) - 1) / ((ADS1299SettingsBoard)currentBoard).getGain(channel) * 1000000.);
475-
}
472+
double scaler = (4.5 / (pow (2, 23) - 1) / ((ADS1299SettingsBoard)currentBoard).getGain(channel) * 1000000.);
476473
double maxVal = scaler * pow (2, 23);
477474
int numSeconds = 3;
478475
int nPoints = numSeconds * currentBoard.getSampleRate();
339 KB
Binary file not shown.

0 commit comments

Comments
 (0)