[BUG]: The payment calculation for Warrant is wrong

Describe the bug
The payment calculation for Warrants is wrong.
To Reproduce

Issue a warrant
The lawyer opens up the pending task to make payment for summons
It shows E Filing Fee instead of I Cops fee
The pay button is also not working

Expected behavior
Only Court Fee needs to be paid at the time of Icops Warrant
The Pay button should redirect to E Treasury
Screenshots
https://github.com/user-attachments/assets/c8549ad7-674f-4d8a-88dc-d30464a5e4b6

Hi @Ramu-kandimalla Can you please add Ajith with this ticket? I'm not sure whether the UI is calling the endpoint or the backend.

@devarajd94 , we don't have any payment information for a warrant in the payment calculator document, and the same goes for notice. Could you please update the document accordingly?

Hi @Ramu-kandimalla @devarajd94 Payment calculator api is just called for E-Filing as far as i know. Please Provide further details on requirements for it's integration.

Hi @devarajd94  this is a duplicate ticket please refer the original ticket


Media query does not override Button hover styles on smaller screens

I've noticed that the media query override that I have provided in the Theme Overrides for Button does not actually override the button styles when viewed on smaller screens.
I wrapped :hover in the same media query that the Button has by default. You can see in this screenshot how I have overridden the raisedPrimary class for Buttons in my theme file:

My override is applied, but the default styles seem to be applied after the overrides. This is the console for the Button after I added the override:

I could be doing something wrong, but I've tried unsuccessfully several ways to do this while also avoiding using !important on my :hover.
We absolutely love using MUI here at Torchlite. Thank you for all of your hard work.

@jermsull The next release of Material-UI (1.0.0-beta.42) will come with a button override example: https://material-ui-next.netlify.com/demos/buttons/#customized-buttons. I can't reproduce the issue you are describing not the issue comes with a reproduction. So, I'm gonna assume it's ok.


Update subler to 1.3.2

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] If the sha256 changed but the version didn’t,
provide public confirmation by the developer: {{link}}

Thank you for the contribution. It was merged directly as d85dc2dc1540d920464a5b61dd57951e5abe44c0. Your contribution is still credited to you.


Add inactivity notification popup that toggles after 3 minutes of no …

Issue-88: Inactivity message
Implementations/Features

Created an inactivity message component in the form of a modal that triggers after 3 minutes.
Only displays if a user is actively signed in
Displays in the center of user's screen.
Allows user to either close the modal and continue session or to log out.
Uses Material UI for base  and  components.
Just has inline styling for now, but could probably be spruced up in the future by the design team.

Notes

Can provide a picture of what the notification looks like. Not sure how to add it to the PR without uploading it to the repo as a whole.

Also changing the merge to Development instead of Master


NB Launcher - replace colab with myBinder, increase popup delay #654

Removed and replaced colab launcher server option with myBinder
Increased time for Notebook Launcher opening on hover from 200ms to 500ms

@jlperla Sounds good to me, merging and then we can confirm with the next test build.


music on hold (moh) for dial out

currently when waiting for a call to connect using "yap" mode on the helpline section, you get dead air.  the point of this would be able to use a stock MOH mp3 for playback.

ive thought about this alot and i don't think stock is the way to go, I think it should be bill withers lovely day. haha. ive noticed even on grasshopper systems people would be more likely to wait a minute on hold with music than just ringing or silence for that matter.

it would be a stock file from Creative Commons (whatever we don't have to pay royalties on / get permission on), and then provide an option for anyone to swap in their own.

Makes sense

The way that I would have to implement this is less than ideal, but it's do-able for sure.

this is implemented in the unstable branch and destined for yap v2.  note that generally this is broken and will only dial the first volunteer.  some more work needs to be done to call this feature complete.  also need to add configuration value for MOH.

custom moh implemented in unstable

I tested this with a mp3 on my server and worked great.


community: update Memgraph integration

Description:

Memgraph no longer relies on Neo4jGraphStore but implements GraphStore, just like other graph databases.
Memgraph no longer relies on GraphQAChain, but implements MemgraphQAChain, just like other graph databases.
The refresh schema procedure has been updated to try using SHOW SCHEMA INFO. The fallback uses Cypher queries (a combination of schema and Cypher) → LangChain integration no longer relies on MAGE library.
The schema structure has been reformatted. Regardless of the procedures used to get schema, schema structure is the same.
The add_graph_documents() method has been implemented. It transforms GraphDocument into Cypher queries and creates a graph in Memgraph. It implements the ability to use baseEntityLabel to improve speed (baseEntityLabel has an index on the id property). It also implements the ability to include sources by creating a MENTIONS relationship to the source document.
Jupyter Notebook for Memgraph has been updated.
Issue: /
Dependencies: /
Twitter handle: supe_katarina

I initially had all imports in langchain package as well, since I followed the implementation of others, but now I removed it based on the PR template.
I also ran make format and make lint, but based on the PR template, I didn't want to commit changes on other integrations. Let me know if that's okay.

Hi @baskaryan, tagging as noted in PR template. Please let me know what I need to do regarding the test and what's needed to do in order to get this merged.

I managed to fix the CI. Now, all is passing.
Tagging @efriis, as the other person from the PR template for the review. Let me know if someone else should be tagged.

Hey! Here's a link to the review policy: https://python.langchain.com/docs/contributing/review_process/
Status is currently needs support

@efriis is there any update? I have more than 5 upvotes so I am wondering what's the next course of action here and if I can help somehow.

Thank you, @libraua, for approving this PR. Is there something else I need to do to get this PR merged? I wanted to enable Memgraph developers to utilize Memgraph LangChain integration with the latest improvements.

@ccurme is this available as part of langchain-core==0.3.23?

This will be available in the next release of langchain-community (0.3.11)


Hoist the Utility module above Basic, now that path support is in Basic

Flip the layering of Basic and Utility, now that the last reference has been removed.  Basic is where new general-purpose support is to be added, and as things from Utility get cleaned up, they may be moved down to Basic if they are deemed suitable of that.

Can you fix the PR so it doesn't have a merge in it?

Also, I think those two commits should be squashed together. My rule is that each commit in a sequence should ideally build and pass tests...

Definitely.  The merge was a mistake, and the squash is a good idea.

LGTM, otherwise.

Awesome!

Ok, that should be fixed now.  One commit, and no merges.

@swift-ci please test

Linux failure is unrelated (once again).  Merging.


failure installing XCode command line tools

System:
Clean install of Mac OS X 10.11.6 (snapshotted in VMWare Fusion)
I'm being prompted to install XCode command line tools while running the bootstrap script. The scripts halts here:

Unfortunately, after waiting for the install I get an awesome error message:

I suspect the key is:
==> /usr/bin/sudo /bin/rm -f /tmp/.com.apple.dt.CommandLineTools.installondemand.in-progress STDERR: sudo: no tty present and no askpass program specified
Running again bypasses the error, but it is irritating. I want to see if calling build-essential first will resolve the issue. I am able to reproduce this issue.

I'm going to classify this as "meh" or something that can be worked around in documentation. As long as you just ignore the pop-up, it will go away and things will work as expected.


PR TEMPLATE

I can add a pr template , which will have few checklists and options like uploading screenshots and describing about the pr etc , which will give better understanding of pr for the maintainer

I would love to work on this issue as a part of DWOC. Kindly assign me this issue

I would love to work on this issue as a part of DWOC.


GET request is continuously triggered after the DELETE request is issued

What happened?
I’m automating the onboarding of accounts and other resources to Prisma Cloud using provider-http. As an initial step, I successfully created a role. However, when I tried to delete the managed resource (MR), it got stuck in the delete stage. After reviewing the pod logs, I noticed that a GET call is continuously triggered after the DELETE request. The external resource (the role) is deleted correctly, but the MR remains stuck. The GET/OBSERVE call is failing because the external resource was already deleted as part of the original DELETE request. I came across a similar issue reported here. I've tried several configurations in expectedResponseCheck, but none of them have worked so far.
pod logs
{"level":"info","ts":"2024-11-11T07:36:04+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:36:06+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"DELETE\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:36:06+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:36:07+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:36:12+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:36:20+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:37:03+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:37:35+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:38:36+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:39:19+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"POST\",\"body\":\"{\\n  \\\"username\\\": \\\"{{ prismacloud-secrets:crossplane-system:accessKeyId }}\\\",\\n  \\\"password\\\": \\\"{{ prismacloud-secrets:crossplane-system:secretKey }}\\\"\\n}\\n\",\"url\":\"https://api.eu.prismacloud.io/login\",\"headers\":{\"Accept\":[\"application/json; charset=UTF-8\"],\"Content-Type\":[\"application/json; charset=UTF-8\"]}}","disposableRequest":"obtain-jwt-token"}
{"level":"info","ts":"2024-11-11T07:39:37+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:40:37+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:42:53+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}
{"level":"info","ts":"2024-11-11T07:43:54+05:30","logger":"provider-http","msg":"http request sent: {\"method\":\"GET\",\"url\":\"https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03\",\"headers\":{\"Content-Type\":[\"application/json\"],\"x-redlock-auth\":[\"{{response-token:crossplane-system:token}}\"]}}","request":"add-role"}

describe output after delete request is made
iehje9m@AMC-D6YC47CD9H examples % k describe request.http.crossplane.io/add-role
Name:         add-role
Namespace:
Labels:       <none>
Annotations:  crossplane.io/external-create-pending: 2024-11-11T07:16:14+05:30
              crossplane.io/external-create-succeeded: 2024-11-11T07:16:15+05:30
              crossplane.io/external-name: add-role
API Version:  http.crossplane.io/v1alpha2
Kind:         Request
Metadata:
  Creation Timestamp:             2024-11-11T01:46:14Z
  Deletion Grace Period Seconds:  0
  Deletion Timestamp:             2024-11-11T02:05:07Z
  Finalizers:
    finalizer.managedresource.crossplane.io
  Generation:        2
  Resource Version:  2195897
  UID:               dc70ddad-ab1c-4269-bd67-243b0d6e48bc
Spec:
  Deletion Policy:  Delete
  For Provider:
    Expected Response Check:
      Logic:  if .response.body.name == .payload.body.name
 then true
 else false
 end

      Type:  CUSTOM
    Headers:
      Content - Type:
        application/json
      X - Redlock - Auth:
        {{response-token:crossplane-system:token}}
    Mappings:
      Action:  CREATE
      Body:    {
name: .payload.body.name,
roleType: .payload.body.roleType,
accountGroupIds: ["55555f8c-8867-4a9a-b608-1bab38db8c40"],
description: .payload.body.description
}

      Headers:
        Content - Type:
          application/json
        X - Redlock - Auth:
          {{response-token:crossplane-system:token}}
      Method:  POST
      URL:     .payload.baseUrl + "/user/role"
      Action:  OBSERVE
      Method:  GET
      URL:     (.payload.baseUrl + "/user/role/" + (.response.body.id|tostring))
      Action:  UPDATE
      Body:    {
name: .payload.body.name,
roleType: .payload.body.roleType,
accountGroupIds: ["55555f8c-8867-4a9a-b608-1bab38db8c40"],
description: .payload.body.description
}

      Method:  PUT
      URL:     (.payload.baseUrl + "/user/role/" + (.response.body.id|tostring))
      Method:  DELETE
      URL:     (.payload.baseUrl + "/user/role/" + (.response.body.id|tostring))
    Payload:
      Base URL:  https://api.eu.prismacloud.io
      Body:      {
  "name": "test role1",
  "roleType": "Owners",
  "description": "Created from crossplane"
}

  Management Policies:
    *
  Provider Config Ref:
    Name:  http-conf
Status:
  Cache:
    Last Updated:  2024-11-11T02:06:04Z
    Response:
      Body:  {"id":"ce8d2265-048c-43bc-9341-07da17e7bd03","name":"test role1","description":"Created from crossplane","lastModifiedBy":"OnboardingUser","lastModifiedTs":1731289575107,"accountGroupIds":["55555f8c-8867-4a9a-b608-1bab38db8c40"],"resourceListIds":[],"codeRepositoryIds":[],"associatedUsers":[],"restrictDismissalAccess":false,"permissionGroup":null,"additionalAttributes":{"onlyAllowCIAccess":false,"onlyAllowComputeAccess":false,"hasDefenderPermissions":false,"onlyAllowReadAccess":false},"accountGroups":[],"resourceLists":[],"codeRepositories":[],"roleType":"Owners"}
      Headers:
        Cache - Control:
          no-cache, no-store, max-age=0, must-revalidate
        Connection:
          keep-alive
        Content - Security - Policy:
          default-src 'self' https:; font-src 'self' https: data:; img-src 'self' https: data:; object-src 'none'; script-src https:; style-src 'self' https: 'unsafe-inline'
        Content - Type:
          application/json
        Date:
          Mon, 11 Nov 2024 02:06:04 GMT
        Expires:
          0
        Pragma:
          no-cache
        Referrer - Policy:
          no-referrer
        Strict - Transport - Security:
          max-age=31536000 ; includeSubDomains
        Trace - Id:
          882dff959fcae095eced096960b5f692
        Vary:
          Origin
          Access-Control-Request-Method
          Access-Control-Request-Headers
          accept-encoding
          x-redlock-auth
          Origin
        X - Content - Type - Options:
          nosniff
        X - Download - Options:
          noopen
        X - Frame - Options:
          DENY
        X - Permitted - Cross - Domain - Policies:
          none
        X - Xss - Protection:
          0
      Status Code:  200
  Conditions:
    Last Transition Time:  2024-11-11T02:06:06Z
    Reason:                Available
    Status:                True
    Type:                  Ready
    Last Transition Time:  2024-11-11T02:06:06Z
    Message:               observe failed:  failed updating status: HTTP GET request failed with status code: 400
    Reason:                ReconcileError
    Status:                False
    Type:                  Synced
  Failed:                  22
  Request Details:
    Headers:
      Content - Type:
        application/json
      X - Redlock - Auth:
        {{response-token:crossplane-system:token}}
    Method:  GET
    URL:     https://api.eu.prismacloud.io/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03
  Response:
    Body:  {"timestamp":"2024-11-11T02:26:45.265714384Z","status":400,"error":"INVALID ID","message":"invalid_id","path":"GET:/api/v1/user/role/ce8d2265-048c-43bc-9341-07da17e7bd03"}
    Headers:
      Cache - Control:
        no-cache, no-store, max-age=0, must-revalidate
      Connection:
        keep-alive
      Content - Security - Policy:
        default-src 'self' https:; font-src 'self' https: data:; img-src 'self' https: data:; object-src 'none'; script-src https:; style-src 'self' https: 'unsafe-inline'
      Content - Type:
        application/json
      Date:
        Mon, 11 Nov 2024 02:26:45 GMT
      Expires:
        0
      Pragma:
        no-cache
      Referrer - Policy:
        no-referrer
      Strict - Transport - Security:
        max-age=31536000 ; includeSubDomains
      Trace - Id:
        46be61ad0b33ba2b097f744a73c8e838
        46be61ad0b33ba2b097f744a73c8e838
      Vary:
        Origin
        Access-Control-Request-Method
        Access-Control-Request-Headers
        accept-encoding
        x-redlock-auth
        Origin
      X - Content - Type - Options:
        nosniff
      X - Download - Options:
        noopen
      X - Frame - Options:
        DENY
      X - Permitted - Cross - Domain - Policies:
        none
      X - Redlock - Status:
        [{"severity":"error","i18nKey":"invalid_id"}]
      X - Xss - Protection:
        0
    Status Code:  400
Events:
  Type     Reason                         Age                 From                                Message
  ----     ------                         ----                ----                                -------
  Normal   CreatedExternalResource        40m                 managed/request.http.crossplane.io  Successfully requested creation of external resource
  Normal   DeletedExternalResource        20m                 managed/request.http.crossplane.io  Successfully requested deletion of external resource
  Warning  CannotObserveExternalResource  13s (x22 over 20m)  managed/request.http.crossplane.io  failed updating status: HTTP GET request failed with status code: 400
iehje9m@AMC-D6YC47CD9H examples %

How can we reproduce it?
Please find the manifest which used for role creation.
apiVersion: http.crossplane.io/v1alpha2
kind: Request
metadata:
  name: add-role
spec:
  deletionPolicy: Delete
  forProvider:
    # insecureSkipTLSVerify: true
    waitTimeout: 5m
    headers:
      Content-Type:
        - application/json
      x-redlock-auth:
        - "{{response-token:crossplane-system:token}}"
    payload:
      baseUrl: 'https://api.eu.prismacloud.io'
      body: |
        {
          "name": "test role1",
          "roleType": "Owners",
          "description": "Created from crossplane"
        }
    mappings:
      - action: CREATE
        method: POST
        body: |
          {
          name: .payload.body.name,
          roleType: .payload.body.roleType,
          accountGroupIds: ["55555f8c-8867-4a9a-b608-1bab38db8c40"],
          description: .payload.body.description
          }
        url: .payload.baseUrl + "/user/role"
        headers:
          Content-Type:
            - application/json
          x-redlock-auth:
            - "{{response-token:crossplane-system:token}}"

      - action: OBSERVE
        method: GET
        url: (.payload.baseUrl + "/user/role/" + (.response.body.id|tostring))

      - action: UPDATE
        method: PUT
        url: (.payload.baseUrl + "/user/role/" + (.response.body.id|tostring))
        body: |
          {
          name: .payload.body.name,
          roleType: .payload.body.roleType,
          accountGroupIds: ["55555f8c-8867-4a9a-b608-1bab38db8c40"],
          description: .payload.body.description
          }

      - method: DELETE
        url: (.payload.baseUrl + "/user/role/" + (.response.body.id|tostring))
    expectedResponseCheck:
      type: CUSTOM
      logic: |
        if .response.body.name == .payload.body.name
         then true
         else false
         end
  providerConfigRef:
    name: http-conf

What environment did it happen in?
Crossplane version: v1.17.2
Include at least the version or commit of Crossplane you were running. Consider
also including your:

Cloud provider or hardware configuration
Kubernetes version (use kubectl version)

iehje9m@AMC ~ % kubectl version
Client Version: v1.31.0
Kustomize Version: v5.4.2
Server Version: v1.31.0

Kubernetes distribution (e.g. Tectonic, GKE, OpenShift) - Kind
OS (e.g. from /etc/os-release)  - MacOS
Kernel (e.g. uname -a)

Cc: @barunavo

Hi,
Just to explain, the provider sends a GET request after a DELETE to confirm the resource was deleted. Right now, it only supports cases where this GET returns a 404 status. In your case, it looks like the GET is returning a 400, which is causing the issue. I understand the need for more flexibility here.

Thank you so much for the enhancement release @arielsepton 👍


Add borders to "Download all (CSV)" button in Internet Explorer

Is your feature request related to a problem? Please describe.
"download CSV" button has no borders in Internet Explorer
Describe the solution you'd like
To add borders to the "Download all (CSV)" button to be readily spotted/identified by program admins
Additional context

Done when
What conditions need to be met before this issue can be closed?

We don't officially support Internet Explorer, so we should focus on getting #512 done, and then we can close this one out.


implementation of interval function

https://github.com/crsmithdev/arrow/issues/362
From the issue above, implemented an interval function that takes in frame, start, end, tz, and also interval value to give interval on specific time range. (i.e. : frame = 'hour' with interval =2 will give tuples with 2 hour difference within the range)

Current coverage is 100% (diff: 100%)

Merging #386 into master will not change coverage

@@           master   #386   diff @@
====================================
  Files          14     14
  Lines        2944   2962    +18
  Methods         0      0
  Messages        0      0
  Branches      215    217     +2
====================================
+ Hits         2944   2962    +18
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 4996bc9...b2bae22


The footer year should be changed in the academix

Is your feature request related to a problem? Please describe.
The footer year in the academix is a wrong .
Describe the solution you'd like
The footer year should be 2022
Describe alternatives you've considered
Make the footer to 2022
Additional context
N/A
Are you working on it?
Yes

Oh this is already fixed with #68. But it is not deployed. The latest version will be deployed with #70
So, I'm closing this issue


leafref and augment xpath serialization issue

Hi
After fix #224 , it seems new problem introduced in leafref and augment xpath serialization, reproducible code and yang modules are sent by email as security reason. Thanks.
leafref before serialization:
 161         leaf zone {
 162             type leafref

163                 path "/abc:cfw/abc:zone/abc:name";

 810         leaf zone {
 811             type leafref

812                 path "/abc:cfw/abc:zone/abc:name";

leafref after serialization:
 161         leaf zone {
 162             type leafref

163                path "/abc:cfw/abc:abc:zone/abc:abc:name";

 810         leaf zone {
 811             type leafref

812                path "/abc:cfw/abc:abc:zone/abc:abc:name";

Augment before serialization:
<     augment "/abc:cfw/rule-sets/rule-set" {
<     augment "/abc:cfw/rule-sets/rule-set/rules/rule" {

<     augment "/abc:cfw/firewall/servers/server" {
<     augment "/abc:cfw/firewall/servers/server/port" {
<     augment "/abc:cfw/firewall/service-groups/service-group" {
<     augment "/abc:cfw/firewall/service-groups/service-group/member" {

Augment after serialization:
>     augment "//abc:cfw/rule-sets/rule-set" {
>     augment "//abc:cfw/firewall/servers/server" {
>     augment "//abc:cfw/rule-sets/rule-set/rules/rule" {
>     augment "//abc:cfw/firewall/servers/server/port" {
>     augment "//abc:cfw/firewall/service-groups/service-group" {
>     augment "//abc:cfw/firewall/service-groups/service-group/member" {

@RimShao
I published a 0.7.2-SNAPSHOT build, could you please check?
If the issue is fixed, we can prepare a 0.7.2 release.

Released with v0.7.2


Scanning a QR Code for a Transfer Transaction without Mosaics breaks the Wallet

Steps to reproduce
Scan:

Current behavior
The wallet is unable to proceed

Expected behavior
Be able to Scan the QR Code, and the users see that they are asked to send a message to an account without any mosaic.
Affected Version
0.4.3-13
Platform details

OS: Android  11

Additional context
Supporting QR Codes for Transfer Transaction that doesn't include any Mosaic would enable multiple new use cases.

fixed

Fixed in integration branch.


Added escape character support to String Literals

Initially just added string escaping to the decompiler, as CP77 scripts have several log messages with a stylish /!\ in them, breaking the formatting of the rest of the file. Added support on the compiler side too, although I haven't had a chance to thoroughly test it.
Supports character escapes for Swift as documented here
(I know Redscript is not actually Swift, but might as well maintain compatibility)
Rust's str::escape_default(...) uses the same escape formatting, so the decompiler code is a lot simpler.
I'm relatively new to Rust so let me know if I've made any horrible mistakes

Oh boy, the debug formatting on the test platform is outputting differently from my dev machine. Looks like the single quote aren't being escaped back. I'll need to rewrite that test

very nice, I left 1 comment, but otherwise LGTM


Not allow null value for cache default_ttl

Description
Have an error when try to set null value for cache config default_ttl.
Cache plugin support null value https://docs.php-http.org/en/latest/plugins/cache.html?highlight=cache#semantics-of-null-values
but bundle configuration except an integer value
Error message:
Invalid type for path "httplug.clients.***.plugins.0.cache.config.default_ttl". Expected "int", but got "null".
Hint: The default max age of a Response

How to reproduce
httplug:
    clients:
        bo_inner_api:
            factory: 'httplug.factory.symfony'
            plugins:
                - cache:
                      cache_pool: my_custom_cache_pull
                      config:
                            default_ttl: null
                            respect_response_cache_directives: []

hi @tolean, hm, what is the reason to set the default_ttl to null? can't you omit the configuration completely when you don't want to set it? we don't have a default value in the configuration, so not setting it at all should achieve what you want, no?

@dbu, if I omit this config, the default value from the cache plugin is applied, see https://github.com/php-http/cache-plugin/blob/9e4b50f8aea93720ca745cd81b6154bcb6713898/src/CachePlugin.php#L356

oh, thanks you are right, we need to support explicit null.
do you want to do a pull request for the Configuration.php class to allow that?

I'll be happy to fix it. Problem is here https://github.com/php-http/HttplugBundle/blob/1036c17bd58703e41930fae9f854df2485f12d9a/src/DependencyInjection/Configuration.php#L716
But isn't clear for me how to make nullable numeric node

sorry, this went under. it looks like symfony integerNode can not be nullable. i think we need to change this to a scalarNode and then add a custom validation that says its either null or is_int is true, and mark it invalid if its anything else.
do you want to give that a try?

https://github.com/php-http/HttplugBundle/releases/tag/1.26.0


A few optimizations

In particular in MapDecoder we can avoid checking for JSON object and key presence with downField for every key.

Codecov Report

Merging #1190 into master will increase coverage by 0.41%.
The diff coverage is 94.59%.

@@            Coverage Diff             @@
##           master    #1190      +/-   ##
==========================================
+ Coverage   86.53%   86.95%   +0.41%
==========================================
  Files          89       89
  Lines        2808     2813       +5
  Branches      118      116       -2
==========================================
+ Hits         2430     2446      +16
+ Misses        378      367      -11

Impacted Files
Coverage Δ

.../core/shared/src/main/scala/io/circe/Printer.scala
95.21% <100%> (ø)
:arrow_up:

.../core/shared/src/main/scala/io/circe/Decoder.scala
86.75% <66.66%> (+0.28%)
:arrow_up:

...re/shared/src/main/scala/io/circe/MapDecoder.scala
88.23% <96.29%> (+1%)
:arrow_up:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
88.88% <0%> (+0.58%)
:arrow_up:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
95.34% <0%> (+6.97%)
:arrow_up:

...d/src/main/scala/io/circe/NonEmptySeqDecoder.scala
100% <0%> (+8.33%)
:arrow_up:

...rc/main/scala/io/circe/shapes/SizedInstances.scala
92.85% <0%> (+14.28%)
:arrow_up:

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 08af0ef...13d8a55. Read the comment docs.

For the MapDecoder change, here's before:
MapDecoderBenchmark.decodeNestedMap  thrpt   20  1580.659 ± 21.327  ops/s

And after:
MapDecoderBenchmark.decodeNestedMap  thrpt   20  2119.333 ± 8.666  ops/s

So for large maps it's a pretty big win.


include api module in coverage reports

I don't find a way to run coverage reports on the api module from pytest without this configuration change. Coming from https://github.com/caneruguz/osf.io/pull/17#issuecomment-176989103.

Thanks @sloria. :-)

Thank you!


[BUG] VLC plugin / play ID command not working anymore

Is this a bug in companion itself or a module?

[X] I believe this to be a bug in companion

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Hi guys, previously I was running a 2.2.0 RC / beta version from April 2021 on Windows 10 - everything was running stable and smooth, the "play ID" command of VLC module was working 100% reliable and smooth.
Last week I installed the lastest official release 2.1.4 and noticed that VLC "play ID" command is not working anymore. No errors, just the fact that nothing happens if the command is triggered.
I then downloaded the latest 2.2.0 RC2 and also in this version the "play ID" command for VLC is not working anymore.  So I ended up in "downgrading" to the 2.2.0 RC from April 2021 and everything is working again.
I tried to access VLC via browser using http://<IP_ADDRESS>:8080 address - it's accessible in all companion versions I have tried out. So I think the problem might not be located in the VLC module.
Steps To Reproduce

Install and setup VLC module with either latest RC2 or latest official companion build
Create new button with action "vlc: Play ID"
Open VLC player with playlist (make sure ID from step 2 exists in playlist)
Press "Test actions" button and see that nothing happens

Expected Behavior
No response
Environment (please complete the following information)
- OS: Windows 10
- Browser: Chrome 99
- Companion Version: 2.1.4-b03d4f0f

Additional context
No response

Did you read the help in the module?

IMPORTANT UPDATE!
If you used this module prior to the feedback and variables version, the Play ID has changed! The prior versions sometimes worked by starting the playlist at number 3. This version retrieves the current playlist from VLC and numbers from 1 at the top.The first item in the list is always number 1 which means if you click the heading and sort the list, number 1 is still the first one on the list.
VLC does not generate an error if you enter a Play ID that is not in the playlist. It will play the active item as if the Play ID was not there.
Due to the nature of the feedback and play list syncronization, the module status shows ERROR if VLC is not running yet. This is normal. It is to let you know that buttons may not work as expected. The status will automatically change to OK when you start VLC.

Hey @JeffreyDavidsz , thanks for your comment. Yes, I read the help and got that point with the playlist ID. I have a playlist with approx. 40 videos and if I start the video with ID 20 at least one video in the playlist should be played, I guess ;-)
Besides from that the same help text of the VLC module is already displayed in 2.2.0 RC from April 2021 so obviously that was already implemented in that version.

Ok I haven't checked it further but the command seems valid, so need to check the API

Looking over the module, nothing but cosmetic changes to the module since last August

@JeffreyDavidsz Thanks for your help!
@istnv  Okay, I only checked the function with version 2.2.0 RC from April 2021 and with official built from this January, nothing in between - unfortunately.

Closing. No comment for over a year.


style: update copyright

:white_check_mark: Build TheIdServer 4.3.1-ciba1-0014 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/b4a3e5084a by @)


*: use latest Go in tests

@jpbetz Can we also update https://github.com/coreos/etcd/blob/release-3.2/.semaphore.sh#L13? I already uploaded test container image with Go 1.8.7 and 1.9.4.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b309bc6). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #9303   +/-   ##
=========================================
  Coverage          ?   75.54%
=========================================
  Files             ?      365
  Lines             ?    30702
  Branches          ?        0
=========================================
  Hits              ?    23193
  Misses            ?     5884
  Partials          ?     1625

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 b309bc6...6b258bf. Read the comment docs.


Selection highlighting matches regex if activated in local search.

Unless it is a feature:
Correct:

Bug:

@SrTobi Yeah, the Ctrl+D respects the modifiers of the find widget. Match case makes sense, whole words makes sense, I wasn't sure what to do about the regex, so I left it to honor that as well.
I'm not sure how useful it is though... I personally never use it like that. Maybe I'll just take it out.

ok, your right. for Case Matching and whole words it does make sense.
Hmmm... I think it is a little strange for regex, though.
On the other side it is some kind of feature.

@SrTobi Thank you for your thoughts. Just pushed the change that the regex toggle is not taken into account in these cases :+1:


[Rancher2] Document new functionality for resource quotas in existing namespaces

Need to document in 2.6.5 once this Dashboard issue is completed. Re: Updating an existing namespace default limit for a quota at the project level will not result in that value being propagated to existing namespaces in the project.

See related PR.

Dashboard issue had been moved to 2.6.6. Moving this as well.


removed time.Now() when error in datetime function

Fixes #54
Removed the time.Now() in cases of error. Because this produced a not
reproducable output of the function.

Coverage remained the same at 94.357% when pulling 293e6713a04ad2429c4257a6e152b3130063918f on as27:54-refactor-dateime-file into 23e2339a207d61a114c8d11cfea573afb2939017 on frictionlessdata:master.


Rerender the rowHeight

When I change the  the input value of "DynamicDataSheetGrid",  I want to rerender the rowHeight according to my new input each time, how can I do that?

Dynamic row height is still in beta and undocumented, this is an issue I have not fixed yet :/


Done

Hi there!i have solved my assignment.Please check it out

Numpy exercise: For the intersection expected output is [2, 4]  yours is [2,2,4,4]
Also in numpy exercises last question you can also use extended slice for example:
a = np.arange(10)
a[::3]

Thanks 😅

So my  pull request is not acceptable?😶

Change that and again send a pull request it will be merged


EPIC: Result form updates

[ ] #1214 Result form updates to expand/collapse groupings
[ ] #1216 Result form updates to support disaggregation entry

Trying a different label based method for handling epics.


Expand ZCL structures which have an occurence in chip-tool in order t…

…o use simple types as command parameters
Problem
Currently chip-tool does not support ZCL structures when they are used as an argument to a command. The current PR expand the structure to simple types in order to not expose those structures to the high level API.
Summary of Changes

Expand structure to simple types in src/app
Expand structure to simple types in examples/chip-tool/commands/clusters/Commands.h

@vivien-apple can you resolve the merge conflict? Should be good to go after that

@vivien-apple can you resolve the merge conflict? Should be good to go after that

Done !


[$500] Settings - Delay Occurs When Navigating Back from Accounting Settings

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

Version Number: 1.4.43-2
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
1, Click on "Profile."
2, Click on the "Back" button to return from the Accounting Settings
Expected Result:
There should be no delay when navigating back from the Accounting Settings
Actual Result:
A delay occurs when navigating back from the Accounting Settings
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/98026db9-d597-4d33-838f-f576729969df
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0181bbb8ce76a8fa96
Upwork Job ID:<PHONE_NUMBER>551297024
Last Price Increase: 2024-02-20

Looks like regression from https://github.com/Expensify/App/pull/36309

Issue not reproducible during KI retests. (First week)

Agreed, can't reproduce it either. Closing this out.


Add 'what next' to quickstart

Add 'what next' to quickstart linking to the various tutorials
Add 'background tasks' guide
Move/consolidate the docs in http for sessions, middleware etc into their respective guides, making http a lot smaller and using links to the various guides.

Fixes #77

Thanks Brian! 😃


Angular 5 Support

Angular 5 was released nearly a month ago.  Could this get updated to support it?  This should probably come as a major version bump in this module.
Thanks!

@AHelper thanks for opening this. What is ng2-bs3-modal lacking in order to support Angular 5? As far as I know, it should "just work".

Ah, yes, I was scanning through our dependencies again and picked up on the ^4.0.0 devDependencies rather than the >=2.0.0 peerDependencies here. Sorry for the false alarm!


[TW#28894] aws-iot last will

I am fighting for some time with aws-iot last will. When i set plain text in last will there is no issue, but when i am trying to set simple json string then most of the time data is corrupted. Usually its missing 1 or 2 leading chars or sometimes leading bytes (1-2) are replaced with other random char.
I could add leading space if not the problem that sometimes 2 chars are corrupted and in very rare case there is no corruption in string.
Any advice or help will be appreciated because its in production app.

@chegewara Do you suspect this to be IDF side issue? If yes can you please provide sample code to reproduce it?

Hi @mahavirj
to be honest i dont know if this is esp-idf issue or aws broker itself.
This is code i am using to create last will message and few other topics. With regular topics i dont see this issue, it is only last will problem:
    char* name = get_fanet_addr(); // name = "1005";
    ESP_LOGE(__func__, "%s", name);
    char l_will[50];
    sprintf(l_will, "  {\"data\":\"%s\"}", name);

    IoT_MQTT_Will_Options will = iotMqttWillOptionsDefault;
    will.pTopicName = "/last_will";
    will.topicNameLen = strlen("/last_will");
    will.pMessage = l_will;
    will.msgLen = strlen(l_will);

I would suspect my error if this would be trailing character, but with leading characters corrupted it has to be something in library or aws.
Thank You for help.

@mahavirj What i can say in addition is:
when i perform test from aws IoT core and publish to /last_will topic then all works fine. This is suggesting that message sent to aws broker from esp32 aws-iot component is corrupted (some memory allocation problem maybe?).

@chegewara Sorry for late followup, did you analyze this further? Interim we moved aws-iot to independent repository at https://github.com/espressif/esp-aws-iot, in case this still exists I will move it over there.

Hi @mahavirj
since there was no response i left it with settings that worked in my app which is sending just simple plain text, one word or to be accurate numeric value as a string (ie 1005).
Because my project is in advanced stage, some devices are working now, i cant move to aws-iot repository, but thank you for info.


Add cuda to pm-gpu yaml files for spack builds

This is needed for Trilinos and Albany builds that include cuda.

Checklist

[ ] Testing comment in the PR documents testing used to verify the changes

Testing
I used this PR to build a spack environment with Trilinos and Albany, and then successfully used that to run MALI on Perlmutter-GPU.  (There were some restart issues that do not appear to be related to cuda support.)


1Password secure notes editor

We should be able to load secure notes items into a buffer, set vim.bo.filetype = 'markdown' (since I think 1Password secure notes uses markdown), set the vim.bo.buftype = 'nofile', and edit 1Password secure notes in Neovim.
As for updating, there's one of two approaches we can do, the first one needs some research:

Write the buffer to /dev/null and set up BufWritePre autocmds to update the secure note in 1Password (important: would need to validate whether writing to /dev/null compromises the security of the note in some way)
Just use a separate command to update the secure note in 1Password, like :OpWrite or :OpSaveNote` or something like that

This could actually be a really awesome way to use 1Password secure notes. If I can get it working well enough/seamlessly enough it might even replace Notion for me.

After a little research, first option cannot be guaranteed to be secure. It is possible to replace /dev/null with something else if you have administrative privileges. While this is extremely unlikely, we shouldn't even open the possibility.
The buffer must be buftype=nofile and use a separate, dedicated command for updating in 1Password.
@tsyroid thoughts on:

:OpWrite (possibly with :OpW as a shorthand?)
:OpSave
:OpSync (though maybe this should also reload changes from 1Password into the buffer?)
Something else

Should also wait for #26 so we can update the notes in 1Password asynchronously.

Another idea is we could split note titles on > to make a simulated hierarchy/directory structure and make a 1Password secure notes sidebar.


Describe Sysman extension discovery (#149)

@wdamon-intel @sanilkumar0 @jandres742 @saik-intel please review

Since here we are defining some new naming conventions (finally), I think it would be better to leave the PR open until next release, to have time to receive community feedback.


Modified libcupsfilters to use libcups 2 and 3

Based on https://github.com/OpenPrinting/libcupsfilters/pull/24 by RudraGayu
Minor updates have been made over the above mentioned PR.

Solved differently via commit 26f9c3fb7c
But thanks anyway for your pull request which provided a good base for the final solution.


Integrate Google Maps API to determine restaurants in range

NutritionIX requires a 'query' string when we filter by macros.
We need to gather the user location, and a radius of how many far the user wants to look for food.
Use the google maps API to determine how many restaurants are within radius distance from the user, grab all of those names, then include it in the 'query' parameter to fetch from nutritionIX

This issue is being put in the backlog due to competing priorities.
Researching Google APIs unfolded a credit card was required to receive an API key, which we were not comfortable providing despite the $300 start-up credit. For this reason, we will limit the scope of restaurants to those available in Victoria for the time-being, which can be done for free.


gprc blocks on create new account

Possible conflict with javaee wildfly 12 libraries

There seems to be something related here https://github.com/grpc/grpc-java/issues/4544
Notice that this error happens only when sdk is run under wildly; it does not come up when using the demos in a lean jvm. According to the linked ticket, it seems that some queue is getting full too quickly (and I don't think that throttling would resolve this one)


Unable to pull ack-s3 charts manifest from ecr

Describe the bug
I was following the installation instruction and unable to pull the chart manifest for s3-controller with version v0.0.5 and above.
Also, it seems the link to the installation docs under the usage tab on the ecr is broken :)
Steps to reproduce
➜ HELM_EXPERIMENTAL_OCI=1 helm pull oci://public.ecr.aws/aws-controllers-k8s/s3-chart --version v0.0.5 -d /tmp/chart
v0.0.5: Pulling from public.ecr.aws/aws-controllers-k8s/s3-chart
Error: manifest does not contain a layer with mediatype application/tar+gzip

Hi @senior88oqz , I was wondering what is version of helm that are using.
The recommended way to install an ACK service controller for Kubernetes is to use Helm 3.7+.

Thanks @vijtrip2 , i was on v3.6.0, working fine now after upgrading to v3.8.0


merge from template-kronos-app

package.json

chore(deps): add ^10.10.0 remove ^10.9.2 (devDependencies.npm-pkgbuild)

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

npm package (@latest dist-tag)
v1.10.151
GitHub release

Your semantic-release bot :package::rocket:


Add GPT code reviews.

:rocket: Feature Request
Feature description

Implement a code review feature using ChatGPT to analyze and provide feedback on the following aspects of the code: quality, duplicate lines, performance, typing, and adherence to Google style docstrings.
Rationale and use cases

Code reviews are essential in identifying and addressing potential issues in codebases. Automating code reviews using ChatGPT will help developers identify problems, maintain consistent coding standards, and improve overall code quality. This feature would cater to developers seeking an AI-assisted review process.
Proposed implementation (if applicable)

Create a function to analyze code snippets and identify low-quality code, duplicate lines, performance issues, typing problems, and adherence to Google style docstrings.
Use ChatGPT to generate insightful and actionable feedback on the identified issues.
Allow users to submit code snippets for review through the API or chat interface.

Potential drawbacks and alternatives

The accuracy of ChatGPT's code analysis may not be on par with human reviewers, potentially leading to overlooked issues or false positives.
There might be a learning curve for developers to understand and interpret the feedback provided by ChatGPT.
Alternative solutions could include integrating existing linters and code quality tools or exploring other AI-based code review tools.

The following prompt seems to work well: Check the code for the following issues: low quality, duplicate lines, performance, typing and Google style docstrings:


Query/Feature Request - remove/hide Drag Components button with config

Hello,
I am working with the SPA for my component, the component uses the Responsive Grid to wrap two child components.
e.g.
Responsive Grid ->

Sub Component 1
Sub Component 2

However, in this application, we will always have only two components but the spa adds the 'drag components' button, which takes up space and looks out of place.
Is there a way to tell the spa to not add the 'drag components' for the responsive grid?
Alternatively is there any configuration in AEM that might achieve the same effect?

Hi, @TobyGWilliams

Is there a way to tell the spa to not add the 'drag components' for the responsive grid?

Do you then need the responsive grid at all? Instead, you may want to extend the Container class and get rid of the notion of ContainerPlaceholder. Backend side, you can decide to keep the current container model or optimize the payload by exposing a simpler Sling Model.

Alternatively is there any configuration in AEM that might achieve the same effect?

Not OOTB
[0] https://github.com/adobe/aem-react-editable-components/blob/master/src/components/Container.tsx
[1] https://github.com/adobe/aem-react-editable-components/blob/master/src/components/Container.tsx#L148

Thank you for your help, this prodded me to realise that I need to modify our extension of the responsive grid component for this case. In the end it was pretty simple, check for scenario and then omit placeholder.
Thank you for your help.


Can I train with a Mandarin Chinese data set?

Can I train with a Mandarin Chinese data set? What format should the data be in

Hello,
I have tried to summarise how to train it on a different dataset in the wiki here:
https://github.com/shivammehta25/Matcha-TTS/wiki/Training-🍵-Matcha‐TTS-with-different-dataset-&-languages
Please see the link above :) Let me know if you have any questions or require any clarification. Feel free to reopen in case of questions.

感谢大佬的回答，我使用中文数据集 合成出来的都是噪音，感觉可能是因为我的对中文数据集做的音素化处理和symbols使用的不对。我正在修改，数据的输入方式，再重新训练。
Thanks for the big man's （in Chinese it means Superman or handsome man ）answer. All the data I use in Chinese data collection are noise, which may be due to my incorrect phonemization processing and use of symbols in Chinese data set.I use Chinese data set to train and synthesize all the noises, which may be because of my incorrect phonemization processing and use of symbols in Chinese data set. I'm modifying the data input and retraining.


requirements > langchain-serve is no longer maintained, any alternative?

Looks like this dependency is no longer maintained. Is there a replacement for this?
https://github.com/bhaskatripathi/pdfGPT/blob/3cfa4e8ba8d5a2d19512c3ce4ccf7d52f1c7a1ab/requirements.txt#L8

Having the same issue, i try different things but nothing work.

Maybe ollama?
https://python.langchain.com/docs/integrations/text_embedding/ollama
https://github.com/ollama/ollama?tab=readme-ov-file#rest-api

Hi Guys,
I understand that it Langserve doesn't provides support now. In the hindsight, I acknowledge that I shouldn't have accepted their PR. However, you can use the original code of PDF-GPT from this location to build, deploy or customize it as per your requirements:
https://huggingface.co/spaces/bhaskartripathi/pdfGPT_Turbo/blob/main/app.py
https://huggingface.co/spaces/bhaskartripathi/pdfGPT_Turbo/blob/main/requirements.txt


Texture to buffer copies

PTAL, this will be important to be able to check pixel values in the end2end tests. I didn't actually test that the Metal implementation works though.
Part of #53.

Thanks all for the reviews


[SP-2154] Backport of PDI-14124 - While running a transformation in a…

… job on a slave server, the getSystemInfo parent batch id always returns 0/null for first level (5.4 Suite)
@mattyb149, @brosander, @pamval review it please

I'm getting a bunch of errors on running codestyle over the changed files.
Can you review ?

@pamval ,
I checked kettle-engine again and did not get errors over the changed files (run usual "ant checkstyle").
Could you share the errors?

No checkstyle errors found on my box. merging


Font in the guide column

Hi!
When I open csv files for editing. I notice that the content in the Guide column is special characters (HYeQ[!t+v+ ; emfZ/!E,?[....). I don't know if it's a font error or what. thanks for any help!

Hey @vuxvix, feel free to ignore the column guid, those special characters are used by Anki to keep reference of notes.
It stands for Globally Unique Identifier (guid). It is used internally by Anki to identify notes. By storing this in the CSV we can safely update meanings, readings etc. and update decks without affecting your progress in Anki.
If you want to contribute new readings, don't worry about that field either. This project will autogenerate it.

I should probably do a better job explaining the format so I've added a section to https://github.com/jamsinclair/open-anki-jlpt-decks/blob/main/CONTRIBUTING.md#csv-format

Hi jamsinclair,
Thanks for your quick response. I'm looking for a way to show the next columns in Back Templete. It's not very intuitive to me.

show the next columns in Back Templete.

I think your message may have some typos. What do you want to display on the back template?

With some additional time spent learning Anki, I m now able to add content to the backs temp.


Please add an menu entry in "Windows" for opening the VM-Library

With an running a windowed VM, it is impossible to re-open the VM-Library. One has to stop all windowed VMs to open it, which is very annoying. :)

Duplicate of  #3632


Remove reliance on for..of

Fixes https://github.com/sabeurthabti/react-native-css/issues/60

@anzorb I pushed a fix for #60 - please test it and if that doesn't fix then we can merge this.
thanks for the PR


add option for FileType.AUDIO to not open apple music

I want my iOS users to be able to search the Files app for audio, not apple music files, to avoid DRM stuff.
problem with using .CUSTOM is I can't filter multiple extensions at once, at least until #68 is implemented.
so at the moment .ANY with custom handling on the app side is the only way. which seems more cumbersome than
basically this issue is moot if #68 is on the horizon, otherwise it would be nice if .AUDIO could have an option to open the files app on iOS
edit: also, excuse the ignorance, but if the audio files are DRM'd, what can one even do with it when picked in the file_picker?

This is in a way a duplicate of #68. Please, use the #99 to upvote on features.
Thank you!


syscall: the wrong sysctl was gimped on iOS

In issue #34133, @mmaxim reported that the app store was rejecting his app due to the use of __sysctl, which he blamed on the Go runtime. Others didn't seem to have the problem, but @mmaxim said that things were accepted by the app store if he went back to Go 1.10. @eliasnaur considered this to be confirmation of the problem and removed sysctl on iOS from Go 1.14 and backported that to Go 1.13.3.
Unfortunately the diagnosis was incorrect. __sysctl is undocumented, but sysctl is documented. The Go runtime uses sysctl. However, x/sys/unix uses __sysctl. @mmaxim noticed that Go 1.10 worked for him, because going back that far switched his vendored x/sys/unix over to using raw syscalls instead of going through libSystem.
This is a regression, breaking wireguard-apple's use of net.InterfaceByName, which uses sysctl under the hood.

CC @crawshaw

@gopherbot Please backport this to 1.13, because it fixes a grave regression in the net package introduced in 1.13.3.

This change broke macOS 10.14 at least.

import (
	"golang.org/x/sys/unix"
)

func main() {
	unix.Sysctl("kern.boottime")
}

Compiling this program results in:
golang.org/x/sys/unix.libc_sysctl_trampoline·f: relocation target golang.org/x/sys/unix.libc_sysctl_trampoline not defined

The darwin assembly files still refer to __sysctl
zsyscall_darwin_386.s
43:TEXT ·libc___sysctl_trampoline(SB),NOSPLIT,$0-0
44:	JMP	libc___sysctl(SB)

zsyscall_darwin_amd64.s
43:TEXT ·libc___sysctl_trampoline(SB),NOSPLIT,$0-0
44:	JMP	libc___sysctl(SB)

zsyscall_darwin_arm64.s
43:TEXT ·libc___sysctl_trampoline(SB),NOSPLIT,$0-0
44:	JMP	libc___sysctl(SB)
``


Create Deployment Documentation - Development

In this task we want to fill out the template for the DEPLOYMENT.md from the documentation skeleton.

ACs

[ ] The DEPLOYMENT.md template has been filled out

Background information

Filled out template from telemetry

This issue was completed within #709
If we want to extend the current documentation we will create a new issue


Is GpioChangeReader not supported with Lightning ?

Hi,
When I try to use the GpioChangeReader with the direct memory mapped driver I get an exception.
This even occurs in the insider preview version : v.10.0.17035.1000
Is it supported in lightning or is it an implementation error on my side ?
When I switch to the inbox driver everything works just fine.
Best regards

Sorry for the delay in responding. GpioChangeReader is not supported with Lightning (it uses a completely different API). If you need change reader, please use the standard driver.


Change default max_block_time to 30s

Closes: #1998

PR author checklist:

[ ] Added changelog entry, using unclog.
[ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[ ] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

@ancazamfir, I noticed that we're using the max_block_delay from the ChainConfig to also fix https://github.com/informalsystems/ibc-rs/issues/1445 (Update client may cause "new header has a time from the future" chain error). My understanding is that that value (and it's default) is the same as maxExpectedTimePerBlock which is defined in ibc-go here, which is also used in the connection block-delay calculation in hermes in #1917. Can you confirm if this is correct? And if it is okay to set the default to 30s?

AZ: We shouldn't keep max_block_time as a config param in Hermes. Instead, programmaticaly fetch it from the chain (genesis param) and cache it in the relayer.

AZ: We shouldn't keep max_block_time as a config param in Hermes. Instead, programmaticaly fetch it from the chain (genesis param) and cache it in the relayer.

I opened https://github.com/informalsystems/ibc-rs/issues/2037 to address this.


Class support for RefCounted

jakobovrum (@JakobOvrum) reported this on 2015-02-11T03:55:16Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=14168
CC List

nick (@ntrel)
r9shackleford

Description
Instantiating RefCounted with a class or interface type should reference-count the actual instance and not just a class reference. To this end, RefCounted currently rejects instantiating with class types for forward-compatibility purposes (at least I'm pretty sure that's why it does it).

jakobovrum commented on 2015-02-11T04:09:47Z
(In reply to Jakob Ovrum from comment #0)
> Instantiating RefCounted with a class or interface type should
> reference-count the actual instance and not just a class reference. To this
> end, RefCounted currently rejects instantiating with class types for
> forward-compatibility purposes (at least I'm pretty sure that's why it does
> it).

I filed a PR to reject interface types as well until class support is properly implemented:

    https://github.com/D-Programming-Language/phobos/pull/2977


Memory consumption during training

❓ Questions & Help
I'm trying to train pointnet_segmentation on my own data set with graphs of roughly 100k nodes, 400k edges, and node features comprised of 6 floats. But even with a batch size of 1, I get an OOM on my GPU with 11GB.
I tried to use pytorch_memlab to asses where the memory is occupied, but only get very limited information:
-------------------------------------------------------------------------------
Total Tensors: 44640170 	Used Memory: 225.47M
The allocated memory on cuda:0: 228.88M
Memory differs due to the matrix alignment or invisible gradient buffer tensors
-------------------------------------------------------------------------------

What are your recommendations for approaching this problem?

Sadly, your graph is too big to be processed at once :( You could sample subgraphs during training, but IMO the best approach is to divide your data into many (overlapping) subgraphs as a pre-processing step, e.g. like it is done in the S3DIS dataset.

Thanks for your input. Unfortunately, it's not straight-forward to divide the graphs without losing valuable information. Can you tell which parts of the model are especially demanding? Or even better how to look into the memory consumption myself? Any specific tool or technique you're using yourself?

In each layer we create a [num_edges, num_features] matrix that needs to be stored in memory for the complete training cycle, so this is not really efficient but sadly cannot be avoided. I am fairly sure that PyTorch provides ways to access the tensors that are currently referenced, and this should give you a feel about the memory consumption of your model.


Update bitcoin_cigs.rb - Getting groestlcoin fixed

WIP getting groestlcoin to verify

Tested, no code breaking changes it appears. although groestlcoin still does not verify. wip.
these changes pave the way for groestlcoin to work, and also fix bitcoin as there was a code breaking change previously.


Too many warning messages that aren't meaningful or actionable

Recent tester feedback:
"The user experience could be improved by not displaying the many warning messages that are not meaningful or actionable by the typical user."

I agree, it would be nice to streamline any user-facing messages.  @ways2read did you have specific things that stood out to you?

Info from a call: this issue is about the script messages, not dialog boxes that might pop up. Request to simplify presentation in the UI.
Also -- when dealing with all these messages appearing constantly, the screenreader gets overwhelmed and keeps speaking and users don't get important updates like that the job has finished.
Ideas:

collapse entire messages section
just show warnings
turn off aria-live for the messages
show warnings and show last info statement per nesting level

Decided on the team chat to do this one in addition to removing aria-live (already done): "show warnings and show last info statement per nesting level"

The commit above does what we said above ("show warnings and show last info statement per nesting level") except it doesn't show the warnings out of context.  The approach taken here is to show the most recent message per nested level except when there is a warning. In that case, the ancestor messages of the warning are also shown.
The other option is to get rid of the nesting and show the warnings out of context. This would result in fewer messages remaining in the UI messages view at the end of the job.
@bertfrees want to have a look and let me know what you think? I would post a screenshot but it's more meaningful "live" e.g. when running a job.

Discussed https://daisy-dev.slack.com/archives/C064GB8U9/p1719593846632609

Description for the latest commit here:
Turning verbosity on automatically turns warnings on.
Errors are shown regardless.
In non-verbose mode, the outer two levels of messages are shown in a flat list.
In verbose mode, all messages are shown in a hierarchy.
How this looks in practice depends on the script. DTBook to braille looks good; DAISY 202 to EPUB 3 does not have nested messages so verbosity control does little here.

Scripts vary in their messages - some report nested messages, some report a lot of things, some report very little. So the idea of "verbose" is hard to define relative to this.
After trying a few approaches, I think the most consistent and easiest is if we show errors and warnings by default, and then show everything if the user chooses.
This is what's available in the test build here:  https://github.com/daisy/pipeline-ui/actions/runs/9726989779

I think the most consistent and easiest is if we show errors and warnings by default, and then show everything if the user chooses.

I think this solution is fine. Let's see what the users say.


A fatal error has occurred, the connection is terminated

Two of my friends tried going to the moon in my server with the Galacticraft mod and many other add-ons, they got timed out right before going out of the atmosphere, and when they rejoin the error in the title occurs. Now they are stuck in outer space, unable to join
Here is a log consisting of just one of them joining and disconnecting https://mclo.gs/CyksIoO (yes Aternos because dad doesn't let me pay)
Update: I am stuck too ;-;
Another update: it only seems to happen in this server

update i am stuck too
the galaxy map pops up for one second and then

good news! after digging into my client logs it was an issue caused by the galaxyspace addon where i forgot to disable the enableNewGalaxyMap option

tysm i was looking for this, i thought i replaced all the configs but apparently the thing in asmodeuscore didnt save to false. when i opened the config on the aternos server it said enableNewGalaxyMap=true


return {:error, reason} if a connection dies during a Gnat.request call

I was starting to work on this when I realized that GenServer.call already monitors the process that it is calling. So if someone tries to call Gnat.request and the connection has already died, or dies during that request an error will be raised.
Returning an {:error, reason} is slightly better than raising an error, but I think it's good enough for now


Specifications

| BEDHEATERFILM1 | Kapton Film Heater 200mm 100W | Bed heating
For this part- the first link is of robotdigg & there's 6 options to choose from :
PL2020-12V
PL2121-12V
PL2525-12V
PL2020-24V
PL2121-24V
PL2525-24V
Please specify which one should we get. I cross checked with price to find a match but all options cost the same 12 USD & you haven't specified the voltage.

Hello, it is 24V, 100W heater, so it should be PL2020-24V. I’ll add this
spec in.
On Thu, 9 Jun 2022 at 20:09, antordey51 @.***> wrote:

| BEDHEATERFILM1 | Kapton Film Heater 200mm 100W | Bed heating
For this part- the first link is of robotdigg & there's 6 options to
choose from :
PL2020-12V
PL2121-12V
PL2525-12V
PL2020-24V
PL2121-24V
PL2525-24V
Please specify which one should we get. I cross checked with price to find
a match but all options cost the same 12 USD & you haven't specified the
voltage.
—
Reply to this email directly, view it on GitHub
https://github.com/KRALYN/PositronV3/issues/13, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP6KUFL6AES425R7YUDQPH3VOKBTTANCNFSM5YLYNAKA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[BSO] Dyed melee Gorajan pieces dont count for KK torva boost

🐛 Bug Report
What title says. Before I dyed my Warrior helm, I had a 8% boost for Torva items. Once I dyed it, that boost became 4%. The Gorajan warrior set boost functions properly though

Fixed in next update


Feature Request: Input field styling more consistent with Material-UI's TextField

Is it possible to integrate the blue sliding animation when the input is brought into focus, as shown in Material-UI's documentation for autocomplete?

This would make the styling more consistent.

@devsdevsdevs Hmm, I would like to keep the field consistent, but having trouble figuring out why it's not.  It should be handled by the underline styles of Input but not sure if it has to due to focus not registering in the Input correctly somehow (it appears to track focus in it's state).
Will take more investigation but if you have any ideas, I'm always open to reviewing a PR

@devsdevsdevs Found the issue and is now resolved in 1.0.3


[CVCUDA/Holoscan Interop] Set pointer to null after freeing them

As Stroustrup mentions, deleteing/freeing a pointer would not necessarily set it to null. It is good idea to set such  a pointer to null.

Stroustrup's comment: https://www.stroustrup.com/bs_faq2.html#delete-zero


[2.7.x] Fix CountDownLatch not being released(#8845)

What is the purpose of the change
Fix #8845

Codecov Report

Merging #8862 (4cce7a9) into master (e39e494) will decrease coverage by 0.70%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master    #8862      +/-   ##
============================================
- Coverage     60.75%   60.05%   -0.71%
+ Complexity      448      446       -2
============================================
  Files          1100     1100
  Lines         44422    44422
  Branches       6468     6468
============================================
- Hits          26988    26677     -311
- Misses        14462    14781     +319
+ Partials       2972     2964       -8

Impacted Files
Coverage Δ

...he/dubbo/registry/zookeeper/ZookeeperRegistry.java
60.24% <83.33%> (ø)

...ookeeper/ZookeeperDynamicConfigurationFactory.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...gcenter/wrapper/CompositeDynamicConfiguration.java
0.00% <0.00%> (-63.34%)
:arrow_down:

...pport/zookeeper/ZookeeperDynamicConfiguration.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...fig/configcenter/TreePathDynamicConfiguration.java
31.42% <0.00%> (-54.29%)
:arrow_down:

.../dubbo/metadata/report/MetadataReportInstance.java
12.50% <0.00%> (-40.63%)
:arrow_down:

.../configcenter/support/zookeeper/CacheListener.java
0.00% <0.00%> (-34.29%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-27.59%)
:arrow_down:

...gistry/client/migration/MigrationRuleListener.java
55.88% <0.00%> (-23.53%)
:arrow_down:

...mmon/config/configcenter/DynamicConfiguration.java
68.18% <0.00%> (-18.19%)
:arrow_down:

... and 42 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 e39e494...4cce7a9. Read the comment docs.

Please check if should patch in 3.0 branch


Enhancement - html title tag

Would be great to be able to change the html title tag as well as the icon so when you look at the tabs in e.g. chrome you can have a different name as well as a different icon for different instances.

Great idea!
I normally have so many tabs open I don't even see the titles...


Implement dpy-self-like browser info fetch in HttpRequest

7bd8236c1e799f32d3142183f6c913287313d7f1
Needs testing

todo: Undo vscode linting


bug: 3 Test fail

Cloned repo, ran anchor test and 3 test fail

Came here to post this as well. The repo doesn't run out of the box, those three tests fail.

It's Ok now.


Fix invalid catch block in checkSelfUpdate()

This fixes the unexpected crash when there is no internet connection.

Do you think it would be possible to make it to where the app would run if there was no internet connection to allow for people to restore any backups they have?

Yeah, it's possible; if no one else gets to it first, I will try to have a new PR ready by the end of next week.

Awesome, thanks so much!


Prevent gas price manipulation

If the block producer receives the whole fee from transactions included in the block, he can generate many transactions, increasing gas/storage prices and receiving spent tokens back. It allows him price manipulations for free, making the transition very expensive for the end-user.
Maybe we want to consider something like eip-1559, but instead of burning base fee tokens, we could distribute them between fuel token holders.

Let me rewrite it simpler, break it down a bit. So we check if I understand correctly:
Potential problem: Price manipulation, Block manipulation
Potential cause:

Putting transactions into block is practically free for block producer

because all fees they pay , come back to them in 100% from block they produced

Potential outcome:

manipulation of fees, inflation of fees

e.g. to levels very expensive for end users
so block producer can get back fees they paid + gains on what other users had to pay
incentive to inflate prices: to get more from users
the more block producer has, the harder can inflate during this attack

MEV

because it's practically free for block producer to put transactions, creates unmet MEV opportunities

This concern seems more applicable in a decentralized sequencing model, as a single sequencer chain can affect prices in a multitude of more straightforward ways (i.e. at the mempool level) in order to cover expenses (i.e. DA upload or node hosting costs).


Improves GetContractDetails Error Message

Expected Behavior
When we can find the details for a security, inform the user that this security was not found in IB database.
Actual Behavior
We receive the following message:
No security definition has been found for the request. Origin: [Id=42] GetContractDetails: NVDL (CFD NVD L SMART)

which is not very helpful. It means that the CFD for NVDL was not found in IB database. If we search for it in the Produce Search page it cannot be found.
Potential Solution
Catch that error message and explain we couldn't find it on the IB dataset page. We could suggest using TWS search to verify whether the symbol is supported.
Reproducing the Problem
self.add_cfd("NVDL", market=Market.INTERACTIVE_BROKERS)

Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

Replaced by https://github.com/QuantConnect/Lean/issues/7972


Fixed translations mistakes

I have fixed some translation mistakes as well as concepts that differ in spanish from english.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thanks!


Fix check in IsTokenChar

Hi @callumbwhyte
I recommended somebody to use this package on the forum : https://our.umbraco.com/forum/using-umbraco-and-getting-started/111021-examinemanager-search-in-certain-tree#comment-343501
This person is not able to get a query on path working. I knew we had the list value type in one of our own projects (we did not need the entire package). Turned out my colleague was having the same issue and already fixed in our project.
This PR fixes the issue in the package.
Dave

Great spot @dawoe! #h5yr!
It looks like this was overlooked when I migrated to v9+.
I'll get this merged later today and have a release out on NuGet.

Thanks again @dawoe, this is released as v3.0.1 now on NuGet!


IDVA6-691 your companies update invitation link

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-691
Changes:

A banner replaced with a link
Code refactored to remove duplications

CI: No security warnings found


Allows overriding $merge method in resource implementation class

I'm not sure if that Resource.prototype.$merge call was on purpose (can you clarify it?), but with that I'm unable to override method $merge in descendant class and make it not merge some fields (which I need in my application)

This looks good. I will take a closer look and merge shortly.

:+1:

@besuhoff Do you need this released?

@jcassee,  I'll answer for @besuhoff . We are currently using your library in our project and yes it would be very desirable to release new version to npm.

Alright, just released 0.7.6.
@besuhoff Heads-up: you forgot (and I missed it too) to run gulp before committing so dist/hypermedia.js (which dependent projects use) was not updated. Fixed in the commit that updated the version.

Thank you @jcassee!

@jcassee I'll also appreciate a lot if you release a new version of npm package at npmjs.org

Done now! Thanks for reminding me.

@jcassee thanks a lot sir!

@jcassee sorry to disturb but npm registry is still at 0.7.5 https://www.npmjs.com/package/angular-hypermedia

Weird, I ran npm publish again, and it's now updated to 0.7.6. Sorry for the confusion!

No problem sir, thanks! I now see it there


Force to use memory cache not working: cacheType is 'none'

Hi everyone,
Kingfisher is an awesome library, but I can't seem to implement the following:
In order to resolve a slow/jittery UICollectionView on initial scrolling, I checked out Time Profiler and I believe the jitter comes from UIImageView.kf.setImage() which would make sense. I am already using the kingfisher prefetch method with .cacheMemoryOnly to prefetch the images used in the cells of my collectionview long before actually reaching the view where the uicollectionview resides. Then when I populate the cells I use UIImageView.kf.setImage() with the .fromMemoryCacheOrRefresh option, since I believe the lag is caused because it is loading the image from Disk and not fro Memory.
However! when I print out the value.cacheType in the setImage completionHandler, the first results are "none" not "memory" as expected. It is only after scrolling up and down a couple of times that the cache type indeed becomes "memory".
So my question is the folloying:

am I correct in assuming that loading my cell's images from memory instead of disk will improve performance of my UICollectionView?
if so how can I force KingFisher to not use Disk or Network-fetching, but simply prefetch all the images in memory and load them from memory when the time comes. I simply cannot find the correct settings to achieve this behaviour.

Thank you!

.cacheMemoryOnly means Kingfisher will skip caching the image to disk.
.fromMemoryCacheOrRefresh means Kingfisher will try to find an image from memory cache first. If not matched, it will skip searching from disk cache, but download the image again.
The memory is limited on a device. Kingfisher restricts itself to use memory with an upper limitation. If you cache a lot of images in memory, old or unused ones will be purged.

So your experience above is not strange. You indeed downloaded the images and cached them in memory, but you didn't cache them to disk. Later, when you want to show them in a collection view, these images are removed from the memory cache, so you need to download them again, which brings an awful experience.
Now, for your question.

Yes, loading cell's images from memory instead of the disk will improve your time performance. But the cost is more memory usage (you can set a larger number to Kingfisher's memory cache to allow more images to be cached).
I personally suggest you can remove these options (.cacheMemoryOnly and .fromMemoryCacheOrRefresh), just use the default settings. It is designed carefully to get a balance between memory and timing. Furthermore, even loading images from the disk cache will not block your main thread. So your Time Profiler should show Kingfisher's image loading time consuming mainly happens in a background thread. It will not block your UI at all, so the timing should be just fine in most cases.

IMO, it is fine to use ImagePrefetcher to fetch images before where they are really needed. However, it is originally designed for table view and collection view data prefetching. So I suggest just use it in a UICollectionViewDataSourcePrefetching delegate to download only several images before they are about to be shown. This type is NOT DESIGNED as a good general-purpose image downloader for thousands of images (although you are not forbidden to do so).

You indeed downloaded the images and cached them in memory, but you didn't cache them to disk. Later, when you want to show them in a collection view, these images are removed from the memory cache, so you need to download them again, which brings an awful experience.

But why? I just downloaded them and forced them to be cached in memory, why do I need to download them again? How does Kingfisher deside when the 'unused' images should be redownloaded? I prefetch them the first time I arrive at the scroll-view, once the user scrolls to the second view in the scrollview the uicollectionview appears... why does it print out that the images are loaded from cahce type 'none' instead of 'memory'. It just doesn't make sense to me :(
But thnx for the explanation :)

Hi,
Yes, I can understand your confusion. But as I mentioned, there are some limitations on the memory cache. If these conditions are met, the underlying NSCache will purge the cached objects and release them.
By setting .cacheMemoryOnly, you say you want to only use memory, but to skip the disk cache. It does not mean "forced them to be cached in memory". All loaded images will be cached to memory until the cache limitation has been reached.

You skipped the disk cache by .cacheMemoryOnly.
The cached images have been purged due to limitations reached.
Then you want to load these images back, Kingfisher cannot find them in the cache.
So it triggers a redownload.

That's the whole story.
Please read the doc to know more about .cacheMemoryOnly and .fromMemoryCacheOrRefresh. Again, I suggest you just use the default settings or follow examples in the cheat sheet as the first step with Kingfisher until getting more details.
Best regards.

I totally understand now. thanks for the explanation!
As I was getting redownload behaviour pretty early after downloading 20-50 images each of around 50-200 KB in size, maybe using a custom cache with more memory space, would make it work?
If I change the cells to use a static asset image using 'UIImage(named: "") in the 'cellforitem'-method, I do not have the laggy behaviour. If my images are cached in memory, should I get the exact same smooth behaviour as using UIImage(named:...) ?
Thank you, and great work with the library 👍
using a configured Cache with lots's of memory space would that help/force the images to be cached to memory?
Also since I use the default settings and images are already redownloaded after prefetching around 15 images, maybe using a compressor and only caching the

As I was getting redownload behaviour pretty early after downloading 20-50 images each of around 50-200 KB in size, maybe using a custom cache with more memory space, would make it work?

Yes, it should work better for your case. Instead of the encoded size (50-200KB) in disk, the pixel bitmap data (the height and width of the image) in memory is more important to determine the memory usage. If you are loading some big images, the memory purge could happen easily. If the image is much larger that what you need (the size of your image view), I suggest you can apply a DownsamplingImageProcessor when setting image (like this).

If my images are cached in memory, should I get the exact same smooth behaviour as using UIImage(named:...)?

Yes.


[Hardening] Have one atomic function call on compass for all handovers

Instead of making n calls to change ownership of ERC20 tokens and fee manager and potentially ending in a situation where some calls went through and other don't, it would be faster and more efficient to have only one function on compass to call, that forwards handover calls to all tokens and the fee manage.
Since those will all be in the same TX, it will either go through or fail.

@webelf101 Here's the suggested implementation:

Make  update_compass_address_in_fee_manager  an internal function.
Add  a new public function compass_handover()

That new function should:

Verify the call was made from Paloma and assert consensus
Accept a batch of submit logic call arguments
For each item in the batch, execute  the submit logic call (ignoring fees)
Lastly, run update_compass_address_in_fee_manager

That way, when we upgrade compass, we can call the old compass once. The submit logic call batch would contain the calls to new_compass() for each deployed ERC20 token. That way, we can make sure that ALL dependencies will be updated in one atomic transaction.
This would be a great improvement on the current situation, where we have to make n calls instead of one, and deployments can get stuck if one or more tokens fail to be updated.

https://github.com/VolumeFi/paloma-compass-evm/commit/ee37fff6a3ab78f47263d8e14b7518135a4856ed
Can this solve this issue? @byte-bandit


no pip yet?

tried to install keras with pip install keras, but what I got looks like this:
Collecting keras
  Could not find a version that satisfies the requirement keras (from versions: )
  No matching distribution found for keras`

You can install it with pip using the following command:
pip install git+git://github.com/fchollet/keras.git
The library is not available at PyPI yet as it is still alpha. We should probably think about releasing it on PyPI after we reach a stable version.

+1! Tag with a version number < 1.0 and release on PyPi.

thx~


feat: express list of contributors of an element(component, ...)

We currently do not have a way to track the full list of all contributors for a component version. Since contributors could be different from authors, we need a better mechanism to track individual committers and authors using nested components or externalReferences (which could be a URL or BOM-Link).

releaseNotes could also benefit from dedicated organizationContact attributes to track contributors, reviewers, and releasers.


Enhance Application Compatibility with PostgreSQL

Summary
The application currently faces compatibility issues when configured to use a PostgreSQL database. This issue is to outline the specific problems encountered and propose a set of enhancements that will improve overall compatibility with PostgreSQL. These changes are expected to benefit not just PostgreSQL users but potentially enhance the robustness of database interactions for other databases as well.
Problems Identified

ENUM Type Naming: The application fails because PostgreSQL requires ENUM types to be named. Several ENUM declarations are missing names, leading to a failure in creating the database schema.
Configuration Handling: The application does not find the config.ini file and does not clearly handle or report this situation, leading to a confusing startup experience.
Database Permissions and Schema Existence: The application assumes certain database permissions and schema existences without checks, which might not be the case for all users, leading to runtime errors.

Proposed Enhancements

Update all ENUM declarations in the models to include explicit names, ensuring compatibility with PostgreSQL's requirements.
Improve the configuration loading mechanism to provide clear, informative error messages when config.ini is missing or corrupted, and improve the robustness of environment variable fallbacks.
Implement initial checks for database connectivity, permissions, and schema existence, providing users with early and clear diagnostics if the setup is incorrect.

Expected Benefits

By addressing these issues, the application will become more compatible with PostgreSQL, offering users more flexibility in their choice of database systems.
Improved error messages and configuration handling will make the application more user-friendly, reducing setup time and confusion.
Early checks for database setup will prevent runtime errors, improving the overall user experience and reducing the need for troubleshooting.

Additional Context
The need for these enhancements was identified during an attempt to set up the application with a PostgreSQL database, which resulted in multiple errors. Investigating these errors revealed that the application's current database handling is not fully compatible with PostgreSQL's strict requirements, particularly regarding ENUM naming and initial database setup. Implementing these changes will make the application more robust and easier to set up for new users.

Thank you! Merged.


Improve application webhook security

[x] add a filter to limit allowed service operations and payload data
[x] if the access token contains a user ID use it as service call parameters (eg to check authorisations)
[ ] generate a secret associated with the webhook so that an old access token can be revoked

In a first version we can manage access tokens at the API gateway level, which will allow to generate/revoke a token whenever required although this will be a manual process.


'TwitterSignin.login' failed

Follow the instructions as written, but what i get is this
undefined is not an object(evaluating 'TwitterSignin.logIn')

import {
NativeModules,
} from 'react-native';
const { TwitterSignin } = NativeModules;


Untangle olm.sh

Inline multiple layers of helpers to make the code readable. It seems
that the helper tried to avoid repeating the olm repo url; this is easy
to do with a constant.
Use local variable ($cluster_name) instead of cryptic $1.

@BenamarMk @raghavendra-talur cane you review?


Is there a way to convert (or transfer weights from) CuDNNLSTM to TFLiteLSTMCell?

1. System information

OS Platform and Distribution: Ubuntu 18.04 (and Android for inference)
TensorFlow installation: pip
TensorFlow library: version 1.14.0

2. Code
I wish to use CuDNNLSTM for a bidirectional LSTM model in order to make use of GPU for training. A short code snippet is below.
        forward_layer = tf.compat.v1.keras.layers.CuDNNLSTM(164, return_sequences=True, time_major=True)
        backward_layer = tf.compat.v1.keras.layers.CuDNNLSTM(164, return_sequences=True, time_major=True,
                         go_backwards=True)
        output = tensorflow.keras.layers.Bidirectional(forward_layer, backward_layer=backward_layer)(input_feature)

However, this architecture is too heavy to deply on android using TFLite. Also, I need to avoid the Select TF Ops which increase the size of the runtime dramatically. Therefore, I decided to use experimental TFLiteLSTMCell in combination with bidirectional_dynamic_rnn , which works very well when trained on CPU.
        lstm_forward = tf.compat.v1.lite.experimental.nn.TFLiteLSTMCell(164, use_peepholes=False, name="fw")
        lstm_backward = tf.compat.v1.lite.experimental.nn.TFLiteLSTMCell(164, use_peepholes=False, name="bw")
        infer_output, infer_state =  tensorflow.lite.experimental.examples.lstm.rnn.bidirectional_dynamic_rnn(
                lstm_forward,lstm_backward,lstm_inputs, dtype='float32', time_major=True)

I'm using what I believe is a reasonably standard method of conversion:
            converter = tf.compat.v1.lite.TFLiteConverter.from_session(sess, [roi], [y_pred])

            converter.experimental_new_converter = True
            converter.optimizations = [tf.lite.Optimize.OPTIMIZE_FOR_SIZE]

            # converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS], tf.lite.OpsSet.SELECT_TF_OPS]
            # We do not want to use Select TF Ops

            tflite = converter.convert()

Is there any way to convert weights from CuDNNLSTM to TFLiteLSTMCell? Even a way to do it from LSTM (non cudnn) to TFLiteLSTMCell will be most helpful, becuase transferring weights CuDNNLSTM to LSTM is well documented and not a big hurdle.
Any other way to convert CuDNNLSTM to TFLite without using Select Ops is also welcome.
Thanks in advance!

Hi @vighneshhegde !
1.x version are not supported any more. I was thinking of using get_weights then set_weights method to transfer the weights from CudNNLSTM to LSTMcell model.
Attached relevant thread for reference.
Please let us know after your trial.
Thank you!

Hi @mohantym , thanks very much for your response!
I have tried manualy transferring the weights (i.e., "model surgery"), but I must be missing something like a transpose operation or some other thing because the results don't come out right. The documentation I found for TFLiteLSTMCell mentions a transpose operation but was not particularly enlightening. Would you be able to point out to a better source, perchance?
Also, any comments on converting CuDNNLSTM to TFLite without using Select Ops? Thank you!

@vighneshhegde !
Not sure on 1.x apis.  But please feel free to go through this document on LSTMFusion Code lab (I think it will be possible to convert without select ops with this method)
Thank you!

@mohantym this seems to work, even for Bidirectional LSTMs. Thank you very much!


Update datasource example to demonstrate SWARM-14

(wrapping standalone.xml)

Will be merged when the core update is successfully verified


feat: enable use of preexisting weval binary

This commit enables using a pre-existing weval binary -- this is mostly to try to avoid some flaky tests in jco downstream, but this is probably a reasonable thing to have in general.
I also shuffled around some instructions/docs while I was in there.

Hey @guybedford requesting another review just because of the dramatic change to the CI stuff :bow: -- took a while but it's finally :green_circle:
I'm pretty sure I covered the dependency graph properly, the build should now be much faster. We'll have to pay extra attention to cache keys for the three builds in the future, though IMO the current set should work for a while.
Right now we build against latest (22, for now) and 20 -- I'm happy to only build against latest if you'd like!

What happened to the Weval-specific update to just use the direct release URLs? If that doesn't require API gating that sounds ideal to me.
Is this PR still needed if that lands?

Hey @guybedford sorry a bit late getting to this, but I was hoping that the update to use direct URLs and this could land -- this PR would enable using a pre-existing weval binary (which would remove our need to pull the URLs at all -- though we should still pick the more direct URL)

@vados-cosmonic thanks for the confirmation - I don't mean to be pedantic, but I feel that it's important to get this one fixed upstream first, before landing the alternative paths, as that solves the issue for all Weval users instead of just us.

I've posted https://github.com/bytecodealliance/weval/issues/13 for clarity.

Hey @alexanderkjeldaas that sounds like a great idea! Would you mind making a new issue for it? The changes should be relatively straight forward, so we could discuss it there and probably anyone could come along and implement :)


The package "requests" is missing in requirements.txt

Running inspect4py --version will result in an error:
line 6, in <module>
    import requests
ModuleNotFoundError: No module named 'requests'

We need to better capture some of these dependencies with a poetry environment


Provide feedback on the Java Side Unit Tests Coverage

To provide feedback on the Unit Tests Coverage

Moving this to 8.0-2 per our discussion today in triage meeting.