Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Namespace lookup not working in imperative blocks #8084

Closed
dimven opened this issue Aug 4, 2017 · 3 comments
Closed

Namespace lookup not working in imperative blocks #8084

dimven opened this issue Aug 4, 2017 · 3 comments

Comments

@dimven
Copy link
Contributor

dimven commented Aug 4, 2017

Dynamo version

1.3 / 1.31

Operating system

Windows 10

What did you do?

Tried to resolve this issue:
dimven/SpringNodes#29

What did you expect to see?

def NullAllInd(l1:var[],val){
return = [Imperative]
{
i;
z = {};
for(i in GetKeys(l1) )
{
	if(l1[i] == val)
	{
		0;
		z = DSCore.List.AddItemToEnd(i,z);
	}
}
return = z;
}};

NullAllInd({null,1,2,null,3,null,1,2}, 1)

should return {1, 6}

What did you see instead?

null

This is a pretty big issue, because whenever you install some of the most popular packages out there, something in imperative code becomes unusable. (i.e. Lunchbox disables Solid, Rhynamo disables Point, archi-lab.net disables List, etc...)

If I remove archi-lab, the code behaves as expected:

dynamosandbox_2017-08-04_21-03-18

@aparajit-pratap

@ThomasMahon
Copy link

ThomasMahon commented Aug 4, 2017

See also #7697. @dimven agree, its a major issue. For the uninitiated (most of whom are likely to be using Archi-lab given its popularity), I doubt they will be able to pinpoint the cause - the issue really warrants urgent attention.

@mjkkirschner
Copy link
Member

mjkkirschner commented Aug 4, 2017

@dimven are you saying that even when you use the full namespace of a list function the function is not found?

@dimven
Copy link
Contributor Author

dimven commented Aug 4, 2017

Ah, I forgot to check if this has been reported already. Since this is a known issue, i'll close the current one and follow up in #7697.

@mjkkirschner , yes that seems to be the case.

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

No branches or pull requests

3 participants