Equivalent to Flask's path converter

Similar to this Flask snippet, is there a way to do this in hug?
eg
@hug.get('/test/{pathy}')
def tester(pathy):
    ....

and would accept urls like example.com/test/one/two/three with pathy = one/two/three.

Hi @daf,
the given snippet, exactly as defined should work! hug uses {varname} withing the route to automatically inject into hugged methods. Hope this works out well for you and let me know if you have any more questions.
Thanks!
~Timothy

@timothycrosley but it doesn't:
import hug

@hug.get('/test/{pathy}')
def tester(pathy):
    return " ".join(pathy)

Regular:
~ -> http localhost:9999/test/hello
HTTP/1.0 200 OK
Date: Thu, 08 Sep 2016 17:40:40 GMT
Server: WSGIServer/0.2 CPython/3.5.2
content-length: 11
content-type: application/json

"h e l l o"

Attempt to get a "path":
~ -> http localhost:9999/test/hello/a/path
HTTP/1.0 404 Not Found
Date: Thu, 08 Sep 2016 17:40:48 GMT
Server: WSGIServer/0.2 CPython/3.5.2
content-length: 562
content-type: application/json

{
    "404": "The API call you tried to make was not defined. Here's a definition of the API to help you get going :)",
    "documentation": {
        "/test/{pathy}": {
            "GET": {
                "inputs": {
                    "pathy": {
                        "type": "Basic text / string value"
                    }
                },
                "outputs": {
                    "content_type": "application/json",
                    "format": "JSON (Javascript Serialized Object Notation)"
                }
            }
        }
    }
}

The latter is the approach possible in Flask that I'm after.

Ahh sorry, I misunderstood the original issue. To get what you want you would need to use a sink:
import hug

@hug.sink('/test/')
def hello(request):
    """Says hello"""
    return request.path

Thanks for clarifying, sorry I originally misunderstood!
~Timothy

How would I define this flask route in hug?
@app.route('/api/{userId}/<path:path>', methods=['GET'])

It seems like the SinkRouter doesn't work with variables, this is what I tried.
@hug.sink('/api/{userId}')
    def foo(userId, path):
        return userId + path


Update UI LeftPanel

Update PersonListPanel .java and .fxml to have the required fields

close #42
close #35


make it possible to restrict permission to read secrets to allowed namespaces

In a conversation with @jessesuen today, he noted that some users will be reticent to grant cluster-wide read permissions on Secrets to Kargo.
We need to make is possible to restrict Kargo's access to Secrets in particular namespaces, even if we elect not to make that the default.
This is relatively low priory at the moment.

@jessesuen I've been thinking about how to achieve this and it occurred to me that this might overlap with #558.
i.e. I was assuming #558 would be solved with a new controller that is less vulnerable to attack than the API server by virtue of not being exposed to the outside world. It would use broader permissions to orchestrate more narrow permissions for the API server...
Then it occurred to me that the API server doesn't touch secrets at all (at least not yet). Until/unless that is no longer the case, it is only the existing Kargo controller that can read secrets in all namespaces -- and that's the very same permission we'd have to give to the onboarding controller.
In a nutshell... I don't think there's anything to do here until/unless we need to start allowing the API server to access secrets in project namespaces.
wdyt?

Yep, I agree this is probably unnecessary.


New features for n-body simulation

Hello there,
I'm currently working on astrophysics simulations and I'm really excited about kiddo. I would like to use treecode algorithm and/or fast multipole method. Indeed, to perform n-body simulation, grouping bodies is a very powerful technique to approximate the interaction calculation in physics simulation. This is generally done with a quadtree/octree or a k-d tree.
But this kind of algorithm requires iterating over the tree structure, which is to the best of my knowledge not possible yet.
Propositions
The goal is not to include any domain specific features like interaction forces, ODE or potentials. But adding new iterators to iterate over the tree structure may allow it to implement this kind of algorithm.
I studied how kiddo works and I can elaborate my proposition further but I want to keep the issue simple to understand for now.
Are you interested in this type of contribution or should I rather fork the project and work on my own? We can also implement it as an optional feature like the serialize feature or the f16 feature.

Hi Astrid! I would indeed be happy to accept a contribution of this kind. I look forward to finding out more!


add option for configurable new order status

Add option for user configurable new order status.
Set to on hold by default to match up with existing functionality but configurable to other states such as pending or whatever is desired to fit in with the functionality of the specific shop

Thanks for this contribution!  We'll review it asap

No problem, look forward to seeing what you think
On 14 Apr 2015 6:19 pm, "Rich Morgan"<EMAIL_ADDRESS>wrote:

Thanks for this contribution! We'll review it asap
—
Reply to this email directly or view it on GitHub
https://github.com/bitpay/woocommerce-plugin/pull/53#issuecomment-92986412
.

Absolutely!  We're in the middle of a couple other projects at the moment but will definitely get to this in the next week for sure. :)


[Release] Bumped nvidia_triton version to 1.2.0-beta.1

What does this PR do?

This is just for me to keep track of this while I build the beta wheel. No plans to merge.
Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

beta built successfully, closing this


Release 1.0

Codecov Report

Merging #106 into master will increase coverage by <.01%.
The diff coverage is 4.54%.

@@             Coverage Diff             @@
##             master    #106      +/-   ##
===========================================
+ Coverage      5.96%   5.97%   +<.01%
- Complexity       10      11       +1
===========================================
  Files            45      45
  Lines           620     636      +16
  Branches         71      72       +1
===========================================
+ Hits             37      38       +1
- Misses          575     590      +15
  Partials          8       8

Impacted Files
Coverage Δ
Complexity Δ

...xus/service/internal/CancerHotspotServiceImpl.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../exception/EnsemblTranscriptNotFoundException.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../exception/VariantAnnotationNotFoundException.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ce/exception/IsoformOverrideNotFoundException.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../service/exception/PdbHeaderNotFoundException.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...xception/VariantAnnotationWebServiceException.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ome_nexus/service/internal/PdbDataServiceImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ervice/exception/PdbHeaderWebServiceException.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...service/exception/PfamDomainNotFoundException.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...us/service/enricher/HotspotAnnotationEnricher.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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

yummy!


Switch to using Conda lock files to specify requirements and update dependencies

Switches from using pip-compile and requirements.txt files to specify Python dependencies to be installed using pip to using conda-lock to generate lock files specifying (platform specific) Conda environments with both Python and non-Python dependencies.
Also currently set to update dependencies to Python 3.11 and Pandas 2.0.0 - we could potentially pin to the current Python (v3.8) / Pandas (v1.2.2) versions instead to split the change over how we're handling dependencies and the actual updating of dependencies.

I don't think we want to go forward with this one any more so closing.


Shift and Fn1 behavior

Hi,
first of all, great work, your editor makes it so much easier to change the layout! I finally have a reason to use the MPC firmware on my Core.
But one thing I did notice:
Shift in combination with Fn1 seems to be a bit strange, it appears that the order in which I press the keys seems to matter. If for example I want to type a ! I have to press <Shift>, then <Fn1>, then <Tab>.
If I press <Fn1>, then <Shift>, then <Tab> I get „1“, not „!“.
With the normal non-MPC-firmware the order doesn‘t matter.
Any way to fix this?
Thanks
Lawiak

see #15

Yes, that was it! Sorry for not seeing it before I opened this issue.
And when I think about it it really does make sense, so it wasn‘t really an issue in the first place!


Error in big.PCA

Hi,
I have 2 questions:
Q1. I meet this problem at the end when running MVP:-
Error in big.PCA(bigMat = big.geno, pcs.to.keep = pcs.keep) :
could not find function "big.PCA"
Here is my script:
> imMVP <- MVP(
+     phe=phenotype,
+     geno=genotype,
+     map=map,
+     #K=Kinship,
+     #CV.GLM=Covariates,
+     #CV.MLM=Covariates,
+     #CV.FarmCPU=Covariates,
+     nPC.GLM=3,
+     nPC.MLM=3,
+     nPC.FarmCPU=3,
+     perc=1,
+     priority="speed",
+     ncpus=12,
+     vc.method="EMMA",
+     maxLoop=10,
+     method.bin="FaST-LMM",#"FaST-LMM","EMMA", "static"
+     #permutation.threshold=TRUE,
+     #permutation.rep=100,
+     threshold=0.05,
+     method=c("GLM", "MLM", "FarmCPU")
+ )
#--------------------------------------------Welcome to MVP---------------------------------------------#
#          A Memory-efficient, Visualization-enhanced, and Parallel-accelerated Tool For GWAS           #
# Version: 1.0.1                                                                                #
# Authors: Lilin Yin, Haohao Zhang, Zhiwu Zhang, Xinyun Li, Xiaohui Yuan, Shuhong Zhao, and Xiaolei Liu #
# Contact<EMAIL_ADDRESS>                                                                #
#-------------------------------------------------------------------------------------------------------#
[1] "Input data has 34 individuals, 51910 markers"
[1] "Principal Component Analysis Start..."
Error in big.PCA(bigMat = big.geno, pcs.to.keep = pcs.keep) :
  could not find function "big.PCA"

Q2. I have 38 individuals but it appears that it is reading only 34?
My MVP.Data script looks like this and was successful:
> MVP.Data(fileHMP="~/Desktop/SNP_formatted.hmp.txt",
+          filePhe="~/Desktop/Phenotype.txt",
+          sep.hmp="\t",
+          sep.phe="\t",
+          SNP.effect="Add",
+          fileKin=FALSE,
+          filePC=FALSE,
+          out="mvp.hmp",
+          #priority="memory",
+          #maxLine=10000
+ )
[1] "Preparing data for MVP..."
[1] "Preparation for PHENOTYPE data is done!"
[1] "Output numeric genotype..."
[1] "File: ~/Desktop/myGAPIT/SNP_Rlm7_formatted.hmp.txt ; Total markers:  51910  finished!"
[1] "Preparation for numeric data is done!"
[1] "Output mvp genotype..."
[1] "MVP data prepration accomplished successfully!"

Hi,
Thanks for your using of MVP! Maybe i guess that you installed the version of Unix instead of Windows for MVP, so please make sure the type of your platform, and then chose the right installation way.

