az aks install-cli is broken with 0.0.26

The new version seems to be missing some assets, which breaks az aks install-cli
The detected arch is x86_64, would be treated as amd64, which may not match the actual situation due to translation and other reasons. If there is any problem, please download the appropriate binary by yourself.
Downloading client to "/usr/local/bin/kubectl" from "https://storage.googleapis.com/kubernetes-release/release/v1.26.1/bin/linux/amd64/kubectl"
Please ensure that /usr/local/bin is in your search PATH, so the `kubectl` command can be found.
The detected arch is x86_64, would be treated as amd64, which may not match the actual situation due to translation and other reasons. If there is any problem, please download the appropriate binary by yourself.
Downloading client to "/tmp/tmp719hdx74/kubelogin.zip" from "https://github.com/Azure/kubelogin/releases/download/v0.0.26/kubelogin.zip"
Connection error while attempting to download client (HTTP Error 404: Not Found)

Using --kubelogin-version 0.0.25 works for now

My CI/CD is also dependant on this binary and my build just broke.
I have seen this before, this repo's release process is a bit sketchy...the tagged/release is published before the build is complete. So if this build process breaks the .zip/binary never gets added to the release as a package.

Hi @weinong, In your release job, can you move this step after the build step?
https://github.com/Azure/kubelogin/blob/584497114d74eca24f55f078befb11d8955bb0ba/.github/workflows/release.yml#L37-L46
This way the release isn't created if the other steps fail.

☕💡thank you for this - to add context -  I think It’s because the release failed on publish due to OpenSSL issue, we can quickly release so that the assets will start appearing again: (below you can see the assets kot there because of the second link withi shows the OpenSSL connection error on publish)

https://github.com/Azure/kubelogin/releases/tag/v0.0.26
https://github.com/Azure/kubelogin/actions/runs/4129883812/jobs/7135983692

Hope this helps in some context.

@Tatsinnit  I think you are missing the point. When you publish a release without the built "assets" all projects dependent on this fail. There are GitHub actions in the Marketplace that use kubelogin and these actions fail when using a release that has no binary to download.
Just like @philthynz our build failed with,
WARNING: Downloading client to "/tmp/tmpuenv4e9k/kubelogin.zip" from "https://github.com/Azure/kubelogin/releases/download/v0.0.26/kubelogin.zip"
ERROR: Connection error while attempting to download client (HTTP Error 404: Not Found)

In my case our workflow uses the Azure/CLI Marketplace action which is dependant on/and found your latest release but without the "assets". Just now, I see the release job was successful and the assets are there so thanks! But I bet there are others who had a failed build over the last hour.
All I am saying is maybe don't create your GitHub release until the build is successful which should be a simple change to release.yaml
Build -> Create Release -> Add Binaries/Assets right now you have Create Release -> Build -> Add Binaries/Assets

@Tatsinnit I think you are missing the point. When you publish a release without the built "assets" all projects dependent on this fail. There are GitHub actions in the Marketplace that use kubelogin and these actions fail when using a release that has no binary to download.
Just like @philthynz our build failed with,
WARNING: Downloading client to "/tmp/tmpuenv4e9k/kubelogin.zip" from "https://github.com/Azure/kubelogin/releases/download/v0.0.26/kubelogin.zip"
ERROR: Connection error while attempting to download client (HTTP Error 404: Not Found)

In my case our workflow uses the Azure/CLI Marketplace action which is dependant on/and found your latest release but without the "assets". Just now, I see the release job was successful and the assets are there so thanks! But I bet there are others who had a failed build over the last hour.
All I am saying is maybe don't create your GitHub release until the build is successful which should be a simple change to release.yaml
Build -> Create Release -> Add Binaries/Assets right now you have Create Release -> Build -> Add Binaries/Assets

Hiya @trentmillar, thanks for the ping and thoughts. ❤☕ 🙏 Yep, I am with you, (I think my comment above was mainly towards the original thread of why zip was missing)  and we had similar thinking if you see in a commit here : https://github.com/Azure/kubelogin/commit/14dc89e73c52030ee9bf8358a485f42b20cc709b
The publish needed something called release_id as dependency which can only be fed via please, and under #184 we re-arranged the code for the version issue.
What's next? Thank you so much for the thought and I think I am giving some thought how I will make it more simpler and please tag can be created once publish is done, (will loop back and might add you in that refactor PR?) if that's cool.
Thanks heaps 💡

🔈  General note please: ❤☕💡 Thank you for your. patience all release 0.0.26 is out with all 14 artefacts published: https://github.com/Azure/kubelogin/releases/tag/v0.0.26
I have done quick test with vanilla cli and vscode as well and both worked hence I believe other consuming tool will behave now.
Some screenshots to back above details:
Also kind and gentle ping to @weinong and @philthynz if this suffices the needs we can close this issue. ☕💡

Works for me. Thanks for the quick fix.

Thanks @Tatsinnit everything is working great now!
@Tatsinnit sure, I'd be happy to put eyes on this if tagged as a reviewer for your PR 😄

the publishing part has always been unreliable, even before @Tatsinnit's change. It makes me question if I should move it to Azure blob storage...


Not sync shared drive

If google drive has additional shared drives his not synced.
Synced only files in MyDrive folder.

@redzuk Check out the sync_shared_in_drive option in the config file.

cat config.yml | grep sync_shared_in_drive
sync_shared_in_drive: true
But additional shared drives not synced.
Synced only files in MyDrive folder.

I'm not sure which additional files you mean but you have to add shared files to your drive in order for them to be managed by drivesync. There should be a button for that on any shared Drive link.On Aug 26, 2020 23:34, redzuk<EMAIL_ADDRESS>wrote:
cat config.yml | grep sync_shared_in_drive
sync_shared_in_drive: true
But additional shared drives not synced.
Synced only files in MyDrive folder.
—You are receiving this because you modified the open/close state.Reply to this email directly, view it on GitHub, or unsubscribe.

I mean something like this https://imgur.com/3oW5jk5.
Imposible syncing files whis aditional shared disk?


Map buttons don't work properly

Describe the bug
The map buttons are not working properly, they are always loading, what is it due?
Thank you very much!
"silverstripe/recipe-cms": "4.6.0@stable"
"dynamic/silverstripe-locator": "3.1.1"

Screenshots

This is possibly caused by a theme, like simple, adding a background images to image tags.
This can potentially be solved by adding this css to the site:
.locator-map img {
    background: none;
    border: none;
}

Hi @mak001 thank you your suggestion was helpful but I had to overwrite it like this:
.typography img {
	background: none!important;
	border: none!important;
}


Updated 1D plots to be easier to read.

Updated the 1D plots in grapher to use the same default colors as Google Charts as well as increasing the stroke width to 1.5 (up from 1.0).

I'm not sure that these colors are that visually appealing, we may want to consider a future enhancement to allow swapping between a color-blind and non-color-blind mode. People spend a lot of time looking at graphs, we don't want them to be ugly.

Yeah these colours...
Can we just use the Google chart colors? https://developers.google.com/chart/

I have changed the colors to the Google chart colors. They all start looking rather the same after six or so, clearly a pallet chosen for bar/pie chart graphics where you only need neighbors to be differentiated.

Beyond six traces it's a little hard to draw any conclusion anyways, so I wouldn't worry about it. I like this scheme, we can always add more shades or whatever if people complain.
@maffoo how many colors did the delphi grapher have anyways?
This LGTM

The Delphi grapher has 6: blue, red, green, cyan, purple, brown


[Bug]: Metal aSide token, VLAN is requered when it should be "optional"

Terraform Version
v1.6.3
Equinix Provider Version
1.31
Effected Terraform Resources
equinix_metal_connection
Terraform Config Files
resource "equinix_metal_connection" "metal_vc_connection" {
name               = "metal-vrf-to-azure"
project_id         = var.metal_project_id
type               = "shared"
redundancy         = "redundant"
metro              = var.metro
speed              = "50Mbps"
service_token_type = "a_side"
contact_email      = var.notification_email
}
Debug Output
No response
Panic Output
No response
Expected Behavior
I has been working so far, same code
Actual Behavior
equinix_metal_connection.metal_vc_connection: Creating...
╷
│ Error: Error creating Metal Connection
│
│   with equinix_metal_connection.metal_vc_connection,
│   on main.tf line 164, in resource "equinix_metal_connection" "metal_vc_connection":
│  164:  resource "equinix_metal_connection" "metal_vc_connection" {
│
│ Could not create a shared Metal Connection: POST https://api.equinix.com/metal/v1/projects/bd27e9bf-0df2-4187-a2ad-a8cd1e79fa26/connections: 422 VLAN(s) must be set
│ on creation for Fabric VCs.
Steps to Reproduce
just create a new metal connection aSide

This error is a reflection of API behavior:  the Equinix Metal API was updated to require VLANs to be specified when creating a connection and to prevent updating VLANs when updating a connection.  The provider was updated to flag these issues during plan instead of waiting for an API error.  That provider change was included in the v1.32.0 release.


Support for Android N beta

Windows 10, Godot 2.0.3
I'm seeing inconsistent results when deploying to Android N Beta. I realise this wouldn't be 'supported' but I thought I'd report anyway. For example when deploying the Genetic Programming Godot demo I get this:
Installing into device (please wait..): Google Nexus 6P
running cmdline: "H:/Android/sdk/platform-tools/adb.exe" "-s" "CVH7N15B30002131" "install" "-r" "C:\
Users\UserName\AppData\Roaming/Godot/tmp/tmpexport.apk"
13510 KB/s (6824426 bytes in 0.493s)
Success
running cmdline: "H:/Android/sdk/platform-tools/adb.exe" "-s" "CVH7N15B30002131" "shell" "am" "start
" "-a" "android.intent.action.MAIN" "-n" "org.godotengine.geneticprogrammingsimulation/org.godotengine.godot.Godot"
Starting: Intent { act=android.intent.action.MAIN cmp=org.godotengine.geneticprogrammingsimulation/org.godotengine.godot.Godot }
Error type 3
Error: Activity class {org.godotengine.geneticprogrammingsimulation/org.godotengine.godot.Godot} does not exist.

Steps to reproduce:

Install Android N Beta on a device
Import the Genetic Programming Godot demo to Godot
Perform a one-click deploy to the device (I have project defaults set)
Observe the above error

I've also tried the Drag/Drop GUI demo and that deploys and runs ok.
Another project installed but when I ran it the screen was flashing/strobing badly on the splash screen.
My own project gets a similar error to the above.
Should I assume Android N support will come later and wait, or can something be done to make it work?

the error looks like same as http://www.godotengine.org/qa/4268/android-export-error-type-3-activity-class-does-not-exist

maybe the name of the generated application is too long, try setting up a
shorter name in the android project settings
On Mon, May 23, 2016 at 9:20 AM, volzhs<EMAIL_ADDRESS>wrote:

the error looks like same as
http://www.godotengine.org/qa/4268/android-export-error-type-3-activity-class-does-not-exist
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/4777#issuecomment-220964645

I get a similar error on a 4.4 android device with master branch version + 2.0.3 export templates. Maybe something is wrong with them?

I get a similar error on a 4.4 android device with master branch version + 2.0.3 export templates. Maybe something is wrong with them?

Well 2.0.3 templates are not compatible with 2.1-dev.

@akien-mga Well, it doesn't work with 2.0.3 linux x64 + 2.0.3 templates, and results in error:
#...omited apk building and jarsigner output...
Warning:
This jar contains entries whose certificate chain is not validated.
This jar contains signatures that does not include a timestamp. Without a timestamp, users may not be able to validate this jar after the signer certificate\'s expiration date (2043-10-06) or after any future revocation date.
#...omited a bit here...
Failure [INSTALL_FAILED_INVALID_APK]
returned: 10886, waiting for: 10886 # Unrelated?
Starting: Intent { act=android.intent.action.MAIN cmp=io.github.kobuge-games.minilens/org.godotengine.godot.Godot }
Error type 3
Error: Activity class {io.github.kobuge-games.minilens/org.godotengine.godot.Godot} does not exist.

does device already have an app with same package name?

@volzhs No, it doesn't.

@volzhs no, I have never even tried to install that app on the device before. Still, based on above posts perhaps the Android N thing is a red herring and there is another cause. I agree that error can be raised in other circumstances as well. I'll experiment some more and if I find out more I'll update.

Ok, I've just created a new project with just a label, with Godot & templates 2.0.3 and it deployed fine with one-click..

I got same problem with Android 7.0 (as knwon as N, Nougat) official release, not beta.
I tried both way.

one click deploy
exporting release apk and install with adb install

Failed to install C:\Users\volzh\AppData\Roaming/Godot/tmp/tmpexport.apk: Failure [INSTALL_PARSE_FAILED_UNEXPECTED_EXCEPTION: Failed to parse /data/app/vmdl242509773.tmp/base.apk: Corrupt XML binary file]
No problem with Android 4.4 device.

I tested with official 2.1 stable and seems no problem.
I should investigate what is wrong.

Changing version code also makes error.

It seems that problem is in replacing String value in AndroidManifest.xml while exporting.
https://github.com/godotengine/godot/blob/fc61eb37ce004ecfa0656a46c68d111c68bb5c19/platform/android/export/export.cpp#L641
but fixing it is beyond me.
@RandomShaper would you take a look for this?

I'm interested in checking this, but please let me add it to my backlog
until I got enough time. I'm pretty busy developing my game (with Godot, of
course).
So the same APK fails (intermitenty or not) on Android N, but gets
installed OK anytime on former releases. Is that the case?

