bug: IonSelect is not showing the options when clicked (React)

Bug Report
Ionic version:
@ionic/react 5.1.1,

Current behavior:
When clicking on the <IonSelect> element, the options do not get displayed.
Expected behavior:
When clicking on the <IonSelect> element, the options are displayed.
Steps to reproduce: / Related code:
I've tried copy-pasted the code directly from the documentation and still run into this issue.
<IonList>
  <IonItem>
    <IonLabel>Hemisphere</IonLabel>
    <IonSelect
      value={hemisphere}
      interface="popover"
      onIonChange={(e) => {
        SettingsStorage.setHemisphere(e.detail.value!)
        setHemisphere(e.detail.value!)
      }}
    >
      <IonSelectOption value="N">Northern</IonSelectOption>
      <IonSelectOption value="S">Southern</IonSelectOption>
    </IonSelect>
  </IonItem>
</IonList>

Other information:
It appears that when I do the following, I am able to get the options to display:

Right click on the <IonSelect> in the browser.
Inspect element
A <button> gets highlighted in the devtools. I delete this
I click the <IonSelect> in the browser again
Options displayed!

Ionic info:
Ionic:
   Ionic CLI       : 6.9.1 (/Users/grardb/.config/yarn/global/node_modules/@ionic/cli)
   Ionic Framework : @ionic/react 5.1.1

Capacitor:
   Capacitor CLI   : 2.1.0
   @capacitor/core : 2.1.0

Utility:
   cordova-res : not installed
   native-run  : not installed

System:
   NodeJS : v11.14.0 (/Users/grardb/.nvm/versions/node/v11.14.0/bin/node)
   npm    : 6.7.0
   OS     : macOS High Sierra

A gif of what's happening:

Thanks for the issue. Can you provide a full reproduction of the issue? I am unable to reproduce the issue using the code snippet provided.

Hrm, so I think I've "figured it out" (not really, though). I can reproduce it using the code snippet above in a fresh Ionic project (blank template), but there appears to be a caveat: I need to be in Firefox, and I need to switch from the default browsing experience to the mobile-sized/responsive screen using the devtools. This bug doesn't seem to happen when doing the same in Chrome, however.
The bug doesn't seem to occur if I do a normal resize of the Firefox window, so I assume there's no resize event messing things up or something.
Furthermore, it seems that when changing the size of the responsive screen, I'm then able to open the menu.

At this point, I'm not sure if this is a cross-compatibility issue with Ionic or a Firefox bug. My hunch is that it has something to do with the shadow DOM, but I don't know much about that stuff, tbh.

It looks like this is an issue with the touch simulation in Firefox. I was able to reproduce it using those steps but only if the "Touch Simulation" button is enabled.

Ah you know what this is actually a duplicate of an older bug: https://github.com/ionic-team/ionic/issues/20179. This ended up being a bug in Firefox with Touch Simulation, and not a bug in Ionic Framework.
The workaround for now is to not use Touch Simulation when developing in Firefox. We will report this to the Firefox dev team. Thanks!

Hey there, I filed a bug with the Firefox team: https://bugzilla.mozilla.org/show_bug.cgi?id=1640189. Looks like this has been fixed in Firefox 78, so a fix should be available whenever that ships.


about code issues

hi,
I found some issues about your code..first,you don't upload complete files, for example: solver.py and models and so on...
And,i found your code where on main.py 62 line, you should define [1,2,3] ... not a string... you don't upload the way how to train and test,,if it is necessary pls upload them...thx...

again issues, when I was trainings found some code wrong,MAE_illum at train model should be MAE_illum[0].item() not item()...


Properly inherit constraints for nested abstracts

The constraints seem not to be inherited properly:
abstract Component
    abstract Port
        throughput -> integer // [Mbit/s]
    abstract Port10Mbit : Port
        [ throughput = 10 ]

abstract Port100Mbit : Port
    [ throughput = 100 ]

C1 : Component
    R1 : Port10Mbit
    R2 : Port100Mbit

The expected instance (produced by AlloyIG) is:
C1
  R1
    throughput$1 -> 10
  R2
    throughput$2 -> 100

However, chocosolver produces:
C1
  R1
    throughput -> -1001
  R2
    throughput$1 -> -1001

Fixed.


.c file getting saved as cpp file

Type: Feature Request
I am saving my file as .c file but it is getting saved as cpp file
VS Code version: Code 1.74.2 (e8a3071ea4344d9d48ef8a4df2c097372b0c5161, 2022-12-20T10:29:14.590Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: No

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add tether cable

The tether between the piston and the heave cone is currently a rigid link. We should make it more flexible so it can move like a rope. The first naive approach will be to compose the cable of various cylindrical rigid links connected by ball or universal joints.

We have an initial implementation of the tether and its guide, added on the PRs above. The current sausage-links approach is tough on the physics engine, hurting performance and causing instabilities in extreme conditions. If these become problems during regular use, we should look into an alternative approach.


Added ability to delete profile picture

Describe the changes you have made in this PR:
Added a button on the show profile page to delete the profile picture. Issue #595
Screenshots of the changes (If any):

Coverage decreased (-0.3%) to 82.307% when pulling<PHONE_NUMBER>aaff7c32642c6a55d199e27e78f83e on nrs1729:remove-user-photo-595 into 2c01dad8e0b59646b9625313a0da5801dde1e221 on CircuitVerse:master.

Oh sorry forgot about tests, I'll add them

@satu0king Old UI. Please close


Delay model update to stop flickering

Description
Right now when a recipe is done the machine models turns off for a tick. The update is now delayed a tick, making it not flicker if it found a new recipe in the next tick.
Closes #(issue)

Todo

[ ] Things that are yet to be completed for this PR to no longer be a draft.

Checklist:

[ ] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you).
[ ] I have performed a self-review of my own code.
[ ] I have commented my code in areas it may be challenging to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes are ready for review from a contributor.

This solution looks like it'll fix the problem you outlined -but- potentially the problem here is that we should not be wasting that tick? Maybe as a solution to the root problem, when a craft completes we should try and enqueue the new recipe immediately

We could yeah, though this is similar to what for example mekanism does, and user won't notice the 1 tick delay, so imo we don't need to make it more complex.

We could yeah, though this is similar to what for example mekanism does, and user won't notice the 1 tick delay, so imo we don't need to make it more complex.

Yeah that's fair - gonna merge this then, cheers Ferri!


add ios.forceRegister option

Description

ios.forceRegister

boolean

false

Optional. If `true` the app will register for remote notifications, even if the user has denied notification permissions. On iOS, notification permissions only control *user-facing* notifications – background pushes can still be received.

On iOS, notification permissions only control user-facing notifications. Silent/invisible notifications can be received and handled in the background even when the user denies notification permissions.
When forceRegister is true, registerForRemoteNotifications will be called on init even if requestAuthorizationWithOptions comes back with granted=FALSE.
Related Issue

https://github.com/havesource/cordova-plugin-push/issues/335
Motivation and Context

On iOS, notification permissions only control user-facing notifications. Silent/invisible notifications may be received and handled in the background even when the user denies notification permissions.
However this plugin does provide a way to register for remote notifications if the user has denied UserNotifications.

How Has This Been Tested?

In my app, I observe that without forceRegister, if the user denies notification permissions prior to PushNotification.init(), neither push.on('registration', ...) nor push.on('error', ...) are invoked.
When forceRegister = true, and the same scenario, push.on('registration', ...) is called and an APNS token is established.
When forceRegister = false or not defined, the behavior is the same as before.
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

What if the granted conditional wrappers were removed instead of introducing a new flag?
I'm trying to determine if there are viable use cases where an app developer might prefer not to support background notifications by default.
In the current setup, only users who accept disruptive permissions (e.g., sounds, badges, alerts) can receive normal and background notifications. However, since background notifications can be used to update the app in the background, why restrict this functionality based on the denial of disruptive permissions?

I'm trying to determine if there are viable use cases where an app developer might prefer not to support background notifications by default.

I can't think of any cases but I thought it best not to introduce any breaking changes, at least not without a discussion first.
Perhaps the strongest argument for keeping this behind a flag is that it represents a departure from Android. How important is it for this plugin to behave the same on both platforms by default?

How important is it for this plugin to behave the same on both platforms by default?

Ideally, both platforms should behave the same by default.
I think keeping this behind a flag is acceptable for now.
I believe Android can support the same process of registering and receiving background notifications, even if the user denies notification permissions. However, it might require some investigation and adjustments to make it work properly, as the current initialization process seems to return early if permissions are denied. Eventually, we can look into expanding support for Android as well. Once both platforms support the same functionality, we can reassess whether the flag is still needed in a future major release.
As for the flag’s description, how would the following update sound?

Optional. If set to true, the app will continue to register an APNs token for remote notifications. Notification permissions only control disruptive user-facing notifications, such as alerts, sounds, badges, and critical notifications. Even if the user denies these permissions, background push notifications can still be received.

Additionally, I’m considering adding this statement at the end:

If the user disables "Background App Refresh," silent push notifications will be blocked.

As for the flag’s description, how would the following update sound?

Optional. If set to true, the app will continue to register an APNs token for remote notifications. Notification permissions only control disruptive user-facing notifications, such as alerts, sounds, badges, and critical notifications. Even if the user denies these permissions, background push notifications can still be received.

Additionally, I’m considering adding this statement at the end:

If the user disables "Background App Refresh," silent push notifications will be blocked.

Sure, feel free to wordsmith that however you'd like


mdns error

https://github.com/ipfs/go-ipfs/issues/1807
https://github.com/ipfs/go-ipfs/issues/6359

The IPFS image needs to be pulled from master.

Closing as this repo always fetches with the latest go-ipfs release.


Unable to use Klarna Payment with discount code applied

Describe the bug
Our customers is unable to use Klarna Invoice Payments if a coupon code is applied.
With the coupon code applied the customers is not redirected to Klarna in the storefront and an error messages is logged in Log Center.
The error message logged is:
error processing payment. Error message: Cannot call method "divide" of null more details: TypeError: Cannot call method "divide" of null in int_adyen_overlay/cartridge/scripts/adyenGetOpenInvoiceData.js:68
To Reproduce
Steps to reproduce the behavior:

Apply a coupon with promotion class order
Select a Klarna Payment option
Payment is failing at the confirm order step
The error above is logged in the Log Center

Hi @soerensiegjensen,
Thanks for raising this issue.
This issue was addressed before in #424 , and will be fixed in the upcoming release of the cartridge.
Our release is under review by SalesForce right now and will be released when they approve it.
Regards,
Wouter
Adyen

Hi @wboereboom
Sorry for missing the duplicate of the issue.
Do you have an expected timeframe for when the new version will be released?
Best regards
Søren

No worries @soerensiegjensen,
I'd rather risk a duplicate issue than miss a major bug 😉 .
We're dependent on SalesForce to review the cartridge, so unfortunately it is out of our control at this point.
As soon as they approve we can make our release without any delay on our end.
As an intermediate solution you can copy the changes made in the PR I referred to in my previous message.
Regards,
Wouter
Adyen

Hi @wboereboom
Looks like the new release of the cartridge is a bigger upgrade. Would like to wait until after the holiday season.
But I really want this fix for Klarna implemented. Looks like the PR you linked to includes multiple updates.
Which of the changes should I implement to fix the Klarna issue?
Best regards
Søren

Hi @soerensiegjensen,
I'm not sure which version of the cartridge you are on exactly, but I understand the upgrade is a hard one to do, especially with the holiday season around the corner.
The errors you are getting are related to the LineItemHelper class as far as I am able to tell.
the changes in src/cartridges/int_adyen_overlay/cartridge/scripts/util/lineItemHelper.js in the PR I linked should help you out.
This change always ensure a value is returned as opposed to null when retrieving Item and VAT amounts, which in turn caused the errors when calling the divide function in the adyenGetOpenInvoiceData  class.
Don't hesitate to reach out when things remain unclear.
Regards,
Wouter,
Adyen

Hi @wboereboom
Thank you very much for quick and useful help. It is very much appreciated.
We are currently running on v20.1.3. I have seen the release notes for the two new releases, but do you have some documentation or guidelines regarding upgrading to the new version? Anything I need to be aware of when doing the upgrade?
For now I have updated lineItemHelper.js with the 3 changes from the PR you linked. It looks like it is working. I am able to continue to the Klarna redirect page now and I can see the discount is applied on the Klarna summary as well.
Thank you very much for the help.
Best regards
Søren

Hi @soerensiegjensen,
We are currently working to upgrade our docs with more information on how to upgrade.
But basically:

make sure to reimport the metadata so all system objects and services get updated
try to find out what customizations you made, so you can predict which of them might break. On top of that; always try to make customizations in a separate cartridge so you can turn them on/off, and to make sure you can easily update them too once you update the Adyen cartridge.

Regards,
Wouter
Adyen


[KeyVault] API feedback from review and UX study (post Preview 5)

Work items that come out of the API review for Certificates and the most recent UX study for KV. Post Preview 5.
API updates

[x] Change GetCertificate* methods as follows:

GetCertificateWithPolicy -> GetCertificate : CertificateWithPolicy
GetCertificate -> GetCertificateVersion : Certificate
GetCertificateVersions -> GetCertificateVersions : CertificateProperties (no change)

~[ ] Add constructor for CertificatePolicy to require subject, issuerName~
~[ ] Add constructor for CertificatePolicy to require subjectAlternativeNames, issuerName~
[x] Add CertificatePolicy.Default member

Should just return the same values as CertificateClient.DefaultPolicy in .NET
Change that (or make new):

Subject = CN=DefaultPolicy
IssuerName = Self
Nothing else

[x] Remove overload of StartCreateCertificate where CertPolicy is optional
[ ] There are a bunch of generic classes we need to rename:

~Action -> CertificatePolicyAction~
[x] Contact -> CertificateContact
~Error -> CertificateError~
[x] Issuer -> CertificateIssuer

~[ ] Certificate.CER -> Certificate.Cer~
[x] Create CertificateWithPolicy class
[x] Get/List Deleted Certificates defines includePending parameter and sends to server
[x] Remove GetPendingCertificateSigningRequest method from API
[x] Use Start/Begin prefix to CreateCertificate

Doc updates

[ ] [Doc] Update doc for CreateCertificate(string) to mention, "creates a self-signed certificate"
[ ] [Doc] Document 'like "Self" for self-signed certificates or "Unknown" for certificate signing request'

Folding into #5972


[2.0.0 REVIEW] Use JSON Schema Draft 07 as the default schema

After playing a lot these days with AsyncAPI 2.0.0-rc1, I realized most of the pain comes from having the OpenAPI schema as the default. We want to keep supporting OpenAPI schemas but there's no need to make it the default choice.
There's only one thing that worries me and it's the lack of a deprecation property in Draft 07. However, @philsturgeon is working on it and will probably make it for Draft 08. As per discriminator and externalDocs, they can just be extensions in the form of x-discriminator and x-external-docs. Same for deprecated in the meantime, it can be done using x-deprecated.
@handrews do you have a list of changes (or potential changes) for Draft 08? I've seen this but I'd like to know if upgrading the AsyncAPI default schema from Draft 07 to Draft 08 will be a breaking change. Or put it another way, are there breaking changes in Draft 08 compared to Draft 07?

deprecated got merged (albeit a bit hastily) into draft 08 so you're all set.

Awesome!! 🎉 And congrats for your first contribution on JSON Schema, Phil.

@philsturgeon I'm going to (almost) follow your suggestion here: https://github.com/OAI/OpenAPI-Specification/pull/1977.
However, and since tooling is not yet ready for Draft-08, AsyncAPI is going to adopt a superset of Draft-07 as the default schema, and the parser will provide a "translator" from current OpenAPI schemas.
Thanks for the work you did there. It helped clarify the current scenario.


eth_getStorage successfully parses index that is too long

Describe the bug
The eth_getStorage/get-invalid-key-too-large test in the rpc-compat test suite currently fails:
>>  {"jsonrpc":"2.0","id":12,"method":"eth_getStorageAt","params":["0xaa00000000000000000000000000000000000000","0x00000000000000000000000000000000000000000000000000000000000000000","latest"]}
<<  {"jsonrpc":"2.0","result":"0x0000000000000000000000000000000000000000000000000000000000000000","id":12}
response differs from expected:
 {
   "id": 12,
   "jsonrpc": "2.0",
-  "result": "0x0000000000000000000000000000000000000000000000000000000000000000"
+  "error": {
+    "code": -32000,
+    "message": "hex string of odd length"
+  }
 }

This is because we successfully parse 0x00000000000000000000000000000000000000000000000000000000000000000, which is 67 characters long, or 65 characters excluding the 0x. Instead, this should be disallowed in U256 deserialization, and we should return the error hex string of odd length with code -32000.
Steps to reproduce
Run the steps in #851, building the docker image from the branch dan/hive-ci.
The command to run the eth_getStorageAt/get-invalid-key-too-large test:
./hive --client reth --sim ethereum/rpc-compat --sim.limit "/eth_getStorage/get-storage-invalid-key-too-large"

Node logs
No response
Platform(s)
No response
What version/commit are you on?
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

this is actually a U256::deserialize bug, right?

this is actually a U256::deserialize bug, right?

Yeah, will open an issue and track there

fixed in https://github.com/paradigmxyz/uint/pull/3

https://github.com/recmo/uint/issues/389


variable autocomplete does not show variables without quotes

When autocompleting a variable, the function name will autocomplete with:
variables()
and the cursor is between the parens...  If you start typing, the list of variables does not appear, but a generic list.  You have to type a single quote before you get the variable list...  Seems like it would make more sense to show the variable list without requiring the single quote? That will be the common case, generic list is probably never helpful here.

Shipped yesterday


ts-expect-error suppresses wrong error

Bug Report

🔎 Search Terms
ts-ignore ts-expect-error suppresses wrong error

🕗 Version & Regression Information
4.3.0-dev-20210415
⏯ Playground Link

Playground link with relevant code
💻 Code

type Foo<Str extends string, Num extends number> = never

declare const foo: Foo<
    'foo',
    'bar' //correctly errors
>

declare const bar: Foo<
    1, //correctly errors
    'bar' //doesn't throw, presumably because there was already an error so type checker didn't bother checking the rest
>

declare const baz: Foo<
    //@ts-expect-error
    1,
    'bar' //no error, even though there was an error above that was suppressed
>

🙁 Actual behavior
no error on incorrect type that's not suppressed

🙂 Expected behavior
compiler error on incorrect type that isn't being suppressed by ts-expect-error

ts-expect-error doesn't change what errors get issued and what don't. It's more akin to a text-based filter applied at the very end of the process.


Final newline stripped from templated files

If a template file ends with a new line voodoo will strip that new line when outputting the rendered template. This causes flake8 to complain about W292 no newline at end of file for all templated python files. New lines at the end of a template file should be preserved by voodoo.

If keep_trailing_newline=True is passed to the Environment, this will stop happening. I'll make a pull request that adds this to the DEFAULT_ENV_OPTIONS for voodoo.


Invalid publicPath for js files (duplicated nested dir name)

Laravel Mix Version: 6.0.31
Node Version: 12.16.1
NPM Version: 6.13.4
OS: Windows 10 (21H1 19043.1202) in Italian

Let's assume this project structure
├── build
│   ├── src
│   │   └── test.js
│   ├── node_modules
│   │   └── ...
│   ├── package-lock.json
│   ├── package.json
│   └── webpack.mix.js
└── web

Here's the content of the relevant files
// build/package.json
{
  "name": "test",
  "version": "0.0.1",
  "devDependencies": {
    "laravel-mix": "^6.0.31"
  }
}

// build/webpack.mix.js
const mix = require('laravel-mix');

mix.options({
    publicPath: '../web',
});

mix.js('src/test.js', 'js');

When I run npx mix I'd expect this structure:
├── build
│   └── ...
└── web
    ├── mix-manifest.json
    └── js
        └── test.js

But instead I have this:
├── build
│   └── ...
└── web
    ├── mix-manifest.json
    └── web
        └── js
            └── test.js

