Maxscript filein exception ошибка

There’s no need to delete the rest of the objects, you can use «save selected». Look it up in the manual. The problem you have is because of the deleting of objects. Because you’re working directly on the selection set and are deleting objects in the selection set, your basicly cleaning your selection array, but since you start at object 1 in the array and then delete it, object2 becomes object 1. The only way to make it work is to first assign your selection to a array like:

sel = selection as array

and then loop through it back to front.

for i in sel.count to 1 by -1 do

That way you won’t mess up the array order.
But I would go for the save selected… is also a lot «cleaner»

Cheers,
-Johan

I have to delete the rest since the exporter exports everything in the scene, selected or not, hidden or not, frozen or not, and the point of the script is to separate everything out so you only have one mesh, export it, restore everyhting (i am using hold and fetch) select next object, delete the rest, etc.

I should mention that though my script saves now as opposed to exporting it will eventually both save a new max file and an exported file in a new folder.

There’s no need to delete the rest of the objects, you can use «save selected». Look it up in the manual. The problem you have is because of the deleting of objects. Because you’re working directly on the selection set and are deleting objects in the selection set, your basicly cleaning your selection array, but since you start at object 1 in the array and then delete it, object2 becomes object 1. The only way to make it work is to first assign your selection to a array like:

sel = selection as array

and then loop through it back to front.

for i in sel.count to 1 by -1 do

That way you won’t mess up the array order.
But I would go for the save selected… is also a lot «cleaner»

Cheers,
-Johan

this is how i wanted the script to run in the whole deleteing selection part

select object 1
select invert (selecting everything but object 1)
delete (leaving object 1 behind)
move object 1 to 0,0,0
export the file to object1.hsg
save the file to object1.max

restore all objects

select object 2
select invert ……….

Ok, if your objects have no dependencies (like hierarchies, volume select mod on one using another one, skin modifier using bones etc.), then using saveNodes() is better than what you are doing (it is equivalent to File>Save Selected, but you don’t even have to select anything, just pass the object to be saved to the method. If there ARE dependencies, saving one of the nodes using saveNodes() will automatically save all objects it depends on to ensure its look and animation will be preserved.

If you DO want to delete the objects and hold/fetch around the code, it is still ok, but MUCH slower.
And you don’t need the $ sign in front of the array variable, that’s what is probably causing the error.

$ in MAXScript means a prefix of an object path. (In MEL though, it means a user variable!)
So $Box01 means the object called «Box01». $somevariable does NOT mean a variable containing an object, but an object called «somevariable».

Borislav «Bobo» Petrov

Ok, if your objects have no dependencies (like hierarchies, volume select mod on one using another one, skin modifier using bones etc.), then using saveNodes() is better than what you are doing (it is equivalent to File>Save Selected, but you don’t even have to select anything, just pass the object to be saved to the method. If there ARE dependencies, saving one of the nodes using saveNodes() will automatically save all objects it depends on to ensure its look and animation will be preserved.

If you DO want to delete the objects and hold/fetch around the code, it is still ok, but MUCH slower.
And you don’t need the $ sign in front of the array variable, that’s what is probably causing the error.

$ in MAXScript means a prefix of an object path. (In MEL though, it means a user variable!)
So $Box01 means the object called «Box01». $somevariable does NOT mean a variable containing an object, but an object called «somevariable».

I removed the «$» (which is must complain i have read no explanation in the documentation as to what it was, so thank you) but the script does not run past the selection of the object to be kept.

M

I removed the «$» (which is must complain i have read no explanation in the documentation as to what it was, so thank you) but the script does not run past the selection of the object to be kept.

Complain? Just because you haven’t read the whole Reference, it does not mean you have to complain! ;o)

Check out «Pathname Literals» topic for some more info and see the NOTE at the bottom of the page.

Can we now see the WHOLE code you are trying to use (because what you posted looks like a snippet from a larger script and it is difficult to tell where it breaks without being able to see the whole picture).
If this IS the whole script, then it is missing half of the variable declarations.

Thanks in advance.

Borislav «Bobo» Petrov

I removed the «$» (which is must complain i have read no explanation in the documentation as to what it was, so thank you) but the script does not run past the selection of the object to be kept.

Complain? Just because you haven’t read the whole Reference, it does not mean you have to complain! ;o)

Check out «Pathname Literals» topic for some more info and see the NOTE at the bottom of the page.

Can we now see the WHOLE code you are trying to use (because what you posted looks like a snippet from a larger script and it is difficult to tell where it breaks without being able to see the whole picture).
If this IS the whole script, then it is missing half of the variable declarations.

Thanks in advance.

——— hard coding master_folder for now
——- to do: make this variable input from user from a dialog
master_folder=»u:\testbed\»

——— declaring additional strings needed to complete file names
add_folder =»\»
dotmax = «.max»
dothsg = «.hsg»

