[Snyk] Security upgrade com.amazonaws:aws-java-sdk from 1.12.290 to 1.12.720

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

api/pacman-api-admin/pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity
Reachability

475/1000  Why? Has a fix available, CVSS 6.5
Denial of Service (DoS) SNYK-JAVA-IONETTY-1020439
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

460/1000  Why? Has a fix available, CVSS 6.2
Information Disclosure SNYK-JAVA-IONETTY-1070799
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

460/1000  Why? Has a fix available, CVSS 6.2
Information Disclosure SNYK-JAVA-IONETTY-1082235
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

460/1000  Why? Has a fix available, CVSS 6.2
Information Disclosure SNYK-JAVA-IONETTY-1082236
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

595/1000  Why? Mature exploit, Has a fix available, CVSS 5.9
HTTP Request Smuggling SNYK-JAVA-IONETTY-1317097
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
Mature
No Path Found

525/1000  Why? Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-JAVA-IONETTY-1584063
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

525/1000  Why? Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-JAVA-IONETTY-1584064
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

475/1000  Why? Has a fix available, CVSS 6.5
HTTP Request Smuggling SNYK-JAVA-IONETTY-2314893
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

550/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 6.5
HTTP Request Smuggling SNYK-JAVA-IONETTY-469234
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
Proof of Concept
No Path Found

600/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 7.5
HTTP Request Smuggling SNYK-JAVA-IONETTY-543490
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
Proof of Concept
No Path Found

600/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 7.5
HTTP Request Smuggling SNYK-JAVA-IONETTY-543669
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
Proof of Concept
No Path Found

560/1000  Why? Has a fix available, CVSS 8.2
Uncontrolled Memory Allocation SNYK-JAVA-IONETTY-564897
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

475/1000  Why? Has a fix available, CVSS 6.5
Denial of Service (DoS) SNYK-JAVA-IONETTY-5725787
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
No Known Exploit
No Path Found

600/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 7.5
HTTP Request Smuggling SNYK-JAVA-IONETTY-6056419
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
Proof of Concept
No Path Found

490/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Allocation of Resources Without Limits or Throttling SNYK-JAVA-IONETTY-6483812
com.amazonaws:aws-java-sdk:  1.12.290 -> 1.12.720
No
Proof of Concept
No Path Found

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)
🦉 Information Disclosure
🦉 Allocation of Resources Without Limits or Throttling

Checkmarx One – Scan Summary & Details – d0205a96-db33-4d71-a838-469419e46b2c
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2023-34034
Maven-org.springframework.security:spring-security-web-5.0.7.RELEASE
Vulnerable Package

CVE-2023-34034
Maven-org.springframework.security:spring-security-config-5.0.7.RELEASE
Vulnerable Package

CVE-2023-52428
Maven-com.nimbusds:nimbus-jose-jwt-9.8.1
Vulnerable Package

CVE-2023-6481
Maven-ch.qos.logback:logback-core-1.3.5
Vulnerable Package

CVE-2023-6481
Maven-ch.qos.logback:logback-core-1.2.9
Vulnerable Package

CVE-2023-6481
Maven-ch.qos.logback:logback-core-1.2.3
Vulnerable Package

CVE-2023-6481
Maven-ch.qos.logback:logback-core-1.2.11
Vulnerable Package

CVE-2024-22243
Maven-org.springframework:spring-web-5.0.8.RELEASE
Vulnerable Package

CVE-2024-22257
Maven-org.springframework.security:spring-security-core-5.0.7.RELEASE
Vulnerable Package

CVE-2024-22259
Maven-org.springframework:spring-web-5.0.8.RELEASE
Vulnerable Package

CVE-2024-22262
Maven-org.springframework:spring-web-5.0.8.RELEASE
Vulnerable Package

CVE-2024-23672
Maven-org.apache.tomcat.embed:tomcat-embed-core-8.5.32
Vulnerable Package

CVE-2024-23672
Maven-org.apache.tomcat.embed:tomcat-embed-websocket-8.5.32
Vulnerable Package

CVE-2024-24549
Maven-org.apache.tomcat.embed:tomcat-embed-core-8.5.32
Vulnerable Package

CVE-2024-29180
Npm-webpack-dev-middleware-5.3.0
Vulnerable Package

CVE-2024-31573
Maven-org.xmlunit:xmlunit-core-2.5.1
Vulnerable Package

CVE-2024-31573
Maven-org.xmlunit:xmlunit-core-2.9.0
Vulnerable Package

CVE-2024-33883
Npm-ejs-2.7.4
Vulnerable Package

CVE-2024-4067
Npm-micromatch-4.0.5
Vulnerable Package

CVE-2024-4068
Npm-braces-3.0.2
Vulnerable Package

Missing User Instruction
/dockerfile: 1
A user should be specified in the dockerfile, otherwise the image will run as root

Missing User Instruction
/Dockerfile: 2
A user should be specified in the dockerfile, otherwise the image will run as root

Missing User Instruction
/dockerfile: 1
A user should be specified in the dockerfile, otherwise the image will run as root

Missing User Instruction
/dockerfile: 1
A user should be specified in the dockerfile, otherwise the image will run as root

Passwords And Secrets - Generic Password
/mysql-check.yaml: 10
Query to find passwords and secrets in infrastructure code.

Add Instead of Copy
/Dockerfile: 10
Using ADD to load external installation scripts could lead to an evil web server leveraging this and loading a malicious script.

Add Instead of Copy
/dockerfile: 10
Using ADD to load external installation scripts could lead to an evil web server leveraging this and loading a malicious script.

CVE-2023-33202
Maven-org.bouncycastle:bcprov-jdk15on-1.67
Vulnerable Package

CVE-2023-33202
Maven-org.bouncycastle:bcprov-jdk15on-1.56
Vulnerable Package

CVE-2024-29025
Maven-io.netty:netty-codec-http-4.1.79.Final
Vulnerable Package

CVE-2024-29025
Maven-io.netty:netty-codec-http-4.1.77.Final
Vulnerable Package

CVE-2024-29025
Maven-io.netty:netty-codec-http-4.1.27.Final
Vulnerable Package

CVE-2024-29025
Maven-io.netty:netty-codec-http-4.1.32.Final
Vulnerable Package

Image Version Not Explicit
/Dockerfile: 2
Always tag the version of an image explicitly

Image Version Not Explicit
/dockerfile: 1
Always tag the version of an image explicitly

Pip install Keeping Cached Packages
/dockerfile: 3
When installing packages with pip, the '--no-cache-dir' flag should be set to make Docker images smaller

Pip install Keeping Cached Packages
/dockerfile: 10
When installing packages with pip, the '--no-cache-dir' flag should be set to make Docker images smaller

Pip install Keeping Cached Packages
/dockerfile: 4
When installing packages with pip, the '--no-cache-dir' flag should be set to make Docker images smaller

Pip install Keeping Cached Packages
/Dockerfile: 4
When installing packages with pip, the '--no-cache-dir' flag should be set to make Docker images smaller

RUN Instruction Using 'cd' Instead of WORKDIR
/dockerfile: 4
When using RUN command 'cd' should only be used for full path. For relative path make use of WORKDIR command instead.

Unpinned Actions Full Length Commit SHA
/codeQl.yaml: 50
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/codeQl.yaml: 84
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/codeQl.yaml: 65
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Package Version in Apk Add
/dockerfile: 1
Package version pinning reduces the range of versions that can be installed, reducing the chances of failure due to unanticipated changes

Unpinned Package Version in Apk Add
/dockerfile: 10
Package version pinning reduces the range of versions that can be installed, reducing the chances of failure due to unanticipated changes

Unpinned Package Version in Apk Add
/Dockerfile: 4
Package version pinning reduces the range of versions that can be installed, reducing the chances of failure due to unanticipated changes

Unpinned Package Version in Apk Add
/dockerfile: 3
Package version pinning reduces the range of versions that can be installed, reducing the chances of failure due to unanticipated changes

Unpinned Package Version in Pip Install
/Dockerfile: 4
Package version pinning reduces the range of versions that can be installed, reducing the chances of failure due to unanticipated changes

Unpinned Package Version in Pip Install
/dockerfile: 3
Package version pinning reduces the range of versions that can be installed, reducing the chances of failure due to unanticipated changes

Unpinned Package Version in Pip Install
/dockerfile: 4
Package version pinning reduces the range of versions that can be installed, reducing the chances of failure due to unanticipated changes

Unpinned Package Version in Pip Install
/dockerfile: 10
Package version pinning reduces the range of versions that can be installed, reducing the chances of failure due to unanticipated changes

Healthcheck Instruction Missing
/dockerfile: 1
Ensure that HEALTHCHECK is being used. The HEALTHCHECK instruction tells Docker how to test a container to check that it is still working

Healthcheck Instruction Missing
/dockerfile: 1
Ensure that HEALTHCHECK is being used. The HEALTHCHECK instruction tells Docker how to test a container to check that it is still working

Healthcheck Instruction Missing
/Dockerfile: 2
Ensure that HEALTHCHECK is being used. The HEALTHCHECK instruction tells Docker how to test a container to check that it is still working

Healthcheck Instruction Missing
/dockerfile: 1
Ensure that HEALTHCHECK is being used. The HEALTHCHECK instruction tells Docker how to test a container to check that it is still working

IAM Access Analyzer Not Enabled
/CloudFormation.yaml: 2
IAM Access Analyzer should be enabled and configured to continuously monitor resource permissions

MAINTAINER Instruction Being Used
/Dockerfile: 3
The MAINTAINER instruction sets the Author field of the generated images. The LABEL instruction is a much more flexible version of this and you sho...

MAINTAINER Instruction Being Used
/dockerfile: 2
The MAINTAINER instruction sets the Author field of the generated images. The LABEL instruction is a much more flexible version of this and you sho...

Multiple RUN, ADD, COPY, Instructions Listed
/dockerfile: 144
Multiple commands (RUN, COPY, ADD) should be grouped in order to reduce the number of layers.

Fixed Issues

Severity
Issue
Source File / Package

CVE-2016-7051
Maven-com.fasterxml.jackson.core:jackson-core-2.6.0

CVE-2018-3827
Maven-org.elasticsearch:elasticsearch-5.6.10

CVE-2018-3831
Maven-org.elasticsearch:elasticsearch-5.6.10

CVE-2021-44228
Maven-org.apache.logging.log4j:log4j-core-2.10.0