I have edited your comment, please wrap the code with  ```, otherwise it is difficult to see clearly.
Q1:
I found that bigpca has removed form CRAN this month. so you should download it from archive
then:
$ R CMD install bigpca_1.1.tar.gz

Q2:
I haven't encountered a similar problem before, maybe you should check your data carefully.

Thank you very much, YinLiLin and hyacz.
Yes, I actually wrongly installed the Windows "devtools" onto the R studio in Mac.
Now I'm trying to install it in Mac but this warning came up:
Warning messages:
1: In install.packages(pkg[p], repos = NULL) :
installation of package ‘snpStats_1.26.0.tar.gz’ had non-zero exit status
2: In install.packages(pkg[p], repos = NULL) :
installation of package ‘MVP_1.0.1.tar.gz’ had non-zero exit status

install.packages(pkg[p], repos = NULL, dependencies=TRUE)
Installing package into ‘/home/tneik/R/x86_64-pc-linux-gnu-library/3.4’
(as ‘lib’ is unspecified)
ERROR: dependency ‘snpStats’ is not available for package ‘MVP’

Do you have any suggestions how I can solve this please?
Thank you in advance.

It looks like the Snpsta package fails to install.
You can try the following command in R.
> install.packages('snpStats')

or
> install.packages('<your-path>/snpStats_1.26.0.tar.gz', repos=NULL)

Note: You can find snpStats_1.26.0.tar.gz in the MVP offline installation package
If there are still errors, please send all the logs here.

Thank you very much, hyacz. All good now. I used this:
source("https://bioconductor.org/biocLite.R")
biocLite("snpStats")


Add linking to Data Objects

Objectives
Allow users to interact with the result section
Complete When

Tapping the image redirects to a relevant website
Touching the Phone number generates a call

Note to self: after adding plugins, the platforms must be removed and added again.

For website url:
https://www.thepolyglotdeveloper.com/2015/11/launch-the-web-browser-in-a-nativescript-mobile-app


rule idea: Identifying virtual environments via mac address

Hello,
Could be interesting maybe include vm detection using mac address api call like "UuidCreateSequential"
Suggestion link:
https://nakedsecurity.sophos.com/2016/12/13/nymaim-using-mac-addresses-to-uncover-virtual-environments-and-bypass-antivirus
rule:
  meta:
    name: Identifying virtual environments via mac address
    namespace: anti-analysis/anti-vm/vm-detection
    author:  andrea
    scope: basic block
    att&ck:
      - Defense Evasion::Virtualization/Sandbox Evasion::System Checks [T1497.001]
    mbc:
      - Anti-Behavioral Analysis::Virtual Machine Detection::Unique Hardware/Firmware Check
    references:
      - https://nakedsecurity.sophos.com/2016/12/13/nymaim-using-mac-addresses-to-uncover-virtual-environments-and-bypass-antivirus
  features:
    - and:
      - basic block:
        - and:
          - api: UuidCreateSequential
      - optional:
        - or:
          - mnemonic: cmp
          - mnemonic: jz
          - mnemonic: xor

nice idea!
i think we should tweak the rule a bit to detect the known constants related to hardware vendors. otherwise, the rule boils down to UuidCreateSequential which is a little loose.
then, we'll have to find a good example of this in the wild. maybe hunting in VT can yield one.

Thanks. Agree.
Maybe also something similar this. Just an idea.
``
rule:
meta:
name: Identifying virtual environments via mac address
namespace: anti-analysis/anti-vm/vm-detection
author: andrea
scope: basic block
att&ck:
- Defense Evasion::Virtualization/Sandbox Evasion::System Checks [T1497.001]
mbc:
- Anti-Behavioral Analysis::Virtual Machine Detection::Unique Hardware/Firmware Check
references:
- https://nakedsecurity.sophos.com/2016/12/13/nymaim-using-mac-addresses-to-uncover-virtual-environments-and-bypass-antivirus
features:
- and:
- mnemonic: cmp
- or:
- number: 0xf01faf$      # Dell Inc.
- number: 0x005056$      # VMware
- number: 0x000c29$      # VMware
- number: 0x001c14$      # VMware, Inc.
- number: 0x000569$      # VMware, Inc.
- number: 0x00155d$      # Hyper V
- number: 0x080027$      # PCS Systemtechnik GmbH (VirtualBox)
- number: 0x0003ff$      # Microsoft Corporation
- number: 0x001c42$      # Parallels, Inc
- number: 0x00163e$      # Xen, Inc
- optional:
- api: UuidCreateSequential
``

Great ideas @andreisss!
Tip: You can use descriptions in rules like this: - string: mac$ = VMware, Inc.. This way they are rendered in the output :bowtie:

Thanks for your feedback. 👍

@andreisss  if i can find an example binary or two, would you be willing to draft the PR for this rule?

@andreisss if i can find an example binary or two, would you be willing to draft the PR for this rule?

yes, it will be really nice for me if I can get this new rule working. thx

This was my test. Sample is available here: https://app.any.run/tasks/d1b0216f-47d3-433f-85bc-7ddaf4e8477e/#
I was following also this link https://labs.sentinelone.com/gootkit-banking-trojan-deep-dive-anti-analysis-features/ (real example with gootkit banking trojan)
My local test with the rule.

rule:
  meta:
    name: MAC address identifiers used to detect sandboxes or VMs
    namespace: anti-analysis/anti-vm/vm-detection
    author<EMAIL_ADDRESS>    scope: function
    att&ck:
      - Defense Evasion::Virtualization/Sandbox Evasion::System Checks [T1497.001]
    mbc:
      - Anti-Behavioral Analysis::Virtual Machine Detection::Unique Hardware/Firmware Check
    references:
      - https://nakedsecurity.sophos.com/2016/12/13/nymaim-using-mac-addresses-to-uncover-virtual-environments-and-bypass-antivirus
      - https://labs.sentinelone.com/gootkit-banking-trojan-deep-dive-anti-analysis-features/
    examples:
      - c342af62302936720e52679bc431d5a8
  features:
    - and:
      - mnemonic: cmp
      - or:
        - number: 0xF01FAF00
          description: Dell Inc.
        - number: 0x00505600
          description: VMware, Inc.
        - number: 0x000C2900
          description: VMware, Inc.
        - number: 0x00056900
          description: VMware, Inc.
        - number: 0x0003FF00
          description: Microsoft
        - number: 0x0003FF00
          description: PCS System Technology GmbH
        - number: 0x001C4200
          description: Parallels
        - number: 0x00163E00
          description: XenSource
        - number: 0x8002700
          description: PCS System Technology GmbH
      - optional:
        - api: UuidCreateSequential

Very cool, some minor tweaks in the rule below:

rename to complete sentence "this function may...", see https://github.com/fireeye/capa-rules/blob/master/doc/format.md#rule-name
remove duplicate number feature

I haven't tested this on the example, with the correct offset (see -vv output) this could go straight into the rule corpus! 🎉
Thank you!
rule:
  meta:
    name: detect VM using MAC address identifiers
    namespace: anti-analysis/anti-vm/vm-detection
    author<EMAIL_ADDRESS>    scope: function
    att&ck:
      - Defense Evasion::Virtualization/Sandbox Evasion::System Checks [T1497.001]
    mbc:
      - Anti-Behavioral Analysis::Virtual Machine Detection::Unique Hardware/Firmware Check
    references:
      - https://nakedsecurity.sophos.com/2016/12/13/nymaim-using-mac-addresses-to-uncover-virtual-environments-and-bypass-antivirus
      - https://labs.sentinelone.com/gootkit-banking-trojan-deep-dive-anti-analysis-features/
    examples:
      - c342af62302936720e52679bc431d5a8
  features:
    - and:
      - mnemonic: cmp
      - or:
        - number: 0xF01FAF00
          description: Dell Inc.
        - number: 0x00505600
          description: VMware, Inc.
        - number: 0x000C2900
          description: VMware, Inc.
        - number: 0x00056900
          description: VMware, Inc.
        - number: 0x0003FF00
          description: Microsoft
        - number: 0x001C4200
          description: Parallels
        - number: 0x00163E00
          description: XenSource
        - number: 0x8002700
          description: PCS System Technology GmbH
      - optional:
        - api: UuidCreateSequential


fix: typo in az iot hub route update docs

Description
I found a typo in the documentation of the az iot hub route update documentation.  (enpoint must obviously be endpoint).

iot


Add installation instructions for TypeScript users to readme.

Added installation instructions for package type definitions. (@types/coinbase-commerce-node)
With the increased adoption of TypeScript, I feel it is important to let users know that you do have type definitions available as it makes life a lot more convenient for most developers. 😅

@JorgenVatle, thanks! Just checked out @types/coinbase-commerce-node and opened a PR with a few minor changes. Happy to merge this after https://github.com/DefinitelyTyped/DefinitelyTyped/pull/35666 this merged.

Oh, awesome! 👍


MAISTRA-1079: Replace deprecated Kubernetes API's

You forgot to include the generated charts. Running make update-1.0-charts will do

You forgot to include the generated charts. Running make update-1.0-charts will do

Done.

Some weird things are showing up in the changeset now. Did we possibly forget to run the patch scripts for the last release?

Yeah, saw those too, left them in the PR on purpose, I thought there were changes in maistra/istio.
I'll remove them by hand now for this PR's sake and we should investigate what happened.

@dgn Found the issue! I had a previously downloaded tarball (I have no idea of its age) and thus download-charts.sh was not downloading it again. Removed this old file, ran the script again and it worked fine. Pushed the changes.

This PR seems to go beyond the immediate issues, I believe it's only the following which will likely impact us.
* apps/v1beta1

* apps/v1beta2
  The other changes should be removed from this PR.

@knrc extensions/v1beta1 for Deployments is mandatory as well.

Will close this one for now and open a new one when https://github.com/Maistra/istio/pull/79 is merged.

To me, this PR is preferable to what's in Maistra/istio#79, as it may make future merges with upstream more difficult and we'll need to make sure we make these same changes again should the charts change.  Also, the 1.0 changes will need to be in the maistra-1.0 branch of istio-operator as well, so they get pulled into resources/helm/v1.0 properly.


added support for other pythia 160m seeds

Description
I think I fixed whatever was causing the merge conflict. Added support for pythia-160m seeds 1, 2, 3
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have not rewritten tests relating to key interfaces which would affect backward compatibility

@will-hath  is this still a WIP?

Things should be good
From: Joseph Bloom @.>
Date: Sunday, August 20, 2023 at 4:57 PM
To: neelnanda-io/TransformerLens @.>
Cc: Will O Hathaway @.>, Mention @.>
Subject: Re: [neelnanda-io/TransformerLens] added support for other pythia 160m seeds (PR #361)
@will-hathhttps://github.com/will-hath is this still a WIP?
—
Reply to this email directly, view it on GitHubhttps://github.com/neelnanda-io/TransformerLens/pull/361#issuecomment-1685395767, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4KFI6XWIS4TEZE7CQWNDWDXWJ243ANCNFSM6AAAAAA22P2ZM4.
You are receiving this because you were mentioned.Message ID: @.***>

@will-hath Can you remove the poetry.lock changes? Otherwise this LGTM

Note that this is still classified as a draft PR, so we can't merge it in


Android build issue

Describe the bug
When doing Android builds I'm getting:
> Task :app:bundleStagingReleaseJsAndAssets FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:bundleStagingReleaseJsAndAssets'.
> kotlin.io.FileAlreadyExistsException: /Users/henrymoulton/Code/app/android/app/build/generated/assets/react/staging/release/index.android.bundle -> /Users/henrymoulton/Code/app/android/app/build/intermediates/bugsnag/rescued/Staging-release.android.bundle: The destination file already exists.

Running rm -rf android/app/build does the trick but I'm losing some caching ability I suspect.
It specifically mentions /intermediates/bugsnag/rescued/Staging-release.android.bundle: so that's why I've reported here.
Steps to reproduce

Run an android build
Run another android build without running ./gradlew clean or rm -rf android/app/build
See error

Environment

Bugsnag version: 7.10.5

Closing as we haven't been able to reproduce, but if you can share an example of how to reproduce the issue we'd be happy to re-open and investigate further.

This happens to me after every build. I have to manually delete the ./android/app/build/intermediates/bugsnag/rescued/Release.android.bundle file to be able to build again.

Can't share all the files, but the relevant lines in ./android/app/build.gradle are:
apply from: "../../node_modules/react-native/react.gradle"
apply plugin: "com.bugsnag.android.gradle"

bugsnag {
  uploadReactNativeMappings = true
  uploadJvmMappings = true
  uploadNdkMappings = true
  uploadNdkUnityLibraryMappings = false
  reportBuilds = true
  retryCount = 5
  failOnUploadError = true
}

Hi @rikur - unfortunately I've still been unable to reproduce the behaviour described. I've created a new app with npx react-native init and added @bugsnag/react-native v7.10.5, with the same lines you shared above above in my app's build.gradle.
If you're able to provide a cut down reproduction example we'd be happy to look into this for you 👍

Thank you for looking into this. I'll keep you updated on what I find.
Here's a stacktrace:
Caused by: kotlin.io.FileAlreadyExistsException: project/android/app/build/generated/assets/react/release/index.android.bundle -> project/android/app/build/intermediates/bugsnag/rescued/Release.android.bundle: The destination file already exists.
        at kotlin.io.FilesKt__UtilsKt.copyTo(Utils.kt:224)
        at kotlin.io.FilesKt__UtilsKt.copyTo$default(Utils.kt:217)
        at com.bugsnag.android.gradle.BugsnagUploadJsSourceMapTask$Companion$rescueReactNativeSourceBundle$1.execute(BugsnagUploadJsSourceMapTask.kt:206)
        at com.bugsnag.android.gradle.BugsnagUploadJsSourceMapTask$Companion$rescueReactNativeSourceBundle$1.execute(BugsnagUploadJsSourceMapTask.kt:152)```

