remove type field

Because type is required on package.json, when this module is loaded under Node.js 13.x it throws an error
(node:6162) Warning: require() of ES modules is not supported.
require() of /home/travis/build/microlinkhq/metascraper/packages/metascraper-iframe/node_modules/oembed-parser/index.js from /home/travis/build/microlinkhq/metascraper/packages/metascraper-iframe/src/index.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename /home/travis/build/microlinkhq/metascraper/packages/metascraper-iframe/node_modules/oembed-parser/index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /home/travis/build/microlinkhq/metascraper/packages/metascraper-iframe/node_modules/oembed-parser/package.json.
/home/travis/build/microlinkhq/metascraper/packages/metascraper-iframe/node_modules/yargs/yargs.js:1163
      else throw err
           ^
Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /home/travis/build/microlinkhq/metascraper/packages/metascraper-iframe/node_modules/oembed-parser/index.js

but actually the module is not written using imports, so this field looks innecessary.
Related: https://travis-ci.com/microlinkhq/metascraper/jobs/266852612#L368

@Kikobeats thank you, I have plan to use esm for supporting ES6 Modules by default. Do you think that approach is better?

Can I ask why do you need to rewrite the source code to use ES6 Modules?
I feel it's going to make the module unusable at Node.js world.
If you need to target browsers environment, the thing you can do is expose the export in an ES6 Module friendly way, like this:
https://github.com/microlinkhq/mql/blob/6706b5c2a6bffd5f0a6b440933712a41ab806175/src/browser.js#L38

@Kikobeats I have no plan to support browser environment (#41)
But I think ES6 Modules will be standard for modularizing in Node.js in very near future. And I like using import/export syntax better than CommonJS style.
However as you known:

ES6 Modules is still experimental, even it's enabled by default since v13.3
with older versions of node, we need to run with the flag --experimental-modules
ES6 Modules support is not completed, there is many error around it

So, the simplest and safest way is using esm as I'm doing with many other modules and projects.

I think the future is uncertain and the only thing true is the present 😛
Feeling a bit overengineering there; if you keep type on package.json, I'm going to fork the repo since Node.js is throwing me a warning.
If you simply wait for two Node.js versions released or so, probably ES6 modules will be a more stable thing and more common npm modules will be ready for that.

Just to provide you context, browserify didn't implement esm yet in 2019 and it's okay
https://github.com/browserify/browserify/issues/1186#issuecomment-565733976

@Kikobeats ok, I will try to merge your pr after checking why the CI does not pass.

@Kikobeats I've checked. Unfortunately, keeping type field causes error in Node.js v13.x while removing it causes error in all Node.js versions. I'm trying to go with esm, but also hit the wall.

I don't undestand, I removed it manually and all is fine, what's the error?

@Kikobeats  I think the problem relates to tap library, not our source code. I'm finding a work around. At this time, you can see it always throws error like this:
https://travis-ci.org/ndaidong/oembed-parser/jobs/624558023#L942

use npm list || true to avoid npm list error

@Kikobeats npm list does not cause any problem.
Do you have node v13.3 installed locally? If you do, you will see removing type fields makes no sense. Any dependency that use 'require()` will lead to error. We may need another approach.

@Kikobeats sorry, it's my fault. Your approach is correct. The issue relate to one of dependencies. I've merged your PR and fix the related issues. Everything looks fine at v1.3.7


Add example of duplicate top-level definition.

The error message should indicate the lines of all the definitions.

The compiler got the ability to list multiple different sections of the code (I think in 0.19.0) so I am currently seeing this error message for your example:
-- NAME CLASH --------------------------------------------------------- temp.elm

This file has multiple `value` declarations. One here:

2| value = 0
   ^^^^^
And another one here:

5| value = 1
   ^^^^^
How can I know which one you want? Rename one of them!

Instead of checking in all the examples exactly as they appeared, I am planning to go through a bunch of the old issues and pull requests at some point. There are a bunch of great ones, but I'm not sure how to organize them as a historical record that is easy to check on.


add component parser

This is a stripped-down version of the graph component that simply parses components into component objects and wit files.
It should be able to be transpiled using jco into either a core wasm or js in order to use it in the browser.

This CI/CD build should succeed if we re-trigger it after this PR gets merged.

It was merged, retriggering CI/CD now.


[bug] web3modal/siwe - SIWE does not popup when connecting with wallet

Link to minimal reproducible example
https://github.com/2facedsquid/siwe-test
Summary
When connecting with your injected wallet ( in my case I tested metamask ), the siwe message signing does not pop up, it only pops up when already connected and switching the network or the wallet account.
There is also an error in the console of the browser regarding this endpoint call - https://rpc.walletconnect.com/v1/profile/reverse/{address}?projectId={projectId}
with this response:
{
"status": "FAILED",
"reasons": [
{
"field": "address",
"description": "No name for address is found"
}
]
}
I presume this console error is not tied to the @web3modal/siwe integration, since it is also present when siwe is not integrated
List of related npm package versions
Issue was first discovered on this config:
"@tanstack/react-query": "^5.40.0",
"@web3modal/siwe": "^4.2.3",
"@web3modal/wagmi": "^4.2.3",
"viem": "^2.13.3",
"wagmi": "^2.9.8"
The issue is still happening on this config:
"@tanstack/react-query": "^5.45.1",
"@web3modal/siwe": "^5.0.2",
"@web3modal/wagmi": "^5.0.2",
"viem": "^2.15.1",
"wagmi": "^2.10.2"

Experiencing similar behavior on Vue with versions @web3modal/siwe and @web3modal/wagmi above 4.1.11
Very concerned.

Retested after the 5.0.2 release of web3modal and the issue is still present

experiencing similar issue

Retested in 5.0.11, issue still present

There is also an error in the console of the browser regarding this endpoint call - https://rpc.walletconnect.com/v1/profile/reverse/{address}?projectId={projectId}
with this response:
{
"status": "FAILED",
"reasons": [
{
"field": "address",
"description": "No name for address is found"
}
]
}
I presume this console error is not tied to the @web3modal/siwe integration, since it is also present when siwe is not integrated

This is error is not related with siwe

I cannot reproduce this in our laboratory/reference/test environment: https://dev.lab.web3modal.com/library/wagmi-all/

@2facedsquid
The problem is that you are returning a value whatever the session actually is in SIWE configs:
async function getSession(): Promise<SIWESession> {
  return {
    address: '0x1',
    chainId: 1
  }
}

This is the problem bc in real world examples (like see our laboratory SIWE configs), the getSession is a promise which should be fetching the user's authentication status and not returning a static value like you do. When you return a static value, our modal is detecting the returned object and thinking the user is authenticated and directly closing the SIWE signing view.
@Un173 @OladetounJed Are you following the same approach? Is your issues the same?

@2facedsquid
The problem is that you are returning a value whatever the session actually is in SIWE configs:
async function getSession(): Promise<SIWESession> {
  return {
    address: '0x1',
    chainId: 1
  }
}

This is the problem bc in real world examples (like see our laboratory SIWE configs), the getSession is a promise which should be fetching the user's authentication status and not returning a static value like you do. When you return a static value, our modal is detecting the returned object and thinking the user is authenticated and directly closing the SIWE signing view.
@Un173 @OladetounJed Are you following the same approach? Is your issues the same?

What is getSession supposed to return when first connecting where the session is not established yet, meaning that no message was signed? I see that it gets called when the wallet connects, even when substituting for null values ( simulating a non existent session ) the modal does not pop up.
The only time the modal does appear tho is when switching networks even with static values.
This MRE refers to an integration where we have correct data in the getSession and its the same outcome of siwe sign-in pop up not appearing.

@2facedsquid
The problem is that you are returning a value whatever the session actually is in SIWE configs:
async function getSession(): Promise<SIWESession> {
  return {
    address: '0x1',
    chainId: 1
  }
}

This is the problem bc in real world examples (like see our laboratory SIWE configs), the getSession is a promise which should be fetching the user's authentication status and not returning a static value like you do. When you return a static value, our modal is detecting the returned object and thinking the user is authenticated and directly closing the SIWE signing view.
@Un173 @OladetounJed Are you following the same approach? Is your issues the same?

I have encountered this issue as well, and it has persisted for several months without a solution. To troubleshoot why the SIWE modal dialog doesn't pop up, I directly hardcoded the return value in the following function:
async function getSession(): Promise<SIWESession> {
  return null;
}

During multiple tests, I observed that the SIWE dialog didn't appear a few times, but I'm not sure why. I'm using the Next.js framework with the web3modal version 4.2.3.

If possible, I would prefer to know how to manually trigger the SIWE modal dialog. This way, I can display a “Sign Message” button after the wallet is connected to prompt the user to initiate the signature actively, rather than relying on the internal logic of web3modal to handle it automatically. If this feature is not available, could it be considered for future implementation?  @enesozturk @2facedsquid

I can display a “Sign Message” button after the wallet is connected to prompt the user to initiate the signature actively

Isn't this what our flow already does?

@leecobaby @2facedsquid we recently announced new AppKit. This issue should already been resolved since we did improve bunch of logics including SIWE flows. So I'd suggest you to upgrade to AppKit v1 instead of Web3Modal.
Here is the migration guide for you: https://docs.reown.com/appkit/upgrade/from-w3m-to-reown


Refactor to use Math.TAU

DO NOT MERGE, CHANGES IN PROGRESS
This is a little controversial, as these changes modify the standard Math object, and add TAU (6.28318530717958647692)
However, the refactoring of the code is actually refreshing, removing all those PI * 2 multiplications, etc, etc.
Testing was conducted with CLI and WEB UI as well. All browsers work just fine.
All Submissions:

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[X] Does your submission pass tests?

@platypii please review. Also, you should change your user I'd to @playtau 😃

I have mixed feelings on this. I think it is actually very aesthetic in a lot of ways. I personally find the code more readable and understandable in most circumstances. But will this confuse users? I also don't love that it modifies the global Math object. Overall I am supportive, but need to think about it a bit.

I would vote to not modify global Math object. Especially if future will see us moving to es6 modules, such constant is more suited to be imported.

FYI, there is now adoption of tau by the Rust language, Microsoft’s .NET framework, Boost C++ library, and Python.
it's only a matter of time that JavaScript gains Math.TAU

I have mixed feelings on this. I think it is actually very aesthetic in a lot of ways. I personally find the code more readable and understandable in most circumstances. But will this confuse users? I also don't love that it modifies the global Math object. Overall I am supportive, but need to think about it a bit.

I adjusted the code to add Math.TAU if not present. Hopefully, TAU will be added in a few years. :)

@platypii (alias @platytau) any thoughts?

