text:
get() method of unexecuted checkpoint not raising exception

Snakemake version
Occurs in 8.16.0 and 8.13.0, does not occur in 7.32.4.
Describe the bug
I am running into an issue where under certain circumstances, a checkpoint's get() method does not raise the expected IncompleteCheckpointException when it has not yet been run. I've been able to create minimal example below, The problem happens in the input function of the problem rule, which reads the output of the my_checkpoint checkpoint rule.
The example runs fine starting from scratch. However, if I create an empty file for the problem rule's output file, then execute a rule that takes that file as an input, the call to checkpoints.my_checkpoint.get()  succeeds even though the checkpoint has not been run.
Several warnings like the following are generated, which may or may not be related:
RuntimeWarning: coroutine 'DAG.sanitize_local_storage_copies' was never awaited

Minimal example
import os, sys

checkpoint my_checkpoint:
    output: 'checkpoint.txt'
    shell: 'echo 1 2 > {output}'

rule make_problem_input:
    output: 'input-{i}.txt'
    shell: 'echo data-{wildcards[i]} > {output}'

def my_input_func(wc):
    print('!!!!!!!!!! input function called', file=sys.stderr)
    checkpoint_file = checkpoints.my_checkpoint.get().output[0]
    assert os.path.exists(checkpoint_file), f'{checkpoint_file} does not exist!'
    print('!!!!!!!!!! checkpoint file exists', file=sys.stderr)
    data = open(checkpoint_file).read()
    nums = data.strip().split()
    return expand(rules.make_problem_input.output, i=nums)

rule problem:
    input: my_input_func
    output: 'problem.txt'
    shell: 'cat {input} > {output}'

rule final:
    input: 'problem.txt'
    output: 'final.txt'
    shell: 'cat {input} > {output}'

Logs
With directory empty except for the Snakefile:
snakemake --profile none final.txt

Assuming unrestricted shared filesystem usage.
Building DAG of jobs...
!!!!!!!!!! input function called
!!!!!!!!!! input function called
/home/jared.lumpe/opt/mambaforge/envs/signature-finder/lib/python3.12/site-packages/snakemake/dag.py:1612: RuntimeWarning: coroutine 'DAG.sanitize_local_storage_copies' was never awaited
  self.sanitize_local_storage_copies()
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Using shell: /usr/bin/bash
Provided cores: 80
Rules claiming more threads will be scaled down.
Job stats:
job              count
-------------  -------
final                1
my_checkpoint        1
problem              1
total                3

Select jobs to execute...
Execute 1 jobs...

[Fri Aug  9 16:52:43 2024]
localcheckpoint my_checkpoint:
    output: checkpoint.txt
    jobid: 2
    reason: Missing output files: <TBD>
    resources: tmpdir=/tmp
DAG of jobs will be updated after completion.

[Fri Aug  9 16:52:43 2024]
Finished job 2.
1 of 3 steps (33%) done
!!!!!!!!!! input function called
!!!!!!!!!! checkpoint file exists
/home/jared.lumpe/opt/mambaforge/envs/signature-finder/lib/python3.12/site-packages/snakemake/dag.py:1612: RuntimeWarning: coroutine 'DAG.sanitize_local_storage_copies' was never awaited
  self.sanitize_local_storage_copies()
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Select jobs to execute...
Execute 2 jobs...

[Fri Aug  9 16:52:43 2024]
localrule make_problem_input:
    output: input-2.txt
    jobid: 6
    reason: Missing output files: input-2.txt
    wildcards: i=2
    resources: tmpdir=/tmp

[Fri Aug  9 16:52:43 2024]
localrule make_problem_input:
    output: input-1.txt
    jobid: 5
    reason: Missing output files: input-1.txt
    wildcards: i=1
    resources: tmpdir=/tmp

[Fri Aug  9 16:52:43 2024]
Finished job 6.
2 of 5 steps (40%) done
[Fri Aug  9 16:52:43 2024]
Finished job 5.
3 of 5 steps (60%) done
Select jobs to execute...
Execute 1 jobs...

[Fri Aug  9 16:52:43 2024]
localrule problem:
    input: input-1.txt, input-2.txt
    output: problem.txt
    jobid: 1
    reason: Missing output files: problem.txt; Input files updated by another job: input-1.txt, input-2.txt
    resources: tmpdir=/tmp

[Fri Aug  9 16:52:43 2024]
Finished job 1.
4 of 5 steps (80%) done
Select jobs to execute...
Execute 1 jobs...

[Fri Aug  9 16:52:43 2024]
localrule final:
    input: problem.txt
    output: final.txt
    jobid: 0
    reason: Missing output files: final.txt; Input files updated by another job: problem.txt
    resources: tmpdir=/tmp

[Fri Aug  9 16:52:43 2024]
Finished job 0.
5 of 5 steps (100%) done
Complete log: .snakemake/log/2024-08-09T165243.478887.snakemake.log

Remove all files and .snakemake/ directory, touch problem.txt, and rerun:
Assuming unrestricted shared filesystem usage.
Building DAG of jobs...
!!!!!!!!!! input function called
!!!!!!!!!! input function called
InputFunctionException in rule problem in file /home/jared.lumpe/tmp/24-08-09-snakemake-bug/Snakefile, line 20:
Error:
  AssertionError: checkpoint.txt does not exist!
Wildcards:

Traceback:
  File "/home/jared.lumpe/tmp/24-08-09-snakemake-bug/Snakefile", line 14, in my_input_func (rule problem, line 40, /home/jared.lumpe/tmp/24-08-09-snakemake-bug/Snakefile)

Possibly the same as #2958

Also see #2957 for RuntimeWarning: coroutine 'DAG.sanitize_local_storage_copies' was never awaited

repo: snakemake/snakemake


text:
feat(): add network property to strategy

Added network property which will be later used to display the network in the space settings for each strategy.

I think we should auto populate this field with the space network otherwise type validation would fail

source: gharchive/pull-request

authors: bonustrack midgerate

repo: snapshot-labs/snapshot-hub

license: MIT

license_type: permissive

license_source: github-api


text:
[l2-deversifi] L2-deversifi strategy

Implementation of l2-deversifi strategy

@plutoegg please resolve conflicts :)

Done!

Does another snapshot team member need to merge? We are keen to test it out with the UI :)

@plutoegg Thanks for your patience will get the strategy up in a moment

@plutoegg Strategy is live here: https://snapshot.org/#/strategy/l2-deversifi Thanks :)

authors: ChaituVR bonustrack nicob21 plutoegg

repo: snapshot-labs/snapshot-strategies


text:
Add balance-with-min strategy

Changes proposed in this pull request:

If balance is below minimum the voting power is replaces with zero

@bonustrack How long does it usually take until it's live after merge.

@bonustrack How long does it usually take until it's live after merge.

Could the this strategy also notify the voter if the balance is below minimum?

Could the this strategy also notify the voter if the balance is below minimum?

@samuveth When there is a new strategy, the frontend need to be redeployed, i'm doing this now it should be live in 5mn. When user will vote and have less than the minimum he will see 0 as voting power, so somehow he will get notified.

@samuveth When there is a new strategy, the frontend need to be redeployed, i'm doing this now it should be live in 5mn. When user will vote and have less than the minimum he will see 0 as voting power, so somehow he will get notified.

Does this affect both voting and proposing? Would it be possible to support a similar strategy that only blocks proposing for users below the threshold?

authors: bonustrack samuveth sinasabet81


text:
leaderboard users list rendred fixed

Summary
conditional render fixed for listing the users data in the leadboard.

Closes: #339

Will be fixed via #341

repo: snapshot-labs/sx-monorepo


text:
support Lemmy

Background:

https://join-lemmy.org/
https://join-lemmy.org/docs/en/administration/federation_getting_started.html
https://socialhub.activitypub.rocks/t/help-improving-federation-between-lemmy-and-other-projects/2308

I tried federating this like to this post on lemmy.ml (running BE 0.16.7) just now, and the AP inbox delivery to https://lemmy.ml/u/OptimusPrime/inbox returned 400 with invalid type: map, expected a string representing an URL at line 1 column 218. Bridgy Fed log here. Full AS2 object below.
Lemmy's design looks like Reddit, with up/downvotes instead of likes, so I don't know that it actually supports likes, but the error message sounds unrelated to that, more like maybe it can only handle string URLs for actors instead of compound objects.
{
  "published": "2023-01-17T13:03:52-08:00",
  "content": "likes [lemmy.ml](http://lemmy.ml/)",
  "url": "https://fed.brid.gy/r/https://snarfed.org/2023-01-17_lemmy-ml",
  "actor": {
    "url": "https://fed.brid.gy/r/https://snarfed.org/",
    "image": {
      "url": "https://secure.gravatar.com/avatar/947b5f3f323da0ef785b6f02d9c265d6?s=96&d=blank&r=g",
      "type": "Image"
    },
    "type": "Person",
    "name": "Ryan Barrett",
    "icon": {
      "url": "https://secure.gravatar.com/avatar/947b5f3f323da0ef785b6f02d9c265d6?s=96&d=blank&r=g",
      "type": "Image"
    },
    "id": "https://fed.brid.gy/snarfed.org",
    "preferredUsername": "[snarfed.org](http://snarfed.org/)"
  },
  "@context": "https://www.w3.org/ns/activitystreams",
  "type": "Like",
  "object": "https://lemmy.ml/post/713841",
  "id": "https://fed.brid.gy/r/https://snarfed.org/2023-01-17_lemmy-ml",
  "cc": [
    "https://lemmy.ml/u/OptimusPrime",
    "https://lemmy.ml/c/privacy",
    "https://www.w3.org/ns/activitystreams#Public"
  ],
  "to": [
    "https://www.w3.org/ns/activitystreams#Public"
  ]
}

From https://socialhub.activitypub.rocks/t/help-improving-federation-between-lemmy-and-other-projects/2308/22 :

You are right this isnt complicated, in fact we are already doing the same thing in some other places. Unfortunately I dont have time to work on this, but if you want to make a contribution I would gladly guide you through the code.

Progress is a new error message. After #442, we're now getting HTTP 400 data did not match any variant of untagged enum PersonOrGroup on this Like:
{
  "published": "2023-10-11T21:12:28-07:00",
  "content": "Likes https://lemmy.world/post/6663494;..\n
\nA meme for math people – Lemmy.world\n
",
  "url": "https://fed.brid.gy/r/https://snarfed.org/2023-10-11_a-meme-for-math-people-lemmy-world",
  "actor": "https://fed.brid.gy/snarfed.org",
  "id": "https://fed.brid.gy/r/https://snarfed.org/2023-10-11_a-meme-for-math-people-lemmy-world",
  "@context": "https://www.w3.org/ns/activitystreams",
  "type": "Like",
  "object": "https://lemmy.world/post/6663494",
  "cc": [
    "https://lemmy.ml/c/memes",
    "https://lemmy.world/u/hypertown",
    "https://www.w3.org/ns/activitystreams#Public"
  ],
  "to": [
    "https://www.w3.org/ns/activitystreams#Public"
  ]
}

