Mediapipe tasks-vision iOS 17+ inside Web Workers not working, despite iOS 17+ already support it.

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
None
OS Platform and Distribution
iOS 17.4.1 / iPadOS 17.4.1
MediaPipe Tasks SDK version
0.10.12 (latest on npm)
Task name (e.g. Image classification, Gesture recognition etc.)
All
Programming Language and version (e.g. C++, Python, Java)
Javascript
Describe the actual behavior
OffscreenCanvas is not used inside Web Workers when iOS 17+ supports it. It tries to create a normal canvas via document.createElement("canvas"), which obviously does not works inside a Worker.
Describe the expected behaviour
tasks-vision creates the OffscreenCanvas instead of DOM canvas inside Web Workers
Standalone code/steps you may have used to try to get what you need
Load tasks-vision (for example, for Face Landmarks mesh) inside a Web Worker on iOS 17+. It will throw:

 Can't find variable: document

I can make tasks-vision to work perfectly if I change

 typeof OffscreenCanvas !== 'undefined' && !isWebKit()

to

 typeof OffscreenCanvas !== 'undefined'

on graph_runner.ts:111 and vision_task_runner.ts:47

Also, but with this I'm not completely sure, iOS 17+ has full gl.FLOAT support, so checking if isIOS on mask.ts:306 is not necessary anymore. I've made a few test with Mesh / Hand models forcing isIOS to return false and it works correctly, but as I'd said, maybe it requires further testing.

Other info / Complete Logs
No response

Hi @SystemPanic,
Could you kindly provide the detailed steps you are following from our documentation, or alternatively, share the standalone code with us? This will help us to reproduce the issue and gain a better understanding of it.
Thank you!!

We will fix this and verify on iOS 17.

HI @SystemPanic,
We've launched the latest version 0.10.13, which includes support for OffscreenCanvas in iOS 17+, as detailed in the release notes ("Update WebGL2 on OffscreenCanvas support check to include Safari 17+"). Could you please verify now? If the issue persists on your end, please inform us.
Thank you!!

@SystemPanic could you help guide how you were able to import tasks-vision on a web worker
i am not able to find any resource on how to go about doing it and tried a lot but no success

Hi @arcinston,
Currently, we do not have any examples demonstrating web workers with our task-vision. I hope @SystemPanic can assist you with this. If I find any resources showing how to run web workers with task-vision, will inform you.
In the meantime, @SystemPanic, could you please confirm if you can now run Mediapipe inside the web worker on iOS 17+? We have resolved this issue.
Thank you!!


i18n(ja): Add docsearch translation in Japanese

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[ ] New or updated content
[x] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description

Add docsearch.ts in Japanese

Screenshot

Thank you for the contribution @nokazn! I’ve put a call out to our other Japanese language translators to review your PR.
If you would like to join us on Discord, we’d love to have you: https://astro.build/chat
We have a #docs-i18n channel to talk about translations and a dedicated thread where our ja translators co-ordinate and chat.


Prowlarr config template doesn't work after <IP_ADDRESS>33

Upgrading from <IP_ADDRESS>33 to <IP_ADDRESS>20 the entrypoint script does no longer create the prowlarr config from env vars
I don't have any persistent storage mounted to the container.
Example version: ghcr.io/onedr0p/prowlarr-develop:<IP_ADDRESS>33
[Info] Bootstrap: Starting Prowlarr - /app/bin/Prowlarr - Version <IP_ADDRESS>33
2024-04-20T10:35:43.705384522+02:00 [Info] AppFolderInfo: Data directory is being overridden to [/config]
2024-04-20T10:35:43.737502198+02:00 [Debug] Bootstrap: Console selected
2024-04-20T10:35:43.742116511+02:00 [Info] AppFolderInfo: Data directory is being overridden to [/config]
2024-04-20T10:35:44.157850004+02:00 [Info] AppFolderInfo: Data directory is being overridden to [/config]
2024-04-20T10:35:44.277479707+02:00 [Info] MigrationController: *** Migrating Database=prowlarr;Host=holon-pgbouncer.postgres.svc;Username=prowlarr;Password=mypassword;Port=5432;Enlist=False ***
Example version: ghcr.io/onedr0p/prowlarr-develop:<IP_ADDRESS>20
/entrypoint.sh: line 23: /config/config.xml: No such file or directory
[Info] Bootstrap: Starting Prowlarr - /app/bin/Prowlarr - Version <IP_ADDRESS>20
2024-04-20T10:22:44.191697868+02:00 [Info] AppFolderInfo: Data directory is being overridden to [/config]
2024-04-20T10:22:44.198994787+02:00 [Debug] Bootstrap: Console selected
2024-04-20T10:22:44.200191209+02:00 [Info] AppFolderInfo: Data directory is being overridden to [/config]
2024-04-20T10:22:44.552854025+02:00 [Info] AppFolderInfo: Data directory is being overridden to [/config]
2024-04-20T10:22:44.766077679+02:00 [Info] MigrationController: *** Migrating data source=/config/prowlarr.db;cache size=-20000;datetimekind=Utc;journal

I'm curious if this is related to permissions as things changed recently, look at the release notes and the README.

I think there is something to this. Updating to the image below broke my config. Post update, /config/config.xml no longer contains any references to Postgres.
This is the first release that I was able to reproduce this with:
- ghcr.io/onedr0p/prowlarr:1.15.0@sha256:73e30e7c9eddd7aca3de4208e702d6293862bb5c49d7a8547cc1b9e3c35dcb86
+ ghcr.io/onedr0p/prowlarr:1.15.0@sha256:cf646c64fdb90f3acddb5e2ec6ffad064392ad1aaf9da7875b8a7a962263f5fc

I am properly setting the env vars:
a9a397ddab72:/app# env | grep PROWLARR__
PROWLARR__BRANCH=master
PROWLARR__INSTANCE_NAME=Prowlarr
PROWLARR__ANALYTICS_ENABLED=False
PROWLARR__API_KEY=<REDACTED>
PROWLARR__POSTGRES_PASSWORD=<REDACTED>
PROWLARR__POSTGRES_LOG_DB=prowlarr-log
PROWLARR__PORT=9696
PROWLARR__POSTGRES_HOST=<REDACTED>
PROWLARR__POSTGRES_USER=prowlarr
PROWLARR__POSTGRES_MAIN_DB=prowlarr-main

Here is the config post update:
a9a397ddab72:/app# cat /config/config.xml
<Config>
  <BindAddress>*</BindAddress>
  <Port>9696</Port>
  <SslPort>6969</SslPort>
  <EnableSsl>False</EnableSsl>
  <LaunchBrowser>True</LaunchBrowser>
  <ApiKey>9f58b9cbaf3f4c9e89576ff2c4c9c57e</ApiKey>
  <AuthenticationMethod>None</AuthenticationMethod>
  <AuthenticationRequired>Enabled</AuthenticationRequired>
  <Branch>master</Branch>
  <LogLevel>info</LogLevel>

I've also attempted to update to ghcr.io/onedr0p/prowlarr:1.16.2@sha256:7e54724e4119386b9d992350c9fd269fd1b94ad9d51484d7b66b94a7754fbde, but received the same results.

I'm a bit confused on why this is happening since there has been no significant updates to those scripts. The only things that have changed recently is the move to upstream alpine base and the user change.
With that said I'm still using this image and updated from 1.14 to 1.15 to 1.16 without issue.

Using the 1.16 image, when it starts /config/config.xml is missing all of the Postgres related config. However, when I exec into the container and manually run /entrypoint.sh everything populates correctly in the config.xml. Also, I use a few of your other containers, and Prowlarr is the only one having the issue. 🤔
According to my git history, I started using the known-good version (73e30e7) on April 2nd. On April 18th, I attempted to use the 1.15 cf646c6 version. So the change that breaks me should be hiding in one of these commits.
The UMASK commit didn't work on my system ( I still had 0022), but I set it to 0000 via the --umask flag and the problem persisted.
I'm running Prowlarr as root, so it shouldn't be permission issues (famous last words):
/app # ps aux
PID   USER     TIME  COMMAND
    1 root      0:00 /usr/bin/catatonit -- /entrypoint.sh
    2 root      0:03 /app/bin/Prowlarr --nobrowser --data=/config
   17 root      0:00 /bin/sh
   28 root      0:00 ps aux

I'd appreciate any debugging ideas you have. I've run out for the moment.

This is my NixOS config in case you spot something that differs from yours. Besides for my choice in OS, I think everything else is pretty standard.

No volume mounts (including not persisting /config)
Systemd destroys the container as part of my ExecStartPre, so it's a clean container every time it starts.
Uses Podman
Not specifying a user. Podman launches the container with root permissions.
Load envs via file
x86_64 arch

  virtualisation.oci-containers.containers."prowlarr" = {
    image = "ghcr.io/onedr0p/prowlarr:1.15.0@sha256:73e30e7c9eddd7aca3de4208e702d6293862bb5c49d7a8547cc1b9e3c35dcb86"; # Last good version
    autoStart = true;
    environmentFiles = [
      config.age.secrets.prowlarr-env.path
    ];
    dependsOn = [ "gluetun" ];
    extraOptions = [
      "--network=container:gluetun"
    ];
    volumes = [ ];
    environment = { };
  };

@Toastyice Please let me know if you make progress on this. Maybe we can figure it out together.

I will try to reproduce this with a simple docker-compose file when I have time unless someone can get to it first.

Thanks for reopening this.
The following reproduces for me, using Docker Desktop for Mac.
To test:

Make sure the proper image is uncommented
docker compose up -d
docker compose exec -it prowlarr /bin/bash -c "cat /config/config.xml"
docker compose down, swap the image, and 🔁

prowlarr-init.sh
#!/bin/bash
set -e

psql -v ON_ERROR_STOP=1 --username "$POSTGRES_USER" --dbname "$POSTGRES_DB" <<-EOSQL
	CREATE DATABASE "prowlarr-logs";
	GRANT ALL PRIVILEGES ON DATABASE "prowlarr-logs" TO prowlarr;
EOSQL

