Packaging ats3d

[gmhwxi]

Packaging ATS code is largely undocumented at this point.

I suggest that we use ats3d (https://github.com/ashalkhakov/ats3d)
as a concrete example to illustrate some key steps involved in forming
an npm-package for ATS code.

[Artyom Shalkhakov]

I'll do it, will you wait for evening? Oh, I wanted to say: will you wait for 6 hours?

I guess at this point, just creating a dummy package with sources will be enough? I have packaged some tiny library on NPM once.

[gmhwxi]

Sure :)

[Artyom Shalkhakov]

I made this package:

https://www.npmjs.com/package/atscntrb-as-ats3d

This is what I did:

$ cd ats3d

$ npm init

...answered some questions (most notably, named the package [atscntrb-<author-initials>]; added ATS to the list of keywords)

...tried to figure out how .npmignore work (it didn't help me exclude a leftover README.md~ file, no idea why)

$ npm publish

That was very simple!

What are the mylibies/mydepies files? I'm still struggling to understand how external package dependencies are handled.

[gmhwxi]

Thanks!

Just tried it:

/home/ubuntu/workspace/mytmp/node_modules/atscntrb-as-ats3d/src/DATS/quaternion.dats: 7079(line=343, offs=11) -- 7093(line=343, offs=25): error(3): the symbol [.length] cannot be resolved due to too many matches:
D2ITMcst(length_vec3i) of 10
D2ITMcst(length_vec3f) of 10

It seems that a type-annotation is needed for the if-expression.

Also, there are files like xyz~ in the package. You may want to remove them.

[Artyom Shalkhakov]

On Thursday, April 27, 2017 at 9:45:40 PM UTC+6, gmhwxi wrote:

Thanks!

Just tried it:

/home/ubuntu/workspace/mytmp/node_modules/atscntrb-as-ats3d/src/DATS/quaternion.dats: 7079(line=343, offs=11) -- 7093(line=343, offs=25): error(3): the symbol [.length] cannot be resolved due to too many matches:
D2ITMcst(length_vec3i) of 10
D2ITMcst(length_vec3f) of 10

Well, this isn't supported yet, sorry! I'm going to give quaternions a try once a proper scene description is done. I'm currently working on test04 (will break it into separate modules soon); the goal is to create a feature-rich software rasterizer that mimics the OpenGL shading pipeline (I'm following https://github.com/ssloy/tinyrender as an inspiration).

 

It seems that a type-annotation is needed for the if-expression.

Also, there are files like xyz~ in the package. You may want to remove them.

This is what I was trying to avoid. But maybe it's there because it got into version control... We'll see.

[gmhwxi]

I made some changes here:

https://github.com/githwxi/ATS-Postiats-frozen/tree/master/projects/SMALL/atscntrb-as-ats3d-2017-04-27

#####

First, please change

staload "../some-path"

to

staload "./../some-path"

The first '.' means the directory in which the file containing 'staload "./../some-path"
is located.

I also added: mylibies.hats and mylibies_link.dats. The latter is for generating code
needed  at link-time.

Various occurrences of "../CATS/vector.cats" are changed into

"atscntrb-as-ats3d/src/CATS/vector.cats"

#####

[Artyom Shalkhakov]

2017-04-28 9:11 GMT+06:00 gmhwxi <gmh...@gmail.com>:
>
> I made some changes here:
>
> https://github.com/githwxi/ATS-Postiats-frozen/tree/master/projects/SMALL/atscntrb-as-ats3d-2017-04-27
>
> #####
>
> First, please change
>
> staload "../some-path"
>
> to
>
> staload "./../some-path"
>
> The first '.' means the directory in which the file containing 'staload
> "./../some-path"
> is located.
>
> I also added: mylibies.hats and mylibies_link.dats. The latter is for
> generating code
> needed at link-time.
>
> Various occurrences of "../CATS/vector.cats" are changed into
>
> "atscntrb-as-ats3d/src/CATS/vector.cats"
>
> #####
>
>

Thank you very much for the suggestions! I will take a look this weekend.

> --
> You received this message because you are subscribed to the Google Groups
> "ats-lang-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to ats-lang-user...@googlegroups.com.
> To post to this group, send email to ats-lan...@googlegroups.com.
> Visit this group at https://groups.google.com/group/ats-lang-users.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/ats-lang-users/43d61c91-6609-4a14-ba2e-25b81f1faed0%40googlegroups.com.



--
Cheers,
Artyom Shalkhakov

[gmhwxi]

Sounds good!

I only managed to make test01 work.

For packaging, mylibies.hats is considered the entry point for the package.
One should not be asked or should be discouraged to load files in SATS and
DATS directly. If needed, please use ifdef-guards to control loading of files.

In your package, there are also CATS files. When these files are mentioned
in SATS-files (or even DATS-files), please use a path like the following one:

<module-name>/.../CATS/foo.cats

For instance, you can use the following one:

atscntrb-as-ats3d/src/CATS/vector.cats

In your case, one also needs to compile various DATS-files
in order to use ats3d. So I added mylibats_link.hats (where 'link'
means that this file is for producing something needed at link-time).

As for testing, tests inside TEST are for so-called internal testing.
For testing packages externally, please visit:

https://github.com/githwxi/ATS-Postiats-test/tree/master/package

On Thursday, April 27, 2017 at 11:17:50 PM UTC-4, Artyom Shalkhakov wrote:

> email to ats-lang-users+unsubscribe@googlegroups.com.
> To post to this group, send email to ats-lang-users@googlegroups.com.

[Artyom Shalkhakov]

Hongwei,

Sorry for this taking so much time but... at least I've prepared some images this time:

https://github.com/ashalkhakov/ats3d/

Onwards to getting your suggestions to work!

> email to ats-lang-user...@googlegroups.com.
> To post to this group, send email to ats-lan...@googlegroups.com.

[Artyom Shalkhakov]

Finally got around to tackling this issue. The new repo is here:

https://github.com/ashalkhakov/atscntrb-as-ats3d

had to rename it to keep things totally uniform.

It should be working for you if you do:

$ npm i atscntrb-as-ats3d

$ cd node_modules/atsnctrb-as-ats3d/TEST

$ make regress

It should create some files, e.g. test04.exe generates PPM images in data/ subdirectory (they all are named like X-out.ppm, where X is the name of the corresponding Wavefront OBJ file).