Docker build error: Cannot find module<EMAIL_ADDRESS>when running Remix project

Describe the bug
I am building a Docker container for a Remix project and running it with docker-compose. During the startup, the application fails to find the @remix-run/dev module, specifically cli.js, and throws a MODULE_NOT_FOUND error. The build completes without issues, but the container exits with code 1 when it tries to start the Remix server.
Link to the Bolt URL that caused the error

Steps to reproduce

Clone the project and navigate to the directory.
Follow the steps in the video by Cole Medin: https://www.youtube.com/watch?v=31ivQdydmGg&ab_channel=ColeMedin
Run the following commands:npm run dockerbuild
docker-compose --profile development up

Observe the error in the Docker container logs, which indicates that @remix-run/dev cannot be found.

Expected behavior
The container should start successfully, and the Remix server should be accessible as expected in development mode.
Screen Recording / Screenshot
No response
Platform

OS: Windows 11 Pro x64
Docker: Docker Desktop for Windows
Node.js: v20.18.0
Package Manager: pnpm (configured with corepack)

Additional context

Docker build process displays several cached steps for the pnpm install command. I attempted rebuilding without cache to ensure dependencies are installed afresh, but the issue persists.
The @remix-run/dev module is listed in package.json as a dev dependency.
Warnings about secrets in ARG or ENV are displayed, but they do not seem related to the module not being found.

C:\Users\Admin\Projects\bolt.new-any-llm>npm run dockerbuild

> dockerbuild
> docker build -t bolt-ai:development -t bolt-ai:latest --target bolt-ai-development .

[+] Building 7.2s (12/12) FINISHED                                                                                docker:desktop-linux
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 1.94kB                                                                                0.0s
 => [internal] load metadata for docker.io/library/node:20.18.0                                                                               1.5s
 => [auth] library/node:pull token for registry-1.docker.io                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 427B                                                                                0.0s
 => [base 1/5] FROM docker.io/library/node:20.18.0@sha256:a5e0ed56f2c20b9689e0f7dd498cac7e08d2a3a283e92d9304e7b9b83e3c6ff3                    0.0s
 => => resolve docker.io/library/node:20.18.0@sha256:a5e0ed56f2c20b9689e0f7dd498cac7e08d2a3a283e92d9304e7b9b83e3c6ff3                         0.0s
 => [internal] load build context                                                                                4.1s
 => => transferring context: 9.92kB                                                                                3.5s
 => CACHED [base 2/5] WORKDIR /app                                                                                0.0s
 => CACHED [base 3/5] COPY package.json pnpm-lock.yaml ./                                                                                0.0s
 => CACHED [base 4/5] RUN corepack enable pnpm && pnpm install                                                                                0.0s
 => [base 5/5] COPY . .                                                                                0.2s
 => [bolt-ai-development 1/1] RUN mkdir -p ${WORKDIR}/run                                                                                0.3s
 => exporting to image                                                                                0.6s
 => => exporting layers                                                                                0.3s
 => => exporting manifest sha256:2b5579db3bb3e90aa8d7398bd9c64ba8d0a1621eeda382337caecbe011fef8df                                             0.0s
 => => exporting config sha256:0e96f674ef7e2ff11da4ee3976af75b429abf9ab9b36a5f42c10d9ce9f996003                                               0.0s
 => => exporting attestation manifest sha256:a5a4a2fda303ccdac93c4389964e3c3e530e625485b525766ce4ac25a2da1832                                 0.1s
 => => exporting manifest list sha256:e77a5bc2f99eaf75353be2583af0cbaa8d66234187a32c8be8398957d0fa30c2                                        0.0s
 => => naming to docker.io/library/bolt-ai:development                                                                                0.0s
 => => unpacking to docker.io/library/bolt-ai:development                                                                                0.1s
 => => naming to docker.io/library/bolt-ai:latest                                                                                0.0s
 => => unpacking to docker.io/library/bolt-ai:latest                                                                                0.0s

 11 warnings found (use docker --debug to expand):
 - SecretsUsedInArgOrEnv: Do not use ARG or ENV instructions for sensitive data (ARG "OPENAI_API_KEY") (line 51)

What's next:
    View a summary of image vulnerabilities and recommendations → docker scout quickview

C:\Users\Admin\Projects\bolt.new-any-llm>docker-compose --profile development up
[+] Running 1/1
 ✔ Container boltnew-any-llm-bolt-ai-dev-1  Recreated                                                                                4.7s
Attaching to bolt-ai-dev-1
bolt-ai-dev-1  |
bolt-ai-dev-1  | > bolt@ dev /app
bolt-ai-dev-1  | > remix vite:dev "--host" "<IP_ADDRESS>"
bolt-ai-dev-1  |
bolt-ai-dev-1  | node:internal/modules/cjs/loader:1228
bolt-ai-dev-1  |   throw err;
bolt-ai-dev-1  |   ^
bolt-ai-dev-1  |
bolt-ai-dev-1  | Error: Cannot find module<EMAIL_ADDRESS>bolt-ai-dev-1  |     at Module._resolveFilename (node:internal/modules/cjs/loader:1225:15)
bolt-ai-dev-1  |     at Module._load (node:internal/modules/cjs/loader:1051:27)
bolt-ai-dev-1  |     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:174:12)
bolt-ai-dev-1  |     at node:internal/main/run_main_module:28:49 {
bolt-ai-dev-1  |   code: 'MODULE_NOT_FOUND',
bolt-ai-dev-1  |   requireStack: []
bolt-ai-dev-1  | }
bolt-ai-dev-1  |
bolt-ai-dev-1  | Node.js v20.18.0
bolt-ai-dev-1  |  ELIFECYCLE  Command failed with exit code 1.
bolt-ai-dev-1 exited with code 1

Would appreciate any guidance on resolving this issue!

The guide does not say you need it but you can try :
npx create-remix@latest
if you need to install remix.
after installing it will take you into remix environment
just press ctrl+c to exit back to console and try
npm run dockerbuild again.
Pretty sure this is not your issue as it should auto pull it but worth a try i think


Fix protoc version in java build template

Packman was using incompatible versions of protobuf which causes compile erros in the generated grpc package.

@garrettjonesgoogle Can you confirm that this is the right version to be using?

Current coverage is 88.42% (diff: 100%)

Merging #65 into master will not change coverage

@@             master        #65   diff @@
==========================================
  Files             3          3
  Lines           674        674
  Methods         132        132
  Messages          0          0
  Branches        103        103
==========================================
  Hits            596        596
  Misses           78         78
  Partials          0          0

Powered by Codecov. Last update c490fc5...7121765

Our grpc-core-proto is using protoc-beta1, so using beta-3 here causes compile errors.
I can update the grpc-core-proto to use beta3 and see if it fixes the issue.

Updating core-proto fix the issue. PTAL.

LGTM


Fix Spark Pub/Sub connector shading to leave PubsubMessage unshaded

This shades google-cloud-pubsub and unshades the generated Java from protos.
That is because we expose PubsubMessage in the generated Java package com.google.pubsub.v1.

LGTM


Gradle no possibility to specify the ClassName

The plugin takes the filename without the extension and set it as class name. This should be possible to configure.
Example.json
{
  "type": "object"
}

Results in:
@JsonInclude(JsonInclude.Include.NON_NULL)
@Generated("org.jsonschema2pojo")
@JsonPropertyOrder({

})
public class Example {
    // additional properties stuff
}

You can override the class name with javaType:
{
    "javaType" : "com.other.package.CustomTypeName",
    "type" : "object"
}

When using just the embedded version there is also the possiblilty to define the java class name inside the method without the extra property inside the json schema

@jmattheis If either of those methods work for you, we can close this.

See also #217.


Automated PR: Cookstyle Changes

Hey!
I ran Cookstyle 7.23.0 against this repo and here are the results.
This repo was selected due to the topics of chef-cookbook
Changes
Issues found and resolved with resources/app.rb

