[ASM68000] rubik's

[heavy]

vbl issue

test
rubiks.S

[Arnaud.68000]

For your VBL issue, change IFD by IFNE, I do not know what IFD does although ASM2 accepts it.
Here are the commands i found, maybe you wanted a similar one :

IFC Conditional Assembly String Equality Test.
IFDEF Conditional Assembly Symbol Definition Test.
IFEQ Conditional Assembly Equal to Zero Test.
IFGE Conditional Assembly Nonnegative Test.
IFGT Conditional Assembly Greater than Zero Test.
IFLE Conditional Assembly Nonpositive Test.
IFLT Conditional Assembly Less than Zero Test.
IFNC Conditional Assembly String Inequality Test.
IFNDEF Conditional Assembly Symbol Not Defined Test.
IFNE Conditional Assembly Unequal to Zero Test.

I read your code and have changed mine using SECTION.
That's clear that the code is more simple, thanks.

WaitVBL:
Why do you prefer use a variable RASTER=300 instead of using parameters DISPLAY_Y+DISPLAY_DY ?
Same question for the copperlist


CopperList

What does ?

dc.w $ffdf,$fffe
dc.w $2c0f,$fffe,$100,0

and why do you duplicate END instruction ?

;Fin
dc.l -2
dc.l -2

[heavy]

IFDEF not recognized by asmtwo (not diff. with IFNE or IFD, try it yourself....)
I have no more vbl issue with the nop added if DEBUG disabled
and no more bug with coplist when adding or removing lines

raster=300 just for testing other values !
for the coplist, just because it's too long and for testing other values too

$ffdf,$fffe => more than 256 lines, just a test when adding lines on coplist

[Arnaud.68000]

I use ASM Two 0.96w and IFNE is recognized.

I have VBL issue with your code using IFD, if i replace by IFNE your code is working

Amazing !

[heavy]

what ??? I talk about IFDEF
IFNE is recognized
IFD is recognized

I also use ASMTwo 0.96w and ASMone 1.0.5 / TFA

too many timing problem (the code, winuae 32, winuae 64)

[Arnaud.68000]

Yes, i confirm IFNE and IFD are recognized with ASW-Two, I tested on a simpe code.

If I replace (also in my code) IFNE by IFD I got a VBL issue... I need to understand.
Maybe a mistake on a variable ?



For timing, I use winUAE 64, A500 OCS, cycle-exact (full)
I don't know if it solves the pb

[Antiriad_UK]

Originally Posted by a/b

Eh, it's simple stuff made for a demo, nothing more nothing less. Didn't check Arnaud's code in detail, french gives me exception #4 ;p, but it looks like a completely different approach.
As I said, dual playfield is the critical part, makes rendering trivial. Simple "script", takes a series of +/- 1-6 (6 possible moves, sign determines rotation direction), selects one of 6 predefined poly/corner sets, generates inner 3d points, does some math and throws everything at the screen, Tops everything off with shiny line vectors to hide ugly edges ;P. The usual convex vector demo stuff.
Playfield priority (and the consequent remapping of colors) is based on the sum of corner z coords.

I really like that

[KONEY]

Did I compile the correct source?

[heavy]

works fine today (original version and modified ones) !
depend of the start config used of winuae (a500 512k/512k ocs). problem with ecs mode

but with original version from github if you uncomment the line 1684 dc.w COLOR04,MIRROR_SCROLL_COLOR
no more cube and a guru

[Arnaud.68000]

Not sure Koney, take the last release
https://github.com/Amiga68000/Rubiks...tag/2021-02-08

In "conf management" section of the code you may find :

;--- Gestion de CONF ---
...
;2021-02-08 - debug fin rotation 90° => RotCour_TournerCoordsSelonAngle90 au lieu de RotCour_Tourner90EtCopierPOLYSauves

I will add a new release with tricks from Heavy to simplify the code.

[Arnaud.68000]

Originally Posted by heavy

works fine today (original version and modified ones) !
depend of the start config used of winuae (a500 512k/512k ocs). problem with ecs mode