@RandomShaper correct.

I found so~ weird thing.
It works if compiled tool with scons platform=windows target=debug
And problem with scons platform=windows target=release_debug
What's wrong with release_debug?

If you could extract the manifest from both resulting APKs we would be able
to tell if it's the offending piece.

Tool
Generated manifest by tool with debug
Generated manifest by tool with release_debug

Size
33,040 bytes
33,040 bytes

Compressed size in APK
6,773 bytes
6,777 bytes

Found diff with VBinDiff, only one difference.

I found that is last savint string (not sure this is typo or not)
...
savint string :com.google.android.gms.auth.api.signin.RevocationBoundService
savint string :com.google.android.gms.auth.api.signin.permission.REVOCATION_NOTIFICATION
file size: 33040
ADDING: AndroidManifest.xml
ADDING: META-INF/MANIFEST.MF
...

So... is this issue fixed now?
I don't have problem after #6574 merged.


Asynchronous TLS session cache

Issue # (if available):
https://github.com/awslabs/s2n/issues/1028
Description of changes:

Clean up code that is reviewed. The previous review comments can be found here https://github.com/songhu/s2n/pull/5
The pull request only contains changes that need for asynchronous session cache.
Saw and cry code change are due to additional handshake state SERVER_SESSION_LOOKUP

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Is there a way to configure a timeout for session id look up?

Is there a way to configure a timeout for session id look up?

Hi:
Thanks for the feedback! S2N invokes callbacks for the caller to take action on looking up session id. The timeout of looking up might be owned by the caller. For example, the caller might set a timeout on redis cluster which stores session ids.
Can you give me a use case of your idea?

I see. I'm trying to understand how the callbacks should be organized. Is it going to be something like:

static map *session_id_2_promise;
static int lookup_timeout;

int s2n_cache_retrieve_callback (struct s2n_connection *conn, void *, const void *key, uint64_t key_size, void *value, uint64_t *value_size) {
     promise *lookup_promise = map->lookup(map, key, key_size);
     if (lookup_promise == NULL) {
         lookup_promise = cache_client->async_get(cache_client, key, key_size);
         map->put(map, key, key_size, lookup_promise);
     }

     int lookup_result_code = lookup_promise->get(lookup_timeout, value, value_size);
     switch (lookup_result_code) {
        case SUCCESS:
           // we may choose to keep it if the map is an LRU cache
            map->remove(map, key, key_size);
            return S2N_SUCCESS;
        case AWAITING:
             return S2N_AGAIN;
        case TIMEOUT:
            map->remove(map, key, key_size);
            return S2N_FAILURE;
        default:
            return S2N_FAILURE;
     }
}

Or you had something different in mind?
Also in case it returns S2N_AGAIN then how often is s2n going to check the status again?
Is there some sort of polling loop?
P.S. the use case is a distributed TLS resumption based on session-ids (potentially implementing a strike register for TLS 1.3 0-RTT).

Your understanding is correct. The implementation of your callback looks good to me.  I will document the related APIs once this pr is approved.

Also in case it returns S2N_AGAIN then how often is s2n going to check the status again?
The next time when the client calls s2n_negotiate.

Is there some sort of polling loop?
This could be based on epoll events which is owned by the client, the client is responsible for calling s2n_negotiate until getting success or failure (s2n_blocked_status != S2N_BLOCKED_APPLICATION)

The next time when the client calls s2n_negotiate.
This could be based on epoll events which is owned by the client, the client is responsible for calling s2n_negotiate until getting success or failure (s2n_blocked_status != S2N_BLOCKED_APPLICATION)

I see, thank you for the clarifications. So essentially we'll need to do something like

s2n_blocked_status blocked;
int handshake_result_code = s2n_negotiate(s2n_conn, &blocked);

switch(handshake_result_code) {
      case S2N_BLOCKED_ON_APPLICATION_INPUT:
           // we also may register a callback to cache lookup promises to wake up earlier
           schedule_retry(....);
           break;
      .....
}

schedule_retry

Your code looks good for scheduled retry. For performance reason, notification might be better than scheduled retry. If you could get a notification (either an event or a callback) when fetching from cluster completes, you could call s2n_negotiate immediately. Please note that s2n_connection by default "can be operated on by up to two threads at a time, one sender and one receiver". You might need a lock if the callback is called from another thread.

For performance reason, notification might be better than scheduled retry.

Definitely a completion callback needs to be set to promises. However, you still need to schedule a retry to handle the timeout case. I.e.
....
lookup_promise = cache_client->async_get(cache_client, key, key_size);
lookup_promise->then(send_event, &context);
....

while(active) {
    wait_for_events(&context); // new connections, promise callbacks, timeouts
    ....
    int handshake_result_code = s2n_negotiate(s2n_conn, &blocked);
    switch(handshake_result_code) {
         case S2N_BLOCKED_ON_APPLICATION_INPUT:
              schedule_retry(TIME_OUT, send_event, &context);
              break;
    ....
}

Please note that s2n_connection by default "can be operated on by up to two threads at a time, one sender and one receiver". You might need a lock if the callback is called from another thread.

A very important note, thank you for pointing this out. I think that the code above should handle that as s2n_negotiate is called only from the receiver thread.


detect touch mobile device using cnc-tskit

... once https://github.com/czcorpus/cnc-tskit/issues/20 is ready

Search for new AppServices in page/index


once the search bar is closed it should be clear the next time it's opened.

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version:
Operating system and its version:
PDF.js version:1.4
Is a browser extension:

Steps to reproduce the problem:

search for any text in the pdf-viewer
2.close the search bar and re-open it

What is the expected behavior? (add screenshot)
What went wrong? (add screenshot)
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Hey @timvandermeij , I would like to work on this issue . Can this issue be assigned to me?

@timvandermeij  can you please tell me the location in the repository where I can find the JS code written for the search button? Because I was unable to find it in the viewer.js file.

Never mind. I found the JS file.

once the search bar is closed it should be clear the next time it's opened.

This is not how the native findbar works in Firefox, so why should the PDF.js (built-in) findbar behave any different here!? Having different behaviours for the native/built-in findbars seems very confusing, and will result in a bad UX; hence I'd suggest that WONTFIX is an appropriate resolution here.

once the search bar is closed it should be clear the next time it's opened.

This is not how the native findbar works in Firefox, so why should the PDF.js (built-in) findbar behave any different here!? Having different behaviours for the native/built-in findbars seems very confusing, and will result in a bad UX; hence I'd suggest that WONTFIX is an appropriate resolution here.

Good point. Now that I'm looking at this again and having tried both the native find bar and the PDF.js find bar, it indeed wouldn't make sense for them to differ in behavior, so let's close this.


How can this error be solved !

FAILURE: Build failed with an exception.

Where:
Build file 'C:\Users\hp\AndroidStudioProjects\abc\android\app\build.gradle' line: 26

What went wrong:
A problem occurred evaluating project ':app'.

Could not load compiled classes for script 'C:\flutter\packages\flutter_tools\gradle\flutter.gradle' from cache.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 52s
Exception: Gradle task assembleDebug failed with exit code 1

@omarakl This issue looks like a duplicate of https://github.com/flutter/flutter/issues/107523. Closing as such


Docker Swarm support

Hi,
Does Longhorn support Docker Swarm?

Not currently, we support Kubernetes only ATM

Does Longhorn support Docker Swarm? in 2021?

Swarm Cluster Volume Support with CSI Plugins
https://github.com/moby/moby/issues/39624

Swarm Cluster Volume Support with CSI Plugins moby/moby#39624

That support was now released as part of Docker 23.0.0 but Longhorn is one of those CSI plugins which cannot be easily made working with that. More info on my repo https://github.com/olljanat/csi-plugins-for-docker-swarm


[WIP] memcached: passthru filter for memcached

Mega WIP Pr.
For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

@maximebedard I'm going to leave this unassigned until you're ready for someone to review.
/wait

Is there anyone still working on this feature?

Yes, I still have a WIP branch that I plan on getting ready for reviews shortly.

hello @maximebedard , has this PR got any progress?

Bumping this as well. =)