CVE-2021-46877
Maven-com.fasterxml.jackson.core:jackson-databind-2.10.5

CVE-2022-38900
Npm-decode-uri-component-0.2.2

Cxab55612e-3a56
Npm-braces-3.0.2

Cxca84a1c2-1f12
Npm-micromatch-4.0.5

Cxced0c06c-935c
Maven-com.fasterxml.jackson.core:jackson-databind-2.10.5

CVE-2020-9488
Maven-org.apache.logging.log4j:log4j-core-2.10.0


Canopy tutorials

Purpose

Add canopy standalone and coupled hydraulics tutorials to ClimaLSM/docs. See issues #240, #255
To-do

Add links to dynamics docs at locations marked with TODO once dynamic docs are live.

Content

Two usage tutorials included in docs/tutorials/Canopy

[x] I have read and checked the items on the review checklist.

@Espeer5 I will review today!
Just a note: before merging a pull request, after addressing review comments, we squash and rebase (to only merge a single commit with all the changes; it's a bit cleaner that way). I usually follow these directions: https://github.com/openedx/edx-platform/wiki/How-to-Rebase-a-Pull-Request.

@Espeer5 I will review today!
Just a note: before merging a pull request, after addressing review comments, we squash and rebase (to only merge a single commit with all the changes; it's a bit cleaner that way). I usually follow these directions: https://github.com/openedx/edx-platform/wiki/How-to-Rebase-a-Pull-Request.

I can help you with this on Monday if you want - I usually make a backup branch before doing it just in case (rebasing changes the commit history, so there is the potential for lost code if something goes wrong).

@Espeer5 I will review today!
Just a note: before merging a pull request, after addressing review comments, we squash and rebase (to only merge a single commit with all the changes; it's a bit cleaner that way). I usually follow these directions: https://github.com/openedx/edx-platform/wiki/How-to-Rebase-a-Pull-Request.

I can help you with this on Monday if you want - I usually make a backup branch before doing it just in case (rebasing changes the commit history, so there is the potential for lost code if something goes wrong).

That's alright, I went ahead and did it this morning. I also have implemented the changes you mentioned in your comments.

bors r+

@Espeer5 not sure why buildkite failed in the bors run - it passed in github actions, and your changes didnt change the experiments folder at all. let's see if it passes this time..


Google Play link is broken + Releases

Google Play link is broken.
Should be nice to have ready to download releases here on GitHub to avoid the Play Store; this works nice with Obtanium too.

apk
would be nice if you can report if it works, dumped it from my main phone to install it on second one, but it force closes when trying to add an exchange. would be good to know before i try to flash another rom ;)

yeah i expected an answer like this, sorry that i signed up to provide the url. friendly hint: check the md5 if youre paranoid. no one forces you to download it, i just saw that multiple people asked for it & nobody provided the file. cheers.

Thank you for the kindness, but offering an anonymous link is not the proper/educative solution; if an user want to risk with a random apk he can find it online on several (quite shady) websites.
Furthermore, it is not related to the issue that needs to be addressed differently, of course.

check the md5

Which one? I cannot find it in the official release.

if youre paranoid

Paranoid? Are you kidding? This is the basic rule.

I cant upload it on github directly afaik, i provided a dumped apk of the original latest version that was available in play store.
The developer of this app doesnt answer anything for the last like 4 months.
I dont have any md5, i searched for it but i could only find the "old" bitcoin checker versions online.
Just wanted to give people a chance, cause this app is the only one I know with such decent price alerts.
I can reupload it, didnt take it down - i dont know any viable filehosters anymore. Of course its risky to install apks found online, but yeah - I just wanted to give an "option" for people that are willing to "risk" it, because there is no alternative :)

https://filebin.net/5lymx4o87alb7irf

check the md5 if youre paranoid.

I dont have any md5

For me this is one big red flag; when someone tries to push a false security when it is actually not possible to achieve it.
By the way, I'm happy if you are in good faith; otherwise, the issue has been flagged and every user will choose what to do :)


Allow the user to choose what type of peak finding to use for clusters

This will need to be added as an argument to pass in the declaration of GalaxyCluster and ClusterSample, let them choose between hierarchical clustering peak finding and simple peak finding - partly to avoid all this crap with issue #626.

This is done now.


AWS EKS infra: transition to k8s 1.24 and add required storage addon

This adds the required changes to the eksctl cluster configuration files, as generated from a template.jsonnet file (depending on a libsonnet/nodegroup.jsonnet file).
Questions

Should we add iam.withAddonPolicies.ebs=true to the default node group object so all node groups get support for mounting PVCs using EBS storage, or should we just add it to the core node-group that is known to need it?

Closes #2054, but work to migrate existing hubs remain, this is covered by #2057.

Thanks for reviewing @yuvipanda! I just updated inline comments and force-pushed, and updated the PR description to include a question:

Should we add iam.withAddonPolicies.ebs=true to the default node group object so all node groups get support for mounting PVCs using EBS storage, or should we just add it to the core node-group that is known to need it?

@consideRatio the question to me is - where does the actual controller run? does it run just on the core nodes (as does the autoscaler, for example?) - in which case just letting it be there for the core nodes is the way to go. Or, is it a daemonset that needs to run on every node? In which case we should still probably just set it up for the core nodes (least principle and all that) but document that it needs to be explicitly enabled.
My intuition is that the actual controller is a deployment / statefulset that just runs on the core nodes and so having the permissions be just on the core nodes is fine.

However this intuition has no evidence :D Should be easy enough to figure out though.

@consideRatio I think the controller creates PVs, but the ds is what 'binds' them. I would suggest trying to launch a pod that has a PVC that will run on a non-core-nodepool to test this. I think the controller will always run in the core nodepool, the question is, what permissions does the daemonset need to do the attach? So we want to test 'will EBS volumes mount on arbitrary nodes even if they don't have this permission explicitly defined?'

Thanks for your help thinking about this @yuvipanda!!!
Test: to add a new PVC, and a new Pod to mount it, and force the pod to run on the user node without considered iam ebs stuff.
Test situation 1: Run test as defined above
Test situation 2: Run test as defined above, but let the ebs controller run on the user node
Both worked out. Due to this, I figure we go with removing the nodeGroup's addition of iam for ebs stuff entirely and see if it works, if not, we add it. This would be a win for us, as its easier to just add the addon to the EKS based k8s cluster than to node pools with delete/create operations.
In 8fe4009 I remove the nodeGroup's iam ebs stuff and this is ready to merge. In case things won't work, lets add it back to the core node pool.

@consideRatio sounds good to me.

Will do! Thanks @consideRatio!


Sort css properties

Properties should be sorted in order.
SassGuidelines: http://sass-guidelin.es/#declaration-sorting
Google HTML/CSS Style Guide: https://google-styleguide.googlecode.com/svn/trunk/htmlcssguide.xml?showone=Declaration_Order#Declaration_Order

Wow - that's a lot of changes! Thanks!
Shoot us your mailing address and shirt size so we can send you a sweet Yeti shirt!
foundation at zurb .com

@rafibomb Wow, thank you so much. I've mailed you my address.
Feel free to contact me at any time, if there is anything I can do for you :)

@rafibomb Have you received my email with my address? Is everything fine or do you need any more information?


Add READMEs to infra and workflow dirs

Adds READMEs with light descriptions to workflows/ and infrastructure/ directories.
Help people quickly see what's up.

[ ] closes #xxxx
[ ] tests added / passed
[ ] docs reflect changes
[ ] passes flake8 downscale tests docs
[ ] entry in HISTORY.rst

[summarize your pull request here]

this is wonderful 🤩

what @delgadom said 🙂 thanks @brews for adding this documentation!!


Fully support placement rules in app recovery

Applications that are recovered should be treated the same way as newly submitted applications. Currently the replay uses the original queue from the submission without applying the placement rules or ACLs.

fixed as part of the changes that were made for #35


Multi-image inference

Thanks for your great work! LLaMA-VID supports single-image input and video input, but does it support multi-image input? What's the quickest way to adapt to this input?
Thanks in advance!

In current version, we do not support multi-image input. But you can support it by using multi-image instruction data like MIMIC-IT for instruction tuning.


Conversation stays open forever

There seems to be a bug. I can't find timeLimit to be set anywhere for a task.
https://github.com/howdyai/botkit/blob/940c3e004f3b3551e83ce102d1bec9f016c4bf32/lib/CoreBot.js#L381
I have added the following on line 380:
console.log("duration: " + duration + " la: " + lastActive + " tl:" + this.task.timeLimit);
result:
duration: 2612 la: 1003 tl:undefined
In my scenario there is a conversation going on and if the bot asks a question and there is no response in a timely fashion I expect the conversation to end. However, it can stay open for hours waiting for the reply.

Wow, just a closed with not even a comment?

@MiroValchev sorry about that, Was doing some housekeeping and closed by mistake. Have you tried asking this question on slack4dev?

In order to end the conversation you should call convo.stop()
You should look at the Conversation Control Functions @ https://github.com/howdyai/botkit#conversation-control-functions
Let me know, if the problem exists, I will reopen the issue.

@anonrig lets say you have asked a question and expect the user to reply. If that doesn't happen in a timely manner, the conversation should be closed. Pretty sure that's the intent behind the link that I gave:
https://github.com/howdyai/botkit/blob/940c3e004f3b3551e83ce102d1bec9f016c4bf32/lib/CoreBot.js#L381
The only trouble is that timeLimit is undefined.
Sounds like a bug to me, but might be wrong.


Add lookup functions to handle advanced use cases

Lookup all namespaces if enabled to add secrets to all namespaces
Lookup existing certs and use if certification rotation is disabled
Fixes #108

Does it make sense to mention then all-namespaces feature flag in the documentation?


support swift4.2

@phpmaple
Why did you change URLs? (krimpedance/ to phpmaple/)?
And, you shouldn't renew the version.

@phpmaple
It will be necessary for me to finally update it.
So, please send PR to the dev branch.

So when it will be available for updating?

Thanks.


Filenames are not visible

Description
Telescope buffer and Telescope-file-browser does not display filenames.
I can see icons, however there is no filename next to it.
--- see attached screenshots.
https://imgur.com/a/RSpPwn2
--- file browser shows icons but no filename along with it.
Neovim version
NVIM v0.9.0
Build type: Release
LuaJIT 2.1.0-beta3