And for this reply, 400 Client Error: Bad Request for url: https://lemmy.world/u/hypertown/inbox ; deserializing body: ...
{
  "id": "https://fed.brid.gy/r/https://snarfed.org/2023-10-11_a-meme-for-math-people-lemmy-world-3#bridgy-fed-create",
  "actor": "https://fed.brid.gy/snarfed.org",
  "published": "2023-10-12T04:17:52.463875+00:00",
  "@context": "https://www.w3.org/ns/activitystreams",
  "type": "Create",
  "object": {
    "published": "2023-10-11T21:17:47-07:00",
    "content": "\n
\nnice\n
",
    "url": "https://fed.brid.gy/r/https://snarfed.org/2023-10-11_a-meme-for-math-people-lemmy-world-3",
    "id": "https://fed.brid.gy/r/https://snarfed.org/2023-10-11_a-meme-for-math-people-lemmy-world-3",
    "type": "Note",
    "attributedTo": "https://fed.brid.gy/snarfed.org",
    "inReplyTo": "https://lemmy.world/post/6663494",
    "cc": [
      "https://lemmy.ml/c/memes",
      "https://lemmy.world/u/hypertown",
      "https://www.w3.org/ns/activitystreams#Public"
    ],
    "tag": [
      {
        "type": "Mention",
        "href": "https://lemmy.world/u/hypertown"
      }
    ],
    "to": [
      "https://www.w3.org/ns/activitystreams#Public"
    ]
  },
  "to": [
    "https://www.w3.org/ns/activitystreams#Public"
  ]
}

Progress is a new error message. After #442, we're now getting HTTP 400 data did not match any variant of untagged enum PersonOrGroup on this Like:

This also happens on follows.

More on this error:

https://socialhub.activitypub.rocks/t/we-have-created-documentation-on-how-exactly-lemmy-federation-works/1085/46?u=snarfed
https://github.com/LemmyNet/Lemmy/issues/2756

Is there any progress on this at all? The recent lemmy update made some heavy changes and improved the API significantly.

Exciting! Last I checked, we were blocked on Lemmy errors, details above. Feel free to test again and see if anything has changed!

Lemmy maintainer here, just came across this. I threw this activity into Lemmy tests, and it fails because the Create activity is missing cc field. You need to copy that from the inner Note.
By the way, Lemmy users view Bridgy Fed generally positively, as they consider all posts to be public anyway. So theres no need for complicated opt-in solutions in case of Lemmy. Here you can see the discussion.

@snarfed This sound's actionable (or possibly even solved? I think something similar came up regarding another software), so I'm removing the blocked label since we should at least re-test if the same snag is being hit.

Yes! We now duplicate to and cc between object and activity, so this might now work. Worth re-testing!

source: gharchive/issue

authors: Nutomic Tamschi gaviscapes snarfed

repo: snarfed/bridgy-fed


text:
Gradle Sample?

I see that you have Maven in your examples, would is be possible to have gradle examples as well?

+1
I've spent some time trying to get this running in Gradle w/out much luck.

+1
Yes, a sample for gradle would be perfect.

+1

ya should be great

+1

authors: KyleChamberlin glebiuskv marccollin miguelfc neskartu theironcook xuchang86

repo: snicoll-scratches/spring-boot-daemon


text:
Introduce singleton API client with retry policy

Introduce singleton API client with retry policy
Remove all refresh token logic. Turns out the cookie we got back from /refreshtoken didn't do shit. And by that i mean its expiration date wasn't extended.

Merging this without code review because YOLO.

repo: snobu/destreamer


text:
Error on windows

Hi.
I am running a workflow, that runs on Mac and Windows.
Mac works perfectly fine.
But on Windows I get the following error:

D:\a_temp\ba90638b-3217-4c14-b352-615ff786d16c.sh: line 5: /c/Users/runneradmin/.poetry/env: No such file or directory
Error: Process completed with exit code 1.

Hi @NicklasTegner, thanks for submitting the issue.
I'm going to take a look at this tonight and get back to you 👍

hi @sondrelg
Just to add, here's my configuration (that triggered the error)
- name: Install Poetry
  uses: snok/install-poetry@v1.0.0
  with:
    virtualenvs-create: true
    virtualenvs-in-project: true
    virtualenvs-path: .venv

Thanks! Figured out how to make it run on Windows - just trying to clean up the implementation now and then I'll release a new version shortly 🚀

nice!
Do let me know when the new version is out 👍

Hey @NicklasTegner,
Just released v1.1.0, and Windows should now work.
Make sure to use:
- name: Install Poetry
  uses: snok/install-poetry@v1.1.0
  with:
    virtualenvs-create: true
    virtualenvs-in-project: true

(the path setting you've used is not necessary since the default in-project path should be .venv)
and then you will need to set the default shell like this, at the job level:
defaults:
  run:
    shell: bash

I've added a section dedicated to running this action in windows in the updated readme, here that might also be helpful.
If anything is unclear, or it still doesn't work, let me know :)

hi @sondrelg
I just tried with the updated version, and I get the following now:
Poetry (1.1.4) is installed now. Great!

To get started you need Poetry's bin directory (%USERPROFILE%\.poetry\bin) in your `PATH`
environment variable. Future applications will automatically have the
correct environment, but you may need to restart your current shell.

chmod: cannot access 'D:a_actionssnokinstall-poetryv1.1.0/main.sh': No such file or directory
Error: Process completed with exit code 1.

And my config is as follows:
Globally:
defaults:
  run:
    shell: bash

And in my Windows job:
    - name: Install Poetry
      uses: snok/install-poetry@v1.1.0
      with:
        virtualenvs-create: true
        virtualenvs-in-project: true

Yes I see you're right. Must have changed something last minute, because my tests were passing and now they're not 🤔
I'll take another look at it today after work and get back to you! Thanks for the feedback.

When you get the opportunity, could you try using snok/install-poetry@v1.1.1rcN?
Version 1.1.1rcN is the pre-release for what I believe fixes the issue.
The issue seems to have been that the built-in Github Actions variable ${{ github.action_path }} doesn't work correctly on Windows. I had made a pre-release yesterday and tested remotely on ubuntu-latest runners, and I had also tested on windows-latest runner locally in this action repo; but I had of course forgotten to test windows runners remotely. Sorry about that, hopefully this works (fingers crossed).

Hi.
I see that v1.1.1rcN is here on the page, but my workflow seems unable to find download information for it:
Failed to resolve action download info. Error: Unable to resolve action snok/install-poetry@vv1.1.1rcN, unable to find version vv1.1.1rcN

I think you might have included one "v" too many. Try snok/install-poetry@v1.1.1rcN :)

