Code edits not reflected in firmware load via flash command

When I edit code then tap the flash to load it into my device, everything looks normal, the device restarts etc after being flashed etc.. But the running code is a prior version, not the one I just edited.
The fix I've found is to select a different device, then re-select the desired target, then flash...
This is either a bug in the system of some flaw in my development setup. How can I help fix this?

What's the device you're trying to flash (Core/Photon/Electron)?
Does this happen every time? (can you reproduce steps you mentioned each time)
Could you try using CLI to start particle subscribe mine and see if it shows spark/device/app-hash event after flashing from Dev?

Is a Photon.  Does not happen ever time. I don't yet have a correlation to an action so that I can repeat the issue. When it does happen, everything seems to work but it's loading previous code. I include a version ID which is output on the LEDs of the Internet Button so I can tell that it's updated.
When it starts this evil issue, I end up trying many things and sometimes it starts working again. Things like selecting a different Photon, switching projects, close and re-open Dev etc.. Sometimes one of these fixes other times not.  I am thinking there is an issue with the cloud compile/save process.
I've even put in errors in the code to see if they are caught. I will have to re-do this experiment as I'm only 90% sure the errors were not caught on recompile, proving in some way that it's a cloud issue.
Is there a way I can see more information about the cloud process? Like when a file is sent up, when it's compiled, etc...
I did have a initialization issue, calling one of the Accel methods in the InternetButton lib before init was called.  That was crashing my button completely, it's possible that some of my problems were related to that.  Some, but not all.
Last night, I had to move back to the online IDE to continue working as Dev was not updating as I edited.
I am a 35+ year experienced embedded developer and love this platform. I am willing to help figure this out anyway I can.  I will try the CLI suggestion and get back.

OK, output from CLI. Made a single code change, just bumped my version ID, in the editor and simply clicked flash.  Process seemed perfectly normal, device LEDs flashed as if being programmed, serial output showed reset process but version ID did not change.
{"name":"spark/flash/status","data":"started ","ttl":"60","published_at":"2016-03-29T19:25:56.042Z","coreid":"I removed 24 numbers here for privacy"}
{"name":"spark/flash/status","data":"failed ","ttl":"60","published_at":"2016-03-29T19:26:03.667Z","coreid":"I removed 24 numbers here for privacy"}
{"name":"spark/status","data":"online","ttl":"60","published_at":"2016-03-29T19:26:03.665Z","coreid":"I removed 24 numbers here for privacy"}

Without any edits, I tried several different things, "compile in the cloud" followed by "flash", forced save of file, reselected the device etc.. each time I tried and got the old version of code on flashing.  Then I got this and the new version ran.
{"name":"spark/flash/status","data":"started ","ttl":"60","published_at":"2016-03-29T19:33:23.272Z","coreid":"2c0023000747343232363230"}
{"name":"spark/flash/status","data":"failed ","ttl":"60","published_at":"2016-03-29T19:33:31.536Z","coreid":"2c0023000747343232363230"}
{"name":"spark/status","data":"online","ttl":"60","published_at":"2016-03-29T19:33:31.545Z","coreid":"2c0023000747343232363230"}
{"name":"spark/device/app-hash","data":"EB0C292754EFB9344FA5D30FACAF7AFD64AFBCD2D14D2A5C1BAAC5CC59A9204D","ttl":"60","published_at":"2016-03-29T19:33:32.936Z","coreid":"2c0023000747343232363230"}
What is the failed error above. I see the "failed" message on every flash, good or bad.

Maybe a clue/complication... The editor does not seem to save edits before doing a cloud compile. I've seen several cases where the error reported was corrected but did not clear on compile until after a file save.

One of the reasons might be that Dev doesn't automatically save the file you're working on so you have to do it manually before each compile.
spark/flash/status failed is a false negative. If you see spark/device/app-hash this means flash has succeeded and an app has been updated.
Also Flash button will flash latest .bin file in the directory if it exists (instead of compiling latest version) so make sure you don't have any photon_firmware_*.bin files there.
Let me know if this helped.

The "failed" message should be listed as a bug.
Are you saying that I must manually delete the .bin file after each compile?  The dev process is..

Edit your file.
Save your file.
Delete any previously generated .bin files
Flash

Why can't the development tool do steps 2, 3 ,and 4?  Every other IDE does...

I agree with @uchobby, I'm finding it too easy to accidentally compile an unsaved sketch, or flash an old version of firmware.
Also I noticed this behavior which I plan to report as a new issue (if it's not already an issue), it might help here.  If you restart Dev, no devices are selected, and when you press compile, it compiles firmware for the Core.  If you then select a Photon as your device and flash, it will flash with the previous photon_firmware.bin that was in the directory, and doesn't recompile.  If there was no photon_firmware.bin, then it does appear to compile fresh.

There are two options in settings that control this behaviour:

By default flash will try to use existing binary. This is intentional as it could be build using CLI, local compiler or downloaded from Build. I think another option: "Always compile new firmware when flashing" would be helpful in mentioned situations.

The CLI would be used by advanced users so I would focus Dev on the Arduino crowd.  Make it hard for things to fail rather then hard to get things to work.  I totally understand trying to be versatile but it's not a good trade off to benefit 10% of your users at the expense of 90%.
I use the CLI now with my normal code editor, this has been reliable for me so far.
If you need any testing done with the Dev tool, I'd be happy to switch back.

Dev now saves opened editors before compilation and has more sensible platform selection. This should help with unexpected behaviour.


[incubator/kafka] Rework mechanism to expose Kafka brokers externally

What this PR does / why we need it:
Statefulsets from kubernetes 1.9 set label statefulset.kubernetes.io/pod-name to pod name.
Thus setting label pod in initcontainer is not needed anymore.
Also rbac settings, previously needed by initContainer are not needed.
Special notes for your reviewer:
I wasn't sure, to what version should the chart be bumped.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@faraazkhan @h0tbird @benjigoldberg

/ok-to-test

/lgtm -- thanks for the contribution!

/assign

/assign @benjigoldberg @unguiculus

Is anything more needed from me?
/lgtm does not seem to have pushed it forward.

I want to bump this PR.
Is there an blocker?
@unguiculus @benjigoldberg

/lgtm

Leaving this here in case someone else runs into similar issues: if you previously set rbac.enabled: true, upon upgrading you may see errors related to missing kafka service account tokens.  To work around this I had to manually remove the serviceAccount and serviceAccountToken values from the kafka statefulset.
More details here: https://github.com/kubernetes/kubernetes/issues/76367#issuecomment-481689160


[Parse][IDE] Fix document structure request output for interpolated string literals

InterpolatedStringLiteralExpr has a TapExpr, which contains a BraceStmt containing the CallExprs to the builder appendInterpolation/appendStringLiteral methods used to construct the final string. This is all implementation detail, but the BraceStmt wasn't marked implicit and had a valid (but incorrect) source range, so the document structure request treated it as any other BraceStmt and made a structure node for it. The invalid range ended up tripping an assertion in the document structure walker in swift-ide-test.
This patch corrects the BraceStmts produced when parsing interpolated strings to be marked implicit and have the correct range (endloc pointing at the start of the last token - the string itself), and also updates ModelASTWalker to skip over the internal details of interpolated strings when walking them (the call and args of the CallExprs were also being emitted as structure nodes).
Resolves https://bugs.swift.org/browse/SR-11099
Resolves rdar://problem/55183943

@swift-ci please test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 44df35924696eb7163d515c02b3741547cd8897f

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - aa0305ee8010ba549856ba016094f3c432798cc4

Build failed
Swift Test OS X Platform
Git Sha - aa0305ee8010ba549856ba016094f3c432798cc4

@swift-ci please test


Update requirements.txt

The dependencies for torch and torchvision were incompatible. Torch 1.10.0 utilizes torchvision 0.11.1 instead of 0.9.0.

I second this pull request.


collect new globalhub ns data

Related Issue:
story: https://issues.redhat.com/browse/ACM-12718
doc pr: https://github.com/stolostron/rhacm-docs/pull/6971/files#diff-426c3333e3d6698409184255e75546191f82b8b362dc8f713fb9e77a00c289e1R24
Description of Changes:
Globalhub will install some addons in open-cluster-management-global-hub-agent-addon ns, so we need to collect the log in this ns.
What resource(s) are being added:
resoueces in ns open-cluster-management-global-hub-agent-addon
Is this a Hub or Managed cluster change?:

[x] Hub Cluster
[ ] Managed Cluster
[ ] N/A

Notes:

/cc @hchenxa @clyang82

/cc @dhaiducek


Please add a requirements.txt file

I would like to request that you start using a requirements file in your Python project.
A requirements file is a simple text file that lists all the dependencies that your project needs in order to run. It makes it easy for other contributors and users to install all the necessary dependencies with a single command.
Not only does using a requirements file make it easier for people to get started with your project, but it also ensures that everyone is using the same version of each dependency. This can help avoid compatibility issues and make it easier to maintain the project in the long run.
Creating a requirements file is easy. You can use the pip freeze command to generate a list of all the dependencies currently installed in your project's virtual environment. This command will output a list of packages with their version numbers, which you can then copy into a file named requirements.txt in the root directory of your project. For example:
pip freeze > requirements.txt

Once you have created your requirements file, you can use it to install all the necessary dependencies with a single command:
pip install -r requirements.txt

Thank you for considering this request. Using a requirements file will make it easier for contributors and users to get started with your project, and ensure that everyone is using the same versions of the dependencies.

Does this suit?
https://github.com/opendp/dpcreator/tree/develop/server/requirements
It might be too hidden.

For reference, you may find it easier to stand DPCreator up via docker-compose:
https://github.com/opendp/dpcreator/blob/develop/README-DEVELOPMENT.md#building-with-docker-compose


Comments

Add some comments to explain the game.py file

I looked at your code ! i'm going to write some comments to make it easier to understand !


[WIP] Add an example that implements GPyOpt based sampler.

This PR adds an example that implements GPyOpt based sampler.

Close this PR because Optuna already provides a GP based sampler (SkoptSampler).


GH Actions: tweak label management workflow

Follow up on #2127
Let's try this without the quotes around the labels. The example in the action runners' readme didn't contain quotes, I added them as the labels we use have a : in it, which has special meaning in YAML workflows, but clearly that's not working.
If this doesn't work either, I may need to report it as a bug to the action runner (or find another action runner for the same).

I guess it's working now

😄

🎉 with 🍰


Please tell me can I Disable playback video?

Please tell me can I Disable playback video?

@dev3ios
https://github.com/tilltue/TLPhotoPicker/blob/master/Example/TLPhotoPicker/CustomCell_Instagram.swift
use custom cell.
playerView is optional properties.
open class TLPhotoCollectionViewCell: UICollectionViewCell {
    @IBOutlet open var imageView: UIImageView?
    @IBOutlet open var playerView: TLPlayerView?
//...
}

@tilltue  Can you provide some more detail on what to set?  I have tried making player and playerView both nil without any luck.
I just want the video to not play when I select the cell.

@jimsmithm3
open class TLPhotoCollectionViewCell: UICollectionViewCell {
    @IBOutlet open var imageView: UIImageView?
    @IBOutlet open var playerView: TLPlayerView? //<- you don't want to play? remove this.
//...
}


RuntimeError: CUDA error: CUBLAS_STATUS_EXECUTION_FAILED when calling cublasSgemm( handle, opa, opb, m, n, k, &alpha, a, lda, b, ldb, &beta, c, ldc)

RuntimeError: CUDA error: CUBLAS_STATUS_EXECUTION_FAILED when calling cublasSgemm( handle, opa, opb, m, n, k, &alpha, a, lda, b, ldb, &beta, c, ldc)
How to solve it?

This sounds as if the JIT would kick in and compile the CUDA kernels for your architecture, which seem to be missing from the used binary.
How did you install/build PyTorch and which GPU are you using?

Once 1. is solved, this might also be fixed.
Similar error in PyTorch Discuss

I guess maybe your gpu  memory is not enough
you can try run your code on one bigger gpu device or small you mode


Be explicit with cluster option

Avoid digging deep into documentation to understand it.

Merge it if you want ;)