NEW DB Driver defaults to PDO

We should default to the PDO driver out of the box as it has better performance.
This wasn't done in 3.x for backwards-compat reasons.

Can we do a tiny bit of extension detection, maybe copy a bit of code from https://github.com/silverstripe/silverstripe-framework/blob/master/_register_database.php#L17
$default = (class_exists('PDO') && in_array('mysql', PDO::getAvailableDrivers()))
    ? 'MySQLPDODatabase'
    : 'MySQLDatabase';

If we don't, then we should add ext-pdo to https://github.com/silverstripe/silverstripe-framework/blob/master/composer.json.

We don't currently check that mysqli extension is installed when we default to mysqli so I'm not sure we need to do the same for PDO. If someones not running with the recommended extensions they'll need to make manual adjustments.
A site can run on mysqli or PDO so neither should to be enforced in the composer.json - perhaps they could be recommendations?

Ok, then don't make it a requirement, and use the environment checking as suggested?

What happens if neither PDO nor mysqli are available? At the moment we don't check the default is available, if someone doesn't have it they need to make an adjustment.

It also feels like we're tightly coupling our knowledge of what libraries the classes are using when this should only be information the classes themselves need to know about...

We'd also slow down sites for people relying on the default by calling in_array on every request

What happens if neither PDO nor mysqli are available?

Nice error message?

We'd also slow down sites for people relying on the default by calling in_array on every request

You can rewrite the code so it only runs if driver isn't selected.

I invoke the mantra "if it's an improvement it should be merged".
This PR changes nothing except that PDO is the preferred MySQL driver, which was my intend.
Adding checks that PDO/mysqli are installer and/or that PDO has the mysql driver available and/or that there's a nice error if no compatible lib is available are all nice additions but ultimately outside the scope of what I intend to achieve which is PDO drivers are our default option.

It was an improvement so I merged it.


Add an option to restrict tweets to a preferred language

@andela-doladosu
Language support is now apart of: https://github.com/Preposterous/twitter-lite/pull/18
Thanks!


Feature Request: SSH login and access settings for debugging

Is your feature request related to a problem? Please describe.
In Azure Container Apps (ACA) integrated Custom VNet, ACA wants to check if communication between other Azure services (e.g. Azure Database) is via VNet.
Describe the solution you'd like.
For that my purpose, I want to feature that use SSH login (like Azure Container Apps for Linux) to ACA.
Describe alternatives you've considered.
A clear and concise description of any alternative solutions or features you've considered.
I have installed curl and phpmyadmin on ACA and checked it, but it is necessary to prepare a container for communication check, and it takes time to check the communication status from Log Analytics.

We're working on the ability to connect to a console in a container #54. You'll be able to connect to the container and run commands from the Azure portal, Azure CLI, and other supported clients. Would that work?

Closing as duplicate of #54.


[1.32.4] Picker of sap.m.DateRangeSelection is not displayed instead throws "Uncaught TypeError: D.calculateWeekNumber is not a function"

[1.32.4] Picker of sap.m.DateRangeSelection is not displayed instead throws "Uncaught TypeError: D.calculateWeekNumber is not a function" at CalenderUtils.js
See https://openui5.hana.ondemand.com/explored.html#/sample/sap.m.sample.DateRangeSelection/preview
Note that https://openui5.hana.ondemand.com/explored.html?sap-ui-debug=true#/sample/sap.m.sample.DateRangeSelection/preview works so this seems a problem in minification

is this only happening for specific agents?
I tried IE11 und Chrome 46 and could not reproduce.

Sorry, I just updated my Desktop Chrome to Version 46.0.2490.80 m and the error does not happen anymore, also does not happen in IE and Firefox, but I had this at least since openui5.hana.ondemand.com started delivering 1.32.x .


Error screen after npm start (typescript)

Describe the bug
I don't really know if this can be a create-react-app issue, but...
I tried to create a typescript app, and when I do npm start it shows me this error:
<EMAIL_ADDRESS>TypeScript error in C:/.../typescript/test/node_modules/@types/testing-library__react/node_modules/pretty-format/build/index.d.ts(7,13):
'=' expected.  TS1005

     5 |  * LICENSE file in the root directory of this source tree.
     6 |  */
  >  7 | import type * as PrettyFormat from './types';
       |             ^
     8 | /**
     9 |  * Returns a presentation string of your `val` object
    10 |  * @param val any potential JavaScript object

I tried creating a normal javascript and runing, and it worked ok
Did you try recovering your dependencies?
I installed with create-react-app test --template typescript --use-npm, tried deleting and redownloading dependencies, deleted package-lock.json, but nothing seems to fix it...
I also tried to put "esModuleInterop": false and "allowSyntheticDefaultImports": false, but didn't worked.
Which terms did you search for in User Guide?
I actually looked in the issues tab to see if it happened with someone else.

import type as PrettyFormat from
TypeScript error in '=' expected
TypeScript error in '=' expected PrettyFormat

Environment
Environment Info:
 current version of create-react-app: 3.4.1
 running from C:\Users\...\AppData\Roaming\npm\node_modules\create-react-app

System:
   OS: Windows 10 10.0.18363
   CPU: (4) x64 Intel(R) Core(TM) i7-7500U CPU @ 2.70GHz
 Binaries:
   Node: 12.16.1 - C:\Program Files\nodejs\node.EXE
   Yarn: 1.22.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
   npm: 6.13.4 - C:\Program Files\nodejs\npm.CMD
 Browsers:
   Edge: 44.18362.449.0
   Internet Explorer: 11.0.18362.1
 npmPackages:
   react: ^16.13.1 => 16.13.1
   react-dom: ^16.13.1 => 16.13.1
   react-scripts: 3.4.1 => 3.4.1
 npmGlobalPackages:
   create-react-app: Not Found

Steps to reproduce

create-react-app test --template typescript --use-npm
cd test
npm start

Expected behavior
It should just start the normal "boilerplate"
Actual behavior
Shows the normal react error screen.
Reproducible demo
https://github.com/mateusbds/test-repo

The problem is caused by pretty-format/build/index.d.ts so the temporary solution is to change
import type * as PrettyFormat from './types';

to
import * as PrettyFormat from './types';

Path : @types/testing-library__react/node_modules/pretty-format/build/index.d.ts

I'm closing this issue as I found the solution here https://github.com/facebook/create-react-app/issues/8717#issuecomment-604145928


Unable to find tunctl binary when running "runqemu qemux86 core-image-ros-groovy-qemux86-running-roscore"

Hi I'm trying to set up the qemu environment that you guys are using for development.
@koenkooi already included meta-ros in the Angstrom distribution so you just need to follow this instructions http://www.angstrom-distribution.org/building-angstrom to set up an Angstrom environment.
Then, as specified in the README:
bitbake core-image-ros-groovy-qemux86-running-roscore

/home/victor/Escritorio/GSOC/setup-scripts/sources/bitbake/bin:/usr/lib/lightdm/lightdm:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
Parsing recipes: 100% |##############################################################################################################################################################################| Time: 00:01:31
Parsing of 2612 .bb files complete (0 cached, 2612 parsed). 3105 targets, 123 skipped, 0 masked, 0 errors.
WARNING: No recipes available for:
  /home/victor/Escritorio/GSOC/setup-scripts/sources/meta-handheld/recipes-core/udev/udev_164.bbappend
  /home/victor/Escritorio/GSOC/setup-scripts/sources/meta-intel/meta-fri2/recipes-core/tiny-init/tiny-init.bbappend

Build Configuration:
BB_VERSION        = "1.17.0"
TARGET_ARCH       = "arm"
TARGET_OS         = "linux-gnueabi"
MACHINE           = "beaglebone"
DISTRO            = "angstrom"
DISTRO_VERSION    = "v2012.12"
TUNE_FEATURES     = "armv7a vfp neon cortexa8"
TARGET_FPU        = "vfp-neon"
meta-angstrom     = "angstrom-v2012.12-yocto1.3:7539ad5baf71b8906826d96db0bd4f3c7c305d42"
meta-oe
meta-systemd
meta-efl
meta-gpe
meta-gnome
meta-xfce
meta-initramfs
toolchain-layer
meta-multimedia
meta-networking
meta-webserver
meta-ruby         = "angstrom-staging-yocto1.3:58f77c0b1d303a5fcc29f3ac57829b006ae679ca"
meta-kde          = "master:316d9c27e72cfebc7618268f8766624ee3f5e18e"
meta-opie         = "master:c242efc6ca55772a88602f1f8f94dff697b38430"
meta-java         = "master:75dee76c7cfbf1418e5c581f50bb57b3282013f7"
meta-browser      = "danny:de6eb680c0efe0df9ed12526380a10d3aab4359a"
meta-mono         = "master:adf47f078dbf788a80ff1398a15a4fd5a334d320"
meta-ros          = "master:430f67cdd8c3b54270a08a2921083843406b4f74"
common-bsp        = "master:b81ea32039c5dc80c47f49a9f1a09125225f9a5c"
meta-ti           = "angstrom-staging-yocto1.3:d74487009d97649586b561a0355fdf1871cc6188"
meta-efikamx      = "master:07f2a5c777b4f866985735303bc6d696592760a9"
meta-nslu2        = "master:8948458fe3ec2b3c713b2a13a87123b64e22b0d9"
meta-htc
meta-nokia
meta-openmoko
meta-palm         = "danny:924338ee2d44987da71088da31e1cd31d3a9dfa9"
meta-handheld     = "danny:d641e71a7c09cd24c72e690ad1981e49da612bcd"
meta-intel
meta-sugarbay
meta-crownbay
meta-emenlow
meta-fri2
meta-jasperforest
meta-n450         = "danny:98342e9e685ac8066f7fe2517fd382127a286648"
meta-allwinner    = "danny:3603ce61c075d2f6a23dbe0bd71ee604f7eb24ea"
meta-raspberrypi  = "danny:1ad98cf651054982c1a69576c1663285f0090034"
meta-minnow       = "danny:38deaab4cb52acc2232d5b424e689b7c20450dfe"
meta-linaro       = "danny:d0558fa4304592622423582f8b464e1c22cfe6f1"
meta              = "angstrom-staging-yocto1.3:d52ac598829dc35c6eac8ce404424751309ca60d"

NOTE: Resolving any missing task queue dependencies
NOTE: Preparing runqueue
NOTE: Executing SetScene Tasks
NOTE: Executing RunQueue Tasks
NOTE: Tasks Summary: Attempted 3254 tasks of which 2555 didn't need to be rerun and all succeeded.

and finally, launch qemu:
runqemu qemux86 core-image-ros-groovy-qemux86-running-roscore

runqemu qemux86 core-image-ros-groovy-qemux86-running-roscore
Assuming core-image-ros-groovy-qemux86-running-roscore really means /home/victor/Escritorio/GSOC/setup-scripts/build/tmp-angstrom_v2012_12-eglibc/deploy/images/core-image-ros-groovy-qemux86-running-roscore-qemux86.ext3

Continuing with the following parameters:
KERNEL: [/home/victor/Escritorio/GSOC/setup-scripts/build/tmp-angstrom_v2012_12-eglibc/deploy/images/bzImage-qemux86.bin]
ROOTFS: [/home/victor/Escritorio/GSOC/setup-scripts/build/tmp-angstrom_v2012_12-eglibc/deploy/images/core-image-ros-groovy-qemux86-running-roscore-qemux86.ext3]
FSTYPE: [ext3]
Setting up tap interface under sudo
Error: Unable to find tunctl binary in '/home/victor/Escritorio/GSOC/setup-scripts/build/tmp-angstrom_v2012_12-eglibc/sysroots/i686-linux/usr/bin'

Can anybody reproduce this? Anybody has some clue about this tunctl binary?

I close this issue as is it probably outdated by now and is only weakly related to the meta-ros layer.


fix: open footer integrations link as new tab incorrectly opens swap page

This fix makes the footer use router links when the href doesn't start with HTTP like in the header/nav. This does not fix the issue of opening 'integrations' in a new tab opening the swap page, but that's a different issue.

I also fixed the integrations link in the sidebars, except for the DAO app, because that app doesn't have an intgerations link at all. Need to create a task for that perhaps.


processes RGBD input streams

Hi,
I noticed on the to-do list that you are planning to release an online demo that processes RGBD input streams. This is very exciting, and I'm eagerly looking forward to it. Do you have a rough timeline for when this feature might be available?
Best regards,
Zhening

Hi, thanks for your interest! We will release the Demo code in the next month.


Can't pickle <function cli at 0x105140a60>: it's not the same object as pyspider.run.cli

pyspider version: 0.3.10 & 0.4.0
Operating system: macos 14.0 (M1)
Start up command: pyspider

Can't pickle <function cli at 0x105140a60>: it's not the same object as pyspider.run.cli
$ pyspider
Traceback (most recent call last):
File "/Users/haibing6/Library/Python/3.9/bin/pyspider", line 8, in
sys.exit(main())
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/pyspider/run.py", line 839, in main
cli()
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/click/core.py", line 1130, in call
return self.main(*args, **kwargs)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/click/core.py", line 1055, in main
rv = self.invoke(ctx)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/click/core.py", line 1635, in invoke
rv = super().invoke(ctx)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/click/core.py", line 1404, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/click/core.py", line 760, in invoke
return __callback(*args, **kwargs)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/click/decorators.py", line 26, in new_func
return f(get_current_context(), *args, **kwargs)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/pyspider/run.py", line 177, in cli
ctx.invoke(all)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/click/core.py", line 760, in invoke
return __callback(*args, **kwargs)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/click/decorators.py", line 26, in new_func
return f(get_current_context(), *args, **kwargs)
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/pyspider/run.py", line 531, in all
threads.append(run_in(ctx.invoke, phantomjs, **phantomjs_config))
File "/Users/haibing6/Library/Python/3.9/lib/python/site-packages/pyspider/libs/utils.py", line 68, in run_in_subprocess
thread.start()
File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/process.py", line 121, in start
self._popen = self._Popen(self)
File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/context.py", line 224, in _Popen
return _default_context.get_context().Process._Popen(process_obj)
File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/context.py", line 284, in _Popen
return Popen(process_obj)
File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/popen_spawn_posix.py", line 32, in init
super().init(process_obj)
File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/popen_fork.py", line 19, in init
self._launch(process_obj)
File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/popen_spawn_posix.py", line 47, in _launch
reduction.dump(process_obj, fp)
File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/reduction.py", line 60, in dump
ForkingPickler(file, protocol).dump(obj)
_pickle.PicklingError: Can't pickle <function cli at 0x105140a60>: it's not the same object as pyspider.run.cli
How to resolve this issue?

me+1,
Is there anything I can do to fix this?

+1

+1


Can we offer code review for scientific code?

How would this work?
A few thoughts:

maybe a list of people with domain expertise (e.g. biology, mathematics, physics) and language expertises?
is this the right place to do this? Is there somewhere more appropriate?

Please comment with your thoughts how we could enact this!
This issue suggested by @leotrs

Before taking it up on ourselves to offer this to the wider public, I want to run a local experiment. In the following weeks I want to offer code reviews for people at my lab. I'll see how that goes and report the results here. With that experience we can move on to thinking about the big picture!

I really like the idea of scientific code reviewers and having a list of people with domain expertise.
How about pairing people according to discipline and programming language, so that they can review each other's code?
I feel that giving some good incentive to review will be important here, as it takes a long time if you want to do it well and it's a big responsibility!

should we make a list of people who have domain expertise and programming
expertise of various sorts? linked to orcid somehow, maybe. Is that even a
feasible thing or insanity?
On Thu, May 10, 2018 at 7:13 PM Athina Tzovara<EMAIL_ADDRESS>wrote:

I really like the idea of scientific code reviewers and having a list of
people with domain expertise.
How about pairing people according to discipline and programming language,
so that they can review each other's code?
I feel that giving some good incentive to review will be important here,
as it takes a long time if you want to do it well and it's a big
responsibility!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yochannah/code-is-science/issues/19#issuecomment-388138086,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI14jqzfWNdFPPt4LrTru4xtcbKKEKApks5txINHgaJpZM4QO4pQ
.

Orcid is a good resource, but I guess it might be hard to mine information from there about the coding that someone did. Maybe if there's a spreadsheet people can write down themselves which programming language and which domain/techniques are within their comfort zone?
I don't know if it's insanity, but I would personally love to have an extra pair of eyes review my code just before submitting work to a journal for example. And I'd be more than happy to take the time and review someone else's code in exchange :)

ooooh, like a group of code review volunteers and / or a resource journals
could use to source reviewers? I like it!!
we could start with a google form for input, I guess. What columns would we
need?
mandatory:

name
email address
domain expertises
language expertises

optional(?)

orcid
github user

anything else?
On Thu, May 10, 2018 at 7:25 PM Athina Tzovara<EMAIL_ADDRESS>wrote:

Orcid is a good resource, but I guess it might be hard to mine information
from there about the coding that someone did. Maybe if there's a
spreadsheet people can write down themselves which programming language and
which domain/techniques are within their comfort zone?
I don't know if it's insanity, but I would personally love to have an
extra pair of eyes review my code just before submitting work to a journal
for example. And I'd be more than happy to take the time and review someone
else's code in exchange :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yochannah/code-is-science/issues/19#issuecomment-388141623,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI14jgQRFysPr3NatwnHX3mZIt3jDtE3ks5txIYPgaJpZM4QO4pQ
.


feat(note): add Tonality major and minor getters

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
483

Relevant Lines:
483

💛  - Coveralls


Add zh_tw translations; Improve translation help wanted strings in settings page

説明 / Description
Update New Translations for Traditional Chinese support
チェックリスト / Checklist

MUST

[x] npm test passed
[x] npm run lint was applied without warnings
[x] changes of /docs/webapp not included (except release branch)
[x] console.log not included (except script file)

MUST for Outside Contributor

[x] understand プロジェクトへの関わり方について

RECOMMENDED (it depends on what you change)

[ ] unit test added/updated
[x] i18n

Summary by CodeRabbit

New Features

Added translation assistance messages in English, Japanese, and Traditional Chinese.
Introduced notifications for app restart requirements after changing the language.

Localization

Updated various text strings for better clarity in Traditional Chinese.

UI Updates

Displayed new notes in the settings dialog regarding translation help and restart requirements based on the selected language.

Thanks!


Fullscreen without _NET_WM_FULLSCREEN_MONITORS

Some window managers do not support _NET_WM_FULLSCREEN_MONITORS.
In that case multimonitor fullscreen does not properly work, so
add a path resizing the window over all screens instead.
Based on @erbth pull request, adding proper X11 atom checks.
Fixes #3439, #2707

@T3N4K4 can you check if that removes the need of your /openbox hack ?

@akallabeth BTW you need a rebase for this one

@hardening Don't know for @T3N4K4 but tests with openbox on my debian jessie show a working multimonitor fullscreen (as opposed to the current nightly build)

@hardening @akallabeth Sorry for the delay, tight schedule this week.
Seems to be working for me.


Upgrading from 0.8.1 to 0.23.3:  module Plug.Swoosh.MailboxPreview is not available

When I try to compile after updating the package to 0.23.3 from 0.8.1, I get an error saying that the Plug.Swoosh.MailboxPreview module is  not available.  As far as I can tell everything is set up in the application correctly.  The error comes from the router.ex file:
== Compilation error in file lib/refleq_web/router.ex == ** (UndefinedFunctionError) function Plug.Swoosh.MailboxPreview.init/1 is undefined (module Plug.Swoosh.MailboxPreview is not available)     Plug.Swoosh.MailboxPreview.init([base_path: "/dev/mailbox"])
Router:

scope "/dev" do
  pipe_through [:browser]
  forward "/mailbox", Plug.Swoosh.MailboxPreview, base_path: "/dev/mailbox"
end

Seems as though I needed to add {:plug_cowboy, "~> 1.0"} to my mix.exs file for this to work.

I already had {:plug_cowboy, "~> 2.5"} in my mix.exs deps, and I was getting that error.   I searched for Plug.Swoosh.MailboxPreview and the first hit was:
HexDocs
Plug.Swoosh.MailboxPreview — Swoosh v1.14.2

So, I looked up Swoosh, and it contains the module Plug.Swoosh.MailboxPreview, so I added Swoosh as a dependency in mix.exs:
defp deps do
    [
      {:phoenix, "~> 1.7.10"},
      {:phoenix_html, "~> 3.3"},
      {:phoenix_live_reload, "~> 1.2", only: :dev},
      {:phoenix_live_view, "~> 0.20.1"},
      {:floki, ">= 0.30.0", only: :test},
      {:phoenix_live_dashboard, "~> 0.8.2"},
      {:esbuild, "~> 0.8", runtime: Mix.env() == :dev},
      {:tailwind, "~> 0.2.0", runtime: Mix.env() == :dev},
      {:telemetry_metrics, "~> 0.6"},
      {:telemetry_poller, "~> 1.0"},
      {:gettext, "~> 0.20"},
      {:jason, "~> 1.2"},
      {:plug_cowboy, "~> 2.5"},
      {:auction, in_umbrella: true},
      {:swoosh, "~> 1.14.3"},
      {:hackney, "~> 1.20.1"}
    ]
  end

Then I did mix.deps get, then I started up my server: mix phx.server.  Yeah, the warning:
 warning: Plug.Swoosh.MailboxPreview.init/1 is undefined (module Plug.Swoosh.MailboxPreview is not available or is yet to be defined)
    │
 41 │       forward "/mailbox", Plug.Swoosh.MailboxPreview
    │       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/auction_web/router.ex:41

was gone...but it was replaced by this:
00:29:05.113 [notice] Application runtime_tools exited: :stopped
** (Mix) Could not start application swoosh: exited in: Swoosh.Application.start(:normal, [])
    ** (EXIT) an exception was raised:
        ** (RuntimeError) missing hackney dependency
            (swoosh 1.14.3) lib/swoosh/api_client/hackney.ex:26: Swoosh.ApiClient.Hackney.init/0
            (swoosh 1.14.3) lib/swoosh/api_client.ex:43: Swoosh.ApiClient.init/0
            (swoosh 1.14.3) lib/swoosh/application.ex:7: Swoosh.Application.start/2
            (kernel 9.1) application_master.erl:293: :application_master.start_it_old/4

Wtf, mix deps.get ??!  I checked the Swoosh docs, and sure enough you have to add a hackney dependency.  So I did this:
defp deps do
    [
      {:phoenix, "~> 1.7.10"},
      {:phoenix_html, "~> 3.3"},
      {:phoenix_live_reload, "~> 1.2", only: :dev},
      {:phoenix_live_view, "~> 0.20.1"},
      {:floki, ">= 0.30.0", only: :test},
      {:phoenix_live_dashboard, "~> 0.8.2"},
      {:esbuild, "~> 0.8", runtime: Mix.env() == :dev},
      {:tailwind, "~> 0.2.0", runtime: Mix.env() == :dev},
      {:telemetry_metrics, "~> 0.6"},
      {:telemetry_poller, "~> 1.0"},
      {:gettext, "~> 0.20"},
      {:jason, "~> 1.2"},
      {:plug_cowboy, "~> 2.5"},
      {:auction, in_umbrella: true},
      {:swoosh, "~> 1.14.3"},
      {:hackney, "~> 1.20.1"}
    ]
  end

Ta da!  No more warnings or errors.  Good luck.


Sqlite: dbContext.Update() and dbContext.Attact() get information from database, not from POCO entity

I have a parent and a child class.
Parent
{
    long IDParent;
    long IdType;
}

Child
{
    long IDParent;
    long IdType;
}

The primary key of the parent is an autonumeric and the primary key of the child is not an autonumeric, it is set the same key than the parent.
The initial values in the database, are: myParent.IdType = 1; myCHild.IdType = 2 and I wanted to update the data in the database, I am trying this code:
Child myChildUpdated = new Child();
myChild.IDParent = 4;
myChild.dType = 3;

Parent myParentUpdate = new Parent();
myParent.IDParent = 4;
myParent.IDType = 6;

myParent.Child = myChild;
myChild.Parent = myParent;

Entities myDbContext = new Entities();
myDbContext.Update(myParent);

At this time, myParent.IdType = 1 and myChild.IdType = 3. So myParent get the data from database but the child not, so the data is not consistent.
The state of both entities are "modified".
I have also try myDbContext.Attach(myParent) but it has the same behavior, the myParent get the information of the database and the child not. In this case the state of both are unchanged.
At the end I have tried myDbContext.Entry(myParent).State = State.Modified. In this case myParent has the expected value, 6, not the value from database. But in this case the child is not add to the dbContext, I have to add it manually.
I guess that the expected behavior of dbContext.Update() and dbContext.Attached() it is to add entities to the context keeping the information of the entities, not going to the database to get the data, because I want to save a trip to the database, if not, I would send a query to the database and I would need these methods.
Anyway, as I commented, the data is not consistent because the parent has the data from database and the child keeps its data.
What I want to do is to add the parent and the child to the dbContext keeping their data in modified state to update the data that there are in the database.
Thanks.

EF Team Triage: This issue is lacking enough information for us to be able to effectively triage it. In particular, it is missing the following information requested in the new issue template. Can you please provide this information?

Steps to reproduce

Ideally include a complete code listing that we can run to reproduce the issue.
Alternatively, you can provide a project/solution that we can run.
Also, please provider the product and provider versions you are using.
BTW we're not just doing this to be mean :smile:... we get a lot traffic on this project and it takes time to attempt to reproduce an issue based on fragments of information. In addition, our attempt is often unsuccessful as the exact conditions required to hit the issue are often not explicitly included in the code provided. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we ask that folks give us a self-contained way to reproduce an issue.
For a guide on submitting good bug reports, read Painless Bug Tracking.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.

At the end the problem was mine, I had a navigation propierty that isn't consistent with the ID, so in the update() EF set the ID with the ID that there is in the navigation property.
So the behavior is correct and it was my problem.
Thanks.


Screen: Confirmed New Infections (EXPOSUREAPP-4510)

Implementation of Infections Card

The primary label with the date is not correct. It should be Gestern/Yesterday

The primary label with the date is not correct. It should be Gestern/Yesterday

The primary label with the date is not correct. It should be Gestern/Yesterday

The data are clearly wrong, however, I implemented only card functionality and design in this PR. The card shows data it gets and since it worked with mock data, I guess the issue could be somewhere deeper.

The primary label with the date is not correct. It should be Gestern/Yesterday

The data are clearly wrong, however, I implemented only card functionality and design in this PR. The card shows data it gets and since it worked with mock data, I guess the issue could be somewhere deeper.


Better error for outside references

Motivated by @teto in https://github.com/NixOS/nixpkgs-check-by-name/issues/64#issuecomment-2308447186, improves the error messages when a path outside a package directory is referenced:
- pkgs/by-name/aa/aa: File package.nix at line 1 contains the path expression "../." which may point outside the directory of that package.
  This is undesirable because it creates dependencies between internal paths, making it harder to reorganise Nixpkgs in the future.
  Alternatives include:
  - If you are creating a new version of a package with a common file between versions, consider following the recommendation in https://github.com/NixOS/nixpkgs/tree/master/pkgs/by-name#recommendation-for-new-packages-with-multiple-versions.
  - If the path being referenced could be considered a stable interface with multiple uses, consider exposing it via a `pkgs` attribute, then taking it as a attribute argument in package.nix.
  - If the path being referenced is internal and has multiple uses, consider passing the file as an explicit `callPackage` argument in `pkgs/top-level/all-packages.nix`.
  - If the path being referenced is internal and will need to be modified independently of the original, consider copying it into the pkgs/by-name/aa/aa directory.

The This is undesirable because it creates dependencies between internal paths, making it harder to reorganise Nixpkgs in the future. part is nice to have. As for the rest, it depends how you prefer to manage the project but one solution is to do like shellcheck and reference a wiki page like "https://www.shellcheck.net/wiki/SC2155" which can be edited without needing another release.

Yeah I'd really like that better, maybe let's keep it simpler for now though

@philiptaron I think it's kind of important for multiple failures: https://github.com/NixOS/nixpkgs-check-by-name/blob/better-error-outside-reference/tests/multiple-failures/expected

The after release action didn't fire automatically yet again, just like before

I think this is the culprit: https://github.com/orgs/community/discussions/54574#discussioncomment-10119733


select range add class on middle values on hover

On select range mode, I'd like to add a class to middle values between first date (already selected) and the new value (being selected). To show the user the resulted range.
Something like that:

What do you think ?

I like the idea- is that a Bootstrap class or something you created? It
looks nicer than giving them all a primary style.
The end date style is a little subtle, though.
On 6:40pm, Sat, Sep 26, 2015 Breno Vieira<EMAIL_ADDRESS>wrote:

On select range mode, I'd like to add a class to middle values between first
date (already selected) and the new value (being selected). To show the
user the resulted range.
Something like that:
[image: screen shot 2015-09-26 at 19 35 02]
https://cloud.githubusercontent.com/assets/7158650/10120179/bd5aa1c0-6485-11e5-9328-f4dcc02241f1.png
[image: screen shot 2015-09-26 at 19 34 48]
https://cloud.githubusercontent.com/assets/7158650/10120180/cb2f7d48-6485-11e5-9467-52543f764c2b.png
What do you think ?
—
Reply to this email directly or view it on GitHub
https://github.com/spongessuck/gm.datepickerMultiSelect/issues/24.

It is actually not mine. hahaha
I had the idea, googled for datepicker range and found it here: http://www.daterangepicker.com/
But there are many sites of airline companies that have this, like http://www.united.com (United Airlines).
I've already seen this "feature" in some sites using jQuery datepicker, but I didn't find one using bootstrap.
This light blue is actually on hover (could be after selected also), but it pretends the selected range before the user selects.
Tomorrow morning I'll give it a try =]

I'll take a look at this later today.


Converting from SimpleShell to Shell?

HI,
When trying to convert GadgetStoreApp to use Shell instead of SimpleShell I am receiving the error  "The property 'ShellContents' was not found in type 'AppShell'" at AppShell.xaml with this line of code:
BindableLayout.ItemsSource="{Binding Source={x:Reference thisShell}, Path=ShellContents}"
Is it possible to user Shell instead of SimpleShell please?
Thanks,
AG

Hi @asafgo,
why would you like to do that? You can use normal Shell, but you will lose the ability to create custom navigation controls from scratch. ShellContents is a SimpleShell property, as are some other properties (such as the Content property) which are used in this sample app.


[Evals] Consistent testCaseId, support patch updates

In this PR:

Consistent and persistent testCaseId throughout evalsland. This facilitates partial dataset update.
UpdateDataset now supports partial updates to dataset.
Refactor EvalFlowInput -> EvalInferenceInput
Minor changes to runEvaluation API
make EvaluatorFn always get testCaseId, enforcing consistency with testCaseId in plugins.
JSDoc updates + tests

Checklist:

[x] Tested (manually, unit tested)

@ssbushi FYI, whenever making changes to tools-common types that the Developer UI relies on, you need to follow these steps to update package/genkit-ai-tools-common.tgz. Currently, the UI has errors compiling on next because of this.

Sorry about that. I had https://github.com/FirebasePrivate/genkit-ui/pull/1075 in the works but did not push.

No worries, looks good. Thanks!


Add files via upload

Day 5 Lab: The Beginning of 5 branches [Problems 1 through 5, all building upon the previous set]

Day 5 Lab


Linux 1GB RAM: virtual memory exhausted: Cannot allocate memory

$ ./bootstrap-vcpkg.sh

I think it needs to take available memory into account before doing parallel building.

This issue hasn’t been updated in a year; if it is still an issue, please reopen this issue.


Fixed typo for assets version update task

Hi, i propose this fix because with:
file_content = "parameters:\n    assets_version: #{assets_version}"

capifony ran:
echo 'parameters:\\\n    assets_version: nyop6h' |  tee /path/to/app/config/assets_version.yml

with a resulting error for symfony yaml parser in the assets_version file.

looks like tests are broken now

I ran the suite on my local machine and it looks great, it seems travis temporary cannot download dependencies for build, no? 😀
Thanks for response


import neptune instead of import neptune.new

What does this PR do?
Updates the importing of neptune according to https://docs.neptune.ai/setup/neptune-client_1-0_release_changes/
This probably breaks NeptuneLogger with neptune-client<1.0.0. Should neptune-client>=1.0.0 be added as a requirement?
Fixes #16887

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

cc @Blaizzy for review too

cc @Blaizzy for review too

lgtm :)

Hi @wouterzwerink and @carmocca
Thanks for the initiative!
However, this PR solves part of the problem but not all.
There are a few more issues precisely:

Users still using the legacy client(<=1.0.0) will have their workflows broken.
The new client no longer supports automatically converting unsupported data types to strings. (Docs: https://docs.neptune.ai/api/utils/#stringify_unsupported)

We are preparing a PR to address the breaking changes and ensure backward compatibility.
So, could you please revert this PR?
Here is the draft PR we have been working on for a few weeks:
https://github.com/Lightning-AI/lightning/pull/16761/files

@Blaizzy sure, reverting in #16898.

@Blaizzy you can incorporate those changes and backwards compatibility in #16761 after rebasing this PR's changes. Without needing to revert

@carmocca sure, no problem!

Thanks, guys!
I really appreciate the initiative, @wouterzwerink!


Non-uniform scaling results in wrongly shaped SVG strokes

<svg viewBox="0 0 16 16" xmlns="http://www.w3.org/2000/svg"><path d="M8 0l4 1-4 1-4-1Z" fill="none" stroke="#def" stroke-linejoin="round" transform="matrix(1, 0, 0, 5, 0, 3)"/></svg>

Expected:

Hi @hermet
This is tvg code for this issue.
Hope this help you.
    auto shape1 = tvg::Shape::gen();
    shape1->moveTo(8, 0);
    shape1->lineTo(12, 1);
    shape1->lineTo(8, 2);
    shape1->lineTo(4, 1);
    shape1->close();

    tvg::Matrix m = {50, 0, 0,
                     0, 250, 150,
                     0, 0, 1};

    shape1->transform(m);

    shape1->strokeFill(255, 0, 0);
    shape1->strokeJoin(tvg::StrokeJoin::Round);
    shape1->strokeWidth(1);

    if (canvas->push(std::move(shape1)) != tvg::Result::Success) return;

Same issue: https://github.com/thorvg/thorvg/issues/954

Same issue: #954

#954 still contains bugs that need to be solved


MIT License?

Hi,
This looks really interesting!
I understand CertiCoq is open source and distributed under the MIT license as stated in the CertiCoq repo and that VeriFFI is "part of [the] CertiCoq project". Does that mean that this VeriFFI repository is also under the MIT license? If so, it would be helpful if an appropriate LICENSE.md file was added here too.
Thanks :)

Thanks for your interest! Added MIT license.

Great, thanks 😃


Install error on OSX

Tried to install it but got this error...
Error detected while processing ~/.vim/bundle/LeaderF/plugin/leaderf.vim:
line   82:
E806: using Float as a String
E116: Invalid arguments for function <SNR>62_InitVar
line   88:
E806: using Float as a String
E116: Invalid arguments for function <SNR>62_InitVar```

has('python') return 1
has('python3') return 0
Python is at version 2.7.13

I updated MacVim and now I have python 3 (3.6.0)
I removed, cleaned and reinstalled LeaderF and got the same error:
Error detected while processing /Users/romainf/.vim/bundle/LeaderF/plugin/leaderf.vim:
line   82:
E806: using Float as a String
E116: Invalid arguments for function <SNR>43_InitVar
line   88:
E806: using Float as a String
E116: Invalid arguments for function <SNR>43_InitVar```

Please hack the code in /Users/romainf/.vim/bundle/LeaderF/plugin/leaderf.vim. change the code call s:InitVar('g:Lf_WindowHeight', 0.5) as call s:InitVar('g:Lf_WindowHeight', '0.5'), and change call s:InitVar('g:Lf_NeedCacheTime', 1.5) as call s:InitVar('g:Lf_NeedCacheTime', '1.5'), and have a try.

Does :echo has('float') return 1 for you?  Maybe your version of Vim does not support float.

It seems to work! Thanks!
The first launch of f is quite long though, like 10 or 15 seconds. Some feedback might be great here.
Also, is there a way to remove some directories from the browsing? I'd like to remove .hg, .git and some project specific directories

Yep, :echo has('float') returns 1

The hack makes it work?

I'll optimize the first launch later on.
You can see here

To ignore some directories, please refer to here.

This is a bug of some version of vim. We are not alone, https://github.com/nathanaelkane/vim-indent-guides/issues/10 is the same.

https://groups.google.com/forum/#!topic/vim_dev/knwTpoNiTvE may be the root cause.

Thanks!

The hack makes it work?
Yep, it seems to work now! (although I'm fairly new to this plugin, so I don't really know how it should work ;) )

To ignore some directories, please refer to here.
Otherwise, it would be nice if you could add '.hg' too by default.

https://groups.google.com/forum/#!topic/vim_dev/knwTpoNiTvE may be the root cause.
Strange vim didn't fix that already 🤔


Dear Trust Wallet Team,

Dear Trust Wallet Team,
If we make a payment for the listing, can our token be accepted with 1,500
holders, or should we reach out once we have 10,000 holders?
Thank you for your guidance.
Best regards,
Umid Hasanov
SportPoint
Originally posted by @UmudHasanov in https://github.com/trustwallet/assets/pull/28915#issuecomment-2276582437

Sorry, but I didn't understand your answer. Is the 10,000 hodlers limit for
everyone? Or does it not apply to those who made the payment?
пт, 9 авг. 2024 г. в 00:20,<PHONE_NUMBER>319764 @.***>:

Closed #46 https://github.com/worldcoin/awesome-zkml/issues/46 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/worldcoin/awesome-zkml/issues/46#event-13811834940,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATVREBMKMG4OVLM3BE6YMJ3ZQPHJPAVCNFSM6AAAAABMHF6XO6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTHAYTCOBTGQ4TIMA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Umid Hasanov
Tel & Whatsapp : +994557758507


Ported the project to maven and wiki

You may like to absorb these changes:

Project build system is maven based
Moved .md files to github project wiki
No tangible changes in algorithm

Thanks

Wow, thanks you so much, @ExpertNLP !!


"TypeError: rebuild is not a function" when using with electron-packager and gulp.js

I came across this project today while reading the overview of the electron-packager repo and wanted to integrate it in my build process.
As this requires to use the js api of electron-packager (according to the README) I moved the shell command to my gulpfile. This seemed the most comfortable to me as I already use Gulp.js for several tasks.
I also expected this combination to work as there are sections in the README for both, the electron-packager and gulp as well.
Now whenever I want to run my build the following error occurs:
TypeError: rebuild is not a function
    at Object.packager.afterCopy (.../gulpfile.js:112:9)
    at internal/util.js:278:30
    at new Promise (<anonymous>)
    at Object.<anonymous> (internal/util.js:277:12)
    at .../node_modules/electron-packager/src/hooks.js:11:61
    at Array.map (<anonymous>)
    at Object.promisifyHooks (.../node_modules/electron-packager/src/hooks.js:11:29)
    at WindowsApp.copyTemplate (.../node_modules/electron-packager/src/platform.js:128:17)
    at async WindowsApp.buildApp (.../node_modules/electron-packager/src/platform.js:112:5)
    at async WindowsApp.initialize (.../node_modules/electron-packager/src/platform.js:107:7)
    at async WindowsApp.create (.../node_modules/electron-packager/src/win32.js:103:5)
    at async Promise.all (index 5)
    at async packager (.../node_modules/electron-packager/src/index.js:167:20)
    at async exports.package_app (.../gulpfile.js:77:19)

(I've shortened the paths for readability. '...' is equal to my project root directory.)
The error occurs after several minutes but I assume that this delay is related to the packaging process of electron-packager. Before that, only the usual status messages are output:
Packaging app for platform win32 arm64 using electron v7.1.7
Packaging app for platform linux arm64 using electron v7.1.7
Packaging app for platform linux x64 using electron v7.1.7
Packaging app for platform linux armv7l using electron v7.1.7
Packaging app for platform win32 ia32 using electron v7.1.7
Packaging app for platform linux ia32 using electron v7.1.7
Packaging app for platform win32 x64 using electron v7.1.7
Packaging app for platform darwin x64 using electron v7.1.7
Packaging app for platform mas x64 using electron v7.1.7

Find my gulpfile.js below for reference:
const packager = require('electron-packager');
const rebuild = require('electron-rebuild');
const log = require('fancy-log');

// Stripped out a lot of unrelated code here in favor of readability.

exports.package_app = async (finish) => {
  const Now = new Date(),
    manifest = require('./package.json');

  let distPaths = await packager({
    all: true,
    appCopyright: ['(c)', Now.getFullYear(), manifest.author.name].join(' '),
    appVersion: manifest.version,
    asar: true,
    buildVersion: 'v' + Now.toISOString(),
    derefSymlinks: true,
    dir: process.cwd(),
    download: {
      cacheRoot: './.tmp',
      rejectUnauthorized: true,
    },
    icon: null,
    ignore: [
      '\\.editorconfig',
      '\\.gitattributes',
      '\\.gitignore',
      '\\.nvmrc',
      '\\.tmp',
      '\\.vscode',
      'config',
      'gulpfile\\.js',
      'README\\.md',
      'scripts',
      'yarn\\.lock',
    ],
    junk: true,
    out: './dist',
    overwrite: true,
    prune: true,

    // Run electron-rebuild after copying the files
    // see https://github.com/electron/electron-rebuild#how-can-i-integrate-this-into-electron-packager
    afterCopy: [
      (buildPath, electronVersion, platform, arch, callback) => {
        rebuild({ buildPath, electronVersion, arch })
          .then(() => callback())
          .catch((error) => callback(error));
      }
    ],
  });

  log('Done. Built the following packages:');
  for (const path of distPaths) {
    log(' -> ' + path);
  }

  finish();
}

I'm kind of confused about this error message and could not find any related information in other Issues or during my web research on the problem.
It seems as the function is not available within the callback but I can't really figure out why.
If you need any further information please let me know.
Greetings, Max
Package Versions

OS: macOS Mojave 10.14.6
Node: v12.13.1
electron: v7.1.7
electron-packager: v14.1.1
electron-rebuild: v1.8.8
gulp: v4.0.2
gulp-cli: v2.2.0

electron-rebuild is an ES module, this is the export structure.

Your code would work if you did.
const { rebuild } = require('electron-rebuild')

@MarshallOfSound I am really sorry for bothering you with such a simple mistake 🤦♂
After changing the require statement as mentioned it worked perfectly.
Thanks a lot for your help!


Fail to install using Python 3

akismet is breaking the install on Python 3
$ pip install django-fluent-comments                                                                                node:system
Collecting django-fluent-comments
  Using cached django_fluent_comments-1.0.5-py2.py3-none-any.whl
Collecting akismet>=0.2 (from django-fluent-comments)
  Using cached akismet-0.2.0.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/private/var/folders/tp/d0cgdkdd1c32w57b4c_kl9d40000gn/T/pip-build-4au2m7mf/akismet/setup.py", line 5, in <module>
        import akismet
      File "/private/var/folders/tp/d0cgdkdd1c32w57b4c_kl9d40000gn/T/pip-build-4au2m7mf/akismet/akismet.py", line 136
        except Exception, e:
                        ^
    SyntaxError: invalid syntax

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/tp/d0cgdkdd1c32w57b4c_kl9d40000gn/T/pip-build-4au2m7mf/akismet

What I did was checking the requirements and installing them one by one, except akismet. Then I installed django-fluent-comments with the option --no-deps. You will be able to install it this way and if you don't use akismet everything will work perfectly.

Thanks, it's working.
What do you use in replacement for the anti-Spam Akismet (which is quite old: last version is from 2009!)

I don't use any replacemenet. My comment system requires the user to be logged in. I also added a captcha.

There is Python 3 compatible version of Akismet. So you can add following line to your requirements file before django-fluent-comments:
-e git+https://github.com/allieus/python-akismet.git#egg=akismet

Fixed in 1.1!
I've removed akismet as a fixed requirement in setup.py using conditional dependencies. If you install it, using the requirements.txt method above, it will be picked up and used.


Pushcart sprite rendering issues.

After renting a pushcart, the character is playable the cart is not rendered. Also, no other sprite is renderer in the scene.

A search with grep -rni "cart" src shows these results

It doesn't seem like carts are supported at all right now as there's very little mention of that word in source.


Quotes around references to other files are incorrectly handled

Hi,
When formatting this file, that has references to schemas defined in other files:
openapi: 3.1.0
info:
  title: Example
  version: 1.0.0
paths:
  /v1/examples:
    get:
      summary: Get
      responses:
        '200':
          content:
            application/json:
              schema:
                type: object
                properties:
                  data:
                    type: array
                    items:
                      $ref: '#/components/schemas/Example'
components:
  schemas:
    Example:
      type: object
      properties:
        id:
          $ref: './otherfile.yaml#/components/schemas/Id'

The following command  npx openapi-format ./public/docs/example.yaml -o ./public/docs/example.yaml
Returns the following file:
openapi: 3.1.0
info:
  title: Example
  version: 1.0.0
paths:
  /v1/examples:
    get:
      summary: Get
      responses:
        '200':
          content:
            application/json:
              schema:
                type: object
                properties:
                  data:
                    type: array
                    items:
                      $ref: '#/components/schemas/Example'
components:
  schemas:
    Example:
      type: object
      properties:
        id:
          $ref: ./otherfile.yaml#/components/schemas/Id

As you can see, quotes around $ref: './otherfile.yaml#/components/schemas/Id' get removed.
Can I fix this by modifying my file?

Temporarily fixed my issue by appending
 sed -i '' "s/^\( *\$ref: \)\(.*\.yaml#\/.*\)$/\1'\2'/g" $filepath
To every OpenAPI format command

hi @florentausha
We use @stoplight/yaml for handling the YAML files, which manipulates the "".
https://github.com/thim81/openapi-format?tab=readme-ov-file#stoplight-studio

We have adopted the YAML parsing style from Stoplight Studio, by leveraging the @stoplight/yaml package for handling the parsing of OpenAPI YAML files.

By using the Stoplight YAML parsing, the results will be slightly different from when using a normal YAML parsing library, like js-to-yaml. We appreciate the Stoplight Studio tool, since it is an excellent GUI for working with OpenAPI documents for non-OpenAPI experts who will be contributing changes. By adopting the Stoplight Studio YAML parsing, the potential risk of merge conflicts will be lowered, which is the main reason why we opted for using the @stoplight/yaml package.

@thim81 Thanks for your detailed answer, I'll close this issue then 👍


Add CNPs stale node updates GC controlplane test

Add a controlplane unit test to verify that the stale policy enforcement updates are cleared from the Status section of CNPs and CCNPs when the option skip-cnp-status-startup-cleaning is set to false (and thus the GC is enabled).
Also, to enable controlplane unit tests as this one, a small change has been done in the operator initialization, in order to avoid panicking when starting it multiple times in controlplane tests. See the commit message for more details.
Relates: https://github.com/cilium/cilium/pull/20366

/test

Smoke test workflow is hitting https://github.com/cilium/cilium/issues/22168


Add documentation for [DeploymentItem] attribute

fix: https://github.com/microsoft/testfx/issues/1320 and https://github.com/microsoft/testfx/issues/676

@engyebrahim I don't understand how this PR solves #676. My understanding is that people where using .testsettings (e.g. legacy configuration file) which isn't supported by MSTest. I feel like the issue should be simply closed.


Graduate the AssetConfiguration API to stable

We've implemented the AssetConfiguration API as an alpha API in order to support use cases such as Amazon EKS-D. The API is still missing some important features and only works on Amazon Linux 2.
This is an Epic about completing the AssetConfiguration API and graduating it to beta/stable.

The AssetConfiguration API has been removed starting with the v1beta2 API.


chore(keymap): lsp keymap should effective at LspAttach event

Lsp's keymap should not work when I'm editing a normal file。

https://github.com/ayamir/nvimdots/pull/699#issuecomment-1532371932


Timing, frame-rate consistency improvements and features

Features

improves all round consistency of the engine with respect to timing and frame-rates (delta)
adds several new velocity and speed functions to Matter.Body
adds updateVelocity argument to Body.setPosition etc.
for e.g. moving bodies around directly but with imparted velocity
adds substeps feature to Matter.Runner for applying multiple updates per-tick
changes body velocity and speed values to be frame-rate independent (via getters and setters)
changes examples to be frame-rate independent
removes correction from Engine.update as it is now built in

Notes
For most purposes, these changes are intended to be backwards compatible.
If you're using the engine at the standard browser 60fps (16.666ms delta) then results should be essentially the same as before (or better) but if they are not, please leave a comment and an example here if possible.
If you're using Body.setVelocity, Body.setAngularVelocity at something other than 60fps (16.666ms delta) then you may need to change your velocity values or remove any code you may have been using that factored in frame-rate to the velocity values, since velocity is now constant with respect to delta as expected.
While body.velocity  and body.speed remain the same, they are internal and not expected for use going forward, instead you should now use Body.getVelocity and Body.getSpeed which are frame-rate independent. Code that previously relied on these properties should still function as before for the time being, but it is strongly suggested to update these as they are not consistent or frame-rate independent, they may be removed in future.
The internal standard base time unit remains as 16.666ms for the time being, rather than 1sec as used in SI. If this were to change later, 1ms would seem the natural choice. For most users this is not usually necessary to be aware of.
All examples have been updated to also be frame-rate independent but using 1sec as their time unit to be more in line with expectations, they are also using the new updateVelocity argument in e.g. Body.setPosition which makes it easier to correctly control bodies.
Requesting reviewers, testers and comments
If anybody has any time to help me test this update out further and feedback any issues, it would be greatly appreciated:

try experiment with the standard examples in different environments (browsers, devices)

consider friction, air friction, restitution, sleeping, collisions, constraints

compare these to using the previous version of Matter.js (e.g. see the demo), results should be very similar or better
try this branch with your code base (e.g. use the alpha build on this branch)
try using different delta values in Engine.update (more than 33.333ms won't be stable)
try using engine.timing.timeScale (as another way of modifying delta)
try using runner.substeps > 1, quality should improve
try out the new updateVelocity, body velocity and speed setters / getters

Changes

Added readonly body.deltaTime
Added delta property to engine update event
Added delta argument to various internal functions
Changed timeScale argument to use delta instead on various internal functions
Fixed issues when using an engine update delta of 0
Improved time independence for friction, air friction, restitution, sleeping, collisions, constraints
Removed optional correction argument from Engine.update
Removed correction and timeScale from Body.update and Matter.Runner
Added updateVelocity argument to Body.setPosition, Body.setAngle, Body.translate, Body.rotate
Added Body.setSpeed, Body.setAngularSpeed
Added Body.getSpeed, Body.getVelocity, Body.getAngularVelocity
Changed all velocity functions to be time independent
Changed examples to be delta independent
Added substeps feature to Matter.Runner
Updated edge build

I just bought a 144 hz monitor so I'll be testing this branch asap.

@liabru, any update on how this branch looks to you?

@liabru , any update on this branch? Would you like some help resolving some of the above comments?

I am now updating my physics at a fixed timestep so I am no longer using this but FYI there seems to be something wrong with the frictions. Bodies were sliding much faster on a body with friction set to 0 at 144Hz than at 60Hz.

Thanks for the feedback everyone.
So far I've added an initial value for body.deltaTime. I think this should mean that setting velocity etc. immediately after creation should now work correctly.
I also adjusted Body.applyForce for timing. I didn't think this was necessary, but it looks like it is.
As for differences in friction between timestep sizes, a small difference is to be expected but maybe there is still a problem to fix on that.
It's definitely advised to always use a fixed timestep e.g. 60hz, then do frame skipping for faster monitors.
Once this work is done, I have some experimental updates to Matter.Runner that make it fixed by default and do this for you.

@liabru

It's definitely advised to always use a fixed timestep e.g. 60hz

why though? wouldn't you want to sync the refresh rate to whatever hz from the monitor the user is using?
I've used this PR in the past, just so I can bypass the 60hz limit. I feels laggy on my screen to run on lower refresh rates.. https://github.com/liabru/matter-js/issues/818

why though? wouldn't you want to sync the refresh rate to the monitor of the user?

In my opinion there's no "correct" answer to this. You have to choose what you need. If the timestep is synced to a monitor's refresh rate, you'll achieve smooth-looking motion but your simulation will lack a deterministic quality, aka multiples runs of the same physics simulation will yield different results since the series of delta times won't be exactly the same between runs.
On the other hand, if you fix your timestep you get this deterministic quality since the physics simulation will always be a series of steps of an exact amount. But if your physics step is 60hz, and refresh rate is 144hz, your simulation will likely appear choppy since some frames will encompass a physics step and some won't. You can still achieve smooth movement in this fixed-timestep approach through interpolation. This article explains it well.

why though? wouldn't you want to sync the refresh rate to the monitor of the user?

In my opinion there's no "correct" answer to this. You have to choose what you need. If the timestep is synced to a monitor's refresh rate, you'll achieve smooth-looking motion but your simulation will lack a deterministic quality, aka multiples runs of the same physics simulation will yield different results since the series of delta times won't be exactly the same between runs.
On the other hand, if you fix your timestep you get this deterministic quality since the physics simulation will always be a series of steps of an exact amount. But if your physics step is 60hz, and refresh rate is 144hz, your simulation will likely appear choppy since some frames will encompass a physics step and some won't. You can still achieve smooth movement in this fixed-timestep approach through interpolation. This article explains it well.

I got a new trouble. Results different on server and on client, but the game field and code the same.
Different is runners realisation. It depends on result?
Server code:
let runner = Matter.Runner.create({ isFixed: true });
while (collectedBalls.length < generatedBallsCoords.length) {
Matter.Runner.tick(runner, engine);
}
Client code:
let runner = Matter.Runner.create({ isFixed: true });
Matter.Runner.start(runner, this.engine);

why though? wouldn't you want to sync the refresh rate to the monitor of the user?

In my opinion there's no "correct" answer to this. You have to choose what you need. If the timestep is synced to a monitor's refresh rate, you'll achieve smooth-looking motion but your simulation will lack a deterministic quality, aka multiples runs of the same physics simulation will yield different results since the series of delta times won't be exactly the same between runs.
On the other hand, if you fix your timestep you get this deterministic quality since the physics simulation will always be a series of steps of an exact amount. But if your physics step is 60hz, and refresh rate is 144hz, your simulation will likely appear choppy since some frames will encompass a physics step and some won't. You can still achieve smooth movement in this fixed-timestep approach through interpolation. This article explains it well.

someone here?...

bump!

bump

So I am pulling my hair out.
After 3 months full time work on my game, I just found out that my game does not work at all on 144Hz monitors.
Gravity is much higher, and forces are not correct, breaking my game completely.
I can see that some of you seem to of managed to come up with work arounds ?
Can anyone give me any advice on how to proceed.
I am using the phaser 3 implementation of matter.js.
I am not clear on how to fix the timestep of matter, within the update loop of my phaser 3 game.
Thankyou in advance.

@mturneruk since you're using Phaser, first check with them about their support for 144Hz etc.
In terms of Matter.js, you need to make sure that Matter.Engine.update is always called with the intended fixed timestep at the intended rate, accounting for the user's refresh rate which may be different.
As an example, if your intended update rate is 60Hz the timestep is 1000 / 60 = 16.666ms and the user's refresh rate is 144Hz 1000 / 144 = 6.944ms, then you'll need to do some frame skipping (update every 144 / 60 = 2.4 frames) to maintain the same speed when using a single thread.
This is known generally as the game loop, it's worth reading the article Fix your timestep which gives some great examples of techniques to account for this problem.
The built in Matter.Runner does not currently do this for you because it is primarily intended for debugging and prototyping rather than production games, which require a much more complex game loop to handle a broad range of hardware, so check if Phaser has that feature already (maybe @photonstorm has some ideas to help).
I'd like to implement some of these techniques into Matter.Runner eventually since higher refresh rates are becoming more common, but for now I consider that a task for game engines to handle.

@driescroons exactly as @wmike1987 says. Essentially it's all about balancing performance with quality for your intended hardware support. Many games run physics as low as 30hz and but render at e.g. 60hz or more by interpolating the motion in the renderer, which generally works really well and allows for sub-stepping without the performance hit.

Thankyou so much for your excellent assistance. I now understand the problem from your description.
I have solved my 144Hz problem by setting autoUpdate to false and manually indexing the Matter engine on each step of my game loop which is running at 60hz.
https://phaser.discourse.group/t/question-about-matter-js-and-its-update-loop/4824/9

This branch fixes my timing issues with my Gas Retention Simulator - I can now successfully slow down the animation using engine.timing.timeScale. I would love for these improvements to be in a matter.js release as using this alpha build is a bit awkward.

I tried this branch since I liked the improvement, but the bodies are not going to sleep with this version: https://codepen.io/tylercorleone/pen/yLoKjad
After a couple of seconds, using matter.js: baloon.isSleeping == letters.bodies[0].isSleeping == true
using matter.alpha.js none of the body gets sleeping
Could you give please a look? Thanks :)

Hello!
Any updates on that?

These changes seem to finally fix velocity being framerate dependent. Any chance to merge this to master soon?

@liabru I see you're still working on this, from time to time - are there any plans to merge it into master? I think a lot of people are waiting for this :)

@photonstorm that's the plan, a major goal was to get constraint stiffness and friction are consistent across deltas, which I think is now working nicely. Did you try it out at all?

Hi, any chance to merge this to master soon?

@photonstorm that's the plan, a major goal was to get constraint stiffness and friction to be consistent across deltas, which I think is now working nicely. Did you try it out at all?

Not yet, but I'm working on the 3.60 release of Phaser right now, so would love to have this merged in! I'll try it out next week with our set of examples - if those all work without issue, that would be a really healthy sign.

@photonstorm that's the plan, a major goal was to get constraint stiffness and friction to be consistent across deltas, which I think is now working nicely. Did you try it out at all?

Not yet, but I'm working on the 3.60 release of Phaser right now, so would love to have this merged in! I'll try it out next week with our set of examples - if those all work without issue, that would be a really healthy sign.

Looking forward to the merge! I have a few games that are crying out for this update.

@liabru I don't suppose there's any chance of you merging this into master, is there? :) I really think it could be beneficial!

Look out for these changes in the next release. Thanks again to all those who provided feedback here!


fixed broken links

Signed-off-by: Gaurav Shukla<EMAIL_ADDRESS>Describe your changes
I've fixed broken links in _index.md english language
Related issue number or link: resolves #1547
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Thanks, @withgaurav. Where did you see broken images? It looks right on the website. The preview won't display the images correclty, but that is normal.

Here in this README file (_index.md) images are not showing. I'm not talking about website.I have seen that images in the glossary repository
Below link will redirect you to that file https://github.com/cncf/glossary/blob/main/content/en/contribute/_index.md

Hi @withgaurav
They are not broken links. :)
This repository is to generate the Cloud native Glossary website https://glossary.cncf.io/ (based on HUGO).
The links are for website and they work in the website.
Those links does not work in page view of Github repository.

Okay @seokho-son thankyou I also noticed the same thing right now

Thanks ~!
Let me close this PR. :)


Repository Housekeeping

Beefing up GitHub repo to "tidy house".

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


add cx-ssi-lib README

I am working on the same thing pluse examples how to use this lib,

I am working on the same thing plus examples of how to use this lib.

Alright. How should we proceed here? Could you merge your stuff into the "general description" I did here for Tractus-X?
I've added the sections I would also like to add with 'TBD'.

I am working on the same thing plus examples of how to use this lib.

Alright. How should we proceed here? Could you merge your stuff into the "general description" I did here for Tractus-X?
I've added the sections I would also like to add with 'TBD'.

Merging my stuff is done with some documentation/examples under the docs folder, the only remaining TBD is contact information.


[videolectures.net] download error: unable to download video data: HTTP Error 403: Forbidden

Salam
When I try to download videos from videolectures.net the following error occurs:
› youtube-dl --verbose http://videolectures.net/pcw05_zisserman_1voci/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', 'http://videolectures.net/pcw05_zisserman_1voci/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2014.06.26
[debug] Python version 2.7.5 - Darwin-13.2.0-x86_64-i386-64bit
[debug] Proxy map: {}
[videolectures.net] pcw05_zisserman_1voci: Downloading XML
ERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1020, in process_info
    success = dl(filename, info_dict)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 996, in dl
    return fd.download(name, info)
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/common.py", line 291, in download
    return self.real_download(filename, info_dict)
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 58, in real_download
    data = self.ydl.urlopen(request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1225, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 410, in open
    response = meth(req, response)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 523, in http_response
    'http', request, response, code, msg, hdrs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 448, in error
    return self._call_chain(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 382, in _call_chain
    result = func(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 531, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
HTTPError: HTTP Error 403: Forbidden

This might be a problem related to videolectures.net admins changing things in their website.

Closed by 0d93378. This should be fixed in youtube-dl versions 2<IP_ADDRESS> and newer. Refer to our FAQ if you need help updating. Thanks for the report.

Thanks.


Minimizing the App closes the app

Is there an existing issue for this?

[X] I have searched the existing issues.

What's the problem?
Minimizing the App on macos close the app not opening back until you close and open again
How did you install the app?
downloaded one of the provided packages
Operating System
macOS
Additional context

Hi @itfidele thanks for the report
As you may expect, this issue is still linked to #197 that you opened some time ago.
I tried to fix it but I didn't find a suitable solution yet.
I'll try other methods as soon as I have some spare time.
I'll keep you posted in case of updates.
Please keep referring to #197, but it's not a bad idea to also leave this issue open.


build: update docker in docker image for SMA to latest 20.10.7 to …

…address snynk issue
Signed-off-by: jpwhitemn<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/master/.github/Contributing.md.
What is the current behavior?
updated Docker in Docker from 20.10.6 to 20.10.7 to address snyk issue
Issue Number:
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

New Imports

[ ] Yes
[x] No

Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

Codecov Report

Merging #3650 (6cb9866) into main (c934d26) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3650   +/-   ##
=======================================
  Coverage   47.29%   47.29%
=======================================
  Files         112      112
  Lines        9174     9174
=======================================
  Hits         4339     4339
  Misses       4462     4462
  Partials      373      373

Continue to review full report at Codecov.

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