It's working now, but my problem is, that my Windows job now fails at the end, at the "Complete job" step.
No output, but on the overview I get
github-actions/` Windows Builder
.github#L1
Process completed with exit code 1.

This error came, after I set the shell to be bash. Any suggestions?
PS: Also still need to verify that caching work, will do that while I try to fix the above error.

Is your workflow closed source? It might be easier to help if you could link me the full code.
If it is, perhaps this example can help you: https://github.com/snok/django-swagger-tester/blob/upgrade-action/.github/workflows/testing.yml
This is the branch of another repository that I've used to test an ubuntu/windows matrix on the current pre-release.

yes, the repository is closed, but if you want you are welcome to take a look at my workflow configuration here: https://gist.github.com/06a177cdeaa2946be83077c2f0f4352b

Sorry for the late response, but my best bet is that one of your other commands will need to be adapted to the shell type. I've finally been able to test this properly, and I thought I would share the workflow I've run.
name: test

on: push

jobs:
  linting:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-python@v2
        with:
          python-version: "3"
      - uses: actions/cache@v2
        with:
          path: ~/.cache/pip
          key: ${{ runner.os }}-pip
          restore-keys: |
            ${{ runner.os }}-pip-
            ${{ runner.os }}-
      - run: python -m pip install black flake8 isort
      - run: |
          flake8 .
          black . --check
          isort .
  test:
    needs: linting
    defaults:
      run:
        shell: bash
    strategy:
      fail-fast: false
      matrix:
        python-version: [ "3.6", "3.7", "3.8", "3.9"]
        django-version: [ "2.2", "3.0", "3.1" ]
        os: [ubuntu-latest, windows-latest]
    runs-on: ${{ matrix.os }}
    steps:
      - name: Check out repository
        uses: actions/checkout@v2
      - name: Set up python ${{ matrix.python-version }}
        uses: actions/setup-python@v2
        with:
          python-version: ${{ matrix.python-version }}
      - run: echo "${{ runner.os }}"
      - name: Install poetry
        uses: snok/install-poetry@v1.1.1rcN
        with:
          virtualenvs-create: true
          virtualenvs-in-project: true
      - name: Load cached venv
        id: cached-poetry-dependencies
        uses: actions/cache@v2
        with:
          path: .venv
          key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}-1
      - name: Install dependencies
        run: poetry install
        if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
      - name: Install django ${{ matrix.django-version }}
        run: |
          source $VENV
          pip install "Django==${{ matrix.django-version }}"
      - name: Install DRF 3.11 on Django 2.2
        if: matrix.django-version == '2.2'
        run: |
          source $VENV
          poetry add "djangorestframework==3.11.1"
      - name: Run tests
        run: |
          poetry run pytest --cov=django_swagger_tester --verbose --assert=plain
          poetry run coverage report

and here is a link to the workflow run in case it could be helpful.
I believe with the changes made in v1.1.1 the poetry install should be stable, but I'm still a little stumped as to why it doesn't work on other shells than bash. Feel free to test further and make commit a PR if you want.

authors: NicklasTegner sondrelg

repo: snok/install-poetry


text:
Cleaner infrastructure dbodor

Removed some unnecessary files
Cleaned up pyproject.toml
Added main init file

most of these changes have been implemented already


text:
Update the snowplow-mini UI to make it more functional

Proposed UI:

Looks awesome - scheduling!

Pushing back

authors: alexanderdean yalisassoon

repo: snowplow/snowplow-mini

license_source: bigquery


text:
🛑 Interseed (interseed.app) is down

In cf5710e, Interseed (interseed.app) (https://interseed.app) was down:

HTTP code: 404
Response time: 139 ms

Resolved: Interseed (interseed.app) is back up in bda2ae6 after 1 hour, 38 minutes.

repo: so-more/proof-monitoring


text:
Присоединить стили к проекту

Присоедини стили добавив следующую строку внутрь  секции.
 <link rel="stylesheet" href="styles.css">
Также добавь данный мета тег в  секцию для responsive design.
<meta name="viewport" content="width=device-width, initial-scale=1" />

Здесь главное не перепутать, что куда добавлять...

Здесь главное не перепутать, что куда добавлять...

Ну, не говори...


text:
Add possibility to manually select Extraction zones

In some very special cases there might happen that the automatic target identification and extraction will not work properly, so there should exist the possibility to, by parsing an argument (most likely), manually select the regions for the target and for background subtraction.
This could be particularly helpful for targets that are too close or too close to the border (nobody should put their science target near the border though).

Isn't this related to #1?

Absolutely, but I think this can be treated independently because one thing is to improve the target detection (automatically) and another is to add a manual option (similar to the interactive mode for wavelength calibration). Despite this I would consider merging both issues. By the way, this came out from the discussion after the talk in the SOAR 2020 Workshop.

This is one of the reasons that it has to be considered for the object extraction: if an input from the user may be an option, we have to have the code written to make this available in the future. If not, it may be too entangled so adding this new capability will be much more difficult.

Yes, we definitely want this feature!  Simon should code things so that this can be added later, but I would not include it for the Beta release.

It will require some time, since we need to program a graphic user interface for this but if we are planning to create a special library for graphic treatment (see #42 )we better leave the effort for then.

I'm currently having this issue with data that contains an extended source. Usually the background subtraction zone is selected an n-sigmas apart from the edge of the target extraction zone. What usually works for point sources does not work for extended sources Such as a planetary nebula (LEFT)

At least there should be a command line argument that allows to adjust the separation of the background zone.

Could this be a solution for #13 ?
Not for fully automatic.

related #298

Upon review with @Allonck we think that despite this issue being very old it is still a desired feature in 2024.

authors: b1quint cbaorion simontorres


text:
Single '}' encountered in format string

Today I tried syncing my wiki with anki and got the following error:
Debug info:
Anki 2.1.29 (bbff62bf) Python 3.8.0 Qt 5.15.0 PyQt 5.15.0
Platform: Mac 10.14.6
Flags: frz=True ao=True sv=1
Add-ons, last update check: 2020-10-20 14:54:16
Add-ons possibly involved: TiddlyRemember

Caught exception:
Traceback (most recent call last):
  File "/Users/kebi/Library/Application Support/Anki2/addons21/60456529/__init__.py", line 142, in join_thread
    raise self.extract_thread.exception
  File "/Users/kebi/Library/Application Support/Anki2/addons21/60456529/__init__.py", line 60, in run
    self.notes = twimport.find_notes(
  File "/Users/kebi/Library/Application Support/Anki2/addons21/60456529/twimport.py", line 189, in find_notes
    notes = _notes_from_paths(
  File "/Users/kebi/Library/Application Support/Anki2/addons21/60456529/twimport.py", line 91, in _notes_from_paths
    notes.update(_notes_from_tiddler(tid_text, wiki_name, tid_name))
  File "/Users/kebi/Library/Application Support/Anki2/addons21/60456529/twimport.py", line 113, in _notes_from_tiddler
    return TwNote.notes_from_soup(soup, wiki_name, tiddler_name)
  File "/Users/kebi/Library/Application Support/Anki2/addons21/60456529/twnote.py", line 73, in notes_from_soup
    notes.update(subclass.parse_html(soup, wiki_name,
  File "/Users/kebi/Library/Application Support/Anki2/addons21/60456529/twnote.py", line 328, in parse_html
    parsed_text = ankify_clozes(text)
  File "/Users/kebi/Library/Application Support/Anki2/addons21/60456529/clozeparse.py", line 128, in ankify_clozes
    return placeholder_text.format(*(i.anki_occlusion for i in occlusions))
ValueError: Single '}' encountered in format string

Does this mean that there is a closing curly bracket without its opening curly bracket somewhere in my tiddlers?
If so, is there a way to debug the plugin and see where it gets stuck?

I am guessing that you accidentally closed a cloze note with }} instead of }.
A quick way to debug would be to open the file mentioned at the end of the traceback (clozeparse.py) and above the offending line 128 add:
raise Exception(text)

That should show the text of the cloze note responsible, which will likely be enough for a full-text search in TiddlyWiki to find it. When you find it, please let me know exactly what text it involved -- this could potentially be an exploitable security issue the way it's written and I'll want to follow up on it.

It looks like I forgot to remove a closing curly bracket after making a correction to my tiddler:
<$macrocall $name="remembercz" id=<<currentTiddler>> text='
Make the radio input checked by default:<br>
{`<input type="radio" name="some-name" checked}>`}
' reference="HTML5 Elements"/>

After removing one of the closing curly brackets everything worked fine thanks :)

A nicer error message for this situation will be provided in the next release.

authors: kebifurai sobjornstad

repo: sobjornstad/TiddlyRemember


text:
Better install instruction

I think I need to create a folder sobjornstad in the plugins folder first, then download zip of this repo, then extract it, and drag tw-plugin folder into sobjornstad folder, and rename tw-plugin to TiddlyRemember

Current instruction in the website is not friendly enough for new tw users.

Hi @linonetwo,
It sounds like maybe you didn't find the installation instructions on the front page of the TR documentation? While you can install to a folder wiki by extracting the source like you are doing, that is not required – the simplest way (and recommended for anyone who doesn't already understand how to do that) is to simply drag and drop the provided link into your wiki.
See the "Installing TiddlyRemember" tab in the bottom half of this screenshot:

If you were following these instructions, please elaborate on what part is unclear or insufficiently detailed and I will take a look at improving them.

I think step 1's "Alternatively," is unclear, I know it will take words to describe the instruction, but I think it worth.
The problem with drag&drop is it won't save plugin to the plugins/ folder for nodejs wiki (for now).

I think step 1's "Alternatively," is unclear, I know it will take words to describe the instruction, but I think it worth.

I don't really think that explaining how to install plugins on Node.js belongs in TiddlyRemember's documentation. That is the same for any TiddlyWiki plugin. I wouldn't explain in detail how to double-click on an installer in my downloads folder if this were a Windows program either, because that's not something my application controls; I would just say “run the installer” and then provide any instructions needed after that.
However, linking to the TiddlyWiki documentation on the structure of Node.js plugin folders makes good sense here. I've changed this section to read (with appropriate links):

First you need to install the TiddlyWiki plugin into your wiki. To do so, drag and drop this link into your TiddlyWiki: TiddlyRemember.
Alternatively, if you're running on Node.js, you might prefer to (but do not have to) install the plugin as a plugin folder instead. See Installing custom plugins on Node.js in the TiddlyWiki documentation for details on the available options. You can find TiddlyRemember's code in this GitHub repository.

The problem with drag&drop is it won't save plugin to the plugins/ folder for nodejs wiki (for now).

Aside from making it a little harder to update, AFAIK it doesn't really matter if it's there or not. I do prefer installing into the plugins folder myself, but either way works fine in practice.
I actually don't understand why, because according to the docs plugins installed via drag-and-drop are not usable when running directly under Node rather than a browser. But I've tested it, and syncing TiddlyRemember (which requires tiddlers in the TiddlyRemember plugin to work!) still works fine when you install TiddlyRemember via drag-and-drop. Maybe that caveat only applies to JavaScript modules or something.

Thanks, I agree that makes sense.
I will work on extract plugin automatically to plugins folder later this year.

authors: linonetwo sobjornstad


text:
Async functions are stringified

Bug report
Link to a working example, because we pass a function that is synchronous: https://jsfiddle.net/k3prLfcd/
Passing an async function results in the function being stringified. It should be usable as a function. Link to example demonstrating the issue: https://jsfiddle.net/ope5y79f/1/
Steps To Reproduce
Steps to reproduce the behavior:

Go to https://jsfiddle.net/ope5y79f/1/
Click on the button
See error

Expected behavior
Should be able to call async methods
Actual behavior
Async functions are stringified
Screenshots

Desktop (please complete the following information):

Browser chrome
Version 117.0.5938.149 (Official Build) (arm64)

https://github.com/socketsupply/tonic/releases/tag/v15.1.2

authors: bcomnes nichoth

repo: socketsupply/tonic


text:
sodalite/unstable fails to build

On the relase/unstable branch, doing ./build.sh base (or any variant) crashes while it configures packages with:
Running pre scripts... done
error: Package 'authselect-libs' has (currently) unsupported <lua> script in '%prein'
error: Failed to build tree
🛡  Correcting permissions for build directory...

Related to https://github.com/coreos/fedora-coreos-tracker/issues/1057.

Took the package lock from https://github.com/coreos/fedora-coreos-config/blob/03e3950438ece41db38bf1786da183a002318690/manifest-lock.overrides.yaml, but now I'm stumbling into this error:
Importing rpm-md... done
rpm-md repo 'copr_stsdc_monitor'; generated: 2022-01-11T03:27:39Z solvables: 4
rpm-md repo 'fedora-rawhide'; generated: 2022-01-19T08:03:06Z solvables: 67264
rpm-md repo 'copr_decathorpe_elementary-staging'; generated: 2021-12-14T20:42:00Z solvables: 232
error: Couldn't find locked package 'authselect-1.3.0-3.fc36' (pkgs matching NEVRA: 0; mismatched checksums: 0)
error: Failed to build tree

Removing all the package locks causes an entirely different error from the initial ticket too:
Downloading from 'copr_stsdc_monitor'... done

error: failed to add install element: /proc/self/fd/16/cache/repomd/fedora-rawhide-rpmostree-unset-releasever-x86_64/packages/editorconfig-libs-0.12.5-2.fc35.x86_64.rpm [1]
error: Failed to build tree

repo: sodaliterocks/sodalite


text:
Not Unzipping Files > 2 GB

I'm trying to Unzip a file over 2GB (2.69 GB Precisely).
I'hv implemented the following method to unzip File;

(BOOL)unzipFileAtPath:(NSString *)path toDestination:(NSString *)destination delegate:(id)delegate;

[SSZipArchive unzipFileAtPath:sourcePath toDestination:destinationPath delegate:self];
Which returns NO.
I Debugged the problem & found out that I Get the UNZ_BADZIPFILE in function unzOpenInternal(). I cross checked with the zip file & its unzipping correctly.
I'm stuck, Please let me know how do I get this working. A quick response will be highly valuable & appreciable.

Was this ever solved?

repo: soffes/ssziparchive

license: mit


text:
Reimport sinon

https://github.com/softlayer/sl-ember-components/commit/27a05e2aee726b46cc4e3948a2dc454bf2d57825 removed it

repo: softlayer/sl-ember-components


text:
add thead and tbody tags

I was trying to initialize DataTables on top of a table generated by json2html, and it was failing due to the lack of thead tags.

I totally ignored the test cases. I will update those and submit a new pull request.


text:
Enrich error message when derivation fails.

We're using this pattern(#258) in our project quiet often, and I think it would be great if it provides better error message.
For example,
package magnolia1.examples

import magnolia1.{Magnolia, ReadOnlyCaseClass, SealedTrait}

import scala.language.experimental.macros

trait Print[T] {
  def print(t: T): String
}

trait GenericPrint {
  type Typeclass[T] = Print[T]

  // we only have split method
  def split[T](ctx: SealedTrait[Print, T]): Print[T] = { value =>
    ctx.split(value) { sub =>
      sub.typeclass.print(sub.cast(value))
    }
  }

  implicit def gen[T]: Print[T] = macro Magnolia.gen[T]
}

sealed trait Entity
case class Person(name: String, age: Int) extends Entity
case class Place(name: String) extends Entity

object Print extends GenericPrint {
  implicit val person: Print[Person] = p => p.name + " is " + p.age
//  implicit val place: Print[Place] = _.name
  implicit val entity: Print[Entity] = gen
}

object PrintExample extends App {
  println(implicitly[Print[Entity]].print(Person("John", 32)))
  println(implicitly[Print[Entity]].print(Place("London")))
}

If we don't have an instance for Place, the current error message is
the method 'join' must be defined on the derivation object Print to derive typeclasses for case classes
So, it tells you that it needs join method to derive a type class instance, but it doesn't tell you which instance it was trying to derive.
With this PR, it prints,
magnolia: unable to derive magnolia1.examples.Print.Typeclass[magnolia1.examples.Place] -- the method 'join' must be defined on the derivation object Print to derive typeclasses for case classes
This one tells which one it was trying to derive.

Thanks!

repo: softwaremill/magnolia


text:
UC4/US6 Collect & present usage information of resources

User story
As a coordinator of a soil carbon monitoring network,
I want to be able to get an overview of projects and products that are enabled by the data and information produced by my monitoring network. With open data publications it is often difficult to get a good overview of who is using your data and what products and result are they creating. You often have to rely on mouth to mouth to get this information or follow up scientific publications in detail. Systems requesting information from users to access the data/results such as JRC throw up what can perceived to be an unnecessary barrier, run into GDPR issues and are not able to capture data that is shared further by users.
So that I can justify the expenses and the need for future funding to management and funding authorities.
Critical success factor:
This US relies on other US that cover the ingestion of the relevant metadata about projects, datasets and knowledge this US needs to connect to.
Extended user story description:
I am monitoring network coordinator/data manager, who works at government agency/research institute, which focuses on soil monitoring in Flanders, Belgium in the role of scientist/data manager, my educational level is university (soil scientist, engineer, ….). My main target group are researchers and policy advisors who work in all land use types. I do/don’t have a soil science background. My digital expertise encompasses office programs. I usually use portals google and web of science to find information or data.
I want to be able to connect datasets, projects and knowledge (papers, reports, …) that use data and knowledge produced by my monitoring network to my monitoring network.

[ ]	I need to get an overview of scientific papers that use results from my monitoring networks.
[ ]	I need to get an overview of products and datasets that use results from my monitoring networks.
[ ]	I want to be able to easily connect from the main entry on my monitoring network to the entries that rely on data and results from my monitoring network.

For example this could be achieved by having a section in the entry with URL’s linking to other sections.

Some categorisation could be useful

Data
Report
Paper
….

I need to be able to determine some statistics form the connections that can be made

Statistics:

Number of datasets
Number of papers
Number of reports

Type of the reports

Building further on the results
Evaluation of the network
Methodological study
Applied research
Fundamental research

Number of and type of Stakeholders

Creators of the product
Targeted by the product

Impact factor of scientific results
Open data/open access/FAIR adherence of linkend entries
Data quality of the product
Themes and domains of the derived products
Type and organisations that use the results (scientific, government, business, land managers, citizens, international, …)
Nationality
Area covered (region of monitoring network, …)
…

This can be provided in multiple ways

Part or all of these statistics can be calculated in the platform itself and shown as additional information with the entry for the monitoring network
A download functionality or API that allows me to pull the necessary metadata to be able to calculate those statistics

Business reason:
This will allow me to more effectively justify expense and continued investment in my monitoring network to the funding authority. It will better allow me to visualise the impact the network has and have hard figures on the usage. I will be able to reduce the time that needs to be spend to get this information or justify the spending through other means (consultancy, extensive networking, …). ….
Acceptance criteria:

[ ] I have an overview of at least four key statics on products using data and results from my monitoring network (numbers of products, impact, data quality fairness, …) or I can pull the necessary data from the repository that allows me to create those key statistics.

meeting 28-2-2024:

Linkage between academic articles and used datasets in that research are available via citations on the article. OpenAire offers some linkage between datasets and articles (as added by the article writers). But the article itself may offer additional citations to dataset linkage.
The frontend should offer some filtering/dashboard mechanism to display results of linked items to a certain dataset
A user login mechanism is not needed
Nice to have: Soilwise may derive dataset/knowledge items linkages from context.

Extend with information about how results can be shown (API-access is must have and GUI is should have).
Extend with functionality on storing the queries/results (is it needed?) +
Both for DOI and non-DOI (but with other uuid) content.
User access management functionality, not really necessary at face value, because information is assumed to be public (metadata) or it is assumed that direct access to private information is not necessary, only after processing and aggregation by the SoilWise platform.
Potential new user story on requesting users to add linkages when providing metadata to SoilWise (a new independent functionality from providing metadata or an extension of existing functionality).

Extracted Requirements:

01.1 - Data Discovery - Query Catalogue: Query using a filter to identify metadata records for knowledge and data related to a soil monitoring network. Further filtering between knowledge records and data records, records about resources produced by the network itself and those that are produced by external parties (organization filter, filtering based on the linked references, filtering based on the data sources, ...) and filtering based on target group.
01.1 - Data Discovery - Query Catalogue: An overview page for the search results, with access to additional filters (filter options not yet used), for each record a link to the record in soilwise and a short overview (title, preview image, start abstract, type, organization, geographical scope, ...)
02 - Metadata creation: Collecting references to current item from other datasets, projects, papers, reports,…
02 - Metadata creation: Having an interface or API with analysed references to my datasource
02 - Metadata creation: Create metadata about a monitoring network or other project (can be done by the project owners or also automatically if several resources in SoilWise mention a project but no project information is available).
02 - Metadata creation: The ability for the user to add linkages between metadata records
03 - Metadata feedback: On the metadata record is an overview given of impact statistics of the record
04 - Download data: A download functionality or API that allows me to query and pull the enhanced metadata from SoilWise so that the data can be analysed to create statistics about the use of results from a certain project.
06 - Data quality checks: Open data/open access/FAIR evaluation of the records
08 - Vocabulary: Complex query required vocabs
09 - Knowledge: need for metadata on papers, reports, products (scientific and non-scientific), who are related to soil projects and soil monitoring networks. (Not only scientific/soil mission sources).
11 - Processing: analysing the type of references (dataset, report, paper) (out-of-scope)
11 - Processing: FAIR score (out-of-scope)
11 - Processing: Link records based on the references mentioned (not only for records with DOI's but for alle records), this should be clickable in UI, queryable (so attributes of the references need to be accesible). Do this linking not only for the references but also based on keywords, information and data mentioned in metadata, tekst and data + keep information about how the link was made and the link type. (out-of-scope)
11 - Processing: calculating an impact score based on the harvested references, and a large number of other statistics such as number of referecers by type, estimation of the stakeholders,…. (out-of-scope)
13 - Networking: Metadata, core public information about organizations (and maybe also persons?) who are using my data
14 - Security: A user can login to edit the autmatically generated record

authors: KathiSchleidt dluts pvgenuchten

repo: soilwise-he/Soilwise-userstory-epics


text:
BufferLayout encoding does not support arrays

While encoding instruction data, I need to use seeds as an argument which is a uint8Array.  It seems like that is not an option offered by the Layout:
`
 const dataLayout = BufferLayout.struct<any>([
     BufferLayout.u8('instruction'),
     BufferLayout.nu64('seeds'), // I don't think this is right; should be [u8; 32] but can't find that
     BufferLayout.u32('number_of_schedules')
   ]);
   const data = Buffer.alloc(dataLayout.span);
   dataLayout.encode(
     {
       instruction: 0, // 0 for create instruction,
       seeds: Uint8Array.of(bump),
       number_of_schedules: new BN(5),
     },
     data
   );