docker-compose.yaml
version: '3'
services:
  prowlarr:
    # Good Image
    # image: ghcr.io/onedr0p/prowlarr:1.15.0@sha256:73e30e7c9eddd7aca3de4208e702d6293862bb5c49d7a8547cc1b9e3c35dcb86
    # Bad Image
    image: ghcr.io/onedr0p/prowlarr:rolling@sha256:7e54724e4119386b9d992350c9fd269fd1b94ad9d51484d7b66b94a7754fbde6
    ports: []
    volumes: []
    environment:
      - PROWLARR__POSTGRES_HOST=db
      - PROWLARR__POSTGRES_MAIN_DB=prowlarr-main
      - PROWLARR__POSTGRES_LOG_DB=prowlarr-logs
      - PROWLARR__POSTGRES_PASSWORD=prowlarr-pass
      - PROWLARR__POSTGRES_USER=prowlarr
      - PROWLARR__API_KEY=1234567890
    depends_on:
      - db
    healthcheck:
      test: ["CMD", "curl", "-f", "http://localhost:9696/api/v1/health"]
      interval: 30s
      timeout: 10s
      retries: 3

  db:
    image: postgres:16
    ports: []
    volumes:
      - ./prowlarr-init.sh:/docker-entrypoint-initdb.d/prowlarr-init.sh
    environment:
      - POSTGRES_USER=prowlarr
      - POSTGRES_DB=prowlarr-main
      - POSTGRES_PASSWORD=prowlarr-pass

Going to close this issue again since we discovered the issue.

I figured out the issue in a way that doesn't require mounting a /config volume.
If /config does not exist when envsubt runs, it fails and doesn't populate the config. When we see a /config/config.xml without Postgres information, that's a default config that prowlarr creates.
https://github.com/onedr0p/containers/blob/47d0967233421118cbd307d6fa8aae7b37e408b2/apps/prowlarr/entrypoint.sh#L23
There are two possible solutions to this that I tested:

Have the entrypoint create the directory if it doesn't exist:

if [ ! -d /config ]; then
    mkdir -p /config
fi

Specify a VOLUME in the Dockerfile:

VOLUME [ "/config" ]

Would you be open to a PR that adds one of these, and if so which would you prefer? The advantage of Option 1 is that the directory would get created with the user that the container is running as.

Yep that makes sense, Option 1 would be best, but I'll have to add it to all the containers even non-arr ones.
I'll try to get to it today.
Thanks!

@kusold done.

I can confirm that the newly released version is now working for me.
Thanks for troubleshooting this with me.


move only some of the default arguments

Introduce a new keyword (e.g. count=K) to move only the first K default arguments to the left. This should be no problem at all with how the moving is implemented (simply stop after K defaults have been processed).

Maybe even better might be a which='x, y, z' option that allows specifying the exact names and order (!) in which defaults are moved to the left?

I tested the second idea earlier, and it works nicely!


Include running nozzle logs in standard out

Let's not merge this PR, take a look at:
https://www.pivotaltracker.com/story/show/132357265
And investigate a bit first.


Feature/made input to confirm password while creating password

Added input to confirm password on create password screen.
Screenshots:

related story: https://www.pivotaltracker.com/story/show/158364432
@raymundjacobs I can't find corresponding screens in Zeplin

@ror-shubham let me merge it by myself next time
firstly we had to wait for @raymundjacobs review


fix(sqlparser): add FROM to reserved words

Signed-off-by: TennyZhuang<EMAIL_ADDRESS>I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
Fixes #5111

Tried the query and it still returned the old confusing error. Looking...

Tried the query and it still returned the old confusing error. Looking...

Keywords are banned only for table/column alias, but not table/column names..
dev=> create table table(table int);
CREATE_TABLE

😂 maybe we can port another parser framework later...

Just checked the code. It seems to be a feature of sqlparser-rs to have keywords as identifiers 😄
https://github.com/risingwavelabs/risingwave/blob/aa827ce2ff2dc0a8c03885701f6c1c7ed7bcc93e/src/sqlparser/src/parser.rs#L2147-L2162
https://github.com/risingwavelabs/risingwave/blob/aa827ce2ff2dc0a8c03885701f6c1c7ed7bcc93e/src/sqlparser/src/parser.rs#L2255-L2261
https://github.com/risingwavelabs/risingwave/blob/aa827ce2ff2dc0a8c03885701f6c1c7ed7bcc93e/src/sqlparser/src/tokenizer.rs#L220-L234

Ready for review


[release-1.9] listener: match rebalancer to listener IP family type

Cherry-pick 7f8abd142da83ab48cb6da2c86afa404fffa99ae from envoyproxy/envoy
See PR https://github.com/envoyproxy/envoy/pull/17262

@googlebot I fixed it.


Added walk-and-graph-svelte-components

Just adding my tool for the great society :)

this PR would need to be rebased


Think about what we can offer the group - thanks and recognition

What
Community participation is very important to the Design System team, and we would like to take this opportunity to offer candidates a form of reward or recognition
Why
Encourage participation and also appreciate individuals who share knowledge and experience
Who needs to work on this
Community Manager
Associate Delivery Manager
Who needs to review this
SDM & SPM
Done when
### Tasks
- [ ] Options considered
- [ ] Costing if a financial reward
- [ ] Budget approval for financial rewards
- [ ] Tracking for rewards and recognition
- [ ] Decide how to consider potential nominations for reward and recognition-minimum criteria

Options:-
provide feedback for End-of-year review
financial reward - (budget approval required) and logistically how will this happen?

Represent group on our site - Kudos in general

need to note options and costings on a document
Once the team budget is confirmed - we can finalise


Added syntax where Select on "FieldX" returns actual key names rather than FieldX1, FieldX2,...FieldXn [moved]

This is Issue 631 moved from a Google Code project.
Added by 2011-12-05T23:33:10.000Z by<EMAIL_ADDRESS>Please review that bug for more context and additional comments, but update this bug.
Original labels: Type-Enhancement, Priority-Low, v1.3.0
Original description
Given the following json
{
  "gender": {"name": "Male"},
  "firstName": "Robert",
  "lastName": "Smith",
  "account": {"checking": 12, "savings": "1234"}
}

Query "select account.checking, account.savings from Person" results in
account=[]
account2=[]

and "select account[checking], account[savings] from Person" results in
account=12
account2=1234

One can use "select account[checking] as checking, account[savings] as savings from Person" to get:
checking=12
savings=1234

But a shortcut syntax is desirable. These 3 syntaxes could be supported:

1) select account.* from person
2) select account[*] from person
3) select account.@record from person

Luca comments...
The 1) seems the cleaner but using * means that the @rid and @version will not be taken.
2) it's very similar to (1) so both could be supported.
3) @record, as the opposite of @rid, returns the entire record. This doesn't exist but it will be the same as you don't use nothing (select from XXX)

Erik comments...
1) is cleaner, intuitive, and would be consistent if IMO "select account.checkings, account.savings from Person" returned records like...
checking=1234
savings=1234

rather than the current ...
account=1234
account2=1234

Perhaps this could be changed?

2) this seems more consistent with the current syntax, though less desirable than 1) if that approach is or can be made consistent

3) this seems least consistent or least intuitive

This is not relevant anymore, as in V 3.0 we have nested projections: http://orientdb.com/docs/3.0.x/sql/SQL-Projections.html#nested-projections
Closing
Luigi


software bug: PR CI result not correct

Describe the bug
when e2e fail, the PR CI result is still success.
the
https://github.com/kubean-io/kubean/actions/runs/3080169472

Expected behavior
the summary result should correctly show the pipeline steps result: when e2e fail, the summary result should be fail

/kind bug

the PR CI result should be failed when e2e test is failed.

Contact with reporter that this issue is to be observation.


Shadows get clipped in exported 3D project

Godot version:

v3.0.3.stable.official.e649ec7, x64
OS/device including version:

Windows 10 Home x64, version 10.0.17134 (1803)
Nvidia GTX 1080, driver version 388.59
Issue description:

Shadows work as expected when the game is run from the editor, but shadows in the exported version get clipped like in the above screenshot.
Steps to reproduce:

Open project
Export a runnable executable using a default Windows desktop template (debug off)
Run the executable
Move camera to examine the issue (WASD to move, QE to rotate, wheel to zoom)

Minimal reproduction project:

ShadowExport.zip

the clipping moves with the camera?


Resnet output doesn't match the FPN_FEAT_STRIDES

The default cnn network is res101.
However, on the fpn.py line 184, the size of p2, p3, p4, p5, p6 are 1/4,1/8,1/16,1/16,1/32 of the input image's size, so the FPN_FEAT_STRIDES should be [4,8,16,16,32].
But on the trainval_net.py line 173, default FPN_FEAT_STRIDES is [4, 8, 16, 32, 64], the config and code don't match.
I not sure whether the result will be affected, but i think you can modify it. @jwyang

Good suggestion brother, but I guess the author is not deal with this project any more......

yeah... so i will close this issue


Security vulnerability in external links

When our page links to another page using target="_blank", the new page runs on the same process as your page. If the new page is executing expensive JavaScript, your page's performance may also suffer. See The Performance Benefits of rel=noopener for more information.
On top of this, target="_blank" is also a security vulnerability. The new page has access to your window object via window.opener, and it can navigate your page to a different URL using window.opener.location = newURL.
See more about this here

This is fixed in this PR https://github.com/openforge/OPENFORGE.IO/pull/87


App crashes if you try to sign up to Coinstash with an email that's already signed up

https://user-images.githubusercontent.com/50130291/175461878-d466ed0b-b4b6-46c9-b085-492ae2181d44.mov

i found that this is case,  coinstash api return error code 400 and make axios throw axios error in different shape than the current catch code

axios error should be handle differently following error type as seen in the doc https://axios-http.com/docs/handling_errors
i will try open PR to fix just this case first, but ultimately i think we should extract handle axios error function out into axios-util or some sort

Turn out the error return is GraphQL error, need to handle that as well


Fix climate temperature buttons in horizontal layout

Description

Related Issue

This PR fixes or closes issue: fixes #1040
Motivation and Context

How Has This Been Tested

Types of changes

[x] 🐛 Bug fix (non-breaking change which fixes an issue)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 🌎 Translation (addition or update a translation)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have tested the change locally.
[ ] I followed the steps if I add a new language .

this change squishes the increase and decrease buttons a lot closer together. It was easier to reach and tap the right one with the old layout. Is this intended?


Update nf-sddl-convertsecuritydescriptortostringsecuritydescriptorw.md

Fixed formatting.
And while there is no immediate way to provide feedback, let me abuse this issue to report, that the documentation for the StringSecurityDescriptorLen needs to be fixed as well (for both the -W as well as the -A API variant). The following issues need to be addressed:

Remove references to TCHAR. Neither API variant uses the generic-text mappings.
Make it clear what "The size represents the size of the buffer in WCHARs, not the number of WCHARs in the string." really means (is this just a convoluted way of saying "number of characters including the NUL terminator" or is there something more to it?).
References to WCHAR need to be removed from the -A API variant (presumably).