(Please remark that the generated js file is placed in the ../web/web/js directory, instead of in ../web/js).
PS: CSS assets are correctly saved to ../web/css; only js files are saved to ../web/web/js
PPS: I also tried to use an absolute path when configuring publicPath (setting it to `${path.dirname(__dirname)}/web`), with the same results.

I'm not surprised honestly. Relative paths outside of the "root" can be problematic. Thanks for bringing this up. I'll take a look this weekend to see what we can do here.


Swift Lint

Description

Added swift lint and updated Swift syntax to comply to RW guide style.
Main changes to existing code:

Favour type inference
Remove use of self when not needed
Remove trailing white spaces
Removing parenthesis from if statements

Status

[ ] Code review

Current coverage is 99.20% (diff: 99.08%)

Merging #22 into master will increase coverage by 1.92%

@@             master        #22   diff @@
==========================================
  Files             4          4
  Lines           514        503    -11
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            500        499     -1
+ Misses           14          4    -10
  Partials          0          0

Powered by Codecov. Last update b6e47fa...a47621e


Update licenses

What is this PR for?
Update outdated license version
What type of PR is it?
License

LGTM

Thanks for catching it, looks great to me

LGTM

LGTM

Merging it into master


Displays wrongly in Safari

The code samples seem to render OK after 1f13c06 - seems to be caused by -prefix-free not kicking in.
Gradients still don't display even though they're generated and the SVG is present in the DOM...

Yup, it’s in my "to look into asap" list. Thanks for reporting!

FWIW, gradients work in WebKit version r185802

FWIW, the root of the issue is that this doesn’t work: http://dabblet.com/gist/0d5018f6c81484458472

Looks like that dabblet is working in Safari as of at least version 9.0 (11601.1.56)
Works for me.

+1 working in Safari 9.0

Yep, but not in lower.

Woohoo!! Closing this then!


docs: cleanup 03_celery.md

Fixes a typo with the import and variable names
Adds a missing url

Thanks for the tutorial!

Just finished applying the suggestion. And no problem.

Great! I will merge it as is. Next time, I would rather you fix the commit instead of adding a new one on top (with fixup commit+squash or interactive rebase). This makes the git history clearer :)


Alias DURATION for INTERVAL

In Neoj4J/Cypher/Memgraph the data type is called DURATION, so add this as alias for INTERVAL (or the other way round).

Added in #3284


[curl] fix idn2 dependency

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

When you install curl[tool,idn2] with vcpkg, and the you do:
curl --version

You can see the lib that it install with, and even you install the feature idn2, it missing. I fix it.
It should look similar to that:
curl 8.7.0-DEV (Linux) libcurl/8.7.0-DEV ... libidn2/2.3.4  ...

Also may I ask why you disable LIBPSL?
Curl on ubuntu it ship enable. Can we add it as a feature (off by default)?
https://github.com/microsoft/vcpkg/blob/master/ports/curl/portfile.cmake#L86

Also may I ask why you disable LIBPSL? Curl on ubuntu it ship enable. Can we add it as a feature (off by default)? https://github.com/microsoft/vcpkg/blob/master/ports/curl/portfile.cmake#L86

Looking at the upstream code, you can set libspl as a feature. It is turned off by default and can be turned on if necessary.

Also may I ask why you disable LIBPSL? Curl on ubuntu it ship enable. Can we add it as a feature (off by default)? https://github.com/microsoft/vcpkg/blob/master/ports/curl/portfile.cmake#L86

Looking at the upstream code, you can set libspl as a feature. It is turned off by default and can be turned on if necessary.

Maybe next PR

@jimwang118 It actually fixing port bug.
Can you change the correct label?

Usage test pass with following triplets:
x64-windows


Proposal to add Mo Khan as an associate member

In the spirit of adding people with three-letter github handles, I propose adding Mo Khan (@enj) as an associate member of the Product Security Committee. Mo and I worked together at Red Hat, he is deeply familiar with Kubernetes security controls and architecture, currently serves as a chair of sig-auth, and is very involved as a reviewer and contributor.
/cc @cjcullen @joelsmith @lukehinds @micahhausler @tallclair

LGTM Does this mean I can page Mo for code reviews?

/lgtm

/lgtm

/approve

Looking forward to getting started 👍


New-bccontainer (18.x) fails when using option new DB

When creating a container based on BC 18.x using option new DB it fails when trying to install the system application.
See sample script below:
$BCArtifactURL="https://bcartifacts.azureedge.net/sandbox/18.0.23013.24722/nl"
$DockerImageName="local"
$BuildCredential=New-Object System.Management.Automation.PSCredential ('navuser', (ConvertTo-SecureString 'Welkom01!' -AsPlainText -Force))
$LicenseFileName="C:\Projects\License\dev-bc.flf"

$AdditionalParameters = @()
# Disable feature until is has been fixed. Bug in BC when using QueryCategory.
$AdditionalParameters += ("-e customNavSettings=FeatureSwitchOverrides=disableSmartListActions")
$additionalParameters += ("-e customNavSettings=UsePermissionSetsFromExtensions=false")

$UseNewDatabase=$true
$TablesToCopy = @()
if ($UseNewDatabase) {
    $TablesToCopy = @("Add-in", "Permission", "Permission Set", "Page Documentation", "Web Service")
}
$ContainerName="Test18x"

New-NavContainer `
    -imageName $DockerImageName `
    -artifactUrl $BCArtifactURL `
    -containerName $ContainerName `
    -auth 'NavUserPassword' `
    -bakFile $BackupFilePath `
    -Credential $BuildCredential `
    -licenseFile $LicenseFileName `
    -memoryLimit '20G' `
    -additionalParameters $AdditionalParameters `
    -accept_eula `
    -accept_outdated `
    -updateHosts `
    -includeAL `
    -doNotExportObjectsToText:(-not $ExportObjectsToText) `
    -includeTestToolkit:$ImportTestToolkit `
    -includeTestLibrariesOnly:$ImportTestToolkit `
    -alwaysPull `
    -EnableTaskScheduler:(!$DisableTaskScheduler) `
    -multitenant:$MultiTenant `
    -useNewDatabase:$UseNewDatabase `
    -copyTables $TablesToCopy `
    -doNotCheckHealth

Error message at the end...":
Publishing C:\ProgramData\BcContainerHelper\Extensions\Test18x\4e6dc48c-2a93-4944-87d9-ff24767d9277\System.app
This Extension cannot be published as it has the same Publisher, Name, and Version as a previously published Extension.
at , : line 18
at Invoke-ScriptInBcContainer, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\Invoke-ScriptInNavContainer.ps1: line 44
at , C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\AppHandling\Publish-NavContainerApp.ps1: line 257
at Publish-BcContainerApp, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\AppHandling\Publish-NavContainerApp.ps1: line 117
at Clean-BcContainerDatabase, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\AppHandling\Clean-BcContainerDatabase.ps1: line 177
at New-BcContainer, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\New-NavContainer.ps1: line 2033
at , : line 18
This Extension cannot be published as it has the same Publisher, Name, and Version as a previously published Extension.
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\Invoke-ScriptInNavContainer.ps1:45 char:13

        throw $_.Exception.Message

        ~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (This Extension ...shed Extension.:String) [], RuntimeException
FullyQualifiedErrorId : This Extension cannot be published as it has the same Publisher, Name, and Version as a previously published Extension.

Adding complete logfile...
PS C:\Windows\system32> C:\Projects\docker\Test18x.ps1
BcContainerHelper is version 2.0.11
BcContainerHelper is running as administrator
Host is Microsoft Windows 10 Enterprise - 20H2
Docker Client Version is 20.10.5
Docker Server Version is 20.10.5
Removing container Test18x
Removing Test18x from container hosts file
Removing Test18x-* from container hosts file
Removing C:\ProgramData\BcContainerHelper\Extensions\Test18x
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
Image local:sandbox-18.0.23013.24722-nl already exists
Using image local:sandbox-18.0.23013.24722-nl
Disabling Health Check (always report healthy)
Creating Container Test18x
Version: 18.0.23013.24722-NL
Style: sandbox
Multitenant: No
Platform: 18.0.22893.24704
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.19042.928 (20H2)
Host OS Version: 10.0.19042.928 (20H2)
Using process isolation
Using locale nl-NL
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file C:\Projects\License\dev-bc.flf
Additional Parameters:
-e customNavSettings=FeatureSwitchOverrides=disableSmartListActions,EnableTaskScheduler=True
-e customNavSettings=UsePermissionSetsFromExtensions=false,EnableTaskScheduler=True
Files in C:\ProgramData\BcContainerHelper\Extensions\Test18x\my:

AdditionalOutput.ps1
license.flf
MainLoop.ps1
SetupVariables.ps1
updatehosts.ps1
Creating container Test18x from image local:sandbox-18.0.23013.24722-nl
05f20e23775aeff4ff8773b38a5d3669c5b7863b54076bf3a66961f1ccec435c
Waiting for container Test18x to be ready
Initializing...
Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is Test18x
PublicDnsName is Test18x
Using NavUserPassword Authentication
Starting Local SQL Server
Starting Internet Information Server
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 11E7DA84823EC2181E54C2CE088175997C023B44
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting UsePermissionSetsFromExtensions to false
Setting EnableTaskScheduler to True
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Enabling Financials User Experience
Using license file 'c:\run\my\license.flf'
Import License
Creating http download site
Setting SA Password and enabling SA
Creating navuser as SQL User and add to sysadmin
Creating SUPER user
Container IP Address: <IP_ADDRESS>
Container Hostname  : Test18x
Container Dns Name  : Test18x
Web Client          : http://Test18x/BC/
Dev. Server         : http://Test18x
Dev. ServerInstance : BC
Setting Test18x to <IP_ADDRESS> in host hosts file