I updated all bugsnag libraries to the latest version and the issue persists, with the same stacktrace.
info Direct dependencies
├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>info All dependencies
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>

Hi @rikur - could you let us know what version of the Bugsnag Android Gradle Plugin you're using too please?

Hmm, I take that back. Latest version fails with a different error after ./gradlew clean. Looking into it.

I have limited understanding of gradle, I hope this answers your question:
+--- project :bugsnag_react-native
|    +--- com.bugsnag:bugsnag-android:5.15.0
|    |    +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
|    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.72 -> 1.4.10
|    |    |    +--- org.jetbrains.kotlin:kotlin-stdlib-common:1.4.10
|    |    |    \--- org.jetbrains:annotations:13.0
|    |    +--- com.bugsnag:bugsnag-android-core:5.15.0
|    |    |    +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
|    |    |    \--- org.jetbrains.kotlin:kotlin-stdlib:1.3.72 -> 1.4.10 (*)
|    |    +--- com.bugsnag:bugsnag-plugin-android-anr:5.15.0
|    |    |    +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
|    |    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.72 -> 1.4.10 (*)
|    |    |    \--- com.bugsnag:bugsnag-android-core:5.15.0 (*)
|    |    \--- com.bugsnag:bugsnag-plugin-android-ndk:5.15.0
|    |         +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
|    |         +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.72 -> 1.4.10 (*)
|    |         \--- com.bugsnag:bugsnag-android-core:5.15.0 (*)
|    \--- com.bugsnag:bugsnag-plugin-react-native:5.15.0
|         +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
|         +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.72 -> 1.4.10 (*)
|         \--- com.bugsnag:bugsnag-android-core:5.15.0 (*)

Configured with:
classpath("com.bugsnag:bugsnag-android-gradle-plugin:5.+")

I may opt for using the bugsnag cli to upload source maps instead. Thank you again for looking into this :)

Hi @rikur, thanks for sharing that.

Latest version fails with a different error after ./gradlew clean. Looking into it.

Sounds like you're now seeing a different issue after upgrading and running ./gradlew clean - can you confirm if that's the case?

Yes, the different issue seemed to be related to gradle/kotlin version and I don't have the bandwidth to look into it more.
Uploading the source maps via the CLI works great.

Thanks for the update, we will close this issue for now as it sounds like you have a workaround and we were not able to reproduce. Happy to reopen if any new information comes to light.

Sounds like you're now seeing a different issue after upgrading and running ./gradlew clean - can you confirm if that's the case?

rm -rf app/build/intermediates/bugsnag/rescued/filename.bundle before running the script seems to work

[15:21:44]: ▸ FAILURE: Build failed with an exception.
[15:21:44]: ▸ * What went wrong:
[15:21:44]: ▸ A problem was found with the configuration of task ':app:uploadBugsnagStaging-releaseSourceMaps' (type 'BugsnagUploadJsSourceMapTaskGradle53Plus').
[15:21:44]: ▸ > File 'path/android/app/build/intermediates/bugsnag/rescued/Staging-release.android.bundle' specified for property 'bundleJsFileProvider' does not exist.

New error actually as a result of this so perhaps @rikur was right to disable sourcemaps in gradle and use the CLI.

We'd still be interested in trying to determine the root cause of this if you're able to share a cut down reproduction example project which demonstrates the issue, as we've so far been unable to reproduce. Feel free to email to<EMAIL_ADDRESS>if you prefer.

Hi @henrymoulton, @rikur  - I've just managed to reproduce the FileAlreadyExistsException by enabling Hermes in an example app, which wasn't something we tried initially. Could you confirm that you have Hermes enabled in your projects?
We're going to look into a fix for this - will keep you updated here.

Yep using Hermes

I have the same issue, using Hermes as well

Thanks for the info. We've scheduled a fix for this early next month.

@mattdyoung - I just tried 7.16.0 and I'm still getting the same error, do you perhaps know when the fix will be in?

Hi @madandrija - we're working on getting the fix released. We'll let you know here as soon as it's available.

Hi all,
Just letting you know we have now fixed this issue in v5.8.2 of our gradle plugin.
Let us know if you have any issues once you upgrade to this version.

@johnkiely1 Any plans to cherry-pick this fix to v7 while it's recommended by docs?

Hey @luke-belton will this fix be applied to v7 too? I'm also experiencing the same.

Hi @hardworker, @bradleyflood.
We are currently investigating porting this fix to v7 to see whats involved. We will let you know of updates as soon as we can.

Hi all,
We've just released this fix in v7.4.0 of the plugin. Do let us know if you see any issues when you upgrade to it.

Epic. Thank you so much @johnkiely1  it works! :)


Is is required to call indexDoc before search

Riot version (or commit ref):0.10.0
Go version: 1.14.1

Operating system and bit:Windows 10 64bit

I copied the following code from example and add some options in Riot. It works well if I call Index before query. In reality, all the text data will be read from database, and make indexes if required. But I found if I set variable 'buildIdx' to false and run again, nothing is returned. Does that mean I have to query database and make indexes before query?
package main

import (
	"log"

	"github.com/go-ego/riot"
	"github.com/go-ego/riot/types"
)

var (
	searcher = riot.Engine{}
)

func main() {
	searcher.Init(types.EngineOpts{
		Using:       3,
		GseDict:     "zh",
		StoreFolder: "./indexes",
		UseStore:    true,
	})
	defer searcher.Close()
	buildIdx := true

	if buildIdx {
		text := "《复仇者联盟3：无限战争》是全片使用IMAX摄影机拍摄"
		text1 := "在IMAX影复仇院放映时者"

		searcher.Index("你好,世界", types.DocData{Content: text})
		searcher.Index("2", types.DocData{Content: text1}, false)
		searcher.Flush()
	}

	// 搜索输出格式见 types.SearchResp 结构体
	log.Print(searcher.Search(types.SearchReq{Text: "复仇者"}).Docs)
}

Sorry, I finally got it, I need to call searcher.Flush().


Installation problem for Windows 10 but not Mac OS X: figures not appearing inline with PyPi installation

Would you kindly push the new fixed inline plotting code fix of imatlab to PyPi server?
A colleague wanted to install the imatlab kernel in python on his Windows 10 machine and when trying to install from GitHub  with (python -mpip install git+https://github.com/imatlab/imatlab) it generates an error (this error was not resolved after upgrading setuptools and ez_setup):
"command "python setup.py egg_info" failed with error code 1 in C:\Users....."
However he is able to install from PyPi, but we believe it's the older version that did not have the inline plotting fix, and has the same errors generated as in the last revision.
Thanks!

I can push a new release of course, but would like first to know what error he is getting (may just be git not being installed, or something else...).

imatlab 0.3 released.


Eager activity execution should be limited to 3 per workflow task completion by default

In #946 we set MaxConcurrentActivityExecutionSize default to 3, instead we should have set the max number of eager activities requested per workflow task completion to 3.
I wish we hadn't exposed MaxConcurrentActivityExecutionSize and DisableEagerActivities, next time when we add new options we should mark them experimental, but in this case I think it might be okay to remove these, risking a breaking change.
In other SDKs, we've allowed disabling eager activity execution per activity invocation via activity options and have not yet exposed a way to configure the number of eager activity requests, it's hardcoded to 3.
I would like us to expose a way to configure the number of eager activity requests eventually but would rather wait until we know more about the implications of turning this on before exposing so we can document how to tune that parameter.
Since eager activities are still disabled by default server side that should be enough for now.

In https://github.com/temporalio/sdk-go/pull/946 we set MaxConcurrentActivityExecutionSize default to 3, instead we should have set the max number of eager activities requested per workflow task completion to 3.

It is MaxConcurrentEagerActivityExecutionSize and I still it is best IMO. The whole point of having this throttle was to not overload the worker or unexpected consequences of doing thousands of these things. If you made this per workflow, you're only stopping the rare case of ExecuteActivity called 3+ times before either is waited. That's hardly ever. In the meantime, I'd start 1000 workflows that each start an activity, and then they would all be eager. It would make the option useless.
We have MaxConcurrent settings for activities, local activities, sessions, pollers, tasks, and now eager activities. These are consistent and across the worker. It'd be inconsistent and almost worthless to do per workflow or per workflow task.

I wish we hadn't exposed MaxConcurrentActivityExecutionSize and DisableEagerActivities

Why not? These are perfectly valid settings and are inline with other settings this SDK uses. They both have worker-level value. I'd like to see these options in every SDK. Everyone should be able to set max concurrent and disable eager activities at the worker level. Remember however that 0 in Go is "unset" which means "default" and we don't want to default to disabled. That's why there are two settings.

next time when we add new options we should mark them experimental

We don't usually do this with new option additions, e.g. https://typescript.temporal.io/api/interfaces/common.ActivityOptions#alloweagerdispatch. Are you saying all SDKs should have marked them experimental, or are you saying Go uniquely did something wrong here?

but in this case I think it might be okay to remove these, risking a breaking change.

Concur, these are unused and such a breaking change to these options is fine. Can you clarify exactly what they should change to?

I would like us to expose a way to configure the number of eager activity requests eventually but would rather wait until we know more about the implications of turning this on before exposing so we can document how to tune that parameter.

I remember being asked for this explicitly when I developed this feature, but maybe I wasn't supposed to make it user overridable? It would be a deviation from the norm with all the other configuration options we create on new features like this. I think Go was the first SDK to offer a user-facing API for eager activities so there wasn't a model to follow that I recall. Why does how one tunes the default affect whether the user-overridable option exists? We can remove and hardcode if we must.

After discussion, we just need to remove the "3" default from max concurrent eager (so leave that as unlimited) and hardcode a new setting of max 3 per workflow task.


chore: create prefixed amplify-prompts

Description of changes
We have consumers of amplify-prompts outside of this repository.
Therefore creating @aws-amplify/prompts as "an alias" to amplify-prompts so that they can migrate before we discontinue amplify-prompts.

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

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

Are changes in yarn.lock needed? If not, LGTM

Kamil please weigh in here but I don't think so. We are essentially aliasing the amplify-prompts to @aws-amplify/amplify-prompts

Are changes in yarn.lock needed? If not, LGTM

They are not. We don't bring any external dependency here. Nor we're changing dep graph layout, i.e. @aws-amplify/prompts -> amplify-prompts nodes in graph are equivalent to single amplify-prompts .


unitTestSupport.compareArray() only checks for first 3 columns

Describe the bug
def compareArray(trueStates, dataStates, accuracy, msg, testFailCount, testMessages):
    """Compare two arrays size and values and check absolute accuracy"""

only checks for first 3 columns.
To reproduce
from Basilisk.utilities import unitTestSupport
import numpy as np
#some mock data
wheelSpeeds1 = np.ones((500,4))
wheelSpeeds2 = np.ones((500,4))
#add a 0.01 error
wheelSpeeds2[:,3]+=0.01
accuracy = 0.001

testFailCount = 0
testMessages = []
[testFailCount, testMessages] = unitTestSupport.compareArray(wheelSpeeds1, wheelSpeeds2, accuracy, "error message", testFailCount, testMessages)
print(testFailCount, testMessages)

Expected behavior
This should have caused compareArray to fail. I think the documentation doesn't make it clear enough that only first 3 columns are checked? Same with unitTestSupport.compareArrayRelative().
Instead of if not isArrayEqual(dataStates[i], trueStates[i], 3, accuracy): , should it be if not isArrayEqual(dataStates[i], trueStates[i], len(trueStates[i]), accuracy): in compareArray's implementation?

My suggestion here is that you migrate off of unitTestSupport for numerical comparisons. There are more appropriate packages like Numpy Testing.

@patkenneally  Will try numpy testing, thanks!


chore(icon): accessibility checklist

✅ Closes: COMUI-3328

Added a follow-on ticket here but I'm not sure that it merits failing the component since it works generally https://inindca.atlassian.net/browse/COMUI-3359


MINOR: [Python][CI] Add DYLD_LIBRARY_PATH to avoid requiring PYARROW_BUNDLE_ARROW_CPP on macOS job

Could you open a Jira issue because this pull request needs more changes?
The following log shows that import _json in the json library imports pyarrow's _json not the json library's one:
https://github.com/apache/arrow/actions/runs/3470117791/jobs/5797917201#step:6:481
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/scanner.py", line 5, in <module>
    from _json import make_scanner as c_make_scanner
  File "pyarrow/_json.pyx", line 1, in init pyarrow._json

I think that we can fix it by preparing a proper path. I'll take a look at this on my local M1 Mac. Please wait for a while.


Switchto babel preset env

Fixes: #1667
Possible fix for: #1665
Possible fix for: #1570
What I did
I changed all usages of babel-preset-es20xx to babel-preset-env
I also added babel-minify
And I enabled webpack tree-shaking
How to test
Run the examples
Build the static versions of the examples

So do we use both babel-minify and UglifyJS at the same time? Why?

I'd be interested in running prepack on the bundle too.

babel-minify operates per file, and removes dead code

So it's only for dead code in terms of file, unused exports can't be eliminated that way because it runs before webpack, right?

Thanks everyone for reviewing.
I hope you also feel this PR is an improvement. maybe not quite perfect.
We should improve the method of configuring webpack and babel.


Error while setting up securitas platform for alarm_control_panel

For some reason, this integration has broken lately. I am not sure if it has anything to do with the latest HA patches... It stopped working for me approximately 1 week ago, it has been working fine for the past months (since the last breaking patch of the await update)
This is the error I can find in the logs:
Error while setting up securitas platform for alarm_control_panel
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 320, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/securitas/alarm_control_panel.py", line 58, in async_setup_entry
    current_state: CheckAlarmStatus = await client.update_overview(
                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/securitas/__init__.py", line 407, in update_overview
    alarm_status = await self.session.check_alarm_status(
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/securitas/securitas_direct_new_api/apimanager.py", line 486, in check_alarm_status
    response: ClientResponse = await self._execute_request(
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/securitas/securitas_direct_new_api/apimanager.py", line 127, in _execute_request
    error_login: bool = await self._check_errros(response_text)
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/securitas/securitas_direct_new_api/apimanager.py", line 149, in _check_errros
    response = json.loads(value)
               ^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

BTW, there is also a warning, but not sure if it's a consequence of the error, or the cause or completely unrelated.
Unable to find services.yaml for the securitas integration

The issue is that the code is trying to parse some json but is not a json file. You can enable the debug output to see the requests and response for the Securitas Direct API and then try to figure out what would be the issue.

The issue is that the code is trying to parse some json but is not a json file. You can enable the debug output to see the requests and response for the Securitas Direct API and then try to figure out what would be the issue.

Where  I'm supposed to view such debug logs?

In System > Logs I can't see anything related Securitas direct plugin

I have reconfigured it and I find in the debug that there is an error authenticating (but it asks for the 2FA and it sends the right code and everything seems right
2023-06-17 21:06:36.030 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] Making request with device_id e2f704e1575f4ce6, uuid e2f704e1575f4ce6 and idDeviceIndigitall
2023-06-17 21:06:36.131 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] --------------Response--------------
2023-06-17 21:06:36.131 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] {"data":{"xSCheckAlarmStatus":{"res":"WAIT","msg":"Petición en proceso","status":null,"numinst":null,"protomResponse":null,"protomResponseDate":null}}}

2023-06-17 21:06:38.263 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] --------------Response--------------
2023-06-17 21:06:38.263 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] <META NAME="ROBOTS" CONTENT="NOINDEX, NOFOLLOW"><meta name="format-detection" content="telephone=no"><meta name="viewport" content="initial-scale=1.0"><meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1"> Request unsuccessful. Incapsula incident ID:<PHONE_NUMBER>56145721
2023-06-17 21:06:38.266 ERROR (MainThread) [homeassistant.components.alarm_control_panel] Error while setting up securitas platform for alarm_control_pane

I can't understand whats going on

I've tried configured via configuration.yaml and the only difference I see is that it goes straight to the 2FA in the device configuration system, but after that, throw the same errors, and it doesn't configure.
The weird thing is that I have not changed anything in my installation, and it has been working with this integration for the past                 months without issues. Even more weird is that anyone has reported yet the same scenario as me.

@SirLouen, I can confirm that I am having the same issue.
I have removed and reconfigured the integration twice, on latest HA (2023.6.3) and once configured I see the same errors as you and only '1 x Entry' on the integration page, no devices or entities.

@zav13r I have an automation set with this integration, that it connects every day at 00.00 and disconnects every day at 8.00
So basically I can spot when things start going weird pretty soon because I use this every single day.
But the thing is that I did not have much time last 2 weeks to research the issue and go into logs to report earlier.
From what I've read in the logs, it seems that despite the first steps of the login procedure are being done and the first login with user/pass goes well, the 2FA seems to be failing and after multiple 2FA login intents, the Securita's Incapsula firewall blocks any future intent.
I'm not 100% confident, maybe they have modified the login method, so the plugin has to be updated.

@guerrerotook is your setting working? This issue is not generalized?

I have no idea about this, I can have my own installation working with the latest version. How do you know about this login issue?  Sometime it fails because of that

I have no idea about this, I can have my own installation working with the latest version. How do you know about this login issue?  Sometime it fails because of that

What is a managed user and the main user?
I'm using the main user (the owner)
It says that the configuration is created but not real entities appear

Have you tried to reconfigure your integration from scratch? Maybe you are living on an old token.

Little extra info
From the four installations the only one that has the idService = 11 it's the one working
{"id":"8","idService":"11","active":true,"visible":true,"bde":false,"isPremium":false}
So digging into the code, I've set in the line
https://github.com/guerrerotook/securitas-direct-new-api/blob/a5bd47ca96daff3f8211518bd3b87cac46b76861/custom_components/securitas/securitas_direct_new_api/apimanager.py#L474
A debug line to see which installation is being referenced, as I exactly expected, it's referencing only one of the  legacy installations that doesn't have an alarm anymore
2023-07-26 14:23:26.071 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] --------------INSTALLATION ID--------------
2023-07-26 14:23:26.071 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] 33117
I have contacted Securitas, to see if they could clean up the installations I don't use anymore (except for two). In this case I have an special installation because of a minor issue I had to have with Securitas (for invoicing purposes). So I have to keep one active installation and one dummy installation permanently and I cannot ditch it unfortunatelly yet.
But after cleaning the other two installations that were hindering the system, now new logs appear:
First appear the current active alarm, and all services and devices appear in the logs, as it should be. Perfect. The logs end like this:
2023-07-26 14:56:35.836 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] {"data":{"xSCheckAlarm":{"res":"OK","msg":"Su solicitud ha sido enviada","referenceId":"OWA"}}}

After that it jumps into the second installation and after an endless loop, it crashes at it happened before.
So now I can say 100% for sure that the system is faulty when multiple installations are involved (and not all have an alarm).
A possible super-fast solution could be, by simply enabling optionally to set the installation ID on the configuration.xml file and picking it from there (and not anywhere else) in case it has been set.
I'm going to keep digging into the code to see if I can avoid the dummy installation to be called, but I've tried to trick the code and since I'm not confident with data types in Python, so far I have not been able to hard-code this at least to make it work temporarily (if someone could give me a hard-code fix it would be great to make it work until @guerrerotook finds a way to perma-solve this). Anyway if I find anything I will report back.

Final update: It has been semi-automatically sorted out.
In fact, both installations now appear in the entities' table which is even weirder (because in the past, only one showed up)
How so? Not 100% sure but here some steps:
Firstly, I modified all the strings that said "instalation" instead of installation

But not 100% confident this was the solution or it has been pure coincidence.
I was trying to debug the update_overview function to see if I could stop the calls to the dummy installation. But after restarting HA and reconfiguring the integration, it suddenly popped not only the right installation but also the dummy installation...
The installations part is definitely buggy. Probably the idea of using "INSTALATION" with one L in the code has not been the best idea.
So far so good, I've tried restarting the system and it still works, so I'm not going to touch it more until @guerrerotook returns 😝

Let me take a look at all of this!

Thanks for all the detailed information. The integration when is setup for the first time, list all the installations that the user has (in my case for example two), by calling this method here, https://github.com/guerrerotook/securitas-direct-new-api/blob/a5bd47ca96daff3f8211518bd3b87cac46b76861/custom_components/securitas/init.py#L187 Then for each for the installation found on the system it will create a new SecurirtasDirectDevice.
So, in your scenario what would happen is that you should have multiple alarm panel in your installation. Check your home assistant installation and tell me what do you find.

Thanks for all the detailed information. The integration when is setup for the first time, list all the installations that the user has (in my case for example two), by calling this method here,
https://github.com/guerrerotook/securitas-direct-new-api/blob/a5bd47ca96daff3f8211518bd3b87cac46b76861/custom_components/securitas/__init__.py#L187
Then for each for the installation found on the system it will create a new SecurirtasDirectDevice.
So, in your scenario what would happen is that you should have multiple alarm panel in your installation. Check your home assistant installation and tell me what do you find.

Yep, that was broken
It did not list all the installations until I did the mod. It just got stucked as I showed on the logs trying to retrieve info.
After I did the mod, it showed the two installations (because I deleted the other two by calling the Securitas service because I thought this could solve the issue)
The only difference I can say is this:
I had 4 installations
Two installations with alarm, and two installations without
I turned down one of the two installations with alarm some months ago (March). But maybe they did not deactivate it until May (and days after I created this bug). So maybe this disabled installation within my control panel was creating some kind of issue.
But after they disabled my other two installations, I kept having this issue, until I modified the "instaLlation" thing I comment here
https://github.com/guerrerotook/securitas-direct-new-api/issues/137#issuecomment-1651906840
Definitely this is buggy and by the logs I can clearly see this that this is not being handled
Those WAIT "Petición en proceso"
2023-07-26 13:02:23.663 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] Making request with device_id knCuPS
2023-07-26 13:02:23.763 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] --------------Response--------------
2023-07-26 13:02:23.763 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] {"data":{"xSCheckAlarmStatus":{"res":"WAIT","msg":"Petición en proceso","status":null,"numinst":null,"protomResponse":null,"protomResponseDate":null}}}

Should be handled before it returns (maybe like 5 retries or something like that before stopping)
2023-07-26 13:02:40.419 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] Making request with device_id knCuPS
2023-07-26 13:02:40.493 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] --------------Response--------------
2023-07-26 13:02:40.493 DEBUG (MainThread) [custom_components.securitas.securitas_direct_new_api.apimanager] <META NAME="ROBOTS" CONTENT="NOINDEX, NOFOLLOW"><meta name="format-detection" content="telephone=no"><meta name="viewport" content="initial-scale=1.0"><meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1">Request unsuccessful. Incapsula incident ID: 1732655
2023-07-26 13:02:40.497 ERROR (MainThread) [homeassistant.components.alarm_control_panel] Error while setting up securitas platform for alarm_control_panel

The thing is that the alarm cannot configured if one of the installations doesn't have an alarm. Or maybe the "instaLlation" solves it. I'm not 100% sure because as I said, I don't have python skills to progress on this. These are the two installations after I modified that:

But do you have like two panels where you can arm or disarm your alarms?
Something like this?

I don't have two panels configured on Lovelace. Only one
Still I won't be able to use one becuase one of the two installations doesn't have an alarm.

Te explico en Español porque no nos estamos enterando de nada. Por lo general, te escribo en inglés para que otros puedan seguir la conversación (pero como se ve que no le afecta a casi nadie porque es un poco raro este escenario y a los pocos que les afecto, pues le debieron de dar la patada a la integración y la olvidaron, pues tampoco nos vamos a volver locos), y dadas las circunstancias creo que es más importante ir al tiro para ver si le damos salida a este problema.
Yo tenía 4 instalaciones.

2 instalaciones tenían alarma contratada
2 instalaciones eran una instalación virtual sin alarma, solo para facturación (una chapuza de Securitas Direct)

En Abril o Mayo, me dieron de baja una de las alarmas de las instalaciones con Alarma. Creo que ese fue el momento en el que la integración empezó a darme problemas porque pase a tener:

1 Instalación con alarma contratada
1 instalación dummy, sin alarma y sin nada (la que di de baja)
2 instalaciones virtuales para facturación

La semana pasada que fue cuando ya decidí coger el toro por los cuernos y revisar tu código, lo primero que hice fue llamar a Securitas para que eliminaran 1 instalación virtual para facturación y la instalación dummy y entonces me quede con:

1 Instalación con alarma contratada
1 instalación virtual para facturación

Esto es a nivel de instalaciones. Porque el problema está 1000% en el código que se encarga de analizar las instalaciones.
Dicho esto, a pesar de dar de baja las que ya no usaba, seguía dando el problema que te explicaba antes: tu integración configuraba bien la instalación con alarma contratada, pero luego revisaba la instalación virtual se quedaba en un bucle de WAIT y al final moría, quedándose toda la integración sin configurar y sin entidades ni objetos ni nada. La configuración de la integración moría y se quedaba en un estado inerte (que fue lo que te pase el 9 de Junio, un error en el JSON porque basicamente el error final no es un JSON es un HTML que tira Securitas cuando después de intentar configurarse comprobando JSON vacios 12 veces, te bloquea el sistema).
¿Qué solución tomé?
Teniendo más claro que el agua que el bug viene del código que gestiona las instalaciones, me dispuse a revisar tu código y me di cuenta que en los strings ponias instalation con una sola L en vez de installation. Así que entendí que lo mismo al revisar los campos del JSON que te devuelve securitas, no estaba leyendo bien el campo que corresponde a la instalación. Después de arreglar el string, funcionó.
Pero como no entiendo el código a la perfección, no estoy 100% seguro si fue cambiar ese string, o que casualmente se arregló todo por repetir el intento de configuración como unas 500 veces como hice que la API de Securitas devolvió algo novedoso por arte de magia.
Dicho esto:
Ahora mismo que todo me funciona bien, tengo dos instalaciones en la integración:

Pero como te he explicado una es totalmente inútil, es la instalación virtual que usan los de Securitas para facturación (una chapuza que se inventaron para resolverme un problema de facturación). Por lo tanto, no puedo armarla y desarmarla, así que no tiene sentido que le meta una tarjeta en Lovelace. Solo tengo la tarjeta de Lovelace de la instalación que sí puedo manejar porque si tiene una alarma de verdad.
Conclusión:
Ahora mismo me funciona todo, sí, pero hay que tener presente que existe un bug en el código. No sé exactamente cuál es. ¿Puede sea lo del string de instalation? Podría ser. Te he dado todas las pistas que he encontrado después de intentar echar un ojo a un código en Python que no entiendo demasiado bien. Lo dicho, si quieres ver algo del API mi Telegram es @mkt4p
También te adjunto mi fichero init.py con la pequeña modificación que le hice por si te sirve de algo (no me deja subir ficheros python y tampoco te lo quiero pegar aquí te lo paso por paste bin:
https://pastebin.com/TdWT4uJb


Show Unread Channels in Channel Selector when input is empty

Summary
As a user, I want to press CMD+K and see the channels I'm in with unread messages at the top of the list, so that I can quickly switch to that channel.
CMD+K shows unread channels, if any. Unread channels are filtered out if the user starts typing anything.
NOTE: @jasonblais this could go under a config value to enable/disable.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Has UI changes

Thank you @stanchan! Love it.
We would definitely be open to taking this improvement. It was actually on our list of improvements to make.
I'll queue this for our UX team to review.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-09394c55e84f98c46.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-09394c55e84f98c46

Rebased to latest from master and fixed conflicts.

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spintmint test server creation failed. See the logs for more information.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Sorry for the delay:
We discussed this in the UX channel, along with Link.
First, I wanna say that we're all very excited for this feature! We've wanted to add this for a while so we really appreciate this contribution. Just two pieces of feedback and this should be good.

Propose we match the sorting of unread channels to how unreads are sorted in the channel sidebar when ExperimentalGroupUnreadChannels is enabled. That is, channels with unread mentions first, followed by unread channels.

https://github.com/uber-uchat/mattermost-webapp/blob/dcbbd694e842d52a7044742cf3c9958e1ce706fe/components/suggestion/switch_channel_provider.jsx#L81

1b) A nice-to-have improvement would be to sort

direct message channels, including multi-party direct message channels, with mentions first, then
channels with mentions, then
all channels with unreads

Those channels with unread mentions (e.g. autem and town-square in the screenshot above), can we add a mention badge on the right side of the autocomplete suggestion.

We can use the same colours as used for the square G that indicates the channel is a group message. I can grab one of our designers if that colour choice doesn't work.

And of course, let me know if you have any questions :)

@jasonblais thank you for the call today and answering my question. Here are the changes requested and rebased to the latest from MM master. The order is changed to be the same as the unreads on the sidebar. Also added the mention count badges.
This is how it looks:

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0ae890b2c103470a5.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-0ae890b2c103470a5

Spinmint test server destroyed

@saturninoabril @jasonblais rebased to latest from master and made the requested changes.

Note for QA: No JIRA ticket for this one, but I tested just now on pre-release and it is working as described above. Added to release tests.


Testing for Find Tasks, Find Modules, AddExam, EditExam commands

Codecov Report

Merging #254 (ea98432) into master (bae868c) will increase coverage by 7.81%.
The diff coverage is 92.30%.

@@             Coverage Diff              @@
##             master     #254      +/-   ##
============================================
+ Coverage     39.21%   47.02%   +7.81%
- Complexity      534      649     +115
============================================
  Files           160      160
  Lines          3198     3207       +9
  Branches        419      419
