Skip to content

Conversation

@ruihengsu
Copy link
Contributor

@ruihengsu ruihengsu commented Sep 21, 2023

Open connections to the lakeshore magnet power supply parameters corrected. Tested to work in the LD system
Errors associated with connecting to the IPS power supply were also corrected.

@ruihengsu
Copy link
Contributor Author

The most important changes are the ipfs for the LS625 and IPS120 power supplies!

rampK2400Voltage(S.instrIDx, startx, ramprate = rampratex)

// Let gates settle
sc_sleep(S.delayy*20)
Copy link
Member

Choose a reason for hiding this comment

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

It is probably better to keep delayy instead of hard coding a 2 second sleep. It would be better to add an optional parameter to Scank2400 that allows you to set delayy and if not set will use default of 2.

rampK2400Voltage(S.instrIDx, setpointx, ramprate=S.rampratex)
else
setK2400Voltage(S.instrIDx, setpointx)
endif
Copy link
Member

Choose a reason for hiding this comment

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

Not essential, but nice to keep orderly indenting.

fast=0
endif


Copy link
Member

Choose a reason for hiding this comment

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

In IGOR if the parameter is default it is already set to 0. But it is good to be explicit.
A nice 1 liner would be:
fast = paramisdefault(fast) ? 1 : fast

sc_sleep(S.delayx)
if(fast==1)
setlS625field(S.instrIDx, setpointx)
sc_sleep(max(S.delayx, (S.delayx+60*abs(finx-startx)/numptsx/ramprate)))
Copy link
Member

Choose a reason for hiding this comment

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

What does this math do? Looks like you have the time in minutes for a ramp to complete divided by the number of points in x. So if you take 1 data point this is the same as setlS625fieldwait(S.instrIDx, setpointx) but if you take 1e6 data points then you will start collecting data immediately??

writeInstr(instrID, "APHS \n")
end

function AutoSRSSens(instrID)
Copy link
Member

Choose a reason for hiding this comment

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

Write a comment on what this function does.




function Scan_n_limit(instrIDx,instrIDy,fixedD,startn,finn,numptsn,delayn,rampraten, limitx, limity, [y_label, comments, nosave]) //Units: mV
Copy link
Member

Choose a reason for hiding this comment

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

What does this function do? Write a comment.

@johann997
Copy link
Member

Everything looks fine to me. Most of the changes were in Scan_n_D_Bxyz.ipf which I never use.
In general, it would be nice to see more comments, especially when you add a new function or new parameters to a function. But overall seems fine to me.

FolkLab added 2 commits August 4, 2024 17:42
…he gate capacitance

New updates to the scan_n_d codes incorporating a division ratio in the gate capacitance
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants