Jump to content

acet-help not loading from Lisp


Recommended Posts

Posted
yes I did run it and noticed that depending on the selection (interior door, cabinet door etc), the list choices change. Interesting.

 

What ya think about this?

If Comp Qty is picked at 2, then default 3 4 & 5 to blank values, or maybe a "dash".

If the value is blank, the layers are turned off.

 

comp-pid.jpg

 

 

 

-SlimP

 

I think you got the idea now. Notice there are fewer lists to deal with. Cannot get much better than that.

 

The big question is now. Can you interpret my program to see how I did it?

 

And can you fiqure out how you are going to do it?

 

I see you are off to a good start.

 

I would suggest using mode_tile instead of blanking.

  • Replies 91
  • Created
  • Last Reply

Top Posters In This Topic

  • The Buzzard

    36

  • slimpickinz

    34

  • CAB

    21

  • gile

    1

Top Posters In This Topic

Posted Images

Posted

I believe I will attempt this method.

I see the code and it looks pretty clear.

This project is scheduled for a review presentation to the department Tuesday.

I may have to stick with my v1.0 for that, however I will be moving on with the new and improved version for the final product.

 

I would suggest using mode_tile instead of blanking.
Yes, mode_tile is the answer here.

 

Kudos Mate!

Good night.

Posted
I believe I will attempt this method.

I see the code and it looks pretty clear.

This project is scheduled for a review presentation to the department Tuesday.

I may have to stick with my v1.0 for that, however I will be moving on with the new and improved version for the final product.

 

Yes, mode_tile is the answer here.

 

Kudos Mate!

Good night.

 

If you need any clarification to anything in that code, I will be more than happy to give you answers. If you read my comments, They do direct you where the code go next. This I realize is a bit time consuming, But you will learn this. There is no sense to placing random code in your program unless you understand it fully. That is why I want you to do the homework for the sake of understanding all this.

 

So far you have done well as I thought you would.

Great work and good luck.

The Buzzard

Posted

I have started to sort through the IDS app. I think the best way for me is to start removing sections, one by one, and running it along the way, to make sure I am catching on. When I get the basic main section working, I will post.

You do not know how much your help has meant to me so far.

I will keep you posted.

Thanks!

Posted
I have started to sort through the IDS app. I think the best way for me is to start removing sections, one by one, and running it along the way, to make sure I am catching on. When I get the basic main section working, I will post.

You do not know how much your help has meant to me so far.

I will keep you posted.

Thanks!

 

Not a problem and glad to help,

 

You are doing the work, I am only pointing you in certain directions.

Posted

Tell you what.

 

I will make this easier on you. I have removed everything in the IDS.lsp and IDS.dcl that does not pertain to what you need for the list example.

 

Attached is EXAMPLE.lsp and EXAMPLE.dcl

There is no output function in this lisp. This will only show the list operation and help make it easier for you to understand.

 

I left the comments in for you.

Hope this helps.

The Buzzard

EXAMPLE.zip

Posted

Is there something better for building these DCLs than VLIDE?

GEEZ! What a pain!

Posted
Is there something better for building these DCLs than VLIDE?

GEEZ! What a pain!

 

 

I use VLIDE every now and then. I have a preference to Notepad.

I guess I am just use to it.

Posted

Just saw the example.

Man, you just saved me a couple hours at least!

Thanks.

 

I am a visual object oriented person.

How do you preview your DCL as you go?

Just load the lsp and run it?

Posted
Just saw the example.

Man, you just saved me a couple hours at least!

Thanks.

 

I am a visual object oriented person.

How do you preview your DCL as you go?

Just load the lsp and run it?

 

I basically start with the dcl written in Notepad and write the portion needed in the lisp to run the program to see the dcl. I just adjust it from there as I go. I will then build the program from around all this. It pays to have a plan or goal set in mind as to what your program is to accomplish. Sometimes as you learn new ways to do something during the course of writting your program may seem like a hinderence, But pays off in the long run. Most of the code I have put together are copies and paste from other codes I have written.

 

Sort of like assembly line coding.

No point in writting code all over if you already have some good stuff.

In most cases I also have the comments in place and adjust it as needed.

If you take notice to the ////////////////////////////////////// between functions

helps to see where a function starts and ends.

It is these borders that designate a function that will do a specific task

and can be used in other codes needing a function for the same thing.

Much faster to spot your functions. It helps to have a heading or title as well.

Posted

I just realized I can do this whole thing with radio buttons only.

Would that be much simplier? No lists to contend with....

Is it easier to control variables with radio buttons?

I have a sketch of the final product dialog.

I will post it directly.

Posted
I just realized I can do this whole thing with radio buttons only.

Would that be much simplier? No lists to contend with....

Is it easier to control variables with radio buttons?

I have a sketch of the final product dialog.

I will post it directly.

 

Not sure about that. As long as your dialog does not increase in size where it may be two big for the monitor. I still think the list method is ideal, But you seem pressed for time.

 

Its up to you as again this is your program. I am only making suggestions. You will need to evaluate this and see how well this will work. I do not mean to imply anything by this, But you really need to get past things that may seem difficult. You are putting a limit on yourself and this I feel is not a good thing to do because the programming has limits placed on it as well.

 

Your choice.