@tim-weis : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@tim-weis : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Sorry for the delay on this.
Thanks for the feedback, Tim. I'm passing it on internally so it doesn't get abandoned here.


UnknownCallFrameInstruction(DwCfa(45))

Hi
On aarch64 arm64v8 I encountered such errors:
UnknownCallFrameInstruction(DwCfa(45))
Seems that 45 points to DW_CFA_GNU_window_save which is not implemented.
Is there any chance for implmenting that?

It seems that it can be also DW_CFA_AARCH64_negate_ra_state - it's value is also 45

I can implement if I have time, but it won't be soon. Needing to support different vendors is going to complicate things; it's not something we've handled anywhere yet.

Can you test whether #673 meets your needs? I haven't done any testing on it myself yet.

Thanks, I'll check that next week ;)

I tested the dwarfdump example on an AArch64 binary produced by clang.


Azure training image classification validation option is not disabled

The bug has been fixed on the latest main build <IP_ADDRESS>7901.


Add a way to override velero/datamover replicas for debug purposes.

For local debugging, I need a way to avoid starting the velero and volume-snapshot-mover pods. This pull request adds two environment variables to control the respective deployment replicas, which seems like the simplest way to get this to work. There should be no change in behavior when they are not set.

You can edit CSV replicas to 0 then you can edit deployment replicas of these without operator undoing them

You can edit CSV replicas to 0 then you can edit deployment replicas of these without operator undoing them

If I understand this correctly, that's spec.install.spec.deployments(openshift-adp-controller-manager).spec.replicas in the OADP ClusterServiceVersion instance? That's what I have been using to shut down oadp-operator. But I also want to launch oadp-operator as a debug target, and when I do it immediately sets the velero/datamover replicas to 1.

Ok cool make sense.
/approve
/lgtm
On Sat, Jun 10, 2023, 8:24 AM Matthew Arnold @.***>
wrote:

You can edit CSV replicas to 0 then you can edit deployment replicas of
these without operator undoing them
If I understand this correctly, that's
spec.install.spec.deployments(openshift-adp-controller-manager).spec.replicas
in the OADP ClusterServiceVersion instance? That's what I have been using
to shut down oadp-operator. But I also want to launch oadp-operator as a
debug target, and when I do it immediately sets the velero/datamover
replicas to 1.
—
Reply to this email directly, view it on GitHub
https://github.com/openshift/oadp-operator/pull/1068#issuecomment-1585395273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACVVG6DB7ZWQYBE4GOGEV63XKPEFPANCNFSM6AAAAAAZA53TXM
.
You are receiving this because you commented.Message ID:
@.***>

Thanks, I added some unit tests.


[TASK] Add v2 data engine performance benchmark to the pipeline

What's the task? Please describe
https://ci.longhorn.io/job/private/job/longhorn-benchmark-test/
Describe the items of the task (DoD, definition of done) you'd like

Please use a task list for items on a separate line with a clickable checkbox https://docs.github.com/en/issues/tracking-your-work-with-issues/about-task-lists

[ ] item 1

Additional context
Add any other context or screenshots about the task request here.

cc @longhorn/qa @longhorn/dev-data-plane


CDN_TIMESTAMP makes endpoints that serve files from a place other than static_folder not work

I have a static file endpoint that doesn’t load the files from the app’s static_folder, but want to put it behind Flask-CDN’s url_for anyway. Now if I enable CDN_TIMESTAMP, it will fail as it recalculates the path of the file.

you can try my version
https://github.com/libwilliam/flask-cdn/pull/21


Migrate existing functionality to MVVM

I've migrated  the existing working screens to MVVM . Please let me know if there are any questions :)

I need some time to review this. Probably will be able to get to it tomorrow.

I will fix the merge and there is a small  bug on the details scree which i will address

Fixed merge issues. should be fine now. I also tested it in a  couple of games and everything seems in order. Please try it yourself before merging.

I merged this PR as a separate commit d63e0e5f because I had to make a few changes:

All line endings should be in Unix style (\n) as per git guidelines
I don't see any reason to duplicate settings as ViewModel properties

By the way it looks like tab indentation tends to create a mess because I still see spaces in a few files. I think I will convert everything to spaces to keep indentation consistent. I've already configured my scripts that convert between tabs and spaces on checking-checkout so I will continue to work with tabs locally. Wanted to make sure that you don't have any unmerged changes to avoid merge conflicts.

Hi. Thank you for  reviewing the  code  and for  taking the time to merge it. I will do my best to make sure there are no  spacing conflicts in the future.  I'd also want to send my apologies for not replying for this long. I had a really hard period at work  in the past month
As for your second point, I   tend  to disagree. One  source of problems that I've seen throughout the projects that I was involved with was  global state, and that is what our settings object is. As you probably noticed,  I'm always passing the setting object  in the  view-model and update it as needed. My plan was to further refine this and use an interface rather than  the settings object itself. The motivation for  this  is that it improves the testability of the code. I'd  strongly suggest that we start introducing some unit tests soon, as it will allow us to catch a lot of bugs before releasing a new version in the wild and it will greatly  help us with refactoring and improve the codebase as the project grows.
One good start is to try and  replace all tight coupled dependencies with abstractions and  use IOC (I'm talking about the inversion of control pattern, not  DI containers) to provide them to our components.
We can pick this up in a different thread :)


loads_object() doesn't raise MiniJSONError-based exceptions when is used against wrong classes

When loads_object() is used against class with different parameters as expected class, TypeError is raised instead of MiniJSONError-based exceptions that are only ValueErrors.
case 1 - trying to loads_object against class with different parameter names raises TypeError and not MiniJSONError-based exceptions.
case 2 - trying to loads_object against class that has more parameters again raises TypeError.
from minijson import loads_object, dumps_object, MiniJSONError

class A:
    def __init__(self, a, b):
        self.a = a
        self.b = b

class B:
    def __init__(self, c, d):
        self.c = c
        self.d = d

class C:
    def __init__(self, a, b, c):
        self.a = a
        self.b = b
        self.c = c

# case 1
try:
    dumped_a = dumps_object(A(1,2))
    loads_object(dumped_a, B)
except MiniJSONError:
    print('graceful exception handle not going to happen')

> TypeError: __init__() got an unexpected keyword argument 'a'

# case 2

try:
    dumped_a = dumps_object(A(1,2))
    loads_object(dumped_a, C)
except MiniJSONError:
    print('graceful exception handle not going to happen')

> TypeError: __init__() missing 1 required positional argument: 'c'

Don't you think that raising a TypeError is the correct behaviour here? I mean, we're talking about a misadjusted Type, rather than Value.


Add hasFriend Method

Just another suggestion. A method that would be helpful is a hasFriend(id_or_name) method for the client.

Added in version 1.1.0.


Correction for raw affiliation School of Social Work, King’s University College at Western University, London, ON, Canada

Correction needed for raw affiliation School of Social Work, King’s University College at Western University, London, ON, Canada
raw_affiliation_name: School of Social Work, King’s University College at Western University, London, ON, Canada
new_rors: ;02grkyz14
previous_rors: 048f5b183;02grkyz14
works_examples: W4366527641;W4387497932
contact: a32f646bbe25f11d1f1675a258bf58ad:916171f182d684c9 @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


baseStyle shouldnt work when there's custom styles

When options.style is passed onto an element, the default styles should completely go away.
On top of my head, it's useful when you just want to design the card elements with the classes option https://stripe.com/docs/stripe-js/reference#element-options, you can just pass style:  {} to undo the default styles.

Awesome, thank you. ;-)


GetCompletion Network request is pending and not resolving

In my application, I'm using the Codeium editor for editing JSON code. It was functioning well and providing helpful suggestions, but I recently noticed that it has stopped working—all requests remain in a pending state. Could this issue be related to a usage limitation, or is a plan upgrade required?
Also, is there a way to implement debouncing in the editor to prevent it from calling the API with every keystroke?
Please review the attached screenshot below for reference.

I am facing the same issue here


how to make viewControllers contraint inside of overlay

how can i make viewController items constraint to the overlay top left right and bottom. Also when i present a view view controller over the overlay when that view controller dismiss i get the full view of the overlay and then it shrink. can this be fix.

Hi!
When you create your own view controller in storyboard you can set up Auto-Layout on it's views the way you normally would.
Can you elaborate on the 2nd part. What is the "overlay" you are referring to? Do you have sample code or project I can use to check it out?
Thanks!


disabling of links does not work

find a simple solution

style="filter: grayscale(1);" seems to be the simplest solution


  PLEASE DON'T ASK FOR WORKAROUNDS OR TIMELINES - THIS IS FREE SOURCE CODE
Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[ ] iOS
[x] Android
[ ] UWP

PLATFORMS THAT YOU CANNOT FILE ISSUES ON RIGHT NOW: Tizen, watchOS, tvOS, macOS
Feature Library
beta components are not fully done - issues MUST include full reproducible sample and stacktrace or they will be closed without explanation!!!

[ ] Core
[ ] Locations
[ ] HTTP Transfers
[ ] Notifications
[ ] Push
[ ] Push - Azure Notification Hub
[ ] Push - Firebase (beta)
[ ] Sensors
[ ] AppCenter Logging Integration
[ ] Location Sync (beta)
[ ] Photo Sync (alpha)
[ ] BluetoothLE (beta)
[x] Beacons (beta)
[ ] SQLite Integration

Version of Library
v<IP_ADDRESS>5
Version of OS(s) listed above with issue
v10
Steps to Reproduce