Thanks for you suggestion but I think the documentation is fine. Pushers own docs are pretty straight forward. Since this is only a wrapper for Laravel I'm going to pass on this one.


No "exports" main defined in /node_modules/libp2p/package.json v 0.37.3

will commonjs modules be supported?

CommonJS is supported for<EMAIL_ADDRESS>and below<EMAIL_ADDRESS>and later is ESM only.


abouchez/mormot: performance enhancements

It seems to be a good idea to share the hash table between threads.

It does, doesn't :smile:


ASP.NET Core logging

This page links to Logging in ASP.NET Core but there are no instructions on how to configure EF Core logging there.

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

ID: ed83069f-39a8-e992-95f7-a8b41154b6f4
Version Independent ID: 144fe168-080f-6557-e4b2-a18adb4b89ed
Content: Logging - EF Core
Content Source: entity-framework/core/miscellaneous/logging.md
Product: entity-framework
GitHub Login: @rowanmiller
Microsoft Alias: divega

@paulomorgado the first paragraph in this page is the shortest answer:

EF Core integrates automatically with the logging mechanisms of ASP.NET Core whenever AddDbContext or AddDbContextPool is used.

That means that we automatically wire up things so that the DbContext will use the ILoggerFactory dependency of the application. For example, if you create an ASP.NET Core application with ASP.NET Core Identity, the code that you get from the templates already configures EF Core to use the logging configured in ASP.NET Core. Let us know if there is anything specific we could add to make this clearer.

If you're creating the DbContext manually, just make sure to call UseApplicationServiceProvider on the options builder, so that it uses the same services as the rest of the application.


Need: Tablet for speaker timing

You mean like a countdown clock? I have a tablet we could use.
Chris Mathews
On Wed, Sep 14, 2016 11:10 PM, Jason Hibbets<EMAIL_ADDRESS>wrote:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Exactly, countdown clock AND a volunteer to run it.
On Wed, Sep 14, 2016 at 11:14 PM, Chris Mathews<EMAIL_ADDRESS>wrote:

You mean like a countdown clock? I have a tablet we could use.
Chris Mathews
On Wed, Sep 14, 2016 11:10 PM, Jason Hibbets<EMAIL_ADDRESS>wrote:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/CityCampNC/issues/273#issuecomment-247222534,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADgoV2qstxDB2IPrtYsJ0o12nFvm7Aoaks5qqLgFgaJpZM4J9dRI
.

I have the tablet with a countdown clock


User report: Replace: id matched 2 resources

From discord:

In case anyone runs into a similar issue, I recently updated my github flux diff workflow to the latest version of flux-local. After the update it started to fail with this error ✗ in Replace: id matched 2 resources. It did not like the substitution in my certificate config with 2 domains.

After I changed the SECRET_DOMAIN2 to SECRET_DOMAIN_TWO, flux-local passed all tests.
---
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
  name: "${SECRET_DOMAIN/./-}-staging"
spec:
  secretName: "${SECRET_DOMAIN/./-}-staging-tls"
  issuerRef:
    name: letsencrypt-staging
    kind: ClusterIssuer
  commonName: "${SECRET_DOMAIN}"
  dnsNames:
    - "${SECRET_DOMAIN}"
    - "*.${SECRET_DOMAIN}"
---
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
  name: "${SECRET_DOMAIN2/./-}-staging"
spec:
  secretName: "${SECRET_DOMAIN2/./-}-staging-tls"
  issuerRef:
    name: letsencrypt-staging
    kind: ClusterIssuer
  commonName: "${SECRET_DOMAIN2}"
  dnsNames:
    - "${SECRET_DOMAIN2}"
    - "*.${SECRET_DOMAIN2}

Example repo:
https://github.com/clarknova99/home-cluster/blob/main/kubernetes/apps/cert-manager/certificates/app/production.yaml
Example failed run:
https://github.com/clarknova99/home-cluster/actions/runs/9410448182/job/25922128913
Context about affected versions:

flux looked to work fine with that substitution. flux-local did too in the older version. i think i was on 4.2 before updating to 5.3.1

I got the same error with 5.3.1 but dont have any numbers in the domain name.
https://github.com/drag0n141/home-ops/blob/master/kubernetes/main/apps/networking/traefik/certificates/production.yaml

I believe the issue happens because in the first pass of traversing kustomizations post build expansion is not happening, then it builds the objects without passing in any variables and they get expanded to the same object name (-staging). Looking into solutions.

A workaround is to specify a default value that is different for each object.


Simplify repository element in npm's project.json to use https URL

Why:
The repository element in the package.json file can be dramatically simplified from:
"repository" : {
    "type" : "git",
    "url"<EMAIL_ADDRESS>    "directory": "packages/name"
  },

..to just:
"repository" : "https://github.com/OWNER/REPOSITORY",

This will be less error prone because it can be easily copy/pasted from GitHub repository URLs.
We could simplify it further to just "repository" : "OWNER/REPOSITORY",, because npm treats GitHub as it's default Git repository host. This would break symmetry with GHES, so maybe the https form is preferable?
See here for "repository" : "OWNER/REPOSITORY", form:
https://docs.npmjs.com/cli/v6/configuring-npm/package-json#repository:~:text="repository"%3A "npm%2Fnpm"

What's being changed:

Check off the following:

[ ] All of the tests are passing.
[ ] I have reviewed my changes in staging.
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

This is different than how npm init configures the repository information if you have a git remote already configured, so it might potentially confusing?
[09:47:37] $ npm init -y
Wrote to /Users/toddkennedy/src/vimwiki/package.json:

{
  "name": "vimwiki",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "repository": {
    "type": "git",
    "url": "git+https://github.com/toddself/vimwiki.git"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "bugs": {
    "url": "https://github.com/toddself/vimwiki/issues"
  },
  "homepage": "https://github.com/toddself/vimwiki#readme"
}

Ideally maybe we should say to use npm init (-y just accepts the default answers) to generate the package.json file?

Interesting, I didn't know that. I tend to npm init before git init.
If I'm adding it by hand, I much prefer the "repository" : "https://github.com/OWNER/REPOSITORY", form. 🙂

👋 @jcansdale @toddself, let me know when this is ready for a writer review!

希望

希望

希望

@toddself,

This is different than how npm init configures the repository information if you have a git remote already configured, so it might potentially confusing?

Thinking more about this. If a user does npm init inside an existing repository, the repository element will automatically be acced and everything will just work.
I think the following section:
https://github.com/github/docs/blob/a73479193b2287ea49fe19ab44546d0895cf0a1f/content/packages/using-github-packages-with-your-projects-ecosystem/configuring-npm-for-use-with-github-packages.md#publishing-multiple-packages-to-the-same-repository
... is most relevant for users who are adding the repository element after the fact. In this case, I think the simplicity of a copy/pastable https element makes more sense.
What do you think?

@jcansdale i think that makes total sense!

@janiceilene I think we're ready for a writer review. 👍

Hi @janiceilene, I've just marked this as not-stale. It's ready for a writer review. 🙂

We will review this as soon as we can!


Module 'Mixpanel' not found.

Integration Method: CocoaPods
Xcode Version: 12.1
Library Version: 3.6.3
Platform: iOS
Language: Objective-C
Description: Issue when integrating Mixpanel Notification Service, Module Mixpanel not found in NotificationService.h
Expected Behavior: For the Notification Service target to work properly following Mixpanel documentation.
I followed the steps in Mixpanel/Mixpanel-iphone Readme. Because the website documentation has a typo for Rich Push Notifications  NotificationService.h being repeated should a one .m file.
I followed everystep well, installed the pods but I couldn't run no matter what. Because we use Flipper the usage of use_frameworks! gives us some issues. I have tried use_frameworks! :linkage => :static instead and some workaround to make use_frameworks! works. I tried changing Require only App-Extension-Safe Api to No, adding Mixpanel path to Podfile, adding paths to Notification target build settings all didn't work and I get Module 'Mixpanel' not found in NotificationService.h
I also tried starting everything over as I thought maybe my project got corrupted. I see Mixpanel being installed with pods but I cannot import it from NotificationService.h
Here the end of my podfile:
target 'NotificationService' do
  pod 'Mixpanel'
end

Here is our NotificationService.h file:
@import Mixpanel;

@interface NotificationService : MPNotificationServiceExtension

@end

Is there any solution to make this work? Thank you.

It might be similar to https://github.com/mixpanel/mixpanel-iphone/issues/905
However Mixpanel is deprecating Messaging and Mobile A/B testing features(more details), please contact our support if you need to continue the conversation for any critical issues. Sorry for the inconvenience.


Update documentation by my suggestion in #889

Fixes #[889].
Changes proposed:

Noticing cdn install in docs because I'd to check  tag in docs example but didn't see this in Installation section :)