Files:
http://Test18x:8080/ALLanguage.vsix
Container Total Physical Memory is 63.8Gb
Container Free Physical Memory is 40.0Gb
Initialization took 128 seconds
Ready for connections!
Reading CustomSettings.config from Test18x
Creating Desktop Shortcuts for Test18x
Creating .net Assembly Reference Folder for VS Code
Copying DLLs from C:\Windows\assembly to assemblyProbingPath
Copying DLLs from C:\Windows\Microsoft.NET\assembly to assemblyProbingPath
Copying DLLs from C:\Program Files\Microsoft Dynamics NAV\180\Service to assemblyProbingPath
Copying DLLs from C:\Program Files (x86)\Microsoft Dynamics NAV\180\RoleTailored Client to assemblyProbingPath
Copying DLLs from C:\Test Assemblies\Mock Assemblies to assemblyProbingPath
Copying DLLs from C:\Program Files (x86)\Open XML SDK to assemblyProbingPath
Stopping ServiceTier in order to replace database
Copying Database on localhost\SQLEXPRESS from CRONUS to mytempdb
Taking database CRONUS offline
Copying database files
Attaching files as new Database mytempdb
Putting database CRONUS back online
Removing Database CRONUS from localhost\SQLEXPRESS
Creating new database CRONUS on localhost\SQLEXPRESS with Collation Latin1_General_100_CS_AS
Copying table [Add-in] from original database
Copying table [Permission] from original database
Copying table [Permission Set] from original database
Copying table [Page Documentation] from original database
Copying table [Web Service] from original database
Copying table [Entitlement] from original database
Copying table [Entitlement Set] from original database
Copying table [Membership Entitlement] from original database
Removing Database mytempdb from localhost\SQLEXPRESS
Starting Service Tier
Synchronizing
Importing license file
Importing License C:\ProgramData\BcContainerHelper\Extensions\Test18x\my\license.flf
Creating user navuser
Creating User navuser
Assigning Permission Set SUPER to navuser
Publishing System Symbols
Publishing C:\ProgramData\BcContainerHelper\Extensions\Test18x\6461b0bd-00e2-43b4-b7b3-768542d1b979\System.app
This Extension cannot be published as it has the same Publisher, Name, and Version as a previously published Extension.
at , : line 18
at Invoke-ScriptInBcContainer, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\Invoke-ScriptInNavContainer.ps1: line 44
at , C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\AppHandling\Publish-NavContainerApp.ps1: line 257
at Publish-BcContainerApp, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\AppHandling\Publish-NavContainerApp.ps1: line 117
at Clean-BcContainerDatabase, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\AppHandling\Clean-BcContainerDatabase.ps1: line 177
at New-BcContainer, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\New-NavContainer.ps1: line 2033
at , C:\Projects\docker\Test18x.ps1: line 18
This Extension cannot be published as it has the same Publisher, Name, and Version as a previously published Extension.
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\Invoke-ScriptInNavContainer.ps1:45 char:13

        throw $_.Exception.Message

        ~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (This Extension ...shed Extension.:String) [], RuntimeException
FullyQualifiedErrorId : This Extension cannot be published as it has the same Publisher, Name, and Version as a previously published Extension.


[Question] Cache the result of the get() function?

Sorry for the basic question from a beginner, but I'd like to know about the price of calling registry.get<T>(). I have a Entity class like this:
class Entity {
  private:
    std::string name = "";
    entt::entity id = entt::null;
    entt::registry* registry;

  public:
    Entity(const std::string& name, entt::entity id, entt::registry* registry);
    ~Entity() {}

    template<typename T>
    T& GetComponent() {
        CORE_ASERT(registry->all_of<T>(id), "Component T not found in {0}!", name);
        return registry->get<T>(id);
    }
}

Since I come from Unity, whose GetComponent() method is known to be slow and not supposed to be invoked every frame, I'm thinking about caching the result of registry->get<T>() so that subsequent calls won't need to query the registry again. While caching is easy outside the class (on caller's side), it's a bit difficult to cache inside the class with generics (I'm still new to C++), and I wonder if this is at all necessary......
Can you tell me a bit more about the get() function? Does it check registry->all_of<T>() automatically? Is it costly? Also, I don't understand why the documentation says get_or_emplace() is slightly faster. Thank you~!

The underlying pool in EnTT is based on sparse sets and getting an element is a O(1) lookup.
Caching would be still faster, granted, though pointers aren't stable unless you enable pointer stability on all pools (available since 3.8.x). Also, I confirm that get uses all_of (well, sort of actually, but let's assume this is the case) under the hood to assert in case the entity doesn't own the component.
get_or_emplace is faster than the following:
if(registry.all_of<T>(e)) {
    return registry.get<T>(e);
} else {
    return registry.emplace<T>(e);
}

This is due to the fact that the pool lookup takes place only once. Probably not something you'll ever be able to measure but technically speaking it's slightly faster.

Thanks for clearing things up. Glad to hear that it's a O(1) operation, that's amazing. I think I won't cache then since the performance gain is negligible, but easy stability is definitely my first concern.
Thanks for sharing such a great open source library. I hope I could get a better understanding of it by reading your ECS back and forth series, cheers! 😄

You're welcome. Feel free to join gitter or discord if you have any question. Usually easier than chatting on GH. Also, you can find many other devs there (by far more skilled than me) that can help you. 😉


Incorrect parsing for URLs with parenthesis in them

Given a file that includes:
// https://en.wikipedia.org/wiki/ANSI_escape_code#SGR_(Select_Graphic_Rendition)_parameters

I would expect https://en.wikipedia.org/wiki/ANSI_escape_code#SGR_(Select_Graphic_Rendition)_parameters to be parsed as URL. However, when I inspect the parse tree, I see that it says:
(comment ; [28, 0] - [28, 91]
  (source ; [28, 0] - [28, 91]
    (uri))) ; [28, 3] - [28, 79]

it considers it as URL up to, but not including, the closing parenthesis: https://en.wikipedia.org/wiki/ANSI_escape_code#SGR_(Select_Graphic_Rendition.

Hi, this is mostly because ) is considered a stop words like (URL)wods.. or (URL). words... But I'll see if the rules can be relaxed.

As per the URL RFC spec:

Thus, only alphanumerics, the special characters "$-_.+!*'(),", and reserved characters used for their reserved purposes may be used unencoded within a URL.

that URL is perfectly legal. Thank you for looking into this :-)


Python3 Support?

I haven't tried using NSGT in Python3, but would you be open to pull requests for any necessary changes, if any, to make it Py3 compatible?

Definitely, yes! Thanks for making an effort.

Am 16.11.2015 um 21:19 schrieb Jian Liu<EMAIL_ADDRESS>I haven't tried using NSGT in Python3, but would you be open to pull requests for any necessary changes, if any, to make it Py3 compatible?
—
Reply to this email directly or view it on GitHub https://github.com/grrrr/nsgt/issues/5.

Status?

@carlthome
I haven't had a chance to do this yet

@carlthome
I haven't had a chance to do this yet

Alright, mind if I do a pr?

Great, please go ahead!

https://github.com/grrrr/nsgt/pull/13

Fixed with commit 6ea2240687c85b9633fdf08600feaf381c784c99


split d_attention

split 4663 lines of assembly in d_attention.s using my tool from #39

fixed some unrecognized functions thanks to @lepelog's map file tweaks


can : move bytes2dlc/dlc2bytes to common header file

only keep can_bytes2dlc & can_dlc2bytes
Note: Please adhere to Contributing Guidelines.
Summary
Update this section with information on why change is necessary,
what it exactly does and how, if new feature shows up, provide
references (dependencies, similar problems and solutions), etc.
Impact
Update this section, where applicable, on how change affects users,
build process, hardware, documentation, security, compatibility, etc.
Testing
Update this section with details on how did you verify the change,
what Host was used for build (OS, CPU, compiler, ..), what Target was
used for verification (arch, board:config, ..), etc. Providing build
and runtime logs from before and after change is highly appreciated.

[Experimental Bot, please feedback here]
No, this PR does not fully meet the NuttX requirements. While it mentions keeping can_bytes2dlc and can_dlc2bytes, it lacks crucial information.
Specifically, it's missing:

Clear Explanation of the Change: The summary doesn't explain why only these two functions are being kept. What is being removed? What problem does this solve? What's the motivation?
Detailed Impact Assessment: The impact section is completely empty.  This needs to be filled out describing the effects on users, build, hardware, documentation, security, and compatibility.  Removing code almost certainly has some impact.
Testing Information: The testing section is also empty.  What platforms were used for testing? What were the results?  "Works as intended" is not sufficient; concrete examples of tests and their outcomes are needed.  The logs sections are empty.

To make this PR acceptable, the author needs to:

Explain the "why" behind the change. Provide a clear and concise explanation of the reason for removing the other functions.  What problem did they cause?  What are the benefits of removing them?
Complete the Impact Assessment.  Consider all aspects listed in the template (users, build, hardware, docs, security, compatibility) and explain how this change affects each. Even if the answer is "NO", it should be stated explicitly.
Provide detailed testing information.  List the build host(s) and target(s) used for verification.  Include the actual testing logs from before and after the change to demonstrate the effect of the modification.

Only then will the PR meet the NuttX requirements.

why not put common code in a separate header like include/nuttx/can_cmn.h. We shouldn't couple CAN chardev headers with SocketCAN headers. These are two separate implementations

'include nuttx/can_cmn.h' is ok

@xucheng5 I think it should be flexible enough to let some less common usage scenarios: i.e.: it could be possible to use the internal CAN peripheral from a MCU with SocketCAN and an external MCP2515 with CAN char dev. Using global variables shared by both will introduce issues to this case of use.

@xucheng5 I think it should be flexible enough to let some less common usage scenarios: i.e.: it could be possible to use the internal CAN peripheral from a MCU with SocketCAN and an external MCP2515 with CAN char dev. Using global variables shared by both will introduce issues to this case of use. I think your modifications are fine, but we need to test these usage case to confirm everything still working fine.

The conversion of DLC into byte length/ byte length into DLC  does not care about socketcan or character-can, which is the processing of CAN protocol.


Pythia v.s. GPT-neox

Hi, thanks for the awesome work.
I notice there are Pythia models and GPT-neox models on Eleuther's HF hub, both seem to be very recent. Wonder what are the the main differences between them, and if there is any preference on one of them.
Thank you.

Hi,
The Pythia models and GPT-NeoX-20b model were all trained in the same library, and use a very similar architecture.
The Pythia models are primarily intended for research as a consistent set of models. If you’re only looking for performance, then the GPT-J-6B and GPT-NeoX-20b models have been trained on more data and will outperform Pythia models, though the Pythia models should have standard performance for models of their size!