I still think TAU should be exported from constants, NOT added to the global Math object. I don't like modifying the global math object. It breaks encapsulation of the jscad code, and is generally considered bad practice.
There are no current proposals to add Math.TAU to the js standard, and the idea was generally opposed here. So this doesn't feel like a polyfill for future TAU support. It just feels like JSCAD intruding into the standard library.

I still think TAU should be exported from constants, NOT added to the global Math object. I don't like modifying the global math object. It breaks encapsulation of the jscad code, and is generally considered bad practice.

Done. TAU is now defined in the maths contants, and exported.
@platypii if fine then I will refactor the other parts of modeling.

@platypii I completed all the changes / refactoring. So, the changes are now ready for a full review. All comments are welcome.
I like the code changes. The TAU constant makes all rotations easier to conceptualize, and of course the code just looks cleaner. I'm hoping that performance gains slightly as well.

@platypii I completed all the changes / refactoring. So, the changes are now ready for a full review. All comments are welcome.
I like the code changes. The TAU constant makes all rotations easier to conceptualize, and of course the code just looks cleaner. I'm hoping that performance gains slightly as well.

@platypii can you take another look at this PR? thanks

Looks good! Happy to see JSCAD leading the way for clearer mathematical notation :-)

@platypii
HA HA HA! The idea came initially from your post. ;)
By the way, see your email... I've been pinging you to set up a time to discuss V3.


Separate state crate

Objective
Extracts the state mechanisms into a new crate called "bevy_state".
This comes with a few goals:

state wasn't really an inherent machinery of the ecs system, and so keeping it within bevy_ecs felt forced
by mixing it in with bevy_ecs, the maintainability of our more robust state system was significantly compromised

moving state into a new crate makes it easier to encapsulate as it's own feature, and easier to read and understand since it's no longer a single, massive file.
Solution
move the state-related elements from bevy_ecs to a new crate
Testing

Did you test these changes? If so, how? all the automated tests migrated and passed, ran the pre-existing examples without changes to validate.

Migration Guide
Since bevy_state is now gated behind the bevy_state feature, projects that use state but don't use the default-features will need to add that feature flag.
Since it is no longer part of bevy_ecs, projects that use bevy_ecs directly will need to manually pull in bevy_state,  trigger the StateTransition schedule, and handle any of the elements that bevy_app currently sets up.

Yeah, we should move the examples to a separate folder now :)

You need to update the publish.sh script, but otherwise this LGTM.
https://github.com/bevyengine/bevy/blob/main/tools/publish.sh

Done!


Audit Report Issue due to 'marked' dependency

Both esdoc and esdoc-publish-html-plugin depend on 'marked', which has a security warning in the npm audit report.
Please upgrade to >=0.6.2 of marked to resolve this audit failure.
See: https://nodesecurity.io/advisories/812 for more information
npm audit --registry https://registry.npmjs.org
                       === npm audit security report ===

                                 Manual Review
             Some vulnerabilities require your attention to resolve

          Visit https://go.npm.me/audit-guide for additional guidance

  Moderate        Regular Expression Denial of Service

  Package         marked

  Patched in      >=0.6.2

  Dependency of   esdoc [dev]

  Path            esdoc > marked

  More info       https://nodesecurity.io/advisories/812

  Moderate        Regular Expression Denial of Service

  Package         marked

  Patched in      >=0.6.2

  Dependency of   esdoc-standard-plugin [dev]

  Path            esdoc-standard-plugin > esdoc-publish-html-plugin > marked

  More info       https://nodesecurity.io/advisories/812

found 2 moderate severity vulnerabilities in 859520 scanned packages
  2 vulnerabilities require manual review. See the full report for details.

Check #543

+1

Other than that it seems that all dependencies are locked to a single version which makes deduplication a no-go.
Also, if a vulnerability is found in any other dependency it's impossible to npm audit fix it automatically.

The author of the package seems mostly inactive, I'd suggest using a fork directly from GitHub in the meantime


Direct POST to http(s)://site.com/wp-login.php?action=register bypasses Auth0

If you are using auth0 to manage registration, you get the login widget if you go to site/wp-login.php?action=register (as expected).  However if someone just does a POST to site/wp-login.php?action=register with a body of user_name and user_password set, it will actually register those users.  Bypassing auth0 entirely.    I remember this not working in the past, but I was actually able to signup a new user doing this (perhaps because I am migrating users to my auth0 database?).  Anyway seems like something we should handle one way or another.

Workaround:
https://wordpress.org/support/topic/creating-registration-restriction/


consider moving taxon constraints back into ontology

It might simplify the workflow to edit the taxon constraints directly within the ontology, rather than in TSVs. The challenge is that never_in_taxon annotations must be expanded into their logical form.

Ah I see the challenge here - it's that the expanded form can't live in the .obo edit file it must live in an import

A suggestion from developers is that there would be a special keyword we could add in the PR that would automatically regenerate the imports

Maybe a GH action?

Related to https://github.com/geneontology/go-ontology/issues/24489

Discussion with @balhoff : the current system is easier for reasoning. We wont switch back (at least not now)


"Suciu de Sus", "Larga" and "Suciu de Jos" are listed in the drop-down menu

Description:
"Suciu de Sus", "Larga" and "Suciu de Jos" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Suciu de Sus" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Suciu de Sus, MARAMUREȘ (Suciu de Sus)" and "Comuna Suciu de Sus, MARAMUREȘ".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Larga" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Larga, MARAMUREȘ (Suciu de Sus)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Suciu de Jos" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Suciu de Jos, MARAMUREȘ (Suciu de Sus)".


Evaluate RHS of binding/assignment pattern first

This forces caching the destructured variable prior to evaluating non-literal-valued computed property names in a binding or assignment pattern. We only cache up front when we detect non-literal-valued computed property names so that we can avoid emitting excessive temporary variables.
Fixes #32305

CC: @RyanCavanaugh, @weswigham, @sandersn


Use Brotli instead of brotlipy

brotlipy is stuck at brotli 0.6 and upstream is inactive. Let's switch
to the official binding which is up-to-date and has same interfaces.

The CI failure should be https://github.com/postmanlabs/httpbin/issues/554

bump


describe_project_versions function not working

what should we keep in the place of
def get_project(project_name="construction-object-recognition-v3"):
try:
project = reko.create_project(ProjectName=project_name)['ProjectArn']
except reko.exceptions.ResourceInUseException:
project = 'arn:aws:rekognition:us-east-1:113147044314:project/construction-object-recognition-v3/1582839055056'
return(project)
it is showing  cross account access denied

This repo was for a demo from when I worked at AWS, for a specific case - that's why the exception has a hardcoded project. That project and the associated account don't even exist anymore, and TBH I should never have hardcoded the AWS account ID in there.
The general solution would be to use the describe_projects to find the project ARN. ResourceInUseException essentially means that a project with that name already exists.
A possible solution could look like this:
def get_project(project_name="construction-object-recognition-v3"):
    try:
        project = reko.create_project(ProjectName=project_name)['ProjectArn']
    except reko.exception.ResourceInUseException:
        project = [
            p['ProjectArn']
                for p in reko.describe_projects(ProjectNames=[project_name])['ProjectDescriptions']
        ][0]
    return project

Note that if there are more than 100 Rekognition projects, you'd have to implement pagination on the call to describe_projects. You can find more information on the boto3 client for Rekognition docs

but thanks though it worked! 🙂
On Wed, May 4, 2022, 10:33 PM Roberto Bruno Martins <
@.***> wrote:

This repo was for a demo from when I worked at AWS, for a specific case -
that's why the exception has a hardcoded project. That project and the
associated account don't even exist anymore, and TBH I should never have
hardcoded the AWS account ID in there.
The general solution would be to use the describe_projects to find the
project ARN. ResourceInUseException essentially means that a project with
that name already exists.
A possible solution could look like this:
def get_project(project_name="construction-object-recognition-v3"):
try:
project = reko.create_project(ProjectName=project_name)['ProjectArn']
except reko.exception.ResourceInUseException:
project = [
p['ProjectArn']
for p in reko.describe_projects(ProjectNames=[project_name])['ProjectDescriptions']
][0]
return project
Note that if there are more than 100 Rekognition projects, you'd have to
implement pagination on the call to describe_projects. You can find more
information on the boto3 client for Rekognition docs
https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/rekognition.html#client
—
Reply to this email directly, view it on GitHub
https://github.com/bobbruno/aws-rekognition-cl-demo/issues/1#issuecomment-1117592080,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZAKGREHGGITVHYE5LTVPBDVIKUVZANCNFSM5VBPQQQQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

do you have any idea on how much time will it take to train a model in
Amazon rekognition?
I have put 500 images as an input... it is taking more than 5 hours to
train... please help me
On Wed, May 4, 2022 at 10:36 PM Saketh @.***> wrote:

but thanks though it worked! 🙂
On Wed, May 4, 2022, 10:33 PM Roberto Bruno Martins <
@.***> wrote:

This repo was for a demo from when I worked at AWS, for a specific case -
that's why the exception has a hardcoded project. That project and the
associated account don't even exist anymore, and TBH I should never have
hardcoded the AWS account ID in there.
The general solution would be to use the describe_projects to find the
project ARN. ResourceInUseException essentially means that a project
with that name already exists.
A possible solution could look like this:
def get_project(project_name="construction-object-recognition-v3"):
try:
project = reko.create_project(ProjectName=project_name)['ProjectArn']
except reko.exception.ResourceInUseException:
project = [
p['ProjectArn']
for p in reko.describe_projects(ProjectNames=[project_name])['ProjectDescriptions']
][0]
return project
Note that if there are more than 100 Rekognition projects, you'd have to
implement pagination on the call to describe_projects. You can find more
information on the boto3 client for Rekognition docs
https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/rekognition.html#client
—
Reply to this email directly, view it on GitHub
https://github.com/bobbruno/aws-rekognition-cl-demo/issues/1#issuecomment-1117592080,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZAKGREHGGITVHYE5LTVPBDVIKUVZANCNFSM5VBPQQQQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm sorry, but I don't know exactly how long - big datasets can take a while. I suggest you start with a smaller set of images (I've seen decent results with just 20, maybe a 100 could be enough). But I don't work with AWS anymore, and even then, I never did much with Rekognition.

the data set that you kept in the github repo... 100 images from that
dataet is enough?
On Thu, May 5, 2022, 6:29 AM Roberto Bruno Martins @.***>
wrote:

I'm sorry, but I don't know exactly how long - big datasets can take a
while. I suggest you start with a smaller set of images (I've seen decent
results with just 20, maybe a 100 could be enough). But I don't work with
AWS anymore, and even then, I never did much with Rekognition.
—
Reply to this email directly, view it on GitHub
https://github.com/bobbruno/aws-rekognition-cl-demo/issues/1#issuecomment-1118069208,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZAKGRDYMVEYNQQMJNGNDFLVIMMOPANCNFSM5VBPQQQQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

so, I ran the code in google colab with GPU.... while training the model in
Amazon rekognition... will it use the GPU or my laptop's CPU?

Message ID: @.***

com>

so, I ran the code in google colab with GPU... while training in amazon
rekognition will it use my laptop CPU or google colab GPU?... can you
please help me
On Thu, May 5, 2022 at 6:31 AM Saketh @.***> wrote:

the data set that you kept in the github repo... 100 images from that
dataet is enough?
On Thu, May 5, 2022, 6:29 AM Roberto Bruno Martins <
@.***> wrote:

I'm sorry, but I don't know exactly how long - big datasets can take a
while. I suggest you start with a smaller set of images (I've seen decent
results with just 20, maybe a 100 could be enough). But I don't work with
AWS anymore, and even then, I never did much with Rekognition.
—
Reply to this email directly, view it on GitHub
https://github.com/bobbruno/aws-rekognition-cl-demo/issues/1#issuecomment-1118069208,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZAKGRDYMVEYNQQMJNGNDFLVIMMOPANCNFSM5VBPQQQQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

how can I get good f score... I have trained the entire dataset but still
it is just 0.368.. please help me

Message ID: @.***

com>


Callback handler feature

https://github.com/rekab-app/background_locator/issues/31

@gpibarra Please update the wiki to show necessity of this feature and how to use it.


Asset pipeline, aka Sprockets path calculation.

Closed in favor of https://github.com/mbleigh/princely/pull/54


I want to set the focus to the next element, when pressing Enter key

first , focus is on the select element.
at this time, press Enter key , i want to set the focus to the next element.
Could you tell me how to code.
thank you very much.

We cannot implement this on web-form, because web pages differ win-form absolutely. I dont agree with @akgargo code above, so that it prevent default behavior of html process page on browser!

Best way to highlighting keyboard tab event in select2().

I also have the same problem. I have a select2() drop-down and a input box. At first the focus will be on drop down so when the user hit Enter key the drop down will be opened. In onchange method of drop down I am adding focus to the text box using focus() method of jQuery. But it  is not focusing to text box, rather it focus to the drop down itself. What to do guys!!

Fixed: setTimeout(yourFocusFunction, 0);

export class Select2Override extends Select2 {
  attachEventHandlers = (props) => {
    props.events.forEach((event) => {
      if (typeof props[event[1]] !== 'undefined') {
        if (event[0] === 'change.react-select2-wrapper') {
          const changeHandler = props[event[1]];
          this.el.on(event[0], (e) => {
            changeHandler(e);
            setTimeout(() => { e.target.focus(); }, 0);
          });
        } else {
          this.el.on(event[0], props[event[1]]);
        }
      }
    });
  };
}

I am working on HTML5, Angular 4 application using Tyepscript. My problem is I am able to set focus to next cell when I press enter. But when I refresh/reload page it is not working. Here is my stackoverflow question.
https://stackoverflow.com/questions/47949110/page-refresh-enter-arrow-keys-not-working-in-table-cells

Usage questions should be discussed on the forums instead.


Add CI/CCD

This PR will:

update all precommit hook
add Containerfile
add copmose.yml
add GitHub action to:

Build python module and upload to GitHub
Build Container image and upload to GitHub

Excuse me, what is the intention of this PR?
I'm struggling to understand.

Excuse me, what is the intention of this PR? I'm struggling to understand.

I am unsure what to answer to this question except copying and pasting the PR description. It should do everything listed there.

I mean why are they needed?
I really don't want to approve/merge what I don't understand.


[Feature request]Using calculation engine in headless mode on the server

Is your feature request related to a problem? Please describe.
For our scenario it is necessary sometimes to recalculate the spreadsheet formulas on the server. For example some data that spreadsheet depends on are changed by some background process and we need to update some cells and recalculated dependent ones while user is not working with the spreadsheet in UI.
We also would like to develop a set of automatic tests that will compare results of Luckysheet calculation against other calculation engine. I assume these tests will run much faster if we can do it w/o browser context.
Describe the solution you'd like
API to create/load/manipulate/import/export workbook in headless mode with pure javascript.
Additional context

Great idea. We will consider the proposal.


sdcv: 0.5.3 -> 0.5.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/sdcv/versions.
meta.description for sdcv is: "Console version of StarDict"
meta.homepage for sdcv is: "https://dushistov.github.io/sdcv/"
meta.changelog for sdcv is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.4 with grep in /nix/store/1fdanjqsgdpff7kx8rwwpfwpmhr062pa-sdcv-0.5.4
found 0.5.4 in filename of file in /nix/store/1fdanjqsgdpff7kx8rwwpfwpmhr062pa-sdcv-0.5.4

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
koreader
sdcv

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1fdanjqsgdpff7kx8rwwpfwpmhr062pa-sdcv-0.5.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A sdcv https://github.com/r-ryantm/nixpkgs/archive/04ce4c987c1ad6d116f07561b6fb2f768eb75115.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1fdanjqsgdpff7kx8rwwpfwpmhr062pa-sdcv-0.5.4
ls -la /nix/store/1fdanjqsgdpff7kx8rwwpfwpmhr062pa-sdcv-0.5.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

koreader
sdcv

Maintainer pings
cc @lovek323 for testing.

nixpkg-review went fine:
2 packages built:
koreader sdcv

koreader which depends on sdcv is working well. sdcv version 0.5.4 looks like a minor bug fix release with some improvements. LGTM


Need BF V4 documentation for "Send and receive files through your bot"

This is not documented for V4 Bots. Please add new page to document this. Update this code snippet to V4 sample.
Link following samples in the new doc page.

Sample
Description
.NET Core
JavaScript
Python

Teams File Upload
Exchanging files with a bot in a one-to-one conversation.
View
View
View

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

ID: 4b69599c-1da4-8689-fd0c-4e4d8a348496
Version Independent ID: 872fa9a2-0543-4a94-5416-6e6dc389f0d0
Content: Sending and receiving files from a bot - Teams
Content Source: msteams-platform/resources/bot-v3/bots-files.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

This is handled in another PR https://github.com/MicrosoftDocs/msteams-docs/pull/2702.


Support for Burst

Hi,
is Elementum going to support Burst provider(s)?

Hi.
Elementum has own Burst (forked it, to be able to update it). It is available from Elementum repository.

Hi @elgatito, can you please provide details to Elementum repository so I could add it to my local repo on Kodi on LibreElec?

Repository works same as it was in Quasar, it's a virtual repository and Kodi is querying the elementum daemon for listing addons and their versions.
If you only need addon.xml for each module, you can try this:
http://elementum.surge.sh/packages/elgatito/plugin.video.elementum/addon.xml
http://elementum.surge.sh/packages/elgatito/script.elementum.burst/addon.xml
These urls are updated automatically with the latest addon.xml

thank you men greet job


v1.7.2 - fix node10 resolution

v1.7.2 - 06/21/2023
Fixed

Added try-catch best effort for gzip json response decompression from nuget.org
Compile with nodenext moduleResolution so it implements a Promise resolver intead of yield on dynamic module resolution (node v13.2+)

Resolves node and node10 task runners

Approved through the extension release


Fix behavior of SilkMarshal.StringToPtr and related methods on Linux

Summary of the PR
This fixes https://github.com/dotnet/Silk.NET/issues/2374 by checking for whether the platform is Windows and using 2-byte LPWStrs if it is Windows and 4-byte LPWStrs if it is not.
Related issues, Discord discussions, or proposals
https://github.com/dotnet/Silk.NET/issues/2374
Further Comments
This pull request includes the changes made in https://github.com/dotnet/Silk.NET/pull/2376.
In that pull request, I added a test project (Silk.NET.Core.Tests) which is used here to test the changes I made.

I built the package locally and overrode the package version that I was using.

Doing this makes my project run successfully on both Windows and Linux without any further changes.
I also ran the test cases I wrote on both Windows and Linux. The CI Build seems to take care of Mac.

I changed the code to use Encoding.UTF32 for both converting to a pointer and from a pointer. The test cases pass, but I'm still wondering if this is correct (see my reply to your review comment).

Will merge following #2376

Hold on. I want to do a bit more research and make sure the behavior here is correct.

Interesting. It looks like on Windows, LPWStr is UTF-16 as expected. However, on Linux, LPWStr is UTF-32.
For example, the memory for 🧵:

On Linux: 0xf5f901
On Windows: 0x3ed8f5dd

I checked this by inspecting the memory of L"🧵" on both Windows and Linux.
The codepoint for 🧵 is U+1F9F5.
The surrogate pair for 🧵 is 0xd8e3 and 0xddf5.
Ignoring byte order, everything matches up.
This means that it is indeed correct to use UTF32 on non-Windows platforms.
I'll update the code to reflect this.

I found another issue where on Windows, characters like emojis get garbled.
This apparently was always the case.
I'll look into fixing this as well.

Okay, never mind. That technically is expected behavior since that case is testing LPStr (which is 8 bits, but not necessarily UTF-8) and non-ascii characters are not guaranteed to be supported.


404 reporting with Google Analytics

There are article about how to get GA to track 404 errors.  Is it possible for Metron to help with that?
An article I read says to change the GA script to:
_gaq.push('[_trackPageview', '/404.html?p={{ path }}"]);

instead of the standard:
_gaq.push("[_trackPageview']);

I'm just wondering what's the best way to do that without hard coding the script in my 404 page.

This issue has been around for quite some time. Unless I'm mistaken, it appears that this can be done through GA dashboard (see example link). Is this issue still relevant? https://www.searchviu.com/en/404-errors-google-analytics/


Fix theme submodules

This fixes the following theme submodules:

emily
hermit
hephaestus
nasm-theme
zolastrap
zhuia

Something I would like to check before merging that is whether the themes are updated. If they are not I don't really want to show them on the zola website. I'm wondering what's the best way to do that, maybe try to build the demo site of each?

Just give me a list of which ones you want.... some of these themes look stellar even if they have not been updated in a while.

What you may eventually need to do is have your script check which themes are getting the most attention.... be that stars, contributors, combination of both, and sort the list of themes accordinly, putting the cream of the crop at the top, or like you said just be selective in which ones you include...

Do you mean that some of the themes may no longer build?? I assumed the generate script checked that... if thats what you mean then i totally get what your saying.

Do you mean that some of the themes may no longer build?? I assumed the generate script checked that... if thats what you mean then i totally get what your saying.

Exaclty, if the themes hasn't been updated in 2 years for example, it's unlikely to still work. Thinking about it a bit more, I'll handle that later - no need to block this PR.


Update the replica start page with new copy

This updates the temporary start page with new copy, as outlined in https://docs.google.com/document/d/17jGw6JSuGSadfDWfcqC27nNvR2vRWgA_hrHa30sbAco/edit#

 Update replica start page


Inline math brackets are reversed while text is set to an RTL language

Description
This source code:
תהא סדרה $a_n$: $[a_n: 1, 1/2, 1/3, dots]$
Produces this:

Instead of:

My template contains the following set rule: set text(font: "", lang: "he"). Changing the language to "en" shows the brackets in the correct order (but messes up the rest of the document as it is written in Hebrew).
This occurred only after the new update.
Reproduction URL
No response
Operating system
Web app
Typst version

[X] I am using the latest version of Typst

Oh, that doesn't look good. I think it's a result of https://github.com/typst/typst/pull/2938.

It seems to be not only brackets, but inline math in RTL as a whole:
A paragraph in 0.10.0:

The same paragraph in 11.0.0:

Yeah, previously an inline equation would be one atomic box. But now, it consists of multiple breakable parts and those are affected by BiDi reordering in unexpected ways. This piece of code might be a temporary workaround:
#show math.equation.where(block: false): it => box({
  set text(dir: ltr)
  it
})