Operating system and version
MacOS 13.3
Steps to reproduce
deafult configuration for Telescope.
Expected behavior
Filename along with icons in telescope-file-browser, and telescope buffers.
Actual behavior
Icons along with filename/buffer title displayed while using default config for telescope.
Minimal config
require("telescope").setup {
  picker = {
    theme = "ivy"
  },

  extensions = {
    file_browser = {
        theme = "ivy",
        -- disables netrw and use telescope-file-browser in its place
        hijack_netrw = true,
--        cwd_to_path = true,
--        grouped = true,
--        files = true,
       auto_depth = 2,
        git_status = true,
        mappings = {
          ["i"] = {
            -- your custom insert mode mappings
          },
          ["n"] = {
            -- your custom normal mode mappings
          },
        },
      },
  },
}
require("telescope").load_extension "file_browser"

---- installed plugins

require('packer').startup(function(use)
--   [[ Plugins Go Here ]]

    use 'tjdevries/colorbuddy.nvim'
    use 'bkegley/gloombuddy'
    use 'neovim/nvim-lspconfig'
    use 'anott03/nvim-lspinstall'

    -- nvim cmp related plugins
    use 'hrsh7th/cmp-nvim-lsp'
    use 'hrsh7th/cmp-buffer'
    use 'hrsh7th/cmp-path'
    use 'hrsh7th/cmp-cmdline'
    use 'hrsh7th/nvim-cmp'

    -- snip config
    use 'hrsh7th/cmp-vsnip'
    use 'hrsh7th/vim-vsnip'
    use 'hrsh7th/vim-vsnip-integ'
    -- sneaking some formatting in here too
    use {'prettier/vim-prettier', run = 'yarn install' }
    use 'nvim-treesitter/nvim-treesitter'
    use 'nvim-lua/plenary.nvim'
    use {
      'nvim-telescope/telescope.nvim', tag = '0.1.1',
    -- or                            , branch = '0.1.x',
      requires = { {'nvim-lua/plenary.nvim'} }
    }
      use 'BurntSushi/ripgrep'
    use 'natecraddock/nvim-find'
    use 'nvim-tree/nvim-web-devicons'
    use {
        "nvim-telescope/telescope-file-browser.nvim",
        requires = { "nvim-telescope/telescope.nvim", "nvim-lua/plenary.nvim" }
    }
    use 'sheerun/vim-polyglot'
    use 'nvim-lua/popup.nvim'
    use 'jremmen/vim-ripgrep'
    -- [[ Theme ]]
end)

Are you on telescope master? I think there was a temporary bug. If you are, try updating telescope.

I am on telescope 0.1.1 tagged version, not sure this is the latest version.

Strange, your issue seems identical to https://github.com/nvim-telescope/telescope-file-browser.nvim/issues/272
https://github.com/nvim-telescope/telescope.nvim/issues/2528
I tried to replicate this using your telescope config and using the same tagged version but I was unable to. When did you start experiencing this issue. the 0.1.1 release hasn't been changed since mid-January.

Hey,  Yes my issue is identical to this, I am new to nvim. Thanks for pointing to the original issue, it looks like this issue is with telescope and not the file browser extension. Closing this issue here, will followup on Telescope repo for the bug.


Intellisense, etc. no longer working for version 2.34.0+

VS Code 1.15.1
Windows 10 Pro
Ionide-fsharp 2.34.0-4
I'm no longer seeing intellisense/function signatures/error underlining as of version 2.34.0  I have confirmed that everything works fine after downgrading to 2.33.2.
From language service output:
Data={"FileName":"c:\\code\\src\\foo\\bar.fs","Line":68,"Column":15,"Filter":""}
[12:49:10 DEBUG] RES (007) <- {tooltip} in 4 ms: Kind={"error"}
Data={"Code":1,"Message":"File 'c:\\code\\src\\foo\\bar.fs' not parsed","AdditionalData":{}}
[12:49:10 ERROR] Cannot execute tooltip, File 'c:\code\src\foo\bar.fs' not parsed
[12:49:33 DEBUG] REQ (008) -> {project}, File = "~\src\foo\bar.fsproj"
Data={"FileName":"c:\\code\\src\\foo\\bar.fsproj"}
[12:49:33 ERROR] RES (008) <- {project} ERROR in 15 ms: Error: Request failed with status code 500 Data=undefined
[12:49:55 DEBUG] REQ (009) -> {project}, File = "~\src\foo\bar.fsproj"
Data={"FileName":"c:\\code\\src\\foo\\bar.fsproj"}
[12:49:55 ERROR] RES (009) <- {project} ERROR in 4 ms: Error: Request failed with status code 500 Data=undefined

Duplicate of #514


Running on Windows Docker vs WSL versions

Hi
Not really an issue, unless you include "more explanation" as an issue. But you guys don't have a discussion section here on Github.
Is there any benefit (or con) to running in WSL vs Docker for Windows? (as still no sign of a Win version coming).
I am leaning towards WSL simply because I've had issues in the past trying to get non-docker LLM apps to communicate with docker apps and vice versa.
Docker seems simple, but the instructions aren't specific to windows, are they? Otherwise wouldn't the Docker version count as this app being available for windows (which the main page still says is coming soon)?
Will it be any slower or faster in docker?
I have also heard via WSL will use less VRAM.
Where do models get downloaded to, if we're running in either? Can we point the docker version or the WSL  version to a common repo of LLM models on our drive locally?
Many other LLM apps "require" Ollama as their backend, so I really hope to start using this soon.
I have both docker desktop and WSL/Ubuntu installed already.
If I have another LLM app, say, Cheshire Cat AI, already running in docker, maybe I would be better off running the dockerised Ollama. But then other LLM apps that do NOT run in docker, also want it. Not sure what option is going to give me the more simple setup in the long run.
Thanks!

Hi there.
I'm on Win11, wsl2, docker.
I've been using a lot wsl2, doing things straight inside it
It worked for a while, but with time, it got pretty ugly.
I liked to try every AI project and each had own version requirements for some common package.
When I was updating one, often an upgrade was done, which, in turn, blow the others. And so on.
I started to make intensive use of miniconda (TGWUI came with it by default), but still had minor issues.
Then I started to use Docker.
And besides other unforseen problems which poped out but weree workable (increase host ram allocated to docker, swap space, network accessibility between containers, common place to store LLMs, etc),  I am declaring now happy.
No more hustle.
An I'm wondering why others not use it 😉

And actually host ram, swap space, are directly related to wsl2, not to docker.

@EmmaWebGH When starting with WSL2 on laptops/workstations, users typically love the userfriendly use of Ubuntu. When it comes to a specific distro, which is not listed in wsl --list --online, there is an import function to make use of a dockerized distro. You can use Windows Docker Desktop for that as well. Nvidia GPU drivers support WSL2.
There are configuration settings for ram and cpu for the underlying linux kernel in WSL2, see https://learn.microsoft.com/en-us/windows/wsl/wsl-config.
To strictly separate research environments, I recommend VMware by Broadcom Workstation because it's userfriendly, easy to maintain different configurations without thinking about guest os and its components, windows patches, drivers, etc. You can do snapshots during testings, save the virtual machines on portable media, etc. One actual limitation are the defaults to present an NVidia GPU to a guest os, hence a linux guest os with installed Ollama will only use cpu power.
In theory you can use probably every Linux distro on WSL2 to use Ollama, but there is scripting know-how needed. I've assembled all the necessary steps e.g. for VMware By Broadcom Photon OS, see https://github.com/vmware/photon/issues/1532 and https://communities.vmware.com/t5/Photon-OS-Discussions/Ollama-ai-on-Photon-OS-on-WSL2/m-p/2997394.

Thanks for this @dcasota
For me, pretty much the ONLY reason to use WSL is that Docker is not yet windows-friendly, so I'm not too worried about separate linux environments. I actually doubt I'll be using WSL/Ubuntu for anything else.
For all the other stuff I do, I mainly use conda environments, and occasionally Docker on windows, to keep things separate.
I got Ollama running yesterday via WSL, so this looks OK so far.
But I'm still  hazy on where to put models or if we can point Ollama to a folder of already-downloaded models on a local drive somewhere. Every LLM seems to want their models in their own special location and there's a ton of duplication going on right now with my model files! :)

Copied datasets are the root cause. Thirty years old text snippets in ten years old free libraries in actual LLMs.

The root cause is every install of every LLM app doesn't have an easy way to direct itself to a folder specified by the user... ?
Anyway we're off topic now
I suppose I'll go search for a clear answer on where the models are downloaded to and if/how we can direct Ollama to look in a folder of our choosing.

see https://github.com/jmorganca/ollama/issues/1836 about vsan. Deduplication is a datacenter storage server functionality. Computers, laptops, workstations typically are missing this feature.
In addition, using Ollama in docker layering, this is a maintenance technique question. Unfortunately optimal docker image size maintenance is not a built-in automatic docker task. My reference learning container images is Cora Ciberkleid's advice in https://tanzu.vmware.com/developer/tv/tanzu-tuesdays/0082/.

What are you even talking about? Are you a troll? You're speaking words that have nothing to do with the intent of my original question.

3. "if/how we can direct Ollama to look in a folder of our choosing"
I would call this feature as distributed storage solution. It is a well-known feature in data centre environments.

Datacenter? Where did anyone mention data centre.
 A folder with .safetensor models in it, for example. Turns out we can't do it, I've learned elsewhere, no thanks to these confusing replies.
Maybe english isn't your language, I could understand miscommunication then.

Ultimately it's up to the users preference.  We now have a native windows version, but still support WSL2 if that's what users want to run.  They come with their own trade-offs, but if you're on Windows, I'd suggest the native windows version as the preference.

I installed ollama on the windows version, how can I have wsl point to it?

I installed ollama on the windows version, how can I have wsl point to it?
@piranhap
You don't, you either use the WSL version of Ollama or the Windows version of Ollama. One doesn't point to the other.

@piranhap WSL2 has its own network identity, so "localhost" is different from the host windows "localhost".  By default we only expose Ollama to localhost (<IP_ADDRESS>:11434) but you can expose it on other addresses via the OLLAMA_HOST variable. (Be careful not to expose on an open/untrusted network)  Check out the https://github.com/ollama/ollama/blob/e72c567cfd18a6e48de0acaeda60896af4bff3fd/docs/faq.md for more information.


Lables on multiple lines instead of twisting

Please consider the following before submitting a pull request:
Guidelines for contributing: https://github.com/chartjs/Chart.js/blob/master/CONTRIBUTING.md
Example of changes on an interactive website such as the following:

http://jsbin.com/
http://jsfiddle.net/
http://codepen.io/pen/
Premade template: http://codepen.io/pen?template=JXVYzq
when there is no rotation and the labels are too long, they do not show. In this case, make the long label a multi-lines label.

this is already possible in the latest versions by using an array of strings for each label

This is more for dynamic labels. In my case, i got data from the db and cannot format them before.

Closing since splitting the labels should be done outside the chart for now


Atom not detecting this package

I have installed atom and i was writing some html tags but it was not completing them automatically.

I have installed the auto complete html tag package.

You have to press tab at the end of the name of the tag. For example you typed div and pressed tab, it will create the proper tags. Or div.container and press tab, this should be created -

@trilokborah26 did the trick i explained work?

I already knew that and i tried multiple times...It did not work...now i am
visual studio code...and it is better than ATOM.
On Sat, 19 Oct 2019 at 09:30, Md. Abdul Munim Dibosh <
<EMAIL_ADDRESS>wrote:

@trilokborah26 https://github.com/trilokborah26 did the trick i
explained work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dibosh/html-tag-auto-complete/issues/4?email_source=notifications&email_token=AMQZ4BTHMI3JFO4ESSOFKC3QPKA4RA5CNFSM4I6AJHN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBXBCPI#issuecomment-544084285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMQZ4BV5ZXAE2HLJVB5UZSDQPKA4RANCNFSM4I6AJHNQ
.


Ignore constructor args if proxy without contructor

Hello,
I have a contract without constructor that uses a proxy. When defining args of the proxy method, i get this error
Error: ERROR processing /var/www/thesandboxgame/sandbox-smart-contracts/deploy/06_asset/00_deploy_asset.ts:
Error: The number of arguments passed to not match the number of argument in the implementation constructor.
Please specify the correct number of arguments as part of the deploy options: "args"

It checks that if there is no contructor, it should not have args defined. I believe it should not fail, it's possible to have a proxied contract without contructor.
The documentation stipulates also
It is also possible to then have a constructor with the same arguments and have the proxy be disabled

My deployment
  await deploy('Asset', {
    from: deployer,
    contract: 'Asset',
    args: [sandContract.address, deployer, deployer],
    proxy: {
      owner: upgradeAdmin,
      proxyContract: 'OpenZeppelinTransparentProxy',
      methodName: 'init',
      upgradeIndex: 0,
    },
    log: true,
  });

I had no way to test my fix, don't take it for granted

this PR won't change that, options.proxy is always truthful as this part of the code is executed in the proxy path
But anyway note that you can do the following

 await deploy('Asset', {
    from: deployer,
    contract: 'Asset',
    proxy: {
      owner: upgradeAdmin,
      proxyContract: 'OpenZeppelinTransparentProxy',
      execute {
         methodName: 'init',
         args: [sandContract.address, deployer, deployer],
      },
      upgradeIndex: 0,
    },
    log: true,
  });

Perfect thank you !


Adjust cursor for method with arguments

@kasperpeulen, I have a PR out to this PR :)

lgtm, thanks!


Introduce AnatomicalCoordinatesTable

From the introductory comment on #1.