— define arrays as arrays
objname_array = #()

——- get names of all objects in the scene and put their names into array «objname_array»
for x = 1 to $objects.count do (append objname_array $objects

.name)

—— folder creation based on objs in scene
for y = 1 to $objects.count do (
new_folder = master_folder + objname_array

makeDir new_folder
)

—— save scene state so we can then later restore it
max hold

—— take each object and save it out to it’s own max file, within it’s own folder
for z = 1 to $objects.count do (
—— create string for file location
save2folder = master_folder + objname_array

save2file = save2folder + add_folder + objname_array + dotmax
exported = save2folder + add_folder + objname_array + dothsg
—— selection mess

—— save to a max file
saveMaxfile save2file useNewFile:false quiet:true
)

Could be better optimized maybe…
You have lots of errors in your code, select invert is not a valid function, delete requires extra parameters, and as bobo said the array shouldn’t be called with $ in front…

Try and see if this works.

searchArr = objects as array
sa_count = searchArr.count
for o in sa_count to 1 by -1 do 
(
  ----- create string for file location
  save2folder = master_folder + objname_array
  save2file = save2folder + add_folder + objname_array + dotmax
  ------ select each object, delete the rest, center it, and save it out as it’s own file
  select searchArr

  max select invert
  delete selection
  searchArr
.position = 
  saveMaxfile save2file quiet:true
  fetchMaxFile quiet:true
)

-Johan

Your code works for me, although it is a bit overcomplicated.
Here is how I would have approached this:

(--start local scope
local master_folder="c:\temp\testbed\" --define output path
--------- declaring additional strings needed to complete file names
local add_folder ="\"
local dotmax = ".max"
local dothsg = ".hsg"
--Snapshot the complete objects database into an array
objname_array = objects as array 
for obj in objname_array do 
(
local save2folder = master_folder + obj.name --create folder name
makeDir save2folder all:true --create the folder on disk
local save2file = save2folder + add_folder + obj.name  + dotmax --build the save name
local exported = save2folder + add_folder + obj.name + dothsg --build the export name
format "Saving to %n" save2file --print what is being saved
saveNodes obj save2file --save the single object to its own file
)
 )--end script



  


                       
  

I tested this with a scene containing a Box, a Sphere, a Target Camera and an Omni Light.
Since Camera01.Target is a LookAt target of Camera01, when the Camera01 was saved, the target came with it. In addition, a separate Camera01.Target folder with just the target was saved. This is a case that could require additional attention if using SaveNodes(). All the others came out as expected.
No need to delete objects or hold/fetch. Same can be done with exporting using
exportFile … selectedOnly:true

Borislav «Bobo» Petrov

Could be better optimized maybe…
You have lots of errors in your code, select invert is not a valid function, delete requires extra parameters, and as bobo said the array shouldn’t be called with $ in front…

Try and see if this works.

searchArr = objects as array
sa_count = searchArr.count
for o in sa_count to 1 by -1 do 
(
  ----- create string for file location
  save2folder = master_folder + objname_array
  save2file = save2folder + add_folder + objname_array + dotmax
  ------ select each object, delete the rest, center it, and save it out as it’s own file
  select searchArr

  max select invert
  delete selection
  searchArr
.position = 
  saveMaxfile save2file quiet:true
  fetchMaxFile quiet:true
)

-Johan

Your solution, though helpful still yields the same error:
upon reaching «select searchArr

» the error reads: —No «»select»» fucntions for «teapot64»

and i had to modify your code at the very top since i only want the obj names to be stored, not any other values, no biggie, let me post what i basterdized:

——- hard coding master_folder for now
——- to do: make this variable input from user from a dialog
master_folder=»u:\testbed\»

——— declaring additional strings needed to complete file names
add_folder =»\»
dotmax = «.max»
dothsg = «.hsg»

— define arrays as arrays
— objname_array = #()
searchArr = #()

——- get names of all objects in the scene and put their names into array «objname_array»
for x = 1 to $objects.count do (append searchArr $objects

.name)
— searchArr = objects as array
sa_count = searchArr.count

—— folder creation based on objs in scene
for y = 1 to $objects.count do (
new_folder = master_folder + searchArr

makeDir new_folder
)

—— save scene state so we can then later restore it
max hold