("Might" because I'm not 100% sure how it should really look.)

Generally show math.equation: set text(lang: "en") should give correct results

I tried that, but without the box, it didn't work.

Is there a way to disable line breaks in inline equations altogether (i.e. restore 0.10.0 behavior)? I personally prefer it over TeX's behavior in RTL documents, at least most of the time.

@ocantor #show math.equation.where(block: false): box should do the trick

#show math.equation.where(block: false): box disables line breaks without fixing the issue, your workaround disables line breaks and fixes the issue (which is what I wanted), and show math.equation: set text(lang: "en") doesn't seem to do anything. I wonder if there's a different workaround that fixes the issue without disabling line breaks.

I wonder if there's a different workaround that fixes the issue without disabling line breaks.

I don't think that one exists.

Does it even make sense to have a multi-line inline equation in RTL? (And not keep them as blocks for RTL)
As far as I know, all equations are LTR (even in RTL languages). Trying to break an LTR equation over RTL text will probably be very confusing for people.
Although this is the indeed default behavior in TeX.

Does it even make sense to have a multi-line inline equation in RTL? (And not keep them as blocks for RTL)

It can definitely make sense, though it can be confusing (though if you ask me not much more confusing than breaking equations in LTR text)

Does it even make sense to have a multi-line inline equation in RTL? (And not keep them as blocks for RTL)
As far as I know, all equations are LTR (even in RTL languages). Trying to break an LTR equation over RTL text will probably be very confusing for people.
Although this is the indeed default behavior in TeX.

I'm not a fan of breaking inline equations even in LTR. It should be the absolute last resort.

Yeah, previously an inline equation would be one atomic box. But now, it consists of multiple breakable parts and those are affected by BiDi reordering in unexpected ways. This piece of code might be a temporary workaround:
#show math.equation.where(block: false): it => box({
  set text(dir: ltr)
  it
})