@bendichter @stephprince and I are also proposing a separate object (CoordinatesTable?) to store the estimated localization and confirmed localization of electrodes in some reference space and atlas. It would have coordinates, brain area, method, and reference space information. One object for estimated, one object for confirmed using method 1, one object for confirmed using method 2 (in case there are two). Each CoordinatesTable would point to the ElectrodesTable. (Under the model of https://github.com/catalystneuro/ndx-extracellular-channels/pull/1, each CoordinatesTable would have had to point to both ChannelsTable objects above or be duplicated, which are not ideal.)

I think this is a great idea for the reasons defended on #1 here

I don't really like having the estimated and confirmed positions as canonical attributes in the channels table. I think that the proposal on https://github.com/catalystneuro/ndx-extracellular-channels/issues/3 of relegating this to an AnatomicalCoordinatesTable is better for the following reasons. First, separation of concerns between the description of the ecephys setup and experiments of localization. I think is nice to decouple those two things so they can evolve on their own. If the localization experiments improve we then only need the change to AnatomicalCoordinatesTable or link into a better extension without making parts of our schema obsolete. Second, in the case of SpikeGLX where two channels map to one contact we will end up with redundant information. Plus, even within the current schema I don't like the canonization of estimated_{} as a set of single columns. I think that confirmed_{x} is a great name but what if there is more than one estimate? Right now those are optional attributes and maybe something like this is needed urgently in which case is fine but I still wanted to point out what I think are flaws of this design.

So I think that the ChannelsTable can be linked per-row to a new AnatomicalCoordinatesTable and that would decouple and solve some of the problems that we have been discussing.
One thing with this approach though is that the mapping between an anatomical site (a row of the AnatomicalCoordinatesTable) and a ProbeInsertion is not explicit but indirect in the current schema.

Important discussion how does this fit into the considerations of #4 .

yes but the anatomomical coordinates table is in a separate repo here: https://github.com/bendichter/ndx-anatonical-localization

I think is even better if a different extension takes responability of specifying surgical experiments.
Reading the schema it seems that the current estimated_position_{x}_in_mm would be a single column maping to an AnatomicalCoordinatesTable which is a dynamic table so that makes sense.
I also think that another good reason to decouple localization from here is that this extension can be used to store in-vitro experiments and the current estimated_position_{x}_in_mm do not really make sense for those cases.


Possible to set custom FPS of 10?

Required Info |
-- | --
Camera Model | { D415}
Firmware Version | {<IP_ADDRESS>}
Operating System & Version | {Android}
Platform | {Arm-Odroid}
SDK Version | { 2.30.0 }
Language | {C++/Java }
Segment | {others }
I tried running colour(1280x720x15fps) and depth stream(1280x720x15fps) together on android using Odroid N2 SBC and after around 62 minutes, the stream gives Frame didn't arrive error. The system throws error sooner when the color stream is set at 1920x1080x15fps resolution. If it helps can I set the FPS to lower value? I would prefer something around 10 rather than available option of 6, which I think is too less for my use case.
Also, can I reset the camera if it starts giving me error and re-run with same config? I am not able to hardware reset it properly. Currently only reboot seems to allow the rerunning of camera with same config.
I have posted the code and adb logs below.
The issue looks related to https://github.com/IntelRealSense/librealsense/issues/4091
I truly need some answers to make the stream stable. My last question did not grant a single answer. That's why I am opening a new issue. Desperate times, desperate measures.
CODE:
private void stream() throws Exception {

        mPipeline = new Pipeline();

        Config config = new Config();
        try(DeviceList devices = mRsContext.queryDevices()) {
            if (devices.getDeviceCount() == 0) {
                return;
            }

            try (Device device = devices.createDevice(0)) {
                if(device == null){
                    Log.e(TAG, "failed to create device");
                    return;
                }

                if (!device.isInAdvancedMode()){
                    device.toggleAdvancedMode(true);
                    if (CommonConstants.getInstance().DEBUG) {
                        Log.d(TAG, "Device not in advance mode. Set to advance mode now!");
                    }
                } else {
                    if (CommonConstants.getInstance().DEBUG) {
                        Log.d(TAG, "Device already in advance mode");
                    }
                }

                try {
                    byte[] presetData = getByteArrayFromResource(FacialProcessingApp.getContext(), R.raw.high_res_high_density_preset, "high_res_high_density_preset.json");
                    device.loadPresetFromJson(presetData);
                    Log.d(TAG, "Preset Set!");
                } catch (IOException e) {
                    Log.e(TAG, "failed to set preset, failed to open preset file, error: " + e.getMessage());
                } catch (Exception e) {
                    Log.e(TAG, "failed to set preset, error: " + e.getMessage());
                }

                config.disableAllStreams();
                config.enableStream(StreamType.COLOR, 0, CommonConstants.getInstance().colourStreamWidth, CommonConstants.getInstance().colourStreamHeight, StreamFormat.YUYV, CommonConstants.getInstance().depthCamFPS);
                config.enableStream(StreamType.DEPTH,0,CommonConstants.getInstance().depthStreamWidth,CommonConstants.getInstance().depthStreamHeight,StreamFormat.Z16,CommonConstants.getInstance().depthCamFPS);
            }
        }

        mPipeline.start(config);

        int frameCounter  = 0;
        while (!mStreamingThread.isInterrupted())
        {
            if (depthCameraConnected) {
                try {
                    try (FrameReleaser fr = new FrameReleaser()) {
                        FrameSet frames = mPipeline.waitForFrames(DEFAULT_TIMEOUT).releaseWith(fr);
                        mIsStreaming = true;
                        frameCounter++;
                        if (frameCounter <= 30) {
                            continue;
                        } else {
                            frameCounter = 100;
                        }

                        FrameSet processedSet = frames.applyFilter(mDecimationFilter).releaseWith(fr)
                                .applyFilter(mTemporalFilter).releaseWith(fr)
                                .applyFilter(mHoleFillingFilter).releaseWith(fr);

                        Frame depth = processedSet.first(StreamType.DEPTH).releaseWith(fr);
                        DepthFrame depthFrame = depth.as(Extension.DEPTH_FRAME);
                        depthFrame.releaseWith(fr);

                        Frame colour = processedSet.first(StreamType.COLOR).releaseWith(fr);
                        VideoFrame colourFrame = colour.as(Extension.VIDEO_FRAME);
                        colourFrame.releaseWith(fr);

                        LibraryLoader.getInstance().getPeopleCountingInterface().SetDepthCamValues(depthFrame,colourFrame);

                        synchronized (Streamer.this) {
                            mCameraCombinedFrameChain[mChainIdx] = new CameraCombinedFrame(colourFrame, depthFrame);
                            mCameraFrameReady = true;
                            Streamer.this.notify();
                        }

                    }
                } catch (Exception e) {
                    Log.e(TAG, "streaming, error: " + e.getMessage());
                    e.printStackTrace();
                }
            }
        }

        mPipeline.stop();
        mPipeline.close();
        mPipeline = null;
    }

ADB LOGS (When the error appeared):
2019-11-27 07:24:25.503 26820-26974/com.test.app W/librs: usb_request_queue returned error, endpoint: 132 error: Out of memory, number: 12
2019-11-27 07:24:25.503 26820-26974/com.test.app E/librs: failed to submit UVC request, error: -11
2019-11-27 07:24:25.546 26820-26971/com.test.app W/librs: usb_request_queue returned error, endpoint: 130 error: Out of memory, number: 12
2019-11-27 07:24:25.546 26820-26971/com.test.app E/librs: failed to submit UVC request, error: -11
2019-11-27 07:24:25.557 26820-26974/com.test.app W/librs: usb_request_queue returned error, endpoint: 132 error: Out of memory, number: 12
2019-11-27 07:24:25.557 26820-26974/com.test.app E/librs: failed to submit UVC request, error: -11
2019-11-27 07:24:26.179 26820-26834/com.test.app I/sense.retailap: Background concurrent copying GC freed 315(63KB) AllocSpace objects, 5(8MB) LOS objects, 49% free, 10MB/21MB, paused 2.424ms total 464.110ms
2019-11-27 07:24:26.433 26820-26974/com.test.app E/librs: uvc streamer watchdog triggered on endpoint: 132
2019-11-27 07:24:26.433 26820-26974/com.test.app I/librs: USB pipe 132 reset successfully
2019-11-27 07:24:46.406 3068-3161/? D/MDnsDS: MDnsSdListener::Monitor poll timed out
2019-11-27 07:24:46.406 3068-3161/? D/MDnsDS: Going to poll with pollCount 1
2019-11-27 07:24:54.488 26820-26966/com.test.app W/librs: control_transfer returned error, index: 768, error: Broken pipe, number: 32
2019-11-27 07:27:01.564 26820-26966/com.test.app W/librs: control_transfer returned error, index: 768, error: Broken pipe, number: 32
2019-11-27 07:27:01.628 26820-26966/com.test.app I/chatty: uid=10062(com.test.app) Thread-3 identical 1 line
2019-11-27 07:27:01.688 26820-26966/com.test.app W/librs: control_transfer returned error, index: 768, error: Broken pipe, number: 32
2019-11-27 07:29:26.546 26820-26971/com.test.app W/librs: usb_request_queue returned error, endpoint: 130 error: Out of memory, number: 12
2019-11-27 07:29:26.546 26820-26971/com.test.app E/librs: failed to submit UVC request, error: -11
2019-11-27 07:29:27.554 26820-26971/com.test.app E/librs: uvc streamer watchdog triggered on endpoint: 130
2019-11-27 07:29:27.554 26820-26971/com.test.app I/librs: USB pipe 130 reset successfully
2019-11-27 07:29:29.896 26820-26953/com.test.app E/MRApp_1.1 :Streamer: streaming, error: Frame didn't arrived within 3000

Though 10 FPS is not officially supported, a workaround for a custom FPS that has been tried is to run at 30 FP and only capture every nth frame in order to simulate the desired frame rate.  The link below details a user who used this method.
https://github.com/IntelRealSense/librealsense/issues/3169
A hardware reset can be performed automatically within scripting with the hardware_reset() instruction.
https://github.com/IntelRealSense/librealsense/issues/2161#issuecomment-408779150

@MartyG-RealSense @matkatz Can you confirm whether the hardware reset will solve this issue? My previous experience somehow made me believe that when this error is encountered then the particular configuration gets locked for further use, till a system reboot is made.

   If you find that a hardware reset does not work for you though, an alternative approach that can be tried is a USB reset.
https://github.com/IntelRealSense/librealsense/issues/4113#issuecomment-513649319

@MartyG-RealSense Currently I have written some code to perform hardware reset when it encounters "frame not received" error. I am testing with colour stream at 1920x1080. The thing during my first testing I noticed the frame gets stuck after few minutes. Basically each gives me same colour frame (haven't check the depth frame) again and again, without throwing any "frame not received" error. Is their any id that can be compared with the last received colour frame, to check whether it changes or not, so to that I can trigger hardware reset.

I researched your new question very carefully.  In regard to detecting a problem as a condition for reset, I wonder if you could use try_wait_for frames.  This returns a False value when timeout occurs instead of throwing an exception.
https://github.com/IntelRealSense/librealsense/issues/2422#issuecomment-423254709

@MartyG-RealSense Another question, currently I am using the D415 module, the one without case. In the original hardware(cased one), their is heatsink plate touching the back of the vision processor board on two chips: Vision D4 processor and ISP. ISP remains cool to touch and D4 heatsup after around an hour of operation, my guess around 60-65 degree celcius, I haven't checked the temperature through API, as currently there are none on android, maybe I can call it through the JNI. So the question is does it needs heatsink for long operation, I am intending to use it indoors 24x7 at ambient room temperature. Same query for the camera module, specially the region behind the laser projector.

The 400 Series cameras can run indefinitely as long as they remain within recommended temperature tolerances (35 degrees C maximum officially in the data sheet document, though in the real world you may get away with 42 degrees before problems manifest).  So it is important to draw heat away from the camera board.
The data sheet says "The Vision Processor D4 Board should be screw mounted on to a heat sink or a heat dissipating structure element using screw forks on Board.  Thermal conductive tape (electrically non-conductive) should cover the entire back side area (non-component side) of the ASIC Board for thermal transfer onto heat sink or heat dissipating structure element".
The data sheet can be obtained as a PDF from the link below.  Page 82 describes recommendations for designing a heat sink for the depth module.
https://dev.intelrealsense.com/docs/intel-realsense-d400-series-product-family-datasheet?elq_cid=1206596&erpm_id=1567221

I researched your new question very carefully. In regard to detecting a problem as a condition for reset, I wonder if you could use try_wait_for frames. This returns a False value when timeout occurs instead of throwing an exception.
#2422 (comment)

The android api does not have a method that exposes try_wait_for_frames. Looks like I need to call it via JNI. If the bandwidth permits, the team should expose rest of the APIs on Android in future.
Also, the colour camera is unstable on Android, and resetting the device/usb hub feels more like a hack for a device of such calibre, which is designed to run 24x7. Also I am not sure about effectiveness of this device reset approach, although I will test this soon and report back.
In any case, I hope Intel looks into this and sorts out underlying issue. The problem is that the previous threads that reported the issue, acknowledges the problem but mentions about cluelessness about the underlying cause. Maybe the team can think about bringing in some additional help from some expert.
Also, is there a chance that using infrared camera stream instead of color camera would improve stability on Android? Is that path worth exploring? Given I am already unable to use 1920x1080 resolution with color camera for stability issues, I can then use 1280x720 infrared camera stream, rather than same resolution stream from color camera, if it helps it in making it stable. In such share some links on how to remove laser projector pattern from the infrared camera.

Thanks for the detailed feedback!   If you post your feature request as a separate issue on this forum then it will enable the RealSense team to tag it specifically as a feature request and track it if they believe that it is something that they might be able to add at a future date.
If a member of the RealSense user community is able to write their own code to address a particular issue and would like it to be incorporated into the SDK, they are welcome to submit their code as a 'pull request' (PR) so that the RealSense development team can look at it and try to merge it into the SDK.
https://github.com/IntelRealSense/librealsense/pulls
I do not have the specialist knowledge about the Android wrapper to comment reliability on whether a certain tactic with it will work, so hopefully one of the Intel RealSense team members on this forum will be able to address your questions about using the IR sensor.  Good luck!

@MartyG-RealSense
Thanks for your help. I am not sure whether you are part of intel team or rather an independent open source contributor. But atleast you replied on this thread, when no one else did.
The request for 10 fps was due to the fact that D415 is practically unusable on android, inspite of being supported atleast for last 1 year. I believed that lowering fps on android would help but it did not. Even 6 fps stream eventually faces the issue, not to mention that 6 fps brings additional problem of blurring which, may be corrected to an extent by lowering exposure time. Not sure how to go about that using Android API.
One would believe a product launched by Intel would have better support and more robustness, specially when it is already more than 2 years the product was introduced. But looks like it is not, probably because it is not intel's core line of products. I have seen better support for product launched by much smaller companies or young startups. Only shame is Intel launched their product at a very reasonable price. Intel needs to think whether they want this product to be regarded as just another DYI project for enthusiast or want to provide it for actual production environment.
Intel now keeps adding new line of products like "i" series etc. Lets hope someone from intel goes after actual product stability, before adding more half supported and unstable products. Sorry for the rant, I have literally pulled my hairs for last 20 days fixing and trying different approaches and posted on threads with my findings. The least you can expect is a reply from someone from engineering team that they are looking for a solution, but I don't see any activity from the developer team. They conveniently avoid replying to questions that they don't have answers too, keeping users in total dark. I would open new thread regarding that.

@kafan1986  Thanks very much for your further feedback.  I am not an Intel RealSense employee.  As an end-user myself, I do very much understand your frustration
An Intel RealSense team member responds to every case at least once, in order to be able to close it after checking if the user still requires assistance.    There is no predictable schedule for when this first response occurs though.  I respond myself if I can offer at least an educated opinion / best guess, but there are inevitably gaps in my knowledge that mean that some cases are best responded to by RealSense team members with specialist knowledge.  Sometimes RealSense team members also need to refer questions to colleagues with specialist knowledge.
Other than the custom FPS approach discussed earlier about capturing every "nth" frame, an alternative approach that some RealSense users have used is to use a timer.  One user developed a RealSense ROS node that is controlled by a 'rate'.  The link below provides an explanation about 'Rates' in ROS.
https://roboticsbackend.com/ros-rate-roscpy-roscpp/
Another user set up a timer in QT instead of using ROS.
If you are aiming for 10 FPS, the best built-in compromise for avoiding errors that typically occur at 6 FPS is to use the 15 FPS speed.
Computer vision is becoming a key sector for Intel as fields such as AI and autonomous vehicles mature, and RealSense will have a presence at the forthcoming CES 2020 show at the start of January.
Intel aims for RealSense products to run on as many different hardware and software platforms as possible and so it is very open and non-proprietary, minimizing the number of pre-requisites that need to be installed in order for it to work.   Inevitably when attempting to "be everywhere and on everything", there are cases where problems occur that require some extra work to resolve.
The RealSense team try to address issues as quickly as possible, though some issues require a greater amount of time and development steps (e.g gradually building towards a solution over multiple SDK releases).  A fix may be especially slow to arrive if it has to be built into the firmware driver instead of the SDK, due to the need for the firmware to be rock-solid reliable before a new version is released to avoid "bricking" cameras.
Again, I truly understand your criticisms, and I appreciate that you provide constructive and well expressed feedback.


File download does not work in IE 5.5 SP1 on Win95

Browservice seems to be working pretty much flawlessly on Windows 95 OSR2.5 with Internet Explorer 5.5 SP1, with one major exception; File download does not work. The download button will show, the file will download to the browservice server, but IE will only download a HTML file with a scrambled file name, seemingly the same HTML page as is currently shown in the browser (the browservice client).
IE version 5.50.4522.1800
Windows 95 OSR2.5
Browservice <IP_ADDRESS> armhf running on an Raspberry Pi 3 Model B
File download works perfectly fine in Netscape 7.2 and Firefox 1.0

Did you actually check that the downloaded .html file is a HTML page, instead of the file you were trying to download under a wrong name and extension? Old IEs on old Windows versions are weird about downloads, even when not using Browservice; they simply force some file downloads to have the .html extension but if you change it the file itself will be correct. I don't see why the browser would download the client web page instead; if changing the extension doesn't work, could you show an example (complete with screenshots) of downloading a publicly available file and getting wrong file contents?

Yes, it is indeed a HTML page. I will find an example later today. I should also mention that IE loses it's connection to browservice after downloading the file.
Upload works great tho. This github issue is created with this setup :)

I had the exact setup as yours and well, the file that I wanted to download downloaded successfully.

Seems like the exact same thing on my side. There's nothing wrong there.

I have tried multiple files, this was just an example as it was easy to find.

I was also able to reproduce this on MSIE 5.50.4522.1800 (English) on Windows 95. However, the issue does not appear on any of the following versions I tested:

MSIE 5.0: 5.00.3314.2101
MSIE 5.5 (without SP): 5.50.4134.0600
MSIE 5.5 SP2: 5.50.4807.2300
It seems that for some reason in this particular version, MSIE 5.5 SP1, the browser does not handle downloads from iframes correctly, instead downloading the parent page HTML. Because it only seems to affect the SP1 version and not the original or the SP2 version, I think it's really not worth it to try to fix it; the fix would most likely require major changes to the download logic and risk breaking compatibility with other client browsers.

I suggest that you simply upgrade to SP2 (or downgrade to pre-SP1, if possible).


Device manager logs processed by another rule

Device manager rules are being processed by another rule. This is causing the fields:

name
class
guid
deviceID

to be captured as part of the event description.

Rule was updated to resolve the issue.


How can I get user shares?

Hello!
I tried something like this ln_client.network_updates(:type => 'SHAR'), but even in this way I got all updates without including user shares.. Do I have opportunity to get only user shares or get all updates with including user shares? (when I tried ln_client.network_updates, without specifying any parameters - I got all updates without user shares too)

I've encountered the same problem, using gem from v2 stable branch does not help too


Using a link containing "=" makes the extractParametersFromTags crash

Using this link "www.facebook.com/agencearcantide/photos/a.346501409312886/346503839312643/?type=3&theater" makes the extractParametersFromTags function crash.
I'm not an expert with regex but perhaps the expression has an issue with the "="?

Just tested the pull request #26, and it seems to be resolving my issue as well.

I've tried and it works thanks to #68 .
That's an example:
let baseStyle = Style {
	$0.font = UIFont.systemFont(ofSize: self.baseFontSize)
	$0.lineSpacing = 2
}

let linkStyle = Style {
	$0.font = UIFont.boldSystemFont(ofSize: self.baseFontSize)
	$0.linkURL = URLRepresentable.tagAttribute("href")
}

let text = "Stile link"
let styles = StyleGroup(base: baseStyle, ["a" : linkStyle])
self.textView?.attributedText = text.set(style: styles)


Annotation probably fails on Windows due to no $EDITOR

This code will most probably fail on Windows as there is no $EDITOR (not your PR, the annotate function as it is).
To make it more robust, on Windows you could use os.startfile to open the file in question with the default application.
(From comment on #97 by @bilderbuchi)

Just tried this on Windows and recipy annotate does not work because Windows has a different format for environments variables (uses %EDITOR% format).
Changing to:
os.system('%s %s' % (os.environ.get('EDITOR'), f.name))

did work for me as long as I set the environment variable first as it is not a standard Windows variable:
set EDITOR=notepad.exe

Thanks. I've done a bit of playing too, and I think something like this will work:
editor = os.getenv('EDITOR')
if editor:
    os.system('%s %s' % (editor, f.name))
else:
    touch(f.name)
    os.system(f.name)

So, it will try and get the EDITOR variable (which should be set on most Linux/OS X systems) - and if it can't, it will 'touch' the file (like the Unix touch command, but in pure Python) and then execute the file which, on Windows, will open the file in Notepad.
For reference, the touch function is defined at http://stackoverflow.com/questions/1158076/implement-touch-using-python

I assume you're also adding a .txt extension to the temp file to get that last system call to work.

Fixed in #155.


use 'npx relative-deps' instead of 'yarn relative-deps'

In a non-yarn NPM environment 'yarn relative-deps' obviously doesn't work, but 'npm run relative-deps' also doesn't work, which makes sense as there's no script. But the 'npx relative-command' does work so I propose we put this in instead.

Good points. Thanks!


Add a maxWidth parameter to LabelLayout

maxWidth limits the width of the Label.
The idea is that in scenario like:
StackHorizontal[Label1(max 100), Label2]
Label1 can use horizontal space up to 100 Label 2 can use all the available space. All the other rules of StackLayout apply like before.
If there are other ways to achieve with current architecture please let me know. Otherwise happy to add more tests around this.

Thanks for the pull request!
I think this is a specific instance of a problem that we can solve more generally. It would be useful to constrain any layout to have a minimum and/or maximum width and/or height.
I would rather solve the more general problem instead of making a special case for LabelLayout so I am going to close this PR. I have created issue #18 to discuss a solution to the more general problem. Would love to hear your thoughts there.


Override Indicators now don’t hold on to compiler symbols.

This should remove one source of memory leaks. Also, only create one
override indicator per definition, towards the closest override.
Fix #1002293

LGTM

Thanks @wpopielarski! merging this.


Fix#37 : Migrate from 2.7 to 3.6

Update .gitignore and a couple of file. Manage conflict between 2.7 and 3.6 about str and byte concatenation. Also update next() method from generator to __next__()
#37

Coverage decreased (-0.09%) to 46.852% when pulling 4d4130c5de1fe238475487e7ad75507136f1ef42 on AwesomeTurtle:awesometurtle-migrate-to-36 into fb09de9d920227f2b27b17086cd0b3a73ea5fb39 on google:master.

Thank you for the contribution though!

Coverage decreased (-0.01%) to 46.933% when pulling 84ef5ccd2dc7c5b366211cf85c37c94074ea519c on AwesomeTurtle:awesometurtle-migrate-to-36 into fb09de9d920227f2b27b17086cd0b3a73ea5fb39 on google:master.

Coverage increased (+1.02%) to 47.959% when pulling 132b96282c1bb35626d025af9dd7185157df2e36 on AwesomeTurtle:awesometurtle-migrate-to-36 into fb09de9d920227f2b27b17086cd0b3a73ea5fb39 on google:master.

AdbCommands doesn't seem to work with python3 when connecting over a socket, should Send() encode the data before sending it to the wire on line 208?

Coverage increased (+0.6%) to 47.585% when pulling 81f7c12e03fee9f2329ea847f8c5e2803b868fb7 on AwesomeTurtle:awesometurtle-migrate-to-36 into fb09de9d920227f2b27b17086cd0b3a73ea5fb39 on google:master.

Coverage increased (+0.6%) to 47.585% when pulling 50333effa49caf5aadecf16cb7dd5a68a8bda85e on AwesomeTurtle:awesometurtle-migrate-to-36 into fb09de9d920227f2b27b17086cd0b3a73ea5fb39 on google:master.


Memset

Added memset krnl operation, used it in GEMM and Matmul, will let us eventually optimize this op, possibly with library call to optimized versions.
This PR also add a convenient createAffine.forIE() method where we can pass lists of loops. It makes them perfectly nested.

krnl.memset is quite useful.

Thanks, and my hope is that we can connect it to an efficient implementation in LLVM, at least for the value=0 case.

Jenkins Linux amd64 Build #2072 [push] Memset (#884)  * initial... started at 21:43

Jenkins Linux s390x Build #2070 [push] Memset (#884)  * initial... started at 22:43

Jenkins Linux ppc64le Build #1180 [push] Memset (#884)  * initial... started at 22:43

Jenkins Linux s390x Build #2070 [push] Memset (#884)  * initial... passed after 48 min

Jenkins Linux amd64 Build #2072 [push] Memset (#884)  * initial... passed after 55 min

Jenkins Linux ppc64le Build #1180 [push] Memset (#884)  * initial... passed after 1 hr 2 min

@chentong319 thanks for your feedback.

Is the recursive function for ForIE is really needed? Can a loop work?

We are working with lambda functions, and I am building on that support. This forced me to do recursion. Potentially, we could build a loop with a lambda function just at the end if I was not reusing existing infrastructure based on lambda. Took the modular approach to reuse existing infrastructure.

To me, the intention of the PR could be provide some utility (optionally in krnl) to easily construct memref operations. for example: (memref<10x20xf32>: with constraint 2, 4 <= I,j < 6, 7 ) = memref<4x3xf32>.

At this stage, the LBs/UBs on the loop drive the iterations. So you could simply say
createAffine.forIE({2,4}, {6, 7}, {1, 1}, [&](AffineBuilder &createAffine, ValueRange iter) {
  createAffine.store(val, memref, iter)'
}

The name memset somehow made me think of c library memset. Is this intentional? Should we check the element type of memref and the type of Value? Not clear about our rule about common practice

yes, the idea is to eventually lower memset to an optimized LLVM call. Did not want to risk adding additional dependences (think it comes from string.h). Would ideally have an implementation that is embedded in LLVM or stdlib


Added updated guidelines link

Added this so that you don't have to handle with spams.

Thanks for contributing, but I don't think that this disclaimer is needed, people who are making spam aren't even reading the first line of the README, and they do know what they are doing.
So I'm going to close this pull request, but I will not mark as invalid, so it stills counts for you.

Will you please label it as "hacktoberfest accepted" due to the new guidelines.


failed to login: : challenge_required

When I execute:
instadiff-cli followings --list
I got the following fatal error:
FATA[2022-06-06 22:33:48] service setup: make client: failed to login: : challenge_required
The application should work the Instagram challenge_required issue, right? What could be wrong?
Thanks a lot for your support.

@marceloavilaoliveira Please try a newer version v0.16.0

Thanks @marceloavilaoliveira for your detailed feedback.
I'll take a look on it.
I have some questions:

Have you set up an application for 2fa (like 1password, Google Auth)?
Have you check a screenshot that is mentioned on logs?

Hi @obalunenko
Actually, I didn't have set 2FA on Instagram yet, so I configured it to use Google Authenticator, and it worked:

The application listed correctly all the followings. Unfortunately, in the end, one error was presented:

But this didn´t compromise the result.
I'm doing more tests but I think the application is working pretty well now. I'll let you know.

Hi @obalunenko,
I executed some other commands and everything seems to be working as expected. The only "weird" thing is the "405 Method Not Allowed" error at the end of execution.
Thank you very much for your help, and congratulations on your application.
Let me take this opportunity and ask a few questions:

Why does Instagram show "Samsung SM-G975F" in the Login Activity (I'm using the application on a Linux computer)?

Can I use usernames in the whitelist or must I use numbers?

Is there any instructions on how to build/install from code?

Thanks in advance.

Hi @marceloavilaoliveira,
Thanks a lot for the testing and for your feedback.
Answering your questions:

This application utilizes the mobile API, a device is also emulated. That's why you can see  "Samsung SM-G975F" as user-agent in the Login Activity.
Yep, I've added the support of both in new version
I added information to the README how to develop and test locally. In brief, just use make build command in the root of repository.

Unfortunately, the v1.0.1 didn't work for me.
Trying to clean followers:
> instadiff-cli clean-followers
...
INFO[08-06-2022 22:20:58] Logged-in                                     username=xxxxx
INFO[08-06-2022 22:21:02] Cleaning from not mutual followings...        cmd=clean_not_mutual
INFO[08-06-2022 22:21:41] Exit...
FATA[08-06-2022 22:21:41] Failed to run                                 error="clean notmutual: get not mutual followers: get followers: make users list: users iterate: No more posts availible, page end has been reached"

Trying to list followings:
> instadiff-cli followings --list
...
INFO[08-06-2022 22:22:41] Logged-in                                     username=xxxxx
INFO[08-06-2022 22:22:53] Exit...
FATA[08-06-2022 22:22:53] Failed to run                                 error="get followings: make users list: users iterate: Error while calling friendships/xxxxx/following/, status code 560:  (560 )"

Yeah, that's my bad about "No more posts available" error - late night coding issue 😄
FIxed this in v1.0.2
About status code 560 - this seems to be an Instagram server issue, smth went wrong on their side at that moment.

Now it worked pretty well.
Thanks a lot.

Thank you, @marceloavilaoliveira, for your feedback and help to improve the application.
I appreciate that^^

@marceloavilaoliveira did you eventually get past the challenge with the chromium browser, or how did you resolve it?

From time to time, the challenge appears but I was not able to make it works yet. AFAIR I click on the "Submit" button and nothing happens (I'm not sure about this). Every time this happens my account is blocked and I have to set a new password. I'll pay more attention to the details the next time it happens.

Hm interesting. Next time it happens, could you open your dev tools when you get the browser, and check the response to the request that submits the code? >> does it go through or not
What you could also try is after submitting the code, and the request has gone through, manually closing the browser window, and checking if it works. Could also be that it just doesn't redirect properly after submitting the code

Today some different happened... An Instagram message said my account was blocked and sent me a 6-digit code, I typed the code and clicked on "Submit", then it asked me to set a new password, I set it, and a new window opened saying "Open xdg-open? https://i.instagram.com wants to open this application", I clicked on "Open xdg-open" button but nothing happens. Then I opened the Chrome "Inspect" and when I was trying to discover something, the window just closed.
The following message is showed in my terminal:
...
INFO[18-09-2022 15:39:28] Cleaning from not mutual followings...        cmd=clean-followings
Fetching followers▌Request Status Code 400: fail, challenge_required
Encountered a captcha challenge, goinsta will attempt to open the challenge in a headless chromium browser, and take a screenshot. Please report the details in a github issue.
INFO[18-09-2022 15:44:41] Exit...
FATA[18-09-2022 15:44:41] Failed to run                                 error="get not mutual followers: get followers: make users list: users iterate: failed to process challenge automatically with: context deadline exceeded"

Hmm yeah I see. For now it's best to solve challenges directly in the app.
The difficulty with making this work is that it's very hard to reproduce often, and make it work reliably.
Did it give you a reason for the block ?

After that, when I tried to use the app, it said that some automation (or any other term like that) is trying to use my account and, because of it, my account was blocked. I had to resolve two challenges and change the password. :-(
So, when the instadiff-cli asks for a challenge, is it better to use the app first to resolve the issue?

Hm I see. Yes, it is

@Davincible could these new query parameters added in version v3.2.4 possibly trigger Meta ML AI that these requests are send via automation script, not from the app?
commits

https://github.com/Davincible/goinsta/commit/27d9fde1d6cacd50780b8e79789a1f0fb9a94f99
https://github.com/Davincible/goinsta/commit/1837cc64b613fa26ea92ab975cca97cf8dda7c74

I don't think it's the get params. I expect it's related to the update to v<IP_ADDRESS>.109, where perhaps they implemented either extra requests in the login procedure, or added extra headers, and detect that they are missing. Just a hunch though.
It's possible to add them in, just requires more work.
If you have the time I could teach you how I go about analyzing it and we could look at it together, two pairs of eyes are always better than one.

@Davincible Sure, I'll gladly help you

@obalunenko how can I best reach you? (telegram, discord, smth else?)

@Davincible telegram is my primary channel for communication
https://t.me/obalunenko


In a dashboard, show a list of dropdowns with the variables used in the cells in that dashboard

Each dropdown in the list allows the user to set the value for that variable.

Depends on: https://github.com/influxdata/influxdb/issues/12470


HTTP 501 when downloading files from Artifactory

I can't download RPM files from https://packages.adoptium.net/artifactory/rpm
For example: GET https://packages.adoptium.net/artifactory/rpm/rocky/8/x86_64/Packages/temurin-21-jdk-<IP_ADDRESS>.0.12-1.x86_64.rpm
results in HTTP 501.

Hi folks, we're aware of the issue and are investigating with our CDN and package repo providers.

@shollander I believe this is now working correctly, would you mind checking if its working for you?

@steelhead31 Yes, it is working for me now.
Thanks for your help!


Upgrade to MJML 4.0.x

Hello,
Here's a PR for the upgrade to MJML 4, and I also took the opportunity to replace the deprecated gulp-util package.
I've been using it without issues, but if anything arises, feel free to poke me !
Should solve #34

Hi @fmauNeko
Thanks a lot for this PR ! That's look good to me, didn't  know that gulp-util was deprecated !

any update about a release with this path for MJML 4?

@iRyusa I think the npm repo has not updated with the latest version as the master branch is different.


Fixed issue with if statement checking for errors

Resolved Xcode analyze warning which is worried about "errors" being nil. Also looks like if the error was not the last object of the for loop, the userInfo would not be properly set

Could you please add new test coverage?

Well through trying to make a new test, it turns out there is no way for the userInfo could not be set, since the method returns if there is a successful transformation. So really this pull request just silences the Xcode analyze warning about "errors" potentially being nil even though it really can't be

Um, this created bugs.
The "if (error)" check was to see if the passed-in error pointer was NULL or not; if NULL the caller does not care about errors and we should not create them.  This change will now cause a crash if there are errors but the pointer is NULL, since it will dereference the pointer without checking.
The second problem is if the transformation failed but there were no underlying errors -- no error object will be created anymore, even though the return value is failure.

@carllindberg in that case, please make a PR reverting this and adding regression tests?


Add dbprocessing as new project

Adds dbprocessing as a new PyHC package. This work was funded via the 2020 HDEE call.

I'm going to go ahead and merge this.


There  are  no  knime.types and knime.types.geospatial

There  are  no  knime.types and knime.types.geospatial  ,why ?
'import knime.types.geospatial as gt'

The geospatial types are par of a KNIME extension that you need to install manually into the KNIME AP that you use for development. To do so go to File->Install KNIME Extension in the Install window unselect Group items by category and search for geospatial. Than select the KNIME Geospatial Extension.

In your extension you need to add a dependency to the feature as well as its done here: https://github.com/spatial-data-lab/knime-geospatial-extension/blob/main/knime_extension/knime.yml#L12
What kind of nodes/use cases do you plan to develop? We are currently working on several new nodes which might solve you some work :-)

Thank you very much

Excuse me ,no   knime  python integration (labs)  ,why?

Unselect Group items by category and search for geospatial.

Am I using the wrong version? I use 4.7. Do I have to use 4.7 Nightly Build


[Android] JS not being fully reloaded after user close the app via back button

Issue Description
JS not being fully reloaded after user close the app via back button
Steps to Reproduce / Code Snippets
// index.android.js

import React, { Component } from "react";
import { View, AppRegistry } from "react-native";

console.log("loading source..");

class Example extends Component {
  componentDidMount = () => {
    console.log("mounting example");
  };
  render() {
    return (
      <View>
      </View>
    );
  }
}

AppRegistry.registerComponent("example", () => Example);

The first time the user open the app, it will print both messages.
After the user closes the app via back button, and reopen the app, it will:

on RN0.33+ only prints "mounting example" message (which means the JavaScript sources was not fully reloaded.)
on RN0.32 it will print both messages.

Expected Results
I think both messages should be printed. because closing via back button should trigger the same effect as closing the app via app switcher (which make sure everything is clean as whistle).
And also this will break some library that require setup before the element are rendered.
e.g.

https://github.com/AlexanderZaytsev/react-native-i18n needs to load the i18n before rendering, so it could get the proper translation before the first component is rendered
Edit: It would also break https://github.com/Microsoft/react-native-code-push because the new code-pushed bundle is not being reloaded at all (or at least until the app is killed)

Additional Information

React Native version: 0.33+
Platform(s) (iOS, Android, or both?): Android
Operating System (macOS, Linux, or Windows?): macOS

edit: This happens for both debug & release build

closing via back button should trigger the same effect as closing the app via app switcher

Is this true? My understanding was that Android keeps the app alive and backgrounds it when you tap the back button in order to conserve battery life and preserve application state.

closing via back button should trigger the same effect as closing the app via app switcher

Is this true? My understanding was that Android keeps the app alive and backgrounds it when you tap the back button in order to conserve battery life and preserve application state.

I'm only speaking in term of the initial view rendering process. Which on RN033+ looks like this:

Initial Launch: "load JS bundle" -> render
After closing w/ back button: "use JS bundle that've been loaded before w/o reloading" -> render

And also even though the app is still alive on background, the component will be unmounted and you'll want a freshly loaded JS bundle to render the next time anyway

It seems code-push has a different issue all together..
I guess the most annoying thing with this issue is that developers need to make sure to clean up their singleton/globals when the root component is unmounted..

@sbycrosz @ide does it got fixed .?

I've reverted to 0.32 for our latest release. But then I'm not really sure if this still an issue b/c the code-push one is a different issue all together.
Have you encountered this @saleeh93 ?

@sbycrosz I had different issues where i get my navigator gots unmounted and i was keeping the reference to the old navigator , I got fixed that by setting null on componentWillUnmount

It sounds like nobody is left with this issue so I am going to close it.


Update of README_ch.md

Update of Chinese Translation

😓Sorry, I forgot to delete it in README.md


bin\Runner.Server does not appear to use the MemoryDB if no external sqlite is specified

Describe the bug
If I have read the code in https://github.com/ChristopherHX/runner.server/blob/main/src/Runner.Server/Startup.cs and the documentation correctly, but I could be wrong, then the code does not for some unknown reason not use the "conf.UseInMemoryDatabase("Agents");"
I get the following error and info:
$ bin\Runner.Server --urls http://localhost
Relational-specific methods can only be used when the context is using a relational database provider.
   at Microsoft.EntityFrameworkCore.RelationalDatabaseFacadeExtensions.GetRelationalService[TService](IInfrastructure`1 databaseFacade)
   at Microsoft.EntityFrameworkCore.RelationalDatabaseFacadeExtensions.Migrate(DatabaseFacade databaseFacade)
   at Runner.Server.Startup.ConfigureServices(IServiceCollection services)
info: Microsoft.Hosting.Lifetime[14]
      Now listening on: http://localhost:80
info: Microsoft.Hosting.Lifetime[0]
      Application started. Press Ctrl+C to shut down.
info: Microsoft.Hosting.Lifetime[0]
      Hosting environment: Production
info: Microsoft.Hosting.Lifetime[0]
      Content root path: C:\actions-runner\bin

To Reproduce
Run the following
bin\Runner.Server --urls http://localhost
In the C:\actions-runner directory with no specific configuration files.
Expected behavior
The bin\Runner.Server --urls http://localhost to not cause a DB error to show.
I have not tried to configure the server to use SQLLite .
Runner Version and Platform
$ .\bin\Runner.Client.exe --version
3.8.0
OS info:
Windows 10 21H2 patched up to the latest from earlier this week (16-MAy-2022)
What's not working?
See above
Job Log Output
See above
Runner and Worker's Diagnostic Logs
See above

Everything is working, enshured by CI integration Tests. The only issue is that I call the following method with an in memory db ( not a Relational database ) https://github.com/ChristopherHX/runner.server/blob/a290adbe80266f68053926c5e78387752aa2314d/src/Runner.Server/Startup.cs#L126

Sorry, but is it a bug or am I not using the bin\Runner.Server correctly?

Sorry, but is it a bug or am I not using the bin\Runner.Server correctly?

I think you are using bin\Runner.Server correctly.
If the only error is your quoted error message, then you can ignore the error message. This is no fatal error.


[TODO] Add Jest/React Testing Library tests

A good app needs some tests. The one we have is the default for Jest/RTL that comes with Create React App 5.
Add a test folder and create some initial tests for the UI,etc.

@gbowne1 hi there, i would like to work on this, please assign this to me
Thanks

Ok @Sky-De will assign you

@gbowne1 thanks for assignment, for start i will add packages and initializing testing setup with one sample test, because it is lot of works testing all app at once , then I'll test components one by one , is it ok with you?

there are already testing packages installed.
That is fine with me @Sky-De


ruuvitag: only include entities that are available

I'm not sure how to achieve this.  How can I make the set of entities more dynamic.  My attempt has resulted in:
Traceback (most recent call last):
File "/usr/bin/ble2mqtt", line 8, in
sys.exit(main())
^^^^^^
File "/usr/lib/python3.11/site-packages/ble2mqtt/main.py", line 171, in main
File "/usr/lib/python3.11/asyncio/runners.py", line 190, in run
File "/usr/lib/python3.11/asyncio/runners.py", line 118, in run
File "/usr/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
File "/usr/lib/python3.11/site-packages/ble2mqtt/main.py", line 127, in amain
File "/usr/lib/python3.11/site-packages/ble2mqtt/ble2mqtt.py", line 95, in register
File "/usr/lib/python3.11/site-packages/ble2mqtt/devices/base.py", line 504, in init
File "/usr/lib/python3.11/site-packages/ble2mqtt/devices/base.py", line 210, in init
File "/usr/lib/python3.11/site-packages/ble2mqtt/devices/ruuvitag.py", line 50, in entities
if self._state.humidity:
^^^^^^^^^^^
AttributeError: 'RuuviTag' object has no attribute '_state'
I'm trying to avoid warnings from Home Assistant:
WARNING (MainThread) [homeassistant.helpers.template] Template variable warning: 'dict object' has no attribute 'pressure' when rendering '{{ value_json.pressure }}'

It can be a problem as properties are exported to MQTT HA discovery topics after first device appear. So it has to know what entities it has. So, this dynamic creation will not work. Possible solution is to define all items and populate only available.

Okay.  Another option would be to create subclasses for the two RuuviTag Pro models which override the entities property.

Yes, that would be better in terms of internal structure. 👍

@devbis I've now implemented the subclassing idea.  I've tested it and now it works.

Thanks!


fix(Tabs): visual fixes

TACO-1038
Description
This PR fixes few visual issues related to the newly introduced (by me :see_no_evil: ) vertical tabs:

The active tab has no background and according to the design it should have #FCFCFC https://www.figma.com/file/5SCTOPrCDcHuk5We091GBn/Product-Library?node-id=1413%3A15705
The labels are center aligned and they should be left aligned. This one wasn't obvious because my example was using the same label :unamused: Now the example is using different labels
The active tab shadow gets trimmed on the left side because the Tabs root container has a overflow: hidden. I've added a :pinching_hand: of padding-left

How to test

Use the example in storybook

Screenshots

Before.
After.

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

Tested from temploy. Looking good +1 We should catch these while reviewing but man_shrugging
Thanks for the contribution once again bowing_man

Yeah I totally missed those :facepalm: and now they seem so obvious


Enough: Change the date on the upcoming drop

This drop:
https://kodadot.xyz/ahp/drops/enough
Move date from 16th to 23rd pls.
There is an issue with the code artist is trying to change. @vikiival maybe?

Moved on CF

Thank you!


alerts with errors blocking website

How to reproduce:
Open website https://tailwindconverter.app/
Expected result
Page is loaded
Actual result
Alerts is shown. Clicking on "OK" another alert is shown. And it happens a lot of times before you can access the website.
Example of alert: Invalid Color: Error: "var(--bs-body-color)" is not a valid color

Thanks for reporting this - the alert has been replaced with a console.error. Fixed in https://github.com/kt474/tailwind-converter/commit/c3fe08ed12733c51941ebf07c8d082348bb1ae44


Saving changes in Settings UI changes Precision for Decimal Values in settings.json

Environment
Windows build number: Microsoft Windows [Version 10.0.19041.746]
Windows Terminal version (if applicable): Windows Terminal Preview v1.6.10272.0

Any other software? NA

Steps to reproduce

Note decimal values for fields acrylicOpacity & backgroundImageOpacity
Open Settings UI, make some change & click on Save
Observe that precision for the values  have changed.

Expected behavior
The precision for the decimal values should not change
Actual behavior
Values now have precision of 17 digits instead of 2.
Before:

After:

So, this is only half correct. The precision isn't changing, it's just being exposed to you. IEEE954 rules!

Should we just include this with #8991 for ""settings UI reformats my file?

Yes, that would be good.

/dup #8991


parseCEL method is not working for does not contain, does not begin with, does not end with, not in

React Query Builder version
No response
Platform
No response
Description
I have converted the RuleGroupType to CEL Expression using formatQuery(query, "cel") method, it is working fine. When i try to convert the CEL Expression back to RuleGroupType using the parseCEL method, the RuleGroupType is not as same as the initial RuleGroupType. Operators which are not working when converting CEL Expression to RuleGroup Type: does not contain, does not begin with, does not end with, not in

This is the query generated when converting to RuleGroup Type to CEL Expression :  field == "marks" && (!value.startsWith("68")).
When converting it back to Rule Group type using parseCEL getting this value:  field == "marks"

Reproduction
Construct any query which consists of !(not) in front of it like this !value.startsWith("68") when converting to CEL Expression. You can do it  by selecting any of these operators: does not contain, does not begin with, does not end with, not in.
Then try to convert the same CEL expression to RuleGroupType, you can find only the expression which doesn't include these operators
Expected behavior
No response
Additional information
No response

I can reproduce the behavior with your query.
The rootcause seems the position of the !.
In your query, it's inside the brackets.
If you create the same query via the demo ui, you get the following query.
firstName == "firstname" && !(lastName.startsWith("lastname"))

If you import the generated CEL query, you get the following result:

If you import then the generated query:
firstName == "firstname" && (!lastName.startsWith("68"))

it results in your error:

The error occurs also if you do not use the brackets around the second part:
firstName == "firstname" && !lastName.startsWith("lastname")

Thanks for the investigation @noxify! This definitely seems like a fixable bug. I'll take a stab at it soon...unless someone else wants to have a go.

I just got around to looking at this over the weekend. Funny enough, I had put a // TODO: comment to fix this in the test file a long time ago:
https://github.com/react-querybuilder/react-querybuilder/blob/221085b07f16926e9436968f26c31fd50276201c/packages/react-querybuilder/src/utils/parseCEL/parseCEL.test.ts#L134-L135
Anyway, #689 should fix the issue. You can try out the new parseCEL behavior in the preview website demo or a sandbox with the new build.


Add missing MSRV-bumps in changelog

I posted my comments based on https://crates.io/crates/toml_edit/versions

As there are no tags for the toml_edit versions I don't have a good way of figuring this out.

There are tags for toml_edit

Thanks for merging.

There are tags for toml_edit

I didn't know that toml_edit was the "main crate", so I was looking for toml_edit-x.y.z tags.