`
It would be great if there was an option to do this.

https://github.com/pabigot/buffer-layout isn't part of this repo.

repo: solana-labs/solana-program-library


text:
solana token decimal reduction from default

i would like to reduce the decimal from default to lesser in order to mint 1 trillion worth of the token

Hi there, that's not possible unfortunately, and there aren't any plans to add the functionality to either token or token-2022.


text:
added coinGeckoId for TheSolanDAO token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

https://github.com/solana-labs/token-list/commit/a731c642f0089e1fcba2372aab658d965c16ae98

authors: keone thesolandao

repo: solana-labs/token-list


text:
add O-Inari Mask token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

Wasn't aware it had to check if the website was reachable to pass.
It's not up yet, I'll start working on getting a construction page up at least.

I can help resolve automerge issues.
Feel free to check out J-a-m-e-s-o-n/token-list#1

authors: J-a-m-e-s-o-n ObscenityIB


text:
Adding Granny Coin

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

I can help refer this zakimzf/profile#1
PS: if you don't need my help.. please ignore this!

authors: blackgrant zakimzf


text:
DRAFT: Add wallet adapter for Strike

Implemented a wallet adapter for the soon-to-be-released Strike Protocols multisig program-based wallet.

Thanks! Please mark this as a draft until the wallet is actually released.

I can do that, sure. We are planning to launch with an invite-only beta, will that present any issues with getting this approved?
Thanks!

Yeah, we try not to merge or release anything in here until it's publicly testable and usable. But I can review it such that it's ready to merge when you go live.

Gotcha, thanks! I can certainly understand that. We are definitely going to be working to get dApps to support our wallet even during our invite launch, so would love to discuss the possibility of getting this merged even before our wallet is fully open to the public, as I suspect it will be a lot easier for the dApps. Happy to chat in a more interactive medium if that would be helpful.

Got it, makes sense. This is less of a policy than just uncharted territory. We'll figure something workable out! Just let me know when it's ready to beta test.

Got it, makes sense. This is less of a policy than just uncharted territory. We'll figure something workable out! Just let me know when it's ready to beta test.

Cool. I will, but if you want you can also sign up to our waitlist here: https://waitlist.strikeprotocols.com/

I've converted this PR to a draft until you let me know it's ready!

This PR is still marked as a draft and hasn't been updated for some time. Can you either close it, or update it, merge the latest master, and mark it ready for review?

We actually will be rolling out our prod site in the next week or two — we will merge the latest master and mark this ready for review then

We actually will be rolling out our prod site in the next week or two — we will merge the latest master and mark this ready for review then

Hi Jordan, we have updated this PR and marked it ready-for-review. You should be getting an invite code to be able to test against wallet.strikeprotocols.com early next week.

Sure, will do, makes sense.

@jordansexton Hi, we have updated to make this a loadable wallet adapter

Thanks for the review @jordansexton, I believe I've addressed all of your comments

Thank you! Excited to try this out

Thank you! Excited to try this out

Thanks @jordansexton !

Published:

@solana/wallet-adapter-strike@0.1.0
@solana/wallet-adapter-wallets@0.17.2

Please test and let me know if it works!

Please test and let me know if it works!

I tested using a local hacked-up version of the react-ui-starter package, and was able to execute a dapp transaction successfully.

authors: bholzman jordansexton

repo: solana-labs/wallet-adapter


text:
Allow apps to request permissions, and explain why

As part of building the React SDK application, we recently added a "tic tac toe" game. The idea isn't that Solid now supports TTT, but rather to demonstrate to developers the best way to create a game that sends notifications, creates it's own data files, shares the data files in a collaborative way, and so on.
Because of this, the SDK application requires specific permissions. We create game files, then update the permissions for others to edit. We also create an inbox and change the permissions there so anyone can push notifications into it. For this reason, the app (and likely MANY apps) need Control permissions. However, this is not part of the default set of checkboxes when granting permissions, and additionally many people when using our test servers and deployed application at generator.inrupt.com had already visited previous versions, meaning they'd already granted now out-of-date permissions via the handy popup.
For this reason, we think it would be a nice addition to have the following features:

Ability to request certain permissions for my application and override the default checkboxes. My app knows it needs Control permission but currently doesn't have a nice way to indicate this.
Ability to pass over a string for each permission, so we can explain what we're using the permission for. Informing the user why the app needs Control for example.
Ability to re-request the popup, for example if the user attempts to do something requiring a missing permission.

That last one came up during testing, as some users (I believe Vincent and Michiel) had run the app prior to Control permissions being required, and couldn't receive notifications as Everyone was not set to the proper permission on the inbox. But they had no way of knowing it, so it just failed silently. Ideally, we could show the popup again saying "Hey you tried to do something that requires Control permission but you didn't give this app permissions, change it?"

Ability to request certain permissions for my application and override the default checkboxes. My app knows it needs Control permission but currently doesn't have a nice way to indicate this.

I don't understand what do you mean by 'default checkboxes'. In #29 I suggest that person should have a way to chose which app they want to use to manage permissions for all the other apps, so we can't assume to much details about how this application UI functions.

Ability to pass over a string for each permission, so we can explain what we're using the permission for. Informing the user why the app needs Control for example.

👍  I thing we should have this as requirement. Since we talk about text presented to user it may also have some @solid/internationalisation-panel related aspects.

Ability to re-request the popup, for example if the user attempts to do something requiring a missing permission.

👍 It sounds related to #24 - general management of app authorizations, in that case changes initiated by application iself.

To clarify your general use case, I understand that user always wants to grant to application some subset of permissions they have for certain resource.  Does it require application to know what permissions the user oneself has to that resource? Could it happen that application want to request permission which user onself doesn't even have?

@elf-pavlik Regarding the "default checkbox" comment, right now the app permissions popup that is shown has 3 out of 4 of the permissions checked. What I am proposing here is that the application itself should be able to show what permissions it needs to function properly, instead of having 3 permissions checked automatically.
I'm approaching this from an application developer perspective, and we had several people not know they were missing permissions on our sample app because they had already granted incorrect permissions. We also had people who just click "Ok" assuming the popup will take care of everything they need, when in fact it didn't because the Control permission we needed was unchecked.

What I am proposing here is that the application itself should be able to show what permissions it needs to function properly, instead of having 3 permissions checked automatically.

and earlier

We also create an inbox and change the permissions there so anyone can push notifications into it. For this reason, the app (and likely MANY apps) need Control permissions.

I still lack clarity if we talk about user specific access modes in ACL files or app specific access modes - which to my knowledge we currently don't even have proposed way to express.
Maybe you could share two snippets with 'permissions' in question

one before intended changes
one after intended changes

As part of building the React SDK application, we recently added a "tic tac toe" game.
Because of this, the SDK application requires specific permissions.

I must admit lack of familiarity with mentioned React SDK. Given 'tic tac toe' app lacks specific permissions, with what app user will modify the permissions (and where those permissions get stored)?
I appreciate your patience with helping me clearly understand your scenario.

My apologies, I lost track of this!
In this request I am specifically talking about application specific access modes, not user-based. Those are, in my opinion, two different things. The React SDK deals with both, but the application itself can help manage user-access permissions (either by displaying an error, or by granting access if appropriate).

I must admit lack of familiarity with mentioned React SDK. Given 'tic tac toe' app lacks specific permissions, with what app user will modify the permissions (and where those permissions get stored)?

The root problem is that the example app requires Read, Write, Append, and Control permissions for tic tac toe to work. This is due to many factors - Read is pretty obvious, as well as Write. We need Append control as we create notifications, and we need Control permission as well to grant access to your opponent.
Right now, with NSS pods, the trusted app dialog is displayed showing the 4 choices, and 3 are pre-checked by the dialog. They are Read/Write/Append. Control is unchecked by default. Many people have simply clicked "ok" and accepted these settings, but in fact we need all 4, and this breaks parts of the app. What I am asking for is a way for the app requesting the permissions to explain what they need (which it sounds like you agree with) but also perhaps give a list of permissions we want, to pre-check what we need. Either that, or uncheck everything by default, instead of providing a list of 3 permissions that may-or-may-not be correct.

@james-martin-jd I'd say this is a feature request for NSS not a spec request.
The full solution to attenuated access control will be much more complicated, but if I'm understanding you correctly, this is something desired in the short term.

@jaxoncreed yes, this is a problem we're currently having to work around in the SDK apps. Currently we show a little popup with a link to a dokieli doc that explains how and what permissions to set for the SDK to function properly. It's also an issue we've seen many, many people have, because it seems most people just rubberstamp the permissions dialog and select whatever is already checked by default.

I think we can take it as one of Use Cases here. For that it would come helpful to besides read, write, append, control access modes document what kind of information the app wants to access.

When it comes to sharing, it probably would also need to access some kind of contacts / address book information, only in read mode.
When it comes to sending notifications, what kind of notifications this app wants to send?Just notification that board was shared or also that opponent made move?
Does game use some dedicated inbox for send and received notifications?
I understand that anyone can host a board and invite someone else, which means that Alice with app she uses would need to allow it to write 'tic, tac, toe moves' to any storage, since any of her friends could host a board and invite here to make moves.

All those question relate to three common aspect of permissions / authorizations

access mode - read, write, append, control
kinds of data - shapes which use types
location of data - LDP containers in solid storage instances

Often, for each 'kind' of data authorization will have specific access mode and sometimes location detail (or left global).

That is a very good point. Some folks have commented that having to give an SDK-generated application Control permissions is "scary" because it can do a lot of things. Same with Write and Append. Really it only needs access to a few things (profile info, public and it's own app-specific inbox, and a few files it manages in its own folder).

Some folks have commented that having to give an SDK-generated application Control permissions is "scary" because it can do a lot of things.

i think that giving any app (other than a designated sharing-control app) the Control permission is scary.

authors: elf-pavlik james-martin-jd jaxoncreed zenomt


text:
tests fail against production

curl 'https://solid-auth-cli-test-user.solidcommunity.net/login/password'   -H 'Content-Type: application/x-www-form-urlencoded'   --data-raw 'username=solid-auth-cli-test-user&password=123'   --compressed -i

gets a cookie, but the
SERVER_ROOT=https://solid-auth-cli-test-user.solidcommunity.net ALICE_WEBID=https://solid-auth-cli-test-user.solidcommunity.net/profile/card#me USERNAME=solid-auth-cli-test-user PASSWORD=123 npm run jest

instructions from the readme lead to a null cookie in the helper.

Fixed by setting the jest timeout to 60000 :)

repo: solid/solid-crud-tests


text:
Bundle (or don't) rdflib, document installation

This PR should resolve some confusion around how a client application should include the solid.js client.
There are two basic options:

Include the minified bundle inside <script> tags.

Bundles with and without rdflib are provided. Note that the bundles will have new names - solid-client.min.js and solid-client-no-rdflib.min.js (let me know if you have any less verbose suggestions)

Use the CommonJS modules

Both examples are now documented in docs/installing.md
@dmitrizagidulin please review?
FYI @deiu

Reviewed -- Great job on this! Merging.

Both examples are now documented in docs/installing.md <-- should be a link above

Good point, @timbl! I've updated the comment above to point to the installing.md for this particular commit. The readme will always link to the current version.

authors: dan-f dmitrizagidulin timbl


text:
How to migrate 2 or more fields on the same field in Azure DevOps, by appending those data?

Its A question
How to migrate 2 or more fields on the same field in Azure DevOps, by appending those data? Is it possible through this tool? Basically, I want to map custom fields in Azure DevOps or tfs. Since we may have a lot of custom fields which may not be available in azure devops.
Do we have ay way to achieve this?

This feature is part of our Pro offering. The pro version of the tool is still in the making, but we can offer this feature as a part of our migration-as-a-service offering. Let me know if that sounds interesting and I'll give you the contact information :)

This feature is part of our Pro offering. The pro version of the tool is still in the making, but we can offer this feature as a part of our migration-as-a-service offering. Let me know if that sounds interesting and I'll give you the contact information :)

Can I know more about Pro featuring?

Sure, I answered your discussion here :) https://github.com/solidify/jira-azuredevops-migrator/discussions/774

repo: solidify/jira-azuredevops-migrator


text:
Document how to return errors from actions

Currently the documentation makes no mention of how to return an error code from a createRouteAction or createServerAction. I assume it's one of either throw some error, or return a response object with an HTTP status code.

Some specific asks:

Document the proper uses of FormError.
Explain how to throw an exception that causes a specific HTTP error code to be returned (404, 401, etc.)

repo: solidjs/solid-start


text:
Setter<bigint> should not be assignable to Setter<number> and vice versa

Describe the bug
Setter<number> is assignable to Setter<bigint>. Accessor<number> is not assignable to Accessor<bigint>.
The same goes for the opposite case: Setter<bigint> is assignable to Setter<number>. Accessor<bigint> is not assignable to Accessor<number>.
The type error should be there for Setter too, just like it is for Accessor.
Your Example Website or App
https://stackblitz.com/edit/solidjs-templates-6dfgbm?file=src%2FApp.tsx
Steps to Reproduce the Bug or Issue

Open stackblitz as given above.
Once packages are installed, you will see the type error for Accessor but not for Setter.

Expected behavior
Setter<number> should not be assignable to Setter<bigint>.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Brave (Chrome)
Version: see below

  "devDependencies": {
    "typescript": "^5.1.3",
    "vite": "^4.3.9",
    "vite-plugin-solid": "^2.7.0"
  },
  "dependencies": {
    "solid-js": "^1.7.6"
  }

Additional context
No response

This isn't specific to number and bigint, in fact any function with one generic type that returns that type, with compatible args is assignable to any Setter: example. See https://github.com/microsoft/TypeScript/issues/50050.
It might be reasonable to remove the generic from Setter to solve this, and no longer narrow its return type to that specified by its parameters. This would break code which expects set(v) to return the type of v rather than the type of the signal, which was the intent of the initial change https://github.com/solidjs/solid/commit/851bd63202d6e9eb6907c77a26ac323ce728d206.
https://playground.solidjs.com/anonymous/b5d0dba5-83ca-4bbe-bbfd-8d268e0dfc9f is what this could look like with the existing type test cases.

I don't know if this is reasonable do. @otonashixav I trust your opinion on this. If the current types are what make the most sense we can close this as known given how TS works. Otherwise we can revisit it. But I admit I don't think I quite get the subtleties. The desire for the Setter to not use the type of Signal was just to make it more specific? Also @Jager567 if you have any opinions.

Thanks for the detailed explanations. I haven't used Solid enough to make more comments on this subject now that there seems to be a bit more depth to why it was designed this way, so I'm leaving this to your hands.

Okay weirdly I managed to fix? this only slightly changing the current type. So honestly I have no idea what's going on, potentially typescript is inferring variance differently due to the change.

authors: erhant otonashixav ryansolid

repo: solidjs/solid


text:
Custom event names with periods in them

I can see that we have on:MyCustomEvent which is really nice. However some frameworks like bootstrap use event names with periods in them E.g. hidden.bs.modal and when you try on:hidden.bs.modal then the JSX/TSX compiler doesn't like it.
Its actually not a big deal, we can just make use of onMount / onCleanup to do it, just thought I'd bring it to your attension.

I see I wasn't aware of frameworks using periods. But yeah that's invalid JSX. So wouldn't work in React or any JSX library. I guess that's beyond what we can do something about. As you said manual registration will work.

Manual registration did not work for me either. Must not be a standard event listener.
addEventListener("hidden.bs.modal", handleColumnNameDialogHidden); <-- failed
The bootstrap website wants me to do it like this using JQuery:
$('#myModal').on('hidden.bs.modal', function (e) {
  // do something...
})

So I'd probably not expect it to work by default in any JSX libraries anyway.

authors: clinuxrulz ryansolid


text:
nginx 404 Not Found error

I clone repo done below actions:
1-Change docker-compose.yml configs
2-Set .env file
3-docker-compose up -d
every things going well but when I open app url in browser i get below error

What can i do?

I don't think this image even uses/contains nginx, indicating this may be an issue elsewhere.
Are you using an nginx proxy in front of your containers or on the host server?

As @ssddanbrown mentioned, this image doesn't use nginx anywhere, so there's something else going on here. Also note that there should not be an .env file in use - instead, set variables in the docker-compose.yml file. Finally, to troubleshoot issues please provide logs of the Bookstack container to help us see what's going on.

I found problem.
I have another services on nginx on my vps.
I need to set port number on APP_URL http://kms.xxxxx.com:8080
I try to setup a revers proxy on my Nginx.to remove 8080 but it doesn't work.
Can you help me how can i do that?

@razzaghi What is your current config for this site/host in nginx?

My docker-compose
version: '2'
services:
mysql:
image: mysql:8.2
environment:
- MYSQL_ROOT_PASSWORD=xxxxxxx
- MYSQL_DATABASE=bookstack
- MYSQL_USER=bookstack
- MYSQL_PASSWORD=xxxxxxx
volumes:
- mysql-data:/var/lib/mysql
bookstack:
image: solidnerd/bookstack:23.12.1
depends_on:
- mysql
environment:
- DB_HOST=mysql:3306
- DB_DATABASE=bookstack
- DB_USERNAME=bookstack
- DB_PASSWORD=xxxxxxx
#set the APP_ to the URL of bookstack without without a trailing slash APP_URL=https://example.com
- APP_URL=http://kms.xxxxxxx.com
# APP_KEY is used for encryption where needed, so needs to be persisted to
# preserve decryption abilities.
# Can run php artisan key:generate to generate a key
- APP_KEY=xxxxxxx
volumes:
- uploads:/var/www/bookstack/public/uploads
- storage-uploads:/var/www/bookstack/storage/uploads
ports:
- "8080:8080"
volumes:
mysql-data:
uploads:
storage-uploads:
My Nginx config
server {
server_name kms.xxxxxxx.com;
client_max_body_size 200M;

# Reverse proxy the uwsgi Django request
location / {
    #include proxy_params;
    proxy_pass http://localhost:8080;

    proxy_http_version 1.1;
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection "upgrade";

    proxy_redirect off;
    proxy_set_header Host $host;
    proxy_set_header X-Real-IP $remote_addr;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    proxy_set_header X-Forwarded-Host $server_name;
    proxy_set_header X-Forwarded-Proto https;

}

listen 443 ssl; # managed by Certbot
ssl_certificate /etc/letsencrypt/live/kms.xxxxxxx.com/fullchain.pem; # managed by Certbot
ssl_certificate_key /etc/letsencrypt/live/kms.xxxxxxx.com/privkey.pem; # managed by Certbot
include /etc/letsencrypt/options-ssl-nginx.conf; # managed by Certbot
ssl_dhparam /etc/letsencrypt/ssl-dhparams.pem; # managed by Certbot

}
server {
if ($host = kms.xxxxxxx.com) {
return 301 https://$host$request_uri;
} # managed by Certbot
server_name kms.xxxxxxx.com;

listen 80;
return 404; # managed by Certbot

}
~

@razzaghi Since you're using HTTPS via nginx, update your APP_URL to start with https:// instead of http://.
Then re-up your bookstack container (not just restart, since the container needs to be re-created).
After that, what do you see when going to https://kms.xxxxxxx.com/?

Great.
It's worked. Thank you so much

Before this, kms was loaded, but the media was not loaded. it just got fixed.

authors: razzaghi rjhenry ssddanbrown

repo: solidnerd/docker-bookstack


text:
Dummy app not compatible with main search autocomplete?

Solidus Version
3.4.0.dev
To Reproduce
Run bin/rails-dummy-app, then run bin/rspec spec/system/search_spec.rb. Specs should fail:
7 examples, 7 failures

Failed examples:

rspec ./spec/system/search_spec.rb:12 # searching products falls back to normal search for plain HTML requests
rspec ./spec/system/search_spec.rb:60 # searching products with JS enabled closes autocomplete suggestions clicking outside the search input
rspec ./spec/system/search_spec.rb:42 # searching products with JS enabled clicks on a suggestion pressing enter
rspec ./spec/system/search_spec.rb:30 # searching products with JS enabled scrolls up and down through suggestions using up/down arrow keys
rspec ./spec/system/search_spec.rb:20 # searching products with JS enabled shows autocomplete suggestions
rspec ./spec/system/search_spec.rb:51 # searching products with JS enabled closes autocomplete suggestions pressing esc key
rspec ./spec/system/search_spec.rb:25 # searching products with JS enabled automatically selects the first suggestion

To test in development,

Create the dummy app like above, then run bin/rails-dummy-app db:seed and bin/rails-dummy-app spree_sample:load to seed your development database.
Run bin/rails-dummy-app server to run the server,
In https://localhost:3000, try to search for a product. You should see no response:

If you enter the search term, you'd see the following error:

Additional context
The new autocomplete search was introduced in https://github.com/solidusio/solidus_starter_frontend/pull/300.
I'm seeing the same issue on my ongoing work with SolidusBraintree (See https://github.com/solidusio/solidus_braintree/issues/91 and https://github.com/solidusio/solidus_braintree/tree/gsmendoza/91-ssf-dummy-app).
However, on a regular Solidus app, the search autocomplete seems to work fine with SolidusPaypalCommercePlatform. See https://github.com/gsmendoza/solidus_store/tree/rails-7-0-4--solidus-3-4-0-dev--spcp:

@kennyadsl @elia I tried removing --skip-javascript from the sandbox and dummy-app scripts. This seems to have fixed the autocomplete in the Rails dummy app, but I'm still encountering the following issues:

bin/rspec spec/system/search_spec.rb is still failing.
I'm getting the following errors when I try the search autocomplete feature on the sandbox app:

I'll continue investigating these issues tomorrow, but if you can look into them, I would appreciate it!

I think I identified the issue: dummy and sandbox rails apps are generated with --skip-bundle option, which skips the importmap, turbo and stimulus installation. At that point they are added to the app Gemfile, but they are not installed. Trying to remove the --skip-bundle option.

#189 should fix it!

authors: gsmendoza kennyadsl


text:
🛑 orda.kz is down

In f1246b2, orda.kz (https://orda.kz/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: orda.kz is back up in 0c9349c after 48 minutes.


text:
Automate build using multiple FROM

The original Dockerfile assumes local build, while https://hub.docker.com/r/solsson/prometheus-exporter-apache/ is Automated.

This seems to work great. Thanks for setting it up!

authors: dpetersen solsson

repo: solsson/prometheus-exporter-apache


text:
add FC37 build

this PR adds a build for FC37

Hey @stefanleh !!!  Did you get a chance to test out FC37?  I have not yet upgraded my desktop.
I'm going to merge even though the job failed.  FC35 and 37 built successfully, but 36 was killed once it hit the 6 hour mark.
https://github.com/some-natalie/fedora-acs-override/actions/runs/3475885794

repo: some-natalie/fedora-acs-override


text:
Crash on non-patient node selection

The patient data is changed when a node is selected. Currently, there is a crash when a node other than a patient (like city) is selected.
Also, we need a better handler for different node selections.

I think you're referring to the crash on the 'City' filter? Yes, that has to be fixed, but I'm fairly sure it's an issue with the data, and I've taken it up with the Ops team on Telegram.

I pushed a fix for this

authors: SahajR someshkar


text:
Problem: sticky seems to calculate sidebar height wrong, overlaps footer

Hi!
I'm using your plugin on one of my sites, and I'm very happy with it, except for one thing that I'm not able to resolve:
It seems it calculates the height wrong.
You can see it in action here, with the ad on the right:
http://delbart.se/saab-blocket/
When the white are ends, the sticky item is supposed to stop, but it keeps on for a couple of hundred pixels.
Any idea what might be wrong?

Sorry, I solved it by myself. I had misinterpreted a setting, but now it all works as expected! Thanks for a wonderful plugin!

repo: somewebmedia/hc-sticky


text:
native-echarts，在安卓上有时能显示，有时不能显示

1.刚开始可以显示图表，过一段时间之后，就无法显示图表了，所有的图表都不显示，
2.我把安排删掉重装之后，又可以显示，但是会一直重复这个过程，
3.有些手机会出现这个问题，有些手机不会，之前以为是手机内存的原因，但是有些手机明明还有很多内存，怎么会导致这个问题呢。
4.我已经参照网上的方法，配置了安卓打包不显示之类的

我也有这个问题，你是怎么解决的呢？

我换了一个框架，react-native-charts-wrapper，原生的，显示速度更快

谢了哈，我试试

repo: somonus/react-native-echarts


text:
Embedding tfrecords doesn't work

Tensorflow complains about the datafile being passed as a PosixPath. this is fixed if you cast the path to a string in run_embed.py

@goaaron I was having issues with run_embed.py as well, but they seemed to be syntax issues, which I believe was because I was using python 3.5. However when switching to 3.7 I'm not able to install tensorflow-gpu, which appears to be required for TAPE installation. What python version are you using, and did you come across any similar issues?

authors: goaaron proto-tardigrade

repo: songlab-cal/tape


text:
Initial import of p4-symbolic: symbolic-packet, parser and guarded_map

Build Results:
divya@e9e9b2e05579:/sonic/src/sonic-p4rt/sonic-pins$ bazel build $BAZEL_BUILD_OPTS ...
INFO: Build option --cache_test_results has changed, discarding analysis cache.
...
INFO: Elapsed time: 48.366s, Critical Path: 32.15s
INFO: 58 processes: 6 internal, 52 linux-sandbox.
INFO: Build completed successfully, 58 total actions
Test Results:
divya@e9e9b2e05579:/sonic/src/sonic-p4rt/sonic-pins$ bazel test $BAZEL_BUILD_OPTS --cache_test_results=no ...
INFO: Build option --cache_test_results has changed, discarding analysis cache.
INFO: Analyzed 281 targets (0 packages loaded, 17436 targets configured).
INFO: Found 178 targets and 103 test targets...
INFO: Elapsed time: 71.450s, Critical Path: 17.40s
INFO: 104 processes: 112 linux-sandbox, 16 local.
INFO: Build completed successfully, 104 total actions
//gutil:collections_test PASSED in 0.7s
//gutil:io_test PASSED in 0.6s
//gutil:proto_matchers_test PASSED in 0.6s
//gutil:proto_test PASSED in 0.8s
//gutil:status_matchers_test PASSED in 0.5s
//gutil:table_entry_key_test PASSED in 0.1s
//gutil:testing_test PASSED in 0.6s
//gutil:version_test PASSED in 0.6s
//p4_fuzzer:constraints_util_integration_test PASSED in 0.8s
//p4_fuzzer:fuzz_util_test PASSED in 1.0s
//p4_fuzzer:fuzzer_showcase_test PASSED in 0.8s
//p4_fuzzer:switch_state_test PASSED in 0.8s
//p4_fuzzer:table_entry_key_test PASSED in 0.2s
//p4_pdpi:ir_tools_test PASSED in 1.1s
//p4_pdpi:p4_runtime_matchers_test PASSED in 0.9s
//p4_pdpi:sequencing_util_test PASSED in 0.9s
//p4_pdpi/netaddr:ipv4_address_and_network_address_test PASSED in 0.5s
//p4_pdpi/netaddr:ipv6_address_test PASSED in 0.4s
//p4_pdpi/netaddr:mac_address_test PASSED in 0.7s
//p4_pdpi/packetlib:packetlib_fuzzer_test PASSED in 17.2s
//p4_pdpi/packetlib:packetlib_matchers_test PASSED in 0.8s
//p4_pdpi/packetlib:packetlib_test PASSED in 0.3s
//p4_pdpi/packetlib:packetlib_test_runner PASSED in 0.1s
//p4_pdpi/packetlib:packetlib_unit_test PASSED in 1.7s
//p4_pdpi/string_encodings:bit_string_test PASSED in 0.6s
//p4_pdpi/string_encodings:byte_string_test PASSED in 0.4s
//p4_pdpi/string_encodings:decimal_string_test PASSED in 0.2s
//p4_pdpi/string_encodings:decimal_string_test_runner PASSED in 0.2s
//p4_pdpi/string_encodings:hex_string_test PASSED in 0.1s
//p4_pdpi/string_encodings:hex_string_test_runner PASSED in 0.2s
//p4_pdpi/string_encodings:readable_byte_string_test PASSED in 0.5s
//p4_pdpi/testing:helper_function_test PASSED in 0.7s
//p4_pdpi/testing:info_test PASSED in 0.3s
//p4_pdpi/testing:info_test_runner PASSED in 0.2s
//p4_pdpi/testing:main_pd_test PASSED in 0.2s
//p4_pdpi/testing:mock_p4_runtime_server_test PASSED in 0.8s
//p4_pdpi/testing:packet_io_test PASSED in 0.3s
//p4_pdpi/testing:packet_io_test_runner PASSED in 0.2s
//p4_pdpi/testing:rpc_test PASSED in 0.3s
//p4_pdpi/testing:rpc_test_runner PASSED in 0.2s
//p4_pdpi/testing:sequencing_test PASSED in 0.3s
//p4_pdpi/testing:sequencing_test_runner PASSED in 0.2s
//p4_pdpi/testing:sequencing_util_test PASSED in 0.2s
//p4_pdpi/testing:sequencing_util_test_runner PASSED in 0.2s
//p4_pdpi/testing:table_entry_gunit_test PASSED in 0.8s
//p4_pdpi/testing:table_entry_test PASSED in 0.3s
//p4_pdpi/testing:table_entry_test_runner PASSED in 0.2s
//p4_pdpi/utils:annotation_parser_test PASSED in 0.6s
//p4_pdpi/utils:ir_test PASSED in 0.6s
//p4rt_app/event_monitoring:app_state_db_port_table_event_test PASSED in 0.8s
//p4rt_app/event_monitoring:app_state_db_send_to_ingress_port_table_event_test PASSED in 1.0s
//p4rt_app/event_monitoring:config_db_node_cfg_table_event_test PASSED in 1.2s
//p4rt_app/event_monitoring:config_db_port_table_event_test PASSED in 1.2s
//p4rt_app/event_monitoring:debug_data_dump_events_test PASSED in 1.3s
//p4rt_app/event_monitoring:state_verification_events_test PASSED in 1.4s
//p4rt_app/p4runtime:ir_translation_test PASSED in 0.7s
//p4rt_app/p4runtime:p4info_verification_schema_test PASSED in 0.9s
//p4rt_app/p4runtime:p4info_verification_test PASSED in 0.8s
//p4rt_app/p4runtime:packetio_helpers_test PASSED in 1.4s
//p4rt_app/p4runtime:resource_utilization_test PASSED in 0.8s
//p4rt_app/sonic:app_db_acl_def_table_manager_test PASSED in 0.8s
//p4rt_app/sonic:app_db_manager_test PASSED in 0.7s
//p4rt_app/sonic:app_db_to_pdpi_ir_translator_test PASSED in 1.2s
//p4rt_app/sonic:hashing_test PASSED in 0.7s
//p4rt_app/sonic:packetio_impl_test PASSED in 0.5s
//p4rt_app/sonic:packetio_port_test PASSED in 0.7s
//p4rt_app/sonic:response_handler_test PASSED in 0.6s
//p4rt_app/sonic:state_verification_test PASSED in 0.5s
//p4rt_app/sonic:vrf_entry_translation_test PASSED in 1.2s
//p4rt_app/sonic/adapters:fake_sonic_db_table_test PASSED in 0.3s
//p4rt_app/tests:acl_table_test PASSED in 1.3s
//p4rt_app/tests:action_set_test PASSED in 0.8s
//p4rt_app/tests:api_access_test PASSED in 1.1s
//p4rt_app/tests:arbitration_test PASSED in 1.7s
//p4rt_app/tests:debug_data_dump_test PASSED in 0.7s
//p4rt_app/tests:fixed_l3_tables_test PASSED in 2.4s
//p4rt_app/tests:forwarding_pipeline_config_test PASSED in 5.2s
//p4rt_app/tests:grpc_behavior_test PASSED in 5.2s
//p4rt_app/tests:p4_constraints_test PASSED in 0.5s
//p4rt_app/tests:p4_constraints_test_runner PASSED in 0.2s
//p4rt_app/tests:p4_programs_test PASSED in 1.6s
//p4rt_app/tests:packetio_test PASSED in 3.5s
//p4rt_app/tests:port_name_and_id_test PASSED in 2.6s
//p4rt_app/tests:resource_limits_test PASSED in 2.9s
//p4rt_app/tests:response_path_test PASSED in 2.9s
//p4rt_app/tests:role_test PASSED in 0.8s
//p4rt_app/tests:state_verification_test PASSED in 1.4s
//p4rt_app/tests:vrf_table_test PASSED in 1.3s
//p4rt_app/tests/lib:app_db_entry_builder_test PASSED in 0.1s
//p4rt_app/utils:event_data_tracker_test PASSED in 0.1s
//p4rt_app/utils:table_utility_test PASSED in 0.9s
//sai_p4/instantiations/google:clos_stage_test PASSED in 0.8s
//sai_p4/instantiations/google:fabric_border_router_p4info_up_to_date_test PASSED in 0.2s
//sai_p4/instantiations/google:middleblock_p4info_up_to_date_test PASSED in 0.2s
//sai_p4/instantiations/google:sai_p4info_fetcher_test PASSED in 1.0s
//sai_p4/instantiations/google:sai_p4info_test PASSED in 1.0s
//sai_p4/instantiations/google:sai_pd_proto_test PASSED in 0.2s
//sai_p4/instantiations/google:sai_pd_util_test PASSED in 0.7s
//sai_p4/instantiations/google:tor_p4info_up_to_date_test PASSED in 0.2s
//sai_p4/instantiations/google:union_p4info_up_to_date_test PASSED in 0.3s
//sai_p4/instantiations/google:wbb_p4info_up_to_date_test PASSED in 0.2s
//sai_p4/instantiations/google/test_tools:test_entries_test PASSED in 1.0s
//sai_p4/tools:p4info_tools_test PASSED in 0.6s
Executed 103 out of 103 tests: 103 tests pass.
INFO: Build completed successfully, 104 total actions
Keyword Check:
divya@eonms3vm12:~/new_code/sonic-buildimage/src/sonic-p4rt/sonic-pins$ ~/keyword_checks.sh .
Keyword check Passed.

@kishanps , could you please review

authors: bibhuprasad-hcl divyagayathri-hcl

repo: sonic-net/sonic-pins


text:
Try to find ways to tweak performance on appspot

So it runs smoother and doesn't close down lobbies randomly.
Might work by already calling backend on homepage so instances are started earlier?

Some of the issues that occur consistently are:

On cold start, lobby creation takes ages and often closes down the lobby (starts working perfectly fine after server is properly started)
After you start the game from the lobby, it displays the spinner (maybe infinitely or until the creation phase ends) until you reload the page

Check issue on server branch; we might run 1 instance constantly (but it might become bloated with data so we need to do a refresh before our presentation). We can also temporarily up the instance specs to F3 or F4 to have more ram and cpu resources

authors: Cnordg pablotanner


text:
Use Rust for starknet sign/verify/pedersen

Problem
sign/verify/pedersen functions are CPU intensive and the current implementation uses https://github.com/indutny/elliptic which is very slow in JS.
Solution
I took a look at what we can do to improve this and there seem to be several options:

Use Starkware's crypto-cpp repo with their crypto.js JS bindings
Use https://github.com/xJonathanLEI/starknet-rs/tree/master/starknet-crypto which is a pure Rust implementation of starknet crypto operations and write our own bindings
Rewrite src/starkware/ in Rust with https://github.com/RustCrypto/traits/tree/master/elliptic-curve and write our own bindings

I chose option 2. as the repo is seriously maintained, has better performance and portability than starknet-cpp, and was way easier than rewriting our own stark curve.
Results :zap:

sign took 200ms before vs 1ms now -> 200x faster
verify took 1000ms before vs 3ms now -> 333x faster
Test suite passed successfully in 30s before vs 2s now -> 15x faster

More serious benchmark can be done but overall it's way faster!
TODO

Clean the code and figure out a better way to switch between the JS and the Rust implementations. An env variable? I think the JS one is still great for ease of use (no need to compile Rust) and browser support
Add CI steps to build and ship the napi addon (dynamic library) in the JS lib

Closing in favor of https://github.com/sorare/crypto/pull/3

repo: sorare/crypto


text:
LSP hover on method shows wrong return types

Input
→ View on sorbet.run
# typed: true
extend T::Sig

class ActiveSupport::TimeWithZone
  extend T::Sig

  sig { returns(FalseClass) }
  def blank?
    false
  end
end

class NilClass
  extend T::Sig

  sig { returns(TrueClass) }
  def blank?
    true
  end
end

@start_date = T.let(@start_date, T.nilable(ActiveSupport::TimeWithZone))
T.reveal_type(@start_date.blank?)

FWIW this is how Rails does define these: NilClass, TimeWithZone
Observed output
Sorbet is smart enough to put TrueClass and FalseClass together as T::Boolean:
editor.rb:23: Revealed type: T::Boolean https://srb.help/7014
    23 |T.reveal_type(@start_date.blank?)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  Got T::Boolean originating from:
    editor.rb:23:
    23 |T.reveal_type(@start_date.blank?)
                      ^^^^^^^^^^^^^^^^^^
Errors: 1

But in VS Code I'm getting two possible return types of T::Boolean on hover:

Using 'go to definition' does show the two definitions correctly.
Expected behavior
Hover should show the two possible TrueClass/FalseClass returns or a single T::Boolean on hover.

Ran into a different example using presence, with the following signatures:
class ActiveRecord::Base
  sig { returns(T.self_class) }
  def presence; end
end

class NilClass
  sig { returns(NilClass) }
  def presence; end
end

Here current_user!.member returns T.nilable(Member) (an ActiveRecord::Base subclass). Hover shows the merged return values twice:

repo: sorbet/sorbet


text:
After rbi gems run suggest-typed (#1146)

Motivation
Per Jake's suggestion
(https://github.com/sorbet/sorbet/issues/1146#issuecomment-507883405)
Fixes problem where rbi gems produces incorrect sigils
[Fix #1146]
Test plan
I couldn't figure out how to test this. I read the "Writing tests" section of the readme, and I tried to write a test in test/cli but maybe that was a bad choice because main/sorbet in the .sh doesn't seem to work the way srb does? For example, if I do main/sorbet rbi gems, I get: "gems: File Not Found". I'm obviously missing something fundamental, sorry.

This should also probably be done for sorbet typed

@jez could you take this one? You know how this works much better than I do

It's not possible to test this right now.

Thanks Jake, That makes me feel better. I really tried to do a test, and I couldn't figure it out!

authors: DarkDimius connorshea jaredbeck


text:
Error during building

Run-time dependency wayfire found: NO (tried cmake)

meson.build:18:10: ERROR: Dependency lookup for wayfire with method 'pkgconfig' failed: Could not generate cflags for wayfire:
Package glm was not found in the pkg-config search path.
Perhaps you should add the directory containing `glm.pc'
to the PKG_CONFIG_PATH environment variable
Package 'glm', required by 'wf-config', not found

