Skip to main content

scoping - function fails to use local variable when it is called inside a Module


I have these two functions fun and microstep.Fun makes use of a Module construct within which I define the Array I need to store the values of magnetization for different temperatures (each case stored in a different row). microstep is the function that store the data at the correct position at each step of the Monte Carlo algorithm. The monte Carlo procedure doesn't matter really much now, what bothers me is that when I define the magnetization array inside fun, the function doesn't work properly:


   fun [numbofsets_, nsteps_] := Module [{confinit, magnetization, index},

index =
MapIndexed[ { #2[[1]], # } &, numbofsets ]; (* {index,
temp} tuple*)
confinit = RandomChoice[{-1, 1}, {10, 10}]; (*
initial random matrix *)

magnetization =
ConstantArray[ 0, {Length@numbofsets, nsteps}];


Table[

NestList[
microstep[ ##[[1]], ##[[2]], ##[[3]], ##[[4]] ] & \
, { index[[i, 1]], index[[i, 2]] , confinit, 2 } ,
nsteps];


, {i, 1, Length@numbofsets}];

]

and


microstep[tindex_, temp_, matrix_, mcindex_] :=
Module[{ tempmatrix = matrix, dimx, dimy, x , y , it = 1/temp, down,
up, left, right, spinsum , randnum, bool = False , J = 1 },


(* generic Metropolis Alghoritm *)
dimx = Dimensions[matrix][[1]];
dimy = Dimensions[matrix][[2]];
x = RandomInteger[{1, dimx}];
y = RandomInteger[{1, dimy}];
randnum = RandomReal[];
spinsum =
Plus[Compile`GetElement[matrix, Mod[x + 1, dimx, 1], y],
Compile`GetElement[matrix, Mod[x - 1, dimx, 1], y],
Compile`GetElement[matrix, x, Mod[y - 1, dimy, 1]],

Compile`GetElement[matrix, x, Mod[y + 1, dimy, 1]]];

If[2*J *spinsum*tempmatrix[[x, y]] < 0 \[Or]
randnum < E^(- it*2*J*tempmatrix[[x, y]]*spinsum)
,
tempmatrix[[x, y]] = -Compile`GetElement[matrix, x, y]; bool = True
];
(* tricky part starts here *)

If[bool,

magnetization[[tindex, mcindex]] =
Abs[(magnetization[[tindex, mcindex - 1]] +
2 *tempmatrix[[x, y]])] ;
,
magnetization[[tindex, mcindex]] =
magnetization[[tindex, mcindex - 1]];

];
{tindex, temp, tempmatrix, mcindex + 1}


]

now if i run


fun [{2, 3, 4}, 10]

i get



"Part specification magnetization[[1,1]] is longer than depth of \ object"



Meanwhile If I declare the magnetization array outside the Module, the function works properly giving me the correctly stored values, but it forces me to use global variables :



magnetization = ConstantArray[0, {3, 11}];
fun [{2, 3, 4}, 10];
magnetization


{ {0, 2, 0, 2, 4, 6, 8, 6, 8, 6, 6}, {0, 2, 2, 0, 2, 2, 4, 4, 2, 0, 2}, {0, 2, 4, 6, 8, 10, 8, 6, 6, 4, 4} })



I think the problem rise up from the fact that module is a scoping construct but I thought that a function called inside it would see the local variable but it doesn't and I don't know how to solve the problem. In c-like languages pointers can be used, is there anything similar in mathematica? Also, as always, any suggestion is appreciated



Answer



Does this work out for you?



Here I added magnetization as additional argument and gave microstep the attibute HoldAll to allow for call by reference.


SetAttributes[microstep, HoldAll];
microstep[tindex_, temp_, matrix_, mcindex_, magnetization_] :=
Module[{tempmatrix = matrix, dimx, dimy, x, y, it = 1/temp, down, up,
left, right, spinsum, randnum, bool = False,
J = 1},(*generic Metropolis Alghoritm*)

dimx = Dimensions[matrix][[1]];
dimy = Dimensions[matrix][[2]];
x = RandomInteger[{1, dimx}];

y = RandomInteger[{1, dimy}];
randnum = RandomReal[];
spinsum = Plus[
Compile`GetElement[matrix, Mod[x + 1, dimx, 1], y],
Compile`GetElement[matrix, Mod[x - 1, dimx, 1], y],
Compile`GetElement[matrix, x, Mod[y - 1, dimy, 1]],
Compile`GetElement[matrix, x, Mod[y + 1, dimy, 1]]
];
If[2*J*spinsum*tempmatrix[[x, y]] < 0 ∨
randnum < E^(-it*2*J*tempmatrix[[x, y]]*spinsum),

tempmatrix[[x, y]] = -Compile`GetElement[matrix, x, y];
bool = True];
(*tricky part starts here*)
If[bool, magnetization[[tindex, mcindex]] = Abs[(magnetization[[tindex, mcindex - 1]] + 2*tempmatrix[[x, y]])];,
magnetization[[tindex, mcindex]] = magnetization[[tindex, mcindex - 1]];
];
{tindex, temp, tempmatrix, mcindex + 1}]

There was also a second issue within fun: Apparently, the array magnetization was set up a bit too short, so I prolonged it by 1. Removing also some ;, the function fun executes without error and returns a result. Checking wether the result is correct is up to you.


fun[numbofsets_, nsteps_] := 

Module[{confinit, index},
index = MapIndexed[{#2[[1]], #} &, numbofsets];(*{index,temp} tuple*)

confinit = RandomChoice[{-1, 1}, {10, 10}];(*initial random matrix*)

magnetization = ConstantArray[0, {Length@numbofsets, nsteps + 1}];
Table[
NestList[
microstep[##[[1]], ##[[2]], ##[[3]], ##[[4]], magnetization] &,
{index[[i, 1]], index[[i, 2]], confinit, 2}, nsteps

]
,
{i, 1, Length@numbofsets}]
]

Comments

Popular posts from this blog

functions - Get leading series expansion term?

Given a function f[x] , I would like to have a function leadingSeries that returns just the leading term in the series around x=0 . For example: leadingSeries[(1/x + 2)/(4 + 1/x^2 + x)] x and leadingSeries[(1/x + 2 + (1 - 1/x^3)/4)/(4 + x)] -(1/(16 x^3)) Is there such a function in Mathematica? Or maybe one can implement it efficiently? EDIT I finally went with the following implementation, based on Carl Woll 's answer: lds[ex_,x_]:=( (ex/.x->(x+O[x]^2))/.SeriesData[U_,Z_,L_List,Mi_,Ma_,De_]:>SeriesData[U,Z,{L[[1]]},Mi,Mi+1,De]//Quiet//Normal) The advantage is, that this one also properly works with functions whose leading term is a constant: lds[Exp[x],x] 1 Answer Update 1 Updated to eliminate SeriesData and to not return additional terms Perhaps you could use: leadingSeries[expr_, x_] := Normal[expr /. x->(x+O[x]^2) /. a_List :> Take[a, 1]] Then for your examples: leadingSeries[(1/x + 2)/(4 + 1/x^2 + x), x] leadingSeries[Exp[x], x] leadingSeries[(1/x + 2 + (1 - 1/x...

How to thread a list

I have data in format data = {{a1, a2}, {b1, b2}, {c1, c2}, {d1, d2}} Tableform: I want to thread it to : tdata = {{{a1, b1}, {a2, b2}}, {{a1, c1}, {a2, c2}}, {{a1, d1}, {a2, d2}}} Tableform: And I would like to do better then pseudofunction[n_] := Transpose[{data2[[1]], data2[[n]]}]; SetAttributes[pseudofunction, Listable]; Range[2, 4] // pseudofunction Here is my benchmark data, where data3 is normal sample of real data. data3 = Drop[ExcelWorkBook[[Column1 ;; Column4]], None, 1]; data2 = {a #, b #, c #, d #} & /@ Range[1, 10^5]; data = RandomReal[{0, 1}, {10^6, 4}]; Here is my benchmark code kptnw[list_] := Transpose[{Table[First@#, {Length@# - 1}], Rest@#}, {3, 1, 2}] &@list kptnw2[list_] := Transpose[{ConstantArray[First@#, Length@# - 1], Rest@#}, {3, 1, 2}] &@list OleksandrR[list_] := Flatten[Outer[List, List@First[list], Rest[list], 1], {{2}, {1, 4}}] paradox2[list_] := Partition[Riffle[list[[1]], #], 2] & /@ Drop[list, 1] RM[list_] := FoldList[Transpose[{First@li...

front end - keyboard shortcut to invoke Insert new matrix

I frequently need to type in some matrices, and the menu command Insert > Table/Matrix > New... allows matrices with lines drawn between columns and rows, which is very helpful. I would like to make a keyboard shortcut for it, but cannot find the relevant frontend token command (4209405) for it. Since the FullForm[] and InputForm[] of matrices with lines drawn between rows and columns is the same as those without lines, it's hard to do this via 3rd party system-wide text expanders (e.g. autohotkey or atext on mac). How does one assign a keyboard shortcut for the menu item Insert > Table/Matrix > New... , preferably using only mathematica? Thanks! Answer In the MenuSetup.tr (for linux located in the $InstallationDirectory/SystemFiles/FrontEnd/TextResources/X/ directory), I changed the line MenuItem["&New...", "CreateGridBoxDialog"] to read MenuItem["&New...", "CreateGridBoxDialog", MenuKey["m", Modifiers-...