("Might" because I'm not 100% sure how it should really look.)

This work for me ( I use arabic language in my document )

I've opened #4150 to fix this by adding Unicode LTR embedding chars around equations. The original markup from this issue now produces this result:

This is not exactly what @yonatanmgr showed as intended, but it's what the BiDi algorithm outputs and I think it's technically correct? The ":" and " " are weak chars, so they adopt the LTR-ness of the two surrounding equations. Any feedback from native speakers / BiDi experts on this?

You should try LRI (U+U+2066) instead of LRE: https://www.unicode.org/reports/tr9/tr9-46.html#Directional_Formatting_Characters

@khaledhosny Thanks for the pointer. Using LRI for equations indeed yields exactly the result @yonatanmgr wanted.
We also use LRE on segments of text that have their direction specifically specified (e.g. Hello #text(dir: ..)[..] World). When the PR that implemented that landed, there was a bit of discussion about LRE vs LRI. The problem there was that we individually isolated consecutive pieces of text instead of figuring out the longest subsequence with the same direction, and LRE seemed to handle that better. Do you happen to know whether that's a sensible usage of LRE or whether this should also be changed in some way?

Embedding control characters basically works as strong LTR/RTL characters, while isolates act as neutral characters.
(using ASCII hack syntax with lower case representing LTR and upper case representing RTL)
so
ABC [LRE]a[PDF] [LRE]b[PDF] [LRE]c[PDF] DEF

would render as:
FED a b c CBA

While:
ABC [LRI]a[PDI] [LRI]b[PDI] [LRI]c[PDI] DEF

would render as:
FED c b a CBA

I’d say the later is more desirable, since if one wants the whole embedded segment to be LTR, it should set direction to it as whole:
ABC [LRI]a b c[PDI] DEF

Consider when there is punctuation between the segments:
abc A, B, C def

This is certainly the more desirable rendering (which is what you get with isolates):
FED c ,b ,a CBA

Okay, thanks for the detailed explanation, that definitely makes things clearer!
I'm still not 100% sure what's the right thing for Typst because of Typst's speciality that
ABC #ltr[abc] DEF

and
ABC #ltr[a]#ltr[b]#ltr[c] DEF

is effectively equivalent and indistinguishable (here let ltr = text.with(dir: ltr)). There is no distinction between setting the RTL style for segments individually or for a whole. So, I think in the context of Typst right now the LRE happens to sort of do the right thing / best we can.
The alternative would be to manually find consecutive segments and use LRI on them, but it seems to me that the effect would be quite similar to LRE, except perhaps for small differences in the effect on surrounding text / punctuation.

What about:
ABC #ltr[a] #ltr[b] #ltr[c] DEF

is it the same as:
ABC #ltr[a b c] DEF

?

No, the spaces indeed are the substantive parts here.

Then if the original example here used LTR text segments instead of math, using LRE would give different output than the expected here, which is inconsistent. I think you should either used embedding or isolated control chars for both to have consistency in the output.

Probably the best would be to use LRI on automatically detected consecutive segments, then. We can look into that for the future.

Another option would be to follow CSS and provide control over this: https://developer.mozilla.org/en-US/docs/Web/CSS/unicode-bidi


Replace "default" for any animation name when importing clips

The clips method only worked if the imported model had an animation named "default". This PR allows clips to be created no matter what the one-track animation name is.
This closes my other PR #36709.
Bugsquad edit:

Closes https://github.com/godotengine/godot/issues/25599.
Closes #64486.

I attach a sample project anim-clips.zip

Is this merged on stable? Any ETAs?

Well this needs a review from @reduz who is more familiar with animation workflow.
For what it's worth, while I see value in allowing the use of other default animations than one named "default", I don't like that it arbitrarily picks the first one. It's a valid workflow to actually have an animation named "default", which may not be the first one, but which you want to be used as default.

I don't like that it arbitrarily picks the first one.
Me neither but I've noticed more than one animator struggling with this and it seems that when using clips having one animation is the most common use case, while having it named "default" isn't, but rather the default named assigned by the 3dcg sofware.

The engine doesn't report any error when you try to clip with a different name, it just creates the wrong animations, so it becomes a RTFM moment for the integrator/animator.
If you a dozen of purchased assets it means re exporting every one of those which might even involve getting the software and learning how to do it in what would otherwise work by default.
I think a way it can work without breaking the workflow of those who are following the docs would be the following:

If there is an animation named 'default' use that for clipping
Otherwise use the first one as a fallback rather than failing

That would work well for people explicitly using "default" as well as for assets with only one animation with whatever name.

The clips method only worked if the imported model had an animation named "default".

I can sense the usefulness of using the first track if it isn't named "default". However, the usage of "default" when there there are two tracks needs to be designed.

Is this still the case in 4.0?

I haven't been able to use clips successfully in master branch with the test model I attached above. It keeps being imported as one animation track

@godotengine/animation Can you confirm if this is still needed?

I think it is good, but @fire is the best judge of this feature.

Common workflow is to use one track and then split. Very wanted

Right, then this needs a rebase and a review.

This makes sense, would be good to salvage.

I'll take another look into it

All right. Here's what I did:

Remove slicing options from the animation player import menu
Add such options to the animation import menu
Rename clips to slices wherever was left to be done. This was actually breaking the current workflow.

So the import menu looks like this.

Now instead of using "default" or picking arbitrarily the first one, the user can decide which animation to slice.
Here for example there are two timelines that needed slicing: One the body movement and another for facial animations. Such workflow is now possible with this PR.

Here is my demo project. Let me know what you think:
AnimationSlicesDemo.tar.gz

Thanks!


Cookie jar option is not writing file to disk

It seems to work properly, but the file never goes to disk, i suppose because curl_easy_cleanup is never called.
Also, documentation on this option, or the fact that most of the CURL options are actually available and are described in docs for Ethon is really hard to find
This kind of shows what people do because they can't find it: http://stackoverflow.com/questions/9810150/manually-login-into-website-with-typheous/9867538#9867538.

any progress on this?

would love this to werk :-(

@Zapotek same issue right? or at least v similar

No idea, I used the cookie-jar option recently without issue, didn't check the files or anything though.

It may be something to do with the file handle not being closed / something
GC related. I’m just doing a very simple POC w/ sinatra.
On Tue, Jul 7, 2015 at 1:43 PM Tasos Laskos<EMAIL_ADDRESS>wrote:

No idea, I used the cookie-jar option recently without issue, didn't check
the files or anything though.
—
Reply to this email directly or view it on GitHub
https://github.com/typhoeus/typhoeus/issues/291#issuecomment-119280134.

Just a few hours ago I submitted pull request which can solve this problem (#519).

@i0rek, please, merge the @mrazicz pull request. For me it solves the problem.

hey @i0rek this issue can be closed as it looks like it has been solved.


make ppc64le/ppc64be opt-in

In the analysis phase, we could scan for C files in the opam package and/or a x-test-on-arch: ppc64 flag and turn off that architecture if not found.
The reason is that it would be nice to activate ppc64be testing, but we only have about 8 CPUs available on that arch. So that means we do need to minimise the number of packages using it.

Closing this as we have ppc-hardware now.


Installation of non-required dependency

One of my dependencies is using ruamel.yaml, which has a requirement on ruamel.ordereddict if the installation is happening on CPython <= 2.7.
This dependency does not install on Python 3, and it shouldn't be installed – something pip honors.
Actually, pipenv honors this too, for an initial installation. However, it adds ruamel.ordereddict to the Pipfile.lock anyway, and the wheels come off there when doing a fresh installation of the pipenv. I.e.:
pipenv --rm
pipenv install --three  # now attempts to install ruamel.ordereddict, which has c bindings that won't compile.

Maybe I'm throwing edge cases at you today. But can we figure this out?
Here's the lockfile:
{
    "_meta": {
        "hash": {
            "sha256": "0078ef581d9a09fffae8a4d729aaa6130bcbd078ecb6fb6be73221a9104b41f1"
        },
        "host-environment-markers": {
            "implementation_name": "cpython",
            "implementation_version": "3.6.1",
            "os_name": "posix",
            "platform_machine": "x86_64",
            "platform_python_implementation": "CPython",
            "platform_release": "16.7.0",
            "platform_system": "Darwin",
            "platform_version": "Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64",
            "python_full_version": "3.6.1",
            "python_version": "3.6",
            "sys_platform": "darwin"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.6"
        },
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.python.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {
        "ruamel.ordereddict": {
            "hashes": [
                "sha256:150ce8e6c514a2a2b62753622a75874962561f8e5eeec81a3172ab952807bf0b",
                "sha256:45541836cbfdde630033cae7bbbe35acbac87a0ceec79f944b7a3bedd940fe78",
                "sha256:aee2fa23e884249b4284b728888c553d551e5bfd4de2731f10153fd7813ec55f",
                "sha256:08b4b19fe518d32251a5338e039c4dc9eb0876f2919f94c9b8d2f9446ea80806",
                "sha256:bf0a198c8ce5d973c24e5dba12d3abc254996788ca6ad8448eabc6aa710db149"
            ],
            "version": "==0.4.13"
        },
        "ruamel.yaml": {
            "hashes": [
                "sha256:4bdec0b1a4a12e4a35b788ef53e0b2efd1e9f815ca48615b5244ccdbc8f0b56b",
                "sha256:6432088ef1c3cdc4b304dcec6044e560160c1f51dd0360ae3f0887057d310056",
                "sha256:64868964b64cad9103f37d0b4fc0ec950e5b683bbc47bf4a1ea893fd70aaabf0",
                "sha256:c8da0ea47df5f2dee58f6f904335ac55098b7e72303b01414ce80f37dd3091df",
                "sha256:87bdfb04981f1040ec60b1d1bfa8e7e4d89f7d06c86d4b2fa935002c62e43380",
                "sha256:c20e853cb585167fd5e4ce6e1b57fefe7bd4a61659f8fec5f77e12efa59a7a77",
                "sha256:e884979944642b4ad3222cb102a0d4c17fed2d37abfffdf0f319fae6e41d436a",
                "sha256:10493c92c0e5bd57d576d73b708ed900532846ead1b381c5236c41d0c39f5f71",
                "sha256:3127a87b274e516b3bf2058f64d784089322fddd7322f8e78e9f5380bdfee064",
                "sha256:530bec874292bacbbb80a9143e5182ce436c2a2434a2ea83dda24d30b8e572df",
                "sha256:d92d90c9bc0945223e47223a67808dd97ac9390ed914cc6871479b7ba489e607",
                "sha256:3e87114aac9553c39eea4b88e258eb7fdce39b81a2c399a775efe1f64e4f3d7b",
                "sha256:dfd14829070728159d0dac55a19e4c77514cb8ad3ae3fc2ed065d7f24038b60f",
                "sha256:2f28a3b6665697c20d841a4aee21cfb932bb0db91f293ff97daf845b914ddddb",
                "sha256:2de7bd2d5713c46be9d1d489b028178c9497813f78bd0486a31bbe1c268d7f21",
                "sha256:ddf0e1277664fafff0ae692e8ea2fca56f1ee4daf686d9be785ccdf3a9542744",
                "sha256:f1e29054c6e477963e302b007b6cd1d6c7a58c38d78fabe64fde9ce170d2d1fd"
            ],
            "version": "==0.15.34"
        }
    },
    "develop": {}
}

And, here's the output of pip freeze:
$ pipenv run pip freeze
ruamel.yaml==0.15.34

this has been biting me too  — for maya.

I just noticed maya shares that dep. I guess the temporary workaround would be to clear ruamel.ordereddict from the lock file.

I dont' think we can do anything about this — a proper setup.py file would specify ramuel.ordereddict; python_version > 3, but they're doing something crazy complicated, that i don't think we can get around.

fixed this!

I'm amazing.

https://github.com/kennethreitz/pipenv/commit/05933ae9fdca722e8bf49b617f37b5053a40b825

v8.0.1 fixes this. :)

💫✨🍰✨💫

Hmm. I'm still having this problem. Can you verify that pipenv --three install ruamel.yaml doesn't actually add ruamel.ordereddict to the lockfile?

Oh boy. For some reason this is working in a docker container I created specifically for pipenv, but not on my Mac.

Could there be a cache issue on my mac?

Ah ha, so I've created a regular virtualenv (python3.6 -m venv env) which I install pipenv into, then I've deactivated the virtualenv (deactivate), and run env/bin/pipenv install --three ruamel.yaml and gotten a proper lockfile.
Using python2 (the problem!):
virtualenv env
source env/bin/activate
pip install pipenv
deactivate
env/bin/pipenv install --three ruamel.yaml

... and with this approach, I still get ruamel.ordereddict.
This reflects why my docker image (with only Python3 installed) has no issues, and my Mac, with a pipsi install of pipenv against Python2.7 fails.

you have to use the proper version of python when generating the lockfile.

that's why the lockfile says requires_python

--three is not doing anything for you if you're running in a virtualenv. run pipenv outside of a virtualenv.

So what are we supposed to make of the "fancy install process" that uses pipsi? Is that not a good approach?

pipsi does not run pipenv in a virtualenv. it just installs it in one.

@dfee I was just having this same problem and wanted to comment that your solution works.
For all the Googlers out there if you're on OSX check this:
~/.local/venvs/pipenv/bin/python --version
~/.local/venvs/pipsi/bin/python --version
If it doesn't say Python 3, do this:
pipsi uninstall pipenv
pipsi uninstall pipsi
curl https://raw.githubusercontent.com/mitsuhiko/pipsi/master/get-pipsi.py | python3
pipsi install pipenv


Remove code to configure workbench.editorAssociations based on experiment status

For https://github.com/microsoft/vscode/issues/128495

Codecov Report

Merging #6768 (24afa12) into main (a0c3dd3) will increase coverage by 0%.
The diff coverage is 91%.

:exclamation: Current head 24afa12 differs from pull request most recent head 363fc9c. Consider uploading reports for the commit 363fc9c to get more accurate results

@@          Coverage Diff           @@
##            main   #6768    +/-   ##
======================================
  Coverage     68%     68%
======================================
  Files        411     411
  Lines      28581   28563    -18
  Branches    4271    4267     -4
======================================
+ Hits       19497   19699   +202
+ Misses      7420    7183   -237
- Partials    1664    1681    +17

Impacted Files
Coverage Δ

src/client/common/constants.ts
96% <ø> (-1%)
:arrow_down:

src/client/common/experiments/groups.ts
100% <ø> (ø)

src/client/common/experiments/service.ts
91% <66%> (+2%)
:arrow_up:

...c/client/common/application/customEditorService.ts
81% <100%> (+22%)
:arrow_up:

src/client/datascience/notebook/integration.ts
80% <100%> (+9%)
:arrow_up:

src/client/extensionActivation.ts
98% <100%> (+1%)
:arrow_up:

...t/common/variables/environmentVariablesProvider.ts
75% <0%> (-3%)
:arrow_down:

...ience/kernel-launcher/localKernelSpecFinderBase.ts
79% <0%> (-3%)
:arrow_down:

src/client/datascience/raw-kernel/rawSocket.ts
84% <0%> (-2%)
:arrow_down:

...t/datascience/kernel-launcher/localKernelFinder.ts
83% <0%> (-2%)
:arrow_down:

... and 32 more


How to reinitiate the plugin?

Can you please help me out with how to reinitiate the plugin?
Do you have any destroy/reset function?

Hello @fahimxyz ,
Probably you can trigger click for play/pause button.
If you have any suggestion feel free to send pull requests.
Best Regards!

How to trigger an event loop? eg: onFinish scroll


[bug] keyerror when print scikit-learn interface classifier (and regressor)

How you are using LightGBM?
python package: version=='2.3.1'
Environment info

Python 3.7.7 (default, Mar 26 2020, 15:48:22)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Operating System: Ubuntu 16.04.6 LTS / running on Docker
CPU/GPU model: CPU Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz
C++ compiler version:
CMake version:

LightGBM version or commit hash: 2.3.1
Error message and / or logs
When I try to print an aliased variable by passing it at instantiation, I will get a KeyError.  When I looked it up, I found that it was occurring in most of the variables with aliases
Python 3.7.7 (default, Mar 26 2020, 15:48:22)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from lightgbm import LGBMClassifier
>>> from lightgbm.basic import _ConfigAliases
>>>
>>> error_key_values = []
>>>
>>> for k, values in _ConfigAliases.aliases.items():
...     for v in list(values):
...         clf = LGBMClassifier(**{ k: v })
...         try:
...             print(clf)
...         except KeyError as e:
...             print(f'{k}\t{v}')
...
bin_construct_sample_cnt	subsample_for_bin
bin_construct_sample_cnt	bin_construct_sample_cnt
boosting	boost
boosting	boosting_type
boosting	boosting
categorical_feature	categorical_feature
categorical_feature	cat_column
categorical_feature	categorical_column
categorical_feature	cat_feature
data_random_seed	data_seed
data_random_seed	data_random_seed
early_stopping_round	early_stopping
early_stopping_round	n_iter_no_change
early_stopping_round	early_stopping_rounds
early_stopping_round	early_stopping_round
enable_bundle	bundle
enable_bundle	is_enable_bundle
enable_bundle	enable_bundle
eval_at	eval_at
eval_at	ndcg_eval_at
eval_at	map_eval_at
eval_at	ndcg_at
eval_at	map_at
group_column	group_id
group_column	group
group_column	query_column
group_column	query
group_column	group_column
group_column	query_id
header	has_header
header	header
ignore_column	ignore_column
ignore_column	blacklist
ignore_column	ignore_feature
is_enable_sparse	enable_sparse
is_enable_sparse	is_enable_sparse
is_enable_sparse	is_sparse
is_enable_sparse	sparse
label_column	label
label_column	label_column
machines	machines
machines	workers
machines	nodes
metric	metric
metric	metrics
metric	metric_types
num_class	num_classes
num_class	num_class
num_iterations	num_boost_round
num_iterations	num_round
num_iterations	num_rounds
num_iterations	n_iter
num_iterations	num_iterations
num_iterations	n_estimators
num_iterations	num_iteration
num_iterations	num_trees
num_iterations	num_tree
LGBMClassifier(objective='objective')
LGBMClassifier(objective='application')
LGBMClassifier(objective='app')
LGBMClassifier(objective='objective_type')
pre_partition	pre_partition
pre_partition	is_pre_partition
two_round	use_two_round_loading
two_round	two_round
two_round	two_round_loading
verbosity	verbose
verbosity	verbosity
weight_column	weight_column
weight_column	weight

The details of the error are as follows (when set verbose).
KeyErrorTraceback (most recent call last)
/opt/conda/lib/python3.7/site-packages/IPython/core/formatters.py in __call__(self, obj, include, exclude)
    968
    969             if method is not None:
--> 970                 return method(include=include, exclude=exclude)
    971             return None
    972         else:

/opt/conda/lib/python3.7/site-packages/sklearn/base.py in _repr_mimebundle_(self, **kwargs)
    461     def _repr_mimebundle_(self, **kwargs):
    462         """Mime bundle used by jupyter kernels to display estimator"""
--> 463         output = {"text/plain": repr(self)}
    464         if get_config()["display"] == 'diagram':
    465             output["text/html"] = estimator_html_repr(self)

/opt/conda/lib/python3.7/site-packages/sklearn/base.py in __repr__(self, N_CHAR_MAX)
    277             n_max_elements_to_show=N_MAX_ELEMENTS_TO_SHOW)
    278
--> 279         repr_ = pp.pformat(self)
    280
    281         # Use bruteforce ellipsis when there are a lot of non-blank characters

/opt/conda/lib/python3.7/pprint.py in pformat(self, object)
    142     def pformat(self, object):
    143         sio = _StringIO()
--> 144         self._format(object, sio, 0, 0, {}, 0)
    145         return sio.getvalue()
    146

/opt/conda/lib/python3.7/pprint.py in _format(self, object, stream, indent, allowance, context, level)
    159             self._readable = False
    160             return
--> 161         rep = self._repr(object, context, level)
    162         max_width = self._width - indent - allowance
    163         if len(rep) > max_width:

/opt/conda/lib/python3.7/pprint.py in _repr(self, object, context, level)
    391     def _repr(self, object, context, level):
    392         repr, readable, recursive = self.format(object, context.copy(),
--> 393                                                 self._depth, level)
    394         if not readable:
    395             self._readable = False

/opt/conda/lib/python3.7/site-packages/sklearn/utils/_pprint.py in format(self, object, context, maxlevels, level)
    168     def format(self, object, context, maxlevels, level):
    169         return _safe_repr(object, context, maxlevels, level,
--> 170                           changed_only=self._changed_only)
    171
    172     def _pprint_estimator(self, object, stream, indent, allowance, context,

/opt/conda/lib/python3.7/site-packages/sklearn/utils/_pprint.py in _safe_repr(object, context, maxlevels, level, changed_only)
    412         recursive = False
    413         if changed_only:
--> 414             params = _changed_params(object)
    415         else:
    416             params = object.get_params(deep=False)

/opt/conda/lib/python3.7/site-packages/sklearn/utils/_pprint.py in _changed_params(estimator)
     96     init_params = {name: param.default for name, param in init_params.items()}
     97     for k, v in params.items():
---> 98         if (repr(v) != repr(init_params[k]) and
     99                 not (is_scalar_nan(init_params[k]) and is_scalar_nan(v))):
    100             filtered_params[k] = v

KeyError: 'verbosity'

Other features, such as learning and reasoning, work just fine.
This is probably due to a version of scikit learn. The version that gave the previous error was 0.23.0. Downgrade this to 0.22.1, and it works fine.

I'm getting the same KeyError (_changed_params raised in sklearn/utils/_pprint.py) but for a different LGBM hyperparameter - n_estimators when trying to print an imported model object (LGBM regressor model) which worked many times before for the same binary object until the latest upgrade in scikit-learn.
We train many LGBM models versions each day (thank you for their stability!) and preserve both their model objects and the output of pip freeze, so I can rather confidently pinpoint the dependencies versions where a bug was introduced.
So now it looks like the compatibility issue was introduced in scikit-learn==0.23.0:
First sklearn version with the error:
(modeling pipeline run from 2020-05-18 19:28:06)
scikit-learn==0.23.0

Last scikit-learn version without the error:
(modeling pipeline run from 2020-05-14 18:55:47)
scikit-learn==0.22.2.post1

I can now confirm that indeed the problem was introduced in scikit-learn==0.23.0, but downgrading the package to scikit-learn==0.22.2.post1 eliminates it.
I think @nyk510 that it should be an issue for scikit-learn devs too - will you report it there or shall I?

@mirekphd
That's certainly true.
I've created an issue here:D

This will be fixed in 0.23.1 which should be out soon
https://github.com/scikit-learn/scikit-learn/issues/17206

In fact 0.23.1 is already out so the issue can probably be closed (it's a scikit-learn issue anyway)

@NicolasHug Thank you for your polite comment!

This will be fixed in 0.23.1 which should be out soon
scikit-learn/scikit-learn#17206

@nyk510, the issue negatively impacting lightgbm was indeed fixed in scikit-learn==0.23.1, so the issue here can be probably closed.
On the other hand, the fixed version of scikit-learn 0.23.1 does not get installed by default by pip in my containers, and it has to be pinned or else the buggy version 0.23.0 gets installed by default (probably other packages that use scikit-learn did not get upgraded yet).
So I suppose it takes time and the buggy version to be rooted out in all dependencies, and it cannot be simply taken down from PyPI (probably will need to stay there for a long time).

the fixed version of scikit-learn 0.23.1 does not get installed by default by pip

This is surprising, what are you using? On a clean env, pip install scikit-learn will install 0.23.1. If you already have a scikit-learn version installed you'd need to use the --upgrade flag.

the fixed version of scikit-learn 0.23.1 does not get installed by default by pip

This is surprising, what are you using? On a clean env, pip install scikit-learn will install 0.23.1. If you already have a scikit-learn version installed you'd need to use the --upgrade flag.

Right, but I thought that the need for a forced upgrade does not apply to docker builds... apparently the use of docker's cache replicates the situation of a previous package version being already "installed".
When I look into Jenkins log for a build triggered by my unpinning of scikit-learn (from previous working version 0.22.2.post1), I see that package installations layers with both pip and conda were retrieved from docker cache, restoring the buggy version 0.23.0:
Step 30/50 : RUN pip install -r /tmp/python-packages/pypi-packages.txt --no-cache-dir

 ---> Using cache

 ---> 0975530a5deb

Step 31/50 : RUN conda install --channel conda-forge --yes --file /tmp/python-packages/conda-forge-packages.txt

 ---> Using cache

 ---> 59b63c286bb4

But wait, there's more.:) During the next automatic build of this python container 15 minutes later (this time a time-triggered one intended to check for available package and security upgrades) did correctly identify that a newer version of scikit-learn is available, but since only build has changed, the script ignored this change and left the buggy version 0.23.0. So I suppose we had the well konwn scenario "it's not a but, it's a feature":)
scikit-learn :
- version installed:  0.23.0
- latest  available:  0.23.1
Note: ignoring build differences

- package upgradeable:  False


Plugin does not work in v1.7.7

The plugin does not work in Obsidian v1.7.7. I think #9 has the reason and solution.

me too


Calendar must show items before 9am

Yes, students, there is an 8 AM.
I just entered this event that was missing from the calendar. Or was it? Now that I've entered it, it's not visible, because the day view starts at 9am — maybe there was one there already.

Resolved by #201.


vite preview can't use --mode option

Describe the bug
When I use vite preview -h,
vite/5.2.12

Usage:
  $ vite preview [root]

Options:
  --host [host]           [string] specify hostname
  --port <port>           [number] specify port
  --strictPort            [boolean] exit if specified port is already in use
  --open [path]           [boolean | string] open browser on startup
  --outDir <dir>          [string] output directory (default: dist)
  -c, --config <file>     [string] use specified config file
  --base <path>           [string] public base path (default: /)
  -l, --logLevel <level>  [string] info | warn | error | silent
  --clearScreen           [boolean] allow/disable clear screen when logging
  -d, --debug [feat]      [string | boolean] show debug logs
  -f, --filter <filter>   [string] filter debug logs
  -m, --mode <mode>       [string] set env mode
  -h, --help              Display this message

But the vite preview command doesn't use the mode option.
A similar problem was solved here, but apparently without success:
https://github.com/vitejs/vite/issues/6391
Reproduction
any project
Steps to reproduce
any project
System Info
System:
    OS: Linux 5.15 Ubuntu 20.04.6 LTS (Focal Fossa)
    CPU: (4) x64 Intel(R) Core(TM) i3-9100 CPU @ 3.60GHz
    Memory: 23.49 GB / 31.22 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 20.12.2 - ~/.nvm/versions/node/v20.12.2/bin/node
    npm: 10.5.0 - ~/.nvm/versions/node/v20.12.2/bin/npm
  Browsers:
    Chrome: 125.0.6422.141
  npmPackages:
    @vitejs/plugin-vue: ^5.0.4 => 5.0.5
    vite: ^5.2.0 => 5.2.12

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

It is passed here https://github.com/vitejs/vite/blob/e3c8278dc7a58aa738675fe35c449de30344ce58/packages/vite/src/node/cli.ts#L352. Can you provide a repro?

I wrote a tool which may behave a little more like you expect. It puts together vite build with watching and a preview server as a single command.

https://www.npmjs.com/package/vite-live-preview
https://github.com/Shakeskeyboarde/vite-live-preview

Hope this helps. Feel free to post issues on the repo or contribute.


Suggestion from web: missing function arguments in PowerPC binary

Binary sent to us by user through https://retdec.com/.
Reported problems:

Missing/bad arguments in several functions (e.g. Max_int__Fii() at 0xcc730).
Bad return types.

Check it problem still occurs.

This problem does no longer occurs. I am closing this issue.
Current output:
  // Address range: 0xcc730 - 0xcc754
  int32_t Max_int__Fii(int32_t a1, int32_t a2) {
      // 0xcc730
      int32_t v1; // 0xcc730
      __asm_cror_38(v1, v1, v1);
      uint32_t v2 = __asm_mfcr(a1, a2, v1, v1, v1, v1, v1, v1); // 0xcc738
      uint32_t v3 = v2 % 2; // 0xcc73c
      return v3 - 1 & v2 | -v3 & a2;
  }


Using SSR with defined dynamic components, not rendered classes at tag (component)

Vue.js version
2.0.2
Steps to reproduce

Create component

Vue.component('testcomponent', {
  render: h => h('div', 'contain')
})

Use the components within the parent component template with className

    <testcomponent class="testclass" />

What is Expected?
Expected to render on the server side

contain

Similarly, as the client-side
What is actually happening?
The server side is not renders classes in dynamically generated components

contain

At the same time, the client side, renders everything with classes

contain

This behavior causes a twitch of the interface, due to the belated addition of classes with styles.
Also incorrect display if the js file not to load on slow internet.

Hello @PowerfulPony,
thank you for your filing this issue.
Please follow the guidelines on how to report an issue. In particular, provide an example on www.jsfiddle.net (or a similar service) that reproduces the problem.
Thank you.

Hi @LinusBorg,
I do not understand how you can demonstrate through the server render jsfiddle.

In this case, a repository would be the better option, oviously.

Seems to be a bug, similar to #4076


Add Project CI Variables

Description of your changes
This PR implements GitLab Project Variables, allowing Crossplane to manage project-level CI/CD variables and exert control over the build process.
Currently this only supports plain-text variable values so should not be used with sensitive content, although it does support protecting and masking values once in GitLab.
I've taken a slightly different approach to implementation of upToDate in this resource, converting the gitlab.ProjectVariable representation back into our own v1alpha1.VariableParameters struct, and then comparing the two with cmp.Equal. I've found this approach to be robust and more succinct in provider-cloudflare, but I know this doesn't follow convention within this provider so happy to implement differently if necessary.

Adding the same functionality for Group Variables should be a simple copy-and-paste - happy to add to this PR, or submit separately. I've not done this yet as I think the new code will be easier to review without it.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Tests have been added for the additional controller and client, and the provider itself has been run against a kind cluster and a manually created project to confirm functionality against real GitLab.

lgtm, thanks @benagricola!
would you please squash the commits?

Done @janwillies :)


https://docs.rs/futures-concurrency/latest/futures_concurrency/ is stale

When I load https://docs.rs/futures-concurrency/latest/futures_concurrency/ I see version 6.0.1, even if I use Chrome dev tools to disable my browser's cache.

response headers:
age: 78002
alt-svc: h3=":443"; ma=86400
content-encoding: gzip
content-type: text/html; charset=utf-8
date: Thu, 17 Nov 2022 00:56:29 GMT
server: nginx/1.14.0 (Ubuntu)
vary: Accept-Encoding
via: 1.1 bf7159e30a38421f642619d6da9a8eb4.cloudfront.net (CloudFront)
x-amz-cf-id: 3y6ebsTcpSWEqI8LuMKQ9oyVgKPwhFB4FPuxbQco8XXkWmm6wUUaAQ==
x-amz-cf-pop: IAD12-P2
x-cache: Hit from cloudfront

Version 7.0.0 was released 21 hours ago, according to https://docs.rs/releases/recent/32 (long enough ago that I had to hit next a bunch to find it!).

Ahh here's the build: https://docs.rs/crate/futures-concurrency/7.0.0/builds/674685
7.0.0 shows up in search and takes me to https://docs.rs/futures-concurrency/7.0.0/futures_concurrency/ which looks as I'd expect. Its headers:
age: 19452
alt-svc: h3=":443"; ma=86400
cache-control: stale-while-revalidate=86400
content-encoding: gzip
content-type: text/html; charset=utf-8
date: Thu, 17 Nov 2022 17:12:56 GMT
server: nginx/1.14.0 (Ubuntu)
vary: Accept-Encoding
via: 1.1 b7f525be96cecf61bbec66a423b622b0.cloudfront.net (CloudFront)
x-amz-cf-id: 2bKT_Q6Jz_-KIk6ayeMz7Q1NjbK4RlUsjGbgWEN64hRHRGjsT-OHPw==
x-amz-cf-pop: IAD12-P2
x-cache: Hit from cloudfront

I see mention of CDN invalidations in other issues. E.g. #1877 says they take up to 15 minutes. That's well past, so is it possible that step got entirely skipped for this release for some reason?

It definitely looks like an invalidation issue, the date is from a couple minutes before the build completed and the path should have been invalidated. Looking through a couple more recent crates I found https://docs.rs/testdir/latest/testdir/ which similarly isn't updated.
We don't have any error logs from the invalidations and at a glance it seems to be configured correctly, but I don't know whether we have any other tools to see if the invalidations are being processed correctly. Hopefully @syphar has some ideas what to look at.

Thank you for reporting this!
This is really very odd. We don't see any error in the logs, so creating the invalidation request with CloudFront succeeded.
The only possible explanation I have is that somehow the invalidation failed, since we don't query & check the invalidation status yet.
Currently the CDN caches only for a day, so the pages should be up-to-date now (@scottlamb please ping me if they are not up-to-date now for you, then I'll manually invalidate again).
I'll check with infra about CloudFront if they can see any failed invalidations, possibly then reducing the caching time again, and/or writing code to check & retry the invalidations.

related discussion: https://rust-lang.zulipchat.com/#narrow/stream/242791-t-infra/topic/docs.2Ers.20.2F.20more.20CloudFront.20caching

Since #1961 we have an invalidation queue & reporting on it.
We are running with a one-day TTL for some weeks already, so I'm closing this issue as solved.


wikieditor not showing

MediaWiki = 1.26.2
PHP = 5.6
Ive just installed the extension WikiEditor and bootstrap-mediawiki skin.
When I edit a page in my wiki, the WikiEditor toolbar doesn't show.(The first show once)
I've installed another skin(vector). And I try again.
In other skin it appears normal.

If the refresh is shown.


Support Lua5.2

We received users using lua5.2 instead of luajit with torch, and that will cause errors as #31 and #30 etc.

I tried to do fresh installation with lua5.2 but still have problem with lutorpy:
ctypes.CDLL(os.path.join( "/home/wei/torch/install/lib/liblua.so"), mode=ctypes.RTLD_GLOBAL)
ctypes.CDLL(os.path.join( "/home/wei/torch/install/lib/libluaT.so"), mode=ctypes.RTLD_GLOBAL)
ctypes.CDLL(os.path.join( "/home/wei/torch/install/lib/libTH.so"), mode=ctypes.RTLD_GLOBAL)

Then I got the following error:
In [27]: import lutorpy
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-27-db94a7dcb3df> in <module>()
----> 1 import lutorpy

/usr/local/lib/python2.7/dist-packages/lutorpy-1.3.7-py2.7-linux-x86_64.egg/lutorpy/__init__.py in <module>()
     42 # the following is all that should stay in the namespace:
     43
---> 44 from lutorpy._lupa import *
     45
     46 try:

ImportError: /usr/local/lib/python2.7/dist-packages/lutorpy-1.3.7-py2.7-linux-x86_64.egg/lutorpy/_lupa.so: undefined symbol: luaL_openlib
``

Hi,
Any possibility of supporting Lua5.2 ? I am using Lua5.2 to bypass limit on table size from luajit.
Kind regards


Restore Config value read performance

Restore the performance of Config on value reads back to its state before #143.
Here we capture all values on read to Config's internal _values hash. This means that after first read, all subsequent reads should be fast.
Then to preserve the functionality of #configured?, we keep a separate _configured Set that tracks the names of values that have either been assigned, or mutable values that have been read. For the very small amount of memory (one set per config object, since most of the names, as symbols, should already be in memory) this is a worthwhile tradeoff for the best possible performance on read, which should reasonably be expected by our users to be as fast as possible.

@_values is the hash where the config values are kept, regardless of where they came from (i.e. via a setting default or explicit assignment). We keep them all in this hash to ensure that reads remain quick. This is how it was in previous releases of the library.
In #143 I changed how we populated _values so that it could be used to determine whether a user had explicitly assigned a value. But this had some negative performance ramifications. This change was NOT a part of the memory optimisation effort; it was me adding what I believed to be a missing feature, based on places we'd been using dry-configurable.
To restore config read performance, in this PR I'm letting _values be populated again on every read. This means we now need a separate place to track whether a user has explicitly assigned a config value, hence _configured.

@solnic @flash-gordon Folks, this is ready to go. Since it is a fix to a performance regression I'm incline to get it in sooner rather than later and ship a patch release. Please let me know if you have any concerns :)


Support creation of probes by json

Probes can be exported to json.
Support importing these probes directly via json.

Resolved by #206


Print full n-inference failures dict

This PR changes the printing of the n-inference failures dict to not truncate printing the dict, which happens in the log at the moment:
┌ Info: n-jet failures (excluding n=0)
│   nfailures =
│    Dict{Any, Any} with 13 entries:
│      "implicit_tendency!"             => 674
│      "hyperdiffusion_tendency!"       => 642
│      "additional_tendency!"           => 15
│      "update_surface_conditions!"     => 598
│      "fill_with_nans!"                => 3
│      "limited_tendency!"              => 649
│      "horizontal_advection_tendency!" => 584
└      ⋮                                => ⋮

The dict is 16 entries long, so it shouldn't bloat the log too much.

bors r+

bors r+

bors r+


ESP32: Add optional NimBLE support for BLE transport

Problem
This PR adds NimBLE as optional BLE transport layer for ESP32 platform. The PR is still work in progress because it has not been validated or tested against any test suite plus there are still few pending items.  I have verified rendezvous session establishment with chip-tool example.

Summary of Changes

Adds BLEManagerImpl.cpp and BLEManagerImpl.h for NimBLE.
Refactor ESP32 into bluedroid and nimble sub-directories with corresponding BLEManagerImpl.cpp files.
Modified src/platform/Makefile.am and src/platform/BUILD.gn to accommodate nimble support in make and GN build system.

fixes https://github.com/project-chip/connectedhomeip/issues/1274 and https://github.com/project-chip/connectedhomeip/issues/1115

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Given this is WIP, moving to Draft

How can I validate the implementation using rendezvous ? Is there any documentation available for rendezvous ?

How can I validate the implementation using rendezvous ? Is there any documentation available for rendezvous ?

cc @vivien-apple @shana-apple

How can I validate the implementation using rendezvous ? Is there any documentation available for rendezvous ?

cc @vivien-apple @shana-apple

Assuming the ESP32 is configured and flashed properly, you need to run out/debug/standalone/chip-standalone-demo echo-ble 3840 1234567 from your macOS build.
Basically 3840 correspond to 0x0F00 (default discriminator) and 1234567 is the default PIN Code used for testing.
I will try to update the documentation with complete build steps ASAP.
In the meantime feel free to ping me if you have any questions.

@vivien-apple @turon @andy31415 @mspang I have modified the PR, please take a fresh look. I have verified the rendezvous session establishment using chip-tool. Thanks.

the Dockerfile update implies a version change. can you please split the dockerfile update into a separate PR (and include an update to integrations/docker/images/chip-build/version)?
The Dockerfile PR would need to land first, and a docker.io push will be necessary. Then this PR would have updated CI scripts pointing to the new version of the docker images.

Hi @rwalker-apple Dockerfile change is unnecessary one, I have made relevant change in my code so it negates the necessity of updating esp-idf from tags/v4.1 to release/v4.1.  Please feel free to remove integrations label from the PR.

Here is the summary of the footprint consumed by Bluedroid vs NimBLE BLE stack (for wifi-echo example).

Memory footprint
Bluedroid (Bytes)
NimBLE (Bytes)
Savings with NimBLE (KB)

Static memory
1185018
922782
> 262KB

Free Heap
112784
140028
> 27 KB

@prasad-alatkar could you resolve the merge conflicts please?

@prasad-alatkar could you resolve the merge conflicts please?

I have resolved the merge conflicts. Thanks.

@saurabhst @BroderickCarlin @jelderton @chrisdecenzo @hawk248 could you take a look please?


Autofix for functional/prefer-readonly-type produces confusing formatting

Bug Report
Autofix for functional/prefer-readonly-type on a mapped type like the following inserts readonly on a wrong line:
let a: {
  [B in 'c']: 'd';
};

Expected behavior
let a: {
  readonly [B in 'c']: 'd';
};

Actual behavior
let a: { readonly
  [B in 'c']: 'd';
};

Proposed changes
Put readonly on the correct line.
Additional information
Related to #384

I'm not sure why this would be happening. I can look into it but formatting isn't the goal of this plugin; another rule or tool should be used to manage/fix formatting.


MdSelect Input field for filtering (space key prevented)

First, love the work you are doing here, I use it in one of my projects, I come from Angular Material so all of this is extremely familiar to me and soon I hope to contribute to your work.
I tried implementing a search filter field that sits above the menu items in the select component. Similiar to Angular Materials structure. It works great, except that md-menu-content prevents the space key as an action, causing me not being able to press the space bar in the input field.
I'm trying to think of ways to get around this. Another named slot that will put stuff outside of md-menu-content perhaps?
Right now I can live with it in my project, but I'm curious as to a way you would like this implemented or a thought for the future.
Thanks a lot!

Thanks for the words.
Yeah. The <md-select> are intended to use the 'space' key event to confirm the selection. I have plans to implement the select header with a search behaviour. There is no workaround for now, sorry about that.

I will close this issue for now and reopen when I start to develop this improvement. Thanks!


Unstable connection on Docker for M1 Macs

Bug description
Recently upgraded to an M1 mac from an Intel mac. Running a docker-compose config that worked perfectly on old mac.
The issues seems to be coming from prisma. I cannot have a reliable connection between my prisma client / cli and the database (tried postgres, mysql and mariadb but unable to make it work).
This is the output I get when running a prisma migration and have enabled all logs on postgres
postgres_1        |
postgres_1        | PostgreSQL Database directory appears to contain a database; Skipping initialization
postgres_1        |
postgres_1        | 2021-05-26 16:00:02.571 UTC [1] LOG:  starting PostgreSQL 12.7 (Debian 12.7-1.pgdg100+1) on aarch64-unknown-linux-gnu, compiled by gcc (Debian 8.3.0-6) 8.3.0, 64-bit
postgres_1        | 2021-05-26 16:00:02.572 UTC [1] LOG:  listening on IPv4 address "<IP_ADDRESS>", port 5432
postgres_1        | 2021-05-26 16:00:02.572 UTC [1] LOG:  listening on IPv6 address "::", port 5432
postgres_1        | 2021-05-26 16:00:02.575 UTC [1] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
postgres_1        | 2021-05-26 16:00:02.607 UTC [28] LOG:  database system was interrupted; last known up at 2021-05-26 15:55:50 UTC
postgres_1        | 2021-05-26 16:00:04.357 UTC [28] LOG:  database system was not properly shut down; automatic recovery in progress
postgres_1        | 2021-05-26 16:00:04.374 UTC [28] LOG:  redo starts at 0/17B13B8
postgres_1        | 2021-05-26 16:00:04.375 UTC [28] WARNING:  could not open directory "base/16701": No such file or directory
postgres_1        | 2021-05-26 16:00:04.375 UTC [28] CONTEXT:  WAL redo at 0/17B1468 for Database/DROP: dir 1663/16701
postgres_1        | 2021-05-26 16:00:04.375 UTC [28] WARNING:  some useless files may be left behind in old database directory "base/16701"
postgres_1        | 2021-05-26 16:00:04.375 UTC [28] CONTEXT:  WAL redo at 0/17B1468 for Database/DROP: dir 1663/16701
postgres_1        | 2021-05-26 16:00:04.377 UTC [28] LOG:  invalid record length at 0/17B1510: wanted 24, got 0
postgres_1        | 2021-05-26 16:00:04.377 UTC [28] LOG:  redo done at 0/17B14D8
postgres_1        | 2021-05-26 16:00:04.410 UTC [1] LOG:  database system is ready to accept connections
migration_1       | Prisma schema loaded from prisma/schema.prisma
migration_1       | Datasource "db": PostgreSQL database "lantern", schema "public" at "postgres:5432"
migration_1       |
postgres_1        | 2021-05-26 16:00:21.964 UTC [35] LOG:  statement:
postgres_1        | 	            SET search_path = "public";
postgres_1        | 	            SET NAMES 'UTF8';
postgres_1        |
postgres_1        | 2021-05-26 16:00:22.062 UTC [35] LOG:  execute s0: SELECT EXISTS(SELECT 1 FROM pg_namespace WHERE nspname = $1)
postgres_1        | 2021-05-26 16:00:22.062 UTC [35] DETAIL:  parameters: $1 = 'public'
postgres_1        | 2021-05-26 16:00:22.115 UTC [36] LOG:  statement:
postgres_1        | 	            SET search_path = "public";
postgres_1        | 	            SET NAMES 'UTF8';
postgres_1        |
postgres_1        | 2021-05-26 16:00:22.119 UTC [36] LOG:  execute s0: SELECT EXISTS(SELECT 1 FROM pg_namespace WHERE nspname = $1)
postgres_1        | 2021-05-26 16:00:22.119 UTC [36] DETAIL:  parameters: $1 = 'public'
postgres_1        | 2021-05-26 16:00:22.152 UTC [36] LOG:  execute s1: SELECT "id", "checksum", "finished_at", "migration_name", "logs", "rolled_back_at", "started_at", "applied_steps_count" FROM "_prisma_migrations" ORDER BY "started_at" ASC
postgres_1        | 2021-05-26 16:00:22.174 UTC [36] LOG:  statement: CREATE DATABASE "prisma_migrate_shadow_db_5d09e104-f1e8-4639-b71a-c03b1ade7df2"
postgres_1        | 2021-05-26 16:00:29.151 UTC [36] LOG:  statement: DROP DATABASE IF EXISTS "prisma_migrate_shadow_db_5d09e104-f1e8-4639-b71a-c03b1ade7df2"
migration_1       | Error: P1001
migration_1       |
migration_1       | Can't reach database server at `postgres`:`5432`
migration_1       |
migration_1       | Please make sure your database server is running at `postgres`:`5432`.
migration_1       |

As you can see, postgres boots properly, the migration script is able to connect and perform some steps but fails saying it cannot reach the database (while it clearly was able to run sql queries).
When migration is made from the host it works properly.
Also some services that are using the prisma client cannot reach the database, and it happens in a relatively random manner.
How to reproduce
Running prisma migrate from a node container on another postgres container, on an M1 mac.
Expected behavior
Prisma works on M1 Mac + Docker
Prisma information

"@prisma/client": "2.23.0"
"prisma": "2.23.0"

Environment & setup

OS: Mac OS M1
Database: Postgres/MySQL/Mariadb in docker containers
Node.js version: 16

Prisma Version
    "@prisma/client": "2.23.0"
    "prisma": "2.23.0"

Sorry wasn't aware of the extremelly slow db issue on docker + mac M1. If anyone comes across this, add timeout arguments to your database uri (ex: postgres://user:pwd@postgres:5432/db?connect_timeout=30&pool_timeout=30&socket_timeout=30) and it should work !


Nextflow processes with exec do not show up in the Legacy Renderer

Just an observation, not sure if this is intentional or a known issue, but if you have a Nextflow process like this

process EXEC_FOO {
    input:
    tuple val(prefix), val(constant)

    output:
    path(outputfile)

    exec:
    println ">>> EXEC: ${prefix}, ${constant}"
    outputfile = new File("${task.workDir}/${prefix}.exec.txt")
    outputfile.write(constant)
}

it does not seem to show up in the legacy manifest JSON output

Does it appear in the BCO output?

No it does not seem to appear in either. See the results from running the PR #31


Update versions

Update golang and kubectl versions to the latest ones.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@jessesuen can you retrigger the CI? Or I need to force push to retrigger it? Is there something blocking the merge?

/assign @jessesuen


Unable to use static IP address before Ignition on VMWare

Description
We are unable to use a static IP before Ignition following https://docs.flatcar-linux.org/ignition/network-configuration/#using-static-ip-addresses-with-ignition
Impact
Every provisioned Flatcar VM gets two leases from DHCP and we need them to get only one lease
Expected behavior
We create a template using an Ignition with the following grub.cfg
set linux_append="$linux_append ip=IP:::NETMASK::ens192:none"
set oem_id="vmware"

Once the template is created with that Ignition applied, we use that template to create Flatcar VMs that don't run DHCP before Ignition, and thus they only get one lease assigned to them.
Notes
We have tried many combinations of ip=<ip>::<gateway>:<netmask>:<hostname>:<iface>:none[:<dns1>[:<dns2>]]
with/without <gateway>,with/without <hostname>, with/without <iface>, with/without <dns1> and none of them have worked.
The server always runs DHCP before Ignition and gets a lease

Hi,
thanks for reporting, this should be fixed in the new Beta release, can you please give it a try? https://www.flatcar-linux.org/releases/#release-2513.2.0

Hi,
I ran some tests and it worked like a charm!
Thank you so much for all your help.
I love Flatcar!


Support with big dataset

Hi,
If we want to find architectures on big datasets such as ImageNet, how to change our code to support it?
Thanks

You should start here, which can take an architecture and evaluate it on your problem: https://github.com/kirthevasank/nasbot/blob/master/demos/cnn_function_caller.py
For specifics on converting our nn representation into TF, look at this directory: https://github.com/kirthevasank/nasbot/tree/master/cg

Thank you, I will try it again.


Add: Managing Events with Notification Center abstract

Notes for Reviewers
This PR fixes #10037
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 100a5a5038a457aa49635d330871b0484ab0d67d at: https://65ba19aa6f52ecb01c7638a7--meshery-docs-preview.netlify.app