This turns up when I was running meson setup build --prefix=/usr, I've installed glm and wayfire packages (tried both AUR and chaotic-aur).

I think the basic problem is that, due to a change in glm, you should install cmake and then reinstall wayfire and pixdecor.

I'm still unable to build this, but anyway thx for ur efforts!

Thank you for following up. Do you still get the same error even after installing cmake before rebuilding wayfire and wf-config? If you can post the complete output starting with the command you're running to build, we might be able to spot the problem.

Actually I can't build wayfire manually, either. The built wayfire binary can't find wf-config

I see. It might be that you need to set LD_LIBRARY_PATH to the directory containing libwf-config.so. In any event, if you would like to try to fix it, we can keep communicating here, on IRC/Matrix/Discord or you can create a discussion on github.com/wayfirewm/wayfire.

Thanks for your advice, however I am currently busy and can just accept building from AUR.
I'll try to figure out the problem as soon as I have some free time.

authors: soreau vollowx

repo: soreau/pixdecor


text:
Complexes vs. Families

Hi guys,
We have the following additional entity name strings that we need to ground:
Activin A
Activin AB
Activin B
Inhibin A
Inhibin B
AMPK alpha1beta1gamma1
AMPK alpha1beta1gamma2
AMPK alpha1beta1gamma3
AMPK alpha1beta2gamma1
AMPK alpha1beta2gamma2
AMPK alpha2beta1gamma1
AMPK alpha2beta2gamma1
AMPK alpha2beta2gamma2
AMPK alpha2beta2gamma3
AMPK a1b1g1
AMPK a1b1g2
AMPK a1b1g3
AMPK a1b2g1
AMPK a1b2g2
AMPK a2b1g1
AMPK a2b2g1
AMPK a2b2g2
AMPK a2b2g3
alpha1beta1gamma1
alpha1beta1gamma2
alpha1beta1gamma3
alpha1beta2gamma1
alpha1beta2gamma2
alpha2beta1gamma1
alpha2beta2gamma1
alpha2beta2gamma2
alpha2beta2gamma3

