Unable to use fetch / foursquare-maps example

I am unable to run the foursquare-maps example in the repo.
After running npm install and npm run render it displays the following error:
SyntaxError: Unexpected keyword 'const'. Const declarations are not supported in strict mode..
Plugin “react-sketchapp: Foursquare + Google Maps”, line 12175:
» const SuperInit = (0, _runtime.SuperCall)(NSStringFromSelector("init"), [], { type: "@" });

I see the same error ( Unexpected keyword 'const'...) if I try to add fetch to basic-example.
Also seeing it in Profile Cards w/GraphQL:
SyntaxError: Unexpected keyword 'const'. Const declarations are not supported in strict mode..
Plugin “react-sketchapp: Profile Cards w/ GraphQL”, line 6801:
» const SuperInit = (0, _runtime.SuperCall)(NSStringFromSelector("init"), [], { type: "@" });

I'm using node v7.10.0 and npm 4.2.0

@nicolasantryhoot thanks for reporting, sorry for the hiccup 😭  — can confirm I'm seeing this on one of my Macs too 🤔
I think cocoascript-class should be set up to transpile const -> var, will check it out and get back to you.

same here

@darknoon can you take a look at that PR please 🙏

@jongold doesn't seem to solve the fact that transpilation uses arguments to transpile (..args), which AFAIK isn't present with cocoascript.


Delete successfully put data chunks of unsuccessful put operations.

This patch ensures that if a composite blob put operation fails
after successfully putting one or more data chunks, then such chunks
are deleted in the background.
New code is getting covered 100%.

Mostly look good, left a few comments.

@xiahome @cgtz addressed comments, please take a look.

Looks good!

LGTM, merging.


[NoodleMagazine] Unable to extract player path

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Brazil
Provide a description that is worded well enough to be understood
Unable to download from NoodleMagazine
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt -vU https://noodlemagazine.com/watch/-199846690_456248029                                                                           2
[debug] Command-line config: ['--cookies-from-browser', 'firefox', '--concurrent-fragments', '6', '--downloader', 'aria2c', '--downloader-args', 'aria2c:--optimize-concurrent-downloads=true', '-vU', 'https://noodlemagazine.com/watch/-199846690_456248029']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [4d9231208] (pip)
[debug] Python 3.12.5 (CPython x86_64 64bit) - macOS-11.7.10-x86_64-i386-64bit (OpenSSL 3.3.1 4 Jun 2024)
[debug] exe versions: ffmpeg 7.0.2 (setts), ffprobe 7.0.2, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, mutagen-1.47.0, requests-2.32.3, sqlite3-3.46.1, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
Extracting cookies from firefox
[debug] Extracting cookies from: "/Users/my_user/Library/Application Support/Firefox/Profiles/nuylb6l4.default-release/cookies.sqlite"
Extracted 577 cookies from firefox
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1830 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[NoodleMagazine] Extracting URL: https://noodlemagazine.com/watch/-199846690_456248029
[NoodleMagazine] -199846690_456248029: Downloading webpage
ERROR: [NoodleMagazine] -199846690_456248029: Unable to extract player path; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "/Users/my_user/.local/pipx/venvs/yt-dlp/lib/python3.12/site-packages/yt_dlp/extractor/common.py", line 740, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/my_user/.local/pipx/venvs/yt-dlp/lib/python3.12/site-packages/yt_dlp/extractor/noodlemagazine.py", line 47, in _real_extract
    player_path = self._html_search_regex(
                  ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/my_user/.local/pipx/venvs/yt-dlp/lib/python3.12/site-packages/yt_dlp/extractor/common.py", line 1369, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/my_user/.local/pipx/venvs/yt-dlp/lib/python3.12/site-packages/yt_dlp/extractor/common.py", line 1333, in _search_regex
    raise RegexNotFoundError(f'Unable to extract {_name}')

Duplicate of #9936


Usage can be more elaborate

Hi,
I am intending to use this, but the Usage section in the Readme, is not very descriptive,
neither is there a list of all goals that this plugin supports, (More like a feature-list, that would be great to see all capabilities offered by this)
what goals to include in build, or any of that ?
Just by adding the dependency , would that trigger jalopy execution ?

We can close this issue.


Improve application.properties textmate grammar

Fixes #233
The PR https://github.com/textmate/ini.tmbundle/pull/1 that fixes this apostrophe problem:

uses the \K regex pattern, which VS Code does not seem to support: https://github.com/microsoft/vscode/issues/93426
Therefore in VS Code, the regexes with \K in that PR does not match with anything, therefore does not highlight anything in orange, therefore appearing to solve the issue in VS Code.
The PR I am making right now, contains TextMate grammar from https://github.com/microsoft/vscode/blob/master/extensions/ini/syntaxes/ini.tmLanguage.json, removes source ini specific grammar (like using ; for commenting), removes matching the single or double quotes (which removes the orange string highlighting alltogether), and adds support for multi-line property keys and number highlighting.
Examples:
quarkus\
.application\
.name=name

quarkus.application.version=123

quarkus.application.name=long\
application\
name

quarkus.http.port = 1234

quarkus.application.name=David's application
quarkus.http.port=8080

org.acme.restclient.CountriesService/mp-rest/url=https://restcountries.eu/rest
configKey/mp-rest/url =

# comment
quarkus.\
http.\
port = { \
  "name": "EpochMillis" \
}

# comment
quarkus.application.name=my app name

Signed-off-by: David Kwon<EMAIL_ADDRESS>

Fixed

weird, it doesn't work for me

It should work now

Nice work @xorye !


New Use Case: Add Kadi4Mat and Material Science

https://kadi.iam.kit.edu/ linked to ELN page. Also has Zenodo integration.

Example RO-Crate https://doi.org/10.5281/zenodo.12165686

Can be added to https://github.com/ResearchObject/ro-crate/blob/main/docs/pages/use_cases/eln.md


feat: added prims algorithm

Description of Change
Added prims algorithm to graphs.
Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] Sort by alphabetical order
[x] I acknowledge that all my contributions will be made under the project's license.

#750 already implemented this.
can we close this PR @cclauss ?


Use scikit-build to build and install with pip

This builds and installs using scikit-build.
Key points:

makes libtab "pip-installable" - pip3 install --user . will work, for example.
Can also use cmake to build the project, if desired (with -DCMAKE_INSTALL_PREFIX etc.)
Can also use python3 setup.py install if desired.
Installs both libtab.so and libtab.cpython.foo.so (pybind) to the same location, and sets RPATH to fix linking of wrapper code.
So far, does not install any C++ headers or C/C++ API, but this could be added, once it is decided on the structure.

Reduced pre-requisites down to just having Eigen and scikit-build installed, and testing on MacOS.


[CT-1507] part 2: Expand configuration to all strategies

resolves #402
Description
Take the new logic from #389 and expand and configuration to all incremental strategies to allow simplification of conditional logic for readability.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

@dbeatty10 tests are failing because as of right now the default value being passed in for the condtional logic is none which fails for the delete+insert check due to not being a table by configuration. playing around with it right now.
could probably get passing by just specifying == "view" but as mentioned above doesn't handle other possible unsupported configs later on
could also get passing by adding the configuration to models being tested (looking for which ones they are specifically already incase that is what we want to do)


Feature request: support for integral terms

Help! All the PDE models I'm interested in integrate over one of the dimensions, and NeuralPDE.jl is too slow for many of them. As MOL is the standard way to discretize these models in the literature, would it be possible to add in support for integrals?

This library could evolve to solve integro-differential equations, but it does not at this time and it's probably not going to for at least a bit.

Support for integrals from say 0..x are now merged, was this the type of limits you needed?

The limits I need are integrals over the entire domain - here's a snippet:
@parameters t a
Dt = Differential(t)
Da = Differential(a)
Ia = Integral(a in DomainSets.ClosedInterval(0,52*75.0))
domains = [t ∈ (0,40.0), a ∈ (0,75*52)]

alright, see you in MOL v0.7.6 :)


include importance on monitoring

It will allow put importance on monitoring system, like if something is huge important or low importance.

I didn't use priority because, I was worried about misunderstanding it to be a run priority, but now I think it doesn't have difference at all.
Which is better? What is it priority on keep running or what is it importance on keep running?

It's alright to merge?

Aproved !


Refactor TC [run_process_replay]

This PR is a refactor on tensor cores:

