Issue: Can't install it on mac Cannot tap mingrammer/flog: invalid syntax in tap!

Hi!
I was going to try your interesting app, but following the instructions, it seems that it is not possible to install it via brew.
This is what I got after running brew tap mingrammer/flog:
==> Tapping mingrammer/flog
Cloning into '/usr/local/Homebrew/Library/Taps/mingrammer/homebrew-flog'...
remote: Enumerating objects: 39, done.
remote: Counting objects: 100% (3/3), done.
remote: Compressing objects: 100% (2/2), done.
remote: Total 39 (delta 0), reused 0 (delta 0), pack-reused 36
Unpacking objects: 100% (39/39), done.
Error: Invalid formula: /usr/local/Homebrew/Library/Taps/mingrammer/homebrew-flog/flog.rb
flog: Calling bottle :unneeded is disabled! There is no replacement.
Please report this issue to the mingrammer/flog tap (not Homebrew/brew or Homebrew/core):
  /usr/local/Homebrew/Library/Taps/mingrammer/homebrew-flog/flog.rb:6

Error: Cannot tap mingrammer/flog: invalid syntax in tap!

Do you know what we can do to install it in Mac using brew?
Best

I'm getting this instead
❯ brew tap mingrammer/flog

==> Tapping mingrammer/flog
Cloning into '/opt/homebrew/Library/Taps/mingrammer/homebrew-flog'...
remote: Enumerating objects: 39, done.
remote: Counting objects: 100% (3/3), done.
remote: Compressing objects: 100% (2/2), done.
remote: Total 39 (delta 0), reused 0 (delta 0), pack-reused 36
Receiving objects: 100% (39/39), 5.49 KiB | 2.75 MiB/s, done.
Resolving deltas: 100% (8/8), done.
Tapped 1 formula (12 files, 11.2KB).
❯ brew install flog
Error: flog: wrong number of arguments (given 1, expected 0)


Bugzilla save preparation

This PR contains various prerequisites to enable saving data through Bugzilla and generally enable the write operations through the API. There are also various random changes which I did along the way. I hope the commit messages explains the changes enough. I am creating this as a separate PR to bring it to the review sooner and in smaller pieces.
Except the random stuff there are three main types of changes.

One is preparation for the Bugzilla API key substitution from the user to the proxied Bugzilla query (in BBSync) so the change is done by the user and not by OSIDB service.
Second is restricting some API operations which does not make sense now. I am doing it in the backend not to break any API as it will be in the future allowed again. Please comment if you think it should be done other way.
Third is introducing API view validations. Some things are not available on the model level - namely user data and query parameters - and some are not mandatory going from the collector unlike from the user (ACLs).

It may be not clear enough how the BBSync part and the API view part will be connected in the middle - coming soon.
Closes OSIDB-692

Closing in favor of other better designed PRs. Though this was an important PoC.


Metrics sending moving to micrometer

See https://github.com/micrometer-metrics/micrometer/issues/66
spring-cloud-aws-actuator can be removed.

Closed via 33778b7


Adjust title order in learning toctree and fix some typos

Thank you for the tutorial. I am new at Fortran (I only wrote python before) and found the tutorial helpful.
But the tutorial toctree is alphabetical. I adjust the title order in https://fortran-lang.org/en/learn/quickstart/ sidebar
based on https://github.com/fortran-lang/webpage/blob/main/data/learning.yml.
I also fix some typos in my reading.
Welcome review. I am happy to modify if something I did wrong.

Thanks @0ut0fcontrol sir.

#build_preview

Do I need to run python3 intl.py and commit?  @henilp105

@0ut0fcontrol please feel free to do so. Thanks

I can rebase this branch or revert commit 7fa5c1a11174804020dfaa549c09fe51d9a22cb8 to resolve the conflicts, which one do you like?

#build_preview


MPI_Gather and geopm

I'm running into troubles doing a simple MPI_Gather of rank IDs when using geopm's profiling feature. The number of ranks is correct, but the results are garbage values. I'm running my example with 4 nodes and 8 tasks, 2 tasks per node. With geopm, I would expect the rank IDs to be 0 and 1.
I've put a reproducer code here:
git clone https://github.com/slabasan/mpigather-geopm.git
Let me know if you have any questions. Thank you!

Thanks for finding this bug and for the nice reproducer!  I think the fix is here:
https://github.com/cmcantalupo/geopm/tree/cmcantalupo-fix-gather-wrapper
Let me  know if that works for you.  We had forgotten to swap the communicator for the MPI_Gather() and MPI_Gatherv() wrapper routines.

@cmcantalupo Thank you! Problem fixed!


Aaron's solution

I'd say give this one another shot.  Use two spaces to indent.  And see if you can't substantially reduce the amount of code, and avoid the need for the return keyword.

Looks a lot better. :+1:
If you call a #split on the string first, you'll end up with an array of words, and then your regex can be a lot simpler.
Also, === is probably better than =~ here.  Whereas the latter returns the index of the first match, which can be cast into a boolean, the former returns a boolean directly.  Just remember that it's a regexp method, so the regexp has to go on the left side.


Adding books to a database using Node.js

I'd like to create a form that allows adding books to a database: (GSSoC'23)
Form may contain fields such as:

Name of the Book
Category
Short summary (optional)

Can u please assign me this issue?

@SonakshiA sure, go on

@SonakshiA sure, go on

Thanks! Do you mind if I use Bootstrap for the front-end part of the form? I'll do the DB part using Node.js and MongoDB

@SonakshiA certainly! using Bootstrap for the front-end part of the form while implementing the database functionality with Node.js and MongoDB is a great approach.

@rohansx
Might come off as a silly question since this is my first time contributing to open-source.
When connected to MongoDB Atlas, I surely can’t use my personal account, since you as the owner should also have the access to the Cluster and the Database?
In such situations, what MongoDB Atlas account should the Cluster and Database be connected to?

Hey @rohansx , I would also like to help with this issue, if it is not a problem with @SonakshiA .

I want to work with Backend as well Using ExpressJs and MongoDB

@rohansx
Might come off as a silly question since this is my first time contributing to open-source.
When connected to MongoDB Atlas, I surely can’t use my personal account, since you as the owner should also have the access to the Cluster and the Database?
In such situations, what MongoDB Atlas account should the Cluster and Database be connected to?

Hello @SonakshiA, you could create .env file for MongoDB URL.

Hi , i can help in adding these datas from the forms to mongodb. kindly assign this issue to me


README: Note instructions are for classic systems

thanks

ping me if I wait too long next time. somehow I missed the notification for this.


ASM: suggest block arg name for secret.generic

Similar to #1219 but does not depend on that.
In the past, we have seen secret.generic op like this
    %0 = secret.generic ins(%arg0, %arg1 : !secret.secret<tensor<8xi16>>, !secret.secret<tensor<8xi16>>) {
    ^bb0(%arg2: tensor<8xi16>, %arg3: tensor<8xi16>):
      %1 = arith.muli %arg2, %arg3 : tensor<8xi16>
      %2 = tensor_ext.rotate %1, %c4 : tensor<8xi16>, index
      %3 = arith.addi %1, %2 : tensor<8xi16>
      %4 = tensor_ext.rotate %3, %c2 : tensor<8xi16>, index
      %5 = arith.addi %3, %4 : tensor<8xi16>
      %6 = tensor_ext.rotate %5, %c1 : tensor<8xi16>, index
      %7 = arith.addi %5, %6 : tensor<8xi16>
      %extracted = tensor.extract %7[%c7] : tensor<8xi16>
      secret.yield %extracted : i16
    } -> !secret.secret<i16>

where the input and block argument are both named argX, which will cause confusion.
This PR intends to distinguish them by setting block argument to inputX.
By OpAsmOpInterface, there could be more readablity improvement, like change ^bb0 to ^body and secret.yield to yield, yet there are ~30 tests failing (heavily depends on the argX/bb0 behavior).
The full version is
    %computed = secret.generic ins(%arg0, %arg1 : !secret.secret<tensor<8xi16>>, !secret.secret<tensor<8xi16>>) {
    ^body(%input0: tensor<8xi16>, %input1: tensor<8xi16>):
      %0 = arith.muli %input0, %input1 : tensor<8xi16>
      %1 = tensor_ext.rotate %0, %c4 : tensor<8xi16>, index
      %2 = arith.addi %0, %1 : tensor<8xi16>
      %3 = tensor_ext.rotate %2, %c2 : tensor<8xi16>, index
      %4 = arith.addi %2, %3 : tensor<8xi16>
      %5 = tensor_ext.rotate %4, %c1 : tensor<8xi16>, index
      %6 = arith.addi %4, %5 : tensor<8xi16>
      %extracted = tensor.extract %6[%c7] : tensor<8xi16>
      yield %extracted : i16
    } -> !secret.secret<i16>

I was also thinking about simplifying the asm format like the below (as we have setArgAttr now), but does not know if such direction is favorable (definitely many changes to the test). If it is acceptable, I'll add an issue on this and work on it.
secret.generic (%arg0 -> input0 : tensor<8xi16> {attr}, %arg1 -> input1 : tensor<8xi16> {attr}) attrs = {} {
  %0 = arith...
} -> !secret.secret<i16>

I had an old issue to do a marginal improvement to the syntax: https://github.com/google/heir/issues/481
Part of the difficulty with your suggested syntax is that the operand for generic need not be secret (though it usually is). So the syntax actually needs to have both types. IMO the less custom we can make the syntax, the simpler our lives will be when migrating from upstream changes.
The other suggested changes are fine with me.


Add disableKeyboard option to disable changing slides on left/right a…

When slides contain input elements (text or number) moving in the inputs with left or right arrows makes the slides move.
Added a disableKeyboard optional flag to prevent this.

@express-crowbot Any chance this can be merge soon'ish? Thanks in advance.

@jpdelima I merged this. Will get it built.

This is published: v1.13.1


Scope for Cosmos Service Managed Key

Is the service managed key scoped to a specific Cosmos account or entire cosmos service?
Thank you!
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7c068a1f-f3bf-0ea8-ffff-c5626f6da77a
Version Independent ID: 3adef505-3bd5-a55b-9697-1febe37d0a6a
Content: Encryption at rest in Azure Cosmos DB
Content Source: articles/cosmos-db/database-encryption-at-rest.md
Service: cosmos-db
GitHub Login: @markjbrown
Microsoft Alias: mjbrown

@RonnieQuan We are checking this and we will get back to you.

It is scoped per account.

Thank you @markjbrown
@RonnieQuan - Please let us know if you have any questions in this regard.

@RonnieQuan We will proceed to close this issue now.
If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


fix: tsRestFetchApi content type parsing

fixes #335

I would've loved to see a test for this - but to be frank this is a pretty trivial change so I'm not worried

yup, I think it should be covered in https://github.com/ts-rest/ts-rest/blob/d122bb7dffc17c1a4bb5bde184ab0850c9f31ea4/libs/ts-rest/core/src/lib/client.spec.ts


KeyDown Throwing A System.NullReferenceException