Upgrade Path (for changed or removed APIs): -
Acceptance Checklist:

[ ] The commit message follows the guidelines in CONTRIBUTING.md.
[x] Documentation (README.md) and examples have been updated as needed.
[ ] If this is a code change, a spec testing the functionality has been added.
[ ] If the commit message has [changed] or [removed], there is an upgrade path above.

LEGIT. Can you also add a link to this page on README?

Thanks, @magistrfox.


Fix coqPackages.mathcomp-altreals dependencies

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@vbgl CI green both here and on https://github.com/coq-community/coq-nix-toolbox/pull/279 (and this time I didn't forget to test master on https://github.com/math-comp/analysis/pull/1373 ), sorry about my stupid mistake

Thanks


Add default layout option

Is your feature request related to a problem?
no
Describe the solution
Add option to settings menu to choose an default layout mode for all folders.
Describe alternatives you've considered
Additional comment
Using Files <IP_ADDRESS> on Win 10 (OS Build 19042.844)

@d2dyno1 does setting a default layout make sense with the adaptive layout feature?

@michelebiancucci you can set a default layout for all folder by turning off "Enable individual preferences for folders" options in Files settings

@gave92 I am pretty sure the default is already the details view.

Thank you. Much appreciated.


Create download file lambda

This is mostly copied from the tdr-file-format project, there isn't much that's new.
The idea behind this is that this lambda runs when a file is uploaded to S3 and downloads that file to the EFS file system. It then sends a message to the file-format queue. There are a few advantages to doing it this way.

It makes the file-format code shorter
It allows retries without resorting to deleting the file on failure. At the moment, the AWS SDK won't allow you to download a file from S3 if it already exists. We could probably get around this but it seems like good practice. If the file-format lambda isn't downloading the file, it can retry as many times as it likes without hitting the error with the SDK.
It will make it easier to move the other backend checks to use EFS and will simplify the code for those as well.

Looks good, thanks. Just one minor comment which is still open.
Also, @TomJKing are you happy with the changes from your comments?


RTSP to local file?

@starkillerOG are you interested in a PR to save a recording from the rtsp of the camera to a local file? Here's an example using ffmpeg: https://github.com/nicolasbrailo/zigbee2mqtt2web/blob/master/zigbee2mqtt2web_extras/reolink_cam/rtsp.py
If you think it could fit as an extra utility in this repo, I can prepare a PR.

I am trying to keep this library mainly focused on Home Assistant.
Therefore I don't want too many extra's that won't be used in HomeAssistant because they increase the maintenance burden.
(Which is already pretty high with about 70.000 HomeAssistant installs using this library).
Maybe you could create a seperate repo for addon-utilities that just uses reolink-aio as a dependency.
Something like "reolink_aio_utility"
How does that sound?


Checkboxes translation error

In color_calc_GUI.py not all titles can be translated after the program launch due to PySimpleGUI limitations.

Checkbox text update was fixed in PySimpleGUI v4.29


Refactor binaries into packages

This is a big PR. It:

Moves the build system to setuptools+pyproject.toml, fixing #559.
Moves all the bin/gftools-* scripts into a modules under a gftools.scripts package, and setting them as endpoints installed by setuptools, fixing #390 and probably also fixing #560.
Refactors most of the absl-based scripts to use argparse, fixing #531.
Deprecates a few scripts (font-diff and space-check) and removes a few obviously broken ones (gftools-cldr, gftools-list-variable-source, gftools-udhr), fixing #9 and #14 and closing #633.
Moves the tests from Lib/gftools/tests to tests/, since they don't need to be installed on the users' system.
Updates tests so that they actually begin to test some of the scripts' functionality.
Fixes an issue where the builder tests had to be run separately.

Overall looks good but I have a lot of beef with poetry. What does it bring to the table that pip doesn't?
Installing on my system isn't working due kurbopy which needs cargo. Which package uses kurbopy?

There's no poetry here.

kurbopy is now used by collidoscope which is used by fontbakery. And I think there's another problem with collidoscope which now prevents it being used in fontbakery, ugh.

Thanks.
Sorry, I thought pyproject.toml was a poetry thing, didn't realise it was established in pep 518.


Spotify Premium sponsorship for MiczFlor

Hi all,
as discussed in another thread, I truly belive that for all Spotify+ Phoniebox users it would be benefical to enable @MiczFlor with Spotify. However, as @MiczFlor is the creator and major contributor of this great project, I don't think he should pay this on his own, as he has no intentions do use Spotify for privat reasons (at least not yet ;)).
Therefor I suggest the community to contribute to a sponsorship for @MiczFlor.
I order to collect the required 99€ for the annual premium account I have created a paypal moneypool. As I never did something similar, this was for me the easyest way to get it done. If there are other suggestions do so please feel free. Here is the moneypool: https://paypal.me/pools/c/8lQ7tMIYad
downside of the moneypool as far as I have seen it, I cannot direct the money directly to @MiczFlor so it goes through my account first. However I hope this is somehow trustful. I have already put 30€ into the bucket to start over.
Looking forward for some contributions.
thanks!!!
Dominik

The Paypal Moneypool has ended on Friday. 18€ have been contributed. @MiczFlor I have just send you a spotify e-voucher of €60.
unfortunately we did not manage to get you the full fund for a yearly premium account

20 € für Spotify Premium sind im Topf für 2021.

20 € für Spotify Premium sind im Topf für 2021.


Information from logs is too low

Hello!
There is some log record in my installation '| DEBUG | falling back to AS number lookup"
This record does not answer on question: who? where?
Issue is understood. But object of issue is not.

Logging should indeed be improved and properly documented so it can fit all users needs.


Delete duplicated entries in ingest

Description of the task:
Project entries are being duplicated in ingest. We need to delete the duplicated entries for the following projects:
Easy to delete: only 1 project contains a submission
I have marked the one that shouldn't be deleted

lymphaticEndothelialCells

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=23062add-79c2-40da-955e-e17254171521
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=05b5c725-364d-46a3-b2a2-809fdeab2729
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=92d1e974-a69f-46ef-b3dc-50cdafa323c5
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=e6ef32d5-09d2-4b2c-8dde-ea53eb428dde
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=ba55a108-cc3f-4e71-8fc6-b85df1329e51
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=376ac0e1-0c52-45ef-b4d3-881ceead0589
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=5ee710d7-e2d5-4fe2-818d-15f5e31dae32

HumanDCsFromPre-cDCs

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=75d92425-4823-4e80-8d2c-6887c4ef21cc
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=05be4f37-4506-429b-b112-506444507d62

HumanPreimplantationEmbryosESCs