I have the some questions about this set and the use of the Relations table:

It is my belief (however misinformed) that these are all protein complexes,
1a) yet the Relations file lists some of them as families. What is the correct mapping for each of these strings?
1b) Aren't the Inhibin dimers also part of the Activin family? (I looked for entries like:
BE,Inhibin_B,isa,BE,Activin but didn't find any).

Some (all?) of these are synonym strings for existing entries in the Relations file so...
2a) Can your Relations file handle the synonyms that entities often have? How?
2b) Should we map the synonyms ourselves (i.e., in a separate table we ground these strings to Bioentities entities) or do you have a way to add these strings directly to the BE Relations table?

Thanks for your help.
-t

I think the confusion comes from the fact that Activin is a family of complexes.
Here is the relevant portion of relations:

HGNC	INHBA	partof	BE	Activin_A
HGNC	INHBA	partof	BE	Activin_AB
HGNC	INHBB	partof	BE	Activin_AB
HGNC	INHBB	partof	BE	Activin_B
BE	 Activin_A	isa	    BE	Activin
BE	 Activin_AB	isa	    BE	Activin
BE	 Activin_B	isa	    BE	Activin

So INHBA and INHBB are parts of specific complexes (Activin_A, Activin_B and Activin_AB), which are then referred to together as the Activin family.
Inhibin is also a family of complexes but it is currently not defined in the relations file, so it needs to be added -  I will do this today.