—— take each object and save it out to it’s own max file, within it’s own folder
—for z = 1 to $objects.count do (
for o in sa_count to 1 by -1 do
(
—— create string for file location
save2folder = master_folder + searchArr

save2file = save2folder + add_folder + searchArr + dotmax
exported = save2folder + add_folder + searchArr + dothsg
—— selection mess
select searchArr
max select invert
delete selection
—— save to a max file
saveMaxfile save2file useNewFile:false quiet:true
fetchMaxFile quiet:true
)

I know it’s a bit overcomplicated, but I think he also wants to export the objects and the exporter takes all objects in the scene in account.. and not only the selected… that’s what the deletion part is for. If I understand correctly… but I think there’s lots of code in this thread to make it work

-Johan

Ok,

That’s off course when it goes wrong, if you only store a name string in the array and not a node reference then selection it won’t work, you need to tell max that your selecting a node not a string!

replace «select searchArr

«
with
select (getNodeByName searchArr)

A object name string value in an array is not the same as a reference to an object…
You have to convert the string to an object reference for select to understand which object.. hence the GetNodeByName function… Personally I would store the node references in the array not the name… I even think it would consume less memory, cause it only holds a pointer, but I could be wrong… bobo?

-Johan

Ok,

That’s off course when it goes wrong, if you only store a name string in the array and not a node reference then selection it won’t work, you need to tell max that your selecting a node not a string!

replace «select searchArr

«
with
select (getNodeByName searchArr)

A object name string value in an array is not the same as a reference to an object…
You have to convert the string to an object reference for select to understand which object.. hence the GetNodeByName function… Personally I would store the node references in the array not the name… I even think it would consume less memory, cause it only holds a pointer, but I could be wrong… bobo?

-Johan

You guys are both Gems! got the damned thing working only to realize that i will now have to write another script that goes thru those folders and then does the export, to do it all in one swift move was greedy, but this is good, thank you both, I really mean it.

Maybe i can have it all happen in the same script, we shall see.

Thanks a bunch!

I know it’s a bit overcomplicated, but I think he also wants to export the objects and the exporter takes all objects in the scene in account.. and not only the selected… that’s what the deletion part is for. If I understand correctly… but I think there’s lots of code in this thread to make it work

-Johan

This is of course WRONG.
You can export one selected object only, by passing the selectedOnly:true flag to the exportFile() method (see my previous post). NO NEED to delete anything from the scene.

(See my «MAXScript For The Masses» DVD, it has a whole chapter exactly on this topic, with code and all)

Borislav «Bobo» Petrov

I know it’s a bit overcomplicated, but I think he also wants to export the objects and the exporter takes all objects in the scene in account.. and not only the selected… that’s what the deletion part is for. If I understand correctly… but I think there’s lots of code in this thread to make it work

-Johan

This is of course WRONG.
You can export one selected object only, by passing the selectedOnly:true flag to the exportFile() method (see my previous post). NO NEED to delete anything from the scene.

(See my «MAXScript For The Masses» DVD, it has a whole chapter exactly on this topic, with code and all)

Bobo,

I love you, but this is a 3rd party exporter, and export selected is ignored by it. I might new to maxscript but i am not stupid.

MB

I know it’s a bit overcomplicated, but I think he also wants to export the objects and the exporter takes all objects in the scene in account.. and not only the selected… that’s what the deletion part is for. If I understand correctly… but I think there’s lots of code in this thread to make it work

-Johan

This is of course WRONG.
You can export one selected object only, by passing the selectedOnly:true flag to the exportFile() method (see my previous post). NO NEED to delete anything from the scene.

(See my «MAXScript For The Masses» DVD, it has a whole chapter exactly on this topic, with code and all)

Bobo,

I love you, but this is a 3rd party exporter, and export selected is ignored by it. I might new to maxscript but i am not stupid.

MB

Oops, sorry, no offense intended!
That sucks then. It might be actually faster to load each Max file you saved and just reexport (since each file will already contain the correct object at the right position etc.) I think it would be faster since the FETCH() call would be relatively slow with large scenes. Unless your scenes are relatively small…

Good luck!

Borislav «Bobo» Petrov

I know it’s a bit overcomplicated, but I think he also wants to export the objects and the exporter takes all objects in the scene in account.. and not only the selected… that’s what the deletion part is for. If I understand correctly… but I think there’s lots of code in this thread to make it work

-Johan

This is of course WRONG.
You can export one selected object only, by passing the selectedOnly:true flag to the exportFile() method (see my previous post). NO NEED to delete anything from the scene.

(See my «MAXScript For The Masses» DVD, it has a whole chapter exactly on this topic, with code and all)

Bobo,

I love you, but this is a 3rd party exporter, and export selected is ignored by it. I might new to maxscript but i am not stupid.

MB

Oops, sorry, no offense intended!
That sucks then. It might be actually faster to load each Max file you saved and just reexport (since each file will already contain the correct object at the right position etc.) I think it would be faster since the FETCH() call would be relatively slow with large scenes. Unless your scenes are relatively small…

Good luck!

No offense taken, I have been at this all day and perhaps did not post what I should have posted. I have started the second part of the script where I getDirectories and then load each file and then proceed to export it out.

Really appreciate all the help, I wished this task was easier, but it is what it is.

Thank you both

M