Use method WhenBeaconRanged( on IBeaconManager to scan for bluetooth beacons
This works on an HTC device with Android v6.0 but the same code does not work on a Samsung Galaxy S10 with Android v10

Expected Behavior
Beacon is successfully detected and major and minor information provided
Actual Behavior
App crashes with the following stack trace:
Java.Lang.IndexOutOfBoundsException: Array index out of range: 0
at Java.Interop.JniEnvironment+InstanceMethods.CallIntMethod (Java.Interop.JniObjectReference instance, Java.Interop.JniMethodInfo method, Java.Interop.JniArgumentValue* args) [0x0006e] in <2937ce765d544e92973abcb98c88fc99>:0
at Java.Interop.JniPeerMembers+JniInstanceMethods.InvokeVirtualInt32Method (System.String encodedMember, Java.Interop.IJavaPeerable self, Java.Interop.JniArgumentValue* parameters) [0x0002a] in <2937ce765d544e92973abcb98c88fc99>:0
at Android.Util.SparseArray.KeyAt (System.Int32 index) [0x00022] in :0
at Shiny.BluetoothLE.Central.AdvertisementData+<>c__DisplayClass1_0.<.ctor>b__0 () [0x00015] in <287e9b4508ee4aa79b61aef478f7fb65>:0
at System.Lazy1[T].ViaFactory (System.Threading.LazyThreadSafetyMode mode) [0x00043] in <0b466cf954e0401081e4cbbe155f07c9>:0    at System.Lazy1[T].ExecutionAndPublication (System.LazyHelper executionAndPublication, System.Boolean useDefaultConstructor) [0x00022] in <0b466cf954e0401081e4cbbe155f07c9>:0
at System.Lazy1[T].CreateValue () [0x00074] in <0b466cf954e0401081e4cbbe155f07c9>:0    at System.Lazy1[T].get_Value () [0x0000a] in <0b466cf954e0401081e4cbbe155f07c9>:0
at Shiny.BluetoothLE.Central.AdvertisementData.get_ManufacturerData () [0x00000] in <287e9b4508ee4aa79b61aef478f7fb65>:0
at Shiny.Beacons.CentralManagerExtensions.IsBeacon (Shiny.BluetoothLE.Central.IScanResult result) [0x0000a] in <0861096243cb4454a6ec61f0761440e8>:0
at Shiny.Beacons.CentralManagerExtensions+<>c.b__0_0 (Shiny.BluetoothLE.Central.IScanResult x) [0x00000] in <0861096243cb4454a6ec61f0761440e8>:0
at System.Reactive.Linq.ObservableImpl.Where`1+Predicate+_[TSource].OnNext (TSource value) [0x00002] in :0
--- End of stack trace from previous location where exception was thrown ---
at System.Reactive.PlatformServices.ExceptionServicesImpl.Rethrow (System.Exception exception) [0x00006] in :0
at System.Reactive.ExceptionHelpers.Throw (System.Exception exception) [0x0000a] in :0
at System.Reactive.Stubs+<>c.<.cctor>b__2_1 (System.Exception ex) [0x00000] in :0
at System.Reactive.AnonymousSafeObserver1[T].OnError (System.Exception error) [0x00010] in <f524960edd7647578011457067449862>:0    at System.Reactive.Sink1[TTarget].ForwardOnError (System.Exception error) [0x00000] in :0
at System.Reactive.Sink2[TSource,TTarget].OnError (System.Exception error) [0x00000] in <f524960edd7647578011457067449862>:0    at System.Reactive.Sink1[TTarget].ForwardOnError (System.Exception error) [0x00000] in :0
at System.Reactive.Sink2[TSource,TTarget].OnError (System.Exception error) [0x00000] in <f524960edd7647578011457067449862>:0    at System.Reactive.Subjects.Subject1[T].OnError (System.Exception error) [0x00065] in :0
at System.Reactive.Sink1[TTarget].ForwardOnError (System.Exception error) [0x00000] in <f524960edd7647578011457067449862>:0    at System.Reactive.Sink2[TSource,TTarget].OnError (System.Exception error) [0x00000] in :0
at System.Reactive.Sink1[TTarget].ForwardOnError (System.Exception error) [0x00000] in <f524960edd7647578011457067449862>:0    at System.Reactive.Sink2[TSource,TTarget].OnError (System.Exception error) [0x00000] in :0
at System.Reactive.Sink1[TTarget].ForwardOnError (System.Exception error) [0x00000] in <f524960edd7647578011457067449862>:0    at System.Reactive.Linq.ObservableImpl.Where1+Predicate+[TSource].OnNext (TSource value) [0x00012] in :0
at System.Reactive.Sink1[TTarget].ForwardOnNext (TTarget value) [0x00000] in <f524960edd7647578011457067449862>:0    at System.Reactive.IdentitySink1[T].OnNext (T value) [0x00000] in :0
at System.Reactive.AutoDetachObserver1[T].OnNextCore (T value) [0x00002] in <f524960edd7647578011457067449862>:0    at System.Reactive.ObserverBase1[T].OnNext (T value) [0x0000d] in :0
at Shiny.BluetoothLE.Central.Internals.CentralContext+<>c__DisplayClass26_1.b__1 (Android.Bluetooth.LE.ScanResult sr) [0x00023] in <287e9b4508ee4aa79b61aef478f7fb65>:0
at Shiny.BluetoothLE.Central.Internals.LollipopScanCallback.OnScanResult (Android.Bluetooth.LE.ScanCallbackType callbackType, Android.Bluetooth.LE.ScanResult result) [0x00000] in <287e9b4508ee4aa79b61aef478f7fb65>:0
at Android.Bluetooth.LE.ScanCallback.n_OnScanResult_ILandroid_bluetooth_le_ScanResult (System.IntPtr jnienv, System.IntPtr native__this, System.Int32 native_callbackType, System.IntPtr native_result) [0x00011] in :0
at (wrapper dynamic-method) Android.Runtime.DynamicMethodNameCounter.89(intptr,intptr,int,intptr)

No reproducible sample was provided for a beta library, thus this is being closed.  You can try the myget version as this library is almost ready for release there


Add Clarity smart contract programming language

Description
Add Blockstack's smart contract programming language for the Stacks 2.0 blockchain, Clarity
Checklist:

[X] I am adding a new language.

[X] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[ ] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a syntax highlighting grammar: https://github-lightshow.herokuapp.com/
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

Please follow the instructions on Adding a language, but before you do, take note of this requirement:

we prefer that each new file extension be in use in hundreds of repositories before supporting them in Linguist.

There are only 202 .clar files on the whole of GitHub right now which means you're a long way from meeting this requirement. Unless you expect the popularity to grow rapidly in the next month or so, I'd suggest you close this PR and open it, following the instructions, once it is more popular.

Please follow the instructions on Adding a language, but before you do, take note of this requirement:

we prefer that each new file extension be in use in hundreds of repositories before supporting them in Linguist.

There are only 202 .clar files on the whole of GitHub right now which means you're a long way from meeting this requirement. Unless you expect the popularity to grow rapidly in the next month or so, I'd suggest you close this PR and open a new one, following the instructions, once .clar is more popular.

Hey there @lildude! Thanks so much for your guidance. A little confused by this process.
Thanks for helping out with Clarity. We had considered trying to add Clarity as an official language before, but waited until we had more repositories using the language. Like you said, there are over 200 Clarity smart contracts in the wild now, and I thought that would be enough to warrant addition to Linguist.
Also how did you search for only .clar files?

Also how did you search for only .clar files?

Search for extension:clar NOT nothack, which will perform a site-wide search for files ending with the .clar extension. You can use Harvester to download the actual files if needed.


Implement client preference cookies

We need to implement this decision: https://github.com/epicweb-dev/epic-stack/blob/main/decisions/05-client-pref-cookies.md
Any takers?

If the only thing the app needs to do is display the current date or time, then it's fine to just put the client's UTC offset (what you can get from new new Date().getTimezoneOffset()) in the cookie.
But the offset isn't enough if you need to do more complicated things, like derive new date/time values like "tomorrow at 9AM" or "how many hours of server time do I have to pay for between now and next Friday at midnight?" because there might be a DST transition in between.
To get the time zone ID on the client, it's easy: new Intl.DateTimeFormat().resolvedOptions().timeZone.
But actually using the ID (creating date/time values with it, adding/subtracting, etc.) requires more code. Possibilities:

Temporal, after it's natively available (probably on track for late 2023 or early 2024)
A Temporal polyfill. Disclosure: I'm one of the champions of the Temporal proposal and a maintainer of one of those polyfills.
Another library like day.js or date-fns-tz.

Also: the client's time zone can change if the client is a mobile device or laptop and the user is moving between time zones, e.g. on a plane. So a persistent cookie can get stale. The client should refresh the cookie periodically, and/or before doing some time-zone-sensitive thing, like showing a meeting reminder.

Thanks @justingrant,
I think the implementation I have so far handles these cases as far as we need to in the starter code: #64

I created an example that adds a time zone client hint. It also includes a helper function for formatting dates using the Intl.DateTimeFormat object, so everything uses native browser APIs.
https://github.com/kiliman/epic-stack-time-zone


Update enhance_speach.py

Fix the bug of x_final = 0 due to
len(z) < 0

Thank you for your commit.


Nitro 2.9.7 -> 2.10.4 breaks my Nuxt App with Route Roules behaviour

Environment

Operating System: Linux
Node Version:     v20.6.1
Nuxt Version:     3.13.1
CLI Version:      3.15.0
Nitro Version:    -
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      default
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>maz-ui/nuxt

Reproduction
Unfortunately, I can't reproduce it on Stackblitz cause the Proxy behavior in Webcontainers is strange. (or external requests are blocked maybe?)
If the issue isn't reproducible for you I'm happy to provide a Github Repository with the link or help finding the issue in Nitro, just tell me.
Describe the bug
I have a nuxt app which makes requests like fetch('/api/orders') (native fetch)
The app runs on the baseUrl /app
After updating Nuxt (3.13 -> 3.14), the baseURL seems to be prefixed to the fetch request, could that be possible?
I'm confused how Nuxt influences that, but if I call fetch('/api/orders') now my browser does a request to fetch('/app/api/orders'), is this intentional?
I tried changing my routeRules to proxy /app/api/**,  but the rewrites are not working anymore. Even though the proxy works, my backend receives the call with path /app/api/orders instead of /orders like before.
I tried downgrading Nuxt but had the same issue again.
Since nitro is doing the proxy stuff, I checked and I was using version 2.10.4.
Before, I was using 2.9.7. After forcing Nuxt to use version 2.9.7, everything works fine again though.
This seems to be a big breaking change for a minor version.
Additional context
necessary changes in nuxt.config.ts:
 baseURL: '/app', routeRules: {       "/api/**": {         proxy: 'http://localhost:3000/**',      } }
Logs
There are no error logs, Nuxt is behaving like this is intentional.

Any chance you can share a github repository I can locally reproduce the issue?

@pi0 it's in dev mode, I haven't tested it deployed

Let's track in https://github.com/nitrojs/nitro/issues/2881. 🙏


Demontryte Mine Image

Small issue, demontryte mine thumbnail doesnt load.

This is a Roblox bug, not a game bug.


Saving format

Files are currently saved as jpeg  for every frame.
Decide on a better output and implement.

TIF for now with better filenames (0001, 0002, etc.)


Alter Title on OPSEC report to display game name

Currently when a games COC goes to the OPSEC Report on OpServ, the title will only state OPSEC Report instead of "${game_initials} OPSEC Report" i.e. "SC OPSEC Report"
Let's update that to properly display the game initials, so there aren't any confusion on what perms is being edited.

Deployed


Fix i18n gettext fallback language message loading

Fallback language messages didn't get assigned to $messages variable in yii\i18n\GettextMessageSource::loadMessages().

Merged. Thanks!


Swagger support

Expose the Cabot REST API with SWAGGER.
Swagger is more wide spread and has more tooling than Django REST framework. We use it a lot for client code generation.

We use Cabot at our company. I post this ticket to check if this feature would be accepted in a pull request.


Fix incorrect HookEventType of pull request review comments (#23650)

Backport #23650 by @Zettat123
HookEventType of pull request review comments should be HookEventPullRequestReviewComment but some event types are HookEventPullRequestComment now.

ping lg-tm


Could you give me some guidance on how I can adapt this to image captioning?

Thanks for you great work. In the past I have used your repo for several classical computer vision problems.
I am now trying to use this for video captioning - the model I am using has an image encoder and the encoding is then passed through a causal self-attention layer and then a text decoder to generate a caption. I want to understand which part of the image led to the prediction of a certain word (at index i in the generated caption). Would this be possible if so how would I do this?

Hi,
I would create a model wrapper that as an output, has the certain word you want to target (for example, this wrapper class can get the word index in the constructor).
But would need more details to handle the time dimension here.

For creating the heatmaps, is it fine to just feed one image at a time?
Or do you have to take into account the entire sequence ?

For simplicity lets assume there would be no time dimension and just use images.

Then writting a model wrapper with the work index, should work.
Something like:
class ImageCaptionWrapper:
    def __init__(self, word_index):
        self.word_index = word_index

    def __call__(self, model_output):
        return model_output[:, word_index]

wrapped_model = ImageCaptionWrapper(model)
Then pass wrapped_model into gradcam as usual (example pointers in Readme).

Thanks Let me try it out! :)

Hi, Have you used grad-cam on the image captioning task successfully? I'm now trying on this work. Any suggestions, or demo available? Thank you


Move 'body' parameters to new typed parameters

As part of the ES client upgrade to 8.x, all client method's now expose the JSON
body' field as typed parameters,, and the use of the body parameter is deprecated.
For example:
client.search(
    index="test-index",
    size=10,
    body={
        "runtime_mappings": {
            "day_of_week": {
                "type": "keyword",
                "script": "emit(doc['@timestamp'].value.dayOfWeekEnum)"
            }
        },
        "aggs": {
            "day_of_week_count": {
                "value_count": {
                    "field": "day_of_week"
                }
            }
        }
    }
)

Would become:
client.search(
    index="test-index",
    size=10,
    runtime_mappings={
        "day_of_week": {
            "type": "keyword",
            "script": "emit(doc['@timestamp'].value.dayOfWeekEnum)"
        }
    },
    aggs={
        "day_of_week_count": {
            "value_count": {
                "field": "day_of_week"
            }
        }
    }
)

We need to refactor all of our runners (and custom runners in rally-tracks) to use these new parameters.

Another option here is to use **body instead of body.

Looks like a decision was made to un-deprecate the removal of the body param.
Closing this one out for now.


A fix for kubespray environment check

Fix default runtime check issue:  https://github.com/microsoft/pai/issues/4564
When user replace docker.io or fcr.io, check the corresponding repo instead of the original.
Get devbox ip from hostvars[inventory_hostname]['ansible_default_ipv4']['address'] instead of eth0

Coverage remained the same at 34.607% when pulling 15fcd789d899801bc29e08193f12ab3c2cd2c4a1 on yuye/qk-enhan into 7cbf51353c6992d65961fb4e261743cc78160244 on master.


Add SourceView

This PR isn't working, for now the structs aren't generated because the sys functions aren't generated.
@EPashkin: It seems the correponding sys functions are missing. Any idea why?

Because it don't part of gtk but separate library https://packages.ubuntu.com/zesty/amd64/libdevel/libgtksourceview-3.0-dev 😉

Ah damn, thanks!


clip models fail to load with unicode characters in OLLAMA_MODELS path on windows

What is the issue?
I tried to start llava:1.6 (or any similar llava based modell) an the llama server terminated.
llama3 modell or different non llava models work just fine.
GPU is: NVIDIA GeForce RTX 4060" total="8.0 GiB" available="6.9 GiB"
CPU is: AMD Ryzen 7 4700G with Radeon GPU (ignored by Ollama) ("unsupported Radeon iGPU detected skipping" id=0 name="AMD Radeon(TM) Graphics")
OS in Win 11
Geforce Drivers: 555.99 Studio
Log extract:
time=2024-06-27T16:53:15.204+02:00 level=INFO source=memory.go:309 msg="offload to cuda" layers.requested=-1 layers.model=33 layers.offload=33 layers.split="" memory.available="[7.2 GiB]" memory.required.full="5.3 GiB" memory.required.partial="5.3 GiB" memory.required.kv="256.0 MiB" memory.required.allocations="[5.3 GiB]" memory.weights.total="3.9 GiB" memory.weights.repeating="3.8 GiB" memory.weights.nonrepeating="102.6 MiB" memory.graph.full="164.0 MiB" memory.graph.partial="181.0 MiB"
time=2024-06-27T16:53:15.205+02:00 level=WARN source=server.go:241 msg="multimodal models don't support parallel requests yet"
time=2024-06-27T16:53:15.208+02:00 level=INFO source=server.go:368 msg="starting llama server" cmd="C:\Users\Stefan Hüttner\AppData\Local\Programs\Ollama\ollama_runners\cuda_v11.3\ollama_llama_server.exe --model C:\Users\Stefan Hüttner\.ollama\models\blobs\sha256-170370233dd5c5415250a2ecd5c71586352850729062ccef1496385647293868 --ctx-size 2048 --batch-size 512 --embedding --log-disable --n-gpu-layers 33 --mmproj C:\Users\Stefan Hüttner\.ollama\models\blobs\sha256-72d6f08a42f656d36b356dbe0920675899a99ce21192fd66266fb7d82ed07539 --no-mmap --parallel 1 --port 51274"
time=2024-06-27T16:53:15.742+02:00 level=INFO source=sched.go:382 msg="loaded runners" count=1
time=2024-06-27T16:53:15.742+02:00 level=INFO source=server.go:556 msg="waiting for llama runner to start responding"
time=2024-06-27T16:53:15.743+02:00 level=INFO source=server.go:594 msg="waiting for server to become available" status="llm server error"
INFO [wmain] build info | build=3171 commit="7c26775a" tid="17384" timestamp=1719499996
INFO [wmain] system info | n_threads=8 n_threads_batch=-1 system_info="AVX = 1 | AVX_VNNI = 0 | AVX2 = 0 | AVX512 = 0 | AVX512_VBMI = 0 | AVX512_VNNI = 0 | AVX512_BF16 = 0 | FMA = 0 | NEON = 0 | SVE = 0 | ARM_FMA = 0 | F16C = 0 | FP16_VA = 0 | WASM_SIMD = 0 | BLAS = 1 | SSE3 = 1 | SSSE3 = 1 | VSX = 0 | MATMUL_INT8 = 0 | LLAMAFILE = 1 | " tid="17384" timestamp=1719499996 total_threads=16
INFO [wmain] HTTP server listening | hostname="<IP_ADDRESS>" n_threads_http="15" port="51274" tid="17384" timestamp=1719499996
ggml_cuda_init: GGML_CUDA_FORCE_MMQ:   no
ggml_cuda_init: CUDA_USE_TENSOR_CORES: yes
ggml_cuda_init: found 1 CUDA devices:
Device 0: NVIDIA GeForce RTX 4060, compute capability 8.9, VMM: yes
ERROR [load_model] unable to load clip model | model="C:\Users\Stefan Hüttner\.ollama\models\blobs\sha256-72d6f08a42f656d36b356dbe0920675899a99ce21192fd66266fb7d82ed07539" tid="17384" timestamp=1719499996
time=2024-06-27T16:53:16.512+02:00 level=ERROR source=sched.go:388 msg="error loading llama server" error="llama runner process has terminated: exit status 0xc0000409 "
OS
Windows
GPU
Nvidia
CPU
AMD
Ollama version
0.1.47

I also encountered the same problem

I updated ollama and it worked.

Thanks, but error still persists after update.

The error occured after updating to the latest version. I get the issue now with llama3 which was working finde before.

Same here, opened another issue https://github.com/ollama/ollama/issues/5431

It looks like there's a bug in the clip model loading code in C++ that doesn't handle unicode characters properly.
Until we can get this fixed, a workaround is create a model directory on your C: with only ASCII characters and set OLLAMA_MODELS to that path for the server.
https://github.com/ollama/ollama/blob/main/docs/faq.md#how-do-i-configure-ollama-server

Dear @dhiltgen - thanks a lot. This works like a charm. Unexpected that unicode is not handled, but workaround is fine. Many thanks for our great work!

Item closed, workaround is doing fine

Thanks for submitting the issue.  This definitely should work, so we'll get this fixed properly.


docs: assign remove function

Description
Update Python API in How to define intra-node data path.
Add How to remove intra-node data path.

Related links
https://tier4.atlassian.net/browse/RT2-400

Notes for reviewers

Pre-review checklist for the PR author

[x] I've confirmed the contribution guidelines.
[x] The PR is ready for review.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR has been reviewed.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@atsushi421
Review reflected. Please reconfirm.

@nabetetsu
Please review.


Builder exits with error - teensy & arduino

Mac OSX 10.10.5 (14F1021)
Python 2.7.10
PlatformIO, version 2.3.4
Can't think of a reason for this.
If I setup the board myself with a main.ino with setup(), loop() and my own config file I don't seem to have the problem.
platformio ci --board teensylc
.pioenvs/teensylc/libFrameworkArduino.a(main.o): In function main':
/private/var/folders/r4/pmcrnkrj5rzg_4x2mf6hj3400000gn/T/tmp57Olfd/.pioenvs/teensylc/FrameworkArduino/main.cpp:21: undefined reference to 'setup'
/private/var/folders/r4/pmcrnkrj5rzg_4x2mf6hj3400000gn/T/tmp57Olfd/.pioenvs/teensylc/FrameworkArduino/main.cpp:23: undefined reference to 'loop'
collect2: error: ld returned 1 exit status
scons: *** [.pioenvs/teensylc/firmware.elf] Error 1

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27752028-builder-exits-with-error-teensy-arduino?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github).

You should specify the path to source file or define global environment variable. See documentation http://docs.platformio.org/en/latest/userguide/cmd_ci.html

If you still need help, please reopen this issue.


Auto-complete for function snippet in the VS Code extension missing field

The tab auto-complete for the function snippet in the VS Code extension doesn't tab though the input tag.
To Reproduce
In a *.qs file in VS Code where you have the Quantum Development Kit installed start typing "function" and arrow down to select the function snippet. The function name, output type and contents can be tabbed though but not the argument.
Expected behavior
The input field should be tab-able like the rest of the fields.
Screenshots
If applicable, add screenshots to help explain your problem.
System information

Extension version:

Name: Microsoft Quantum Development Kit for Visual Studio Code
Id: quantum.quantum-devkit-vscode
Description: Microsoft Quantum Development Kit for Visual Studio Code provides support for developing quantum algorithms in the Q# programming language.
Version: 0.11.20033107
Publisher: Microsoft DevLabs
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=quantum.quantum-devkit-vscode

Windows 10:

dotnet --version
3.1.101

VS Code versions:

Version: 1.43.2 (system setup)
Commit: 0ba0ca52957102ca3527cf479571617f0de6ed50
Date: 2020-03-24T07:38:38.248Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

Thanks for filing & fixing!


Allowed characters in keys

Would you please explain the rationale behind allowing only string.ascii_letters + string.digits + "_" in keys?
I would like to use greek letters and end up with silent errors like:
>>> a = Box()
>>> a.σeq = 1
>>> a.µeq = 2
>>> a
<Box: {'σeq': 2}>

Is there anything wrong with adding more utf-8 characters to allowed?

Honestly never considered using non-ascii characters, because I was operating under the false assumption that they were identifiers that stuck to the conventional rules of:

Identifiers can be a combination of lowercase letters (a to z) or uppercase letters (A to Z) or digits (0 to 9) or an underscore (_).

Always open to PRs to help improve it!

Looks like non-ascii identifiers were introduced on Python 3.0 with PEP 3131. It gives also some details about how to implement, will try to make a proposal based on that.

Finally got around to looking at this, and discovered the str.isidentifier function.
In the process of implementing that I decided to try and optimize the code a little there, and due to that functions importance was able to make load times of large sets twice as fast!
https://github.com/cdgriffith/Box/commit/b3ba1fb7eccd746dd6d56d03f9d6fc036608451b#diff-ded35bb17832f133568fbb4f1627f47f7bf0357bd7770e7576a65dd77542c540R735
Box 6.0 I plan to make Cython optimized which speeds it up another 10x across the board, but these optimizations stack thankfully, so in the end will be 20x faster :tada:

Sounds great! 🎉 Looking forward for it! 😃

While testing this found out something "fun".
>>> a = Box()
>>> a.σeq = 1
>>> a.µeq = 2
>>> a
Box({'σeq': 1, 'μeq': 2})
>>> a == Box({'σeq': 1, 'µeq': 2})
False

Python uses NFKC normalization on variables and attributes behind the scenes. AKA
µ = 1
>>> print(dir()[-1])
μ
>>> ord(dir()[-1])
956
>>> ord('µ')
181

So that's something to be very wary of

Adding these features in 6.0, currently have a release candidate that can be installed and tested with:
pip install python-box==6.0.0rc2

6.0 is ushering in Cython speedups on supported platforms, so please let me know if you run into any issues!

All good so far, great job 👍

Box 6 has been released with this added, thanks for opening the issue!


Enable largeHeap to prevent OutOfMemoryError on some devices (Android)

Issue

N/A

Overview (Required)

Due to recurring OutOfMemoryError issues when loading animated image vector(anim_header_title.xml) resource on certain devices (Medium Tablet API 25), this commit enables the largeHeap configuration in the AndroidManifest.

In an effort to address these memory issues, enabling the largeHeap configuration in the AndroidManifest has been considered as a temporary solution. This change aims to provide more heap space to prevent crashes and improve the stability of the application on devices with limited memory resources.

However, it's important to note that using largeHeap=true is generally discouraged as it can lead to poorer system performance and longer garbage collection times, negatively impacting the user experience. This decision was not made lightly; alternatives such as resizing the animated-vector files or optimizing memory management within the app were considered. Unfortunately, the inherent complexity of the vectors and the anticipated loss of quality in animations from resizing the vector files were problematic.

Links

android:largeHeap

Screenshot (Optional if screenshot test is present or unrelated to UI)

Movie (Optional)

Before
After

Thank you for identifying this issue. I'm concerned that we might have a memory leak

I've checked the heap dump but I couldn't find a problem.

Thank you for checking the heap dump. I also reviewed it on my end and couldn't identify any issues. It's really puzzling why the OOM is occurring on specific devices 😭  (Medium Tablet API 24, 25)

@onseok @takahirom
Excuse me. 🙇
I was investigating this issue as I am the one who made the animation display.
I found out that even if I made it so that a VectorDrawable that is not an AnimatedVectorDrawable is displayed, I cannot prevent OOM as long as the image fills the entire width of the screen on the About screen.
Therefore, we are now working on a fix to prevent the About screen from displaying images that fill the entire width of the screen on devices with low memory.
I would suggest that you look at that response and then start considering whether or not you need a LargeHeap. 🙇

Well, it is difficult to determine if it's a settings issue when users experience crashes. How about having an expect fun canShowLargeVector(): Boolean instead?

@takahirom @onseok
We addressed this issue in #673. 🙇
Therefore, there is no need to use largeHeap.

Thank you for your investigation! I'll close this pull request, but I believe your investigation is also a valuable contribution.


Weapon speed multiplier bug

Hello, I first want to thank the mod author and everyone who helped for this absolutely awesome mod which I cannot play Skyrim whithout (I simply get motion sickness from the 60 FPS slideshow we are still getting in 2024 by default).
However i found a very specific bug that I spend quite a few hours on to get to the bottom of and get the exact circumstances to be able to simulate it (being a Java programmer myself I know how important this is). So here is my bug report (I cannot seem to make a report under the "BUGS"-tab here on the nexus site).
Bug report:
Description:
When you have both the "Bow of Shadows" and another regular one- or twohanded weapon (e.g. "glass sword") and you trigger a
loading screen by activating a door or something, depending on the weapon you had equipped (whether sheathed or unsheathed),
either the "Bow of Shadows" has suddenly slower attack speeds (smaller weapon speed multiplier) or the other regular weapon
(e.g. "glass sword") has suddenly faster attack speeds (bigger weapon speed multiplier).
This bug is potentially not only happening with the "Bow of Shadows", but could be happening for any weapon that has an inherent
attack speed buff or debuff.
Steps to reproduce:

Make sure you have the "Bow of Shadows" (Creation Club content) in your character's inventory.
Make sure you have another one- or twohanded regular weapon in your character's inventory, I chose a regular "glass sword".
Equip the "Bow of Shadows" and check your weapon speed multiplier using the console:
player.getav WeaponSpeedMult
Next equip the other regular weapon (in my case the "glass sword") and check the weapon speed multiplier using the console:
player.getav WeaponSpeedMult
Pick either of the two weapons and equip it (it does not matter whether or not you unsheath it)
Go through a door that triggers a loading screen (I chose to enter/exit my Breezehome house in Whiterun)
Now check the weapon speed multiplier again — for the weapon you equipped — using the console:
player.getav WeaponSpeedMult
Next equip the other weapon (you didn't pick before the loading screen) and check the weapon speed multiplier again, using the console:
player.getav WeaponSpeedMult

Expected result:
The weapon speed multipliers for both weapons should be equal when you compare their value from before the loading screen with their corresponding value from after the loading screen and both weapons should have unchanged attack speed as an effect of that.
Real test result:
Either the "Bow of Shadows" has a lower weapon speed multiplier or the other regular weapon (my "Glass sword") has a higher weapon speed multiplier. As a result of that I can visually tell the difference in attack speed.
Relevant test influencers:

I have unlocked the "Quick Shot" perk in the Archery skill perk tree, which makes bows 30% faster (i.e. weapon speed multiplier 1.30).
The "Bow of Shadows" has an inherent attack speed buff op 20% (i.e. weapon speed multiplier 1.20)
The other regular weapon (my "glass sword") has no attack speed buffs or debuffs (i.e. weapon speed multiplier 0.00) (technically the same as 1.00 or x1 speed).
There are no other attack speed buffs or debuffs active (if survival mode is on, make sure you are well fed to prevent attack speed debuffs)

Irrelevant test influencers:
The bug happens independent of the chosen configuration file, that is the default INI file included in the mod or the so called
"High Performance" INI file included as an optional mod (which can be used to overwrite the default). I did however not change any individual settings in the INI file for I have no clue of what settings could impact this bug (if any at all).
My test data:
Before loading screen:

"Bow of Shadows" weapon speed multiplier: 2.50
(Vanilla Skyrim math says: 20% (1.20) inherent speed + 30% (1.30) "Quick Shot" perk speed = 2.50 (150%) total speed, instead of 1.50 (50%)) (This is not part of this bug, but is rather just ridiculous vanilla Skyrim math and is thus supposed to be this way)
Other weapon (my "glass sword") weapon speed multiplier: 0.00
(Skyrim equals 0.00 to 1.00 or x1 normal speed)

After loading screen when "Bow of shadows" was picked and equipped before it:

"Bow of Shadows" weapon speed multiplier: 2.50 (unchanged)
Other weapon (my "glass sword") weapon speed multiplier: 1.20 (added 20% inherent speed of "Bow of Shadows" it seems)
=> The other weapon (my "glass sword") is suddenly faster when attacking

After loading screen when other weapon (my "glass sword") was picked and equipped before it:

"Bow of Shadows" weapon speed multiplier: 1.30 (subtracted 20% inherent speed of "Bow of Shadows" it seems — again using Skyrim math)
Other weapon (my "glass sword") weapon speed multiplier: 0.00 (unchanged)
=> The "Bow of Shadows" is suddenly slower when attacking

Workarounds found:

Disable this mod (obviously) -> but heck i need this mod to make Skyrim playable ;)
Make a quicksave and immediately do a quickload (the weapon speed multiplier of ONLY the currently equipped weapon will be correctly reset)
Install the mod Unofficial Skyrim Special Edition Patch - USSEP, followed by the mod Unofficial Skyrim Creation Club Content Patches, and in this order. I also made sure these are loaded first (at the top of modlist in Mod Organizer 2). Both of these mods must be installed in the order mentioned for the fix to work!
=> However these mods will make it so that the "Bow of Shadows" gets 30% attack speed (i.e. weapon speed multiplier of 1.30) since it only allows the highest buff to apply and does not add up the buffs (So it also fixes the whole vanilla Skyrim math problem).

PC hardware:
CPU: AMD Ryzen 7 5800X
GPU: AMD Radeon RX 6800XT
Motherboard: MSI MEG X570 UNIFY
RAM: 32 GB DDR4 (2X G.SKILL TRIDENTZ RGB 2X8GB F4-3200C14D-16GTZR CL14-14-14-34 1.35V)
Game running on storage: PCIe 4.0 NVMe M.2 SSD Samsung 990 PRO
Other info:

Using Mod Organizer 2 as mod manager
Bug can be simulated having only this mod (Skyrim SE display tweaks) installed. All other mods have been disabled during testing (offcourse not the required mods SKSE and Address Library for SKSE plugins).

Sorry for the long post, but I wanted to give you as much info as possible in a clear and transparent manner, so you can focus on fixing the bug and not reverse engineering what this bug encompasses. I would honestly be genuinely impressed if you could fix this bug.
Thanks in advance for your time and for your awesome work on this mod!
Kind regards,
Alejandro

Hey, thanks for the detailed report. Could you cap your fps at 60 or lower and see if it still happens? I'm cuirious if it's triggered by high framerate or if it's something in the plugin that's causing it..
Thanks
On December 15, 2024 1:58:49 AM GMT+01:00, Alejandro De Groote @.***> wrote:

Hello, I first want to thank the mod author and everyone who helped for this absolutely awesome mod which I cannot play Skyrim whithout (I simply get motion sickness from the 60 FPS slideshow we are still getting in 2024 by default).
However i found a very specific bug that I spend quite a few hours on to get to the bottom of and get the exact circumstances to be able to simulate it (being a Java programmer myself I know how important this is). So here is my bug report (I cannot seem to make a report under the "BUGS"-tab here on the nexus site).
Bug report:
Description:
When you have both the "Bow of Shadows" and another regular one- or twohanded weapon (e.g. "glass sword") and you trigger a
loading screen by activating a door or something, depending on the weapon you had equipped (whether sheathed or unsheathed),
either the "Bow of Shadows" has suddenly slower attack speeds (smaller weapon speed multiplier) or the other regular weapon
(e.g. "glass sword") has suddenly faster attack speeds (bigger weapon speed multiplier).
This bug is potentially not only happening with the "Bow of Shadows", but could be happening for any weapon that has an inherent
attack speed buff or debuff.
Steps to reproduce:

Make sure you have the "Bow of Shadows" (Creation Club content) in your character's inventory.
Make sure you have another one- or twohanded regular weapon in your character's inventory, I chose a regular "glass sword".
Equip the "Bow of Shadows" and check your weapon speed multiplier using the console:
player.getav WeaponSpeedMult
Next equip the other regular weapon (in my case the "glass sword") and check the weapon speed multiplier using the console:
player.getav WeaponSpeedMult
Pick either of the two weapons and equip it (it does not matter whether or not you unsheath it)
Go through a door that triggers a loading screen (I chose to enter/exit my Breezehome house in Whiterun)
Now check the weapon speed multiplier again — for the weapon you equipped — using the console:
player.getav WeaponSpeedMult
Next equip the other weapon (you didn't pick before the loading screen) and check the weapon speed multiplier again, using the console:
player.getav WeaponSpeedMult

Expected result:
The weapon speed multipliers for both weapons should be equal when you compare their value from before the loading screen with their corresponding value from after the loading screen and both weapons should have unchanged attack speed as an effect of that.
Real test result:
Either the "Bow of Shadows" has a lower weapon speed multiplier or the other regular weapon (my "Glass sword") has a higher weapon speed multiplier. As a result of that I can visually tell the difference in attack speed.
Relevant test influencers:

I have unlocked the "Quick Shot" perk in the Archery skill perk tree, which makes bows 30% faster (i.e. weapon speed multiplier 1.30).
The "Bow of Shadows" has an inherent attack speed buff op 20% (i.e. weapon speed multiplier 1.20)
The other regular weapon (my "glass sword") has no attack speed buffs or debuffs (i.e. weapon speed multiplier 0.00) (technically the same as 1.00 or x1 speed).
There are no other attack speed buffs or debuffs active (if survival mode is on, make sure you are well fed to prevent attack speed debuffs)

Irrelevant test influencers:
The bug happens independent of the chosen configuration file, that is the default INI file included in the mod or the so called
"High Performance" INI file included as an optional mod (which can be used to overwrite the default). I did however not change any individual settings in the INI file for I have no clue of what settings could impact this bug (if any at all).
My test data:
Before loading screen:

"Bow of Shadows" weapon speed multiplier: 2.50
(Vanilla Skyrim math says: 20% (1.20) inherent speed + 30% (1.30) "Quick Shot" perk speed = 2.50 (150%) total speed, instead of 1.50 (50%)) (This is not part of this bug, but is rather just ridiculous vanilla Skyrim math and is thus supposed to be this way)
Other weapon (my "glass sword") weapon speed multiplier: 0.00
(Skyrim equals 0.00 to 1.00 or x1 normal speed)

After loading screen when "Bow of shadows" was picked and equipped before it:

"Bow of Shadows" weapon speed multiplier: 2.50 (unchanged)
Other weapon (my "glass sword") weapon speed multiplier: 1.20 (added 20% inherent speed of "Bow of Shadows" it seems)
=> The other weapon (my "glass sword") is suddenly faster when attacking

After loading screen when other weapon (my "glass sword") was picked and equipped before it:

"Bow of Shadows" weapon speed multiplier: 1.30 (subtracted 20% inherent speed of "Bow of Shadows" it seems — again using Skyrim math)
Other weapon (my "glass sword") weapon speed multiplier: 0.00 (unchanged)
=> The "Bow of Shadows" is suddenly slower when attacking

Workarounds found:

Disable this mod (obviously) -> but heck i need this mod to make Skyrim playable ;)
Make a quicksave and immediately do a quickload (the weapon speed multiplier of ONLY the currently equipped weapon will be correctly reset)
Install the mod Unofficial Skyrim Special Edition Patch - USSEP, followed by the mod Unofficial Skyrim Creation Club Content Patches, and in this order. I also made sure these are loaded first (at the top of modlist in Mod Organizer 2). Both of these mods must be installed in the order mentioned for the fix to work!
=> However these mods will make it so that the "Bow of Shadows" gets 30% attack speed (i.e. weapon speed multiplier of 1.30) since it only allows the highest buff to apply and does not add up the buffs (So it also fixes the whole vanilla Skyrim math problem).

PC hardware:
CPU: AMD Ryzen 7 5800X
GPU: AMD Radeon RX 6800XT
Motherboard: MSI MEG X570 UNIFY
RAM: 32 GB DDR4 (2X G.SKILL TRIDENTZ RGB 2X8GB F4-3200C14D-16GTZR CL14-14-14-34 1.35V)
Game running on storage: PCIe 4.0 NVMe M.2 SSD Samsung 990 PRO
Other info:

Using Mod Organizer 2 as mod manager
Bug can be simulated having only this mod (Skyrim SE display tweaks) installed. All other mods have been disabled during testing (offcourse not the required mods SKSE and Address Library for SKSE plugins).

Sorry for the long post, but I wanted to give you as much info as possible in a clear and transparent manner, so you can focus on fixing the bug and not reverse engineering what this bug encompasses. I would honestly be genuinely impressed if you could fix this bug.
Thanks in advance for your time and for your awesome work on this mod!
Kind regards,
Alejandro
--
Reply to this email directly or view it on GitHub:
https://github.com/SlavicPotato/SSEDisplayTweaks/issues/7
You are receiving this because you are subscribed to this thread.
Message ID: @.***>

Hey, sorry for the late reply (didn't check my GitHub mailaddress lately), but i'm gratefull for your fast reply. After some more testing I can confirm this bug does not happen on 60 FPS capped gameplay. I capped the gameplay by setting all framerate related settings in the INI-file to 60 FPS.
Also worth mentioning is that I use a dragonbow bow and and ebony dagger now in my game and it still happens on uncapped (in my case that means vsync'ed to 165Hz). None of those weapons have inherent speed buffs or debuffs. Only the 30% quick shot bonus from archery applies to the bow. When I go through a door with the bow equipped the dagger is certainly also 30% faster now (weapon speed mult = 1.30). But as said it does not happen when I cap the FPS at 60 using your INI settings.
Thanks in advance for your time and effort.

Alright, atleast we know it's yet another engine bug related to high fps, an odd one too. I'll leave this issue open and get to it when I can - can't promise it'll be any time soon though.

No problem, I understand completely, especially during the holidays now. I'm very glad you are actually willing to take a look at it when you have more time. If it can't be fixed so be it, it won't be the end of the world 😉. I really took the time to report it since I already lost a lot of time trying to figure it out myself and given the hundreds of ours of playtime it seemed worthwhile to me.


TestDepends.testRequire regression in 41.6.0 (py3)

While trying to build the new release of setuptools, I get the following test failure:
==================================================================== FAILURES =====================================================================
_____________________________________________________________ TestDepends.testRequire _____________________________________________________________

self = <setuptools.tests.test_setuptools.TestDepends object at 0x7fbfae31d710>

    @needs_bytecode
    def testRequire(self):
        req = Require('Json', '1.0.3', 'json')

        assert req.name == 'Json'
        assert req.module == 'json'
        assert req.requested_version == '1.0.3'
        assert req.attribute == '__version__'
        assert req.full_name() == 'Json-1.0.3'

        from json import __version__
        assert req.get_version() == __version__
        assert req.version_ok('1.0.9')
        assert not req.version_ok('0.9.1')
        assert not req.version_ok('unknown')

        assert req.is_present()
        assert req.is_current()

        req = Require('Json 3000', '03000', 'json', format=LooseVersion)
        assert req.is_present()
        assert not req.is_current()
        assert not req.version_ok('unknown')

        req = Require('Do-what-I-mean', '1.0', 'd-w-i-m')
        assert not req.is_present()
        assert not req.is_current()

        req = Require('Tests', None, 'tests', homepage="http://example.com")
        assert req.format is None
        assert req.attribute is None
        assert req.requested_version is None
        assert req.full_name() == 'Tests'
        assert req.homepage == 'http://example.com'

        from setuptools.tests import __path__
        paths = [os.path.dirname(p) for p in __path__]
>       assert req.is_present(paths)
E       AssertionError: assert False
E        +  where False = <bound method Require.is_present of <setuptools.depends.Require object at 0x7fbfae0d0b38>>(['/tmp/portage/dev-python/setuptools-41.6.0/work/setuptools-41.6.0-python3_5/setuptools'])
E        +    where <bound method Require.is_present of <setuptools.depends.Require object at 0x7fbfae0d0b38>> = <setuptools.depends.Require object at 0x7fbfae0d0b38>.is_present

setuptools/tests/test_setuptools.py:120: AssertionError

I can reproduce it reliably with at least pypy3.6 (7.2.0) & python3.5 (3.5.7). I haven't tested other versions yet.
Full build log: dev-python:setuptools-41.6.0:20191030-083347.log

In 41.5.1 all tests pass.

We have the same problem when trying to update to the new release in Fedora.

I have checked that paths are equivalent on both versions:
Old, 41.5.1:
DEBUG: __path__ = ['/builddir/build/BUILD/setuptools-41.5.1/setuptools/tests']
DEBUG: paths = ['/builddir/build/BUILD/setuptools-41.5.1/setuptools']

New, 41.6.0:
DEBUG: __path__ = ['/builddir/build/BUILD/setuptools-41.6.0/setuptools/tests']
DEBUG: paths = ['/builddir/build/BUILD/setuptools-41.6.0/setuptools']

Yet the test passes on 41.5.1 and fails on 41.6.0.

looks like this part is buggy:
https://github.com/pypa/setuptools/blob/master/setuptools/_imp.py#L22-L24
imp.find_module(self.module, paths)
(None, '/home/abuild/rpmbuild/BUILD/setuptools-41.6.0/setuptools/tests', ('', '', 5))

but importlib.findspec(self.module, paths) returns None and throws Importerror


docs: Add changelog for v.3.0.0

Related tasks
no tasts
Checklist for self-check

[x] Commits and PRs have been filed according to the rules on the project.
[x] The author is marked as an assigne and assigned mandatory reviewers.
[x] Required labels marked
[x] Specified related tasks and/or related PRs.
[x] Specified Changes.
[ ] Attached videos/screenshots demonstrating the fix/feature.
[x] All unspecified fields in the PR description deleted.
[ ] New code covered by tests.

Checklist for reviewers

[ ] CI passed successfully (with a green check mark).
[ ] PR is atomic, by volume no more than 400 (+-) corrected lines (not including codogen).

Design:

[ ] System design corresponds to the agreements on structure and architecture on the project.
[ ] The code is decomposed into necessary and sufficient components.

Functionality:

[ ] The code solves the problem.
[ ] Any changes to the user interface are reasonable and look good.

Complexity:

[ ] The code is clear, easy to read, functions are small, no more than 50 lines.
[ ] The logic is not overcomplicated, there is no overengineering (no code sections that may be needed in the future, but no one knows about it).

Tests:

[ ] Updated or added tests for mandatory components.
[ ] The tests are correct, helpful, and well designed/developed.

Naming:

[ ] The naming of variables, methods, classes and other components is understandable.

Comments:

[ ] The comments are understandable and helpful.

Documentation:

[ ] All labels are correct
[ ] Technical documentation updated (after approval, updates last reviewer).

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@e08b25e). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Current head 3735db3 differs from pull request most recent head 772bce5. Consider uploading reports for the commit 772bce5 to get more accurate results

Additional details and impacted files
@@          Coverage Diff           @@
##             dev      #62   +/-   ##
======================================
  Coverage       ?   96.34%
======================================
  Files          ?        4
  Lines          ?       82
  Branches       ?        0
======================================
  Hits           ?       79
  Misses         ?        3
  Partials       ?        0

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


Can we increase receive timeout for WinHTTP transport?

The default receive timeout for WinHTTP connections is 30 seconds. Storage service usually takes longer than that to respond to Query requests. As a result, client usually needs to retry 3 or 4 times to get a response within 30 seconds. This wastes quite some time and bandwidth.
So we'd like to see if we can increase the timeout to a larger value, or expose this as an option so that client can customize as needed.
As a reference, libcurl doesn't timeout once the connection is established.
REF:
https://docs.microsoft.com/en-us/windows/win32/api/winhttp/nf-winhttp-winhttpsettimeouts
https://curl.se/libcurl/c/CURLOPT_TIMEOUT.html

@Jinming-Hu, what would you see the timeout option too, specifically for query APIs, given 30 seconds is too short? Does the service has a guaranteed upper bound for how long it will take to respond?

There's also a difference in the default connection timeouts between the curl transport adapter (we set to 5 min) and winhttp (set to 1 min).

@ahsonkhan I didn't find SLA about response time. I think 3 minutes is a reasonable time. It's not too long, and it's long enough for Query.
Connection timeout doesn't break storage at the moment, so I'm fine to leave it as is.

I found this link. We don't have SLA for Query API yet.

Bump up this ask from Azure Storage service team. We are exploring to consume Azure Storage CPP SDK inside backend service. However, there is a requirement to fine grained control HTTP timeouts settings. For example, 4 kinds of timeout settings to be consumed by upper layer.
   ::WinHttpSetTimeouts(
        requestHandle,
        timeoutInMs /* dwResolveTimeout */,
        timeoutInMs /* dwConnectTimeout */,
        timeoutInMs /* dwSendTimeout */,
        timeoutInMs /* dwReceiveTimeout */);

@RickWinter @ahsonkhan Can you prioritize this feature request? We have different cases that require customizable timeouts for transport layers.

It's possible that this can increase the receive response timeout.
this can be used to increase the connect timeout.
this can be used to increase the DNS resolution timeout.
and finally this can be used to set the send timeout.
We can expose all 4 of these timeouts in seconds in the WinHTTP client options which storage can then use if it needs to modify the WinHTTP options.
Exposing WInHttpSetTimeouts seems like a poor choice since it forces all the values to be set explicitly.


Unable to suppress an error thrown in a thread

I have some trouble with an array input of ID for block function.
It seems when one ID is given, the return value whether or not it is valid can be handled in rescue in the main code. However, when an array is used, a thread handling an invalid ID throws an error in itself, interrupting the main code before the rescue in the main code handles it. Use of thread is the same result.
Here are the test codes.
def block(client,id)
	client.block( id, open_timeout: 10 )
end

Thread.abort_on_exception = true
begin
	threads = []
		ids = [abc, 0]
		ids.each do |id|
			threads << Thread.new do
				p block(client,id)
			end
		end
	threads.each(&:join)

rescue Twitter::Error::NotFound => error
  puts "User not found"
ensure
end
exit

where client is defined as usual, and abc is a valid twitter account ID in integer in the real code.
When executing the code, it throws the following error and stops.
#<Thread:0x0000000003471848@test.rb:39 run> terminated with exception (report_on_exception is true):
Traceback (most recent call last):
        11: from test.rb:40:in `block (2 levels) in <main>'
        10: from test.rb:24:in `block'
         9: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/users.rb:177:in `block'
         8: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/utils.rb:122:in `parallel_users_from_response'
         7: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/utils.rb:23:in `pmap'
         6: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/utils.rb:23:in `collect'
         5: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/utils.rb:23:in `block in pmap'
         4: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/utils.rb:123:in `block in parallel_users_from_response'
         3: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/utils.rb:72:in `perform_request_with_object'
         2: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/utils.rb:50:in `perform_request'
         1: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/request.rb:39:in `perform'
C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/request.rb:81:in `fail_or_return_response_body': User not found. (Twitter::Error::NotFound)
User not found

Or if I use array of IDs as option...
def block(client,id)
	client.block( id, open_timeout: 10 )
end

begin
	id = [abc, 0]
	p block(client,id)
rescue Twitter::Error::NotFound => error
  puts "User not found"
ensure
end
exit

a thread error interrupts the main code before it handles the error.
#<Thread:0x00000000037028a0@C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/utils.rb:25 run> terminated with exception (report_on_exception is true):
Traceback (most recent call last):
        5: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/utils.rb:25:in `block (2 levels) in pmap'
        4: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/utils.rb:123:in `block in parallel_users_from_response'
        3: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/utils.rb:72:in `perform_request_with_object'
        2: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/utils.rb:50:in `perform_request'
        1: from C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/request.rb:39:in `perform'
C:/Ruby/lib/ruby/gems/2.5.0/gems/twitter-6.2.0/lib/twitter/rest/request.rb:81:in `fail_or_return_response_body': User not found. (Twitter::Error::NotFound)
User not found

Now, the following code, using single integer as option...
def block(client,id)
	client.block( id, open_timeout: 10 )
end

begin
	id = 0
	p block(client,id)
rescue Twitter::Error::NotFound => error
  puts "User not found"
ensure
end
exit

stops on the same error, but this time only rescue in main code catches it.
User not found
How can I catch Twitter::Error::NotFound error when using array of ID or threads while suppressing the thread error?

You can set Thread.report_on_exception = false but this is probably not what you want.
See https://bugs.ruby-lang.org/issues/14143


add subnet ids variables

add subnet ids variables

I have a vpc with both private and public networks, so i need the option to chose the elastic group subnet location.
I test on my private enviroment on my fork, with both inserted and default  arguments

OK, important sanity check: as mentioned here, here, and here the code in the root of this module is example code. It is meant to show one simple way to use the modules in the modules folder and not to be used directly in prod. I appreciate the PR, and the changes you propose are reasonable, but they sound like you are trying to use the code in the root directly for a real use-case...

You right, my environment is not yet prod, but i should switch to use the inner modules directly.
nonetheless  I think that the changes will slightly increase the example code flexibility

I think that the changes will slightly increase the example code flexibility

True, but at the cost of making the examples harder to read, maintain, and test. If you think this is a critical change to have, I can merge it in, but otherwise, I'm not sure it's worth the extra overhead.

Well, I cant really call this change critical, I should just switch and use the inner modules directly as I supposed to.
thanks for the review.


[Symbol.iterator] Syntax highlighting

Version: 2.0.3
When using [Symbol.iterator](){} Syntax highlighting breaks.
Before:

After:

I am seeing this for all computed method names as well as for computed get and set accessor names within a class body.

Probably belongs here : https://github.com/Microsoft/TypeScript-TmLanguage :rose:

This issue was moved to Microsoft/TypeScript-TmLanguage#299


BAEL-1253 - jgrapht image generation

jgrapht image generation using jgraphx.
In progress article:

closed in order to re-create the PR as it now has conflicts