Posted

I see what you mean. I will work with what I now have, just make a few enhancements.

Actually it is working fine now, good enough for first viewing on Tuesday.

This is a rough crowd, and I am sure they will find some things to improve upon.

Posted
I see what you mean. I will work with what I now have, just make a few enhancements.

Actually it is working fine now, good enough for first viewing on Tuesday.

This is a rough crowd, and I am sure they will find some things to improve upon.

 

Thats OK,

 

Just do not settle for radio buttons because it easier. Sometimes excessive use of these can look tacky even if they work properly. Also you miss out on some better complex programming methods. Take your programming to the max. You will impress these people more. As I mentioned they will require more room than you are currently using with the popup lists you had before. Seems to me a backward direction if any. These methods are from some of the best programmers on this site.

 

You cannot go wrong here.

Take your programming up a few notches.

Posted

Came up with a way to set the components to none without using the "Select One" option. This works much better. Looks cleaner as well.

Still having problems with the "obscure variables" thing...

This is kicking my butt.

Using my code, can you please show just one variable like that, I can take it from there.

I do not remember any other coding I have learned (CSS, PHP, AS3 etc) as being this difficult to figure...

 

Thanks

SlimP-

pid907-1.LSP

pid4.dcl

PID3.zip

Posted
Came up with a way to set the components to none without using the "Select One" option. This works much better. Looks cleaner as well.

Still having problems with the "obscure variables" thing...

This is kicking my butt.

Using my code, can you please show just one variable like that, I can take it from there.

I do not remember any other coding I have learned (CSS, PHP, AS3 etc) as being this difficult to figure...

 

Thanks

SlimP-

 

I looked over what you have. I would write this in a different manner. The format is very easy for me to get confused. You are getting your data thru one function outside of the main function. I would have done it all in the main function where all the variables would be obscure. I would then upon clicking enter send this data to a variable function where the value are passed to another variable to be used in the program and can be declared. After that the data would then be sent to an output function to be processed.

 

I understand the programs intention as to whats suppose to happen, But I write my programs sort of different. I would need some time to study yours more thoroughly and put it into a format with comments that we can both understand. This will involve some time. You have alot of popup lists here.

 

You are also using alot of if statements for each variable group when one conditional would suffice. Using alot of if's will slow your program down because there is more to process this way.

 

If you have something you can get by with for your Tuesday meeting, I will see what I can come up with. Please understand I need to get familar with your code before all this can happen.

Posted

Ok,

Just to get you started. I still recommend a program overhaul.

 

I edited your program file for one edit box concerning the compressor qty. This popup now remembers the last selection. See the attached zip file and compare the edited lines with yours.

 

Here are the lines that were changed or added and indicated by ; This is where I edited

 

 Also take note that atof was changed to atoi
(setq c:ompqty1 (atoi (get_tile "compqty1")) c:ompqty1:DEF c:ompqty1)    ; This is where I edited

 

This is a new line added.
(if c:ompqty1:DEF (set_tile "compqty1" (itoa c:ompqty1:DEF)))                  ; This is where I edited

 

(setq c:ompqty1 (fix c:ompqty1))                                   ; This is where I edited

 

(setq c:ompqty1 (nth c:ompqty1 NAMEScompqty))                         ; This is where I edited

 

This is a new line added.
(setq compqty1 c:ompqty1)                                              ; This is where I edited

 

 

Look for all these edits/additions and just do the same thing to the rest of your program.

 

I almost forgot to mention. The obscure variables here is

c:ompqty1 & c:ompqty1:DEF

These variables you do not declare local or the program will not remember their last value.

 

I pass the value by

(setq compqty1 c:ompqty1)

So you would declare compqty1 as a local variable. This variable is the one that should lose its value so it does not conflict with other programs that may use a variable by the same name.

 

I hope you can understand all this.

 

One last thing, Try to keep your DCL and LSP file names the same so others will know they belong together.

 

Good Luck

Changed.zip

Posted

Wish had more time to spend on this one. Maybe tomorrow?

 

You can test the DCL from VLIDE with the pulldown menu Tools/Interface Tools/Preview DCL in Editor

 

See attached pdi4.dcl which is your dcl with some house cleaning. Should run with your lisp

See "pdi4 CAB.dcl" which I changed, Added check boxes in some places & reconfigured the DCL

code. Use VLIDE to see it. It will not operate with your lisp. Had no time to modify that.

 

Just food for thought. :)

 

 

PS does not reflect you work today.

pid4.dcl

pid4 CAB.dcl

Posted
Wish had more time to spend on this one. Maybe tomorrow?

 

You can test the DCL from VLIDE with the pulldown menu Tools/Interface Tools/Preview DCL in Editor

 

See attached pdi4.dcl which is your dcl with some house cleaning. Should run with your lisp

See "pdi4 CAB.dcl" which I changed, Added check boxes in some places & reconfigured the DCL

code. Use VLIDE to see it. It will not operate with your lisp. Had no time to modify that.

 

Just food for thought. :)

 

 

PS does not reflect you work today.

 

Since those are a Yes/No optional type answer thats the way to go CAB, Nice

I should have spent more time looking at that, But I got more involved in first few lists.

Its good to have more eyes spot this stuff.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


×
×
  • Create New...