The relations file contains symbols in a name space. The entries have no text names or synonyms associated with them. Synonyms need to be handled when mapping to entries from text to the BE name space. That is, for instance, "Erk", "ERK" and "MAPK1/3" would all be mapped to the BE:ERK entry. Our own mapping table is in grounding_map.csv which does this for ungrounded/misgrounded strings appearing most often in our use cases.

To be able to map
AMPK alpha1beta1gamma1
AMPK alpha1beta1gamma2
AMPK alpha1beta1gamma3
AMPK alpha1beta2gamma1
AMPK alpha1beta2gamma2
AMPK alpha2beta1gamma1
AMPK alpha2beta2gamma1
AMPK alpha2beta2gamma2
AMPK alpha2beta2gamma3

all we need to do is define these specific complexes in relations.csv. I will do this later today.

Thanks Ben, I think I understand now.
As for the a[12]b[12]g[123] complexes: we appreciate their addition. From answer (2) above, I infer that we will then handle the mapping of all these synonym strings to whatever symbolic family name that you will create (or maybe you will map these to the existing AMPK family symbol?)

Great! My plan is to create a BE entry that the string "AMPK alpha1beta1gamma1" can be mapped to. The name would be, for instance, BE:AMPK_A1B1G1. That complex will then be defined in the relations as:
BE AMPK_A1B1G1 isa BE AMPK
HGNC PRKAA1 partof BE MAPK_A1B1G1
HGNC PRKAB1 partof BE MAPK_A1B1G1
HGNC PRKAG1 partof BE MAPK_A1B1G1