Thank you for the clarification!


fix(client): handle normal tests in python runner

Normal in the sense that they just invoke some assertions and do not
return a { test: () => {...} } object.
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Hey Oliver, I'm not sure if this work as expected.
My intention was to be able to pass the test with code that would normally throw an error, like this:
a = [1, 2, 3]
for i in a:

If i have a test checking the a value, it's passing until I write the for loop. Then, it's going fail if the loop does not have a body. I think we can discuss about that but for now it's not a priority.

Ok, my mistake. It seems to work 😅

If i have a test checking the a value, it's passing until I write the for loop. Then, it's going fail if the loop does not have a body. I think we can discuss about that but for now it's not a priority.

That's not something we can easily achieve, because the a value doesn't exist since the code as a whole cannot be run. In principle we might be able to parse the python without running it, get the AST and inspect that. However, my brief experimentation suggests that they won't help much, beyond telling us that there's an error.
What you can do is analyse the source strings. If you're expecting the learner declares a variable a with an array value, you could do this
assert(code.match('a = [1, 2, 3]'))

or whatever string analysis seems appropriate.
Either that or break the steps down further, so that a previous step can check that a is correct and then subsequent steps can ignore it.

One thing I don't like is that in this way we are suppressing every error and the camper might make other mistakes in the step. Maybe the best thing is to leave the things as they were before and try other approaches. Like you said, breaking steps down further, or turning something temporarily into a comment to avoid getting errors.

That's a tradeoff, for sure. If you want syntactically invalid python to pass the tests, it's quite difficult to be confident that the expected error is the only error. That said, I think it's fine to have these kinds of lessons and the test runner shouldn't fail when it encounters tests that don't run the python.
It was a bug and the test runner would fail even if the python was syntactically valid.


IndexError: index 99 is out of bounds for dimension 1 with size 1

作者您好，冒昧的打扰您，感谢您对代码的分享。我最近在复现您给的代码的时候遇到了这样一个错误，不知道如何解决，希望您能指点一二。本人小白一枚，我认为错误的行在index = np.argwhere(true_disp>0)，但是不知道如何更改，谢谢您！（我是这样训练的：用Scene Flow模型运行main进行训练，然后加载训练结果，用KT2015运行finetune进行微调）

已解决。

你好遇到了同样的问题，请问是如何解决的呢？

@pridexin https://github.com/JiaRenChang/PSMNet/issues/217#issue-925628747

@pridexin #217 (comment)

谢谢！我好像遇到了和您当初一样的问题，请问您cpu溢出的问题是怎么解决的呀

我暂时也没找到解决方法....你有什么好的办法不？

我暂时也没找到解决方法....你有什么好的办法不？

还在研究当中，有方案了告诉你

朋友，你有微信吗

可以加好友交流一下。你把你的微信发我吧
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2021年6月28日(星期一) 中午11:23
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
朋友，你有微信吗
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

你有邮箱吗
发自我的iPhone
------------------ Original ------------------
From: pridexin @.>
Date: Mon,Jun 28,2021 0:34 PM
To: JiaRenChang/PSMNet @.>
Cc: Jack47746 @.>, State change @.>
Subject: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)

我关注了你的b站账号，你看下私信
------------------ 原始邮件 ------------------
发件人:                                                                                "JiaRenChang/PSMNet"                                                                                @.>;
发送时间: 2021年6月28日(星期一) 中午12:43
@.>;
<EMAIL_ADDRESS>主题: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
你有邮箱吗
发自我的iPhone
------------------ Original ------------------
From: pridexin @.>
Date: Mon,Jun 28,2021 0:34 PM
To: JiaRenChang/PSMNet @.>
Cc: Jack47746 @.>, State change @.>
Subject: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

你把你的微信发我邮箱吧，b站我没看到..
发自我的iPhone
------------------ Original ------------------
From: pridexin @.>
Date: Mon,Jun 28,2021 3:52 PM
To: JiaRenChang/PSMNet @.>
Cc: Jack47746 @.>, State change @.>
Subject: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
我关注了你的b站账号，你看下私信
------------------ 原始邮件 ------------------
发件人:                                                                                "JiaRenChang/PSMNet"                                                                                @.>;
发送时间: 2021年6月28日(星期一) 中午12:43
@.>;
@.@.>;
主题: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
你有邮箱吗
发自我的iPhone
------------------ Original ------------------
From: pridexin @.&gt;
Date: Mon,Jun 28,2021 0:34 PM
To: JiaRenChang/PSMNet @.&gt;
Cc: Jack47746 @.&gt;, State change @.&gt;
Subject: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