So I am building a little script that uses a KeyDown Event and when I open the game it does nothing and when I added a try-catch statement to catch the error it did nothing too, so I went to check ScriptHookVDotNet2.log and got this:
System.NullReferenceException: Object reference not set to an instance of an object.    at TrackersV.TrackersV.TrackersV_KeyDown(Object sender, KeyEventArgs e)    at GTA.Script.raise_KeyDown(Object value0, KeyEventArgs value1)    at GTA.Script.MainLoop()
I tried to google this but got no results that actually help.

UPDATE: I switched it to keyup but it threw the same error.

Okay so I'm still getting this issue and I really don't get why because visual studio doesn't give me any build errors. here is the code

Okay I asked on GTA5-mods forum and got help there


Fix DependencySpecification test.

Forward port of sbt/sbt#2358.
This is a fixup of 0f616294c4e713dc415f5dc3ae7aef257decb228.
That commit assumed that dealiasing is being done for types referred in
self type. It was changed to not do that but the test wasn't updated.
Unfortunately, that mistake slipped by during PR review because unit tests
of compileInterface were not ran (see #2358).

This commit should be added into #48

Thanks, cherry-picked and pushed.


Make arrays available in Python as np.ndarray objects

Adds functions for returning Numpy views of ArrayView and SortedArrayView objects. With a Python extension module, these views can then be manipulated in Python. All that's needed is to pass the objects to one of the functions added in src/python. These will return a PyObject_t * which should be passed to the Python interpreter. There are two python extension modules, used for testing, in unitTests/python.

I pushed some commits (to be squashed later) which I think address all the comments so far.

This branch is still from your fork. I recommend we close this and switch over to feature/corbetts/python (#186). We can then both add commits to that going forward.


Jira integration - Error 500

Hi,
I'm trying to deal with an error with Jira plugin. All is working perfectly locally but when I'm deploying backstage on a kubernetes cluster. I've got error 500 with no more details. Quite strange since all is ok when I run backstage locally using yarn using same Basic Token.
Trying to call directly jira api, all is ok.
Checking on jira server, my domain is known.
curl -s  -H "Content-Type: application/json" -H "Authorization: ${JIRA_TOKEN}" "http://localhost:7007/api/proxy/jira/api/rest/api/latest/project/XYZ"

-> working

curl -s  -H "Content-Type: application/json" -H "Authorization: ${JIRA_TOKEN}" "https://jira.xxxxx.com/rest/api/latest/project/XYZ"

-> working

curl -s  -H "Content-Type: application/json" -H "Authorization: ${JIRA_TOKEN}" "https://myk8sdomain.com/api/proxy/jira/api/rest/api/latest/project/XYZ"

-> FAILED -> Error 500 - Error occurred while trying to proxy
[EPROTO] (https://nodejs.org/api/errors.html#errors_common_system_errors)

here config used locally and remotely:
proxy:
  '/jira/api':
    target: 'https://jira.xxxx.com'
    headers:
      Authorization:
        $env: JIRA_TOKEN
      Accept: 'application/json'
      Content-Type: 'application/json'
      X-Atlassian-Token: 'no-check'
      User-Agent: "ANYRANDOMSTRINGS"

Issue fixed - Bad network config on my side -  Sorry


Rustup to rust-lang/rust#97206

Thanks! @bors r+

:pushpin: Commit ecbade0016ce43dd6a69c71d2e7121903611f6ad has been approved by JohnTitor

:hourglass: Testing commit ecbade0016ce43dd6a69c71d2e7121903611f6ad with merge 0431504fc3ce5782cb4a27e90de5075a5c6138fc...

:sunny: Test successful - checks-actions
Approved by: JohnTitor
Pushing 0431504fc3ce5782cb4a27e90de5075a5c6138fc to master...


Support for maxBatchRecords

It would be nice to have method on query:
void maxBatchRecords(int max);
When someone will set non zero value on it, then after max additions to Batch, there should be called query.executeBatch().
See e.g.: http://viralpatel.net/blogs/batch-insert-in-java-jdbc/
This is common pattern to avoid OutOfMemory error when adding too many records to batch.
It can be very simply implemented in current solution.

I'll get on and develop this and i'll post a pull request on it.
You can review and feeback on my solution :)
I'll also add test coverage of course.

https://github.com/aaberg/sql2o/pull/198


outdated website

https://deployer.org/download/
does not have the current versions that are on packagist - spent an hour trying to figure why my application broke after running a composer install.
Turns out its at v6.5 since the 12th of october?
https://packagist.org/packages/deployer/deployer

Done. Sorry for the delay. Now I have more time for deployer.


Make the notify update optional

When being run aterm is checking for updates. However it might not be practical when running in a disconnected environement
Can you make the following call optional e.g from a configuration value ?:
https://github.com/ashirt-ops/aterm/blob/54a919d8d70dcce77775863de4a32b82dac74245/cmd/aterm/climain.go#L56

Ya, this is definitely something that we can make optional. Is this something that you're ok with being controlled from the config file and being default on but controllable via the setup routine?

Yeah, I think it would be fine to have it on by default as long as it is at least controllable either via a config file or a command line argument


docker container fails to start - connection error

After attempting to start the docker container Connection error.  docker exec -it chia venv/bin/chia show -s -c says "Connection error. Check if full node rpc is running at 8555
This is normal if full node is still starting up"
The container logs show
" File "/usr/lib/python3.8/asyncio/base_events.py", line 1463, in create_server
raise OSError(err.errno, 'error while attempting '
OSError: [Errno 99] error while attempting to bind on address ('::1', 55400, 0,
Daemon not started yet
Starting daemon
Daemon not started yet
Failed to create the chia daemon"
There is nothing running on the docker host using port 8555.

Seems simmilar to https://github.com/Chia-Network/chia-blockchain/issues/1535. It appears to be fixed in the latest version of docker image: https://github.com/Chia-Network/chia-docker/commit/9a3f5130196adf6a6908ef74c7073d173e79b374
have you tried to docker pull the latest image and only then start the container?

this does appear to be fixed in the latest version of the container. Closing issue. please re open if this issue persists!


Update ru_ru.json

Updated the old translations
Accidentally sorted the file in my IDE

Added to main.


[Model] RWKV Support

I hope the best UI can support the best model :)
https://github.com/BlinkDL/RWKV-LM

I think it looks really cool but for now we're focusing on models that can run on llama.cpp. I hope we can revisit this in the future and be more open but don't think we will tackle this for now :/


CSV Import - Space after "," in Header

Hi,
I want to import a CSV , but there are space behind the "," in the header row.
---
Internal id, Date and time, Transaction type, Coin type, Coin amount, USD Value, Original Interest Coin, Interest Amount In Original Coin, Confirmed
---

in the model I have defined the fields like this:
    internal_id = models.CharField(_('Internal id'), max_length=40, db_index=True)
    date_and_time = models.DateTimeField(_('Date and time'), db_index=True)
    transaction_type = models.CharField(_('Transaction type'), max_length=20, db_index=True)
    coin_type = models.CharField(_('Coin type'), max_length=10, db_index=True)
    coin_amount = models.FloatField(_('Coin amount'), db_index=True)
    usd_value = models.FloatField(_('USD Value'), null=True, db_index=True)
    original_interest_coin = models.CharField(_('Original Interest Coin'), max_length=10, db_index=True)
    interest_amount_in_original_coin = models.FloatField(_('Interest Amount In Original Coin'), null=True, db_index=True)
    confirmed = models.CharField(_('Confirmed'), max_length=5, db_index=True)

If I change the header row and replace the fieldnames to the internal variables the import is running perfect.
Is it possible to map fieldnames with a space in front and in the name to the import fields ??
Thanks a lot !!
Ingo

I believe the Fields API declared as part of your Resource covers the scenario you've outlined, i.e. coin_amount = Field(attribute="coin_amount", column_name="Coint amount").

Closing due to inactivity.  If this is still an issue you would like an answer for please add a new question on StackOverflow


[Question]std::async

I have seen in the CPP videos about std::async(). I have tried creating them but the program crashes at glCreateProgram()  or glCreateTexture()  & sometimes at glCreateProgram() . Googling out I found I need multiple contexts. Thus, in order to load meshes or textures asynchronously, we need multiple contexts (in glfw). But I don't know how is it done.
So my greatest question since then has been,  HOW DID HE LOADED THE MODELS ASYNCHRONOUSLY?
Did he created multiple contexts or did it in some other method?

Seeing as this question is related to the C++ series and not the game engine series, I recommend you head to the Discord. There are multiple people that can answer this question on there! :)

By default, you can only make GL context related calls from the main thread (the one that the context is located on). If you want to create an async task, you can, but you have to create a 2nd GL context that is set to share resources with the main context, or you can let the thread do the hard work, and the (final) context call(s) on the main thread exclusively...
As already mensioned, head to the Discord if you want to have a more in depth answer and/or help with implementation details.


Add to HACS

Please could this be added the the Home Assistant Community Store?

https://hacs.xyz/ (watch the gif)

I am looking at the gif. I assume the animated gif that shows how to use HACS. That does not say anything about why your Card Tools repository is not to be found in HACS.
There are many other front end cards that depend on it so I really wonder why this one is not there. If you click on the + in HACS and search for card tools it does not show up.
Are you and the maintainer of HACS not good friends or ...?

It is strange. I searched for both "card" and "tools" and it did not show up.
Then I tried to add the URL for the git-hub for this repo and got the message it was already there. Then I added my comment above on this bug report.
And after I went back and tried again looking for tools and then it popped up. It was like the attempt to add the repo changed something in HACS. In case someone sees this you may want to try the same

I was having the same confusion. Turns out, you don't have to add this as a custom repository.
To install via HACS:

Go to Community > Frontend
In the bottom-right, click "+ Explore & Download Repositories"
Search "card-tools" and click it
Click "Download"


Add integration test that confirm the hosting goal is correct after a merge

@keith-turner - The tablet availability is set and checked in MergeIT.merge. Is that sufficient to close this?


Update submodule picoruby for cross build on macOS.

Update submodule picoruby for cross build on macOS.
Additionally, I have formatted the README for better readability.
Apologies for the readability issues.

@yuuu
Sorry, but your PR became outdated.
I picked your suggestion at https://github.com/picoruby/R2P2/commit/9939cedd1e47946d0df7de865c56c15ad3cedce2


Cannot import ES modules from 'node_modules' folder

I have been using GraalJS in my Java application to execute Javascript code.
As per the documentation I have tried to customize the default FileSystem class to make it search for a file in the 'node_modules' folder if it is not found as a relative path. This effort was moot because the 'module not found' error occurs before any call to the FileSystem class (https://github.com/oracle/graaljs/issues/92#issuecomment-465859825).
I then configured both the FileSystem and js.commonjs-require-cwd using the engineWorkingDirectory variable below. This points to my 'node_modules' folder (/path/to/node_modules).
FileSystem fs = FileSystem.newDefaultFileSystem();
fs.setCurrentWorkingDirectory(engineWorkingDirectory);

Context.Builder contextBuilder = Context.newBuilder("js")
    .allowAllAccess(true)
    .allowHostAccess(HostAccess.ALL)
    .option("js.commonjs-require", "true")
    .option("js.commonjs-require-cwd", engineWorkingDirectory.toString())
    .fileSystem(fs);

This has provided mixed results.
In my index.mjs file, when I try to import a module from the 'node_modules' folder using var Clava = require("clavaapi/dist/clava/Clava.js"); I get errors complaining about the import and export keywords in 'Clava.js'. This would make sense if the engine assumes this is a CommonJS module. Why is it making this assumption? No idea. The point is, it finds and reads the file. That's good.
However when I try to import it using the ES module syntax (import Clava from "clavaapi/dist/clava/Clava.js";) I get an error saying that the module cannot be found:
org.graalvm.polyglot.PolyglotException: TypeError: Cannot load module: 'clavaapi/dist/clava/Clava.js'
Why is the module loader/resolver not searching in 'node_modules' folder when using the ES module syntax (I asume it is the JSModuleLoader class referenced in #92 )? The resolver algorithm documentation says it is... at least when not executing in Java embeddings mode.
What am I doing wrong here? Is this just not supported? Is there another option akin to js.commonjs-require-cwd that needs to be set? Can the module loader be customized to support this and passed to the Context builder as done with the filesystem?

An additional note:
Using import Clava from "../node_modules/clavaapi/dist/clava/Clava.js"; (relative path to 'node_modules' folder) works flawlessly.

Hi, maybe @iamstolis can answer these questions for you

This is @eleinadani's area.

Hi @lm-sousa,

In my index.mjs file, when I try to import a module from the 'node_modules' folder using var Clava = require("clavaapi/dist/clava/Clava.js"); I get errors complaining about the import and export keywords in 'Clava.js'. This would make sense if the engine assumes this is a CommonJS module. Why is it making this assumption?
...
when I try to import it using the ES module syntax (import Clava from "clavaapi/dist/clava/Clava.js";) I get an error saying that the module cannot be found:
org.graalvm.polyglot.PolyglotException: TypeError: Cannot load module: 'clavaapi/dist/clava/Clava.js'

When js.commonjs-require is enabled, Graal.js tries to resolve a CommonJS/ESM module using an experimental emulation of Node.js' module resolution algorithm.
In your example, I guess that clavaapi/dist/clava/Clava.js is loaded as a bare specifier: Graal.js tries to load the package looking for a package.json file; if the package.json is not found, Graal.js uses the file extension to decide how to load the file. In this case it's .js, so the file is loaded as CommonJS. This is equivalent to what Node.js would try to do (i.e., ESM_FORMAT in the resolution algorithm linked above). When you try to load the same file via import, Graal.js will look for the .mjs extension (and fail with a TypeError in case the file has a different extension).
Note that the actual file/package resolution depends on the content of your clavaapi and node_modules folders, so this is just my guess. If you have a package.json file and/or you think that Graal.js is loading the files differently than what Node.js would do, please share a full reproducer with us and we'll look into that (being the CommonJS/ESM emulation experimental, we might have be a bug on our side).

Can the module loader be customized to support this and passed to the Context builder as done with the filesystem?

The module loader cannot be customized at the moment. We might introduce such a feature in future versions, but for now the only way to customize module loading is by providing a customized FileSystem.

Here it is. All instructions on how to run it are in the README.md
lm-sousa/graaljs-test
In tests 2 and 3 I added an additional unexpected behavior I think is related to this one.

@lm-sousa great, thanks! I'll have a look and let you know

Updated the README. Forgot a couple of npm commands on the setup instructions.

@lm-sousa I had a look at your reproducer, and can confirm that this is a bug on our side. We will work on a fix, but cannot give you an ETA at the moment. In the meantime, you can of course use a package bundler to package your dependencies in a single file and load that without using the experimental js.commonjs-require flag.

Should be fixed by this commit. Please let is know if the fix works for you (and re-open this issue if not)

I've been looking into other things and just now had the time to circle back to this issue.
While your fix helped, it did not completely solved the issue.
The current state of affairs, using graaljs version 22.2.0 is:
Root folder: /home/REDACTED/graaljs-test

With js.commonjs-require as false

import { _hasClavaData } from "clavaapi/dist/clava/js/DataHandler.js"; does not work.

Error reading: /home/REDACTED/graaljs-test/clavaapi/dist/clava/js/DataHandler.js
Note the missing node_modules from the path. It hasn't been added by GraalJS.

import { _hasClavaData } from "node_modules/clavaapi/dist/clava/js/DataHandler.js"; works now, where it did not with graaljs version 21.0.0.

With js.commonjs-require as true

import { _hasClavaData } from "clavaapi/dist/clava/js/DataHandler.js"; does not work.

TypeError: Unsupported package exports: 'clavaapi/dist/clava/js/DataHandler.js'
It finds the node_modules folder but fails to identify the file as an ESModule even though both 'graaljs-test' and 'clavaAPI' folders have package.json files with "type":"module" indicating that the default behavior is to treat .js files as ESMs. Interestingly, changing DataHandler.js to DataHandler.mjs does not solve this issue.

import { _hasClavaData } from "node_modules/clavaapi/dist/clava/js/DataHandler.js"; somehow works with both DataHandler.js and DataHandler.mjs.

I conducted my tests with js.commonjs-require-cwd with one of these options yielding the same results:
Path engineWorkingDirectory = Paths.get(System.getProperty("user.dir"));
Path path = Paths.get(engineWorkingDirectory + "/node_modules");

/* (1) */ contextBuilder.option("js.commonjs-require-cwd", path.toString());
/* (2) */ contextBuilder.option("js.commonjs-require-cwd", engineWorkingDirectory.toString());

[Giving this issue a bump. Still relevant in version 22.3.1]

I think I solved it.
The TypeError: Unsupported package exports: 'clavaapi/dist/clava/js/DataHandler.js' error I am getting is not related to the ESM exports in the javascript file. It refers to the exports field in the package.json file. I noticed this by accident.
As soon as I removed the exports field from the package.json file, it all started working.
Maybe a more explicit error message could be shown as this one is ambiguous.

Documentation on how to achieve NodeJS-like loading of both CommonJS and ES modules from within a polyglot environment would be extremely helpful (whether the CommonJS coverage is via the experimental option, a JavaScript library, or something like https://github.com/transposit/graal-commonjs-modules).  Ideally one should be able to set a root directory beyond which searches for "node_module" would not continue.  Right now this is at best black magic.


[WIP] Added windows_path resource

Description
Porting windows_path resource from the windows cookbook to core chef
Issues Resolved
fixes https://github.com/chef/chef/issues/5809
Check List

[ ] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Specs are remaining.

The Appveyor and Travis failure don't seem to be related to this PR.

The travis failures aren't present on master


Fixed Typo in Big O Complexity table

Just a typo fix in the README.

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #8   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          76     76
  Lines        1596   1596
  Branches      282    282
=====================================
  Hits         1596   1596

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 3e0ac74...78923e8. Read the comment docs.

@breinhart  thank you for noticing and fixing that!


I want to go to the particular box edit!

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

hi @broisnischal that would be a nice feature!
however, I believe that's not possible when it comes to rendering a singular native input
check out https://github.com/guilhermerodz/input-otp/issues/32


remove docker version spec in ci

use the default docker version rather than specifying one per job.

Closing this PR, as it's obsolete.


Updated spelling

Updated spelling and punctuations for easy readability

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure (see below).
Error: Your .blue-bg class should have a background-color of blue

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


Add Inklet Cask

Thank you for the contribution. It was merged directly as 7d209e5bfa9a4f030bd263d45fb0d01ce7236737 to keep commit history cleaner. Your contribution is still credited to you.


nullable slug must not be null if base field has "0"

nullable slug must not be null if base field has "0"

thanks ;)

@l3pp4rd Can you tag maintenance release including this fix please?

should be available ;)


attributes and content undefined for .agx files when using injectContent('slug')

Please provide the environment you discovered this bug in.
Reproducible steps:

npm create<EMAIL_ADDRESS>(Blog)
Create a file example-post-2.agx in the src/content folder:

---
title: My Second Post
slug: 2024-07-18-my-second-post
description: My Second Post Description
coverImage: https://images.unsplash.com/photo-1493612276216-ee3925520721?ixlib=rb-4.0.3&ixid=MnwxMjA3fDB8MHxwaG90by1wYWdlfHx8fGVufDB8fHx8&auto=format&fit=crop&w=464&q=80
---

<template lang="md">

# Dynamic Example Post

Hello, world!

</template>

Run npm i && npm run dev

Which area/package is the issue in?
content
Description
When using injectContent with the slug, the attributes and content of .agx files are undefined. In the reproducible example above, the second post is included in the posts list on http://localhost:PORT/blog but if I try to access http://localhost:PORT/blog/2024-07-18-my-second-post, nothing is rendered.
In the pages/blog/[slug].page.ts I added the following:
readonly post$ = injectContent<PostAttributes>('slug').pipe(
    tap((x) => console.log(x))
  );

The console output is:

There is almost no resources on how to use .agx content files but from what I understand it should work exactly like .md content files without the need to adjust the injectContent or anything else in the code.
The behavior is the same regardless of the .agx file's content.
Please provide the exception or error you saw
core.mjs:6467 ERROR
TypeError: Cannot read properties of undefined (reading 'coverImage')
    at HomeComponent_Conditional_0_Template ([slug].page.ts:15:32)
    at executeTemplate (core.mjs:11264:5)
    at refreshView (core.mjs:12817:7)
    at detectChangesInView (core.mjs:13025:5)
    at detectChangesInViewIfAttached (core.mjs:12987:3)
    at detectChangesInEmbeddedViews (core.mjs:12947:7)
    at refreshView (core.mjs:12844:5)
    at detectChangesInView (core.mjs:13025:5)
    at detectChangesInViewIfAttached (core.mjs:12987:3)
    at detectChangesInComponent (core.mjs:12976:3)

Other information
The only resources about .agx files I found:

What happened to Analog's experimental ".ng" format? (Joshua's video)
https://github.com/analogjs/analog/pull/1012 (the original PR)
https://github.com/analogjs/analog/tree/61b1ea63fffc6eb6510cb1006922a92af9f0f3bb/apps/ng-app (an example app which has a single .agx file but uses .analog format for all pages and components which does not apply to Analog's default blog app)

That's about it, there's very little information about this feature and the official docs do not mention it at all.
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

Did you enable support for .analog/.agx files in the analog plugin in the vite.config.ts?
analog({
  vite: {
    experimental: {
      supportAnalogFormat: true
    }
  }
});


added files for automated workflow testing

Added a workflow that runs unit tests automatically.

Currently runs tests for all commits and all pull requests but this could be altered
Needs a local database every time so added a workflow_settings.py, if there was a way to command line override a variable we could set DB_INSTANCE to local but I've not been able to work this out yet
Analysis uses Django tests rather than unit tests. For future additions (e.g. SVIG, SWGS), we could run additional django tests or unit tests for these modules too

Unit tests run successfully as evidenced by nice green tick (:

Looks good, happy with running on every commit (I assume if you commit a few commits locally and then push, it'll only do the most recent? We can see how that works anyway...)
Happy with the separate settings file too, could probably have a base one and then inherit different configs off of it, but that's a future problem
All running as expected, thanks :)


Return None when remove the root node

Return None when remove the root node

Use
tree = tree.remove(point)
instead of
tree.remove(point)

Use
tree = tree.remove(point)
instead of
tree.remove(point)

Yes of course, I was just suggesting a tiny modification of the definition of remove


Bbl changes - clean up parameters, namespace, gitignore

See #6

Thanks, @bpbond!
Two questions/comments:

why are we gitignoring the .md files?
you deleted lock.cache=T in my make(plan) command. I've got the cache locked, cannot make plan error multiple times. Do you have a way around that?

Your .Rmd file transformed by rmarkdown to an .md file and then to .html; we want the first of those version-tracked, but not the others, as they're automatically generated from your Rmd file.

Hmm. I've never seen that error. Were you trying to invoke the plan from within a debug session perhaps? Otherwise I'm confused.

No, I was running the plan from a regular session. I think drake just doesn't like me. Anyway, the lock.cache=F is my workaround for now.

But if I change the material inside the .rmd, that won't be reflected in the .md unless I commit it, correct? Like if I add a new graph or new text.

same with gitignoring the figures. The .md file uses the .png files stored on GitHub. So if we don't update the figures, we're still displaying the old versions.

That's correct, and in general, that's how version control should work: we commit our 'source' files but not our 'products'. We commit source code, not the resulting software; commit Rmd, not html.


[ServiceBus] Message body type validation

Add validation such that Message construction fails fast if body is not a valid type.  Add changelog notes and a test for this scenario as well.

/azp run python - servicebus - tests

/azp run python - servicebus - tests

/azp run python - servicebus - tests

/azp run python - servicebus - tests

/azp run python - servicebus - tests

/azp run python - servicebus - tests


[DevTask] Support usage on Mobile

#35 reported that the editor is unusable on mobile devices (Phone, Tablet). I think add a button to control the zoom/pan mode is only part of the solution for the extension to be fully usable on mobile.
There are several things to do to make the editor usable on mobile:

The modal should take more screen space
The left panel should be foldable (Drawer)
2 finger pinch should zoom in/out the canvas
drag on canvas should pan the canvas
In selection mode, keypoints should be bigger for easier selection and drag.

Thank you


Added Yammer

I needed to add Yammer authentication for one of my projects, so I added in that functionality. It's my first time delving into the innards of Owin/Katana Authentication so please review. The "AcceptedNetworks" option  lets you optionally restrict authentication to members of specific yammer networks.

@ravind We are looking for help moving to ASP.NET 5. Please read http://www.jerriepelser.com/blog/calling-contributors-for-oauth-providers


web cam camera integration

I am trying to test your network on a web camera, however, since there is no need for LIDAR devices in testing, what steps should be taken as an easy fix to project the estimated 3D bounding boxes in LIDAR devices in testing, what steps should be taken as an easy fix to project the estimated 3D bounding boxes in LIDAR coordinate system to image plabe ( since the model unprojects to LIDAR coordinate system). I have the intrinsic matrix of the camera estimated by opencv library. Is P2 the projection matrix of the camera? Do we need to  change the values of tr_imu_to_velo and tr_velo_to_cam values? And what's your suggestion on how to change P2 projection matrix using only the intrinsic matrix? Do we weed to assume any rotation or translation? or for construction of such matrix, should we assume there is no rotation and transformation?

@codyreading

In other words, would you provide a guidline on how to test the model with new images from a camera and calibration matrices acquired from opencv?
We are using webcam images and replaced the calibration matrices, however we do not get proper results.
Since we are using single cameras, i believe for testing, the model should not require camera extrinsic parameters. (therefore it is confusing, however we assumed any extrinsic parameters givens should not affect the performance- Though not sure if that is the case).
-A general guideline would be helpful on how to test the pretrained model on a custom RGB camera. What hyper-parameters should be considered to be changed in order for the system to run properly and what hyper parameters should be tweaked with respect to the camera intrinsic parameters to have better performance.
-Another question would be, is there any work around to test the network on an image without having the intrinsic parameters of the camera? Let's say we have a youtube video, and all we have is the resolution of the image? Do you think the detector is capable of estimating the depth of the objects up to a scale?( meaning the relative positioning of objects in 3D world is estimated up to a scale and then using a reference(such as size of the vehicles) we can rectify the distances.)


alonzo purple 1.0.1 docker file and readme file

I am trying to create a pull request that adds the folder cardano-nodes to the mlabs folder and then a folder called alonzo to the cardano-nodes folder... And in that folder, there should be the Dockerfile and the readme.md file....

moved to https://github.com/mlabs-haskell/node-dockerimage


alert close/closed events fix

"close" works only after parent determined.
"closed" triggers before real "remove" when event handlers are present.

fixed in 2.0.1 thanks!

fat, it is still buggy :(
If you set "data-target" to some selector - "close" will trigger twice.
And "closed" event should be triggered after real closing alert - so element should be detached (so events are not removed) and only then we trigger "closed" on detached element and after - call remove to empty data & events.
If you try to do $(this).is(':visible') in "closed" event - it will be true, so at current time - close and closed events are same.

ah i see.. hm...

can you create a new ticket - and i'll take a closer look? - github won't let me reopen this for some reason :/

Okay, I will make ticket with my opinion tomorrow.

These are both still problems.
Thank you, v0id3r for the fix. It was driving me crazy.
fat, please update bootstrap with fix for these two issues.


Nav drawer, with topbar

Drawer lists sketches as thumbnails, in 2 columns.
Topbar has icons for accessing settings and creating a new sketch.

Done in #27


Roll Flutter Engine from 6d7842d25f81 to 5c3fff4844e9 (1 revision)

https://github.com/flutter/engine/compare/6d7842d25f81...5c3fff4844e9
2023-07-20<EMAIL_ADDRESS>Roll Skia from 401c85ab1e21 to b8133dda3a8c (4 revisions) (flutter/engine#43853)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Weights for pick n atRandom

There are currently some cards that have duplicated closures given to atRandom. This is a way to adjust the weights of the options given.
pick 1 atRandom (
    { doNothing() },
    { doNothing() },
    { summon 1 of 'Earth Totem' to 'you' zone 'Battlefield' },
    { summon 1 of 'Tree Totem' to 'you' zone 'Battlefield' },
    { summon 1 of 'Burning Totem' to 'you' zone 'Battlefield' }
)

Instead of doing this, there could be an optional argument specifying the weight of the option. The weight would be relative to the number of options: 1 would be the default, and 2 would be equivalent to writing the same option out twice.
Proposed syntax:
pick 1 atRandom (
    2 { doNothing() },
    { summon 1 of 'Earth Totem' to 'you' zone 'Battlefield' },
    { summon 1 of 'Tree Totem' to 'you' zone 'Battlefield' },
    { summon 1 of 'Burning Totem' to 'you' zone 'Battlefield' }
)

I think this syntax can be achieved by adjusting the integer meta class.

I like the idea, but I have doubts about the proposed syntax. I'm not sure that's possible that way, and even if it is I'm not sure it's a good way of implementing it.

Could we perhaps do something along the line of { 2 of doNothing() } or { doNothing() times 2 }?

@Phrancis The second one should work at least.
Another way to implement it could be using a closure as the argument to atRandom.
pick 1 atRandom  {
    option 2, { doNothing() }
    option { summon 1 of 'Earth Totem' to 'you' zone 'Battlefield' }
    option { summon 1 of 'Tree Totem' to 'you' zone 'Battlefield' }
    option { summon 1 of 'Burning Totem' to 'you' zone 'Battlefield' }
}

Not sure what to name the keyword (option in the example).

Alternatively, perhaps we could reuse the verbiage from withProbability(0.n)?
pick 1 atRandom  {
    withProbability(0.4), { doNothing() }
    withProbability(0.2), option { summon 1 of 'Earth Totem' to 'you' zone 'Battlefield' }
    withProbability(0.2), { summon 1 of 'Tree Totem' to 'you' zone 'Battlefield' }
    withProbability(0.2), { summon 1 of 'Burning Totem' to 'you' zone 'Battlefield' }
}

That might be a good (although verbose) idea, @Phrancis. I think we shouldn't require users to do the math to sum up the probabilities and instead make them relative to the number of choices as proposed above.
pick 1 atRandom  {
    withProbability(2) { doNothing() }
    withProbability(1) { summon 1 of 'Earth Totem' to 'you' zone 'Battlefield' }
    withProbability(1) { summon 1 of 'Tree Totem' to 'you' zone 'Battlefield' }
    withProbability(1) { summon 1 of 'Burning Totem' to 'you' zone 'Battlefield' }
}

Maybe the word probability would be confusing in this context? Maybe not. It's not really a probability in the normal sense (0-1) though.

Just to compete with another possible syntax:
pick 1 atRandom  (
    2, { doNothing() },
    { summon 1 of 'Earth Totem' to 'you' zone 'Battlefield' },
    { summon 1 of 'Tree Totem' to 'you' zone 'Battlefield' },
    { summon 1 of 'Burning Totem' to 'you' zone 'Battlefield' }
)

This makes the method atRandom accept a varargs parameter of integers and closures. When it encounters an integer, it uses the next closure X times. Or perhaps this syntax is even more confusing?

@Zomis I like how it is short, and a clever trick, but I do think it's a bit confusing. Mind you, if we document it, I think it could be OK. Personally I'd prefer to have a keyword for it like @jacwah and I mentioned, albeit I'd rather see it be used optionally and it default to "1 of" if the keyword wasn't present...
pick 1 atRandom  {
    withProbability(2) { doNothing() }
    { summon 1 of 'Earth Totem' to 'you' zone 'Battlefield' }
    { summon 1 of 'Tree Totem' to 'you' zone 'Battlefield' }
    { summon 1 of 'Burning Totem' to 'you' zone 'Battlefield' }
}

@Phrancis I think that would need to be done as shown below.
pick 1 atRandom  (
    withProbability(2) { doNothing() },
    { summon 1 of 'Earth Totem' to 'you' zone 'Battlefield' },
    { summon 1 of 'Tree Totem' to 'you' zone 'Battlefield' },
    { summon 1 of 'Burning Totem' to 'you' zone 'Battlefield' }
)

It's kind of a hybrid where withProbability "annotates" the closure.


slow and blocks vim on very slow network over ssh

vim or neovim?

[ ] vim
[x] neovim

Output of vim --version or nvim --version: NVIM v0.8.2
Output of :echo has("python"): 0
Output of :echo has("python3"): 1
Output of :py print(sys.version): 3.9.16 (main, Dec  7 2022, 10:15:43) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
Output of :py3 print(sys.version): 3.9.16 (main, Dec  7 2022, 10:15:43) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
Output of :echo g:Lf_Debug_Cmd: undefined
Output of :echo g:Lf_FilesFromCache: undefined
Operating system:

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Configurations related to LeaderF in vimrc:let g:Lf_DefaultMode = 'NameOnly'

Describe your question, feature request, or bug.
slow and blocks vim on very slow network over ssh
Steps to reproduce

open nvim over ssh

local: Putty on Windows
remote: MacOS

on the remote machine, download something that takes all the bandwidth, or anything else that makes the network very bad
open leaderf by the g:Lf_ShortcutF keymap
input quickly

Actual behaviour

the input area of leaderf keeps flicker, seems processing or redrawing something
vim seems blocked due to heavy script running or redrawing, not possible to stop processing or quit, even for <c-c>
stop input, and it keeps flicker for a very long time (at least for one or more minutes)

NOTE:

can not reproduce
I was downloading and installing Unity compents by the Unity Hub in background (on the remote MacOS), the CPU usage seems not the main issue, since these steps was tested (ordered by time):

the network was really slow, VNC refreshs very slow
use leaderf, the issue occur
close the Putty and start new ssh, use leaderf, the issue still occur
start new ssh, use ctrlpvim/ctrlp.vim, works well (with a bit lag due to the slow redraw)
start new ssh, use leaderf, the issue still occur, and prev step's ctrlp still works well
download finished, and all the issue disappeared

seems caused by frequent redraw, triggered by background jobs

Expected behaviour
do not block vim on this case

Please try let g:Lf_NoAsync = 1.


Return UTC timestamp until when requested power level is valid

What's needed?
The commands SetComponentPowerActive and SetComponentPowerReactive do not return anything at the moment.
The commands should, at least, return the timestamp until which the requested power will be active unless no further command is sent. Otherwise clients cannot know if the service will issue the command for an indefinitely long duration, or will stop it after a definite amount of time.
Proposed solution
The SetComponentPowerActive RPC should return an object of type SetComponentPowerActiveResponse. The SetComponentPowerActiveResponse message should be the following:
message SetComponentPowerActiveResponse {
    google.protobuf.Timestamp valid_until_ts = 1;
}

(Similar for the SetComponentPowerReactive RPC)
Use cases
No response
Alternatives and workarounds
No response
Additional context
No response

Implemented in #183 .


Max Cache limit Reached

Hi,
Max cache limit is reached (3000 vs. 3001), sync flush is performed.
Our orient version is 2.2.20.
Java version -- 8
Can you tell me what is the root cause for this warning..???

It means that we can not keep all operations are logged in operation journal in memory and data will be flushed to disk. I will provide a more meaningful text of message soon.

It is not critical warning at all. It means that because the cache is full some of the operations will suffer from higher latency than usual.

Ok thanks..

When will the cache becomes full..??? What is the data present inside this cache..?? Can u explain ..?

Cache become full because it is full by data of database operation journal. This journal is needed to support database durability and in case of DB crash, data are relayed back to restore data consistency.

@rbaarathinms did I answer your question? May we close the issue?

Yes you can close.. Thanks @laa ..


Ignore dead code warning.

@a-siva can we close this?

seems to have been fixed by another PR, will close this.


[YoutubeDL] Add --client-certificate option

Please follow the guide below
Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[ ] New extractor
[x] New feature

Description of your pull request and other information
Some sites only allow access to HTTPS connections that offer a valid client certificate. In some cases, the client certificate may be built into the firmware of a client device. While such certificates are typically not accessible without jail-breaking the device, it would be desirable to allow this means of authentication as well as username+password, etc.
This PR adds an option --client-certificate whose value should be the filename of a single certificate file containing the certificate and private key (as described in the Python library documentation for the ssl module under 'Combined key and certificate').

The new test case test_check_clientcertificate failed for Python >= 2.7.8 because yt-dl sets the HTTPS client to validate the server certificate, and this failed with the test HTTPS server from test/test_http.py because self-signed certificates don't pass.
If 'nocheckcertificate': True is passed along with the clientcertificate option in the test_check_clientcertificate test case, the test succeeds.
The tests for HTTPS in test/test_http.py don't check that HTTPS connections work in general, but only that  the nocheckcertificate option works. A similar test-case without that option fails, but this will happen unless the server certificate has a valid path to a root CA. Unfortunately, arranging proper SSL certificates for testing seems to be a hopeless task.


Cranelift: don't hoist instructions whose results are only used in cold blocks

.clif Test Case
Wasmtime currently translates this WebAssembly module:
(module
  (type $fn (func (result i32)))
  (table $fnptrs 2 2 funcref)
  (func (param i32)
        loop
          local.get 0
          call_indirect $fnptrs (type $fn)
          br 0
        end))

To this CLIF, before optimization:
function u0:0(i64 vmctx, i64, i32) fast {
    gv0 = vmctx
    gv1 = load.i64 notrap aligned readonly gv0+8
    gv2 = load.i64 notrap aligned gv1
    gv3 = vmctx
    gv4 = load.i64 notrap aligned gv3+72
    sig0 = (i64 vmctx, i64) -> i32 fast
    sig1 = (i64 vmctx, i32 uext, i32 uext) -> i64 system_v
    stack_limit = gv2

                                block0(v0: i64, v1: i64, v2: i32):
                                    v3 -> v2
                                    v28 -> v3
@0027                               jump block2

                                block2:
@002b                               v4 = iconst.i32 2
@002b                               v5 = icmp.i32 uge v3, v4  ; v4 = 2
@002b                               v6 = uextend.i64 v3
@002b                               v7 = global_value.i64 gv4
@002b                               v8 = ishl_imm v6, 3
@002b                               v9 = iadd v7, v8
@002b                               v10 = iconst.i64 0
@002b                               v11 = select_spectre_guard v5, v10, v9  ; v10 = 0
@002b                               v12 = load.i64 table_oob aligned table v11
@002b                               v13 = band_imm v12, -2
@002b                               brif v12, block5(v13), block4

                                block4 cold:
@002b                               v15 = iconst.i32 0
@002b                               v16 = global_value.i64 gv3
@002b                               v17 = load.i64 notrap aligned readonly v16+56
@002b                               v18 = load.i64 notrap aligned readonly v17+72
@002b                               v19 = call_indirect sig1, v18(v16, v15, v3)  ; v15 = 0
@002b                               jump block5(v19)

                                block5(v14: i64):
@002b                               v20 = global_value.i64 gv3
@002b                               v21 = load.i64 notrap aligned readonly v20+64
@002b                               v22 = load.i32 notrap aligned readonly v21
@002b                               v23 = load.i32 icall_null aligned readonly v14+24
@002b                               v24 = icmp eq v23, v22
@002b                               trapz v24, bad_sig
@002b                               v25 = load.i64 notrap aligned readonly v14+16
@002b                               v26 = load.i64 notrap aligned readonly v14+32
@002b                               v27 = call_indirect sig0, v25(v26, v0)
@002e                               jump block2
}

Steps to Reproduce
Optimize this CLIF with the egraph pass enabled using opt_level=speed.
Expected Results
I think the loads of v17 and v18 should stay in the cold block.
Actual Results
The loads of v17 and v18 are hoisted to the entry block, since they have the notrap and readonly flags set and depend only on the vmctx parameter, v0.
Versions and Environment
Cranelift version or commit: tested on afaf1c73f67c42e638590608579b2b02c7f65ca3
Extra Info
This is not a correctness bug but might be a performance problem. I haven't measured it, just noticed it while looking at the CLIF that Wasmtime currently generates for call_indirect.
cc: @cfallin

I suppose we could add a(nother) special case to the block-placement logic, where LICM etc is implemented: if the original block is cold, keep the op in the same block. That's probably limited in scope enough that it's both (i) tractable and (ii) shouldn't have unforeseen impacts?


Filter by file type

Hello,
Would it be possible to have an option to filter by file-type?
The use case is that I have pdf I do not want to delete, no matter whether they are linked or not -- a contrario, I would like to only target .jpg or .png.
Thank you,

Modified the code on my own, but editing main.js and deleting the .pdf from line 142.


Document how to set colors in plot_geometry

I cannot figure out how to set the color of an object when using plot_geometry such as here:
plot_geometry(
    (Segment((0, 0), (1, 0)), 'r')
)

I've tried adding variations on the following with different key names in the dictionary but nothing works for me:
plot_geometry(
    (Segment((0, 0), (1, 0)), 'r', {'color': 'red'})
)

Searching through the matplotlib backend links didn't reveal the magic keyword I need to be using. Maybe it would be good to demonstrate how to do this in one of the documented examples for plot_geometry?

Hello James, that's a bug...
For the moment you can use the rendering_kw keyword argument:
plot_geometry(
    Segment((0, 0), (1, 0)), 'r', rendering_kw={"color": "g"}
)

I just released a new version which fixes this bug. It's already available on PyPi and should become available on Conda in a couple of days.
Thanks James for opening this issue. I'm closing it now.


Ignore pipenv when python.pythonPath is already set

Environment data

VS Code version: 1.23.0
Extension version (available under the Extensions sidebar): 2018.4.0
OS and version: Windows 10 Enterprise 2016 LTSB 10.0.14393.0
Python version (& distribution if applicable, e.g. Anaconda): Python x64 3.6.5
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pipenv
Relevant/affected Python packages and their versions: pipenv 11.10.0

Actual behavior
When I open my project folder I get the following error:

Expected  behavior
I should get no error. My .vscode/settings.json is properly configured. This same project folder was working a week ago, but now when I open it I get the error cited above.
Workspace settings
{
    "python.pythonPath": "C:\\Users\\<username>\\.virtualenvs\\amazing-project-xyz12Abc\\scripts\\python.exe"
}

Steps to reproduce:

Open a project folder configured to use pipenv

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
##########Linting Output - pylint##########
No config file found, using default configuration

--------------------------------------------------------------------

Your code has been rated at 10.00/10 (previous run: 10.00/10, +0.00)

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)
console.ts:136 [Extension Host] vscode-icons is active!
console.ts:136 [Extension Host] Error: No virtualenv has been created for this project yet!
	at PipEnvService.<anonymous> (C:\Users\<username>\.vscode\extensions\ms-python.python-2018.4.0\out\client\interpreter\locators\services\pipEnvService.js:113:31)
	at Generator.next (<anonymous>)
	at fulfilled (C:\Users\<username>\.vscode\extensions\ms-python.python-2018.4.0\out\client\interpreter\locators\services\pipEnvService.js:15:58)
	at <anonymous>
t.log @ console.ts:136

Please could you type the following in your terminal window and let me know what you get:
pipenv --venv

I get the same message on Mac OS. pipenv --venv gives me the correct location of my virtual environment (/Users/<my_username>/.local/share/virtualenvs/<my_folder_name>-50WkzGzt)

@maximegregoire what happens if you unset python.pythonPath in your settings? And what if you have pipenv recreate the virtual environment?
Basically the solution for us would be for us to ignore pipenv if you set python.pythonPath.

@brettcannon Unsetting the pythonPath setting did the trick! Thanks!

@DonJayamanne Sorry for the late reply. I get the following:
PS C:\Users\...\amazing-project> pipenv --venv
No virtualenv has been created for this project yet!

@juanchristian
Ok, this mean you do not have a pipenb environment setup for the current folder. You need to create one using the commands pipenv install or other. Please check the documentation for pipenv.
So the last question (before I close this issue) I have is, do you not want to use pipenv environments?

@DonJayamanne the issue is that I already configured pipenv for this exact folder, hence I had the folder C:\Users\Juan Christian\.virtualenvs\amazing-project-zqy49Rxo automatically generated for me. It was working as usual, but out of the blue it started giving me this error.

Please could you try i/running the command in a native (non-vscode terminal) and provide the output?
Commands to try are pipenv --venv and pipenv --py

The result is:
PS C:\Users\<username>\Code\amazing-project> pipenv --venv
No virtualenv has been created for this project yet!
PS C:\Users\<username>\Code\amazing-project> pipenv --py
C:\Users\Juan Christian\AppData\Local\Programs\Python\Python36\python.exe

By the looks of it, it's not finding my virtualenv :/

I'm sorry to say this, but this doesn't look like a problem with vscode. You seem to be having problems with pipenv. The fact that you are having issues outside of vscode confirms this.

Sorry then, should have opened a ticket on pipenv. Anyway, just to give a proper response to this ticket: I deleted my old pipenv folder and re-install pipenv with pipenv install and now everything is OK again.

Sorry for commenting on a closed ticket, but I just got hit with that too. The behaviour is VSCode refuses to see pipenv environment and cmd/PS can see it just fine. I did some debugging and it turns out (on Windows) cmd/PS always use an uppercase letter for the drive while VSCode started passing a lowercase letter. This causes pipenv to count a different hash to the virtualenv directory and hence the difference. This SHOULD be fixed in pipenv I guess (but could also be handled in getPipenvWorkingDirectory).


LA calculating the wrong address (off by -4)

(Copied from StarDot at https://www.stardot.org.uk/forums/viewtopic.php?p=401976#p401976 for completeness — you've already replied on there.)
If I assemble the following:
10 DIM code 100
20 FOR opt=0 TO 3 STEP 3
30 P%=code
40 [ OPT opt
50 ALIGN
60 .start
70 LA T0,text
80 LI A7,4
90 LB A0,0(T0)
100 ECALL
110 RET
120 :
130 EQUS "ABCD"
140 .text
150 EQUS "EFGH"
160 ]
170 NEXT

... then do CALL start, I get an A printed, when I would expect an E (i.e. T0 appears to be loaded with an address 4 lower than it should be).
I think the actual LB A0,0(TO) is being assembled correctly, though.

Currently the following pseudo instructions can emit either one or two instructions:

LA
LI

I'm open to the idea of removing the optizations, and always emittimg a two of instructions in each case.
Or just doing this for LA, as it would be unusual to use LI with a label?
Do you have any thoughts Robert?

(Remembering my knowledge of RISC-V is one and half days old...)
I haven't thought about LI but I think it's unlikely to use it with a label directly, although I can see people doing calculations based on the label (e.g. to work out the size of a block by subtracting one label from another).  Searching quickly, I haven't found a good explanation of what it encodes into (the cheat sheet says "Myriad"!) — is there a 'most common' size we could assume, if we didn't have a label defined yet?
With LA, though, I would have thought it was very rare to use a single instruction as it encodes to AUIPC for the top 20 bits and then ADDI for the bottom 12 bits: for it to not need the ADDI, the offset from PC would have to be an exact multiple of 2^12 (= 4096).  You could assume it needs two, if the label was undefined, and then assemble one if not (and require a third pass) — at least that would get it right in the majority of cases, I think.
If you really did want to omit the ADDI, I would think you could just use AUIPC manually.  As an aside, I'm not sure how this is supposed to operate — e.g. whether it's supposed to mask out the bottom 12 bits and shift right by 12 to work out the offset from a 32-bit pointer for you, or you have to do that yourself.  Looking at the code here https://github.com/hoglet67/BBCSDL/blob/master/src/bbasmb_riscv.c#L583 I assume you need to do it manually.
I think they made a bit of a mistake here, encoding things to different lengths implicitly.  With ARM, I think you to explicitly specify a 'long' version of a pseudo instruction and would get an error if you used the short one but needed a long.
All this said, there's clearly enough horsepower on any modern machine that's not going to make the size of programs you write in BBC BASIC's assembler to not matter doing three passes: maybe we just need to get in the habit of doing it!

I will definitely update LA to always emit two instructions.
My inclination is to leave LI as variable, for the following reasons:

the one-instuction forms will be very common
it would normally be used with constants, not labels
if used with, it's more likely to be label expressions, e,g, table_end - table_start, which would for small tables evaluate to the one-instruction form in both passes

There are corner cases, e,g.  table_end - table_start for a large table.
Unfortunately in the assembler code expressions are completely opaque, in that I have no visibility of whether they use labels or not, and whether those labels are defined. So doing anything "smart" is not really an option.


[Request]configure what item in a compacting drawer is reported to a external storage system

im trying to autocraft a recipe that uses alot of nuggets using applied energetics 2 but it wont allow me as there is a issue with how the compacting drawers report their contents to the ae system (its reported 3 times basically) it would be nice if there was a upgrade that would allow you to select what slot to report

AE2 can already make this work by configuring your ME Storage Bus with a partition and configuring it to only see items that are in the partition.


ROX-14702: Display origin for access scopes, roles, auth providers and groups

Description
Following up steps and pattern of changes made here https://github.com/stackrox/stackrox/pull/4894, in this PR I add origin field for other resources in the UI: roles, access scopes, auth providers and groups.
As groups are tightly bound to auth provider and are only rendered on the auth provider page, we do not display groups' origin anywhere in the UI. The assumption is that groups and corresponding auth provider always will be of the same type - either both imperative or declarative. The only change made for groups is that groups with Declarative origin are impossible to modify or delete within the UI.
For every other resource next changes are introduced:

Origin field is displayed when listing all resources of a certain type
Based on Origin field resources might be disabled for modification and deletion
Origin field is added to the resource page

Integration tests will be added in the follow-up PR.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed

/test gke-postgres-ui-e2e-tests


GlobalAveragePooling2D causes the inconsistency of output between frameworks

Click to expand!
Issue Type
Bug
Source
source
Tensorflow Version
tf2.0
Custom Code
Yes
OS Platform and Distribution
Ubuntu 20.04.2 LTS (GNU/Linux 5.4.0-74-generic x86_64)
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/Compiler version
gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
CUDA/cuDNN version
CUDA Version: 11.2
GPU model and memory
No response
Current Behaviour?
Operators with Sum and Avg often cause inconsistency between frameworks. And for this inconsistency, only changing the input may cause its nature to change.

（Inconsistency: When testing the three frameworks, the inconsistency here refers to comparing the output of the GlobalAvgPool2d layer with the other two frameworks and computing their absolute values.）

Standalone code to reproduce the issue
structure：GlobalAvgPool2d + Flatten + Dense + Softmax
dataset：mnist
batchsize：25
When batchsize is 25， it will appear as Tensorflow Inconsistency.

Relevant log output
No response

@Crystal-yyy
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!


Syncing of player balance is broken between SSDynamic Shop and CMI-Economy plugin, and possibly other economy plugins.

SERIOUS GAME ECONOMY BREAKING ISSUE    ......(PUGIN IS NOT USABLE)......
This plugin has been broken for several months with CMI Vault economy which I posted previously in discord, and am posting again here.
This issue appears related to issue #2
I have recently discovered a major disconnect between CMI-Vault Economy and DShop.  (I have been running DShop and CMI on my dev server for 9mo w/o issue until recently. (I suspect over the past few months is when things broke)
Example 1)

Player starts with $0 balance
Use /cmi money give  10000
-- Player types /bal  (balance is $10,000 great)
Next player goes to a DShop and tries to buy something (DShop says you have insufficient balance $0)

Example 2)

Player starts with $0 balance
Player goes to DShop and sells a ton of stuff
-- Player has $1,000 in the shop
Player goes to another DShop and can buy items with his $1000
Player types /bal (balance is $0)

Issue has been resolved do to a mis-configuration with my Bank plugin.  SS Dynamic Shop is still working flawlessly as it always had been prior to this issue.


Add notes on template compiler restrictions

There are some undocumented restrictions on how templates need to be written (e.g., with conditionals in element attributes).  Add notes in readme.

#75


Add an exception for partial ident in the space-around-operator rule

This PR fixes #929 which was giving a false positive when a hyphen was used in-between two interpolations. Since this hyphen - is a partial ident, an exception has been added to the rule.
$foo: #{$a}-#{$b};

<DCO 1.1 Signed-off-by: Ben Griffith<EMAIL_ADDRESS>

Coverage increased (+0.004%) to 97.559% when pulling 1fb6ca063ed52304b20dbd5510d29170155a1aba on bgriffith:feature/add-ident-exception into 03e51bbaadcd4d1384f56e34c8d9f13758e770d8 on sasstools:develop.

Coverage increased (+0.004%) to 97.559% when pulling 1fb6ca063ed52304b20dbd5510d29170155a1aba on bgriffith:feature/add-ident-exception into 03e51bbaadcd4d1384f56e34c8d9f13758e770d8 on sasstools:develop.

Coverage increased (+0.004%) to 97.559% when pulling 7b420524afde60e014957a92715e59930a549afc on bgriffith:feature/add-ident-exception into 03e51bbaadcd4d1384f56e34c8d9f13758e770d8 on sasstools:develop.

Coverage increased (+0.007%) to 97.561% when pulling 5d87a11606aa88a8f640c5d98f2c54d475ac368f on bgriffith:feature/add-ident-exception into 03e51bbaadcd4d1384f56e34c8d9f13758e770d8 on sasstools:develop.

Coverage increased (+0.007%) to 97.561% when pulling 68eec7312a57728fb780a7dbc409922fdc9cab45 on bgriffith:feature/add-ident-exception into 03e51bbaadcd4d1384f56e34c8d9f13758e770d8 on sasstools:develop.

Coverage increased (+0.007%) to 97.561% when pulling fc23cd898054131712fc57e5ccbdf31933969bc2 on bgriffith:feature/add-ident-exception into 03e51bbaadcd4d1384f56e34c8d9f13758e770d8 on sasstools:develop.

Coverage increased (+0.007%) to 97.561% when pulling fc23cd898054131712fc57e5ccbdf31933969bc2 on bgriffith:feature/add-ident-exception into 03e51bbaadcd4d1384f56e34c8d9f13758e770d8 on sasstools:develop.


Create an SAP Cloud Application Programming Model Project for SAP HANA Cloud

Tutorials: https://developers.sap.com/tutorials/hana-cloud-cap-create-project.html
Issue - Create a new service instance
In step 3.3 not able to create hana instance because plan hdi-shared has been deleted in BTP trial account so this creation is not possible.
Error when binding:

And service Hana in my trial account:

Can somebody help me? What should I do?

I don't believe this is anything specific with this tutorial but a general issue with the entitlement setup in BTP trial.  Did you just recently create your trial account?  I've seen a few similar reports from people newly created that their entitlements aren't correct.   But you are looking at the wrong entitlement in the above screenshot.  It's not the HDI containers trial.  That's for the older HaaS trial.  Instead look at the middle entry. Do you see an hdi-shared available there?  It should be.  If it's not assigned to your user then add the service plan from this same screen.


ThrowingException 7, Permission denied in /tmp/realm-java/realm/realm-library/src/main/cpp/io_realm_internal_OsSharedRealm.cpp line 107

How frequently does the bug occur?
Sometimes
Description
I am setting the default config of Realm DB as follows :
class RealmDB : Application() {
    override fun onCreate() {
        super.onCreate()
        try {
            Log.i(TAG, "Setting RealmDB Default config")
            Realm.init(this)
            val config = RealmConfiguration.Builder()
                .allowWritesOnUiThread(true)
                .allowQueriesOnUiThread(true)
                .deleteRealmIfMigrationNeeded()
                .build()
            Realm.setDefaultConfiguration(config)
        } catch (ex : IOException){
            Log.e(TAG, "Setting RealmConfiguration failed.")
        }
    }
}

Then I am trying to get the default instance in my app as follows:
Realm.getDefaultInstance()
But a Permission denied exception is thrown sometimes. The strange thing is that, this exception is thrown randomly. Sometimes, the realm instance is created successfully, but sometimes it fails with the exception.
Stacktrace & log output
04-05 16:08:54.606  8888  9006 E REALM_JNI: jni: ThrowingException 7, Permission denied in /tmp/realm-java/realm/realm-library/src/main/cpp/io_realm_internal_OsSharedRealm.cpp line 107, .
04-05 16:08:54.606  8888  9006 E REALM_JNI: Exception has been thrown: Permission denied in /tmp/realm-java/realm/realm-library/src/main/cpp/io_realm_internal_OsSharedRealm.cpp line 107

Can you reproduce the bug?
Sometimes
Reproduction Steps
Reboot device and launch the app.
Version
classpath 'io.realm:realm-gradle-plugin:10.11.1'
implementation 'io.realm.kotlin:library-base:1.7.0'
What Atlas App Services are you using?
Local Database only
Are you using encryption?
No
Platform OS and version(s)
Android Version 13
Build environment
Android Studio version: ...
Android Build Tools version: ...
Gradle version: ...

This is an issue for Realm Java so transferring it.


Dynamic priority

Add DynamicPriority function.

Modify priority values to float64.

Define SystemPriority and HigherSystemPriorityQJ function.

Update SystemPriority values if dynamicpriority is set to true.

Print contents in qjqueue for glog.V(10).

Fix ControllerFirstTimestamp to use metav1.MicroTime

Timestamp for objects in program is nanosecond precision until the object is deployed.

metav1.MicroTime allows timestamps to maintain microsecond level precision when transmitting to etcd. The 3 extra digits for nanosecond are truncated before transmission.

Etcd and local store keeps timestamps with microsecond level precision.

After object is deployed, UpdateQueueJobs extracts objects from local store.  ControllerFirstTimestamp will have only microsecond level precision from this time.

Print latency in seconds format for easier post analysis of log.

Set QueueJobStateRunning if at least one resource is running.  This is in reference to the PodRunning for PodPhase: https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/core/types.go#L2217

Clean up syncQueueJob() to use latest version of qj.

Will begin review of this PR later today.


Error every time I modify and save nuxt.config.js

Version
v2.11.0
Reproduction link
https://github.com/AnthonyWebDev
Steps to reproduce
#1 CURRENT PROJECT
On this project (a project started two weeks ago), every time I modify and save my nuxt.config.js I am getting this error in my console :
Assertion failed: (0), function uv_close, file ../deps/uv/src/unix/core.c, line 187.
[1]    4484 abort      npm run dev

Note : The number before abort change depending on where I am making the change in the file.
#2 NEW PROJECT
So when i saw this bug I created a new (fresh and empty) app to check if I was having this bug too.
This time when I am making a change (and saving it) in  nuxt.config.js, here is what I have in my console :
[1]    4575 segmentation fault  npm run dev

Note : The number before segmentation change depending on where I am making the change in the file.
Thanks for the help !
What is expected ?
To not cut my development server. As it was the case before (no?).
What is actually happening?
Two different errors.
Additional comments?
This is on a fresh install on a MacBook Pro Catalina 10.15.2.
Node (v13.5.0) installed via NVM.

This bug report is available on Nuxt community (#c10185)

Could you try it with the LTS version of node?

Hi there,
For the part #2, this was due to some cache problems I think. I updated the project to last nuxt version. By the way, npx create-nuxt-app is installing the 2.0.0 version. Why ?
I did the switch to LTS as I was on 13.5.0. Same problem. I tried without any "additional" plugins like node-sass & sass-loader. Problem still here. I also tried to delete node_modules and re install it. Same problem.
What did the "fix" (I could not call it a fix, as i don't know why I was getting this error), was to delete node_modules, move my project to a new folder, re install it.

By the way, npx create-nuxt-app is installing the 2.0.0 version. Why ?

It dont install 2.0.  https://semver.org/ . It install latest current version which fits semver ^2.0.0 .

I had to revert to node 10 to fix this.

I'm experiencing this on nuxt 2.12.2 and node 14.4.0 under macOS Catalina 10.15.4.
↻ Updated nuxt.config.js
Assertion failed: (!uv__is_closing(handle)), function uv_close, file ../deps/uv/src/unix/core.c, line 113.
error Command failed with signal "SIGABRT".

I am hitting this issue with node 12.16.3, but not on node 10.20.1 or node 12.18.0. Catalina 10.15.4 too.
So what seemed to fix it for me is upgrading to node 12 latest.

web development sucks with problems like this


Support building for C++23

The std::aligned_storage and std::aligned_union functions were deprecated in C++23.
See here for further discussion, as well as for suggested alternatives.

Add missing include <exception>

required for clang-18 (at least)

Replace deprecated std::aligned_storage
Replace deprecated std::aligned_union


Icon for Sorbet in Ruby

Not sure which project to make the request for an official https://sorbet.org icon to go with the sorbet/ top level folder

Go to mallowigi/iconGenerator
But I did add sorbet recently as far as I remember


Add helpful text for standard --format

Issue extracted from conversation at #340
As of standard@6, standard-format is no longer included in the standard package. This is a feature request to add helpful forwarding text instructing people to instead download standard-format  if they try running standard --format to ensure a smoother upgrade experience.

This is already the case.
$ standard --format
standard: Formatting is no longer included with standard. Install it separately: "npm install -g standard-format"

Is it not for you?

Ah, it is, my path was picking up an older standard bin.

@jden Excellent :)


Support PCM16L capture

What i need?
I am trying to communicate with Telegram (Telegram Group Calls), the code works without problems, but if I try to play an audio file on Telegram I hear nothing
The code i'm using
https://github.com/pytgcalls/gotgcalls
The old code (I'm trying to rebuild the nodejs part in go)
https://github.com/pytgcalls/pytgcalls
P.S.
The files of go part is in dist folder
About my environment
pion/webrtc = v3.0.29
golang = 1.16.4
Platform = Windows 20H2 and Ubuntu 20.04
What type of audio telegram need?
Need PCM16L audio
I hope for your help, and thank you in advance

Hey @at-wat @lherman-cs no rush, but curious if this is possible/how hard it would be?
I am happy to do the work myself!

I totally agree, this is great idea. This Would help thousands of developers who work with telegram group call :)

Correct me if I'm wrong, you want a 16-bit raw audio format (PCM16L) and use Opus to encode.
In this case, pion/mediadevices supports this already. If your audio source supports the format already, you can do the following:
	mediadevices.GetUserMedia(mediadevices.MediaStreamConstraints{
		Audio: func(c *mediadevices.MediaTrackConstraints) {
                        c.SampleSize = prop.IntExact(16) // <-- this will test the source to use 16 bit format.
		},
		Codec: codecSelector,
	})

If not, you would need to create your own converter before feeding it to the encoder, you can use https://pkg.go.dev/github.com/pion/mediadevices#AudioTrack.Transform to convert the audio format.

it's possible to add the support natively? Because i need just only PCM16L Raw and not converted to Opus

@Laky-64 if you just need to read raw data (without WebRTC), you can do the following:
	mediaStream, err := mediadevices.GetUserMedia(mediadevices.MediaStreamConstraints{
		Audio: func(c *mediadevices.MediaTrackConstraints) {
                        c.SampleSize = prop.IntExact(16) // <-- this will set the source to use 16 bit format.
		},
		Codec: codecSelector,
	})
        must(err)

	// since we're trying to access the raw data, we need to cast Track to its real type, *mediadevices.AudioTrak
	audioTrack := mediaStream.GetAudioTracks()[0].(*mediadevices.AudioTrack)
	defer audioTrack.Close()

	audioReader := audioTrack.NewReader(false)

	for  {
		chunk, release, err := audioReader.Read()
		must(err)

                 // do something with the chunk

		release()
	}

If you want to use it with WebRTC, you would need to implement your own AudioEncoderBuilder (https://pkg.go.dev/github.com/pion/mediadevices/pkg/codec#AudioEncoderBuilder), and register the custom codec type to pion/webrtc. This is to trick WebRTC to think that PCM16L is an encoded version (in the implementation, you just need to pass through the raw data). Since PCM16L will be a custom format, you would also need to create your RTP packetizer and depacketizer.

I never done that and i don't how to do😅, i'm new in webrtc, A helping hand to build the way to use PCM16L on webrtc is appreciated, more than using the microphone (if it's that hard), I'd like to read a PCM16L file directly

Greetings to you, ladies and gentlemen in @pion, including @Iherman-cs, @Sean-Der.
I would like to know if you can re-think about doing whatever called to making a connection between Golang and Telegram video/voice calls, as we, the Telegram community, aren't talented and blessed as much as your team.
We think by doing that, it will be a very big achievement for both of us, you will be the first one doing this and your names will be promoted everywhere (perhaps each of you turning into a celeb or a Hollywood star one day? who knows?)!
Plus we, will be very happy using your software to make wonderful things that will make tons of people happy.
Regards,

Greetings to you, ladies and gentlemen in @pion, including @lherman-cs and @Sean-Der.
I would like to know if you can re-think about doing whatever called to making a connection between Golang and Telegram video/voice calls, as we, the Telegram community, aren't talented and blessed as much as your team.
We think by doing that, it will be a very big achievement for both of us: you will be the first one doing this and your names will be promoted everywhere (perhaps each of you turning into a celeb or a Hollywood star one day? who knows?)!
Plus we, will be very happy using your software to make wonderful things that will make tons of people happy.
Regards,

+1

@Laky-64 Sorry for the delay... I've created a quick and dirty example here, https://github.com/pion/mediadevices/tree/pcm16/examples/file.
The APIs are not the most user-friendly. Please let me know if the example is not clear, I can add more comments.
Unfortunately, I don't have a lot of bandwidth to maintain the project due to my full-time work. So, any contributions from the community will be really helpful to make this project healthy! If interested, contact me at<EMAIL_ADDRESS>or slack me directly on the Gophers channel.

Ok, thanks, I'll try it

This example works with jsfiddle, but with telegram nop, i don't know why, what you need to know why isn't working @lherman-cs , thanks in advance

This is the section where source code of telegram use webrtc
https://github.com/TelegramMessenger/tgcalls/tree/master/tgcalls/group

This is the two working SessionDescription of Javascript Webrtc, this can be helpful for you to know the problem?
LocalDescription
v=0
o=-<PHONE_NUMBER>090610582 2 IN IP4 <IP_ADDRESS>
s=-
t=0 0
a=group:BUNDLE 0
a=msid-semantic: WMS
m=audio 48595 UDP/TLS/RTP/SAVPF 111 103 104 9 102 0 8 106 105 13 110 112 113 126
c=IN IP4 <IP_ADDRESS>
a=rtcp:9 IN IP4 <IP_ADDRESS>
a=candidate:2757424436 1 udp<PHONE_NUMBER> <IP_ADDRESS> 48595 typ host generation 0 network-id 1
a=candidate:559267639 1 udp<PHONE_NUMBER> ::1 42404 typ host generation 0 network-id 3
a=candidate:1510613869 1 udp<PHONE_NUMBER> <IP_ADDRESS> 43418 typ host generation 0 network-id 2
a=candidate:3940092356 1 tcp<PHONE_NUMBER> <IP_ADDRESS> 57157 typ host tcptype passive generation 0 network-id 1
a=candidate:1876313031 1 tcp<PHONE_NUMBER> ::1 54633 typ host tcptype passive generation 0 network-id 3
a=candidate:344579997 1 tcp<PHONE_NUMBER> <IP_ADDRESS> 56701 typ host tcptype passive generation 0 network-id 2
a=ice-ufrag:BeRp
a=ice-pwd:0BhF5wpNQ83xaBrAarHTNkTD
a=ice-options:trickle
a=fingerprint:sha-256 E2:9C:02:05:7C:3B:A9:75:C9:22:77:BB:08:25:EE:E4:82:53:46:CE:20:BB:57:A8:BD:0F:4D:5B:E7:CB:50:9D
a=setup:actpass
a=mid:0
a=extmap:1 urn:ietf:params:rtp-hdrext:ssrc-audio-level
a=extmap:2 http://www.webrtc.org/experiments/rtp-hdrext/abs-send-time
a=extmap:3 http://www.ietf.org/id/draft-holmer-rmcat-transport-wide-cc-extensions-01
a=extmap:4 urn:ietf:params:rtp-hdrext:sdes:mid
a=extmap:5 urn:ietf:params:rtp-hdrext:sdes:rtp-stream-id
a=extmap:6 urn:ietf:params:rtp-hdrext:sdes:repaired-rtp-stream-id
a=sendrecv
a=msid:- a7d6f191-91cb-44f2-93d9-40ac90267c55
a=rtcp-mux
a=rtpmap:111 opus/48000/2
a=rtcp-fb:111 transport-cc
a=fmtp:111 minptime=10;useinbandfec=1
a=rtpmap:103 ISAC/16000
a=rtpmap:104 ISAC/32000
a=rtpmap:9 G722/8000
a=rtpmap:102 ILBC/8000
a=rtpmap:0 PCMU/8000
a=rtpmap:8 PCMA/8000
a=rtpmap:106 CN/32000
a=rtpmap:105 CN/16000
a=rtpmap:13 CN/8000
a=rtpmap:110 telephone-event/48000
a=rtpmap:112 telephone-event/32000
a=rtpmap:113 telephone-event/16000
a=rtpmap:126 telephone-event/8000
a=ssrc:3550684645 cname:vAEfVjr8bDzmBmog
a=ssrc:3550684645 msid:- a7d6f191-91cb-44f2-93d9-40ac90267c55
a=ssrc:3550684645 mslabel:-
a=ssrc:3550684645 label:a7d6f191-91cb-44f2-93d9-40ac90267c55

I create before the localOffer with nil options, then i request to the telegram the params for to do the connection and i get this for make the remoteDescription based on this
{
     "transport":{
          "candidates":[
               {
                    "component":1,
                    "foundation":1,
                    "generation":0,
                    "id":"c720d7d10bd934302bad8ea9",
                    "ip":"2001:67c:4e8:f102:7:0:285:703",
                    "network":0,
                    "port":32001,
                    "priority":2130706431,
                    "protocol":"udp",
                    "type":"host"
               },
               {
                    "component":1,
                    "foundation":2,
                    "generation":0,
                    "id":"7c19830c10bd9343061a69f7b",
                    "ip":"<IP_ADDRESS>",
                    "network":0,
                    "port":32001,
                    "priority":2130706431,
                    "protocol":"udp",
                    "type":"host"
               }
          ],
          "fingerprints":[
               {
                    "fingerprint":"BB:06:39:03:AF:C3:82:31:5E:16:20:9B:46:48:0F:21:0A:1D:0C:8B:62:31:79:AC:7F:A0:3C:65:2D:3D:84:44",
                    "hash":"sha-256",
                    "setup":"passive"
               }
          ],
          "pwd":"3v5ejlciut5vtcrtk9q4u68jrm",
          "ufrag":"9atfa1f97dpq4m"
     }
}

RemoteDescription
v=0
o=-<PHONE_NUMBER>288 2 IN IP4 <IP_ADDRESS>
s=-
t=0 0
a=group:BUNDLE 0
a=msid-semantic: WMS
a=ice-lite
m=audio 32001 RTP/SAVPF 111 126
c=IN IP4 <IP_ADDRESS>
a=rtcp:9 IN IP4 <IP_ADDRESS>
a=candidate:1 1 udp<PHONE_NUMBER> 2001:67c:4e8:f102:7:0:285:703 32001 typ host generation 0
a=candidate:2 1 udp<PHONE_NUMBER> <IP_ADDRESS> 32001 typ host generation 0
a=ice-ufrag:9atfa1f97dpq4m
a=ice-pwd:3v5ejlciut5vtcrtk9q4u68jrm
a=fingerprint:sha-256 BB:06:39:03:AF:C3:82:31:5E:16:20:9B:46:48:0F:21:0A:1D:0C:8B:62:31:79:AC:7F:A0:3C:65:2D:3D:84:44
a=setup:passive
a=mid:0
a=extmap:1 urn:ietf:params:rtp-hdrext:ssrc-audio-level
a=recvonly
a=rtcp-mux
a=rtpmap:111 opus/48000/2
a=rtcp-fb:111 transport-cc
a=fmtp:111 minptime=10;usedtx=1;useinbandfec=1
a=rtpmap:126 telephone-event/8000

I published also a code semplification of what i do
https://vitHub.com/pytgcalls/ntgcalls

I published also a code semplification of what i do
https://github.com/pytgcalls/ntgcalls
@lherman-cs

According to RFC 3551, 16-bit PCM should be described by an a=rtmap line with the codec name L16.  I don't see such a line in any of the SDP fragments that you posted.
Please describe accurately what you need.

@jech do you know how to help me to use the following data?
If the problem does not persist in the PCM16L, if you have telegram we can write there

According to this I don't think I am making any configuration errors, but I think it is more instead a lack of PCM16L codec, and I think it needs PCM16L to transmit, or else there is some strange problem not known to me, because same configuration on Javascript works, but I use PCM16L and not opus

This is what I want to do, I made a concept map
Link1: https://github.com/pytgcalls/pytgcalls/blob/master/src/stream.ts
Link2: https://github.com/pytgcalls/pytgcalls/blob/master/src/sdp-builder.ts

@Sean-Der I don't want to bother, but do you know how to help me do this (As Herman said he doesn't have much time as he has a full time job)? I gave any necessary means, created the concept map, explained in detail what I needed and I also did all the necessary tests without asking you every single thing every time, but I kindly ask for help, I tried to read everything about WebRTC, but I can understand to some extent about WebRTC.
I await your reply with patience, thanks in advance

In my understanding, you want to port https://github.com/pytgcalls/pytgcalls to Golang. Is it right?
In https://github.com/pytgcalls/pytgcalls, raw PCM16L audio is passed to node-webrtc's RTCAudioSource.
https://github.com/pytgcalls/pytgcalls/blob/9b181ff53695a99dbe5b003d6f86cfcea426e59c/src/stream.ts#L250-L256
It doesn't mean that the WebRTC communication between Telegram peers are using PCM16L as a codec.
(Codec is automatically negotiated and the audio is internally encoded using the negotiated codec.)
According to the SDPs in https://github.com/pion/mediadevices/issues/339#issuecomment-869238310, candidate of the codecs are:

local: Opus, ISAC, G.722, PCMU, PCMA, iLBC
remote: Opus

(CN and telephone-event are not audio itself and used to with other codecs)
So, the actually used codec in Telegram should be Opus.

@at-wat  Yes, i want to Port pytgcalls to Golang
while for the version in Nodejs it transmits the audio, the one in go fails, I made the code readable to all:
https://github.com/pytgcalls/ntgcalls

@at-wat
I tried to use opus as a codec, but it doesn't stream audio, and as stated in https://github.com/TelegramMessenger/tgcalls/blob/master/tgcalls/group/GroupInstanceCustomImpl.cpp#L888-L964 it implies they use PCM16L , since isRawPcm is always true

@at-wat Where can I understand if PCM16L is used as their internal codec for communication between WebRTC and Telegram?
There is an official telegram lib which is used for the Desktop app.
I don't know if this is useful
https://github.com/TelegramMessenger/tgcalls/blob/master/tgcalls/group/GroupInstanceCustomImpl.cpp#L888-L964

@ashellunts about the use case: Just read the issue description.

Hi, does this issue still pending or a solution has been found ?
Should I keep the issue open ?

@EmrysMyrddin Yes, you keep it open.


docs: added graphql-codegen plugin to the GraphQL docs

:tada: This PR is included in version 3.17.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix the --set-as-default-runtime deprecation warning

Fixes #269.
I believe the previously intended behavior was to check if the two flags are different, but the default value of --set-as-default-runtime was None, so the if statement on line 210 evaluated to True by default. Changed it the warning is written whenever the old flag is set to any value, regardless of the new flag.
Thank you for making MicroK8s better
Please reference the issue this PR is fixing, or provide a description of the problem addressed.
Also verify you have:

[x] Read the contributions page.
[x] Submitted the CLA form, if you are a first time contributor.

@neoaggelos This should fix #272 as well since the default value for set-as-default-runtime seems to be False which overrides the default True of gpu-operator-set-as-default-runtime flag within this if clause.

Thank you @Damowerko, could you also make sure to create a Launchpad account and sign the CLA? Then we can merge this PR.

You don't need a Launchpad account to sign CLA

I hope the CLA wasn't a problem. I signed it but for some reason the tests weren't passing.


update with index file

Coverage remained the same at 100.0% when pulling 48b3552a91884c66f6cd060e5a7e04e88d8495d2 on edgrams:index into 47a36816f7144779fdf377a779cf4b33238cbc5b on avroize:master.