Unifies tensor core code in kernel
~Fixes AMX tensor core into real shape, so hack for counting is no longer needed.~ moved to #6524
Enables much cleaner Turing TC (ignaciosica/tinygrad#15) to live in renderer (only +10 LOC with ops_python, cuda and ptx support)

I can't benchmark on all devices, but based on the tests I ran locally (METAL and CUDA with Turing), TC's performance doesn't seem to be slowed down.

I know it could be better to separate AMX changes from this PR but they are kind of mutually dependent on each other. If preferred I will separate them into different PR's though.

The reduce_axes is the shape of the last dim on the tensor core.
I tried using that shape directly on the TC and calculating dims dynamically but didn't like how it turned out.
axes: Tuple[int,int,Tuple[int,...]]
@property
def dims(self) -> Tuple[int,...]: # N, M, K
  return tuple(prod(ax) if isinstance(ax, tuple) else ax for ax in self.axes)

Currently, every TC unrolls the reduce dim first, which is why it is not included in opts_seq and performed first. Nonetheless, it could be the case for a TC in the future where this is not true, the question here is whether to include it or not.


token-cli: Fix unexpected behavior when closing a wrapped SOL account with another wrapped SOL account as recipient

Fixes: #887
Tacked on a sync-native instruction if the destination is a wrapped account.

I'd appreciate just some direction on whether or not I'm doing this right... I kinda used the same logic as I had with the JS client changes.
Check if the recipient is a token account, if that fails we know its a wallet address and we dont need to do anything. If it is a token account and recipient_accout.is_native is true, then append a sync-native instruction.

Maybe another issue to add a test for this, i.e. recipient account when closing is a wrapped sol account, would also be a good idea?

I've added a first draft of a test for this. Please let me know if its correct/going in the right direction at all.

Looks like you still need to run cargo fmt on your changes -- you can do that from token/cli.  Also, there are some test failures, so be sure to fix those.

Looks great, thanks for the contribution!


Cheatsheet won't load correctly

Go to the docs main page (0 or master) https://master-docs.syncano.io/#/
Click on the Cheatsheet option from the resources tab

An empty screen will show up the url path is (https://master-cheatsheet.syncano.io/#/):

The docs load after clicking on one of the tabs.

@adamwardecki is is still a case?

It's working as expected now.


Feature/update openlayers 3.16.0

Upgrade openlayers to 3.16.0, and fix some bugs.

fix TileVector for Removal of ol.source.TileVector (since 3.11.0)
fix popup for https://github.com/openlayers/ol3/pull/4492 (since 3.12.0)
fix mapbox url for ol.source.TileJSON changes (since 3.13.0)
add using js-polyfills by unit tests for https://github.com/openlayers/ol3/pull/4639 (since 3.13.0)

This is awesome :+1:
Going to review this immediately after lunch break :smiley:

LGTM. Thanks a lot for contributing this!!

Fixes #272, #260


Question about the number of output_projection in Beit3

Describe
Beit3:
Hi!
I found that there is only one output_projection (nn.Linear(768, 64000)) for masked language modeling. However, as Beit-3 is a multimodal model, should there also be a output head for masked image modeling?

Hi @violet-sto,
The head for masked language modeling is "mim_head.weight" and "mim_head.bias". They are in the BEiT3-Large and BEiT3-Base

Thank you so much for your reply! Unfortunately, I didn't find where 'mim_head' was defined in the code of Torchscale.


Strange display when connecting to a non Knative enabled cluster

See the error message:

Thanks. Good find.

fixed


test-branch/execute-job/wrong-commit-sha-on-pr/88b74f7a-75d0-4d92-b2c8-ca316dd8865e

PR for testing a3fda22bb065d4944e5f1bb0be5fd096661dfe09

/canonical/self-hosted-runners/run-workflows 27c1f83b0d25c0128e15755c2f36bcf8b64e8a9f


Recommend isolated partition for /var/lib/docker?

File: config/daemon/index.md
I am running Docker automatically on a server.
A problem I run into is that /var/lib/docker grows to fill the entire disk.
In theory, the solution is trivial: a cron job to docker system prune. In practice, usage is bursty, and the disk tends to fill up between runs of docker system prune. And when the disk is full, nothing works, including docker system prune.
As far as I know, there is no way to configure Docker itself to restrict disk usage. (And I certainly wouldn't know how to make a pull request to do that.) The best solution I know of is to create a separate partition and mount /var/lib/docker there. Then, if a request tries to exceed the storage available, it will simply fail rather than hose the system.
I am thinking of editing the documentation to recommend this as a best practice when running Docker on a server. https://docs.docker.com/config/daemon/ seems like perhaps the natural place to mention it, though it's not exactly tied to Docker's configuration per se (Docker itself doesn't really know about the mount).
Comments? Would there be a better place to address disk usage limitation?

@dHannasch Thanks a lot for the feedback. I believe it would be good to discuss this in the upstream https://github.com/moby/moby/issues repository. I'm closing this issue here for that reason. Please feel free to continue the conversation here after I close. Thanks again.


Make repo ready for collaborative work

Create first tests
Separate functions:

from in-memory data to in-memory data
from file to in-memory data
from in-memory data to file

more OOP ?

Solved by #29


BDC Deployment unclear that other tool dependencies are required

Run resource deployment wizard
Select SQL Server Big Data Cluster card
Error message about missing tools appears with instructions to click Install Tools to install them
Click button and immediately get this error

Going to the output pane shows these errors. It isn't immediately clear that there's other dependencies required for the tools (such as curl, pip3, docker, etc). Making these errors much more clear would be greatly beneficial here - at least listing the required other dependencies.
"c:\\Users\\chgagnon\\AppData\\Roaming\\azuredatastudio\\User\\globalStorage\\microsoft.resource-deployment"
]
    > docker version --format "{{json .}}"
    stderr: 'docker' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $docker version --format "{{json .}}"    ... exited with code: $1
	>>> docker version --format "{{json .}}"   ... errored out: Process exited with code 1	>>> Ignoring error in execution and continuing tool deployment    > pip3 show azdata-cli
Search Paths for tool 'Azure CLI': [
	"C:\\Program Files (x86)\\Microsoft SDKs\\Azure\\CLI2\\wbin",
	"c:\\Users\\chgagnon\\AppData\\Roaming\\azuredatastudio\\User\\globalStorage\\microsoft.resource-deployment"
]
    > az --version
    stderr: 'pip3' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $pip3 show azdata-cli    ... exited with code: $1
	>>> pip3 show azdata-cli   ... errored out: Process exited with code 1	>>> Ignoring error in execution and continuing tool deployment   >pip3 show azdata-cli
   Could not find 'Location' in the output:
   output:
Search Paths for tool 'azdata': [
	"c:\\Users\\chgagnon\\AppData\\Roaming\\azuredatastudio\\User\\globalStorage\\microsoft.resource-deployment"
]
    > azdata -v
    stderr: 'az' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $az --version    ... exited with code: $1
	>>> az --version   ... errored out: Process exited with code 1	>>> Ignoring error in execution and continuing tool deployment    stderr: 'azdata' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $azdata -v    ... exited with code: $1
	>>> azdata -v   ... errored out: Process exited with code 1	>>> Ignoring error in execution and continuing tool deployment	[ deleting previously downloaded kubectl.exe if one exists ... ]
    > IF EXIST .kubectl.exe DEL /F .kubectl.exe
    >>> $IF EXIST .kubectl.exe DEL /F .kubectl.exe    ... exited with code: $0
	[ downloading and installing the latest kubectl.exe ... ]
    > for /f %i in ('curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt') do curl -LO https://storage.googleapis.com/kubernetes-release/release/%i/bin/windows/amd64/kubectl.exe
    stderr: 'curl' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $for /f %i in ('curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt') do curl -LO https://storage.googleapis.com/kubernetes-release/release/%i/bin/windows/amd64/kubectl.exe    ... exited with code: $1
	>>> for /f %i in ('curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt') do curl -LO https://storage.googleapis.com/kubernetes-release/release/%i/bin/windows/amd64/kubectl.exe   ... errored out: Process exited with code 1Search Paths for tool 'docker': [
	"c:\\Users\\chgagnon\\AppData\\Roaming\\azuredatastudio\\User\\globalStorage\\microsoft.resource-deployment"
]
    > docker version --format "{{json .}}"
    stderr: 'docker' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $docker version --format "{{json .}}"    ... exited with code: $1
	>>> docker version --format "{{json .}}"   ... errored out: Process exited with code 1	>>> Ignoring error in execution and continuing tool deployment    > pip3 show azdata-cli
Search Paths for tool 'Azure CLI': [
	"C:\\Program Files (x86)\\Microsoft SDKs\\Azure\\CLI2\\wbin",
	"c:\\Users\\chgagnon\\AppData\\Roaming\\azuredatastudio\\User\\globalStorage\\microsoft.resource-deployment"
]
    > az --version
    stderr: 'pip3' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $pip3 show azdata-cli    ... exited with code: $1
	>>> pip3 show azdata-cli   ... errored out: Process exited with code 1	>>> Ignoring error in execution and continuing tool deployment   >pip3 show azdata-cli
   Could not find 'Location' in the output:
   output:
Search Paths for tool 'azdata': [
	"c:\\Users\\chgagnon\\AppData\\Roaming\\azuredatastudio\\User\\globalStorage\\microsoft.resource-deployment"
]
    > azdata -v
    stderr: 'az' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $az --version    ... exited with code: $1
	>>> az --version   ... errored out: Process exited with code 1	>>> Ignoring error in execution and continuing tool deployment    stderr: 'azdata' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $azdata -v    ... exited with code: $1
	>>> azdata -v   ... errored out: Process exited with code 1	>>> Ignoring error in execution and continuing tool deployment	[ deleting previously downloaded kubectl.exe if one exists ... ]
    > IF EXIST .kubectl.exe DEL /F .kubectl.exe
    >>> $IF EXIST .kubectl.exe DEL /F .kubectl.exe    ... exited with code: $0
	[ downloading and installing the latest kubectl.exe ... ]
    > for /f %i in ('curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt') do curl -LO https://storage.googleapis.com/kubernetes-release/release/%i/bin/windows/amd64/kubectl.exe
    stderr: 'curl' is not recognized as an internal or external command,
    stderr: operable program or batch file.
    stderr:
    >>> $for /f %i in ('curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt') do curl -LO https://storage.googleapis.com/kubernetes-release/release/%i/bin/windows/amd64/kubectl.exe    ... exited with code: $1
	>>> for /f %i in ('curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt') do curl -LO https://storage.googleapis.com/kubernetes-release/release/%i/bin/windows/amd64/kubectl.exe   ... errored out: Process exited with code 1

fixed by: https://github.com/microsoft/azuredatastudio/pull/8039
@ranasaria , please make sure to close the issue after merge.


Add new project links and cleanup setup.py

In GitLab by @arisilon on Oct 16, 2019, 13:44
Look at the pattern in the twine project
long_description
project_urls
install_requires

In GitLab by @arisilon on Oct 16, 2019, 14:32
mentioned in commit df8d3661bd15d4bdb34deae3afe9f68ed5da5ddd


Eaten Bodies Cannot Be Recovered From Space Dragon

Description

Round 33913. A space dragon was butchered, leaving no bodies to recover. Players affected blackscreened and had to reconnect to see again.
Reproduction

Butcher a space dragon with a player's body inside.

This happens in every round with a dragon, often removing syndicate objective items along with the bodies


Draft BVH Refactoring

Adapting the BVHNavigator to work with the modified BVH in VecGeom
Validation done:

Example17, 1 event, 1000 e-, Gun 10-170 in Theta, 0-360 in Phi: Numerically identical to master
TestEM3: Numerically identical to master

Performance comparison:

Example17, 64 events, 1000 e-, Gun 10-170 in Theta, 0-360 in Phi.
Average Event time ratio new/master : 1.00081
Average ECAL time ratio new/master : 1.00423
Average Non EM time ratio new/master: 0.99945

This PR was overseeded by https://github.com/apt-sim/AdePT/pull/264


Action Script to automatically run aggregate_data.py on push

I changed the aggregate_data action so that it only runs if a pull request is merged


Dragging trim editor very difficult

is there anyone experience as i experienced? very difficult to drag the trim start and end separately when the position at start and end video. mostly detected as center drag, so both of circle trim dragged.

Yes, I'm also facing this issue.


Feature request: Show Docker Run command

Currently I save all of my docker run commands to OneNote after I've got a container fully tested and working, in case I ever need to go back and use it again...
It would be great if portainer displayed the command that was used in a container's info page, so that it can always just be copied from there if needed. I don't know if there's a way to just get the command itself, or if it would need to be "built" based on it's inspect info, but seems like it shouldn't be too hard that way as well?
Or if that's a bit much, then maybe just have an editable field where we can paste the command ourselves as a place to store the command. Or perhaps both? :p (i.e. auto create the command from the inspect info, show it in the field, and make the field editable in case we want to tweak it a bit, or just overwrite it entirely.)

related to #3205


jwt

Hello,
Is there any example of using JSON Web Token with restler?
Where should I implement it? should I extract the token from the header?
I hope you can point me to some example.

I'm in the process of trying the same thing. I was able to get the JWT code from https://github.com/rmcdaniel/angular-codeigniter-seed/blob/master/api/application/helpers/jwt_helper.php and have added an apiAuth class that implements iAuthenticate and am able to check the custom header auth with this code:
function __isAllowed()
{
$isAuth = false;
// Check to see if we have an Auth header if not check for an api key
if ($_SERVER['HTTP_AUTH'] && $_SERVER['HTTP_AUTH'] == apiAuth::KEY)
{
// This part will call the JWT class to verify token once I am able to set it from the login
// TODO: Check JWT Token...
$isAuth = true;
} else {
$isAuth = isset($_GET['api_key']) && $_GET['api_key'] == apiAuth::KEY ? TRUE : FALSE;
}
return $isAuth;
}
function __getWWWAuthenticateString()
{
    return 'Query name="api_key"';
}

Right now it is just checking a string, as soon as I can figure out how to put the JWT token on login in the Response Header, I will have my Angular Client app send this token on every api request using an http intercepter (I've already done this before using .Net Web Api as the backend, its easy).
I was able to find that putting the @header tag in the comments above the function I can set the customer header Auth to a string but I can'f figure out how to set it to a variable after creating the JWT Token.
I haven't seen this project worked on in a while, is it still alive? I'm starting to think about moving my API code over Sails which is a NodeJS API Server.


401 Unauthorized while issuing maven deploy

Description
While trying to perform a Maven deploy (mvn deploy) to a gitea hosted registry the operation fails with 401 - Authorization. The corresponding log entry is:
2023/03/19 13:40:49 [641710e1-3] router: completed PUT /api/packages/greeen-ambient/maven/org/example/testpipeline-02/1.0-SNAPSHOT/testpipeline-02-1.0-20230319.134051-1.pom for <IP_ADDRESS>:57133, 401 Unauthorized in 3.2ms @ packages/api.go:38(packages.reqPackageAccess)
The operation doesn't work with user:password credentials nor with an authentication token. I tried all sorts of permissions without success. A deployment of files to the generic endpoint does work as expected.
Many thanks in advance.
Gitea Version
1.20.0+dev-181-g6f9cc617f
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
ubuntu
How are you running Gitea?
Running gitea within a docker container utilizing a postgres database.
Database
PostgreSQL

The operation doesn't work with user:password credentials nor with an authentication token. I tried all sorts of permissions without success.

What exactly did you try? https://docs.gitea.io/en-us/packages/maven/#configuring-the-package-registry shows how the registry should be configured.

I configured the settings.xml with the following  part:

gitea

username
repo01

password
xxxx

I tried with a specific user (username/password) as well as with an authorization token, with scope package.

Works as intended for me with the token:
[INFO] --- deploy:3.0.0:deploy (default-deploy) @ maven-package ---
Downloading from central: https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-utils/3.3.1/plexus-utils-3.3.1.pom
Downloaded from central: https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-utils/3.3.1/plexus-utils-3.3.1.pom (5.3 kB at 267 kB/s)
Downloading from central: https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-utils/3.3.1/plexus-utils-3.3.1.jar
Downloaded from central: https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-utils/3.3.1/plexus-utils-3.3.1.jar (262 kB at 5.5 MB/s)
Downloading from gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/1.0-SNAPSHOT/maven-metadata.xml
Uploading to gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/1.0-SNAPSHOT/maven-package-1.0-20230320.131721-1.pom
Uploaded to gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/1.0-SNAPSHOT/maven-package-1.0-20230320.131721-1.pom (1.1 kB at 6.6 kB/s)
Uploading to gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/1.0-SNAPSHOT/maven-package-1.0-20230320.131721-1.jar
Uploaded to gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/1.0-SNAPSHOT/maven-package-1.0-20230320.131721-1.jar (2.1 kB at 16 kB/s)
Downloading from gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/maven-metadata.xml
Downloaded from gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/maven-metadata.xml (231 B at 3.0 kB/s)
Uploading to gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/1.0-SNAPSHOT/maven-metadata.xml
Uploaded to gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/1.0-SNAPSHOT/maven-metadata.xml (765 B at 6.1 kB/s)
Uploading to gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/maven-metadata.xml
Uploaded to gitea: https://my-gitea/api/packages/KN4CK3R/maven/io/gitea/maven-package/maven-metadata.xml (0 B at 0 B/s)

I've to apologize. While doing some more problem determination, I found, the settings.xml was not in the right place.
While testing with the Actions workflow, it is not always easy to see what's going on under the hoods.
Thank you very much for your support.


ASUS Sabertooth X99 pad-file issue

System

Motherboard: ASUS Sabertooth X99
BIOS Version: 4101
GPU: AMD RX480
[x] CSM is turned off. Make sure to confirm this in the BIOS and not with GPU-Z or similar since it can be inaccurate
[x] 4G decoding is enabled. Make sure to confirm this in the BIOS and not with GPU-Z or similar since it can be inaccurate
[x] UEFIPatch is applied (see Using UEFIPatch for more information). On some motherboards DSDT Patching is also needed
[x] I have read Common issues (and fixes)

Description
First off, thanks a lot for this repo and all the effort put into the wiki. It was really helpful!
I have successfully flashed a BIOS with the ReBarSmiDxe module as well as the NvramSmiDxe patch (required for X99).
However, I end up with no video at POST and I can't see anything if I manage to enter the UEFI settings (similar to issue #4 ).
I realise that the UEFIPatch was also trying to patch my PciBus with the patch PciBus | Don't downgrade 64-bit BARs to 32-bit, which is probably what is causing the no video issue.
Unfortunately, if I apply that patch whether it's through UEFIPatch, UEFITool or even MMTool 5.x (can't use 4.x on my BIOS), it seems to add a new pad-file that wasn't there before:

I have even tried messing around with the hex editor, however I can only do that on the extracted uncompressed version of the module, so it ends up being the same problem at the end.
Any idea how I could proceed to fix that?

I've used another version of mmtool to replace pcibus and now bios is free of padding issue, but beware:
on x79 that version caused CPU voltage to be fixed at 1.5V. Thankfully, it's under water cooling and I've seen the voltage error in BIOS and shut down the system.
Maybe there was some config error or it's x79 specific, but use it at your own risk and check immediately for cpu voltage ...
asus.rom.zip

I've used another version of mmtool to replace pcibus and now bios is free of padding issue, but beware: on x79 that version caused CPU voltage to be fixed at 1.5V. Thankfully, it's under water cooling and I've seen the voltage error in BIOS and shut down the system. Maybe there was some config error or it's x79 specific, but use it at your own risk and check immediately for cpu voltage ... asus.rom.zip

Thanks for the response! So the zip you provided is for which board? Highly doubt I can use it if it's a BIOS for another model.
1.5V on CPU is quite high, even with watercooling, it will degrade the CPU pretty fast. I am on air cooling, so not willing to risk running at 1.5V.

It is for your board, what else?
Voltage problem was with x79 and mmtool that supports both 4 and 5, maybe it's not present here ...

Thanks, opening it in UEFITool NE does show that the pad-file is not there, so it looks pretty good (I'm assuming both the PciBus and the NvramSmiDxe patches were applied).
That's really helpful! I'm not sure when I'll have time to test/flash it, but I'll report back as soon as I do.

The cause of the added pad file is that after the patch you applied the module has changed his size causing it to be unaligned, hence needing to add a pad file to realign the other module after the patched one.
There is more than one solution to this:
Try modding an older BIOS version.
Try compressing the module and the insert it to the BIOS (@Sweet_Kitten’s solution).
If you have more than one module to patch, try patching them (like DSDT or whatever, while ignoring the added pad file) and if you have enough luck it might fix the pad file issue.
I went through the same issue. Here is the post that concludes all the process, in case this would be helpful to you.

Hey I tried to modify the BIOS file from Asus. I don´t own this board and can´t test it, but you can try if you want. I added the NVStrapsRebar module too.
https://easyupload.io/yq82fu


Tutorial Page Modification

Currently the tutorial page's format doesn't match the JupyterLab Documentation style,
will change to the form of using tutorial.md linking youtube videos

Now tutorial is under 'Help' menu, naming 'Voyager FAQ',
Since still need user test to make the final design, still modifying it right now


PENV-347: update reloading for new extractor logic

- What I did
Change reload for new extractor with empty pulses
- How to verify it
Run tests
Run mainnet and be, check pulses in database

Codecov Report

Merging #116 into master will increase coverage by 0.06%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
+ Coverage   73.03%   73.09%   +0.06%
==========================================
  Files          47       47
  Lines        3134     3141       +7
==========================================
+ Hits         2289     2296       +7
  Misses        700      700
  Partials      145      145

Impacted Files
Coverage Δ

etl/extractor/platform_impl.go
76.31% <0.00%> (ø)

etl/controller/controller.go
95.83% <100.00%> (+0.48%)
:arrow_up:

etl/controller/pulsemaintainer.go
87.15% <100.00%> (+0.24%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7447bf4...a3929ac. Read the comment docs.


Eventually implement support for PEP 723

What's the problem this feature will solve?
It's just a new source of deps.
Describe the solution you'd like
$sbj
Alternative Solutions
... not doing this?
Additional context
It is provisional and they ask that the user tooling doesn't expose this just yet: https://github.com/pypa/pipx/issues/1128

https://peps.python.org/pep-0723/
https://discuss.python.org/t/pep-722-723-decision/36763

This sounds good to me, now that it's been accepted!


[stdlib] add withMemory function to UnsafePointers

The approach is the same used in withUnsafeMutableBufferPointer, withUnsafeBufferPointer, etc.

Thank you for the PR.  Since this is a new API, please start a thread on swift-evolution.  Please include the motivation for the new API, and a few usage examples.

Thank you for your contribution, but without a Swift-evolution proposal there isn't much we can do.  You may reopen this at any time with a proposal - especially because it is an additive change that would be very much in scope for Swift 4.


unittest

cmd가 실제로 필요한 테스트와, 아닌 테스트로 나뉘어야 함

--grep 'integrated'
--grep 'mockup'

mocha: https://mochajs.org
assertion: https://www.chaijs.com

one-import-bcq has one-import-onnx

ui와 관련된 test는 제외
positive tests만
BuilderCfgFile: 대다수가 private

test bdd로 두번 임포트하는 경우

리드미 작성

nvm
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.1/install.sh | bash
source ~/.zshrc
nvm install 16.14.0
nvm use 16.14.0
npm 8.3.1

Extension Development Host

full access to the VS Code API

# - Launches VS Code Extension Host
code \
--extensionDevelopmentPath=<EXTENSION-ROOT-PATH> \ # - Loads the extension at <EXTENSION-ROOT-PATH>
--extensionTestsPath=<TEST-RUNNER-SCRIPT-PATH> # - Executes the test runner script at <TEST-RUNNER-SCRIPT-PATH>

The test script (runTest.ts)

uses the @vscode/test-electron API to simplify the process of downloading, unzipping, and launching VS Code with extension test parameters
--extensionTestsPath points to the test runner script

The test runner script (index.ts)

programmatically runs the test suite


[macOS] Untrack /usr/lib by default instead of sealing it

It's important to untrack /usr/lib instead of making it a source sealed directory because the set of dynamically loaded libraries (from /usr/lib) during an execution of a process is not necessarily deterministic.  For example, when the same process---which itself is deterministic---is executed multiple times on same inputs, the set of dynamically loaded libraries can vary from time to time.
Enabled tests:

IntegrationTest.BuildXL.Scheduler.NonDeterminismProbeTests

NonDeterminismOpaqueDirectoryOutputDifferentFiles
NonDeterminismOpaqueDirectoryOutput

Test.BuildXL.FingerprintStore.FingerprintStoreTests

VerifyGarbageCollectWorks
DontOverwriteExistingContentAddressableEntries

AB#1519677
AB#1513463
AB#1520150

/azp run Mac BuildXL PR Validation


TECH-1861 - Migrating cla-assistant to Kubernetes

TECH-1861 - Migrating cla-assistant to Kubernetes

Nice one, thanks Sasha


Have a sane default for RESOURCE_SERVICE_PATH

We have many values floating around for the default value of this settings, and sometimes in non-setting form, like this:
https://github.com/ansible/django-ansible-base/blob/21c652631641e3adf5b9cdbb43bb79cbb04fb1ac/ansible_base/resource_registry/tasks/sync.py#L68
^ that value above is what I would like to be the default. I do not want this to be another thing the installer has to set.
And the silent no-op here makes it very difficult to debug anything, and the path to enabling this very challenging.
https://github.com/ansible/django-ansible-base/blob/21c652631641e3adf5b9cdbb43bb79cbb04fb1ac/ansible_base/resource_registry/apps.py#L104-L107

This is fixed by #582...

https://github.com/ansible/django-ansible-base/pull/597


fix: custom event endpoint was wrong

One is for channel and the other is for user.

Our naming isn't very consistent sendEvent vs sendCustomEvent

@ferhatelmas the naming is consistent with the javascript library as far as I see:

https://github.com/GetStream/stream-chat-php/blob/fix-custom-event/lib/GetStream/StreamChat/Client.php#L1131
https://github.com/GetStream/stream-chat-php/blob/fix-custom-event/lib/GetStream/StreamChat/Channel.php#L97

the naming is consistent with the javascript library

yes but not with others (go etc.)

Good notice. I renamed it to sendUserCustomEvent.


restyled time traveling sidebar and added commit overview

To enhance the browsing ux, I made the following changes to the time traveling sidebar:

rearranged and restyled its elements to enhance the compositional flow
added a navigable commit overview
removed the editor avatar (missing or rather redundant data in this context)
changed the merge date into a more human-readable format
changed the EXIT link into a close symbol

Wow! This looks much better, thanks @arminbw.
Could you fix the capitalization of pull request to Pull request? Then it's LGTM.

Could you fix the capitalization of pull request to Pull request? Then it's LGTM.
Done. :)

@prichodko I reviewed the conflicts with the latest changes (pull request 136 by @d4rekanguok). This should be ready to merge now.

I assume this project is dead. It’s a bit sad, that the merge never happened. But it was a fun little experiment!


Fix lifetime handling in rustler_codegen

This PR builds on the previous work of @turion (https://github.com/rusterlium/rustler/pull/430),
@SeokminHong (https://github.com/rusterlium/rustler/pull/430#issuecomment-1175816881) and @neosimsim (https://github.com/rusterlium/rustler/pull/471), so kudos to them, this PR only attempts to complete their prior work.
Handling lifetimes for encoders within rustler_codegen is obvious: we can directly use split_for_impl() from syn to split the generics in a way which allows us to use them for a generated impl. For decoders, handling lifetimes is more involved. First of all, we need to ensure that the special lifetime of the decoder (named '__rustler_decode_lifetime in the generated code) outlives the lifetimes of the type to which we want to decode. The reason for this is that references to parts of the term may not outlive the term itself. On the other hand, the lifetimes of the type need to be able to outlive the term's lifetime, as we return a new struct/enum from the decoder. To ensure this, we generate a where clause for a lifetime 'a like this:
// .. here could be other predicates
'a: '__rustler_decode_lifetime,
'__rustler_decode_lifetime: 'a

With that, the lifetimes 'a and '__rustler_decode_lifetime are practically the same.
Fix #428
Close #430
Close #471

@turion @SeokminHong @neosimsim Maybe you can take a look into this PR? Note that I will not be very responsive the next two weeks.

The generalized helper functions look so good! I think it would be better if there were some tests for them, like this:

pub mod lifetimes {
    use rustler::{Binary, NifMap, NifRecord, NifStruct, NifTuple};

    #[derive(NifMap)]
    pub struct GenericMap<'a, 'b> {
        pub i: Binary<'a>,
        pub j: Binary<'b>,
    }

    #[derive(NifStruct)]
    #[module = "GenericStruct"]
    pub struct GenericStruct<'a, 'b> {
        pub i: Binary<'a>,
        pub j: Binary<'b>,
    }

    #[derive(NifRecord)]
    #[tag = "generic_record"]
    pub struct GenericRecord<'a, 'b> {
        pub i: Binary<'a>,
        pub j: Binary<'b>,
    }

    #[derive(NifTuple)]
    pub struct GenericTuple<'a, 'b>(Binary<'a>, Binary<'b>);

    #[derive(NifMap)]
    pub struct WhereClause<'a, 'b>
    where
        'a: 'b,
    {
        pub i: Binary<'a>,
        pub j: Binary<'b>,
    }

    #[derive(NifMap)]
    pub struct LifetimeBounded<'a, 'b: 'a> {
        pub i: Binary<'a>,
        pub j: Binary<'b>,
    }
}

@SeokminHong I added your tests as-is as a simple compile tests. Do you think that's ok for now? I think we might hit one-or-another issue some time down the road when actually using something with lifetimes, but more concrete examples will make fixing those issues easier I think.

@SeokminHong I added your tests as-is as a simple compile tests. Do you think that's ok for now? I think we might hit one-or-another issue some time down the road when actually using something with lifetimes, but more concrete examples will make fixing those issues easier I think.

I think it's good. I believe it's a rare case to use lifetimes for serialized things.


Setting TTL with encode_and_sign

Hello!
I'm trying to generate a token with non-default TTL.
iex(1)> alias App.{Repo, Device, DeviceRegistration.Monitor}
[nil, nil, nil]
iex(2)> device = Repo.get_by(Device, id: "db749ee8-1e9f-11e6-9d47-4b5e65de4b92")
[debug] QUERY OK db=2.9ms decode=0.2ms queue=0.2ms
SELECT d0."id", d0."is_enabled", d0."user_id", d0."inserted_at", d0."updated_at" FROM "devices" AS d0 WHERE (d0."id" = $1) [<<219, 116, 158, 232, 30, 159, 17, 230, 157, 71, 75, 94, 101, 222, 75, 146>>]
%App.Device{__meta__: #Ecto.Schema.Metadata<:loaded>,
 id: "db749ee8-1e9f-11e6-9d47-4b5e65de4b92",
 inserted_at: #Ecto.DateTime<2016-05-20 15:31:01>, is_enabled: true,
 updated_at: #Ecto.DateTime<2016-05-20 15:31:01>,
 user: #Ecto.Association.NotLoaded<association :user is not loaded>,
 user_id: nil}
iex(3)> ttl = Application.get_env(:app, :reg_ttl)
{30, :minutes}
iex(4)> claims = Guardian.Claims.app_claims |> Guardian.Claims.ttl(ttl)
%{"exp" =><PHONE_NUMBER>, "iat" =><PHONE_NUMBER>, "iss" => "StreamTo.TV",
  "jti" => "7dc915a5-ff0b-4d41-bc1b-b19804550897"}
iex(5)> exp_at = claims["exp"]
1463768778
iex(6)> %{code: code, exp: old_exp_at} = Monitor.device_added(device.id, exp_at)
%{code: "LCBZH6WW", exp:<PHONE_NUMBER>}
iex(7)> ttl = {old_exp_at - :os.system_time(:seconds) - 1, :seconds}
{1709, :seconds}
iex(8)> claims = Guardian.Claims.app_claims |> Guardian.Claims.ttl(ttl)
%{"exp" =><PHONE_NUMBER>, "iat" =><PHONE_NUMBER>, "iss" => "StreamTo.TV",
  "jti" => "294a5c14-f5e4-4b57-9f30-e6551825ccaf"}
iex(9)> {:ok, token, _full_claims} = Guardian.encode_and_sign(device, :token, claims)
{:ok,
 "eyJhb...<skipped>...Zv3w",
 %{"aud" => "Device:db749ee8-1e9f-11e6-9d47-4b5e65de4b92", "exp" =><PHONE_NUMBER>,
   "iat" =><PHONE_NUMBER>, "iss" => "Issuer",
   "jti" => "294a5c14-f5e4-4b57-9f30-e6551825ccaf", "pem" => %{},
   "sub" => "Device:db749ee8-1e9f-11e6-9d47-4b5e65de4b92", "typ" => "token"}}
iex(10)>

As you can see it doesn't work. Is there a way to manually set TTL for a token available?
Thank you and best regards!

@vheathen not sure I'm following your example. When you use the ttl function you're effectively asking for  '30 minutes after the issue time'. If you want to set the ttl manually I believe you can just set the exp and iat in your encode_and_sign call.
Guardian.encode_and_sign(device, :token, %{"exp" => 123456432, "iat" => 124343534})

@hasson, thanks a lot for the answer!

When you use the ttl function you're effectively asking for '30 minutes after the issue time'.
Yes, you are absolutelly correct. But if you check result token fields you find that token's TTL is default 3 days and not 30 minutes as I believe it supposed to be:

1464026216 -<PHONE_NUMBER> = 259200 / 60 / 60 / 24 = 3 (days)
Even in the following case TTL doesn't change:
iex(21)> Guardian.encode_and_sign(device, :token, %{"exp" => 1100, "iat" => 1000})
{:ok,
 "eyJ...<skipped>...1Tg",
 %{"aud" => "Device:db749ee8-1e9f-11e6-9d47-4b5e65de4b92", "exp" => 260200,
   "iat" => 1000, "iss" => "App",
   "jti" => "c1e911ce-732f-467f-b569-97f5e26be066", "pem" => %{},
   "sub" => "Device:db749ee8-1e9f-11e6-9d47-4b5e65de4b92", "typ" => "token"}}

Token's time fields:  %{ ... "exp" => 260200, "iat" => 1000 ... }
Still default 3 days from Guardian config.
Thanks!

Hmm... sorry I can't pull up the code right now but if you pass it the ttl in the encode and sign it will work. It should work with a specified exp and iat so I think that's a bug.
 ttl = {old_exp_at - :os.system_time(:seconds) - 1, :seconds}
{:ok, token, _full_claims} = Guardian.encode_and_sign(device, :token, %{ttl: ttl})

Thank you very much! I just saw an example in Readme
claims = Guardian.Claims.app_claims
         |> Map.put("some_claim", some_value)
         |> Guardian.Claims.ttl({3, :days})

and tried to do the same, but it doesn't work (in the TTL part). I also tried to set exp manually, but also without success. I believe that this doesn't work because as I can see claims = Guardian.Claims.ttl(claims, {3, :days}) is doing the same - changing exp and iat according to ttl map.
The way with adding :ttl key/value directly to the claims map works!
Anyway, thank you very much for your help! I'm a newbee in the Elixir world so my tries to dig into the sources hadn't given me a lot about this topic. :)

I just tested both
    claims = Guardian.Claims.app_claims              |> Guardian.Claims.ttl({5, :days})
and
{:ok, jwt, _} = Guardian.encode_and_sign(       "thinger",       "my_type",       %{"exp" => exp, "iat" => iat})
and both are them are not working, the exp for both them are getting overwritten here
I am not entirely sure how to fix, we might need to rethink the way we set the ttl

It works with
claims = Guardian.Claims.app_claims
         |> Map.put(:ttl, {5, :days})
{:ok, jwt, _} = Guardian.encode_and_sign(resource, :token, claims)

So may be as a temporary workaround it's possible to put ttl key\value to the claims map on Guardian.Claims.ttl(ttl) in addition to iat and exp (which I'm using at the moment to not reinvent exp time calculation)?

Fixed by https://github.com/ueberauth/guardian/pull/164


Refactor to remove storage.MockClient and just use KV client

Codecov Report

Merging #118 (7ec88de) into main (da92381) will decrease coverage by 0.70%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #118      +/-   ##
==========================================
- Coverage   71.34%   70.65%   -0.70%
==========================================
  Files          54       54
  Lines        3029     3029
==========================================
- Hits         2161     2140      -21
- Misses        708      724      +16
- Partials      160      165       +5

Components
Coverage Δ

garden-app
70.65% <ø> (-0.70%)
:arrow_down:


How do I create customized 3d model??

I have an example fox_face.fbx file using that I created fox_face.sfb file that file works fine but i want to create my own 3d model for hat. How do I do that??

Are you asking how to make 3D models? You can use Blender, or Bforartists for that. There are plenty of tutorials on how to do 3D modelling.
Then you export to FBX and copy it to your Android Studio Project either from drag'n drop or file using the explorer.
Then you use right click your file and click "Import Sceneform Asset" (Assuming you've installed the sceneform plugin). This will generate an SFA which you can use to change Filament's properties and it also creates an SFB in your assets folder.
It will also add 3 lines of code to your app.gradle which will conflict if you try to import the model twice.
If you've changed the SFA you do not need to reimport. Unless some changes are not getting through, in which case you may need to delete the SFA, SFB and gradle code.
The remaining steps which is inclusion in your code i assume you've already set that up.

Thanks for your kind full reply
https://stackoverflow.com/questions/57776919/unable-to-set-proper-orientation-of-3d-model-in-ar-augmented-face/58080315#58080315
I have a problem like this but I don't know how to process further, could you help me with this


[BUG]: createTypedLocalStorage type inference fails with type boolean

Package Scope

Package name<EMAIL_ADDRESS>Describe the bug

createTypedLocalStorage<boolean> returns TypedStorage<false> | TypedStorage<true>.
Its set method has type (next: never) => void.

Expected behavior

createTypedLocalStorage<boolean> returns TypedStorage<boolean>.
Its set method has type (next: boolean) => void.

To Reproduce

TypeScript Playground
import { createTypedLocalStorage } from "@toss/storage/typed";

const isSignedIn = createTypedLocalStorage<boolean>("isSignedIn", {
  initialValue: false,
});

isSignedIn.set(true);  // Argument of type 'boolean' is not assignable to parameter of type 'never'.

Possible Solution

Additional context

thank you for reporting 😀


The library targets are not versioned.

Working on the Yocto Project meta-layer to natively list uvw in the OpenEmbedded Layer index and this sample application (the application delivered by uvw inside the test folder, brought in a separated repo), I got the error that the libraries provided (and installed) by this CMake-based project are not versioned (best-practice is that the libuvw.so lib is a symlink to the versioned "concrete" libuvw.so.2.11.2).
I'll work on it.

Cool, thank you!! Is it a problem to also port these changes to the v3 branch?
From what I see, the libuv crew is working on a v2 of the library and I'd like to merge our rework in the meantime.

Of course yes, let's cherry-pick it once the PR got approved, ok?

Can we close this one? 🤔

the PR is still not merged (because of 'required' workflows we not support anymore).
I cannot remove this blockers from the settings by myself, sorry for this.

Oho cuz I'm stupid! 😅
I merged only one of them, sorry. Is it ready?
Also, what about removing the required checks? I want you to be able to proceed on your own, since I trust you 100%.

I agree with removing the required checks.
The PR is ok to me.

I think you can merge it now? Just to double check that the required checks were turned off. 🙂


samples: driver: i2c_scanner: correct i2c message length

Some SoC i2c driver doesn't support zero length of I2C message, and they will ignore this transaction if so. Then the sample will fail.
Fixes  #14749 for STM32 and ATSAM boards
Signed-off-by: Aaron Tsui<EMAIL_ADDRESS>

@pabigot Could you help me to review this code? The i2c_scanner sample fails on ATSAM and STM32 chipset.

@pabigot
Yes, I agree with you that "Technically, even the zero-length transaction of getting its attention could make the device unusable."  I rise this PR  in a very tight situation,  it was few days before 1.1.4 release, and this i2c_scanner sample fails on STM32 and ATSAM.
I thought it's good enough for a demo. For your information, it "works" on several SoC, including STM, Nordic, NXP, and Atmel. Technically, just send one more frame and will still get NACK if no sensor attached.
Anyway, I will close this PR and try to address this in the I2C drivers for the failing SOCs sometimes.
I feel very sorry for bothering you at the weekend.

@overheat That it correctly diagnoses absence of a sensor isn't the issue; it's what happens when a sensor is present and there isn't a NACK.  Somebody could reasonably adopt this demo into an application and use it to determine which sensors are populated.  If an SDP811 (for example) becomes unresponsive because it only got the first half of its two-byte command that would be a problem.  The zero-length transaction, adopted from Linux i2c-detect, works on most devices so it's as safe as we can get.
No problem about the weekend; if it was I'd have just ignored the request until later.


FancyZone unittest failure - CreateZoneWindowWithActiveZoneTmpFile

This is failing on my machine locally.
CreateZoneWindowWithActiveZoneTmpFile
Source: ZoneWindow.Spec.cpp line 178
Duration: 137 ms
Message:
Exception Code: C0000005
Stack Trace:
_Atomic_storage<int,4>::load() line 922
logger::should_log() line 276
logwinrt::hstring() line 230
log<wchar_t [76],winrt::hstring>() line 94
trace<wchar_t [76],winrt::hstring>() line 100
trace<wchar_t [76],winrt::hstring>() line 22
JSONHelpers::ParseDeviceInfoFromTmpFile() line 698
FancyZonesData::ParseDeviceInfoFromTmpFile() line 498
ZoneWindowCreationUnitTests::CreateZoneWindowWithActiveZoneTmpFile() line 195

This test was poorly designed since it uses the actual temp file used by a running instance of PowerToys.
It will be obsolete when https://github.com/microsoft/PowerToys/pull/8148 will be merged since we will not use the temp file anymore.

This test was poorly designed since it uses the actual temp file used by a running instance of PowerToys.
It will be obsolete when https://github.com/microsoft/PowerToys/pull/8148 will be merged since we will not use the temp file anymore.

This test was removed in the commit 646d61bd4d848e10188e927d369a6459e805d91e


Fix teaser in Readme

An unfortunate missing space character prevented the rendering of teaser
code in the readme.

Pull Request Test Coverage Report for Build 628035872

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.121%

Totals

Change from base Build 604241709:
0.0%

Covered Lines:
223

Relevant Lines:
232

💛  - Coveralls


Typo in docs

This is simple enough where we can merge, but a CLA is needed and appreciated for any future changes! Upon CLA signing we'll send you a T-shirt also. Just DM me your address in slack.fnproject.io.

thanks!


Creating a new keyword in a plugin?

I was wondering if you would be open to exporting more of the parser internals to make it easier to support new keywords using a plugin. I am interested in adding support for DCI, for which I would like to add two new keywords, role and context.
I started working on a plugin for this but realized that many of the functions and variables needed to accomplish this (e.g. makePredicate(), ecma6Keywords) are not exported, so if I want to implement it as a plugin, currently my only option seems to be to do a lot of copying and pasting (and I'm concerned I might run into other issues if there's some internal variable that I actually need to modify).
A simple solution might be to export everything needed to create a new keyword, but prefix it with an underscore to indicate that those exports should only be used by plugins. Would you be open to this?
Thanks,
Matt

Keywords should not be added, not via new versions of spec nor via plugins as that might violate existing code assumptions. JavaScript has fixed set of those and even ES6 didn't add new ones, it just reused reserved words. Instead, the recommended way is to use concept called "contextual keyword" - regular identifiers that have specific meanings in one or other contexts.

Thanks, but now I am confused...the context keyword I want to implement would in some ways be similar to a class. I looked at how the class keyword is defined in acorn's source and it does seem to be implemented as a regular keyword. To be clear, I am intending to extend the language; here is some example code I would like to support:
https://github.com/mbrowne/typescript-dci/blob/master/samples/dci/js/TransferMoney-simplified/TransferMoney.ts
The role keyword should only work inside functions or contexts, but the context keyword should be allowed to appear anywhere that the class keyword could appear.
Also, to be a bit more specific about the issue in my original post, here's some code that I used in my plugin as a start on implementing this:
var ecma5AndLessKeywords = "break case catch continue debugger default do else finally for function if return switch throw try var while with null true false instanceof typeof void delete new in this";
var ecma6Keywords = " let const class extends export import yield super";
var dciKeywords = " context role";

//ES5
identifier.keywords[5] = makePredicate(ecma5AndLessKeywords + dciKeywords);
//ES6
identifier.keywords[6] = makePredicate(ecma5AndLessKeywords + ecma6Keywords + dciKeywords);

I also had to copy and paste the makePredicate function since it wasn't exported.
If there is a way to implement my new keywords without doing this, what existing keyword should I look at as an example?

I looked at how the class keyword is defined in acorn's source and it does seem to be implemented as a regular keyword.

Yes. I said above:

even ES6 didn't add new ones, it just reused reserved words

And class was a reserved word even in ES3. There is a good article by @mathiasbynens on this: https://mathiasbynens.be/notes/javascript-properties

The role keyword should only work inside functions or contexts, but the context keyword should be allowed to appear anywhere that the class keyword could appear.

As I said, the proper way to introduce new special-meaning word is not defining an incompatible keyword but implementing contextual one, that will have special meaning only in specific contexts and this won't break any existing code. Any language extension should be done responsibly enough and should definitely not break the original language rules. Acorn has isContextual, eatContextual and expectContextual helpers for that so that you can use them in any place you want to have your keyword.
One simple example of this is of pseudo-keyword which has special meaning only inside of for ... of statement: https://github.com/marijnh/acorn/blob/master/src/loose/statement.js#L58
Another, more complex one, is Flow plugin for Acorn used by Babel - you can find a lot of usages if you do Ctrl+F "contextual" against it's source code: https://github.com/babel/babel/blob/master/src/acorn/plugins/flow.js (it introduces Flow-specific keywords like module, static, type, implements and from; I'm pretty sure that contextual keywords should match your use-cases as well)

More relevant section from the Flow plugin is this part which parses interface declarations which is exactly what you want to do.

@sebmck IIUC, you didn't use contextual helpers purely for performance reasons there or ..?

@RReverser The contextual keyword has already been parsed.

@sebmck Ah right. Just wondering if any additional API is needed / would be useful in such case or rather redundant.

Thank you for mentioning the examples...I think the flow one will be particularly useful. BTW, I noticed that the flow is implemented as a plugin for Babel. The last time I looked (a couple months ago I think), Babel's fork of acorn didn't seem to support plugins; was plugin support recently added? If so, that's great...I was actually hoping to use Babel for the output generation so that my DCI implementation could be more simply integrated into the Babel stack.

Yes, plugin support was added to Acorn about a couple of months ago (though API is still in flux a bit).

One other question...what is the correct way to add a new property to an AST type? I would like to add a property isDCIContext to all functions to indicate whether or not the function contains DCI roles. I can of course just do:
node.isDCIContext = true;

(where node is the function node), but I was thinking there might be a constructor or prototype for function nodes somewhere that I could extend. It seems that's not how it works though...so maybe I should just extend parseFunction and ensure that isDCIContext gets set to either true or false there...?

Nope, no specific prototype because AST types are typed by type field which can be easily changed by various transformers and thus separate prototypes don't make much sense.

node.isDCIContext = true;

is the usual way of adding extra implementation-specific properties.

Closing this issue, but leaving it open for comments should you have any further questions.

Thanks again :)

I think adding a property like that is most easily done in a second pass, after the parsing. You can use the bundled AST walker (or a 3rd party one) and simply mark all parent nodes whenever the feature is found somewhere in the AST.

Thanks for the tip. Given that there are no references to parent nodes from child nodes, that probably would be the simpler way to implement it...maybe it can be part of the output generation step (for which I'm planning to use Babel).


JSON logging is lacking testing

We seem to be lacking tests for our JSON logging output, since there are reports of users with these unfortunate log messages - emitted directly by logrus when a type is not serializable to JSON, but was added to a log entry. Seen in https://github.com/cilium/cilium/issues/35080.
We should probably switch some of our CI to using JSON logging to catch these.
Failed to obtain reader, failed to marshal fields to JSON, json: unsupported type: map[types.Identity]*types.Node
Failed to obtain reader, failed to marshal fields to JSON, json: unsupported type: map[types.Key]policy.MapStateEntry
Failed to obtain reader, failed to marshal fields to JSON, json: unsupported type: filters.FilterFunc

Put up https://github.com/cilium/cilium/pull/35894 for the first instance.


Scaladoc in comments

Some method comments are not properly formatted for Scaladoc (shame on me), it'd be nice to review them all and format them properly.

https://docs.scala-lang.org/overviews/scaladoc/for-library-authors.html


Extended use of Shift key breaks text display

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Windows 10
Godot 3.0 Beta Build 1
Issue description:
When defining constants (for example PLAYER_MOVE_SPEED) the shift key is used for extended periods of time. This causes text to be displayed as boxes.

Steps to reproduce:

Create Script
Type for extended period of time with shift engaged.

Are you sure this is related to the holding of the Shift key? It seems very odd and I can't reproduce it.

It might not be directly related to holding shift down. It is as far as I know the only way to reproduce it on my pc.
Any ideas on how to find out what it directly related to is?

Is there any message in the console window?

Had similar bug once (but only on Quit dialog title bar), don't know how to reproduce it.

No messages pop up in the console window at the time of the bug. Though it might be useful to have:
Activated GL 3.3 contextOpenGL ES 3.0 Renderer: GeForce GTX 970/PCIe/SSE2 GLES3: max ubo light: 409 GLES3: max ubo reflections: 455, ubo size: 144 ARVR: Registered interface: Native mobile OPENING: E:/CloudSync/Programming/Godot/Locked Down Below (E:::CloudSync::Programming::Godot::Locked Down Below) running cmdline: "C:\Users\user\Desktop\Godot Engine.exe" "--path" "E:/CloudSync/Programming/Godot/Locked Down Below" "--editor" Activated GL 3.3 contextOpenGL ES 3.0 Renderer: GeForce GTX 970/PCIe/SSE2 GLES3: max ubo light: 409 GLES3: max ubo reflections: 455, ubo size: 144 ARVR: Registered interface: Native mobile ERROR: EditorData::get_edited_scene_root: Index current_edited_scene=-1 out of size (edited_scene.size()=0)    At: editor\editor_data.cpp:607 fragment uses function: zero_if_equal

Here's a video of how to reproduce this error.
youtube link

I've since seen this error once. It is not related to the use of the Shift key at all, there's something else going on.

I've faced this issue without any key related.
just moving around with mouse click & drag.

Another way to reproduce this bug #14736

Assuming this all have the same core cause, will use this issue as the main one.

Just recently had something slightly similar. Didn't hold Shift at all. Was also just a PopupDialog title being as buggy as your whole UI is in the screenshot above. The rest was fine and any following PopupDialog aswell.

I think this commit 3ca06e1184e49abd300bce03db1bec0e559c0d03 fix that issue.

Closing as fixed.


ocamlmerlin doe not work with ocpwin

simply execute ocamlmerlin.exe
Fatal error: exception Failure("Config file not found - neither C:\ocamlmgw\et
c\findlib.conf nor the directory C:\ocamlmgw\etc\findlib.conf.d")
I understand that it is expecting findlib conf , is tehre any plan to make it work with ocpwin install
(http://www.ocamlpro.com/pub/ocpwin/ocpwin-builds/ocpwin64/)

Exporting OCAMLFIND_CONF=somepath should fix your problem.
If you don't have an ocamlfind install, you can provide an empty file or maybe even NUL (/dev/null works under unix, I am unsure about windows semantics).

providing empty conf file solved the issue. but seems ocpwin is not the best option , I need to install other tools also ocp-indent , opam etc.
Thanks for response.

closing it as original issue resolved


List all elements that can be used in counter constructor in documentation

https://typst.app/docs/reference/meta/counter/:

Counts through pages, elements, and more.
With the counter function, you can access and modify counters for pages, headings, figures, and more. Moreover, you can define custom counters for other things you want to count.

What does "counters for pages, headings, figures, and more" mean? Are there other elements that can be used inside counter() or "more" refers to non-element things? I find it really frustrating every time I want to know if I can use counter for something built-in. IMO, documentation must state every single built-in element that can be used in the counter constructor. What about enum? Yes? No? I don't know! Documentation should answer this question, but it doesn't at the moment of writing.

I didn't list it at the time because it's a temporary limitation that it doesn't work for everything and it's bound to get out of date.
The countable elements are those that implement Locatable. I'm open to a PR that lists the exposed locatable elements although I think there are some for which it doesn't really make sense to use counter, e.g. bibliography.

I understand that the full list can become outdated, but:

we should avoid that (if we know that new/old element is/not Locatable, then this should be the signal to update docs);
better outdated than (almost) nothing.

Better yet, just paraphrase your concern + add a full list, so that the user will know that the list might be outdated (but at least it would be full). We can also add a version next to the list. This way, everyone will know for which version the list is relevant.


MvpLceActivity can't see Dagger2 injections

I have MyModule.java which provides a Presenter for Activity:
    @Provides
    public MyPresenter getMyPresenter(){
        return new MyPresenter();
    }

There is MyComponent.java which has:
public interface MyComponent {
    void inject(MyActivity act);

    MyPresenter getMyPresenter();
}

Now i am trying to inject that Presenter in MyActivity and use it:
public class MyActivity extends MvpLceActivity... {
@Inject MyPresenter myPresenter;
...
private void injectDependencies(){
    ....
}

 @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        injectDependencies();
        ....
}
@NonNull @Override
    public MyPresenter createPresenter() {
        return myPresenter;
}
...
}

It shows error:
java.lang.NullPointerException: Presenter is null! Do you return null in createPresenter()?

You have to call injectDependencies() before super.onCreate()  because the super.onCreate() will call createPresenter() internally.
Also, there is a variable presenter if you extend from a Mosby base class like MvpLceActivity. So in your MyActivity class you will have two variables pointing at the same presenter instance myPresenter  and ´presenter. The later one comes as protected field from MvpLceActivity`.
I would recommend you to take a slightly different approach regarding dependency injection with dagger2. I would recommend to change your code to something like this:
public class MyActivity extends MvpLceActivity... {
  private MyComponent component;
  ...

  private void injectDependencies(){
     myComponent = DaggerMyComponent.create();
    ....
  }

 @Override
  public void onCreate(Bundle savedInstanceState) {
        injectDependencies();
        super.onCreate(savedInstanceState);
        ....
  }

  @NonNull @Override
  public MyPresenter createPresenter() {
        return component.getPresenter();
  }
  ...
}

:+1:

@sockeqwe, @jemshit
Let me share a solution, which I use for the case if dagger-android extension is used.
We can use Provider<T> injection. Code will look like the following:
public class MyActivity extends MvpLceActivity... {
  @Inject Provider<MyPresenter> myPresenterProvider;
  ...

 @Override
  public void onCreate(Bundle savedInstanceState) {
        AndroidInjection.inject(this);
        super.onCreate(savedInstanceState);
        ....
  }

  @NonNull @Override
  public MyPresenter createPresenter() {
        return myPresenterProvider.get();
  }
  ...
}

Here we do injection in Activity by calling just AndroidInjection.inject(this) and don't need to have component instances.


Reading this TS file is causing MediaInfo to crash, the DLL throws a SystemAccessViolation causing the parent to also crash

When trying to read this file: https://www.dropbox.com/s/5d4ir8riltbb18w/Burgers%2C.Brew.%26.Que.S03E08.Beers%2C.Burgers.and.Bocce.in.Baltimore.ts?dl=0
MediaInfo crashes. While the GUI version throws an exception, the DLL version causes much more serious issues, it throws a System.AccessViolationException (corruption in protected memory) which also causes the calling (.NET) application to crash since managed code cannot handle System.AccessViolationException in .NET in a try catch block.

Fix will be in next release, and also in the next dev snapshots.


Generator non linearity missing?

Hey,
I noticed that the dcgan generator ends with a Tanh non-linearity whereas the mlp has no non-linearity at the end. Is this done on purpose? Am I missing something?

Hi! We just found this a few days ago. We double checked with the code we used for the paper and it's exactly the same (we forgot to add the tanh in there as well).
Strictly speaking the tanh unit is not necessary, since the MLP can (and should) learn by itself to produce pixels between -1 and 1. You can add the tanh if you want, I don't think it will make a difference (some students from the Montréal lab also ran the same experiment with tanh and got the same thing).


Add Fluxus.app v0.17

Fluxus is a livecoding environment for graphics, which can generate
animation from sound input, using a 3D rendering engine.

Thank you for the contribution. It will be merged in a different pull request due to some simple recommended alterations. Your contribution is still included (and still credited to you), with the appropriate modifications, when applicable.
Changes:

Facilitated future updates.
Removed extraneous comment.


Rewrite container docs for custom containers

This rewrites the container guide as we move to launch custom containers for Dataflow runner. There are two methods that should be highlighted - one is using the released Dockerhub images as a base image, and one is building from Beam source itself.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @tysonjh
R: @tvalentyn
R: @alanmyrvold
R: @rosetn
Current page: https://beam.apache.org/documentation/runtime/environments/

(some of my comments got hidden under Show Resolved due to new push, hope you can still find them :) )


Fix TestHtmlToPDF occasional issue

Describe the bug
GitHub actions sometimes failed due to this test sometimes raising an error. I've seen it happened once in development as well.

error line:128: context deadline exceeded

Lines in question:
if resp.StatusCode > 299 {
  b, err := io.ReadAll(resp.Body)
  if err != nil {
    t.Fatal(err)
  }
  t.Log(string(b))
  t.Errorf("expected status 200 got %s", resp.Status)
}

To Reproduce

First remove the t.Skip in this test in extras_test.go line: 116
Running this test multiple times in a row should trigger the issue:

$ make thistest TestHtmlToPDF

Expected behavior
This should always work and not trigger this error
Output of staticbackend -v
This flag isn't implement yet.

I would be willing to take this on.

Hi @dstpierre I took a look and this and managed to reproduce the error by adding --count 50  to the test and making the errors  slightly  verbose. It occurs when the context times out on chromedp.Run  in  htmlToX func.

extras_test.go:135: chome dp context deadline exceeded

The docs for chromedp.Run also state that Note that the first time Run is called on a context, a browser will be allocated via Allocator. Thus, it's generally a bad idea to use a context timeout on the first Run call, as it will stop the entire browser.
Possible solutions  to this would be:

Increasing context timeout period though this would not work cases where conversion to the respective formats take longer
Using contex without  timeout

You can assign this to me and also let me know your thoughts on the above approaches.

@Charlton-A thanks, nice diagnostic.
Ok, seems like the timeout wasn't a good idea after all.
In that case, yes please remove that work-around and worst case the go test default timeout will kick in if something is wrong with the initialization of chrome.


Implement find best m among n variables selection method

(although this may not make much sense in practice)

We can do this simply from what is returned by contribution_score():  contribution_score[contribution_score_per_variable]


[FLINK-32836][JUnit5 Migration] Migrate the checkpoint package of flink-runtime module to junit5

[FLINK-32836][JUnit5 Migration] Migrate the checkpoint package of flink-runtime module to junit5

Hi @X-czh , would you mind helping take a look in your free time? thanks~

Sure, no problem.


Master

Errorhandling...

The main reason to implement the onError prototype was, to have the possibility to overwrite it to handle errors after the first connect was successful.

by the way. commands array was not useful anymore since lightify gateway do not take other commands before it response previous command.  I have removed implement of 'sendnextcommand' months ago. please remove this method. Thanks

Hi, thank you for your answer. First to the handling of groups: The changes in nodeOnOff are kompatible, so it can be decided by caller. To handle all calles by user, you can implement an empty prototype isGroupCommand function returning. So other can overwrite it.
2nd: The usage of sendNextRequest is to handle automatic closing the socket. So you do not have to create a new object of the lightify class when necessary. You only have to use a command and the socket will be opend automatically. The buffers Array is, to not lose a command. sendNextRequest is to quickly send the next outstanding command. If not wanted, you call also implement an empty sendNextRequest function whithin simply writing a buffer if one is given.
If you implement this 2 functions I can close the other library at github and npm.
Thanks.


⚠ Bofrost.lu has degraded performance

In 75d5059, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1456 ms

Resolved: Bofrost.lu performance has improved in aa2da02.


Fixing the issue when the process id vanishes.

Cleans up any non-digit returned in the resulting process id query when the process has terminated.
In the event the process id vanishes, the "let" command in the subsequent line sets the value to 0 (which was the intended result when no process id was found).

ts3server_check fail for the win.

lol


Send native token max without dust

Fixes #378

Test failure does not appear to be related to this PR. It's also failing on master

Mainnet max estimate seems a bit high. Not enough eth left to pay for market gas price.

Mainnet max estimate seems a bit high

Interesting. I'm really glad you checked on mainnet! I actually hadn't. (I needed to republish my keys and I was too lazy/cheap to do it before now. 😅) But since I fixed my mainnet setup, I'm actually seeing the opposite -- estimates are too low:

What fee estimate are you seeing in metamask on mainnet @garyghayrat? This is what I'm currently seeing (on 9/12/22 ~3:45pm ET):

I added some logging and it seems like our mainnet estimates are actually consistently 20% too low, even with the 10% scale factor applied. I'm not sure why that is.
Metamask's estimated gas limit is ~64k for a mainnet sendETH:

And ours is 70671 -- exactly 10% higher than metamask's (as expected, given the scale factor). So the issue seems to be with gas prices.
Sure enough, the MM max gas fee is ~11% higher than what we're getting from our provider. And based on https://www.blocknative.com/gas-estimator theirs is correct. I'll need to look into this

@apbendi I was just able to reproduce the error on Arbitrum

I ultimately got it to work (send tx here) by bumping the native send gas limit but tbh I'm not sure why it was a problem to begin with. The gas used by the tx (372,580) was well under the original estimate (461,905). It seems that this might be an issue with metamask? Unclear. But I bumped the scale factor estimateNativeSendGasLimit for arbitrum by an additional 5% (10% total) and that seemed to fix the issue.
I was also able to reproduce on optimism. Optimism fees are just weird, so there was a real fix needed here. I have to get more OETH before I can test that the fix works though.
I wasn't able to reproduce the issue on mainnet. I was able to drain an account without issue. Could you re-confirm you have this problem?

Hey @davidlaprade, I gave this another look and tested out both Arbitrum and Mainnet. The "allowance" errors on Arbitrum are gone, however now both networks show this message in MetaMask:

This is the error I see when MetaMask is on "Market", which is the default. If I switch it to "Low", I'm able to confirm the transaction. So it seems maybe the price estimate for gas is just a bit lower than what MetaMask ends up using. Is there a way we can bump it up?

@apbendi I just rebased on top of master and retried max sends on mainnet and arbitrum, per:

I gave this another look and tested out both Arbitrum and Mainnet. The "allowance" errors on Arbitrum are gone, however now both networks show this message in MetaMask

I didn't get any errors. Native token max sends worked without issue on both networks for me.

This is the error I see when MetaMask is on "Market", which is the default. If I switch it to "Low", I'm able to confirm the transaction

This comment makes me wonder what your MetaMask settings are. I can't even see Market vs Low options unless I switch on an experimental feature (see screenshot). But actually even with this feature switched on I still had an arbitrum max send work without issue.
At this point I think it'd be helpful to hop on a call so I can see what you're seeing.

Alright @apbendi @mds1, I think I finally figured out the remaining issues we were having. I just rebased on master (again) and I think this is ready for re-review.
I have now tested:

native token max sends (all 4 supported networks)
native token partial sends (all 4 supported networks)
ERC20 token max sends (polygon)
ERC20 token partial sends (polygon)
receive native token (all 4 supported networks)
receive ERC20 (polygon)
native token max sends on the release preview (all L2s)

Everything worked!
cc @garyghayrat as there is one new string to translate


update game result handling

Codecov Report
Base: 72.25% // Head: 72.22% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (f66706b) compared to base (395b60f).
Patch coverage: 35.29% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #237      +/-   ##
==========================================
- Coverage   72.25%   72.22%   -0.04%
==========================================
  Files          60       60
  Lines        3453     3460       +7
  Branches      196      200       +4
==========================================
+ Hits         2495     2499       +4
  Misses        801      801
- Partials      157      160       +3

Impacted Files
Coverage Δ

src/shogi/csa.ts
76.85% <0.00%> (ø)

src/shogi/kakinoki.ts
82.32% <ø> (ø)

src/store/csa.ts
71.32% <28.57%> (-0.99%)
:arrow_down:

src/shogi/record.ts
75.33% <100.00%> (+0.08%)
:arrow_up:

src/store/game.ts
64.28% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Mixer limits

@devbharat This might be the more complete attempt at fixing all limitations. It would be great if you could do your integrator fixes on top of this.

@PX4TestFlights Be careful with this one and take off slowly as we changed the attitude control loop.


improve post-processing

Codecov Report
Attention: Patch coverage is 16.66667% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 88.29%. Comparing base (ef0b753) to head (731a724).

Files
Patch %
Lines

...kits/magnet_segmentation/backend/workflows/aedt.py
16.66%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #190      +/-   ##
==========================================
- Coverage   89.09%   88.29%   -0.80%
==========================================
  Files          12       12
  Lines         541      547       +6
==========================================
+ Hits          482      483       +1
- Misses         59       64       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[COR-618] /extensions/all route fails when updatedAt is empty

After publishing a new extension, registry.pgtrunk.io/extensions/all fails because there is no  value present for  updatedAt .
thread 'actix-rt|system:0|arbiter:0' panicked at 'called `Option::unwrap()` on a `None` value', src/routes.rs:32:39

From SyncLinear.com | COR-618
COR-619

Duplicated of https://linear.app/coredb/issue/COR-618/extensionsall-route-fails-when-updatedat-is-empty


Scale hdfs upgrade and README file

Added HDFS code for upgrade,README file and few other small changes!!
@rajan-mis @mamuthiah

Added few more small changes
@rajan-mis @mamuthiah

Added few more small changes
@rajan-mis @mamuthiah

Some multiple checked hence closing and opening new PR.

Some multiple checked hence closing and opening new PR.


itemsExpand still not work, fix again

fix itemsExpand and add initStyleToLayout in set strand

Hi Taro,
I want to review your change but I need an example where the actual solution fails. I need to understand where is the fail before looking at the change to know what I'm reviewing. Can you provide a simple test case? Thanks for this PR!! :)

Hi Yu,
your first code PR is just merged! Thanks for your help!, hoping to see more of this in the near future! :))

That's actually the third one :) First two were in Royale-doc. Keep up the good work and you will be rewarded! :)

Right! for that reason I said "code PR" since the other ones was for documentation ;-)


How much memory do I need to get the model run?

My mac os x system has 16G RAM and 4G GPU memory. It seems still not enough to run the model.
$ th run_model.lua
THCudaCheck FAIL file=/Users/eryunliu/torch/extra/cutorch/lib/THC/generic/THCStorage.cu line=41 error=2 : out of memory
/Users/eryunliu/torch/install/bin/luajit: /Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:11: cuda runtime error (2) : out of memory at /Users/eryunliu/torch/extra/cutorch/lib/THC/generic/THCStorage.cu:41
stack traceback:
[C]: in function 'resize'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:11: in function 'torch_Storage_type'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:57: in function 'recursiveType'
/Users/eryunliu/torch/install/share/lua/5.1/nn/Module.lua:126: in function 'type'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:45: in function 'recursiveType'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:41: in function 'recursiveType'
/Users/eryunliu/torch/install/share/lua/5.1/nn/Module.lua:126: in function 'type'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:45: in function 'recursiveType'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:41: in function 'recursiveType'
/Users/eryunliu/torch/install/share/lua/5.1/nn/Module.lua:126: in function 'type'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:45: in function 'recursiveType'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:41: in function 'recursiveType'
/Users/eryunliu/torch/install/share/lua/5.1/nn/Module.lua:126: in function 'type'
/Users/eryunliu/torch/install/share/lua/5.1/nn/utils.lua:45: in function 'recursiveType'
/Users/eryunliu/torch/install/share/lua/5.1/nn/Module.lua:126: in function 'type'
./densecap/DenseCapModel.lua:199: in function 'convert'
run_model.lua:148: in main chunk
[C]: in function 'dofile'
...nliu/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x010b13bd50

You are running out of GPU memory, not system memory. To run in GPU mode you need about 6GB of GPU memory.

Thanks! @jcjohnson

You can try running in CPU only mode by adding the flag -gpu -1

Processed image 2395976.jpg (996 / 1000) of split 1, detected 88 regions
Processed image 2395971.jpg (997 / 1000) of split 1, detected 101 regions
Processed image 2395960.jpg (998 / 1000) of split 1, detected 97 regions
Processed image 2395955.jpg (999 / 1000) of split 1, detected 86 regions
Processed image 2395949.jpg (1000 / 1000) of split 1, detected 102 regions
Loss stats:
{
mid_box_reg_loss : 0.0010207249870291
captioning_loss : 22.209516950607
end_box_reg_loss : 0.0019101264009718
end_objectness_loss : 0.040239814032614
total_loss : 22.343282449476
mid_objectness_loss : 0.090594833448529
}
Average loss: 	22.343282449476
IMG 30 PRED: a white and white line, GT: canopy has blue and white stripes. white and black awning falling down. , OK: 1, OV: 0.456080 SCORE: 0.141009
IMG 40 PRED: a man wearing a black shirt, GT: a white and green street bike. , OK: 1, OV: 0.241768 SCORE: 0.028571
IMG 50 PRED: a wooden fence, GT: the steps are stone. , OK: 1, OV: 0.237883 SCORE: 0.000000
IMG 80 PRED: the man is wearing a black pants, GT: surfers long sleeve black shirt. , OK: 1, OV: 0.387980 SCORE: 0.121905
IMG 110 PRED: a large brown tree, GT: ostriches have long necks. , OK: 1, OV: 0.248543 SCORE: 0.000000
IMG 121 PRED: a large tree in the background, GT: dark green trees on hill. , OK: 1, OV: 0.490476 SCORE: 0.000000
IMG 131 PRED: a black and black laptop, GT: black lamp on the right side of the room. , OK: 1, OV: 0.594925 SCORE: 0.067416
IMG 258 PRED: a man wearing a white shirt, GT: a baseball player. baseball player swinging a bat. baseball player wearing blue and gray. batter swinging the bat. a baseball player squatting with his arms and bat in the air. , OK: 1, OV: 0.498223 SCORE: 0.076433
IMG 345 PRED: a train on the train, GT: train tracks. , OK: 1, OV: 0.117586 SCORE: 0.186047
IMG 376 PRED: the sky is blue, GT: the sky is cloudy. , OK: 1, OV: 0.427383 SCORE: 0.323972
IMG 439 PRED: a man wearing a white shirt, GT: child sitting in bottom row far left. , OK: 1, OV: 0.415828 SCORE: 0.000000
IMG 492 PRED: the front of the bus is red, GT: black writing on the white license plate. , OK: 1, OV: 0.308367 SCORE: 0.024390
IMG 523 PRED: a wooden table, GT: a tall thin microphone. a black microphone stand. , OK: 1, OV: 0.144778 SCORE: 0.041885
IMG 638 PRED: the shirt is white, GT: the shirt is plaid. , OK: 1, OV: 0.347591 SCORE: 0.323972
IMG 648 PRED: the ground is made of the water, GT: wooden plank deck. , OK: 1, OV: 0.306353 SCORE: 0.000000
IMG 767 PRED: a white sink, GT: this is a metal. , OK: 1, OV: 0.376215 SCORE: 0.065041
IMG 788 PRED: a white and white plate, GT: a wet nap from  deli. , OK: 1, OV: 0.605932 SCORE: 0.021448
IMG 819 PRED: a white wall, GT: knives inside a glass case. rubber ducks atop a knife case. a  box of wood. a display box with a glass front. , OK: 1, OV: 0.801896 SCORE: 0.033058
IMG 861 PRED: the sky is blue, GT: it is foggy in the city. , OK: 1, OV: 0.367248 SCORE: 0.082474
IMG 872 PRED: a large street, GT: a platform outside of a building. , OK: 1, OV: 0.317051 SCORE: 0.035556
IMG 923 PRED: the sign is red, GT: a stop sign is visible. , OK: 1, OV: 0.427752 SCORE: 0.181107
IMG 944 PRED: a light on the wall, GT: a ladder on the bunk bed. , OK: 1, OV: 0.448346 SCORE: 0.116035
IMG 955 PRED: a black and white motorcycle, GT: this is a sidecar. , OK: 1, OV: 0.552734 SCORE: 0.059259
mAP: 3.937982
wrote checkpoint.t7.json
Warning: cudnn.convert does not work with nngraph yet. Ignoring nn.gModulewrote checkpoint.t7
Warning: cudnn.convert does not work with nngraph yet. Ignoring nn.gModuleTHCudaCheck FAIL file=/tmp/luarocks_cutorch-scm-1-6130/cutorch/lib/THC/generic/THCStorage.cu line=65 error=2 : out of memory
/home/rzai/torch/install/bin/luajit: /home/rzai/torch/install/share/lua/5.1/nn/Module.lua:301: cuda runtime error (2) : out of memory at /tmp/luarocks_cutorch-scm-1-6130/cutorch/lib/THC/generic/THCStorage.cu:65
stack traceback:
[C]: in function 'Tensor'
/home/rzai/torch/install/share/lua/5.1/nn/Module.lua:301: in function 'flatten'
/home/rzai/torch/install/share/lua/5.1/nn/Module.lua:318: in function 'getParameters'
./densecap/DenseCapModel.lua:374: in function 'getParameters'
train.lua:196: in main chunk
[C]: in function 'dofile'
...rzai/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x00406670
rzai@rzai00:/media/rzai/ai_data/prj/densecap$
rzai@rzai00:/media/rzai/ai_data/prj/densecap$
I got gtx1080 8G gpu card, but I ran with another program maybe take 1800M gpu memory and got this error.
I wiil try later to put all 8g mem to this program.