好的那麻烦你加我一下<PHONE_NUMBER>3
------------------ 原始邮件 ------------------
发件人:                                                                                "JiaRenChang/PSMNet"                                                                                @.>;
发送时间: 2021年6月28日(星期一) 下午3:56
@.>;
<EMAIL_ADDRESS>主题: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
你把你的微信发我邮箱吧，b站我没看到..
发自我的iPhone
------------------ Original ------------------
From: pridexin @.>
Date: Mon,Jun 28,2021 3:52 PM
To: JiaRenChang/PSMNet @.>
Cc: Jack47746 @.>, State change @.>
Subject: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
我关注了你的b站账号，你看下私信
------------------&nbsp;原始邮件&nbsp;------------------
发件人:                                                                                "JiaRenChang/PSMNet"                                                                                @.&gt;;
发送时间:&nbsp;2021年6月28日(星期一) 中午12:43
@.&gt;;
@.@.&gt;;
主题:&nbsp;Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
你有邮箱吗
发自我的iPhone
------------------ Original ------------------
From: pridexin @.&amp;gt;
Date: Mon,Jun 28,2021 0:34 PM
To: JiaRenChang/PSMNet @.&amp;gt;
Cc: Jack47746 @.&amp;gt;, State change @.&amp;gt;
Subject: Re: [JiaRenChang/PSMNet] IndexError: index 99 is out of bounds for dimension 1 with size 1 (#218)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@Jack47746 @pridexin
The error of the problem is still the dimension of true_disp and pred_disp, where I put the ‘:’ in the wrong place. true_disp has dimension (1, 368, 1232), while pred_disp has (1,1, 368, 1232). It hence should be corrected as the pred_disp**[:, index[0][:], index[1][:], index[2][:]]**.

@pridexin #217 (comment)

谢谢！我好像遇到了和您当初一样的问题，请问您cpu溢出的问题是怎么解决的呀

请问cpu溢出问题解决了吗？

@Jack47746 @pridexin The error of the problem is still the dimension of true_disp and pred_disp, where I put the ‘:’ in the wrong place. true_disp has dimension (1, 368, 1232), while pred_disp has (1,1, 368, 1232). It hence should be corrected as the pred_disp[:, index[0][:], index[1][:], index[2][:]]**** when calculating disp_true.

Hello, I have an question about the code here. In my opinion, Index[0][:] is a list which contains 4 elements, so I don't understand why this 'disp_true[index[0][:], index[1][:], index[2][:]]' can run.

@Jack47746 @pridexin The error of the problem is still the dimension of true_disp and pred_disp, where I put the ‘:’ in the wrong place. true_disp has dimension (1, 368, 1232), while pred_disp has (1,1, 368, 1232). It hence should be corrected as the pred_disp[:, index[0][:], index[1][:], index[2][:]]**** when calculating disp_true.

Thank you so much!!! I read all your comments in #218 #217 and #216, and solved my problem by setting batch_size to 1, and correcting the calculating expression of "pred_disp". I used NVIDIA 2080TI.


[Bug] window.print() not loaded for estimates print

When printing invoices, the following code is executed:   <script type="text/javascript">window.onload = function() { window.print(); }</script>, prompting the browser print dialog.
However, this is not the case for estimates.

good catch, thanks!


Pass file to renaming function

Rename function could decide the new name based on the old name only.
By passing it also the file object, it can take into account data
extracted earlier in the pipeline (e.g. file.frontMatter, file.data,
...) or raw file contents. Example of renaming a file to a slug of the
title set in YAML front matter:
gulp.task('generate_html', function() {
    return gulp.src('articles/*.md')
        .pipe(frontMatter({
            property: 'frontMatter',
            remove: true
        }))
        .pipe(rename(function(path, file) {
            path.basename = slug(file.frontMatter.title).toLowerCase();
        }))
        .pipe(markdown2html())
        .pipe(gulp.dest('html/'));
}

https://github.com/hparra/gulp-rename/pull/19#issuecomment-87421303


Introduce table hierarchical datatype

Pull Request
🤨 Rationale

#861

This is a staging PR in that it will lay the type groundwork that we can ultimately leverage for representing hierarchical data in the table. There are no functional changes in this PR.
👩💻 Implementation
Creating a new datatype to represent hierarchical data, which Tanstack requires in order to visualize hierarchical data properly. This type is used for the internal representation of the table data, and thus does not affect our public facing datatype used in the setData method. This new datatype basically wraps each record in another object, and also provides another field typed as an array of the new datatype (providing hierarchy).
🧪 Testing
Ensured existing tests pass.
✅ Checklist

[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.

@rajsite @mollykreis I told @atmgrifter00 we can bypass your final review on this one since you're OOO and there will be opportunities for more feedback on the follow up PRs


Unit tests: Are there any?

I don't see any unit tests in the solution. Is there another solution containing tests, or is this something yet to be done?

We have tests in our main Trickster Cards repo. This is a great task for us to do. I've changed the title a bit.

Quick update: we've made some initial progress here.

Set up a new test project for this repo in https://github.com/TricksterCards/TricksterBots/commit/aec8331e8cf8ab7a78608a976c096637999a44c2 (including a few basic Spades tests).
Ported our Euchre bidding tests over in https://github.com/TricksterCards/TricksterBots/pull/20.
Added CI to this repo in https://github.com/TricksterCards/TricksterBots/pull/21 so tests automatically run for new changes.

Leaving this open as we continue to port the rest of our existing bot tests.

Just checked in a set of Spades tests ported from our old unit tests. Includes a Util class with helpful state builders.


⚠ Bofrost.at has degraded performance

In 948152d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1723 ms

Resolved: Bofrost.at performance has improved in 4abfc85 after 28 minutes.


Multi homes account issue since HA 2024.3.3

Hi AlexxIT, My account is a multi homes account, my secondary home's devices become unavailable, and not loading correct home list in configutation menu since recent HA 2024.3.3 upgrade.
Could you have a look into this issue?
SonoffLAN Version: 3.6.0
HA version 2024.3.3
home-assistant_sonoff_2024-03-27T11-53-20.932Z.log.zip

Starting from today I get this error Api Login Response:{"error":408,"msg":"appid has expired","data":null} in OH4. Maybe it's connected to your issue....How to get new appid and appsecret keys?

I also get this
{'error': 406, 'reason': 'Authentication Failed', 'apikey':

Just adding on I also get this issue after upgrading to HA 2024.3.3

Another error just pop up:
This error originated from a custom integration.
Logger: custom_components.sonoff.core.ewelink.cloud
Source: custom_components/sonoff/core/ewelink/cloud.py:360
integration: Sonoff (documentation, issues)
First occurred: 12:56:12 AM (1 occurrences)
Last logged: 12:56:12 AM
Cloud WS Connection error: Cannot connect to host us-dispa.coolkit.cc:443 ssl:default [Try again]

PS. "Authentication Failed" can be if you logged in with same account from another place.


[BUG] TDM is Non-Functional for Bot Matches

Attempting to run a bot game TDM on any map using the local client server puts the player in spectate mode while no bots spawn at all and the timer counts down.
Expected result: For the TDM to function properly like the previous version.

fixed for next release used the wrong game mode


Fetch settings after it is saved in Onboarding.

When onboarding completes, app.settings only has data created by the client. If the About modal is opened before some other process fetches settings, it will throw an error because settings is missing the version data.
This fetches the settings after it is saved in onboarding, so all of the settings data from the server is available.
@rmisio there may be a better way to do this, input welcome here.
Closes #726

@rmisio there may be a better way to do this, input welcome here.

Yes, the server should send back the settings data (including the server generated data) on a successful POST. I ran this by @cpacia and he is fine with that. He says he'll push shortly.
Closing this one for now.

Yes, the server should send back the settings data (including the server generated data) on a successful POST. I ran this by @cpacia and he is fine with that. He says he'll push shortly.

Ok, great. I thought that was the right answer, but figured there was some reason it wasn't being done already.


npm install breaks?

Trying to setup the repo locally after cloning and npm install breaks using node 0.12.8 and 4.1.1? Can anyone else reproduce this locally?

Currently using node 7+ and no issues.


[no-unknown-property] Should data attributes be treated as case insensitive

Technically HTML tags and their attributes are case insensitive [1]. But React types generally prevent usage of the uppercase/mixed case version of the tags and attributes as it conflicts with other established conventions (eg: treating tags with the first letter uppercase as a component in JSX). The typings for React also error out for uppercased attributes.
However, data-attributes are special in this regard; From what I can see data-testId is considered to be valid attribute even by react although it's automatically lowercase by the browser (TS enabled react sandbox). Right now the lint rule creates a false positive for this and I am not sure if that should be the case.

for tags, not generally - jsx explicitly prevents it.

Right, but what I meant to say was that: the lint rule throws an error for it.

The relevant code due to which it throws an error (the regex doesn't have A-Z)

hmm - https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/data-* says

The name must not contain any capital letters.

but i guess React and TS lowercase everything to meet that requirement. I'll update.


Recommend the use of quick_main.

Fixes #59.
I didn't modify the others examples since they didn't already use error-chain. Should I?

I would strongly prefer to use the other form of quick_main.
fn run() -> Result<()> {
    /* ... */
}

quick_main!(run);

This makes it look a lot more similar to the user's functions that they are pasting these code samples into.

Thanks @Yamakaky 🥇
I think I prefer the version without the closure to, because it looks more 'typical' of Rust code.

I was thinking, by using quick_main! for the error chain case, that means that examples that use error-chain will be written quick_main!(main) but those that don't will be written fn main() { run().unwrap(); }. I wonder if that's inconsistent and confusing.
We could demand every example use error-chain, even when there's only one error type.
Opinions @dtolnay @Yamakaky ?

I agree. Let's always use quick_main.

We seem to want to recommend the use of error-chain, so let's do that!

Great! @Yamakaky could you update the PR? The example above this one should also use quick_main.


QR Tracker slow to track QR's if at all

Describe the bug
Trying to use the QR Tracker in Hololens 2 produces unusable results. It takes upwards of 5 mins to scan a QR code and often in the most unlikely scanning positions. Just doing simple QR recognition seems to be faster and easier to read QRs in all positions using PhotoCapture for a texture to decode
To reproduce
Steps to reproduce the behavior:

Load basic scene with included QRWatcher script in the scene.
Attach a Text object for scanning feedback
Build and stand in front of a QR code until it scans

Expected behavior
The QRWatcher script looks for newly found QR codes and filters out any in hololens memory to avoid 'ghost' qr codes. It simply looks for a QR code, adds it to the list of found codes and updates the text object to give basic text feedback when a code is recognized. This should be giving QR text feedback every frame the QR is in camera view.
Screenshots
A screenshot doesn't do much justice but here is a comparison video of performance
https://visuallive-my.sharepoint.com/:v:/p/deynon_b/Ef9H7T-fpSlKuljvQOF3A5oBGiZK4iVBb5aznCA4bosDHg?e=afa6Jf
Your setup (please complete the following information)
-QrWatcher.txt [converted to .txt from .cs]

Unity Version 2019.4.8f1
MRTK Version 2.4

Target platform (please complete the following information)

HoloLens 2

Additional context
When our developers asked for access to PV camera in hololens 2, we were prompted to use the QR Tracker provided by Microsoft. In the early hololens 2 development we did use this system for QR reading. After multiple field and customer tests, we gathered that the system seemed to be too unresponsive for most use cases and have given our proprietary reader access via PhotoCapture. This produces better results but does show a noticeable performance hitch as it is capturing and decoding. For our own QR Reader to work seamlessly, we need access to a performant camera stream.

Relevant discussion in different repo (Microsoft QRWatcher example): https://github.com/chgatla-microsoft/QRTracking/issues/19
I think it comes down to camera resolution. I have printed out a QR code almost the size of a piece of A4 paper sheet, and at 1-2 meters it tracks instantly, without any performance stuttering.
I am a bit disappointed with the experience though, having to workaround by increasing QR code side so much.

I have encountered the same problem while experimenting with the example, it's unusable in it's current state. 10x10cm QR Codes needed a distance <40cm to be detected, smaller ones were even worse.
Is it possible to enable the usage of the main camera for the QR code detection?
(While we are at it, the Unity integration of the QR Tracker is cumbersome due to it being distributed by nuget and is lacking any kind of unity integration. I would wish for it to be included with MRTK, since it seems like a natural extension. )

Any updates on this? Tracking seems to be slow especially when the distance between guessed point and qr marker is small. If the distance gets bigger, it seems to be instant.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Better error handling

A preview of the general idea is committed to the branch https://github.com/unreal4u/telegram-api/tree/errorHandler , but this needs some more work as I'm not convinced that that implementation is the best one.

Starting from #55, an error can be identified by a rejected Promise. The RequestHandler rejects it with a TelegramResponse, which is populated if valid data was received.

Closing this one as it works, I will however check for some pitfalls tomorrow before launching a new version.


Translate configuration/taint-and-toleration.md in Korean

This is a Feature Request

What would you like to be added

Translate concepts/configuration/taint-and-toleration.md in Korean.
Why is this needed
There is no document with concepts/configuration/taint-and-toleration.md file in Korean.
Comments

Page to update: https://kubernetes.io/docs/concepts/configuration/taint-and-toleration/

/language ko
/assgin


Fix constructor link display text

?displayProperty=nameWithType displays as "Classname.Classname()". This does not seem desirable to me as you don't/can't call a constructor that way. Better to just display "Classname()".

Thanks for the review, Bill.


Add more context to malformed JSON errors

Include the JSON text and underlying error.

@swift-ci please test


OLS-791: Remove references to yarn and replace with npm

/override "Red Hat Konflux / ols-e2e-tests-rosa / lightspeed-console"
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-console"
Override tests that are currently broken

/lgtm


Ignore test files

This change adds a regex that ignores test files by default.
I had this issue where props would some times not show up for certain components. Turns out, those components were typically the ones with tests.
When I looked in the docz state, I noticed that the test files were also included in the props array. This seemed a bit odd. After a bit of research, I understood that the docz Props component found the first entry, which often (but, for some reason, not always) was the test file. It was a hit, so the Props component was shown, but there was no props to show (since the test file didn't have any prop definitions).
This change filters out the test files by default, which should mitigate the problem, as well as speed up the process of parsing props for all users.
This has been a new problem, so it might have been a regression of sorts - not sure.

Ah yes, it's indeed a regression from this commit, where the test files were overlooked. The previous implementation seemed to have skipped them almost by accident, by only accepting a single period in any file name 😅


Update translation docs

We do a bad job of explaining how to contribute documentation for readthedocs docs and UI.

I want to work on this issue, as far as I got your point is that the contribute documentation should be in other languages also. right?

@mukul09 this issue is about improving the steps for contributing to translations (but also looks like about contributing to documentation and design).
Here are a links about the current docs:

https://docs.readthedocs.io/en/latest/i18n.html?highlight=translations#administrative-tasks
https://docs.readthedocs.io/en/latest/contribute.html#helping-on-translations (here there is just a link!)

Note: I think the steps about contributing to documentation are covered on #3498.

@humitos added a guide on managing translations. That was the piece i was looking to start with here. Closing this issue.


Adopting of Tuners to New Codebase

It is not 100% clear if it is needed to parameterize tuners by a sampler parameter (given that some tuners are specific to some samplers). If no parameterization is needed, the porting is straightforward.

Done in branch 0.5.0.


Add request saving feature

The idea is to be able to make requests from using this endpoint and being able to get those requests afterward something like requestb.in.
My use case would be to test API that require API keys that changes overtime (like APNS and GCM) and being able to make sure that the request has been done with the correct parameters.

It's an interesting feature but  I would propose some changes:
Change /store and retreive to be /store/:id so that multiple users can send different ids and retrieve different ids
Then we can have 1 ETS table holding all the requests and each tuple is
{ id, request }
And they can have more than 1 with id
Then if retrieve/id is requested all these tuples are removed, aggregated and sent.
We should also have some way to TTL the tuples of an id when they get too old. So that the ETS table does not consume too much memory?
What do you think?

The idea of having an id for each user is great, I din't think about it because I use this for local testing but it's definitely a feature needed for the heroku hosted version.
Would you be ok if we add ecacheas a dependency to resolve the TTL part?

What about https://github.com/sasa1977/con_cache ? I never used ecache but if you have experience maybe it's good enough?

Can't get why this test does not pass, would you give me an hint @edgurgel

Cool after pulling your branch I noticed that we test on travis using:
mix test --no-start (.travis.yml)
We need to remove this --no-start then con_cache will be up an running 👍

I have added the doc fixed the test and added test on the RequestStore.
I had to update the elixir 1.2 version because of a bug with OTP 19 support.
https://github.com/elixir-lang/elixir/issues/4617
Please review.

Thanks @steffenix! I will release a new version soon! 🎉


Reproduce Validate MAE

Hi,
Thanks for your interesting work. I  have a problem regarding the evaluation. I downloaded your checkpoints from here, then I run the following command as mentioned in the Readme (for all_fold_seed0 checkpoint):
conda activate graphormer-lsc
export arch="--ffn_dim 768 --hidden_dim 768 --attention_dropout_rate 0.1 --dropout_rate 0.1 --n_layers 12 --peak_lr 2e-4 --edge_type multi_hop --multi_hop_max_dist 20 --weight_decay 0.0 --intput_dropout_rate 0.0"
export ckpt_path="checkpoints"
export ckpt_name="all_fold_seed0.ckpt"
bash inference.sh

The output log is:
Global seed set to 1
 > PCQM4M-LSC loaded!
{'num_class': 1, 'loss_fn': <function l1_loss at 0x7fc2381b3950>, 'metric': 'mae', 'metric_mode': 'min', 'evaluator': <ogb.lsc.pcqm4m.PCQM4MEvaluator object at 0x7fc1995d2110>, 'dataset': MyPygPCQM4MDataset2(3803453), 'max_node': 128}
 > dataset info ends
total params: 47167841
GPU available: True, used: True
TPU available: False, using: 0 TPU cores
Global seed set to 1
initializing ddp: GLOBAL_RANK: 0, MEMBER: 1/1
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0]
len(val_dataloader) 1487
Validating: 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 1487/1487 [03:04<00:00,  7.42it/s]
0.027769196778535843
--------------------------------------------------------------------------------
DATALOADER:0 VALIDATE RESULTS
{'valid_mae': 0.027769196778535843}
--------------------------------------------------------------------------------
[{'valid_mae': 0.027769196778535843}]

I assumed that I should get results near Table 1 "validate MAE column", but it's different from that. Do I miss something?
Thanks for your help.

Hi @alirezamshi ,
As mentioned in the report, we trained 8 Graphormer models using 8-fold cross-validation, and additional 2 Graphormer models using whole data.
The checkpoint you are using now is all_fold_seed0.ckpt, please try to use the cross-validation split here here and use the correct checkpoint instead of these begin with all_fold.
Best,
Chengxuan

Hi @chengxuanying
Thanks for your reply.
So, If I remove  this line , and modify this line to just include train split ids, then it should be fine, right?
Best,
Alireza

You can alternatively uncomment L71-86, and comment L89-96.
And remember to load the correct checkpoint.
By the way, in the code, I simply set fold index the same as seed.

Got it. Thanks.

@chengxuanying , great work, i have a problem, if we want to reproduce validate MAE 0.1234 (single model) with training from scratch, which branch (main or ogb-lsc) we should use?


chore: update to iced latest

I would be really interested in seeing this merged. Currently this has theming issues for .row and .row-selected. Both elements and all their sub elements do not respect theme directives like background, color, border-width or border-color and a number of directives too.
You can reproduce this using the theme files which are linked in the readme. The most prominent effect of this, is the currently seected row not being highlited in all of these themes.


Add correct defaults for oppositeLength props to docs

Noticed that the defaults in the docs weren't right, thought I would update them! Thanks for the awesome package!

Nice catch! Thank you Brian! :)


enable KVM via DISTRO_FEATURES

This enables controlling the kernel feature CONFIG_KVM=y via DISTRO_FEATURES.

Since the kvm distro feature is defined in the meta-virtualization layer, could you move this to under external/virtualization-layer/recipes-kernel?

I have a quick question: Could you please share how did you test it ? Are you using virtualbox ?

I'll have a full write-up of the stack once I have it sorted.

Is all jetson paltforms compatible with that ?

I believe it only applies to cores that imlement Hardware Virtual Extensions (HVE).  I'm not an NVIDIA SoC expert (mostly Qualcomm), so I would need to look it up as to which ones support this.

I think we should enable CONFIG_VIRTUALIZATION in the kernel config file as well.

This is what I currently see with this PR:
root@jetson-xavier-nx-devkit:~# dmesg |grep -i 'CPU features:'
[    0.639115] CPU features: detected feature: Privileged Access Never
[    0.639141] CPU features: detected feature: LSE atomic instructions
[    0.639178] CPU features: detected feature: User Access Override
[    0.639204] CPU features: detected feature: 32-bit EL0 Support
root@jetson-xavier-nx-devkit:~# dmesg |grep -i kvm
[    1.138391] kvm [1]: 16-bit VMID
[    1.138419] kvm [1]: IDMAP page: 80f52000
[    1.138441] kvm [1]: HYP VA range:<PHONE_NUMBER>:7fffffffff
[    1.140504] kvm [1]: Hyp mode initialized successfully
[    1.140649] kvm [1]: vgic-v2@3884000
[    1.141097] kvm [1]: vgic interrupt IRQ1
[    1.141164] kvm [1]: virtual timer IRQ4

I suspect a dtb change is required to get this:
[    0.762646] CPU features: detected feature: Virtualization Host Extensions

I saw discussion about this topic while ago, please take a look here, I think updates in the DTB are needed not only kernel config.

I see good to know.  The one that gave me the clue to proceed was this one:  https://github.com/b-man/Xavier-KVM
I renamed this to WIP until I see the whole stack running as expected, after which I will document the whole process.  I'm thinking that should be no later than Monday.

I am able to boot Ubuntu 18.04 via Firecracker on the Xavier with this PR, Yocto config below, and following Running Firecracker.
Yocto Config
DISTRO_FEATURES_append = " virtualization kvm seccomp ipv6"

RUST_PANIC_STRATEGY = "abort"

IMAGE_LINGUAS = "en-us"

PREFERRED_VERSION_libglvnd = "1.3.2"

CORE_IMAGE_EXTRA_INSTALL += "\
    packagegroup-firecracker \
    libvirt-libvirtd \
    libvirt-virsh \
    libvirt \
    iptables \
    dmidecode \
    python3 \
    python3-opencv \
    ntp ntp-tickadj \
    tzdata \
    curl \
"

QEMU_TARGETS = "aarch64"

PACKAGECONFIG_pn-qemu = " fdt kvm pie seccomp"

PACKAGECONFIG_pn-libvirt = " gnutls qemu yajl test remote libvirtd netcf udev python fuse firewalld libpcap"

recipes-extended/libvirt/libvirt_%.bbappend - work around build issue
EXTRA_OEMESON_remove = "-Dtests=enabled"
EXTRA_OEMESON += " -Dtests=disabled"

Either add default locatime symlink in rootfs stage, or manually after first boot, then reboot.
mkdir -p /usr/share/etc/
ln -sf /usr/share/zoneinfo/America/Los_Angeles /usr/share/etc/localtime

Info from Ubuntu running via Firecracker - Running Firecracker
root@fadfdd4af58a:~# ls-release
-bash: ls-release: command not found
root@fadfdd4af58a:~# cat /etc/os-release
NAME="Ubuntu"
VERSION="18.04.2 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.2 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic
root@fadfdd4af58a:~# uname -a
Linux fadfdd4af58a 4.14.174+ #14 SMP Mon Nov 23 20:44:24 UTC 2020 aarch64 aarch64 aarch64 GNU/Linux
root@fadfdd4af58a:~# lscpu
Architecture:        aarch64
Byte Order:          Little Endian
CPU(s):              1
On-line CPU(s) list: 0
Thread(s) per core:  1
Core(s) per socket:  1
Socket(s):           1
NUMA node(s):        1
Vendor ID:           Nvidia
Model:               0
Stepping:            0x0
BogoMIPS:            62.50
L1d cache:           64K
L1i cache:           128K
L2 cache:            2048K
L3 cache:            4096K
NUMA node0 CPU(s):   0
Flags:               fp asimd evtstrm aes pmull sha1 sha2 crc32 atomics fphp asimdhp cpuid asimdrdm dcpop
root@fadfdd4af58a:~#

Since the kvm distro feature is defined in the meta-virtualization layer, could you move this to under external/virtualization-layer/recipes-kernel?

done

Thanks!
I merged this before checking the commit headers...  Next time, please add a Signed-off-by line to your commits, per our contribution guidelines.

Just an update on Nano.  Building with identical config for machine jetson-nano-devkit, /dev/kvm is not coming up.  So that will require more effort.

I figured it out.  Unfortunately it requires a change.  I'll open another PR.

Actually it might be best to revert this one


Refactoring tenant management

Test:
Select Tenant,
Create Tenant: Manually and with QrCode,
Delete Tenant: Swipe an organisation to the left and delete it,
Restore Tenant: restore the default list and delete the custom organisation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: LucasBrazi06:x: FABIANOFABIANO seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


need provenanced cmd to work with go get or go install

Summary of Bug
need provenanced cmd to work with go get or go install (my use case: trying to get a docker image with provenanced installed so that i can use it in another application, but this would be useful for user's of provenance also)
Version
provenance release tag 0.1.4
Steps to Reproduce
go get github.com/provenance-io/provenance/cmd/provenanced
go: found github.com/provenance-io/provenance/cmd/provenanced in github.com/provenance-io/provenance v0.1.4
go get<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>reading github.com/gogo/protobuf/go.mod at revision v1.3.3: unknown revision v1.3.3

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

@mtps  any progress on this?
still getting  go get github.com/provenance-io/provenance/cmd/provenanced go: downloading github.com/provenance-io/provenance v0.1.10 go: found github.com/provenance-io/provenance/cmd/provenanced in github.com/provenance-io/provenance v0.1.10 go get<EMAIL_ADDRESS>requires<EMAIL_ADDRESS>reading github.com/gogo/protobuf/go.mod at revision v1.3.3: unknown revision v1.3.3

Looking at gaiad, we might have to restructure how we are using the provenance repo and the provenanced binary. From what I can tell @arnabmitra, they have 0 references to github.com/gogo/protobuf from within the gaia repo, allowing them to override the underlying sdk references as a whole in the vendor dir. We have refs to github.com/gogo/protobuf directly in x/metadata, x/name, x/attributes, and x/marker, requiring us to pull the github.com/gogo/protobuf repo transitively from go get. Aka, it keeps getting added to go.mod because it HAS to be there. In theory, if we changed our references in code from github.com/gogo/protobuf to github.com/regen-network/protobuf, it could work.

Note: v1.3.3 (the version that keeps getting pulled in from upstream cosmos-sdk), does NOT exist in project github.com/gogo/protobuf

Cosmos simapp/simd suffers the same issue.

How does gaiad work, because it is in a separate repo?


Alexa play media not working in everywhere group

Hey!
So once again playing music in the everywhere group via
{
    "media_content_type": "TUNEIN",
    "media_content_id": "Radio Hamburg in uberall Gruppe"
}

isn't working. I'm just getting "I am not sure" from Alexa(Hope thats the right translation since I am using german Alexa).
I am on the newest Alexa Media Player Verison and Home Assistant Version 0.110.7.

Did this work before for you? This is likely an Amazon error as we haven't changed that part of the code recently.

Yea it worked before. And I didn't changed a thing since then. The last time it stopped working the solution was changing "Überall" to "Uberall". Not sure what could be a fix to the problem this time..

I'll leave this up in case someone figures it out, but not much we can do here.

Fixed itself.

@Wibias How Did you fixed it? I have the same problem since yesterday and didn't find a fix yet...

Cant really help you there. Didnt really do anything to fix it expect reinstalling Alexa Media Player. That in itself didn't fix it. Just after like 2 days after reinstalling it it worked again. Now on the newest Alexa Media Player version and its still working just fine. Still using the same stuff as in the picture above.

So its not working once again. So I am reopening this.

So I fixed it by removing the "Gruppe" in "media_content_id": "Radio Hamburg in überall Gruppe". In case of other languages try to remove "group" from the media_content_id.


Added build-task create and show commands.

Added following commands:
az acr build-task create: Creates a new build task with specified parameters.
az acr build-task show: Get the details of a build task.
Currently oAuth flow is not enabled in create. So you will need to provide a git personal access token (pat), which you can get from Github -> Settings -> Developer Settings -> Personal access Tokens -> Generate new (with repo scope).

@ankurkhemani Thanks. Can you also update the extension: azure-cli/doc/extensions/acrbuildext/azext_acrbuildext/. For now, you need to manually copy/paste the code there.


⚠ DesiTorrents has degraded performance

In d0d518d, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 1984 ms

Resolved: DesiTorrents performance has improved in f2c18c3.


Import TestGrid UI code with history

Imports a subset of the code from https://github.com/GoogleCloudPlatform/testgrid that corresponds to the new (MVP) TestGrid UI. Includes the history for these files (but not the rest of the main repo).
Ref #2.

@michelle192837 cla issue? add personal email to github?

Reached out to both contributors above for CLA signing! (Thanks in advance folks :D)

/easy-cla

/easy-cla

/easycla

NOTICE file is no longer needed since everyone has signed the CLA! (Thanks folks! 🎉 )
Based on offline discussion on K8s Slack, once all the human CLAs (e.g. not google-oss-prow, which is a bot) are covered, we should be good to merge.

Forcing merge for just this initial PR.


Unable to use with Arduino IDE 2.3.2

I've only been able to use this board library with the legacy version of the Arduino IDE (1.8.19), albeit with a warning.
When trying to compile with the latest version of the IDE (2.3.2), I'm getting the following error message:
Invalid FQBN: tool release not found<EMAIL_ADDRESS>Compilation error: Invalid FQBN: tool release not found<EMAIL_ADDRESS>It seems esp32-20230419 is missing from the boards.json definition.

This is known issue that I'm fixing as soon as I have some time this week. Here's the discussion about the new method.

Hi,
Thanks for your response, and my apologies for creating this issue: I didn't see the discussion about the workaround (which works great !)
Cheers


Upgrading from 19.1 to 19.2 using homebrew doesn't work

I tried to upgrade from 19.1 to 19.2 on my machine using brew cask upgrade and nothing happened with the cockroach installation. I believe this might be because we moved from the central brew cask to our own tap. I wound up fixing this by running brew uninstall cockroach and then brew install cockroachdb/tap/cockroach
We may need to patch our Mac install instructions to make it clear when upgrading that you have to use the new tap.
I'm not able to reproduce this on my laptop since I am not able to reinstall 19.2 from homebrew since it was removed from homebrew-core as part of the new licensing.
cc @bdarnell

@bdarnell, do you have a sense of how that note at the end of the homebrew section should be updated? https://www.cockroachlabs.com/docs/stable/install-cockroachdb-mac.html#use-homebrew-1

I thought I had tested this upgrade and it worked, but now it's too late for me to test again. If anyone is in a position to test this and can tell me exactly how it failed I might be able to do something about it but we should probably just update the docs:
"If you installed CockroachDB via Homebrew before version 19.2.0, run brew uninstall cockroach before installing the new version. If you previously installed using a different method, you may need to remove the binary before installing via Homebrew."


login failed for user 'SA'

I have followed the documentation and everything went fine except for the step 'connect from outside the container'.  I am getting the error as below
PS C:\WINDOWS\system32> sqlcmd -S XX.XXX.XX.XX,1433 -U SA -P "<YourNewStrong!Passw0rd>"
Sqlcmd: Error: Microsoft ODBC Driver 13 for SQL Server : Client unable to establish connection because an error was encountered during handshakes before login. Common causes include client attempting to connect to an unsupported version of SQL Server, server too busy to accept new connections or a resource limitation (memory or maximum allowed connections) on the server..
Sqlcmd: Error: Microsoft ODBC Driver 13 for SQL Server : TCP Provider: An existing connection was forcibly closed by the remote host.
.
Sqlcmd: Error: Microsoft ODBC Driver 13 for SQL Server : Client unable to establish connection.
Sqlcmd: Error: Microsoft ODBC Driver 13 for SQL Server : Client unable to establish connection due to prelogin failure.

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

ID: b152444b-dfb8-5cbe-cf45-9830bd085a68
Version Independent ID: 31719660-d4d0-7f63-efcc-846c822a63c2
Content: Get started with SQL Server containers on Docker (running SQL Server on Linux) - SQL Server
Content Source: docs/linux/quickstart-install-connect-docker.md
Product: sql
GitHub Login: @rothja
Microsoft Alias: jroth

sorry, it's my bad.  I have use the command
docker run -e "ACCEPT_EULA=Y" -e "SA_PASSWORD=<YourStrong!Passw0rd>"    -p 1434:1434 --name sql1
-d mcr.microsoft.com/mssql/server:2017-latest
which should suppose to be
docker run -e "ACCEPT_EULA=Y" -e "SA_PASSWORD=<YourStrong!Passw0rd>"    -p 1434:1433 --name sql1
-d mcr.microsoft.com/mssql/server:2017-latest

@rajendrarama Thanks for diagnosing the problem in the test.  I gather there is no Documentation bug here after all.  So I will now Close this Issue 1465 as completed.
If further communication is needed about 1465, the @ mention mechanism must be used (@ sign, followed by my name MightyPen, with no intervening spaces).  If a good reason arises, I could reOpen this Issue.
Thank you.

I got the following error when trying to connect to the sqlserver instance from external server:
The login is from an untrusted domain and cannot be used with Integrated authentication.. Check if more detail is available for this error in the errorDetail field of the log.
Any advice please!


Android: Notification will be updated after free text input without displayNotification

Problem
Notification will be updated after free text input without displayNotification

Expected
Notification should keep "pending" state until the notification has been updated

Version

4.0.1: not reproduced
4.1.0: reproduced

Cause
https://github.com/invertase/notifee/blob/f5da72227339574897f37717e4609937025b6887/android/src/main/java/app/notifee/core/ReceiverService.java#L191-L197
This calling notificationManagerCompat.cancel has some issues. This issue is NOT reproduced if I set autoCancel: false
Suggestion
I think adding branch condition will be the solution
if (remoteInput == null && notificationModel.getAndroid().getAutoCancel()) {

What do you think?
I’m worried about you guys, because I can't see some activities on github. I think notifee is one of the most variable library working with react-native. I hope your continues development,Thanks.

hi ı want to implement a messaging style notification like you but ı have a problem like this https://github.com/invertase/notifee/issues/936#issue-2045163371 in my real device s22.


ci: test with Python 3.9

Related Issues

Python 3.8 already reached End Of Life
We are testing Haystack with Python 3.9 (https://github.com/deepset-ai/haystack/pull/8492)

Proposed Changes:

use Python 3.9 in CI

How did you test it?
CI
Checklist

I have read the contributors guidelines and the code of conduct
I have updated the related issue with new insights and changes
I added unit tests and updated the docstrings
I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
I documented my code
I ran pre-commit hooks and fixed any issue

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

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

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
1716

Relevant Lines:
1917

💛  - Coveralls


Update homework.sh

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
loops but need more time
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
the commands when pasted on bash work fine but not when tryin to execute ' chmod +x ./homework.sh ' even when file is in the folder
How were these changes tested?
each line with browser; with ls, nano etc
A reference to a related issue in your repository (if applicable)
@mentions of the person or team responsible for reviewing proposed changes (At least 2 people)

@michaeladrouillard and @dtxe

Checklist

[x] I can confirm that my changes are working as intended

@michaeladrouillard and @dtxe