I would then do the same for all the other specific complexes.

@hickst take a look at #16, it adds all the entities and relations that you need to map your inhibin  and AMPK strings. If this looks good, I'll merge it.

It looks good to me....thanks.

authors: bgyori hickst

repo: sorgerlab/bioentities


text:
sqlite3.OperationalError: disk I/O error

Strange error when initiating an instance:
from sosia import Orignal

au = Original(7201538867, treatment_year=2015)

results in
Traceback (most recent call last):
  File "C:\Users\rosm\AppData\Local\Programs\Python\Python38\lib\site-packages\IPython\core\interactiveshell.py", line 3326, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-12-59a3b9befab2>", line 1, in <module>
    au = Original(author, treatment_year=year)
  File "C:\Users\rosm\AppData\Local\Programs\Python\Python38\lib\site-packages\sosia\classes\original.py", line 183, in __init__
    Scientist.__init__(self, self.identifier, treatment_year, refresh=refresh,
  File "C:\Users\rosm\AppData\Local\Programs\Python\Python38\lib\site-packages\sosia\classes\scientist.py", line 349, in __init__
    df = get_authors(self.identifier, self.sql_conn,
  File "C:\Users\rosm\AppData\Local\Programs\Python\Python38\lib\site-packages\sosia\processing\getting.py", line 35, in get_authors
    data, missing = retrieve_authors(authors, conn)
  File "C:\Users\rosm\AppData\Local\Programs\Python\Python38\lib\site-packages\sosia\processing\caching\retrieving.py", line 27, in retrieve_authors
    insert_temporary_table(df, merge_cols=cols, conn=conn)
  File "C:\Users\rosm\AppData\Local\Programs\Python\Python38\lib\site-packages\sosia\processing\caching\inserting.py", line 98, in insert_temporary_table
    cursor.execute("DROP TABLE IF EXISTS temp")
sqlite3.OperationalError: disk I/O error

Could not replicate, probably fixed long ago.

repo: sosia-dev/sosia