[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=3a694703-3084-4ece-9abe-d935fd5f6748
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=bf49faa6-a205-4c35-b5fd-c1498a9e9d7b

SingleCellsAlzheimers

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=04e68ad1-fd8e-4fb1-a5ff-0f1584d24232
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=b51f49b4-0d2e-4cbd-bbd5-04cd171fc2fa

humanPreimplantationEmbryos

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=fbd914f5-3e7c-4ea4-adb1-841bc8b16da3
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=03c6fce7-789e-4e78-a27a-664d562bb738

HumanCerebralOrganoidsFetalNeocortex

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=b1aa4646-7007-4ab7-bf3c-f788c9e7fe43
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=d2111fac-3fc4-4f42-9b6d-32cd6a828267

HumanT2DPancreas

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=ad51b83e-005f-40d3-8b92-bd315ba38018
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=c6ad8f9b-d26a-4811-b2ba-93d487978446

DropletSequencingNaturalGeneticVariation

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=f6e2ea92-453d-42ea-8451-38a490bc97d1
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=89332ae2-8b06-4acb-9f67-7c63b84c1f2d
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=8292b5b9-ae08-4335-9326-0583d128094f
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=a96b71c0-78a7-42d1-88ce-83c78925cfeb

pyleSkeletalMuscle

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=39225a95-b094-48f8-a899-e3bf1dfc48a1
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=f261e9d4-16b8-4e70-99ca-f70e5e8d2c67
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=4037007b-0eff-4e6d-b7bd-8dd8eec80143

TxDynEndotheilialCellsFromPluriStem

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=d2c430fd-712c-4d8c-a752-58dc17675921
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=1ac08b8f-67a5-4bb4-b6cf-3a00fd20f3f5
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=ef81890e-d383-4753-80bf-520a81258e8b
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=dd900b2f-b08f-4d4b-9f8e-c84c5b7a7c07
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=0cebda64-e64d-4add-b395-4b466430b277
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=abffe8ad-b764-4f6f-b81f-af87b8f40345
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=1ad6cee6-53a5-4804-a870-e5bd991186e0

ProstateCellAtlas

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=3ec7c6ca-2702-4e72-89eb-ba16ecacfd08
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=46d4d3d8-02a9-43c5-b897-d08ea9c8c2f6
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=b570ab7c-714b-4023-8677-835d276d6f16
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=768c0d7d-0be8-40c1-97a8-3b92bf973f30
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=53c53cd4-8127-4e12-bc7f-8fe1610a715c

atlasOfHumanTeeth

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=11d78590-bf38-4ef5-84c8-69035ed1447b
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=5f29fabb-27ad-4869-8583-eff09d0dd5c2
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=7fba6e1c-b275-4169-87ed-a315c29f87d5
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=53f5109b-1ec2-4874-b434-fff7c798dc12
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=fcfa8db5-9e87-436d-a418-60f7129d5ac8
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=d3446f0c-30f3-4a12-b7c3-6af877c7bb2d

HumanRPEandChoroid

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=4d2060ea-0a10-4335-9701-0d15b5aba589
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=8aff548d-7961-4882-acf6-63b0b7e0c06d
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=7880637a-35a1-4047-b422-b5eac2a2a358

CorticalNephrogenicNicheKidney

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=da2ee09e-e542-4329-9605-aee0dd26eeee
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=ab4cbda4-d7c0-4f94-8ee3-adb93552dea3
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=3c02ff88-5f40-4adb-8bb9-2ccc0fb8c108
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=fea71e91-f636-47ad-9b2e-f30b27c86c07
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=71436067-ac41-4ace-be1b-2fbcc2cb02fa

CervicalCancerLandscape

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=f4b29b4b-ecfd-4ab5-ba0e-6cb3a50a1fc2
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=86865343-cc7a-4866-a5e8-daedb703110f
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=d012d476-8f8c-4ff3-89d6-ebbe22c1b5c1

Human5Ovary10xChuva

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=3129daea-3084-4dcc-ae49-950654f5d0c7
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=650ceede-c517-4648-94c7-f439e7f33f24
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=82f70ee3-7f0e-4dc1-897f-eeca977e0498
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=faeedcb0-e046-4be7-b1ad-80a3eeabb066

stemCellTrophoblast

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=22c3fc0f-16f3-4e7b-ad9d-2a3f3397f952
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=da2747fa-2921-42e0-afd4-39ef57b2b88b

MilkEpithelialCells

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=4694e75d-8e73-4f90-b0a1-fa8237a62a14
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=8999b456-6fa6-438b-ab17-b62b1d8ec0c3

humanUrineCells

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=529f2dde-e58e-4123-900d-ad5d0d19f351
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=4af795f7-3e1d-4341-b867-4ac0982b9efd

Needs Manual checks: more than 1 project with submission
I have marked all the projects that contain a submission. Projects that don't contain a submission could be deleted quickly

DevelopingCardiacSystem

[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=e07897dc-c25e-453d-b04c-27ca33adb630
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=2086eb05-10b9-432b-b7f0-169ccc49d270

SapenaVentura-Human-10xv3sequencing

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=e13fda6c-f42f-471a-8d56-035bfde182f2
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=5f1a6376-433f-4960-9fba-5ef6f2de7f17

CordBloodHematopoieticStemCells

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=e77ba921-bacb-4f47-9c35-b55155e28296
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=c92ba772-ef8a-4f50-8ec8-e178e190bf7c
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=63b5b6c1-bbcd-487d-8c2e-0095150c1ecd

HumanFetalKidney

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=115da023-665e-4e69-a25b-860bb89bb2a5
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=a9ddf33d-b225-4959-8b91-4ce45f91eaf8
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=24c402d4-8504-44c5-887d-65e9452558a4
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=a3d1e1f8-aeba-4e29-b6fb-69406307c318
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=8a40ff19-e614-4c50-b23b-5c9e1d546bab

HeterogeneityCD4TCells

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=bbec9297-3f64-4453-bb27-d3cdee53cd75
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=9e468adf-2bf6-41a9-93e8-ffae2e75dc9c
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=28fd8a7e-65d5-457f-bf3a-85335ad73484
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=b425dc5f-8b43-4fc9-b55a-12cdc7ff35e2
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=f274325c-e218-499a-a524-316c3f94eda8
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=df88f39f-01a8-4b5b-92f4-3177d6c0f242

HumanPancreaticIslets

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=2c3b03fa-9fce-4d8e-9364-dde93fe194e1
[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=f0bec3b9-db67-43c6-90cf-f6d33f1d6803
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=78b2406d-bff2-46fc-8b61-20690e602227

HumanIleumCronsNormalHaberman

[ ] https://contribute.data.humancellatlas.org/projects/detail?uuid=0ccd30ef-6a23-4e68-b5a3-3d9ca511b5ed
[x] https://contribute.data.humancellatlas.org/projects/detail?uuid=504e0cee-1688-40fa-b936-361c4a831f87

Acceptance criteria for the task:

[ ] Only 1 tick per project in the list (Indicating remaining project)
[ ] All unticked projects have been deleted (And their associated submissions if there were)

Who is currently working on this? Is it you @ESapenaVentura ?


Annotation to automatically register subscriber

What do you think about creating an annotation that would cause an instance of the annotated class to be automatically registered (e.g. when EventBus is initialised or when an event is first posted)?

I have some questions.

When should the annotated class be automatically registered?
Should the class be automatically unregistered too? If so, WHEN?

Since instance will not be garbage collected when it is hold by other class (Event bus, for instance), so, I think it is necessary to unregister by hand.

It should be registered any time before a relevant event is posted.  That could be when EventBus is initialised, the first time post is called, or maybe even specifically when post is called for the first time with an event that the class listens for.  Whatever makes the most sense for the implementation.  The important thing is just that when a relevant event is posted, an instance of the class is ready to receive it.
In the scenario I am imagining, the class should not be unregistered automatically.  Basically these are single instance services that are listening for events throughout the lifetime of the process.

This is definitely out of scope for EventBus and should definitely be handled externally.

Perhaps this feature would be more appropriate for a sister util library that could take advantage of the new subscriber index and potentially hook into post so that an instance of the subscriber is only instantiated and registered when a relevant event is posted?


Add tests

@hmk I only just noticed this pull request which you opened and then closed more than a month ago...
Are you still interested to add tests to the main branch?
I have no strong opinions - let me know either way.

no need to merge from now... at some point will get around to adding a full test suite. thanks for the reminder!

Ah right, no worries then!
On 2 Dec 2024 at 14:48, Jake Heimark @.***>
wrote:

no need to merge from now... at some point will get around to adding
 a full test suite. thanks for the reminder!

 —
 Reply to this email directly, view it on GitHub
 [https://github.com/newfolder0/chrome-clacks/pull/16#issuecomment-2510517545],
 or unsubscribe
 [https://github.com/notifications/unsubscribe-auth/ACDAC7RKQ7JXV5W5HIJKF7L2DPKBDAVCNFSM6AAAAABQKUFBT2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJQGUYTONJUGU].
 You are receiving this because you commented.Message ID:
@.***>


start-reload.sh is missing in the backend base docker image

Local development build through docker-compose
via docker-compose up -d (everything by default) creates failing backend image:
/entrypoint.sh: 46: exec: /start-reload.sh: not found

Stupid mistake of mine. Had old image with same name (from full-stack project). All work when images are rebuilt fresh.

Thanks for reporting back and closing the issue 👍

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


How to add/change/remove a toolbar or tabs of a layout panel (after render)?

I expected the option to change the toolbar or tabs of a layout actually here:
http://w2ui.com/web/docs/w2layout.set

This is a fidldle showing how to dynamically add a toolbar to a layout: http://jsfiddle.net/UberMario/91pd3wc4/
The only way I got it to work was to first have any empty object assigned to the toolbar property of the layout.


Bounce rate more than 100%

Hi there!
Maybe I'm wrong, but I think the bounce rate shouldn't be more than 100%. In Release 1.6.0 I have a bounce rate of 117% for one of the websites.
Image with that bounce rate

Yeah, I have 143% and 168% on two sites, I really don't care personally since I'm not that interested in bounce rate but something seems wrong.


colorful console log

console logging supports color:
https://stackoverflow.com/questions/5762491/how-to-print-color-in-console-using-system-out-println
very useful, to give logging levels different colors. e.g

Trace: cyan
Debug: green
Info: white
Warning: Yellow
Error: Red
Fatal: Magenta

log appenders need to be modified, such that colors only apply to the console appender:

also see this:
https://en.wikipedia.org/wiki/ANSI_escape_code


Profile - Missing Save & continue button on 'Choose your bank account' selector

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

Version Number: 9.0.3-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Prerequisite: The user is logged in to the website.

Navigate to Account settings > Wallet
Click on Add Bank account> Personal Bank Account
On the Plaid modal, search and select Chase bank
Submit the credentials "user_good / pass_good"
Click on Continue until Plaid modal closes
Observe the 'Choose your bank account' selector

Expected Result:
The user sees a Selector and a 'Save & continue' button
Actual Result:
The 'Save & continue' button is not visible. When the user selects an account from the list, the 'Save & continue' button shows up
Workaround:
Unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/ecd96224-18c9-4797-a822-f436d9c30377

View all open jobs on GitHub

@joekaufmanexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

I don't think this is really a "bug". It makes sense that you can't proceed here until you select an account. Once you select an account, we show Save and continue immediately, allowing you to proceed.
I could see an argument that this is not ideal UX, since it is a bit odd that we don't assume your selection when there is only one account on the list. But this form 100% works as designed now, so I don't think it really makes sense to spend resources changing this before we really have many customers in the product.
https://github.com/Expensify/App/assets/54866469/32beaf5c-631c-4678-9dcf-4b3a4214c53b

Closing this until we get feedback from actual customers that this should work differently than as designed now.

Hmm I think technically we don't hide or disable our form submit buttons. We would make it so that the button is visible, but if it's submitted with no bank account selected, we'd throw a form error. cc @mountiny @koko57 in case we might have missed this one during some of the bank account flow refactoring.

I agree this should be visible. Very odd that this would be missed in the original PR, maybe something caused this regression after. Opening it to the community

Proposal
Please re-state the problem that we are trying to solve in this issue.
The 'Save & continue' button is not visible. When the user selects an account from the list, the 'Save & continue' button shows up
What is the root cause of that problem?
We're making submit button visibility dependent on the bank account selection https://github.com/Expensify/App/blob/74a6dbba2105ceba086f0387f50dcad573d4569f/src/pages/AddPersonalBankAccountPage.tsx#L83
What changes do you think we should make in order to solve the problem?

Simply remove that line, or set it always to true
What alternative solutions did you explore? (Optional)
NA

Proposal updated to add more details

@jayeshmangwani can you please review the proposals? thanks

@mountiny Yes I can but here @abdulrahuman5196 is C+ 😃

Yep sorry, @abdulrahuman5196 can you please review?

Proposal updated to add a small detail

@abdulrahuman5196 What is your ETA on this one? thanks!

Hi @mountiny  will check on this before EOD

Checking now

@nkdengineer 's proposal here https://github.com/Expensify/App/issues/44640#issuecomment-2209846945 looks good and works well. The other proposals are almost similar with minor variations.
🎀 👀 🎀
C+ Reviewed
@mountiny

@abdulrahuman5196 this PR is ready for review

PR in review

Same

Bumped in Slack

Payment here is:

$250 to @nkdengineer for PR via Upwork.

The PR that introduced the bug has been identified. Link to the PR:
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:
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:

Doesn't seem to be a regression.

Determine if we should create a regression test for this bug.

Yes.

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.

Navigate to Account settings > Wallet
Click on Add Bank account> Personal Bank Account
On the Plaid modal, search and select Chase bank
Submit the credentials "user_good / pass_good"
Click on Continue until Plaid modal closes
Observe the 'Choose your bank account' selector
Verify that: sees a Selector and a 'Save & continue' button

Hi @joekaufmanexpensify, I have moved to newdot payments.

@abdulrahuman5196 thanks for confirming on payments. I have updated the checklist and closed your upwork contract for this issue.

@nkdengineer $250 sent and contract ended!

Upwork job closed.

Closing as all that's left here is for @abdulrahuman5196 to request payment via NewDot.

$250 approved for @abdulrahuman5196


BREAKING: Refactor network options for neonKUBE clusters

AWS has defeated our (accidental) well-laid plan for unified cluster network settings across all hosting environments.
Up until now, users would specify the premise and node subnet in NetworkOptions.  These were originally intended for just on-premise deployments where we'd use the premise subnet to configure the node network settings and the node subnet to verify that the assigned node IP addresses fell into the expected range. But this morphed over time and we started referencing these for cloud deployments.  For the Azure hosting manager, we used the premise subnet to configure the Azure VNET and the node subnet to configure the Azure subnet within the VNET.
The problem is that AWS is a bit different.  It actually requires two subnets: a private one where the nodes will live and a public one which my best guess acts as a kind of DMZ for the load balancer and NAT gateways.  On AWS, we'll also need to create a VPC (virtual private cloud) which corresponds to an Azure VNET and which also needs to be configured with a subnet that surrounds the public and private subnet.  So we'll need to specify three subnets for AWS.
Here's the plan to address this in a somewhat consistent way:

NetworkOptions: PremiseSubnet will be by only used for on-premise hosting providers going forward (currently BareMachine, Hyper-V, and XenServer); the other hosting manager will look use settings from AwsHostingOptions, AzureHostingOptions, GoogleHostingOptions, etc.
I'm also going to remove the NetworkOptions.NodeSubnet property.  This isn't really required by on-premise providers and we'd only be using it to verify that node IPs fall within the range.  We'll switch to verifying IPs against the PremiseSubnet instead.

AzureHostingOptions: I'm going to add VnetSubnet and NodeSubnet properties to specify the CIDR for the Azure VNET and the subnet within where the nodes will live.

AwsHostingOptions: We'll add three CIDR properties: VpcSubnet, PublicSubnet, and NodeSubnet.

NetworkOptions.PodSubnet and ServiceSubnet will still be used to configure the internal Kubernetes networks and Nameservers will still be honored in all environments, along with the ingress related rules.
I don't feel too bad about this change.  This will enable us to model more complex cloud network configurations in the future, such as multiple node subnets in different availability zones or even clusters spanning multiple cloud regions.

DONE


Usage line for subcommands should include bin_name prefix

When showing the usage for a subcommand, clap only shows the subcommand name, not the command name before it.  For instance, given a command command, and a subcommand sub, command help sub will show:
sub
Subcommand about string goes here.

USAGE:
    sub [FLAGS] <arg>...

This should look like:
command sub
Subcommand about string goes here.

USAGE:
    command sub [FLAGS] <arg>...

This is also a bug, and I'm curious where this regressed. I should look like:
command-sub
Subcommand about string goes here.

USAGE:
    command sub [FLAGS] <arg>...

Why command-sub rather than command sub?

I think originally I modeled it after git where all subcommands in manpages have a hyphen separating commands.

I have a local branch with this fixed. I'm going to roll it up with a few of the other issues and push the PR.


Is it possible to have rotate feature?

I am looking for a way to have to zoom in out feature as well as rotate feature. but all the library i see doesn't have rotate feature. Can you please guide me an way to add this?

@mgorabbani In order to support rotation in PanZoom there are some internal modifications to make.
I will try to evaluate how much change it needs to work, but that's a feature I would definitely like to add.
To make it work I basically need to rewrite all my scaling / panning calculation with a transformation matrix (see http://angrytools.com/css-generator/transform/)

@mgorabbani I have implemented a PoC in master that supports rotation.
It is only available through the rotate function using refs and not yet available with touch control.
There is one small issue that makes the whole screen flicker when you rotate to +/-180deg, I will try to fix it and release a new version with the rotation feature.

wow! that's cool!

It's suffering some heavy performance degradation that I will have to deal with before releasing a new version. So make sure you don't use the latest code on master for production release.

Oh okay no problem.

@mgorabbani I've just released v0.2.5 that support rotation through the rotate method.
I am still struggling when passing from 180 to 181deg or -180 to -181deg, with some flickery transitions, but the overall performance issue has been fixed and should run smoothly from now on.


Strongly recommended add func about translate normal pic to round pic,it is very useful when create some picture.

Origin avatar:

And translate 1:

And translate 2:

@rhettli hello, how did you finally generate round pic ？


Hire a Developer to deploy an Exchange

Hello, I would like to know an estimate budget to help me deploy an exchange with Peatio.
In the exchange, I want to list cryptocurrency vs ERC20 Token (Ethereum Network). For example, BTC/Token... ETH/Token. No fiat in this exchange, only crypto trading.
If you can help me, please contact me and tell me an estimated budget.
Thanks in advance

Can I email you or do you have skype?

Sure, email me at<EMAIL_ADDRESS>

email me too<EMAIL_ADDRESS>

Hi MaffeProg,
I am also looking for a dev to help deploy and develop my front end for the peatio exchange.
I have just launched my company and will be integrating this software with my finance software and therefore this could be quite a big project.
If you could please contact me at<EMAIL_ADDRESS>that would be great as I have quite a long but interesting project involving peatio.
I need to deploy ( I already have a dedicated server for hosting) and dev of the front end for a pretty standard exchange. Sorry to barge in on your thread eduinvestor hope you don't mind.
Please let me know if you are interested and we can discuss further the requirements as I have a decent budget to get this done.
Thanks for your time.

Hello! Our team specializes in everything related to Peatio. Please feel free to contact me at<EMAIL_ADDRESS>to discuss what you are trying to achieve and how we can be of assistance.
Best regards!

@EagleEyedTiger @eduinvestor please ping to<EMAIL_ADDRESS>(live:apollo447722)
He is good Peatio developer.
I received help from him on Peatio

@eduinvestor @EagleEyedTiger   ///   I can help you. I am having a vast experience in Peatio, Crypto Currency exchange and trading platforms with FIAT currency based on Bitcoin, litecoin, Ethereum, XRP etc, Core Wallets and Daemons. I am having expertise in ICO ie Initial Coin Offering full system development, White Paper creation with full architecture and high level documentation, smart contract writing and many more skills.
My Skype : sushilraghav and email is<EMAIL_ADDRESS>

I've already developed and deployed 3 exchanges based on Peatio.
I can add more coins and customize exchanges with peatio.
Also have experiences in wlox.
Feel free to message me any time.
skype : live:apollo447722
mail<EMAIL_ADDRESS>Regards.


Fix #570

Yes it does work and yes I have seen your comment. HOWEVER, when you say "break", I don't understand what it means, so for me that was something absurd for a query breaking because of no order parameter

The fact that null values comes first with a Desc order have been a normal behavior for postgres. I just forgot about it and didn't add LAST or FIRST to tweak it.


Selected not updating picker UI

I'm writing an app that allows the user to select one Facebook image at a time using the picker. After each selection I want the selected image to reset to nil and the picker UI to update to reflect this. I am able to reset the selected images array, but the UI won't update, it continues to show the previously selected images. Has anyone else experienced this issue? Any solutions?

@rc1911, I have the same logic in the app (allowing to pick a single image). So, I simply dismiss the picker on selecting the first image.
- (void)facebookImagePicker:(OLFacebookImagePickerController *)imagePicker didSelectImage:(OLFacebookImage *)facebookImage {
    [self dismissViewControllerAnimated:YES completion:^{
        // process facebookImage
    }];
}


Add ability to choose font for function keys

I implemented this in a cleaner way and pushed it. Thanks for the PR.


Synthetic population generator and census tracts as region ids

This PR includes a script to generate synthetic populations. The algorithm samples houses from the census datasets available in pums based on the household weight and the desired population size. Region coordinates are downloaded for each region based on their PUMA codes. These coordinates are then assigned to each house based on their PUMA code. Finally, two files are generated: people (age, home_id), and region (region_id, lat, lon). Region id is generated as follows: 2 digits for state, 9 digits for PUMA code. Home id is region id + 6 digits that represent the home number.
Census tracts have been changed to region id in the ixa code.
These populations are for testing purposes only.

Why is the build and test workflow failing here? The branch run works locally for me


yasgui (yasr) doesn't understand HTTP post response

I have my own server, with a SPARQL endpoint and I would like to send queries against it with yasgui. Sending the query works well, but yasgui doesn't understand the query response. I'm wondering if I'm doing something wrong.
When I send a HTTP post request the output looks like this:
% curl -v --data 'query=select+*+where+{?s+?p+?o}+limit+1' 'http://localhost:9999/sparql'                                                                                M
*   Trying ::1...
* connect to ::1 port 9999 failed: Connection refused
*   Trying <IP_ADDRESS>...
* Connected to localhost (<IP_ADDRESS>) port 9999 (#0)
> POST /sparql HTTP/1.1
> Host: localhost:9999
> User-Agent: curl/7.47.1
> Accept: */*
> Content-Length: 39
> Content-Type: application/x-www-form-urlencoded
>
* upload completely sent off: 39 out of 39 bytes
< HTTP/1.1 200 OK
< Server: akka-http/2.4.3
< Date: Thu, 14 Apr 2016 21:39:22 GMT
< Content-Type: application/sparql-results+xml; charset=UTF-8
< Content-Length: 524
<
<?xml version="1.0"?>
<sparql xmlns="http://www.w3.org/2005/sparql-results#">
  <head>
    <variable name="s"/>
    <variable name="p"/>
    <variable name="o"/>
  </head>
  <results>
    <result>
      <binding name="s">
        <uri>http://test.model/_root_/helloWorld</uri>
      </binding>
      <binding name="p">
        <uri>http://www.w3.org/1999/02/22-rdf-syntax-ns#type</uri>
      </binding>
      <binding name="o">
        <uri>http://schema.org/Text</uri>
      </binding>
    </result>
  </results>
</sparql>
* Connection #0 to host localhost left intact

Is anything wrong with that? Because yasr doesn't display the result but tells me the following:
Resource representation is only available with these types:
application/sparql-results+xml; charset=UTF-8

The error message doesn't make any sense to me

This is indeed a strange error. I tried loading your SPARQL response in YASR as string (via yasr.setResponse(<string>)), and that works as expected.
This error does not seem to be thrown by YASR, but by the server.
I saw you use akka, and this ticket seems to be related to the problem you're having: https://github.com/akka/akka/issues/18425
Apparently, there seems to be a accept-header issue here. To debug, I suggest you take the curl request send by the browser (in chrome, open dev toolbar, select network tab, select the sparql request, and select 'copy as curl'), and see whether that curl request throws an error (I suspect it does)

Alright, thanks for investigating this. I couldn't see that the error came from the server side. The curl request from chrome revealed what was wrong. The header -H 'Accept: application/sparql-results+json' was passed, but I delivered XML output because I didn't look at the header. After delivering JSON everything works as it should.


1.5.0 changes random values with seeds

I'm not sure if this is considered a breaking change, but all my tests started failing because my seeded generators started returning different values. I suspect this will also impact a lot of other folks.
What do you think about yanking 1.5 and releasing 2.0 instead with the new algorithm?

Sorry for the breakage!
As for this topic, I tend to follow rand's docs.

Any “value-breaking” changes to the generator should require bumping at least the minor version and documentation of the change.

https://docs.rs/rand/0.8.4/rand/trait.SeedableRng.html#tymethod.from_seed


Transport: Add HttpClientFactory support

This PR adds support for the user to provide a factory for HttpClient instances.
Scenarios
When building ASP.NET Core applications on NET Core 2.1/3, users can take advantage of IHttpClientFactory to reuse HttpClient instances across the application. This also allows HttpClient instances to be reused across CosmosClient instances.
Benefits of using IHttpClientFactory on ASP.NET Core applications: https://docs.microsoft.com/en-us/dotnet/architecture/microservices/implement-resilient-applications/use-httpclientfactory-to-implement-resilient-http-requests#benefits-of-using-ihttpclientfactory
When building Blazor applications on WebAssembly, the applications use a preconfigured HttpClient instance that needs to be injected (reference https://docs.microsoft.com/en-us/aspnet/core/blazor/call-web-api?view=aspnetcore-3.1)
API
This PR adds HttpClientFactory to CosmosClientOptions:
public Func<HttpClient> HttpClientFactory { get; set; }

And WithHttpClientFactory to CosmosClientBuilder:
public CosmosClientBuilder WithHttpClientFactory(Func<HttpClient> httpClientFactory)

To provide a delegate that matches IHttpClientFactory.CreateClient.
The new API is not compatible with CosmosClientOptions.WebProxy and if the user tries to set both, there will be an ArgumentException.
Why not use IHttpClientFactory on the API?
To avoid a reference to Microsoft.Extensions.Http on the Cosmos DB SDK package unnecessarily.
Usage
In a Blazor WebAssembly application:
public static async Task Main(string[] args)
        {
            var builder = WebAssemblyHostBuilder.CreateDefault(args);
            builder.RootComponents.Add<App>("app");

            builder.Services.AddTransient(sp => new HttpClient { BaseAddress = new Uri(builder.HostEnvironment.BaseAddress) });

            builder.Services.AddHttpClient();

            builder.Services.AddSingleton<CosmosClient>(sp =>
            {
                var httpClientFactory = sp.GetService<IHttpClientFactory>();
                CosmosClientOptions cosmosClientOptions = new CosmosClientOptions();
                cosmosClientOptions.HttpClientFactory = httpClientFactory.CreateClient;
                cosmosClientOptions.ConnectionMode = ConnectionMode.Gateway;
                return new CosmosClient("<connection-string>", cosmosClientOptions);
            });

            await builder.Build().RunAsync();
        }

In an ASP.NET Core application:
public void ConfigureServices(IServiceCollection services)
{
    // other service configuration
    // adds in HttpClientFactory
    services.AddHttpClient();
    services.AddSingleton<CosmosClient>(sp =>
            {
                var httpClientFactory = sp.GetService<IHttpClientFactory>();
                CosmosClientOptions cosmosClientOptions = new CosmosClientOptions();
                cosmosClientOptions.HttpClientFactory = httpClientFactory.CreateClient;
                cosmosClientOptions.ConnectionMode = ConnectionMode.Gateway;
                return new CosmosClient("<connection-string>", cosmosClientOptions);
            });
}

Related to #1429. One more PR is required to fully support Blazor WebAssembly, but this is a good first step.
Type of change

[X] New feature (non-breaking change which adds functionality)

        httpClient.Timeout = (requestTimeout > this.requestTimeout) ? requestTimeout : this.requestTimeout;

Overriding these on CX given client seems like overstepping each other.
Any other possibilities?

Refers to: Microsoft.Azure.Cosmos/src/GatewayStoreModel.cs:111 in a4ae553. [](commit_id = a4ae55365e23ef3bd11fcc94411674433411a315, deletion_comment = False)

@kirankumarkolli If the user is setting the RequestTimeout through ClientOptions and also providing an HttpClient from the factory, I don't see the issue of configuring the provided instance.
We have 2 choices, either assume the user would configure the timeout, or not.
If we target the most common scenarios for users to use this API, which are described in the Description, then the user would not be configuring the timeout there, but they could be setting the timeout on the ClientOptions and I would expect the Options to be honored.
If the user is indeed configuring a custom timeout on the HttpClientFactory, and they expect that to be honored, we just tell them that they can configure that through the ClientOptions, as they can do right now.

@kirankumarkolli I added a new Feature flag on the UserAgent, so we can identify when a user is using their own HttpClientFactory


cbuild setup --packs does not install missing packs requested by layers

Describe the bug
When a pack that is referenced by a layer is missing, the layer detection fails with:
cbuild setup USB_Device.csolution.yml --context-set --packs --update-rte
error csolution: required pack<EMAIL_ADDRESS>not installed
error csolution: no compatible software layer found. Review required connections of the project

To Reproduce
Install these packs:
<EMAIL_ADDRESS>ARM::CMSIS-Driver_STM32
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Make sure that ARM::CMSIS-Compiler pack is not installed!
Then use the USB_Device example configured for board: STMicroelectronics::B-U585I-IOT02A as shown above.
Expected behavior
The layer detection does not fail due to missing packs referenced by a potential layer.  Once the layer is specified, the --pack option downloads the pack.
Environment (please complete the following information):

CMSIS-Toolbox 2.6.1
Windows

Root cause: today the list layers command reads all pdsc file from packs listed in potentially matching layers and therefore reports missing packs and results in the corresponding layer no matching.


high latency

Hey,
I just bought a Oculus Quest and tried with using ALVR. Unfortunately I have very high latency.
On Riftcat it was working fine. No matter what, it keeps up the latency high. I am using a 2.4G wifi connection on the pc, and a 5G on the Quest.
PC Specs
1080 Ti
32 GB Ram
i7-8700K
Not sure what could fix this problem.

5g on the pc is also needed

or ethernet

Ordinarily you'd use ethernet to wire your PC to the router, 2.4GHz on a computer (especially of those specs) is barbaric.  Then use 5.4GHz with only limited channel selection to the device.
Alternatively, you can use a USB WiFi adapter and use network sharing so that the device can connect directly to the computer.  Again, 802.11 AC (5.4 GHz) is preferable.  ALVR is superior to riftcat, but requires better networking hardware to facilitate it.


[wip] build.gradle: update jacoco settings

see if we can exclude generated files from the coverage data

Coverage decreased (-7.73%) to 50.04% when pulling 68b5f6fa6432e928b79adb42505adb1ac15d71ca on jacoco-ignore-generated-files into 6573fca6ba809d6d3eaf76d7ba90d1c86dff1518 on master.


build: update formidable to 2.0.1

The Make file doesn't run tests correctly for me (with fresh clone with yarn install && yarn test or after changes with yarn install && yarn test) so unable to test locally - almost all of the tests fail with "Error: Forbidden" when I checkout the master branch
Edit: CI tests via Travis aren't running either as Travis shut down travis-ci.org back in June this year, so the project needs to move to travis-ci.com with limited open source credits (though they have dubious security practises of late) or switch to GitHub Actions.
Closes #1650 #1609

Also why not just go all the way to 3?

Also why not just go all the way to 3?

2.x is the @latest tag on NPM.
3.x is re-written as an ES Module (not a dual ESM/CJS package), a breaking change for users of anything that uses formidable in an environment that does not support, or can't parse ESM

Also why not just go all the way to 3?

2.x is the @latest tag on NPM.
3.x is re-written as an ES Module (not a dual ESM/CJS package), a breaking change for superagent users in an environment that does not support, or can't parse, ESM.

If you will entertain 1 more question what is the reason supertest can't go full ESM stack? What still relies on CommonJS

I don't support ESM (nor TypeScript) right now

Tests seem to be broken right now, possible due to this PR or 3c9c0f7feef61328131ae43b06e628bce1c20c17 (related to #1267).
If someone can fix and submit a PR, then I can publish a new release of this package to npm.
You can reproduce by cloning this repo, running yarn, and then yarn test or just make test works.

I don't support ESM (nor TypeScript) right now

fyi, esm packages can be lazy imported from cjs using import('formidable').then(...)


"Monocle is initializing, please wait" forever on Android browser

When trying to view the web interface on an Android Lollipop browser, the map never appears. The "Monocle is initializing, please wait." message stays there forever.
The Android debug log reveals a javascript syntax error in main.js line 467. Looks like populateSettingsPanels() is using javascript template literal backticks, which are not supported on this version of Android.

Hmm, I did not have that problem on marshmallow or nougat.
Must have been an oops since  newHtml  started out with single quotes and it doesn't look like template literals are even needed?

Can I Use is somewhat helpful here by flagging template literals as unsupported on some Android versions, but it doesn't make it very clear that support is not present in all Android distributions of a particular version. For example, Cyanogenmod 12.1 (Android 5.1) doesn't use Chromium as its default browser, so template literals fail there.
Anyway, yes, maybe the backticks were an accident. They were not necessary here, and replacing them fixes the problem without giving up anything.


Env increases the mirror source address, such as ubuntu

In China, many mirror sources cannot be accessed, so you can increase the configuration mirror source in the env file to increase the Linux image of your country.

CHANGE_SOURCE
Already found

https://github.com/laradock/laradock/blob/master/workspace/Dockerfile
No modification of China mirror warehouse


Some apps are on top of the Full Screen Grab window

Could not reproduce this on Windows 11, need to test with Windows 10


Eval bug for speech recognition facebook/wav2vec2-large-lv60

System Info
docker 1.13-463

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
Error running the readme cmd
RuntimeError: cpu fallback is not supported during hpu graph capturing. You should run inference in lazy mode only with use_lazy_mode=True and use_hpu_graphs_for_inference=False.
Should we remove use_hpu_graphs_for_inference=False from the readme command?
Expected behavior
train/eval to completion without cpu fallback error

https://github.com/huggingface/optimum-habana/pull/588


Move (build infrastructure for) documentation to NCAS-CMS space

Transfer the full cf-plot documentation, as-is, and importantly wit the full capability to re-generate from source updates, from http://ajheaps.github.io/cf-plot/ to a new home of https://ncas-cms.github.io/cf-plot, consistent with the other NCAS CF Data Tools libraries. This change is already promised under the current homepage:

Warning - these pages are static and will be removed at some stage in the future. The new location will be https://ncas-cms.github.io/cf-plot

Closed after I merged #33 and set up GitHub Pages to deploy from the master branch docs/ dir. The URLs will now be under the root https://ncas-cms.github.io/cf-plot/build/* e.g. https://ncas-cms.github.io/cf-plot/build/index.html where the build/ component of the path is so that I can keep the generated built content (HTML, etc.) separate to the source content (mostly RST files), but I can redirect so we don't need that featuring in the URL path.


[Bullet] Unable to use imported collision data in other nodes

Godot version
3.3.2.stable
System information
5.12.12-arch1-1
Issue description
It seems you are unable to use the collision shape data imported from blender in other collision shapes. I'm using the "-colonly" hint in blender which, when imported into Godot, results in a scene with the following nodes and resources:
Spatial -> StaticBody -> CollisionShape.shape(ConcavePolygonShape)
I'm trying to use that ConcavePolygonShape resource in this
Area -> CollisionShape.shape(ConcavePolygonShape)
It doesn't matter if I duplicate the ConcavePolygonShape, duplicate the CollisionShape or even ConcavePolygonShape.set_faces(ConcavePolygonShape.get_faces()). Nothing I do will let me use the imported shape in an Area -> CollisionShape. That same data can be used in a MeshInstance.mesh(ArrayMesh) just fine.
Steps to reproduce
Attached is a reproduction project. The left cube is an Area Mesh with CollisionShape.shape(BoxShape) that was created in Godot. The right cube is an Area Mesh with a CollisionShape and the shape is set in Spatial.gd. The third cube is a MeshInstance.mesh(ArrayMesh) that was created using the imported shape data.
You can move the cylinder left and right with A/D or the arrow keys. When the cylinder collides with a cube it should print "entered" in Output. This only happens with the left cube, which is the problem.
Minimal reproduction project
test.zip

Confirmed in 3.4.beta5, when setting concave shape faces by script, the internal shape doesn't get properly updated in Bullet.
Note: Doesn't occur when switching the 3D physics engine to Godot Physics.

Does this occur with Godot 4.x, or only Bullet in Godot 3.x?


Use common retry/backoff implementation

There are at least three implementations of retry (with optional backoff) logic in our code. Use a common one, either own implementation or from a library.

BUT: I would rather say, that we should probably use some sort of backoff library: https://medium.com/@lestrrat/yak-shaving-with-backoff-libraries-in-go-80240f0aa30c Ultimately this code go into a pkg/backoff package itself, but there are already great implementations for that.

From #1344 comments

Looks like this has been resolved.


Adds support to cast an Ion value to Hive string.

Issue
When create a hive table, it raises an error if assign a Hive String field some Ion value (e.g. Ion timestamp, Ion integer).
For example,
CREATE EXTERNAL TABLE `test`(`starttime` string)
ROW FORMAT SERDE  'com.amazon.ionhiveserde.IonHiveSerDe'
STORED AS INPUTFORMAT 'com.amazon.ionhiveserde.formats.IonInputFormat'
OUTPUTFORMAT 'com.amazon.ionhiveserde.formats.IonOutputFormat'

and the Ion file is:
{starttime: 2020-11-05T07:18:59.968Z}

it throw exception:
Failed with exception java.io.IOException:org.apache.hadoop.hive.ql.metadata.HiveException: java.lang.ClassCastException: com.amazon.ion.impl.lite.IonTimestampLite cannot be cast to com.amazon.ion.IonText

Change
ion-hive-serde XToStringObjectInspector only supported Ion Text, Adds support to other Ion values. In addition, Hive will still return null for values that Hive doesn't support.

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

Comments: We support Ion container type -> Hive string too.
For instance, Ion struct {a:2} will be convert to string "{a:2}"

Below commits worked on comments above and includes two changes:

Changed the method used for getting string value.
Adds more tests including Ion containers, timestamps etc. refactor some helper functions.


refactor: rewrite deposit class service to functions

Key changes:

Move deposit service to a new location and refactor
Add getAvailableDepositRoutes for getting available deposit routes for a given wallet connection and selected network

:tada: This PR is included in version 2.8.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Installation fails for Jazzy

Description
Overview of your issue here.
Your environment

ROS Distro: Jazzy
OS Version:  Ubuntu 24.04
Source or Binary build? Source
If binary, which release version?
If source, which branch? Master
Which RMW (Fast DDS or Cyclone DDS)? Humble or Rolling should be using Cyclone. Dont know

Steps to reproduce
I followed the instructions to install ros2 jazzy.
Then I followed with instructions for moveit. Until building the workspace everything was fine.
Expected behaviour
Tell us what should happen
Actual behaviour
When i executed
sudo apt update && rosdep install -r --from-paths . --ignore-src --rosdistro $ROS_DISTRO -y

I get the following error
executing command [sudo -H apt-get install -y ros-jazzy-gazebo-ros2-control]
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
E: Unable to locate package ros-jazzy-gazebo-ros2-control
ERROR: the following rosdeps failed to install
  apt: command [sudo -H apt-get install -y ros-jazzy-gazebo-ros2-control] failed
  apt: Failed to detect successful installation of [ros-jazzy-gazebo-ros2-control]

Any help would be greatly appreciated.
Edit
I noticed when running
sudo apt update && rosdep install -r --from-paths . --ignore-src --rosdistro $ROS_DISTRO -y

That it tries to find gazebo_ros2_control which is Gazebo classic meaning "Gazebo Classic is not released to Ubuntu Noble. As a consequence, gazebo_ros2_control won’t be released for Jazzy and Rolling anymore."
Is it possible to change this in the moveit installation?

Jazzy is not yet supported by MoveIt, release and support progress is tracked here #2846.
Your issue is indeed caused by gazebo_ros2_control not being available as debian package anymore, and gz_ros2_control being the successor to switch to.
Also, this dependency is not a MoveIt dependency, more likely it is being installed by one of the tutorial resources. If you could track down which one that is and update the dependency, that would be very helpful for the migration. I'm closing this issue for now since it doesn't affect any of the supported distros.


python312Packages.craft-application: 4.4.0 -> 4.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.craft-application is: Basis for Canonical craft applications
meta.homepage for python312Packages.craft-application is: https://github.com/canonical/craft-application
meta.changelog for python312Packages.craft-application is: https://github.com/canonical/craft-application/blob/refs/tags/4.5.0/docs/reference/changelog.rst
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.5.0 with grep in /nix/store/fs2cds3gmh2b221wadsm8jvzn231w10f-python3.12-craft-application-4.5.0
found 4.5.0 in filename of file in /nix/store/fs2cds3gmh2b221wadsm8jvzn231w10f-python3.12-craft-application-4.5.0

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

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath

charmcraft
python311Packages.craft-application
python312Packages.craft-application
rockcraft
snapcraft

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fs2cds3gmh2b221wadsm8jvzn231w10f-python3.12-craft-application-4.5.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/db46aae5a194e106d9224b21dfd14fbcdd8b7199#python312Packages.craft-application

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fs2cds3gmh2b221wadsm8jvzn231w10f-python3.12-craft-application-4.5.0
ls -la /nix/store/fs2cds3gmh2b221wadsm8jvzn231w10f-python3.12-craft-application-4.5.0/bin

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

x86_64-linux

:x: 2 packages failed to build:

snapcraft
snapcraft.dist

:white_check_mark: 8 packages built:

charmcraft
charmcraft.dist
python311Packages.craft-application
python311Packages.craft-application.dist
python312Packages.craft-application
python312Packages.craft-application.dist
rockcraft
rockcraft.dist

Maintainer pings
cc @jnsgruk for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Diff looks good, will just figure out why snapcraft failed to build before merging

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 363440

x86_64-linux

:x: 2 packages failed to build:

snapcraft
snapcraft.dist

:white_check_mark: 8 packages built:

charmcraft
charmcraft.dist
python311Packages.craft-application
python311Packages.craft-application.dist
python312Packages.craft-application
python312Packages.craft-application.dist
rockcraft
rockcraft.dist

Build failure for snapcraft was due to python-apt, which a simple version bump to the latest upstream version fixes. Re-running build and will then merge.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 363440

x86_64-linux

:white_check_mark: 14 packages built:

charmcraft
charmcraft.dist
python311Packages.craft-application
python311Packages.craft-application.dist
python311Packages.python-apt
python311Packages.python-apt.dist
python312Packages.craft-application
python312Packages.craft-application.dist
python312Packages.python-apt
python312Packages.python-apt.dist
rockcraft
rockcraft.dist
snapcraft
snapcraft.dist


Week 3  Day 4 morning session

Good implementations overall, however, you may change some margins in the projects sections as follows:

You could set the img width property to 50%.

Thank you for your review and for the recommendations. 🌟 🌟 ⭐  Changes have been made. 🚀 🥇


[SWOC'24] Issue Templates Ready

Closes #16
There are 5 issue templates-

Bug report issue template
Documentation report issue
Feature request issue
Styling request issue
Other issue

@pinak22dhir please review this pr and requested to merge this pr under SWOC'24

thanks for committing @Avdhesh-Varshney , don't forget to star the repository


Allows usage of URL pointing to a local file in file operations

Description
Allows to provide an URL object pointing to a local file with CommitFile operation.
Providing a non-local file will throw an error.
This is not available in the browser.
Fix https://github.com/huggingface/huggingface.js/issues/106
await commit({
  repo,
  title: "Some commit",
  credentials: {
    accessToken: TEST_ACCESS_TOKEN,
  },
  operations: [
    {
      operation: "addOrUpdate",
      content: new URL("file://tsconfig.json"),
      path: "tsconfig.json",
    },
  ],
});

@coyotte508 About handling file from the browser side, shouldn't we support File object that can be get when using the file picker API?
See also https://developer.chrome.com/articles/file-system-access/


Added Portuguese translation for readme and contributing!

Pull Request
Description:
This pull request adds portuguese translation for readme and contributing! I also added a "Português" button for the main readme (english version).
Related Issue:

Closes #108

@all-contributors add @saxocat for translations


v1.0.13 causes yargs to error

After upgrading to v1.0.13 the generation now fails with the following error and no longer generates the README as expected:
npx github-action-readme-generator --action ./action.yaml --readme ./README.md
Need to install the following packages:
  github-action-readme-generator
Ok to proceed? (y) y
✅ SUCCESS action loaded successfully
(node:96659) Warning: "version" is a reserved word.
Please do one of the following:
- Disable version with `yargs.version(false)` if using "version" as an option
- Use the built-in `yargs.version` method instead (if applicable)
- Use a different option key
https://yargs.js.org/docs/#api-reference-version
(Use `node --trace-warnings ...` to show where the warning was created)
✨ INFO  save Configuration saved successfully.

And when explicitly executed via node to include the --trace-warnings flag I get the following results
./README.md
✅ SUCCESS action loaded successfully
(node:45536) Warning: "version" is a reserved word.
Please do one of the following:
- Disable version with `yargs.version(false)` if using "version" as an option
- Use the built-in `yargs.version` method instead (if applicable)
- Use a different option key
https://yargs.js.org/docs/#api-reference-version
    at Object.emitWarning (/Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8575:560)
    at Xt.[emitWarning] (/Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8185:58)
    at Xt.option (/Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8020:160)
    at Xt.options (/Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8031:21)
    at /Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8017:18
    at Array.forEach (<anonymous>)
    at Xt.option (/Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8016:27)
    at Xt.options (/Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8031:21)
    at exports.Argv.Argv.loadArgv (/Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8669:128)
    at exports.Argv.Argv.loadSync (/Users/sesmith/.nvm/versions/node/v16.13.0/lib/node_modules/github-action-readme-generator/dist/index.js:8664:12)
✨ INFO  save Configuration saved successfully.

Upon downgrading to v1.0.12 generation occurs correctly on the same action.yaml/README.md:
npx<EMAIL_ADDRESS>--action ./action.yaml --readme ./README.md
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
✅ SUCCESS action loaded successfully
::group::🚀 START title
✨ INFO  title Writing 24 characters to the title
✨ INFO  title Title: # GitHub Action: Calculate Version Number
✨ INFO  readme-writer Looking for the title token in /Users/sesmith/git/release-tooling-reusable-workflows/actions/versioning/calculate-version/README.md
✨ INFO  readme-writer Found the start title token
✨ INFO  readme-writer Found the end title token
✨ INFO  readme-writer successfully updated the title section
✅ SUCCESS title
::endgroup::
::group::🚀 START description
✨ INFO  description Writing 114 characters to the description section
✨ INFO  readme-writer Looking for the description token in /Users/sesmith/git/release-tooling-reusable-workflows/actions/versioning/calculate-version/README.md
✨ INFO  readme-writer Found the start description token
✨ INFO  readme-writer Found the end description token
✨ INFO  readme-writer successfully updated the description section
✅ SUCCESS description
::endgroup::
::group::🚀 START usage
✨ INFO  usage Action name: discoveryinc-dtc/release-tooling-reusable-workflows
✨ INFO  usage Version string: undefined
✨ INFO  readme-writer Looking for the usage token in /Users/sesmith/git/release-tooling-reusable-workflows/actions/versioning/calculate-version/README.md
✨ INFO  readme-writer Found the start usage token
✨ INFO  readme-writer Found the end usage token
✨ INFO  readme-writer successfully updated the usage section
✅ SUCCESS usage
::endgroup::
::group::🚀 START inputs
✨ INFO  inputs Action has 6 total inputs
✨ INFO  readme-writer Looking for the inputs token in /Users/sesmith/git/release-tooling-reusable-workflows/actions/versioning/calculate-version/README.md
✨ INFO  readme-writer Found the start inputs token
✨ INFO  readme-writer Found the end inputs token
✨ INFO  readme-writer successfully updated the inputs section
✅ SUCCESS inputs
::endgroup::
::group::🚀 START outputs
✨ INFO  outputs Action has 4 total outputs
✨ INFO  readme-writer Looking for the outputs token in /Users/sesmith/git/release-tooling-reusable-workflows/actions/versioning/calculate-version/README.md
✨ INFO  readme-writer Found the start outputs token
✨ INFO  readme-writer Found the end outputs token
✨ INFO  readme-writer successfully updated the outputs section
✅ SUCCESS outputs
::endgroup::
✨ INFO  save Configuration saved successfully.

The .ghadocs.json file (with owner data redacted)
{
  "owner": "REDACTED",
  "repo": "release-tooling-reusable-workflows",
  "paths": {
    "action": "./action.yaml",
    "readme": "./README.md"
  },
  "show_logo": "true",
  "versioning": {
    "enabled": "true",
    "override": "",
    "prefix": "v",
    "branch": "main"
  },
  "title_prefix": "GitHub Action: ",
  "pretty": "true"
}

I took a quick look at the changes in 1.0.13 and nothing stood out, but will try to find the issue later.

@wwsean08 adding here since it is a reppo issue with @1.0.13
npx install the last version waits indefinetely, and had to stop manually
$ npx github-action-readme-generator
[..................] \ fetchMetadata: sill resolveWithNewModule<EMAIL_ADDRESS>checking install

@fty4 I have addressed this in version v1.2.3


adding social media icons with animation effects

social media icons with animation effects
before

after
https://github.com/user-attachments/assets/06934486-4839-4f0f-831e-8e5bb4b67a04
**Issue #44

@Sujal1201 please resolve conflicts

what is this error sir

please assign the issue


[Bug]: insert_nodes with VectoStoreIndex results in AttributeError: 'tuple' object has no attribute 'embedding'

Bug Description
Hi there, I am trying to set up data management for my vector store and would like to insert on the most granular level, i.e. nodes with 'insert_nodes'. The following code
from llama_index import VectorStoreIndex
from llama_index.schema import TextNode
node1 = TextNode(text="<text_chunk>", id_="<node_id>")
node2 = TextNode(text="<text_chunk>", id_="<node_id>")
nodes = [node1, node2]
index = VectorStoreIndex(nodes)
index.insert_nodes(nodes=node1)

gives the error 'AttributeError: 'tuple' object has no attribute 'embedding''
Regular inserts for documents work.
I tried around quite a bit. Adding service contexts with all necessary embedding information didn't help and tried specific other vector stores (e.g. faiss). All unsuccessfull. Any ideas? Would consider this an important basic functionality.
Thx guys and nice job
Andreas
Version
latest
Steps to Reproduce
from llama_index import VectorStoreIndex
from llama_index.schema import TextNode
node1 = TextNode(text="<text_chunk>", id_="<node_id>")
node2 = TextNode(text="<text_chunk>", id_="<node_id>")
nodes = [node1, node2]
index = VectorStoreIndex(nodes)
index.insert_nodes(nodes=node1)

Relevant Logs/Tracbacks
File ".venv/lib/python3.11/site-packages/llama_index/indices/vector_store/base.py", line 104, in _get_node_with_embedding
    id_to_embed_map = embed_nodes(
                      ^^^^^^^^^^^^
  File ".venv/lib/python3.11/site-packages/llama_index/indices/utils.py", line 131, in embed_nodes
    if node.embedding is None:
       ^^^^^^^^^^^^^^
AttributeError: 'tuple' object has no attribute 'embedding'

@ejmiddle as dosu mentions, insert_nodes() is meant to take a list
This would work: index.insert_nodes([node1, node2])
Note that this would duplicate those nodes in the index, since you already built the index with those nodes

Thx! Got confused as it seems different for inserts of documents


display_formats broken?

I can use display_formats in hv.extension() in the notebook if I specify the default value:

But if I choose svg or png I get errors:

I'm trying to find a pure Python replacement for the %output magic, and I think this is intended to be it, but it doesn't seem to be working.

The docs also make it sound like multiple display formats are allowed (as it's a list), but that gives an additional error:

The double traceback is due to an older IPython version, what version are you using so we can pin past it?

ipython=5.4.1


[Apple]Enable JIT for macOS 11 and iOS14

marble tower runs fine (60fps) on Safari. With Babylon Native, it runs at around 10fps once there are some marbles.

It looks like it's related to JIT that is disabled by default
https://developer.apple.com/documentation/bundleresources/entitlements/com_apple_security_cs_allow-jit?language=objc
We should see the same issue on iOS.

Enabling Hardened mode/JIT gives a crash
https://developer.apple.com/documentation/security/hardened_runtime?language=objc
To investigate more ...

I've tested marble tower on ios14/iphone11 and it run at 0.5fps...1 frame every 2 seconds.
This scene is CPU intensive thanks to the physics.

more tests on iphone11/ios14
Cesium man 60fps
Ryan's spheres branch 60fps

profile of the playground. Note the small amount of time taken by the GPU


docs: add readme to vtl resolvers directory

Issue #, if available: Closes #6325
Description of changes:
Adds a README to amplify/backend/api/<MyApp>/resolvers directory describing the override process for resolvers. More details in the linked issue.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Wording looks good to me.

@edwardfoyle Thank you for the review. I have made the required changes, and fixed the lint issues as well.


[Bug-Fix][BTS-1747] Windows CI do not produce cores (case: hardKills)

Scope & Purpose
This PR fixes the behaviour where we want to crash the server on purpose and do not want core files creation.
This already works on macOS and Linux, but needs this subtle change to also work on Windows operating systems.

[x] :hankey: Bugfix

Related Information

[x] GitHub issue / Jira ticket: https://arangodb.atlassian.net/browse/BTS-1747

NOTE: arangodb-matrix-pr failed overall - but the expected single-server fix worked.