but with original version from github if you uncomment the line 1684 dc.w COLOR04,MIRROR_SCROLL_COLOR
no more cube and a guru

Happy to read this.
I spread a new release including your modifications of the coppelist, it seams to be OK now.
Try to uncomment/comment miror lines (#3123) of the copperlist
https://github.com/Amiga68000/Rubiks...tag/2021-02-26

[heavy]

that work (ocs mode)
I only "fix" the asmone issue with bsr/bra (pc) (nothing else modified)
try to use INCDIR directive for the include "font" , I have not the same dir and each time he ask me the "sources:" drive

rubiksf.S

questions : what the thing with JSR and BSR mixed (sometimes bsr, sometimes jsr) ? is jsr faster ? (I found this)
I'm just puzzled with the IFNE/IFD issue... the result should be the same
I prefere IFD if I often use the parameter as ON/OFF with ";" (DEBUG for example)
same thing with the coplist $FF00 instead of $FFFE. is it a skip ? I never seen/used before

try with ifd : so weird. both works for me
rubiksifd.S

another test with only BSR instead of JSR : better or worse?
rubiksbsr.S

[phx]

Originally Posted by heavy

I only "fix" the asmone issue with bsr/bra (pc)

Which assembler accepts that? Probably doesn't do much syntax checking.

questions : what the thing with JSR and BSR mixed (sometimes bsr, sometimes jsr) ? is jsr faster ?

BSR is faster and shorter. And it branches relative to the current PC, while JSR jumps to an absolute address and requires a relocation entry in an AmigaDOS executable.

I'm just puzzled with the IFNE/IFD issue... the result should be the same

Careful! IFD means "if defined". So "ifne SYM" is not the same as "ifd SYM" when SYM is defined as zero. IFD checks if the symbol was defined at all, no matter which value it has.

[heavy]

Originally Posted by phx

Which assembler accepts that? Probably doesn't do much syntax checking.

ASMTwo

accept also ds xx,yy instead of dcb xx,yy

BSR is faster and shorter. And it branches relative to the current PC, while JSR jumps to an absolute address and requires a relocation entry in an AmigaDOS executable.

he also use jsr (pc)

Careful! IFD means "if defined". So "ifne SYM" is not the same as "ifd SYM" when SYM is defined as zero. IFD checks if the symbol was defined at all, no matter which value it has.

yes. I removed the value before using it.

@arnaud.68000
do you know this doc -> pdf

put your code in Fast with the first Section ,CODE
on winuae config with z-fast that work, else vbl issue

[KONEY]

Originally Posted by Arnaud.68000

Not sure Koney, take the last release

Now it works. I wonder why you're not using github as it's meant, like one can clone the project at its latest state...

anyway I love this thing and I'll study it for sure!

If you're interested I fixed a few things to allow Vasm digest it:

lines 440, 2027, 2088 removed the "(pc)" from BRA argument
line 1223 removed TAB before constant declaration

[heavy]

Originally Posted by KONEY

If you're interested I fixed a few things to allow Vasm digest it:

lines 440, 2027, 2088 removed the "(pc)" from BRA argument
line 1223 removed TAB before constant declaration

same fix than asmone, see previous posts

[Arnaud.68000]

Thanks for you returns and interests

I will try to implement all your suggestions and read your codes

do you know this doc -> pdf

--> ERR 404

Now it works. I wonder why you're not using github as it's meant, like one can clone the project at its latest state...

I'm not confortable using Github, this is the first time I was using it, I will read tutos ;-)

For the copperlist, I resued it from the sinescroll source of YRAGAEL and changed nothing. To be honest, I have to improve my undersanding of copperlist.
https://www.stashofcode.fr/tag/sine-scroll/

I need also to use precise and optimized instructions, asmTwo is a little bit permissive ;-)

rubiksbsr.S

is now my new version, hope that include all your modifications rubiks.S and rubiksifd.S

[heavy]

link fixed (crap after .pdf)
PDF

[Arnaud.68000]

Originally Posted by heavy

link fixed (crap after .pdf)
PDF

No I did not know, seems to be usefull