Implementing a generic SCPI class

Hi Elodie,

I'm afraid it's not working for us, since the implementation on this branch is trying to communicate with our HMP4040 via gpib, but we are just using usb. This is the trace from the "powersupply" binary until the error (from top to bottom)

file	function	line
powersupply.cpp	`main(int argc, char*argv[])`	`std::shared_ptr<IPowerSupply> PSreal = hw.getPowerSupply(name);`
EquipConf.cpp	`EquipConf::getPowerSupply(const std::string& name)`	`ps->checkCompatibilityList();`
IPowerSupply.cpp	`IPowerSupply::checkCompatibilityList()`	`std::string idn = identify();`
SCPIPs.cpp	`SCPIPs::identify()`	`std::string idn=this->receive("*IDN?");`
SCPIPs.cpp	`SCPIPs::receive(const std::string& cmd)`	`m_com->write("++addr " + std::to_string(m_config["communication"]["gpib_addr"].get<uint32_t>()) + "\n\r");`

So it returns an error since we don't have any "gpib_addr" entry the json config for this PS, which looks like

"HMP4040" : {
	"hw-type": "PS",
	"hw-model": "RS_HMPXXXX",
	"communication": {
		"protocol" : "usb",
		"port" : "/dev/ttyACM0"
	}
}

If I add the entry with a dummy value, the error moves to SerialCom.cpp (Error reading from /dev/ttyACM0: Resource temporarily unavailable). I don't know if I'm missing something or doing anything wrong.

Are both the Agilent and the Rigol PSs using the GPIB protocol?

changed the description

changed target branch from master to devel

added 1 commit

a955c1fe - add check for gpib address

Compare with previous version

added 3 commits

b2ab9df6 - 1 commit from branch devel
bfc8dd7a - creating SCPIPs class and migrating PS that use that language to inherit from that class
f2b5db12 - add check for gpib address

Compare with previous version

@eressegu, I've been getting a different error now, coming from SerialCom.cpp, and due to the way we send commands to the PS by calling the write function from SCPIPs.cpp.

The error is

terminate called after throwing an instance of 'std::runtime_error'
  what():  Error reading from /dev/ttyACM0: Resource temporarily unavailable
Aborted (core dumped)

And the solution is to change all the \n\r sequences in the send and receive functions of SCPIPs.cpp to just \n.

I'm wondering now if this would affect the Rigol and Agilent PSs.

marked as a Work In Progress

mentioned in issue #35 (closed)

added 6 commits

f2b5db12...87eddd63 - 4 commits from branch devel
86b5bf64 - creating SCPIPs class and migrating PS that use that language to inherit from that class
259595c9 - add check for gpib address

Compare with previous version