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

multiple NatMap module instances #3

Open
vzaliva opened this issue Oct 30, 2019 · 0 comments
Open

multiple NatMap module instances #3

vzaliva opened this issue Oct 30, 2019 · 0 comments

Comments

@vzaliva
Copy link
Owner

vzaliva commented Oct 30, 2019

We instantiate FMap with natural numbers as a key several time:

$ ag FMapAVL.Make                                       12:41:16
ml/extracted/Memory.ml
438:  module IM = FMapAVL.Make(Z_as_OT)

coq/SigmaHCOL/SigmaHCOLRewriting.v
41:Module NM := FMapAVL.Make(Nat_as_OT).

coq/DSigmaHCOL/TypeSig.v
45:Module TM := FMapAVL.Make(Nat_as_OT).

coq/MSigmaHCOL/Memory.v
50:Module NM := FMapAVL.Make(Nat_as_OT).
~/s/helix (type-signatures|…) $                    

Perhaps this is unecessary as all the module instances are typed by the key only, and do not depend on the lement type.

Pehaps we should create Util/NatMap as use it everywhere.

zoickx added a commit that referenced this issue Jan 30, 2023
zoickx added a commit that referenced this issue Jan 30, 2023
zoickx added a commit that referenced this issue Jan 30, 2023
zoickx added a commit that referenced this issue Feb 1, 2023
Additionally (see comment in .drone.yml):
1) Cache will now be built automatically if none was found
   (this required manual intervention before)
2) Cache will now be rebuilt automatically if cache-defining files
   have been changed
3) It is now possible to manually force a cache rebuild by *promoting*
   a build (can be done from the web interface)
4) *Any* cron job on the branch will now trigger a cache rebuild

All of the above logic is done using a (hacky for Drone, but as clean
as I could get it for shell) "one-liner" and can, of course, easily be
changed.

Squashed commit of the following:

commit 7d582c10fdc048937be792ed884ca23da98b6541
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 21:32:18 2023 +0200

    Remove temporary diagnostic steps

commit ae466ac
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 21:12:52 2023 +0200

    Revert "Slim Dockerfile for testing"

    This reverts commit a368a0e.

commit 40d793d
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 21:11:00 2023 +0200

    Add a missing dependency

commit d4f403b
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 21:02:41 2023 +0200

    More test commands

commit a368a0e
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 20:57:42 2023 +0200

    Slim Dockerfile for testing

commit 2640d7c
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 20:21:44 2023 +0200

    Re-enable slack notifications

commit b5dac29
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 20:11:48 2023 +0200

    Remove redundant step run condition

commit f87b601
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 20:11:37 2023 +0200

    Better comments [ci skip]

commit 0251305
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 19:54:40 2023 +0200

    Fix drone cache naming scheme

commit 3cc6736
Author: Ilia Zaichuk <[email protected]>
Date:   Wed Feb 1 19:51:03 2023 +0200

    Hacky shell for drone cache

commit c272db7
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 17:39:32 2023 +0200

    Fix forgotten variable

commit 78ba7f0
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 17:21:55 2023 +0200

    Revert "Ambitious drone global envrionment attempt"

    This reverts commit adb451a.

commit adb451a
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 17:18:28 2023 +0200

    Ambitious drone global envrionment attempt

commit 3de856b
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 17:08:09 2023 +0200

    Drone template semifinal

commit d547e0d
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 17:00:13 2023 +0200

    Drone templates #3.2

commit c4718df
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 16:57:40 2023 +0200

    Drone templates #3.1

commit 5b3ea36
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 16:55:11 2023 +0200

    Drone templates #3

commit 9bb02ff
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 16:49:01 2023 +0200

    Revert "Another attempt at drone templates"

    This reverts commit 2d57178.

commit 2d57178
Author: Ilia Zaichuk <[email protected]>
Date:   Mon Jan 30 16:43:40 2023 +0200

    Another attempt at drone templates

commit 5c90a9b
Author: Ilia Zaichuk <[email protected]>
Date:   Fri Jan 27 20:39:49 2023 +0200

    Test a variable *directly* from the docs

commit 8afc14d
Author: Ilia Zaichuk <[email protected]>
Date:   Fri Jan 27 16:26:39 2023 +0200

    Decrease redundancy in step names

commit 22ed1d0
Author: Ilia Zaichuk <[email protected]>
Date:   Fri Jan 27 15:13:41 2023 +0200

    Add more variable tests to drone pipeline

commit 62891da
Author: Ilia Zaichuk <[email protected]>
Date:   Fri Jan 27 15:12:30 2023 +0200

    Set up test drone pipeline

commit b2eeae0
Author: Ilia Zaichuk <[email protected]>
Date:   Fri Jan 27 15:06:40 2023 +0200

    Lint CI-related files
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

1 participant