127:1 refactor: Chef/Modernize/ClassEvalActionClass - In Chef Infra Client 12.9 and later it is no longer necessary to call the class_eval method on the action class block. (https://docs.chef.io/workstation/cookstyle/chef_modernize_classevalactionclass)

Issues found and resolved with resources/config.rb

34:1 refactor: Chef/Modernize/ClassEvalActionClass - In Chef Infra Client 12.9 and later it is no longer necessary to call the class_eval method on the action class block. (https://docs.chef.io/workstation/cookstyle/chef_modernize_classevalactionclass)

Issues found and resolved with resources/module.rb

129:1 refactor: Chef/Modernize/ClassEvalActionClass - In Chef Infra Client 12.9 and later it is no longer necessary to call the class_eval method on the action class block. (https://docs.chef.io/workstation/cookstyle/chef_modernize_classevalactionclass)

Issues found and resolved with resources/pool.rb

254:1 refactor: Chef/Modernize/ClassEvalActionClass - In Chef Infra Client 12.9 and later it is no longer necessary to call the class_eval method on the action class block. (https://docs.chef.io/workstation/cookstyle/chef_modernize_classevalactionclass)

Issues found and resolved with resources/site.rb

165:1 refactor: Chef/Modernize/ClassEvalActionClass - In Chef Infra Client 12.9 and later it is no longer necessary to call the class_eval method on the action class block. (https://docs.chef.io/workstation/cookstyle/chef_modernize_classevalactionclass)

Issues found and resolved with resources/vdir.rb

130:1 refactor: Chef/Modernize/ClassEvalActionClass - In Chef Infra Client 12.9 and later it is no longer necessary to call the class_eval method on the action class block. (https://docs.chef.io/workstation/cookstyle/chef_modernize_classevalactionclass)

Released as: 7.6.0


chore(eslint): update eslint config

import ignore from .gitignore
harden eslint npm scripts

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


link element

You forgot to write a description?

Durmits... Who says I remember what I was going to ask? It must have been solved or else I would remember. =D If all our issues were like that...


Remove empty jurisdiction support without filter

A less elegant approach to https://github.com/UnityFoundation-io/Libre311/pull/32

Closing in favor of #32


This part of the code sometimes goes wrong, looks like it's returning a null value, right? Can you help fix him?

This part of the code sometimes goes wrong, looks like it's returning a null value, right? Can you help fix him? I'll look at the root of the problem @AshwinPathi

？

I just put out a fix for this. Try it out again.

Thank you. You're the best


Only one xml is needed

Current Behavior
Is it possible to generate only one XML with all the LODS included in rether than one XML by LOD.
It wood be more simple.
Expected Behavior
Only one XML for multi-lods objects
Steps To Reproduce
When launching Multi-Export glTF 2.0
Environment
- OS:Windows 11
- Blender: 3.1.2
- glTF-Blender-IO-MSFS:1.1.5

Anything else?
No response

This was already adressed in on of the earliest release
To make It work like we were used to with the former exporter,
Create collections with the name
Object_LOD00
Object_LOD01
....
Object_LOD04
In the exporter, select Grouped by collection
Click Reload Lod a couple of times
Should work
The band of the xml Will be "Object", and Will contains all the lods

You can also make it like this and it works fine.


Fe: convert fonts to .woff

https://github.com/webaverse/webaverse.com/tree/white/public/fonts

Not really that prio, just would save bytes and parse time.


Depend on carrierwave 1.0 and google-cloud-storage (#11)

Depend on carrierwave 1.0

updated both google-cloud-storage and carrierwave

@codegoalie changes looking good, except on Ruby 2.0.0 due to a dependency that is no longer supporting Ruby 2.0.0. I'll try to keep support for Ruby 2.0.0 - if it's not too much hassle, If it is, I'll drop support for 2.0.0 as well.

Looking good! @codegoalie @johnmcconnell would you both be okay with trying/testing out a pre-release version of this gem?

I deployed against master in this repo earlier today and things appear to be working correctly!


UTM app has the problem of white screen,when I turn on VM, it will flash back to the desktop.

I created a wrong VM. I want to delete it, but I couldn't find any files (All types of files, someone told me that the suffix of VM's filename was: "*.utm")in the UTM app directory in the file application.
I have some additional information here. I hope it can help you repair this app:
iPad system version: iPad OS 13.3.1
iPad model : iPad 5 WiFi
iPad jailbreak with checkra1n
If I don't use my iPad for checkra1n jailbreak, UTM app will flash back (the IPA file of UTM is use the fake design, I think the flash back should be an IPA signature problem).
Sometimes the UTM app also has the problem of white screen.
When I turn on VM, it will flash back to the desktop.

Who can help me fix it？

#32


MAGEIO FFMPEG_WRITER WARNING

Environment

python3
Ubuntu 18.04 x86_64

TIA
python3 yolo_opencv.py --input sampledata/commuters.mp4 --config cfg/yolov3-tiny.cfg --weights yolov3-tiny.weights --classes cfg/yolov3.txt
IMAGEIO FFMPEG_WRITER WARNING: input image is not divisible by macro_block_size=16, resizing from (1920, 1080) to (1920, 1088) to ensure video compatibility with most codecs and players. To prevent resizing, make your input image divisible by the macro_block_size or set the macro_block_size to 1 (risking incompatibility).
Detecting objects in frame 1
Traceback (most recent call last):
File "yolo_opencv.py", line 188, in
processvideo(os.path.join(args.input))
File "yolo_opencv.py", line 140, in processvideo
image = detect(frame)
File "yolo_opencv.py", line 81, in detect
net = cv2.dnn.readNet(args.weights, args.config)
cv2.error: OpenCV(4.2.0) /io/opencv/modules/dnn/src/darknet/darknet_importer.cpp:214: error: (-212:Parsing error) Failed to parse NetParameter file: yolov3-tiny.weights in function 'readNetFromDarknet'
No frames have been written; the written video is invalid.
Finishing stream 0:0 without any data written to it.

Problem is with yolov3-tiny.weights file. Please check the filename/location.

Thank you


Repeated measurements

The wiki article here  states:

Repeating the same measurement without any evolution of the quantum state will lead to the same result. If the preparation is repeated, subsequent measurements will likely lead to different results.

This simple test shows the 10 measurements do often produce different results (a).
    for(int i=0; i<10; i++)
    {
        std::tie(a, b, c) = measure(bell_state, gt.H, {0});
    }

Does that imply the experiments simulated in your Quantum++ assume the preparation is indeed repeated?
In a physical quantum computer, that would represent a scenario where the input device keeps on producing the bell state. Is that correct?

The functions in Q++ do not modify their arguments (this design was intentional, to resemble more functional-programming style and to facilitate testing). In general, in case you need to modify the argument of a function, you need to have the argument on the rhs of an assignment, like in this case where we permute the subsystems of a 3-qubit state vector |psi>
psi =  syspermute(psi, {0,2,1}); // permutes the subsystems of psi

So if you need to simulate repeated measurements on the same state just measure the second time on the result of the previous measurement, like so:
std::tie(m, probs, states) = measure(bell_state, gt.H, {0}); // states[m] will be our output state
std::tie(n, probs, states) = measure(states[m], gt.H, {0}); // now n == m

Great. Thanks.


Figure out recording of discord calls

@UnevenCoder could you please take this

I think there's a discord bot that can help with recording the calls

I don't think discord has a feature to do that.
https://support.discord.com/hc/en-us/community/posts/360047224131-Record-video-audio-calls
There are some work around ideas like this one -
https://www.howtogeek.com/677198/how-to-record-discord-audio/
We should take consent/have a notice issued before recording with the bot.

Bot idea is the best or we can use OBS


Builds fail with both Xcode and bootstrapping enabled

Describe the bug
When bootstrapping is enabled (the default state) and the --xcode build flag is used, the build fails with the following error:
clang: error: no such file or directory: '/Users/<my_user>/Developer/swift-project/build/Xcode-RelWithDebInfoAssert/swift-macosx-arm64/lib/RelWithDebInfo/libswiftCompilerModules-bootstrapping1.a'

My primary machine is Apple Silicon, but I saw the same error on Intel as well.
To Reproduce
Steps to reproduce the behavior:

Build with the following command:

utils/build-script --skip-build-benchmarks \
  --skip-ios --skip-watchos --skip-tvos --swift-darwin-supported-archs "$(uname -m)" \
  --sccache --release-debuginfo --swift-disable-dead-stripping \
  --xcode

...
Expected behavior
I expected a successful build with a usable Xcode project
Environment (please complete the following information):

OS: macOS 12.3.1
Machine: 16-inch MacBook Pro with M1 Pro (16GB Memory)
Xcode Version: Xcode 13.3 (13E113)
Swift commit: 48208380e33fa021bac429748744a978f6e5145a

Additional context
If you build with ninja, the build succeeds.
If you build with Xcode and also disable bootstrapping with --reconfigure --bootstrapping=off, the build succeeds.
The issue is only with Xcode and bootstrapping.
There are some related threads on Swift forums here:
https://forums.swift.org/t/problems-with-build-script-building-compiler-with-xcode/53477
https://forums.swift.org/t/no-xcode-builds-on-ci/53521
https://forums.swift.org/t/building-swift-compiler-from-source-fails-when-not-using-ninja/54656/8

Also never able to build without --bootstrapping=off
But just for reference #40838 this was where bootstrap was enabled for Xcode builds


Not displaying current slide when chaging from mobile view to desktop view.

Hi,
Currently, my slide contain 8 items.. In mobile view, i am seeing 8th item(one item per view) and when switching to desktop view, it reseting to initial item. Please guide me solve this issue.
Thanks,
Premkumar

Hi,
Any feedback for above post..
Thanks,
Premkumar.

Hey @mpkumar87 , please read https://github.com/kenwheeler/slick/blob/master/CONTRIBUTING.markdown for some help, if you can satisfy the contribution guidelines we'll do our best to help you :)

fixed on master.


Update reqwest to 0.11, so we can persistently store cookies

As far as I can tell, there is no way to store cookies before shutdown and restore them before launch in reqwest = "~0.9".
In reqwest = {version = "0.11", features = ["cookies","json"]} it is possible to do so however.

was fixed by #14


feat: latest news feed #312

@akashsiddamshetty are you using a plugin to generate the .css / source maps? I haven't generally committed these. :-)


Updates Needed for About Us and Cart Pages

Email Input Field: The shape of the email input tag does not match the design of the message input box in services ,collections ,etc pages . I will adjust the styling to ensure consistency.
Social Media Tags: I would like to add more social media tags, such as YouTube and Snapchat, with the same features as the existing ones.

Chagning

The navbar is doubled in this ,also the items in the cart have small font size ,would love to change in this
Given my strong background in HTML, CSS, and JavaScript, I would like to request that this task be assigned to me. I intend to mark this under the GSoC extended projects.

Can you assign me this issue?

i would like to make changes can you assign me this issue?

i was assigned this project first , please dont add other assingess to this

@Anjaliavv51 do you want me to add extra icons like these to all the webpage?

@Anjaliavv51 CAN U CHECK MY 2 PR request regarding 2 issues i.e #183 #343

complete this issue as the pr was raised 3 weeks ago

please assign the level and close this issue


Explicitly close ending TCP connection sockets

This came up while testing tcp connections. Sending many events over individual connections left lots of those connections in the CLOSE_WAIT state.

@dominiklohmann I believe we should also guard the read in the action: https://github.com/tenzir/tenzir/pull/4674/files#diff-a3ebba84face4449e084a934dd97e529546af867e7b4fe32dc73062e8febe5a1R387-R388. Can you verify and in case make the adjustment?

Just talked with @lava about this, he'll push that onto this PR and take that over. We should definitely make that change.


Using .property on maps defined by keyBy should return original variable instance

Is it possible/desirable for maps generated by keyBy to return original variable instances?
class Foo extends Variable<{ Name: string, Id: number }> {}
const Collection = VArray.of(Foo)

const collection = new Collection([{
  Name: 'Kerry',
  Id: 123
}])

const index = collection.keyBy(model => model.get('Id'))

// fails
assert.strictEqual(index.property(123), collection.property(0))

// fails
assert(index.property(123) instanceof Foo)

let newProps = {
  Name: 'Jerry',
  Id: 123,
}

index.set(123, newProps)

// fails
assert.strictEqual(collection.get(0), newProps)

Just noticed that index.get(123) actually behaves this way. So that's cool that it's possible to retrieve the instance, but I'm wondering if this makes behavior inconsistent with valueOf()?
Also, would it be possible/desirable for the below to be equivalent?
// or index.property(123).put(newProps),
// depending on how you feel about above
index.get(123).put(newProps)

index.set(123, newProps)

I am not sure how that would be possible if property is really going to return a variable that represents a given id at any point in time, for example:
const collection = new Collection([])

const index = collection.keyBy(model => model.get('Id'))
const slot123 = index.property(123)
const kerry = new Foo({ Name: 'Kerry', Id: 123 })
collection.push(kerry)
slot123.valueOf() -> value of kerry

But, it is not possible for slot123 === kerry before the kerry instance even exists, as far as I can tell.


2.3 console fix for watched resources

https://github.com/stolostron/backlog/issues/19641
Signed-off-by: James Talton<EMAIL_ADDRESS>

Thanks for the updates.
/lgtm


Added in daq functionality

Attempted to architect daq functionality correctly

Just wanted you to look at how I've implemented the pulse count on the daq and see if it looks right


Sync git remote parsing logic

Issue with git remote parsing logic

User has a private bitbucket server, with .git/config remote:  url =<EMAIL_ADDRESS>

user uses netlify init --manual and we create a repo with a bad spec:  Repo.path is set to ~steve.steve/netlify-test.git meaning we don't try to reach his server when cloning and fail

user uses netlify sites:create --manual --with-ci and it works correctly, setting Repo.path to git@bitbucket.companyxyz.com:~steve.steve/netlify-test.git which works well.

It would appear that init and sites:create are using a slightly different git remote parsing mechanism.
one uses parseGitRemote https://github.com/netlify/cli/blob/master/src/commands/sites/create.js#L119 looks like its working and the other uses getRepoData https://github.com/netlify/cli/blob/master/src/commands/init.js#L61 which appears to not be working
The git remote parsing logic appears to be different for each.
It appears that the parsing logic for netlify sites:create --manual --with-ci is working correctly in the scenario above

The repo_path is added from prompt here https://github.com/netlify/cli/blob/master/src/commands/sites/create.js#L122 in the working path.
It's not added in this https://github.com/netlify/cli/blob/master/src/commands/init.js#L61 init flow

Related: https://github.com/netlify/cli/issues/233

Hi, this issue with git remote parsing logic remains problematic in some cases. We have an existing production website, self-hosted on a private GitLab and running on Netlify, and we deploy using Netlify CLI. We would like to connect our repository to Netlify to use Deploy Previews but the netlify init --manual command can't reach our GitLab server.
Everything is working fine for our production website so a new site creation is not really an option for us.
I agree with @zachhardesty7, a repository link edit via UI would be very usefull in this case :)

@zachhardesty7 or @Thms-Brnrd can you explain what you mean by "a way to modify the repo link via UI"?

@verythorough if I remember this issue correctly, I ran into the same problem as here. netlify init --manual leaves out the host and username needed for the repo to work. The ability to change the "Repository" link in the /settings/deploys#continuous-deployment settings would at least help mitigate this issue.

Ah, I see what you mean now. However, since that UI field is directly connected to the "automatic" setup for continuous deployment, I think that would actually be a much bigger change than fixing the bug in this issue.
In the end, maintaining two different commands that do essentially the same thing doesn't really make sense. I suspect that the best path forward is to make sure sites:create includes all of the functionality of init and then gradually deprecate init.

@verythorough I agree. Having both commands is mostly just confusing.
Idea: instead of the ability to change the link, creating a CI-enabled site via arbitrary Git URL on the "/start" page would also alleviate the issue. Currently it just shows the big three online repos, but since it appears that Netlify supports any git-based VCS repo, a field to enter a link would be nice.

@zachhardesty7 It turns out we have an internal feature request filed for that very thing! I've added a link to your comment for tracking.

Came across this very problem today; it's still a thing with CLI netlify-cli/17.16.2 wsl-x64 node-v20.11.1. After some frustrating steps, the --manual --with-ci parsed the correct repo path, and allowed changing it. I'll remember to use this from now on.
This is the first time though, as in previous attempts, init parsed the path correctly. Not sure what is different.

I celebrated too early; the initial build indeed pulled the correct repo, however after a git push with some small changes, the repo url had changed again to the faulty one with just the project path.
It appears the CLI does have the ability to change the repo url. Would love to have control over its choices!


Dynamodb incorrectly allows a mix of "expression" and "non-expression" parameters

Versions
moto: 3.1.16
boto3: 1.21.46
Code
table.get_item(
    Key={
        'key': 'the-key'
    },
    AttributesToGet=[
        'foo'
    ],
    ProjectionExpression='foo'
)

Expectation
An error. This is what is produced by boto3 when calling this against a real table:
botocore.exceptions.ClientError: An error occurred (ValidationException) when calling the GetItem operation: Can not use both expression and non-expression parameters in the same request: Non-expression parameters: {AttributesToGet} Expression parameters: {ProjectionExpression}

Reality
No error.

Thanks for raising this @paulmdavies - marking it as an enhancement to add this validation.
If this is something you want to implement yourself, PR's are always welcome! Let me know if you need help getting started.


How can I use an ES6 module in a worker?

The wa-sqlite build produces ES6 modules, which is nice for development. But most applications should put time-consuming operations in a Worker, and support for ES6 modules in a Worker isn't universal yet. Worker code generally needs to be bundled.
Emscripten modules need to load their associated .wasm file, and the generated code for ES6 uses import.meta.url as its default location. import.meta.url is not valid outside an ES6 module, however, so the bundler has to translate this to something else. When the target context is a Worker, that something else needs to be valid in a Worker, so something like document.currentScript.src, which is the substitution my bundler chooses, causes a runtime exception.
The fix I use is to configure a bundler string replacement plug-in (@rollup/plugin-replace in my case) to replace import.meta.url with an empty string. There may be better approaches out there but this is sufficient for me so far.

To create a Web Worker and being able to use ESM add a 2nd argument with { type: 'module' }
new Worker(url, { type: 'module '})

You can load an ES module directly in a Worker if browser compatibility is not an issue for you, but the point of this FAQ item is that currently bundling is required to work across all WASM-capable browsers (i.e. including Firefox) and this shows a workaround for a bundling problem.

I know, i just wanted to help out with some faq q/a stuff for others who also wonder how to load esm stuff in workers ;)

As an FYI, it appears there is some activity in Firefox for module workers.

Worker modules are now in the Firefox nightly build, targeted for stable channel release in mid-February. It feels buggy at the moment, but that's just how nightlies are.

If anyone comes here trying to put wa-sqlite in a service worker (which as of this posting still has terrible support for modules...) and is using vite-plugin-pwa, then you can put something like this in your VitePWAOptions config:
  injectManifest: {
    ...
    // @ts-ignore - this is a valid option somehow... and is required for wa-sqlite to work
    plugins: [replace({ "import.meta.url": mode === "development" ? "import.meta.url" : "undefined" })],
  },

And at the very least, it seems to work for me. As mentioned above, an empty string probably also works.


@roadiehq/scaffolder-backend-module-utils to handle multiple documents in the same catalog file

If you have multiple documents in the same catalog file, such as
apiVersion: backstage.io/v1alpha1
kind: Component
metadata:
  name: test
  description: Test
  tags:
    - abc

---
apiVersion: backstage.io/v1alpha1
kind: API
metadata:
  tags: def

when you try and use the roadiehq:utils:merge action to merge content (lets say i wanted to add another item to tags), you get a YAML error:
Source contains multiple documents; please use YAML.parseAllDocuments() at line 50, column 1:

Feature Suggestion
Allow multiple documents in the same catalog file to be parsed.
Possible Implementation
Switch https://github.com/RoadieHQ/roadie-backstage-plugins/blob/main/plugins/scaffolder-actions/scaffolder-backend-module-utils/src/actions/merge/merge.ts#L224 to use YAML.parseAllDocuments instead of YAML.parse and merge everywhere - also do the same for JSON source.
Context
I'm trying to merge new content in an action with multiple docs.

Hm, it's a bit tricky. It works only if you always want to do the action on all of the yamls inside one file as soon as you only want to add tags to a specific yaml inside the file it gets really tricky/cumbersome to set it up figure out a good experience how to do it 🤔


Dockerize application?

Hello @Funami580 ,
will there be a dockerized version of the application? This would be very (!) helpful.
It won't run on fresh Ubuntu server installation with error Chromebrowser err 111, can't download.
Best regards,
Martin

Works for me. Full error?

Proxmox 8.2: Container Ubuntu 22.04 LTS
Installation of "sdl"
wget https://github.com/Funami580/sdl/releases/download/v0.1.3/sdl-0.1.3-linux-x64.gz
gzip -d sdl-0.1.3-linux-x64.gz
mv sdl-0.1.3-linux-x64.gz sdl
chmod 755 sdl
chmod +x sdl
mv sdl /usr/bin/

Trying to download series from s.to
root@sdl:~# sdl https://s.to/serie/stream/breaking-bad
xx:xx:xx.xxx INFO  > uBlock Origin not installed... Installing...
xx:xx:xx.xxx ERROR > Failed to create ChromeDriver: could not connect to ChromeDriver: error creating new session: webdriver server did not respond: error trying to connect: tcp connect error: Connection refused (os error 111): webdriver server did not respond: error trying to connect: tcp connect error: Connection refused (os error 111)

On my laptop (Ubuntu 22.04 Desktop GUI) it works without issues.

Works for me. Full error?

i re-installed google-chrome and chromedriver. now it works. Still dockerized version would be awesome. :)

Nice! I don't feel any personal motivation to create a docker version, but feel free to do so yourself.


[DO NOT MERGE] Send anonymised_user_agent_and_ip to BigQuery

As on Find, we only send a value if we have a remote_ip for the user
Context
We want to be able to group events by user, but we don't want PII (which includes IPs) in BigQuery. Copy the TVS approach already implemented in Find.
Changes proposed in this pull request
Add anonymised_user_agent_and_ip field to the request details we send to BQ.
Link to Trello card
https://trello.com/c/hBfnkeMI/3626-add-user-anonymous-identifier-field-to-event-being-sent-to-bq-apply
Things to check

[X] This code does not rely on migrations in the same Pull Request
[X] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[X] API release notes have been updated if necessary
[X] Required environment variables have been updated added to the Azure KeyVault

Added DNM because we need to create the fields in BQ


Allow Code to be a default editor for json

Visual Studio Code has a pretty great JSON editor, and seems to have defaults for things like Windows Terminal Preview.  However, Visual Studio (not Code) takes the default extension registration for JSON.
It would be nice if Visual Studio Code could be registered as an editor for .json files.  Currently Windows does not appear to provide a means to set file association so that .json files can be edited by Code.

Is #61084 relevant? You haven't posted your version numbers.

I don't believe so.  I just installed vscode today.  Apologies for not including version numbers.
Version: 1.43.0 (system setup)
Commit: 78a4c91400152c0f27ba4d363eb56d2835f9903a
Date: 2020-03-09T19:47:57.235Z
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.18363

For me this procedure works:

In Windows Explorer right-click on a .json file and choose "Open with" to get this dialog:

Choose "More apps"

Find Visual Studio Code in the list, select it and click OK.

My version info:
Version: 1.43.0 (user setup)
Commit: 78a4c91400152c0f27ba4d363eb56d2835f9903a
Date: 2020-03-09T19:47:57.235Z
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.18363

If this doesn't work for you, perhaps the fact that yours is "(system setup)" may be significant.

Right, step 1 requires me to find a .json file.  It seems like the official Windows way to set file associations is through Setup -> Apps -> Default Apps -> Choose default applications by file type.
"Microsoft Visual Studio 2019" shows up in that list, but "Visual Studio Code" does not, and there is no "More apps" option.

This became an issue today because I was trying to change the settings in Windows Terminal Preview, which opens the json config file in the default editor.  Visual Studio 2019 takes a long time to start up, so I thought I would change the file association.
Because I didn't have the location of the actual file, there was nothing to right-click.  I suppose I could have created a new text file, renamed it, and then right-clicked on it to get the "Open With" dialog, but if the recommended way of changing file associations in Windows is through "Choose default applications by file type", and Visual Studio Code is designed to edit json files by default, then Visual Studio Code should appear in that list.
It shows up there for .txt, but not for .json.

Additionally, it should be noted that if I create an empty file and rename it to have an extension that ends in .json, I can right click and change what it opens with there.  In that list, Visual Studio Code does show up:

It still does not show up in the system-wide list of handlers for json, however.

It still does not show up in the system-wide list of handlers for json, however.

That seems like an issue of Windows though, I suggest giving them that feedback instead.

It seems like an issue with VSCode, though.  Visual Studio 2019 registers itself as an editor, but VSCode does not.


feat(config): add api config for branching override

What kind of change does this PR introduce?
Feature add tooling around api config to be used inside branching for config override.
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

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

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 60.283%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.9%

Totals

Change from base Build<PHONE_NUMBER>8:
0.05%

Covered Lines:
6519

Relevant Lines:
10814

💛  - Coveralls

@sweatybridge

I don't quite follow the intended used case on branching. I'd imagine the following code snippet is close to how it's used?
cliConfig, _ := config.NewConfig()
body := cliConfig.Api.ToUpdatePostgrestConfigBody()

It's allowed to call public methods on private struct.

Yes, but the actual use case is for splitting each of the "service config update" in their dedicated scope like so: https://github.com/supabase/branching/blob/41c0391c65126fc9f0da978043a2182cac75c2b7/internal/task/configure.go#L13-L35
The idea is to expose what's "shared" but keep what's only cli specific private.


Allow creating vm with vapp options

Description

This PR allows users to pass vApp Options when creating VM in VMWare vCenter. Through vApp Options vCenter can pass static IP address from configured network protocol profile, so users don't need to install dhcp server.
Related issue(s)

https://github.com/rancher/rancher/issues/15817
https://github.com/rancher/machine/pull/13

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "vsphere-vapp-options" git@github.com:kinolaev/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

I don't understand why travis ci build failed, same commands works well on my machine, I think it not related to my PR.

Hello @shin-, can you please review this PR? Automatic static IP allocation is very useful feature for vSphere users!


Fix build - 64 bit functions do not exist

Hi, see topic again 🤠

Thank you for this contribution!
By the way, I am curious what is your use case that you compile 32-bit code on Windows?

Thanks - You're probably right 👍
It's no longer necessary, but as long as it's still available and not completely out of the world, it can still be supported, right?

Yes :)


Add cluster data source, tidy Makefile

Hi @fhke , before we can merge this PR and make a new release. Could you please update your fork to the latest commit on main? It contains some breaking changes.
See: https://github.com/symbiosis-cloud/symbiosis-go/releases/tag/v1.0.12

@fhke small reminder. Have you seen my earlier message ? We would like to have this PR merged soon. Thanks!

Hi @thecodeassassin, apologies for the delayed response. I've rebased from main, could you review please?

Hi @thecodeassassin, apologies for the delayed response. I've rebased from main, could you review please?

LGTM


JA3 and JA4 test adjustments and additions  - v7

Previous PR: #1634
This PR introduces test cases for TLS and QUIC JA4 calculation, as well as some adjustments for JA3.
Changes to previous PR:

Rebase against current master.
Remove JA4s related data from reference output.
Use version 8 in requires.min-version.
Remove HAVE_NSS and HAVE_LIBJANSSON where unnecessary.
Separate JA3 and JA4 related changes in to single commits.
Clean up requires a bit.

Ticket
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/6379

Next PR: #1761


Keys API changes after the API Review

Changes applied:

Update methods should take IEnumerable instead of IList
Remove CreateKey method that only takes KeyCreateOptions
Change DateTime references to DateTimeOffset
Change KeyOps variable name for KeyOperations
All list operations should be GetKeys - ish (not contain All)
GetKeys should return KeyBase, not Key (async and sync)
GetKeysVersions should return KeyBase, not Key (async and sync)
Remove JsonWeb from types like JsonWebKeyType, JsonWebKeyOperations, and JsonWebKeyCurveName,
Convert to enum the JsonWeb types

It also:

Renames the folder of the project form Keyvault to KeyVault
Add copyright info in each file

Testing @Azure/adp-keyvault

Could someone please review this PR?

@KrzysztofCwalina could you please take a look?


dynamic_update_index_in_slice vs scatter

Right now our outputting of times is probably vmap'ing to scatter, which might be inefficient? Need to think about.

Closing: XLA internally converts back if it can.


improved: update core

Removed cheerio and replaced it with node-html-parser. Cheerio is bloated and outdated while node-html-parser is lightweight, faster and regularly updated. Cheerio depends on 6 external dependencies as well while node-html-parser only depends on he module.

hum just curious, why you say cheerio is outdated? The last cheerio release was shipped in less than one month ago.
I agree with "cheerio is bloated" but also it is the consequence when you need to support a lot of edge cases.
I can't feel too confident with node-html-parser; don't misunderstand me I'm sure it's a good project but I can't feel confident with it in the long term 🤔

No problem just thought I would help since it was easy. Cheerio on npm was published a year ago. Take a look at node-html-parser on npm and github, it's quite popular and old as well. It's a fork of original fast-html-parser. Cheerio might be more appropriate on larger projects but simple like this one is overkill. I would in fact prefer regex over any parser since it's parsing from fixed location.

I decided to keep cheerio, mostly because I use this library in combination with metascraper and since cheerio is already a metacraper dependency, cheerio is deduped and a not big deal in terms of disk space.
BTW, thanks for creating hooman! I have pending add it to html-get 🙂


rel4.5.6

4.5.6 changes since 4.5.5

allow handling of *robots.txt and sitemap.xml asset files

Source changes in release package, I don't have rights to merge on this repository
https://acoustic-jiraconf.atlassian.net/browse/CONT-5627


Provide functionality to package tests in testerina

Description:
When I tried to implement websub related tests, it occurs to me that having the $subject would be an ideal case to implement advanced tests. For example, handling this kind of configurations.
[1] https://github.com/ballerina-platform/ballerina-lang/blob/master/tests/jballerina-integration-test/src/test/resources/testng.xml#L251
Steps to reproduce:
The functionality would be ideal when we try to implement this [2] kind of test in testerina. The reason is websub only contains a static hub and we cannot test different configurations of websub hub within a single packaged test cases.
[2] https://github.com/ballerina-platform/ballerina-lang/tree/master/tests/jballerina-integration-test/src/test/resources/websub/publisher/src
Affected Versions:
1.3.0-SNAPSHOT

In ballerina, a module is considered a test suite and the user can give configurations inline with the test cases which is similar to the testng.xml in TestNG in Java. There are also options to enable/disable groups or to execute a certain set of tests with the CLI. Refer bal test --help for more information.


”Remetea-Pogănici”, ”Scăiuș”, ”Valea Mare” are listed in the drop-down menu

Description:
When user writes ”Remetea-Pogănici”, ”Scăiuș”, ”Valea Mare” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Remetea-Pogănici"

Expected results
The location appears in the drop-down menu as "Remetea-Pogănici, Caraș-Severin(Fârliug)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Scăiuș"

Expected results
The location appears in the drop-down menu as "Scăiuș, Caraș-Severin(Fârliug)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Valea Mare"

Expected results
The location appears in the drop-down menu as "Valea Mare, Caraș-Severin(Fârliug)".


feat: release 0.43.0

Codecov Report

Merging #1082 (7461fc7) into develop (a0db0df) will increase coverage by 1.92%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1082      +/-   ##
===========================================
+ Coverage    57.56%   59.48%   +1.92%
===========================================
  Files          346      406      +60
  Lines        12828    15032    +2204
  Branches      2405     2814     +409
===========================================
+ Hits          7384     8942    +1558
- Misses        5200     5819     +619
- Partials       244      271      +27

Impacted Files
Coverage Δ

src/swiper/props.ts
68.75% <0.00%> (-31.25%)
:arrow_down:

src/dialog/props.ts
70.00% <0.00%> (-30.00%)
:arrow_down:

src/calendar/props.ts
70.00% <0.00%> (-30.00%)
:arrow_down:

src/date-picker/props.ts
70.00% <0.00%> (-30.00%)
:arrow_down:

src/drawer/props.ts
71.42% <0.00%> (-28.58%)
:arrow_down:

src/pagination/props.ts
72.72% <0.00%> (-27.28%)
:arrow_down:

src/affix/affix.tsx
39.06% <0.00%> (-27.01%)
:arrow_down:

src/popup/props.ts
75.00% <0.00%> (-25.00%)
:arrow_down:

src/select/props.ts
75.00% <0.00%> (-25.00%)
:arrow_down:

src/slider/props.ts
75.00% <0.00%> (-25.00%)
:arrow_down:

... and 218 more

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 e56479e...7461fc7. Read the comment docs.


A lot of crashes in the last ß version auto refreshing the TLs

It's very annoying: every time I open AS or change the TL, it crashes, I think it's a bug made every time it refresh automatically the TL.

I've disabled "sync when using the app" option and it doesn't crash when it get opened but it does if I select the "User" TL (my notes).

As I wrote in different places, I need to reproduce this or to receive a crash dump/log with a stack trace in order to know, where and when the problem occurs.
I constantly use AndStatus on two devices - it never crashes. Both devices a manufactured in 2014...

The thing is that, I can't use the correct debugging settings to share you the log because AS crashes and I have to delete the data to use it again.

Maybe it's an AIDE bug or something and I'm installing it wrongly (I'll git pull it and install it again) btw, I'm translating the user manual and I've written a note about Syncing on Crowding.

If application crashes, its own log may not have the needed stack trace. So in order to receive the needed information on a crash you should (as one of ways...) connect your device to PC and read/copy logs from PC...

I have no "logs" from today, but I'll send you the entire "files" folder.

Ok, /storage/emulated/0/android/data/org.andstatus.app/files/logs is empty, but databases is not. What should I do?

@moshpirit  Logs, created by AndStatus itself, are not for crash dumps.
Crash dumps / log records, containing detailed crash information, are recorded by Android system and may be captured by special tools.
Tool examples are below:

If your device is rooted, you may use CatLog application to capture crash logs: https://play.google.com/store/apps/details?id=com.nolanlawson.logcat&hl=en

Please see here about another way to get crash report: https://github.com/andstatus/andstatus/issues/183

Eclipse application on a PC, connected to the device, is one of such tools.

About such ways see e.g. here: https://testlio.com/question/how-to-get-consolecrash-logs-for-android

I think that getting used to such tools may be too much for most of users, this is why I'm usually asking to install AndStatus from Google Play, which enables crash reporting out of the box (and I use it!).
For users downloading AndStatus from other sources (or building AndStatus themselves), I guess the only answer would be integrating such crash reporting tool inside AndStatus (e.g. this: https://github.com/ACRA/acra )

@moshpirit I added ACRA (https://github.com/ACRA/acra ) crash reporting to AndStatus. It automates collecting of all required information and allows a User to send the report via email to AndStatus team.
Please use it for your crash reports!

Of course, thank you!!!

Hmm looks like it doesn't crashes anymore (yet). I'll close it the sunday if it doesn't crashes anymore in the entire week, OK?

@moshpirit  Thank you for the Report!
The "Out Of Memory" Error occurs at a seemingly harmless place while listing AndStatus log files:
for (String filename : dir.list()) {
The problem may be only in a case the directory with log files has too many files (see e.g. here: http://stackoverflow.com/questions/3139073/how-to-list-a-2-million-files-directory-in-java-without-having-a-out-of-memory-e ).
Q1. Could you find the log folder and count (or maybe even list them)?
Q2. How often do you get this error? Please keep me informed...
Suggestions:

In order to clean the storage (including directory with logs) I would suggest Backup AndStatus - > Uninstall -> Install AndStatus -> Restore Backup.
Please investigate memory (RAM) consumption by different applications. Maybe some other app takes too much memory...

That's because I changed the storage settings, letting AS to download more attachments than usual.

I would recommend you to set logging level to default "INFO" and to stop logging to file.
Please also try my above suggestions.
Regarding attachments: Yes, images seem to be the most memory consuming element. I will implement an option to reduce a size of images, displayed in a Timeline: this will certainly help in a cases, where there is not enough RAM. BTW, images are smaller in a portrait mode...
However, I would ask you to investigate, how other applications use your device's memory. Maybe some of the other applications is taking too much memory...

@moshpirit Please test with the latest commit: I changed scaling of attached images to produce smaller images (and hence to occupy less memory).
You should get "Out of Memory" less often now.

Ok, I'll do it! Thank you! Btw, I think I remember that I made an issue asking for it :-)

Thank you but I think that there should be 2 different menus when we hold down a message:

If we push the note/user → Normal menu except "view image"
If we push the picture → View image in full resolution

That would make the menu a little more clear and if we separate it in 3 menus would be something even better:

Note:

reply
reply to all
share
favourite
repeat
open on Internet (I think it should be "open using a browser")
open conversation

User:

Direct message
Messages by "Username"
Follow "Username"

Picture:

View in full resolution

I'll open a new issue for that.

OK, it was Telegram, it was consuming 500+ MB of cache ¬¬


Show until today's date

Hello
I have 2 questions about the usage of the library.

How to limit react-native-modal-datetime-picker calendar until today's date?
How to limit in a range of dates?

I will appreciate if I will have an answer as soon as possible.
Thanks in advance

@fabioplaka please check the documentation of https://github.com/react-native-datetimepicker/datetimepicker for the available props.
TL;DR:

How to limit react-native-modal-datetime-picker calendar until today's date?
Use the minimumDate and maximumDate props

How to limit in a range of dates?

https://github.com/mmazzarolo/react-native-modal-datetime-picker#how-can-i-set-a-minimum-andor-maximum-date


Section 1.6 Reminder to run new systems

In section 1.6 we add three new systems, MapIndexingSystem, MeleeCombatSystem, and DamageSystem. There is a snippet adding MapIndexingSystem to run_systems. It would be really helpful to have a quick reminder for MeleeCombatSystem and DamageSystem to run those similar to how there are reminders when adding a new component. This has been a really fun guide so far, thanks for the hard work of maintaining it!

Thanks! I'll see what I can do. Going a bit more slowly than normal, due to the ISP I work for being REALLY busy with everyone working from home.


Create new roles (button & functionality)

Add functionality for creating new roles

There are two buttons (inside  and inside ) to add this functionality to.

As far as I can see, this is possible. You have to create an Organization, and in the roles view, there is a create role view button on the top right.


Bug Report: Offset Buggin

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
im using windows 64 , when changing the offset thru the options menu , it doesn't change the numbers visually , but instead , it changes when hovering to another option . For example :
https://user-images.githubusercontent.com/81717689/122011374-a05f2f80-cd81-11eb-8ba1-8afe6935ab09.mp4

duplicate #799

#891


Cannot add font path with "xset fp+" to X11 server

Windows Version
Microsoft Windows [版本 10.0.19045.2788]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
RHEL 8
Other Software
No response
Repro Steps
Run: xset fp+ /usr/share/X11/fonts/75dpi
Expected Behavior
Return:
xset:  bad font path element (#2), possible causes are:
Directory does not exist or has wrong permissions
Directory missing fonts.dir
Incorrect font server address or syntax
Actual Behavior
Cannot add font path with "xset fp+" to X11 server
Diagnostic Logs
No response

Hi, elsaco. Thank you for your reply! The command "xset fp+ /mnt/wslg/distro/usr/share/fonts/X11/75dpi" works well on wsl2. It is different from the command "xset fp+ /usr/share/X11/fonts/75dpi" on a physical machine. thinks!

Your WSL distro is being mounted under /mnt/wslg/distro/ and the system distro (a.k.a WSLg) starts a font server.


PubSub client Receive method never returns when context is cancelled

Bug
Client
PubSub
current
Describe Your Environment
Docker on GKE
Expected Behavior
When the context in the PubSub reader's Receive method is cancelled, the receive method will return and stop receiving messages.
Actual Behavior
In the iterator when the stop() method is called, a it.stopped channel is closed https://github.com/GoogleCloudPlatform/google-cloud-go/blob/master/pubsub/iterator.go#L118 and the comments in the receive() method would suggest that this channel should be checked before trying to retrieve messages for the iterator https://github.com/GoogleCloudPlatform/google-cloud-go/blob/master/pubsub/iterator.go#L175 but as far as I can tell the stopped channel is never checked for closed(...) anywhere in the code.
As the code behaves today, if I cancel the context, I need to have a MaxOutstandingMessages or MaxOutstandingBytes set in order for the Acquire to eventually fail for the Receive method to actually return. This is undesirable because it requires that the MaxOutstandingMessages (or Bytes) is hit before the function return because the Acquire function in the flow controller also only checks the context when acquiring messages if the maximum is reached.
This also has the effect that if you choose to set MaxOutstanding settings to 0 and manage those things on the consumer side, the iterator and flow_controller will never return and you'll be stuck in a forever loop. Most people won't see these issues unless they are using high values of MaxOutstanding settings.

cc @epsniff

I was unable to reproduce this (gist here); when I cancel the ctx sent into Receive, my program halted (despite not setting MaxOutstandingMessages or MaxOutstandingBytes) as expected.
Could you provide a small reproduction? I'm happy to look into this.

@jadekler I've taken your gist and modified it to reproduce the issue  (careful running it, it left a routine running on my computer). The important changes are that we do not ack the messages right away in the receive and we set maxoutstanding settings to -1 (i.e. off). In our code we put messages from the receive function on a channel and handle them in another goroutine.

Also we set numgoroutines to 2 which is also important to reproducing the issues I believe. I think that what is happening is that the map which keeps the state of the inflight messages is never "drained" because one of the two goroutines always has something in flight when checkDrained is called.

Roughly how long do you wait before acking messages? Are messages guaranteed to be eventually acked? I believe it's known behavior that until messages are acked, Receive will block. @jba is that correct?

We are guaranteed that when the context is cancelled that outstanding messages are acked within 10 minutes. I can reproduce the issue in the example gist even when I've checked that all outstanding messages are acked/nacked after the cancel is called.

Ok, thank you. Finally: is your expectation that once you call cancel, Receive halts immediately? Or, that Receive waits for all acks and then halts? (I believe the behavior is the latter)

@jadekler The latter, that once all inflight messages are acked/nacked, that receive will return.
Currently I can still even reproduce the issue with something like this
        messages := []*pubsub.Message{}
	ctx2 := context.Background()
	ctx2, cancel2 := context.WithCancel(ctx2)
	go func() {
		time.Sleep(5 * time.Second)
		fmt.Println("canceling2")
		cancel2()
		time.Sleep(5 * time.Second)
		for _, m := range messages {
			m.Nack()
		}
		time.Sleep(5 * time.Second)
		fmt.Println("canceling")
		cancel()
		time.Sleep(5 * time.Minute)
		os.Exit(1)
	}()

	fmt.Println("Receiving", time.Now())
	sub.ReceiveSettings.MaxOutstandingMessages = -1
	sub.ReceiveSettings.MaxOutstandingBytes = -1
	sub.ReceiveSettings.NumGoroutines = 1
	err = sub.Receive(ctx, func(c context.Context, m *pubsub.Message) {
		//fmt.Println("Got message", string(m.Data), time.Now())
		//m.Ack()
		select {
		case <-ctx2.Done():
			m.Nack()
		default:
			messages = append(messages, m)
		}
	})

Thanks. By adding those settings and a timer before the ack, I'm able to reproduce. I'll label this a bug and take a peek.

@jadekler Any updates here?

@ajroetker Sorry, my colleague (and primary reviewer) is on leave. Given the subtle nature of this bug it's good to get a solid review before we push anything through. :) We should have an update soon - please feel free to add yourself as cc on the aforementioned code review!

Just a quick note, I'm still tracking this and the review for that CL continues. Apologies for the delay.


[MeshSync] Issues with KinD deployment

@utkarsh-pro to update with details.
File an issue about MeshSync not working in your environment.
Originally posted by @leecalcote in https://github.com/layer5io/meshery/pull/3037#discussion_r642634489

@kumarabd, as you dig into moving MeshKit's usage into Meshery Server for uses like kubeconfig and kube client handling, you should catch up with @utkarsh-pro on the details here.

@leecalcote I, and @kumarabd went through this issue a while back and got it working on the KinD. The issue was not specific to either KinD or MeshSync rather was probably an issue with my environment.
Please re-open it if something more needs to be added here.


python3Packages.mautrix: 0.8.17 -> 0.9.2

Motivation for this change
https://github.com/tulir/mautrix-python/releases/tag/v0.9.2
https://github.com/tulir/mautrix-python/releases/tag/v0.9.1
https://github.com/tulir/mautrix-python/releases/tag/v0.9.0
https://github.com/tulir/mautrix-python/releases/tag/v0.8.18
Adds alot more optional dependencies.
Cherry-picked out of #123760.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Breaks the pinning of this library on both mautrix-telegram and mautrix-signal, but I guess we're not in a hurry anyway.

Result of nixpkgs-review pr 123761 at 010a58fe run on x86_64-linux 1

2 packages failed to build:

mautrix-signal
mautrix-telegram

3 packages built successfully:

python38Packages.mautrix (python38Packages.mautrix-appservice)
python39Packages.mautrix (python39Packages.mautrix-appservice)
tests.trivial

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

@Ma27 I think mautrix-telegram allows newer mautrix versions by now. Are you interested in picking this PR up?
https://github.com/mautrix/telegram/blob/v0.10.1/requirements.txt#L8

Obsolete.


[release-1.4] Cherry-pick https://github.com/istio/istio/pull/20428

Cherry-picking https://github.com/istio/istio/pull/20428 into 1.4-release
[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@stewartbutler: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.4
0aaec95717790925abcd6101884cfe0a7a0d5f8d
link
/test unit-tests_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio_release-1.4

@stewartbutler: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.4
0aaec95717790925abcd6101884cfe0a7a0d5f8d
link
/test unit-tests_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Added server.js file to initiate backend in comment section

I have added server.js file initiating nodejs using npm init. I have used express.js and on submission it returns me the form data. Any more improvements which I can make now or is it enough

Mention issue number and some screenshots of changes made.

You may resolve the conflicts and raise the PR again.


WIP: add support for .NET Core compilation

add shims for missing features on .NET Core
add xproj-based projects and solution, will be migrated later with VS'15
remove packages.config
update .gitignore

I've done some base work to support .NET Core. There are some questions that need to be addressed before this can be considered more mature though:

how are builds made? at the moment dotnet tooling supports packaging using "dotnet pack -c Release" and can be run in main DLL project folder "PhysicalMeasure"

this requires still additional settings to project.json to add metadata to nuget packages

what will be the .NET Framework target version, I see that 4.6 is now set in project properties but the lib could easily target lower versions
do you need build scripts? I could create simple FAKE build script that produces artifacts and runs tests

In short, with this branch you can already create netstandard1.0 packages which can run basically anywhere.


Note in Heroku guide that EdgeDB deployment is currently broken

Temporarily hiding this while the buildpack is broken due to a recent Heroku change. See https://github.com/edgedb/edgedb/issues/4744

I was hoping there might be a way to just toggle this TOC entry off as a comment, but best I can tell, that's not possible. Comments require multiple lines and break any indented blocks they're in. 🫤

Best way I've found is to delete the TOC tree item which isn't ideal, between this and admonition they're the best takes

Take a look at this alternative approach when you have a chance, @1st1. This better communicates our intent to continue supporting Heroku as a deployment target. It doesn't require orphaning anything.

Maybe include a link to the github issue for this problem? It's nice to be able to get notifications for things like this.

@fmoor Great idea! I added a link to the issue.

Heroku is pretty broken in several ways, iirc, though

Heroku is pretty broken in several ways, iirc, though

@msullivan Meaning solving the issue I've linked here will not fully fix Heroku deployment? Should I just revert to  the warning without the link to the Github issue?

Hm, I might be wrong about that. I think it could just be that a bunch of stuff about the tests are broken

I think this is ready to go if you don't mind approving, @1st1.


Problem getting Pin Change Interrupts to work on ATtiny84

I have been trying to get Pin Change interrupts to work on an ATtiny84.
Code:
#include "PinChangeInterrupt.h"

#define BUTTON  2
#define LED 8

void setup() {

  pinMode(BUTTON, INPUT_PULLUP);
  pinMode(LED, OUTPUT);
  attachPCINT(digitalPinToPCINT(BUTTON), buttonInterrupt, CHANGE);

}

void buttonInterrupt() {
  digitalWrite(LED, digitalRead(BUTTON));
}

void loop() {

}

With the code shown above, nothing happens when the button is pressed or released. However, if I move the digitalWrite() line of code out of the buttonInterrupt() function and into loop(), the led goes off and on perfectly as the button is pressed and released. So it seems the interrupt function is not getting called.
I have tried the same code on Nano 3 and that works perfectly, the interrupt function gets called.
I am using:

IDE 1.6.5

Tiny core: https://raw.githubusercontent.com/damellis/attiny/ide-1.6.x-boards-manager/package_damellis_attiny_index.json. Have tried version 1.0.0 and 1.0.1

Pin change library: https://github.com/NicoHood/PinChangeInterrupt version 1.2.3

Which sensorInterrupt() function? Can you post a working and a not working example so we can compare better?
Also I suggest to upgrade the IDE, it is quite old.

Apologies, I meant to say buttonInterrupt() !
1.6.5 is only one or two versions behind, I will give 1.6.7 a go and let you know.
Worgink version: move the digitalWrite() into loop() and that works. Alternatively, it works as shown on Nano 3.

Then it is a bug of the library.
Can you please double check all wires? Maybe the pin numbers are wrong?
Can you please test:

Pull every button low with the current setup and see if the led changes (wrong cables used)
Test all pins, aka change the code (partly wrong library definitions)

Otherwise I have to double check the 84 definitions

Thanks @NicoHood
Like I am saying, if I move the digitalWrite() into loop(), it works, with no changes to wiring, so the wiring must be OK.
I will try using other pins for the button.

Oh. Right. Good Point.
Testing other buttons would be nice, as well as getting the output of the definition.

Just tried pin 0 on the tiny84. Same issue. once again, works fine if I move digitalWrite() into loop()

Does Serial.print(PCINT0); give an error?

Well, this is a tiny84, so Serial.print() won't be available, will it?

int i = PCINT0;

Only this warning:
/home/paul/sketchbook/PCInterruptTest/PCInterruptTest.ino: In function 'void loop()': /home/paul/sketchbook/PCInterruptTest/PCInterruptTest.ino:19:7: warning: unused variable 'i' [-Wunused-variable]    int i = PCINT0;        ^

I have no idea why it then should not work. I'd have to further look into this issue. Please test the other buttons ans report the result. If you have an 85 please also test this mcu.

OK, will do. Maybe later this eve or tomorrow. I do have tiny85 also. I will also try to find the value of digitalPinToPCINT(BUTTON) by some method. I have the tiny84 wired as an i2c slave to another Arduino, so will try passing the value to the master to print to serial.

I tried my sketch above with all pins 0 - 10 on the tiny84. None of them worked, sorry.
But I have managed to get your debug script working, using https://github.com/jscrane/TinyDebugSerial.
Here is the sketch I uploaded:
`
/*
Copyright (c) 2014 NicoHood
See the readme for credit to other people.
PinChangeInterrupt_Development
Prints the available PCINTs.
This is not the Pin number!
Only useful for developing and checking board definitions.
*/
#include "PinChangeInterrupt.h"
#include "TinyDebugSerial.h"
TinyDebugSerial mySerial = TinyDebugSerial();
void setup()
{
// start serial debug output
mySerial.begin(9600);
}
void loop() {
mySerial.println("Available/used pins");
mySerial.println(PCINT_USE_PCINT0);
mySerial.println(PCINT_USE_PCINT1);
mySerial.println(PCINT_USE_PCINT2);
mySerial.println(PCINT_USE_PCINT3);
mySerial.println(PCINT_USE_PCINT4);
mySerial.println(PCINT_USE_PCINT5);
mySerial.println(PCINT_USE_PCINT6);
mySerial.println(PCINT_USE_PCINT7);
mySerial.println(PCINT_USE_PCINT8);
mySerial.println(PCINT_USE_PCINT9);
mySerial.println(PCINT_USE_PCINT10);
mySerial.println(PCINT_USE_PCINT11);
mySerial.println(PCINT_USE_PCINT12);
mySerial.println(PCINT_USE_PCINT13);
mySerial.println(PCINT_USE_PCINT14);
mySerial.println(PCINT_USE_PCINT15);
mySerial.println(PCINT_USE_PCINT16);
mySerial.println(PCINT_USE_PCINT17);
mySerial.println(PCINT_USE_PCINT18);
mySerial.println(PCINT_USE_PCINT19);
mySerial.println(PCINT_USE_PCINT20);
mySerial.println(PCINT_USE_PCINT21);
mySerial.println(PCINT_USE_PCINT22);
mySerial.println(PCINT_USE_PCINT23);
mySerial.println();
mySerial.println("Available/used pins per port");
mySerial.println(PCINT_NUM_PINS_PORT0);
mySerial.println(PCINT_NUM_USED_PINS_PORT0);
mySerial.println(PCINT_NUM_PINS_PORT1);
mySerial.println(PCINT_NUM_USED_PINS_PORT1);
mySerial.println(PCINT_NUM_PINS_PORT2);
mySerial.println(PCINT_NUM_USED_PINS_PORT2);
mySerial.println();
mySerial.println("Available/used ports");
mySerial.println(PCINT_HAS_PORT0);
mySerial.println(PCINT_USE_PORT0);
mySerial.println(PCINT_HAS_PORT1);
mySerial.println(PCINT_USE_PORT1);
mySerial.println(PCINT_HAS_PORT2);
mySerial.println(PCINT_USE_PORT2);
mySerial.println();
delay(5000);
}`
And here is the output:
`Available/used pins
1
0
1
1
1
1
1
1
1
1
1
1
0
0
0
0
0
0
0
0
0
0
0
0
Available/used pins per port
8
7
4
4
0
0
Available/used ports
1
1
1
1
0
0
`

Great. thx for the output. I will have a look later this day. Remind me if i forget. You can also tox me: www.contact.nicohood.de

In case you need it, here is the debug output from the tiny85:
`Available/used pins
1
1
1
1
1
1
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
Available/used pins per port
6
6
0
0
0
0
Available/used ports
1
1
0
0
0
0
`

Well, don't know if this is good news or bad news, but I can't get PC Interrupts to work on any pin of tiny85 either.

Just to reassure you that the library is not completely broken, I just tested it with Nano 3, using pins 2, 3, 13, A0... works perfectly each time.

Thx. The error occurs on 85 is better to debug.
I now tried to use my trinket (85) to debug this, but the code compiles for the 328 for some reason. Maybe this is also your problem? You have to enable the compiler output and check the -mmcu option. But i cannot even upload the program.
So i have to ask you for now until i get this fixed:

connect leds to pin 0, 1, 2
upload the sketch and trigger the button, report what leds are blinking
uncommend the comment and do not touch anything. report what the leds are doing

#include "PinChangeInterrupt.h"

// choose a valid PinChangeInterrupt pin of your Arduino board
#define pinBlink 3

void setup() {
  // set pin to input with a pullup, led to output
  pinMode(pinBlink, INPUT_PULLUP);
  pinMode(2, OUTPUT);
  pinMode(0, OUTPUT);
  pinMode(1, OUTPUT);

  // attach the new PinChangeInterrupt and enable event function below
  attachPCINT(digitalPinToPCINT(pinBlink), blinkLed, CHANGE);
}

ISR(PCINT0_vect);
void blinkLed(void) {
  // switch Led state
  digitalWrite(2, !digitalRead(2));
}

void loop() {
  // nothing to do here
  delay(1000);
  digitalWrite(0, !digitalRead(0));

  // uncomment for 2nd test
  //PCINT0_vect();
}

Hi.
The sketch blinks the led on pin 0, 1s on, 1s off. Pressing the button has no effect on any led.
Then I uncomment the final code line for second test. There is no change in behavoir of the tiny84.
Finally I add the line digitalWrite(1, digitalRead(pinBlink)); into loop(). Now, the led on pin 1 lights. If I press and hold the button, pin 1 led goes off (after 1s delay). When I release the button, pin 1 led lights again (after 1s).

Thanks for all the help! You can retest, but not it all should work fine (I deactivated those 4 pins by default again, to safe flash) :)


Manifest file reference in the Sway Book has a wrong key listed: implicit-std is spelled as implicit_std

implicit_std should be changed to implicit-std in the manifest reference section of the Sway Book.

Cargo uses dashes instead of underscores indeed: https://doc.rust-lang.org/cargo/reference/manifest.html

Yeah this is a documentation bug only. forc expects a dash as well.


grapheme_substr inconsistency as compared to substr

grapheme_substr from Intl have differing behavior as compared to substr:
var_dump(substr('a', 1, null)); //  string(0) ""`
var_dump(grapheme_substr('a', 1, null)); // bool(false)

It looks like on php 5.x it also produces warning.
This behavior needs to be investigated against other wrappers and corrected if needed.

This repository has been closed and moved to laminas/laminas-stdlib; a new issue has been opened at https://github.com/laminas/laminas-stdlib/issues/2.


[$2000] City and State doesn’t populate for the address I-80 but works well with other address like I-35 , I-40, I-70

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open chrome staging dot
Go to personal information and address
In the address field type in : I
Try to select I-80 and see that the city and state doesn't populate with the recommended address
But if you select I-35, I-40 or I-70, every recommended address like city , state, and country populates without a problem

Expected Result:
Setting address I-80 (Interstate 80) should populate the addresses as per the recommendation
Actual Result:
Setting address I-80 (Interstate 80) doesn’t populate the addresses as per the recommendation
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.2.80-1
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/223845024-748c7542-5ac6-4403-b7e8-8456ce2832a5.mp4
https://user-images.githubusercontent.com/43996225/223845043-fdd65945-6548-400b-9350-7cd817bb8345.mp4
Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1678254121686449
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0195f9f8aaab8dc5c1
Upwork Job ID:<PHONE_NUMBER>687765504
Last Price Increase: 2023-03-17

Triggered auto assignment to @arielgreen (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Note that even for other ones (e.g., I-35), it doesn't populate zip.

Job added to Upwork: https://www.upwork.com/jobs/~0195f9f8aaab8dc5c1

Current assignee @arielgreen is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @sobitneupane (External)

Triggered auto assignment to @francoisl (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Possibly due to Google Map API result. The code in https://github.com/Expensify/App/blob/a34c4a7da4ea7db905c28fff1ebb46850c2affe3/src/components/AddressSearch.js#L194 will fetch the location detail from /api?command=Proxy_GooglePlaces, and update the city, state, country values in the input.
For address such as I-36, the location api returns city and state info

For address such us I-80, the location api does not return city and state info

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0116ef5450a33cc027

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

Proposal
Please re-state the problem that we are trying to solve in this issue.
Setting address I-80 (Interstate 80) doesn’t populate the addresses (city, state, ...) as per the Google Places Autocomplete
What is the root cause of that problem?
The root cause is:

When we fetch the data from the Google Places Autocomplete, it tries to give us predictions on the places even though the places might not exist in Google's database
When clicking on the place in the autocomplete list, we retrieve the place details from the Google Place API, which will only return the address that exists Google's database
The autocomplete option for 'I-80' query does not exist in Google's database, so when we get its details, Google will try to find the closest address in its database and return it, in this case the address in Google's database does not have information like city, state, ...

For more information please refer to Google's response here: https://issuetracker.google.com/issues/35823492#comment2
This is a known behavior.
What changes do you think we should make in order to solve the problem?

To fix this, we need to use the Google's place autocomplete information as a fallback in case the Place Detail does not have that information.
For example, when we query for I 80 and click on the I-80, Seward, NE, USA result in the autocomplete dropdown.
If we don't have the state from the Place Detail API, we'll use NE from the autocomplete result as a fallback.
If we don't have the city from the Place Detail API, we'll use Seward from the autocomplete result as a fallback.
The response from Google Autocomplete is quite structured so we can retrieve the state and city using the following logic:
const fallbackData = autocompleteData.terms;
const stateAutocompleteFallback = fallbackData.length >= 2 ? fallbackData[fallbackData.length - 2].value : '';
const cityAutocompleteFallback = fallbackData.length >= 3 ? fallbackData[fallbackData.length - 3].value : '';

in https://github.com/Expensify/App/blob/4a172c9e9a2bcf02b4a3b0cec3537fdf82195d73/src/components/AddressSearch.js#L85
The autocompleteData can be passed into saveLocationDetails by using the data returned in https://github.com/Expensify/App/blob/4a172c9e9a2bcf02b4a3b0cec3537fdf82195d73/src/components/AddressSearch.js#L157.
There might we wild result format returned from Google Autocomplete for countries other than the US, if we're not sure we can limit the above fallback logic to addresses in the US. It will improve the user experience.
What alternative solutions did you explore? (Optional)
NA
Result
Working well after the fix:
https://user-images.githubusercontent.com/113963320/224467227-5f016b7d-02bf-4222-a2db-724673c45220.mov

Thanks for the proposal @tienifr.
But I don't think we can go with your proposal because it depends on the structure of data returned from an api which is not always consistent.

@sobitneupane sure, the structure of the address within the US is very consistent as I observed.
I agree my solution will be the last one to be considered if there's no other perfect solution.

I'd be ok with this solution, though it sounds like it wouldn't work very well for all non-US addresses. Looking at the terms array for random addresses from other countries, the city is not always at index fallbackData.length - 3, and the state is not always at index fallbackData.length - 2, because other countries don't necessarily use the same <street number> <street name> <city> <state> address format.
Alternatively, we could use this fallback solution only if the predicted country is USA?

Alternatively, we could use this fallback solution only if the predicted country is USA?

@francoisl yes we can do this as well.

@hoangzinh Will your proposal alone yield the expected output? Or should it be integrated alongside this proposal.
Our expected output is

Setting address I-80 (Interstate 80) doesn’t populate the addresses as per the recommendation

The issue here is while selecting I-80 state, it doesn't populate 'City' , 'Sate' and 'zipcode'.

@sobitneupane from the screenshot of the proposal it seems the address in the issue I-80, Seward, NE, USA disappears from the search result now, likely due to the changed query types. Not sure if we want that.

bump @sobitneupane

I agree @francoisl. We can go with @tienifr's proposal.
My only concern is since the solution depends on the structure of response from an third-party api, it might break down if the api response changes.

📣 @tienifr You have been assigned to this job by @arielgreen!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

@francoisl @sobitneupane @arielgreen The PR is ready to review!

Regression Test Proposal
Bug: Typing emoji in CAPS doesn’t highlight the text on the emoji suggestion list
Proposed Test Steps:

Login with any account
Click on Profile avatar > Profile > Personal details > Home address
In the Address line 1 field type in: I
Select I-35 (i.e. I-35 Ottawa, KS, USA)
Verify that other fields are filled automatically just as the suggestion (i.e. Interstate 35 - Ottawa - Kansas - United States)
Select I-80 (i.e. I-80 Seward, NE, USA)
Verify that other fields are filled automatically just as the suggestion (i.e. Interstate 80 - Seward - Nebraska - United States)
In the Address line 1 field, type in: N12 ireland
Select the option N12, Knockaconny, County Monaghan, Ireland
Make sure that the City, State / Province, and Country fields get automatically filled
In the Address line 1 field, type in: BR-101
Select the option that starts with BR-101, Linhares
Make sure that the City, State / Province, and Country fields get automatically filled

Do we :+1: or :-1:

Bug: Typing emoji in CAPS doesn’t highlight the text on the emoji suggestion list

This sounds like an accidental copy-paste from another issue? The rest of the steps sounds good to me.

@francoisl Sorry, my bad, I just updated

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.94-3 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/16388

If no regressions arise, payment will be issued on 2023-04-12. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@sobitneupane / @francoisl] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@sobitneupane / @francoisl] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@sobitneupane / @francoisl] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@arielgreen] Determine if we should create a regression test for this bug.
[ ] [@sobitneupane] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@arielgreen] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

I filled out most of the checklist above. For the regression test steps, we can use this comment @arielgreen.

@tienifr @sobitneupane @priya-zha, please apply for this posting on Upwork asap so I can send payment

@arielgreen Submitted the proposal. Thanks

@arielgreen Proposal Submitted.

@arielgreen friendly bump ^

@priya-zha @tienifr paid, apologies for delay

Just waiting on @sobitneupane to accept, then everything here is done and we can close.

@arielgreen Accepted the offer.

All payments made, we are set here.


Unable to view deployed app

Context
I have a Streamlit App, deployed with JHub Apps with the following options:

The file is in my personal file system
Environment is in a shared (analyst) namespace
Shared with users in the same analyst group

Users in the group can see the app in the Homescreen, but get the following error on opening it:
Traceback (most recent call last):
  File "/opt/conda/envs/default/lib/python3.11/site-packages/tornado/web.py", line 1790, in _execute
    result = await result
             ^^^^^^^^^^^^
  File "/opt/conda/envs/default/lib/python3.11/site-packages/jhsingle_native_proxy/websocket.py", line 102, in get
    return await self.http_get(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/default/lib/python3.11/site-packages/jhsingle_native_proxy/proxyhandlers.py", line 849, in http_get
    return await self.proxy(self.port, path)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/default/lib/python3.11/site-packages/jhsingle_native_proxy/proxyhandlers.py", line 843, in proxy
    return await self.oauth_proxy(port, path)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/default/lib/python3.11/site-packages/tornado/web.py", line 3298, in wrapper
    self.redirect(url)
  File "/opt/conda/envs/default/lib/python3.11/site-packages/jupyterhub/services/auth.py", line 1508, in raise_on_redirect
    raise HTTPError(
tornado.web.HTTPError: HTTP 403: Forbidden (user<EMAIL_ADDRESS>is not allowed.)

Value and/or benefit
Bug-fix
Anything else?
I'm using JHub Apps on a deployed Nebari instance v2024.6.1

This was due to lack of permissions for the user. The required permissions to share with a group is:
# can omit "read:users:name" if not sharing with user
shares,read:users:name,read:groups:name


Simple Payload

Implementation of Simple Payload to enable centralised and decentralised contact tracing.

Implementation complete - pending pull request approval


RIght side menu

You mention this can be displayed both from left and right side, but i've spent time both trying and searching a way to display it at the right side (default behavior is from left) without having success. I tried to replace the segue name with "sw_right" instead "sw_rear" and the animation didn't fire.
Could you give me a heads up about this please?

hey guys, I don't know how to change left side menu to right side, I don't know Objective-c!
Help me Please :)


fix Set query encounter rpc exception default retry times to…

… 3 (#28555)
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


How to convert trained tensorflow SSD models to UFF with custom resolution

@jkjung-avt  Hi, thanks for your work, it is very nice!!!
I follow your tutorial to train the ssd_mobilenet_v1 model with DETRAC datasets,
then I got the trained model(.pb) and it performs well, so i want to convert trained tensorflow models to UFF and tensorRT.
My custom resolution is 256*256, so i modified INPUT_DIMS = (3, 300, 300) to (3, 256, 256) in build_engine.py. Finally, I encountered the following errors:
[TensorRT] INFO: Detected 1 inputs and 2 output network tensors.
python: nmsPlugin.cpp:139: virtual void nvinfer1::plugin::DetectionOutput::configureWithFormat(const nvinfer1::Dims*, int, const nvinfer1::Dims*, int, nvinfer1::DataType, nvinfer1::PluginFormat, int): Assertion `numPriors * numLocClasses * 4 == inputDims[param.inputOrder[0]].d[0]' failed.
I think the problem is with resolution. By the way, i modified the input_order with various permutations and combinations(such as [1, 0, 2]、[0,1,2].....), It still doesn't work.
So how can I optimise model for custom resolution ? It works fine with your 300x300 resolution. But can't work with my (3, 256, 256)....
Thank you in advance for your help!!

When you change input image shape of the SSD model, the output feature map shapes would also change.  You need to figure out the correct output shapes and put them into featureMapShapes.

@jkjung-avt
Thanks for your help!! I have solved this problem.

Hi @jkjung-avt ,
I have trained my ssd-mobilenet-v2 model in 300*300 and was able to convert the model to tensor-rt model using the methods specified in your repo, Thanks to you.
But my doubt is this :  I have trained another ssd-mobilenet-v2 model with 480×272. I want to convert this model to TensorRT model. I modified featureMapShapes=[19, 10, 5, 3, 2, 1] as you said in here . But it seems this only works, if height and width are same(ie, if aspect ratio is 1).
Do we have a solution for this ? Can you please help me solve this issue

If you use TensorRT C++ API, it is possible to specify different heights and widths using the "PriorBox" plugin (https://github.com/NVIDIA/TensorRT/tree/master/plugin/priorBoxPlugin).
But it doesn't seem possible when we use the "GridAnchor" plugin (https://github.com/NVIDIA/TensorRT/tree/master/plugin/gridAnchorPlugin) in the UFF.  The heights and widths are hard-coded to be the same here and here.

Reference: https://github.com/NVIDIA/TensorRT/issues/612 and https://github.com/NVIDIA/TensorRT/issues/247


add dark mode to drawer

Pull request template

Please make sure you read and fill this template out entirely
PRs that don't have the template filled out will be ignored

Description
adds dark mode to drawer
Fixes #(issue)
Screenshots (if appropriate):

Type of change
Please tick the relevant option by putting an X inside the bracket

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings
[x] I have read the CONTRIBUTING document

Looks great! Thank you <3


[Question] how to get request options from the aborted request after all retries are performed and failed ?

hi,
i want to get the request options from the abort error message , when the request is aborted after all retries are performed. can anyone help?
const { headers, _data, ...rest } = await ofetch.raw(url, {
  method: "POST",
  body: { some: "value" },
  timeout: 2000, //ms
  retry: maxRetries,
  retryDelay: 1300, // ms
  ignoreResponseError: true,
  async onRequest(ctx) {
    // here i change some option values based on retry number
    // these values i am looking for in catch / error when request is aborted after all retries are performed
  },
})

thanks for reading

i figured it out,  setting a key on error in "onRequestError" i can pass that to error