============================================
+ Hits           1254     1508     +254
+ Misses         1870     1614     -256
- Partials         74       85      +11

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...c/main/java/seedu/address/model/exam/ExamDate.java
82.35% <66.66%> (+62.99%)
:arrow_up:

.../seedu/address/logic/commands/EditExamCommand.java
98.27% <100.00%> (+98.27%)
:arrow_up:

...edu/address/logic/commands/ListModulesCommand.java
100.00% <100.00%> (+100.00%)
:arrow_up:

...du/address/logic/parser/EditExamCommandParser.java
100.00% <100.00%> (+100.00%)
:arrow_up:

.../java/seedu/address/logic/commands/AddCommand.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/seedu/address/logic/parser/AddCommandParser.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/seedu/address/logic/commands/DeleteCommand.java
25.00% <0.00%> (-41.67%)
:arrow_down:

...c/main/java/seedu/address/model/person/Person.java
66.66% <0.00%> (-23.81%)
:arrow_down:

...java/seedu/address/logic/commands/FindCommand.java
33.33% <0.00%> (-22.23%)
:arrow_down:

... and 29 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I think there is some merge conflict with the typical task and typical index part


Problems setting correct EXT-X-TARGETDURATION via hls_fragment and hls_window configuration

Description
I want to configure my EXT-X-TARGETDURATION with value 5. My segment needs to have  max 5 seconds, and I tried may values for hls_fragment and hls_window. My minimum value is always EXT-X-TARGETDURATION: 9. Is there any limitation?

SRS Version: 4.0

SRS Log:

[2023-05-04 17:29:14.081][Trace][1][2p45mgj7] start publish mr=0/350, p1stpt=20000, pnt=5000, tcp_nodelay=0
[2023-05-04 17:29:14.087][Trace][1][2p45mgj7] got metadata, width=1280, height=960, vcodec=7
[2023-05-04 17:29:14.087][Trace][1][2p45mgj7] 46B video sh,  codec(7, profile=Baseline, level=3.2, 1280x960, 0kbps, 0.0fps, 0.0s)
[2023-05-04 17:29:15.563][Trace][1][2p45mgj7] -> HLS time=100113204ms, sno=146, ts=livestream-145.ts, dur=2866ms, dva=0p
[2023-05-04 17:29:17.122][Trace][1][29f575ni] Hybrid cpu=19.02%,15MB, cid=4,1, timer=61,0,0, clock=0,30,16,1,1,0,1,0,0, free=1, objs=(pkt:298,raw:266,fua:31,msg:1120,oth:1,buf:1)
[2023-05-04 17:29:22.122][Trace][1][29f575ni] Hybrid cpu=18.98%,15MB, cid=4,1, timer=61,0,0, clock=0,30,16,1,1,0,1,0,0, free=1, objs=(pkt:298,raw:266,fua:31,msg:1120,oth:1,buf:1)
[2023-05-04 17:29:25.565][Trace][1][2p45mgj7] -> HLS time=110117297ms, sno=162, ts=livestream-161.ts, dur=3600ms, dva=0p
[2023-05-04 17:29:27.122][Trace][1][29f575ni] Hybrid cpu=17.00%,15MB, cid=4,1, timer=61,0,0, clock=0,29,17,1,0,0,0,0,0, objs=(pkt:381,raw:340,fua:39,msg:1430,oth:1,buf:1)
[2023-05-04 17:29:32.126][Trace][1][29f575ni] Hybrid cpu=20.00%,15MB, cid=4,1, timer=61,0,0, clock=0,29,17,1,0,0,0,0,0, objs=(pkt:381,raw:340,fua:39,msg:1430,oth:1,buf:1)
[2023-05-04 17:29:35.588][Trace][1][2p45mgj7] -> HLS time=120134670ms, sno=178, ts=livestream-177.ts, dur=1934ms, dva=0p
[2023-05-04 17:29:37.126][Trace][1][29f575ni] Hybrid cpu=19.98%,15MB, cid=3,0, timer=60,0,0, clock=0,28,17,1,1,0,0,0,0, objs=(pkt:423,raw:376,fua:45,msg:1582,oth:1,buf:1)
[2023-05-04 17:29:39.083][Trace][1][2p45mgj7] <- CPB time=20002032, okbps=1,0,0, ikbps=3233,0,0, mr=0/350, p1stpt=20000, pnt=5000
[2023-05-04 17:29:42.126][Trace][1][29f575ni] Hybrid cpu=14.97%,15MB, cid=3,0, timer=60,0,0, clock=0,28,17,1,1,0,0,0,0, objs=(pkt:423,raw:376,fua:45,msg:1582,oth:1,buf:1)
[2023-05-04 17:29:45.598][Trace][1][2p45mgj7] -> HLS time=130148717ms, sno=193, ts=livestream-192.ts, dur=4300ms, dva=0p

SRS Config:

vhost __defaultVhost__ {
    hls {
        enabled         on;
        hls_fragment    2;
        hls_window      120;
    }

Replay
Please describe how to replay the bug?
Step 1: Run docker, copying the docker.conf from my file, with the configuration above
docker run --rm -it -p 1935:1935 -p 1985:1985 -p 8080:8080 \
    -v /Users/cynthia.tong/globo/poc-rtmp-lite/docker.conf:/usr/local/srs/conf/docker.conf ossrs/srs:4 ./objs/srs -c conf/docker.conf

Step 2: run ffmpeg streaming
ffmpeg -r 30 -f lavfi -i testsrc -vf scale=1280:960 -vcodec libx264 -profile:v baseline -pix_fmt yuv420p -f flv rtmp://localhost/live/livestream

Step 3: curl localhost with wery string
➜ curl "http://localhost:8080/live/livestream.m3u8"
#EXTM3U
#EXT-X-STREAM-INF:BANDWIDTH=1,AVERAGE-BANDWIDTH=1
/live/livestream.m3u8?hls_ctx=223b622j%
➜ curl "http://localhost:8080/live/livestream.m3u8?hls_ctx=223b622j"
#EXTM3U
#EXT-X-VERSION:3
#EXT-X-MEDIA-SEQUENCE:34
#EXT-X-TARGETDURATION:9
#EXTINF:8.300, no desc
livestream-34.ts
#EXTINF:8.300, no desc
livestream-35.ts
#EXTINF:8.300, no desc
livestream-36.ts
#EXTINF:8.300, no desc
livestream-37.ts
#EXTINF:8.300, no desc
livestream-38.ts
#EXTINF:8.300, no desc
livestream-39.ts
#EXTINF:8.300, no desc
livestream-40.ts
#EXTINF:8.300, no desc
livestream-41.ts
#EXTINF:8.300, no desc
livestream-42.ts
#EXTINF:8.300, no desc
livestream-43.ts
#EXTINF:8.300, no desc
livestream-44.ts
#EXTINF:8.300, no desc
livestream-45.ts
#EXTINF:8.300, no desc
livestream-46.ts
#EXTINF:8.300, no desc
livestream-47.ts
#EXTINF:8.300, no desc
livestream-48.ts

Expect
I believe we should have a smaller EXT-X-TARGETDURATION and EXTINF. Maybe EXT-X-TARGETDURATION:5 and EXTINF:4.8

SRS has a default setting where it waits for a complete GOP of frames before closing a TS segment. This means that if you set hls_fragment to 2, but your GOP is 5 seconds long, the resulting segment may exceed the 5-second limit.
To reduce the size of the GOP, you can modify the encoder settings. For instance, in FFmpeg, you can set the GOP to 2 seconds by using the command -r 25 -g 50 by -g frames.
You can also configure SRS to not wait for a complete GOP before closing a TS segment file by adjusting thehls_wait_keyframe setting.
        # whether wait keyframe to reap segment,
        # if off, reap segment when duration exceed the fragment,
        # if on, reap segment when duration exceed and got keyframe.
        # default: on
        hls_wait_keyframe       on;


Fixes validation of .workflow

Adds validation rules for .workflow files

Hello @JayjeetAtGithub! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file cli/popper/gha.py:

Line 67:80: E501 line too long (108 > 79 characters)
Line 68:77: W291 trailing whitespace

@ivotron Can you please tell me what logic will be in the validate function if the validation is handled in normalize function ?

hey @JayjeetAtGithub

As of now , before normalizing , all the validation errors are being catched in validate() and only if everthing is valid , we move on to normalizing in normalize()

you're right. What I meant in my review was that, we wouldn't be doing any type-checking as part of the validate() method that you're implementing. Instead we would do type-checking in the normalize function. Maybe we can rename it to normalize_types() or something like that. And your method could be validate_syntax or parse?

hey @JayjeetAtGithub . Is there anything you would like to discuss regarding this? Thanks!

@ivotron Can you please tell what will be in there in validate_syntax() , since what my pr adds is only type checking , and if i do that in  normalize_types() , then i cant understand what validate_syntax() / parse() will do ? Do i need to add some other logic ?

@JayjeetAtGithub validate_syntax will only check for the presence of required elements, as specified in #504 . The normalize_types will look at types

Thanks a lot @JayjeetAtGithub !  There are a couple of pep8 issues that pep8speaks is complaining about. Also, could you please rebase master again? Sorry 😩

thanks a lot for your patience @JayjeetAtGithub !


Applying more than one interactive condition on the feedback image yields an app client side error

Cause of the issues is found: interactive element was not present on the canvas.
Proposed fix: make sure the error does not lead to a fatal clientside crash. Add a help text at the feedback image object that notes that elements should be present at the storylinesection in wagtail.

Fallback has been made, also en extra help text is added
Please make sure the parameter is present in this section. Otherwise you assert something that doesn't exist


RAG Integration

We're using the flutter_gemma package in our Flutter project and appreciate its features for on-device model execution, cross-platform support, and ease of use.
I’m curious to know if there are any plans to integrate advanced features such as Retrieval-Augmented Generation (RAG). RAG could potentially improve the functionality of the Gemma models by combining retrieval-based methods with generation, enhancing the accuracy and relevance of responses.

I've reviewed the package code and determined that Retrieval-Augmented Generation (RAG) is not related to the flutter_gemma package. RAG needs to be implemented separately, so I'll be closing this issue for now.


Package Manager - Change deployment strategy from RollingUpdate to Recreate

No downtime upgrades are not supported at this time

https://github.com/rstudio/helm/blob/f30fe61271de9a7f7a7a748add09f0c9863c99ba/charts/rstudio-pm/values.yaml#L46-L51


Add result shorthands for Process fakes

This adds two new shorthands for faking process results in tests. The first is a shorthand to simply set the exit code. The second is a shorthand to throw an exception.
Process::fake([
    'php -l script.php' => 255,
    'cat README.txt' => new \RuntimeException('fake exception message'),
]);

Note: while the latter is not common as it is only possible for the underlying Symfony Process to throw a RuntimeException. This path was hard to test unless a fake process can throw an exception - basically you had to single out the command in a callback, then throw. This is cleaner and can be used to test other paths, such as process timeout, etc.

Thanks!


Fix Concatenating Strings with Plus Operator test

Fixed test case
// not working due to space before semicolon
var myStr = "This is the start. " + "This is the end." ;

Now it works even for this TESTED OK

var

                            myStr     =

    "This is the start. "

+

    'This is the end.'                  ;

Reported by @Philosophist  :point_right: FreeCodeCamp/FreeCodeCamp

Those are some dead sexy back-references.


High availability installation

Is your feature request related to a problem? Please describe.
I need to install few ha clusters. I also need to manage these clusters by using Terraform.
Each cluster will be composed of three servers nodes.
I would like to use the terraform-null-rke2-install module, however :

I didn't find a high availability (ha) example in the "examples" folder
The server and the token flags cannot be passed as variable to make a new node joins an existing cluster.

Describe the solution you'd like
I would like that module to support High availability installations.
I think the easy way to support these use cases is to pass the "server" and "token" values as variables, and then, passing them as flags to the rke2 command.
I would also like to see a high availability (ha) example in the examples folder.
Describe alternatives you've considered

Not using the terraform-null-rke2-install module and create my own module.

Additional context
I have no experience yet with RKE2. I hope I didn't miss something.
Considering this module runs the rke2 command, I think it would be an improvement to be able to pass additionnal custom flags to the command to cover more (advanced) use cases.

Looks like what i was looking for is in the terraform-rancher2-aws module :
https://github.com/rancher/terraform-rancher2-aws/blob/main/examples/prod/main.tf
I close the issue.


Question: Get-SeElement: How to get the parent of -Element?

Good afternoon
thank you very much for sharing your great work!
I try to get the parent of a SeElement, but I always get exceptions.
Based on other Selenium Forums, I guess that this should work:
# This is the Sub-Node
$SubNode = Get-SeElement …

# Try to get the parent SeElement - does not work:
$Parent = Get-SeElement -Element $SubNode -By XPath './..'
$Parent = Get-SeElement -Element $SubNode -By XPath '//parent::*'

Do you know a way to get the parent SeElement?
Thanks a lot for any light!
kind regards,
Thomas

I am very sorry for the confusion!
Getting the parent works perfectly with:
$Parent = Get-SeElement -Element $SubNode -By XPath '..'
$Parent = Get-SeElement -Element $SubNode -By XPath './..'
$Parent = Get-SeElement -Element $SubNode -By XPath 'parent::*'
# This searches for the parent of type a:
$Parent = Get-SeElement -Element $SubNode -By XPath 'parent::a'

Thanks a lot, kind regards,
Thomas


add conditions to patient timeline

Review Checklist

Summary*

[ ] fix: Bug fix for Add the items tackled here as checklists
[x] feat: Completed task
[ ] chore: Incomplete task

[ ] test: Sub-task 1

Structure*

[ ] The Pull Request has a proper title that conforms to our MR title standards
[ ] The Pull Request has one commit, and if there are more than one, they should be squashed
[ ] The commit should have a proper title and a short description
[ ] The commit must be signed off
[ ] Unused imports are not present
[ ] Dead code is not present
[ ] Ensure dry running library to confirm changes

Tests

[ ] Proper and high quality unit, integration and acceptance(if applicable) tests have been written
[ ] The coverage threshold should not be lowered

Sign off*

[ ] All comments have been resolved by the reviewers
[ ] Approved by Czar {replace_with_czar_name}
[ ] Signed off by second reviewer {replace_with_name}
[ ] Ensure all checks are done before merging :warning:
[ ] All PRs needs to be signed before merging :warning:

N/B:

Add a checklist if more than one item is done.
Add screenshots and/or images where necessary
Indicate any breakages caused in the UI :exclamation:
Where necessary, indicate which issue the Pull Request solves (Closes #)
Any new files are updated in the folder structure in the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
31 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 79.482%

Files with Coverage Reduction
New Missed Lines
%

pkg/clinical/usecases/clinical/timeline.go
31
55.51%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
5125

Relevant Lines:
6448

💛  - Coveralls


update Webview version in README

📦 Pull Request
[Provide a general summary of the pull request here.]

Updates the installation instruction in the react native expo README<EMAIL_ADDRESS>to<EMAIL_ADDRESS>Ticket: SC-77511

✅ Fixed Issues

[List any fixed issues here like: Fixes #XXXX]

🚨 Test instructions
[Describe any additional context required to test the PR/feature/bug fix.]
⚠ Don't forget to add a semver label!
Please only add one label:

patch: Bug Fix?
minor: New Feature?
major: Breaking Change?
skip-release: It's unnecessary to publish this change.

@am-hernandez Is there a reason we need to specify the version number? Wouldn't it be better to remove the version number altogether and get folks installing the latest version of the package?

@Ariflo That sounds fair too. I bring this up because of the changes we made to the docs for this exact issue. We never did update the README at that time but would have.

:rocket: PR was released in<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>:rocket:


Support for multiple file extensions in tile-server

Currently, raster tiles are always png. Also, the url to retrieve them, has the format "/map/tiles/{}/{{tileMatrix}}/{{tileRow}}/{{tileCol}}", which does not contain the extension.
Our backend (MapServer) has png and jpg support, and I would love to support that via bbox as well.
I believe each tileset should have a file types property, that specifies what extensions are supported. Then, there should be an link with rel=item for each format, pointing to the file. Lastly, the route handler should take the format as an argument, rather than taking it from the accept header.

There is already support for multiple raster formats. For the OGC API via content-type and when using the XYZ endpoint  via file extension:
curl -o /tmp/tile.jpg -H 'Accept: image/jpeg' http://localhost:8080/map/tiles/WebMercatorQuad/2/2/2

curl -o /tmp/tile.jpg http://localhost:8080/xyz/ne_umn/2/2/2.jpg

Are you using the BBOX FCGI backend with UMN Mapserver, or the HTTP proxy map service?

I was under the impression that the content type negotiation wasn't in spec for the OGC tiles api, but it seems it is. Most implementations I've seen have the links that point to specific types. That for example also allows caching by url only.
The xyz urls don't include the TMS and are thus don't cover all cases.
Shall I PR a new route where the extension is in the url, or rather changes the existing behaviour and move away from content negotiation?

I'm sure that I implemented content type negotation because it was required by the OGC spec. But the specs were in draft back then. So we should check the latest map service and tile service specs. Anyway, I'm absolutely fine with adding support for file extensions for the OGC endpoint and a PR would be welcome!


Unhandled exception. System.Resources.MissingManifestResourceException:

Hello,
I was directed to start an issue in msbuild by https://github.com/davkean  (issue: project-system #2100).
This may be an issue with my port from a Windows Forms application to dotnet 3.1 but I am at a lost as to how to correct it.
Regards,
Ron
mscc.zip
Steps to reproduce

Build the included project.
Run - dotnet mscc.dll

Output of dotnet:
G:\Ham Radio\ominaSDR\Hardware\Linux\dotnettest\mscc\Release\netcoreapp3.1>dotnet mscc.dll
Unhandled exception. System.Resources.MissingManifestResourceException: Could not find the resource "OmniaGUI.Main_form.resources" among the resources "mscc.Properties.Resources.resources" embedded in the assembly "mscc", nor among the resources in any satellite assemblies for the specified culture. Perhaps the resources were embedded with an incorrect name.
at System.Resources.ManifestBasedResourceGroveler.HandleResourceStreamMissing(String fileName)
at System.Resources.ManifestBasedResourceGroveler.GrovelForResourceSet(CultureInfo culture, Dictionary`2 localResourceSets, Boolean tryParents, Boolean createIfNotExists)
at System.Resources.ResourceManager.InternalGetResourceSet(CultureInfo culture, Boolean createIfNotExists, Boolean tryParents)
at System.Resources.ResourceManager.GetString(String name, CultureInfo culture)
at System.Resources.ResourceManager.GetString(String name)
at OmniaGUI.Main_form.InitializeComponent() in G:\Ham Radio\ominaSDR\Hardware\Linux\dotnettest\mscc\Main_Form.designer.cs:line 1403
at OmniaGUI.Main_form..ctor() in G:\Ham Radio\ominaSDR\Hardware\Linux\dotnettest\mscc\Main_Form.cs:line 105
at OmniaGUI.Program.Main() in G:\Ham Radio\ominaSDR\Hardware\Linux\dotnettest\mscc\Program.cs:line 20
OS info:
Windows 10
Microsoft Visual Studio Community 2019
Version 16.4.4
VisualStudio.16.Release/16.4.4+29728.190
Microsoft .NET Framework
Version 4.8.03752
Installed Version: Community
Visual C++ 2019   00435-60000-00000-AA205
Microsoft Visual C++ 2019
.NET Portability Analyzer   1.1.10808.0
Evaluates portability of assemblies across .NET platforms.
ASP.NET and Web Tools 2019   16.4.460.23317
ASP.NET and Web Tools 2019
Azure App Service Tools v3.0.0   16.4.460.23317
Azure App Service Tools v3.0.0
C# Tools   3.4.1-beta4-19614-01+165046097562cfe65b09c2e9a9d8f7cd88526f2c
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
IntelliCode Extension   1.0
IntelliCode Visual Studio Extension Detailed Info
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers
Microsoft Visual C++ Wizards   1.0
Microsoft Visual C++ Wizards
Microsoft Visual Studio VC Package   1.0
Microsoft Visual Studio VC Package
NuGet Package Manager   5.4.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/
NuGetReferenceSwitcher   1.0
Information about my package
ProjectServicesPackage Extension   1.0
ProjectServicesPackage Visual Studio Extension Detailed Info
Test Adapter for Boost.Test   1.0
Enables Visual Studio's testing tools with unit tests written for Boost.Test.  The use terms and Third Party Notices are available in the extension installation directory.
Test Adapter for Google Test   1.0
Enables Visual Studio's testing tools with unit tests written for Google Test.  The use terms and Third Party Notices are available in the extension installation directory.
UpgradeCommand Extension   1.0
UpgradeCommand Visual Studio Extension Detailed Info
Visual Basic Tools   3.4.1-beta4-19614-01+165046097562cfe65b09c2e9a9d8f7cd88526f2c
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio
Visual Studio Tools for CMake   1.0
Visual Studio Tools for CMake
Visual Studio Tools for CMake   1.0
Visual Studio Tools for CMake
If applicable, version of the tool that invokes MSBuild (Visual Studio, dotnet CLI, etc):

Folks,
Any update on this?  I could really use a hand with this.   If this is the wrong area for this issue, just let me know.


Natron: init at 1.2.1

Wasn't an easy thing to package but it runs great. Was able to generate some images, animate them with some keyframes and render to an MP4 file.

Please use lower-case directory names (thinking of "pkgs/applications/video/Natron/").

And lower-case package name: Natron -> natron.

i tried to build on NixOS x86_64 with nox-review pr 9846:
[...]
From https://github.com/devernay/openfx-supportext
 * [new branch]      master     -> origin/master
 * [new tag]         Natron-0.9.6 -> Natron-0.9.6
Switched to a new branch 'fetchgit'
git revision is 0c079bafeab5750e6e26086c67c5947a0ce70097
git human-readable version is 1.9.0
Commit date is 2015-08-05 18:41:49 +0200
Deleted remote-tracking branch origin/master (was c68738a).
Deleted remote-tracking branch origin/gh-pages (was c71528b).
Deleted remote-tracking branch origin/master (was 6eb0a50).
output path ‘/nix/store/nllb1gijq8qv8857dgx6h2m17bw5gadm-openfx-misc’ should have r:sha256 hash ‘0zbwyxblkcyyyj45szhr2vhrbjwlpgcxm24pvp5048ns5rwxwbbh’, instead has ‘0iazhaycbcsv2vxzg9slh32bsvz1yj55453szybac97cm3g70i3k’

That's what fetchgit from github does – the stuff served changes slightly over time. fetchzip and fetchFromGitHub were introduced exactly to fight that.

@puffnfresh ping?

@puffnfresh can you rebase on master?

i'm closing this PR since package in this PR is not the latest and there has been almost one year of inactivity.


Include Lambert domains for LDT forcing reader in LIS

Currently, the LDT-generated forcing reader in LIS has been tested and used for only latlon grid domains.
Test and include Lambert domains within this reader.

Lambert domains have been reviewed and run in LDT.


Fix build in latest nightly rust (1.17.0)

This is a quick (and maybe dirty?) fix for the breaking changes that 1.17 brought to us.

Brilliant, thank you so much! You might want to add yourself as an author to Cargo.toml as well.
As for hacks ... that whole crate is a bit of a hack. ;)


Migrate to Dart Sass from LibSass

What
Migrate to Dart Sass from LibSass.
Why

LibSass is now deprecated
Making changes ahead of Design System team dropping support for LibSass https://github.com/alphagov/govuk-frontend/discussions/2632.

Visual Changes
None.
How can I test these changes?

run yarn jasmine:prepare (or run bundle exec rake app:assets:clobber app:assets:precompile)
what should happen? The Sass files should be compiled. Keep an eye on the command line for warnings indicating the compilation process using Dart Sass

Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.
Recommendation: math.div(300%, 4) or calc(300% / 4)
More info and automated migrator: https://sass-lang.com/d/slash-div
   │
26 │   three-quarters: (300% / 4),
   │                    ^^^^^^^^
   ...

the /builds folder: app/assets/builds should now be populated with CSS files. Note, the
subsequently, all assets ought to be generated successfully within the /public directory
no test failures to indicate the correct number of stylesheets are generated
deploy to Integration. You'll notice that in the 'build and publish image' task, the steps taken resemble those mentioned earlier, dartsass-rails is bundled and any Dart Sass warnings encountered during the compilation process are reported)
inspect several pages, e.g. homepage, UK bank holidays, register to vote, contact DVLA to ensure the absence of errors.

Anything else
Once the PR gets the green light, any temporary commits and references to local gems e.g. gem "govuk_publishing_components", path: "../govuk_publishing_components" will be removed.
See:

https://github.com/alphagov/collections/pull/3447
https://github.com/alphagov/govuk_publishing_components/pull/3726
https://github.com/alphagov/static/pull/3190
https://github.com/alphagov/govuk-developer-docs/pull/4216.

Nice work @jon-kirwan. I've tested locally using live data and integration and everything builds correctly / app runs as normal. Looks like the linter has failed, however it is an easy fix.
Also noted the deprecation warnings (Using / for division outside of calc()), however I understand that we cant fix them without breaking the LibSass implementation in the components Gem. Hopefully we (GOV.UK) can migrate to Dart Sass fully before 2.0 is released and squash the warnings.

Thanks @1pretz1. I've resolved the linter error, and all CI checks are now passing.
Currently, the linter warnings are from GOV.UK Publishing Components, and I've opened an issue to investigate. We also had warnings from the Design System, but have silenced those with this change: https://github.com/alphagov/frontend/blob/d56a8773ebaa6999e89aa47fb778ed1ec7580e6c/config/initializers/dartsass.rb#L22


Use scripts to maintain the supported_releases.txt file

/kind feature
/hold

@yibozhuang
Update
Added timed task to check daily, and send PR if out of sync
#24 Run in my local, that look good.

/unhold


Tweak UPGRADING.md

Fixed some issues in UPGRADING.md. I am not sure the link https://github.com/bundler/bundle-viz should be removed or not, but it could make readers confused (like me), so I removed it. It should be reverted when the link is available.

@deivid-rodriguez Thank you for your feedback. I changed the commit message. Could you re-review it please?

@bundlerbot merge

@deivid-rodriguez Just FYI, I translated UPGRADING.md into Japanese under MIT license. If you have any concerns, please feel free to let me know.
https://qiita.com/jnchito/items/d3257a5f46f4f1aee200


Automatically deploy on branch tags.

It will remove a step from releasing, which is useful.

@rudigiesler @justinvdm Ready for review.

:+1:


[Bug]: After the archived task is annotated, the task details are displayed as default values

ODC version
ODC432
OB version
independent
What happened?
After the archived task is annotated, the task details are displayed as default values

What did you expect to happen?
。
How can we reproduce it (as minimally and precisely as possible)?
。
Anything else we need to know?
No response

pass


'jhack tail' is slow to stop after tailing for some time (20 charms, 100+ events, 20 mins)

Hi,
After deploying test bundle and jhack tailing the process it is slow to abort jhack, the ^C takes time and bring to the another jhack output (all columns shown this time) which you need to abort with another ^C and it takes time, see the recording with speedup:
Screencast from 2024-06-03 23-39-34.webm

I think that's because it's processing so many events.
Is "20 mins" the time it took to close tail (that'd be a lot), or the lifetime of the test model you were deploying?

The closing time is 1-2 mins in the worst case... but annoying and visible. It looks like it is doing some internal calculations... as more column shown. Why those calculations necessary?

those calculations are needed to print out the final 'full table', listing all captured events from the start of the session up to when you close it.
If you pass --output ./foo.txt the full table will be printed to file, otherwise it will be printed to console.
If it's annoying I can disable it (it will still take longer with --output)

if no --output passed, can you skip the final calculations and exit? :-)

If it's annoying I can disable it (it will still take longer with --output)

It is necessary when necessary... otherwise fast.

yeah, thought it'd be nice to see the full summary even without --output, didn't think it'd take annoyingly long on regular runs.

This is probably solved by using the --printer raw option implemented in #175


Copy Gemfile.lock into the docker container if there is one

Should fix #40

Did you test this change in your local? Is it solving your problem?

@navarasu I have tested it and just found that I needed to add a second commit to get everything working. It's doing what I expect locally and does solve the problem of mismatched gem versions.

Thanks, Jeremy. Please add your example for gemfile lock case under the example folder. I will update the test case from my side

Hey @navarasu, I wasn't entirely sure what you'd want for an example, but I did take a first stab at it. Let me know if there's anything else you need.
Of note, the Gemfile.lock specifies aws-partitions (1.440.0) but the most recent version is 1.441.0 which is what you should get if Gemfile.lock isn't being respected.

@navarasu anything stopping this from being merged?

The ruby file has to be update examples/use-docker-with-gemfile-lock/handler.rb need to add test case for it.

Today I have created a proper gemfile.lock example and test case for both basic and docker case.

Basic case without docker works without issue.
With docker case, we need to take care of missing bundler version else it will fail with following error

 Could not find 'bundler' (1.16.6) required by your /var/gem_build/Gemfile.lock. (Gem::GemNotFoundException)
To update to the latest version installed on your system, run `bundle update --bundler`.
To install the missing version, run `gem install bundler:1.16.6`

Created new PR https://github.com/navarasu/serverless-ruby-layer/pull/45 for testcase and fix


Move docs out of repo

The user documentation has moved to a new repository at https://github.com/crc-org/docs
Related to: https://github.com/crc-org/crc-internal/issues/91

👍, also updated the blog to point to the "Getting Started Guide"


Separate image widget support from image codecs.

This optimization with feature flags allows a couple of cases:

You can now use the Image widget without having any codecs configured (Raw pixel buffers).
A release + lto binary shaves off 1.7mb on average removing codecs and still supporting the image widget.

Now, the from_path and from_memory methods are gated and will only be present when one or more codec features are enabled.

Audit issue is unrelated.

Thinking about this it may make more sense to enable the image widget by default, and turn the "image" feature into actual codec support in general. That way it would be a non-breaking change and still a win-win size wise.


[Question] Include own VCF files

Is your feature request related to a problem? Please describe

Describe the solution you'd like

Describe alternatives you've considered

Additional context

This issue is from @jgriss,
Short question: Is it also possible to directly create a FASTA file based on your own VCF files using this pipeline?
Thanks a lot!
Johannes

added a new process to support own vcf
@ypriverol need to test commit: 7ddf11bc34019e32095f686e1af34d288eb4265f after deploying the new version of pypgatk

Hi @jgriss,
The command to translate own vcf would be (pull the latest changes):
nextflow run main.nf -profile conda --ensembl_name homo_sapiens -resume --vcf true --vcf_file input_file.vcf
note: it assumes that your variants are mapped to hg38 because it uses the latest assembly from ENSEMBL to annotate and translate the variants within CDS.

Hi @husensofteng,
Thanks a lot! I'll give it a try right away.
Kind regards,
Johannes

I assume the issue is solved, please feel free to re-open if you get any error.


After croping problem

After second time cropping an image ... CropImageActivity loads again 1 time after preesing crop...
After third time cropping an image ... CropImageActivity loads again 2 time after preesing crop...
Can you help me with this?

can u share the solution please


KAFKA-3020: Ensure CheckStyle runs on all Java code

Adds CheckStyle to core and examples modules
Fixes any existing CheckStyle issues

@granthenke LGTM, merged with minor tweak to remove unused imports in the import-control-core.xml.

@ewencp Thanks!


Feature Request - Can we please add options for manual Rate Limits please?

As Topic suggests. Can we please add options for the following Rate Limits please? Max task per job is too simple especially given anthropics recent errors. I rather have the flexibility to run it slowly stably.
Rate Limits
Requests per minute (RPM)
Tokens per minute (TPM)
Tokens per day (TPD)
Great job on creating this. Has consistently taken over aider now I feel.

Thank you for the feedback! These would be good options to have especially once I've implemented auto-approval for tool use.

Closing as duplicate of https://github.com/saoudrizwan/claude-dev/issues/54