no nodepools found Error for new install of 1.0.5

Description
Observed Behavior:
I have a new EKS 1.31 cluster with Karpenter 1.0.5 install via the Terraform eks-blueprints module.  I have similar clusters up and running successfully that were installed the same way but they are k8s 1.29 and Karpenter 1.0.1.  New nodes are not being created when scale up is needed.  Below is the log.
{"level":"INFO","time":"2024-11-25T15:49:49.205Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"metrics.node","worker count":1}
{"level":"INFO","time":"2024-11-25T15:49:49.198Z","logger":"controller","message":"Starting EventSource","commit":"652e6aa","controller":"metrics.nodepool","controllerGroup":"karpenter.sh","controllerKind":"NodePool","source":"kind s
ource: *v1.NodePool"}
{"level":"INFO","time":"2024-11-25T15:49:49.205Z","logger":"controller","message":"Starting Controller","commit":"652e6aa","controller":"metrics.nodepool","controllerGroup":"karpenter.sh","controllerKind":"NodePool"}
{"level":"INFO","time":"2024-11-25T15:49:49.202Z","logger":"controller","message":"Starting EventSource","commit":"652e6aa","controller":"status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","source":"kind so
urce: *v1.EC2NodeClass"}
{"level":"INFO","time":"2024-11-25T15:49:49.205Z","logger":"controller","message":"Starting Controller","commit":"652e6aa","controller":"status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass"}
{"level":"INFO","time":"2024-11-25T15:49:49.410Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"migration.crd","controllerGroup":"apiextensions.k8s.io","controllerKind":"CustomResourceDefinition"
,"worker count":1}
{"level":"INFO","time":"2024-11-25T15:49:49.411Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"state.pod","controllerGroup":"","controllerKind":"Pod","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.412Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodepool.readiness","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.412Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"lease.garbagecollection","controllerGroup":"coordination.k8s.io","controllerKind":"Lease","worker c
ount":10}
{"level":"INFO","time":"2024-11-25T15:49:49.412Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclaim.podevents","controllerGroup":"","controllerKind":"Pod","worker count":1}
{"level":"INFO","time":"2024-11-25T15:49:49.412Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclaim.consistency","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count"
:10}
{"level":"INFO","time":"2024-11-25T15:49:49.412Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"node.termination","controllerGroup":"","controllerKind":"Node","worker count":100}
{"level":"INFO","time":"2024-11-25T15:49:49.412Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"state.nodepool","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.414Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodepool.counter","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.414Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"metrics.pod","controllerGroup":"","controllerKind":"Pod","worker count":1}
{"level":"INFO","time":"2024-11-25T15:49:49.414Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodepool.validation","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker count":10
}
{"level":"INFO","time":"2024-11-25T15:49:49.414Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclaim.lifecycle","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count":1
000}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclaim.termination","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count"
:100}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"status","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclaim.disruption","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count":
10}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"status","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"migration.resource.nodeclaim","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker
 count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclaim.expiration","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count":
1}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"migration.resource.nodepool","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker c
ount":10}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"provisioner.trigger.pod","controllerGroup":"","controllerKind":"Pod","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodepool.hash","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.416Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"provisioner.trigger.node","controllerGroup":"","controllerKind":"Node","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.418Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"state.daemonset","controllerGroup":"apps","controllerKind":"DaemonSet","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.422Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"state.node","controllerGroup":"","controllerKind":"Node","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.423Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclass.hash","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","worker count
":10}
{"level":"INFO","time":"2024-11-25T15:49:49.423Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclass.status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","worker cou
nt":10}
{"level":"INFO","time":"2024-11-25T15:49:49.423Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclaim.tagging","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count":1}
{"level":"INFO","time":"2024-11-25T15:49:49.423Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"nodeclass.termination","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","worke
r count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.423Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"migration.resource.ec2nodeclass","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeCla
ss","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.424Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"state.nodeclaim","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count":10}
{"level":"INFO","time":"2024-11-25T15:49:49.433Z","logger":"controller","message":"Starting workers","commit":"652e6aa","controller":"metrics.nodepool","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker count":1}
{"level":"INFO","time":"2024-11-25T15:49:49.514Z","logger":"controller","message":"discovered ssm parameter","commit":"652e6aa","controller":"nodeclass.status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","EC
2NodeClass":{"name":"default"},"namespace":"","name":"default","reconcileID":"585d48f9-9c71-4e23-9ef2-54c998f0016e","parameter":"/aws/service/eks/optimized-ami/1.31/amazon-linux-2/recommended/image_id","value":"ami-0541903c03df81c16"
}
{"level":"INFO","time":"2024-11-25T15:49:49.537Z","logger":"controller","message":"discovered ssm parameter","commit":"652e6aa","controller":"nodeclass.status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","EC
2NodeClass":{"name":"default"},"namespace":"","name":"default","reconcileID":"585d48f9-9c71-4e23-9ef2-54c998f0016e","parameter":"/aws/service/eks/optimized-ami/1.31/amazon-linux-2-arm64/recommended/image_id","value":"ami-0104decf27a3
167ca"}
{"level":"INFO","time":"2024-11-25T15:49:49.554Z","logger":"controller","message":"discovered ssm parameter","commit":"652e6aa","controller":"nodeclass.status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","EC
2NodeClass":{"name":"default"},"namespace":"","name":"default","reconcileID":"585d48f9-9c71-4e23-9ef2-54c998f0016e","parameter":"/aws/service/eks/optimized-ami/1.31/amazon-linux-2-gpu/recommended/image_id","value":"ami-07a684ca16d05d
a49"}
{"level":"INFO","time":"2024-11-25T15:49:50.458Z","logger":"controller","message":"pod(s) have a preferred Anti-Affinity which can prevent consolidation","commit":"652e6aa","controller":"provisioner","namespace":"","name":"","reconci
leID":"b9b49daf-3974-49ce-a85c-f3424b99356b","pods":"argocd/argocd-application-controller-0, argocd/argocd-server-6f848b85cb-95j48"}
{"level":"ERROR","time":"2024-11-25T15:49:50.458Z","logger":"controller","message":"nodePool not ready","commit":"652e6aa","controller":"provisioner","namespace":"","name":"","reconcileID":"b9b49daf-3974-49ce-a85c-f3424b99356b","Node
Pool":{"name":"default"}}
{"level":"INFO","time":"2024-11-25T15:49:50.458Z","logger":"controller","message":"no nodepools found","commit":"652e6aa","controller":"provisioner","namespace":"","name":"","reconcileID":"b9b49daf-3974-49ce-a85c-f3424b99356b"}
{"level":"ERROR","time":"2024-11-25T15:50:00.474Z","logger":"controller","message":"nodePool not ready","commit":"652e6aa","controller":"provisioner","namespace":"","name":"","reconcileID":"c9be084b-0855-4fae-9279-60cd53a1c78c","Node
Pool":{"name":"default"}}
{"level":"INFO","time":"2024-11-25T15:50:00.474Z","logger":"controller","message":"no nodepools found","commit":"652e6aa","controller":"provisioner","namespace":"","name":"","reconcileID":"c9be084b-0855-4fae-9279-60cd53a1c78c"}
{"level":"ERROR","time":"2024-11-25T15:50:10.476Z","logger":"controller","message":"nodePool not ready","commit":"652e6aa","controller":"provisioner","namespace":"","name":"","reconcileID":"d56b7ee3-4450-4b0f-a1df-5d64aa84f14f","Node
Pool":{"name":"default"}}
{"level":"INFO","time":"2024-11-25T15:50:10.476Z","logger":"controller","message":"no nodepools found","commit":"652e6aa","controller":"provisioner","namespace":"","name":"","reconcileID":"d56b7ee3-4450-4b0f-a1df-5d64aa84f14f"}

Expected Behavior:
New nodes would be spun up.
Reproduction Steps (Please include YAML):
aws-auth
kubectl get configmaps -n kube-system aws-auth -o yaml
apiVersion: v1
data:
  mapAccounts: |
    []
  mapRoles: |
    - "groups":
      - "system:bootstrappers"
      - "system:nodes"
      "rolearn": "arn:aws:iam::myaccount:role/karpenter-us-east-2-newcluster-prod1"
      "username": "system:node:{{EC2PrivateDNSName}}"

Configs

apiVersion: karpenter.k8s.aws/v1
kind: EC2NodeClass
metadata:
namespace: karpenter
finalizers:

karpenter.k8s.aws/termination
labels:
app.kubernetes.io/instance: karpenter-us-east-2-newcluster-prod1
app.kubernetes.io/managed-by: Helm
name: default
spec:
amiSelectorTerms:
alias: al2@latest
metadataOptions:
httpEndpoint: enabled
httpProtocolIPv6: disabled
httpPutResponseHopLimit: 2
httpTokens: required
role: karpenter-us-east-2-newcluster-prod1
securityGroupSelectorTerms:
tags:
karpenter.sh/discovery: us-east-2-newcluster-prod1
subnetSelectorTerms:
tags:
karpenter.sh/discovery: us-east-2-newcluster-prod1
tags:
IntentLabel: apps
KarpenterNodePoolName: default
NodeType: default
intent: apps
karpenter.sh/discovery: us-east-2-newcluster-prod1
project: karpenter-blueprints

apiVersion: karpenter.sh/v1
kind: NodePool
metadata:
labels:
app.kubernetes.io/instance: karpenter-spiral-siaas-prod1-blue
app.kubernetes.io/managed-by: Helm
name: default
namespace: karpenter
spec:
disruption:
budgets:
- nodes: "0"
schedule: "0 12 * * mon-fri"
duration: 16h
reasons:
- Drifted
- Underutilized
- nodes: "100%"
reasons:
- Empty
- nodes: "10%"
reasons:
- Drifted
- Underutilized
consolidationPolicy: WhenEmptyOrUnderutilized
consolidateAfter: 1h
template:
metadata:
labels:
intent: apps
spec:
expireAfter: Never
nodeClassRef:
group: karpenter.k8s.aws
kind: EC2NodeClass
name: default
requirements:
- key: karpenter.sh/capacity-type
operator: In
values:
- spot
- key: kubernetes.io/arch
operator: In
values:
- amd64
- key: kubernetes.io/os
operator: In
values:
- linux
- key: karpenter.k8s.aws/instance-category
operator: In
values:
- c
- m
- r
- key: karpenter.k8s.aws/instance-cpu
operator: In
values:
- "4"
- "8"
- "16"
- "32"
- key: karpenter.k8s.aws/instance-generation
operator: Gt
values:
- "2"

**Versions**:
- Chart Version: karpenter-1.0.5	1.0.5
- Kubernetes Version (`kubectl version`): 1.31
* Please vote on this issue by adding a 👍 [reaction](https://blog.github.com/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/) to the original issue to help the community and maintainers prioritize this request
* Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
* If you are interested in working on this issue or have submitted a pull request, please leave a comment

An update, my ec2nodeclass is not ready I that appears to be the root of the issue but I am unsure why.
  Type    Reason                Age   From       Message
  ----    ------                ----  ----       -------
  Normal  AMIsReady             38s   karpenter  Status condition transitioned, Type: AMIsReady, Status: True -> Unknown, Reason: AwaitingReconciliation, Message: object is awaiting reconciliation
  Normal  Ready                 38s   karpenter  Status condition transitioned, Type: Ready, Status: False -> Unknown, Reason: UnhealthyDependents, Message: InstanceProfileReady=Unknown, SecurityGroupsReady=Unknown, SubnetsReady=Unkn
own, AMIsReady=Unknown
  Normal  SecurityGroupsReady   38s   karpenter  Status condition transitioned, Type: SecurityGroupsReady, Status: False -> Unknown, Reason: AwaitingReconciliation, Message: object is awaiting reconciliation
  Normal  SubnetsReady          38s   karpenter  Status condition transitioned, Type: SubnetsReady, Status: True -> Unknown, Reason: AwaitingReconciliation, Message: object is awaiting reconciliation
  Normal  AMIsReady             38s   karpenter  Status condition transitioned, Type: AMIsReady, Status: Unknown -> True, Reason: AMIsReady
  Normal  InstanceProfileReady  38s   karpenter  Status condition transitioned, Type: InstanceProfileReady, Status: Unknown -> True, Reason: InstanceProfileReady
  Normal  Ready                 38s   karpenter  Status condition transitioned, Type: Ready, Status: Unknown -> False, Reason: UnhealthyDependents, Message: SecurityGroupsReady=False
  Normal  SecurityGroupsReady   38s   karpenter  Status condition transitioned, Type: SecurityGroupsReady, Status: Unknown -> False, Reason: SecurityGroupsNotFound, Message: SecurityGroupSelector did not match any SecurityGroups
  Normal  SubnetsReady          38s   karpenter  Status condition transitioned, Type: SubnetsReady, Status: Unknown -> True, Reason: SubnetsReady

This is your issue:
Normal  SecurityGroupsReady   38s   karpenter  Status condition transitioned, Type: SecurityGroupsReady, Status: Unknown -> False, Reason: SecurityGroupsNotFound, Message: SecurityGroupSelector did not match any SecurityGroups
I can see you're looking for this sg:
securityGroupSelectorTerms:
tags:
karpenter.sh/discovery: us-east-2-newcluster-prod1
Did you add these tags to a SG in the same VPC?

I'm not sure how I missed that but you are 100% correct.  Thanks!!

I have a similar case, however ec2nodeclass has SubnetsReady=True, SecurityGroupsReady=True and InstanceProfileReady=True however AMIsReady is in AwaitingReconciliation
I tried various amiFamily options with AL2, AL2023. tried even specifying the image id.
eg:
  amiFamily: AL2023
  amiSelectorTerms:
    - alias: al2023@v20241121

Any thoughts on what could be the issue?
Also once created I can't delete an ec2nodeclass even if there are no references to this.

this is fixed and was an issue in my Terraform code and missed the IAM Role on Image Describe.


Dev/1.28

Codecov Report
Attention: Patch coverage is 88.19444% with 17 lines in your changes missing coverage. Please review.

Project coverage is 91.98%. Comparing base (a373ee6) to head (e978abb).
Report is 23 commits behind head on main.

Files with missing lines
Patch %
Lines

weaviate/collections/batch/client.py
50.00%
10 Missing :warning:

integration/test_rbac.py
94.23%
6 Missing :warning:

weaviate/client.py
87.50%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1454      +/-   ##
==========================================
- Coverage   93.98%   91.98%   -2.00%
==========================================
  Files         221      174      -47
  Lines       21332    14848    -6484
==========================================
- Hits        20048    13658    -6390
+ Misses       1284     1190      -94

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Betalgo package not available in Visual Studio 2022

Installed the Betalgo package in the package manager console and it installed no problem. I just can't reference the package with a using statement.

Seems weird, what it says when you put your mouse on red warning?

It says The type or namespace name 'Betalgo' could not be found (are you missing a using directive or an assembly reference?). Could the namespace be misspelled?

Could you share .csproject file to please?

Exe
net6.0
enable
enable

Here you go:

Oh, I just found out what the issue is, please use
using OpenAI.GPT3; instead of using Betalgo.OpenAI.GPT3

Does my answer solve your problem? can I close the issue?

Oh, I thought I tried that. Okay, thanks!


System.Buffers + Json.NET everywhere

The JSON input and output formatters use array pooling from System.Buffers.
Other places to add System.Buffers:

JsonViewComponentResult - https://github.com/aspnet/Mvc/blob/25eb50120eceb62fd24ab5404210428fcdf0c400/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/JsonViewComponentResult.cs#L75
JsonResultExecutor - https://github.com/aspnet/Mvc/blob/8b1bd343ba56152d2a7e677cf041dc9d62b4033f/src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/JsonResultExecutor.cs#L114

Will discuss tomorrow with a few folks. Sorta kinda related to the mess described in https://github.com/aspnet/Mvc/issues/3992.

Discussed with @dougbu and @rynowak (The Outlaw):
In #3992 we decided to remove JsonViewComponentResult, so that's not an issue here.
For JsonResultExecutor we took a look at the code and it already uses pooling by virtue of IHttpResponseStreamWriterFactory, which in its implementation uses the pooling feature.

Json.NET doesn't automatically use pooling for its internal buffer because of IHttpResponseStreamWriterFactory.
From JsonOutputFormatter: https://github.com/aspnet/Mvc/blob/4123d83d26e528fe09c870cc92d4aecf0e80d3f7/src/Microsoft.AspNetCore.Mvc.Formatters.Json/JsonOutputFormatter.cs#L120

Ah ok yeah, you're right. We should do this.


To support namespaced installation

Summary
Numaflow controller/server should be abled installed in namespaced scope.

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

https://github.com/numaproj/numaflow/pull/112


Doesn't work with python-click > 6.6

I get an error when trying to use tmuxp that makes me think it is requiring exactly click==6.6.  My system has click 6.7.1.
Here is the error:

This error is fixed by this commit. You can fix this problem by manually editing your local requires.txt file, which if you're running Arch is here: /usr/lib/python3.6/site-packages/tmuxp-1.2.3-py3.6.egg-info/.
This is the second time this week that updated packages have caused tmuxp to stop working for me. I understand the utility of dependency pinning for development purposes and that tmuxp has a very short list of dependencies. However, the current situation is that whenever one of those dependencies is updated, no matter how trivial the update (click 6.7 changed two non-functional lines), tmuxp is broken until a matching update in tmuxp can propagate through to end users. Might it be less user-hostile to have the dependencies be >= instead of ==?

@thebopshoobop Thanks, that did indeed work.  And I am running Arch, too :D.
I really do think it should be >=, but I'm kind of an outsider to the python package ecosystem, and not sure if they have good support for semver resolving.

Actually the fact that it's not even a semver version means no, at least in this case...  I guess I meant it more generally than semver, as in I don't know if python packages have a good solution for managing dependency versions.

@rosshadden @thebopshoobop How is v1.2.4? Better?  Give it a shot and let me know
pyup.io did lock packages, but I'm not sure if it affected the release or not, as it happened after 1.2.3 was already tagged...

@tony v1.2.4 does indeed work just fine. Thanks for the speedy tag.
@rosshadden I've posted an updated PKGBUILD here, and I've flagged tmuxp out-of-date in the AUR.

@tony Yep, works great.  Thank you!
@thebopshoobop Good thinking.


Using Custom Pagination in Django Ninja

Hi there!
I think the PageNumberPaginationExtra is something I need. I tried to add it in two unsuccessful ways:

First by installing this package. Rapidly, I noticed I have to make multiple changes to my code, and I don't especially want to change the whole ninja mechanism which works well for me. I don't need class based endpoints, and other stuff. I'd like to avoid the dependency.
Then I tries to copy paste Pagination code and use directly in my code. Again, I had weird errors, suggesting that maybe I can't use this code as it, outside of ninja extra.

What should I do to "extract" PageNumberPaginationExtra properly from Django ninja extra?
Thanks for your help.

@ddahan The feature is tied to the package but you understand pagination in Django Ninja, you should be able to implement something similar to PageNumberPaginationExtra

TBH I didn't understand why the PageNumberPagination was so complex in this package code source. There must be a reason, but out of my knowledge.
Here is a working custom PageNumberPagination implementation example without this package:
from typing import Any

from ninja import Field, Schema
from ninja.pagination import PaginationBase

class MyPageNumberPagination(PaginationBase):
    class Input(Schema):
        page: int = Field(1, ge=1)

    class Output(Schema):
        nb_items: int
        nb_pages: int
        page: int
        previous_page: str | None
        next_page: str | None

    def __init__(self, page_size: int = 100, **kwargs):
        self.page_size = page_size
        super().__init__(**kwargs)

    def paginate_queryset(self, queryset, pagination: Input, request) -> Any:
        offset = (pagination.page - 1) * self.page_size
        nb_items = self._items_count(queryset)
        next_page, previous_page = None, None
        if pagination.page > 1:
            previous_page = request.build_absolute_uri(
                request.path + "?page=" + str(pagination.page - 1)
            )
        if offset + self.page_size < nb_items:
            next_page = request.build_absolute_uri(
                request.path + "?page=" + str(pagination.page + 1)
            )
        return {
            "nb_items": nb_items,
            "nb_pages": (nb_items + self.page_size - 1) // self.page_size,
            "page": pagination.page,
            "previous_page": previous_page,
            "next_page": next_page,
            "items": queryset[offset : offset + self.page_size],
        }

And how are you using MyPageNumberPagination

@eadwinCode I updated my previous answer to add a usage example.

@ddahan You need to fix your output schema MyPageNumberPagination
    class Output(Schema):
        nb_items: int
        nb_pages: int
        page: int
        previous_page: str | None
        next_page: str | None

The page field needs to be defined
Once you fix that, the next thing is to resolve the response schema in your endpoint
@router.get("", response=MyPageNumberPagination.Output[MySchema])
@paginate(MyPageNumberPagination, page_size=10)
def list_badges(request):
    return Badge.objects.all()

Read more on Pydantic Generics

The page field needs to be defined properly... What you return from paginate_queryset shows, page field is not an int field

Can you elaborate on this? It is the same page variable that I defined myself in:
class Input(Schema):
        page: int = Field(1, ge=1)

As it is defined, it's an int that must be greater or equal at 1. What's wrong here?

the next thing is to resolve the response schema in your endpoint

You're right. I forgot to change the response type. I'd have expected this to create an error, but it works as is.

@ddahan oh it's an oversight... I meant the items... The output schema is not having the items field defined


Can't install locally

OS: Macos Sonoma 14.2.1
Node: 18.17.0
When I clone the repo, main branch and run yarn install I'm getting this error
[1/5] 🔍  Validating package.json...
[2/5] 🔍  Resolving packages...
error An unexpected error occurred: "Invalid URL".
info If you think this is a bug, please open a bug report with the information provided in "/Users/user/projects/frames.js/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

If I upgrade to yarn >2.x the install works but I get a bunch of other building errors. Do I need a specific node 18.x version? yarn automatically gets set to 1.22.19 which is declared in the package.json.

Update: I just had to update to the latest 18.19.0 we should probably update the version requirement to match.


Re-process the journal list from Medline and add additional information to our journal data records

Update journal data in the assets image and in production with more data from this list ftp://ftp.ncbi.nih.gov/pubmed/J_Medline.txt
this is a sample entry from the file:
JrId: 1
JournalTitle: AADE editors' journal
MedAbbr: AADE Ed J ----> map to NLMTA
ISSN (Print): 0160-6999 -----> first ISSN in the list
ISSN (Online):  -----> second ISSN in the list
IsoAbbr: AADE Ed J
NlmId: 7708172

@jrmartino please contact @birkland for his old code for loading journal records, incase it helps.

closed by https://github.com/OA-PASS/pass-journal-loader/pull/4


RFC 03: Relationship methods should be versionable?

Proposal
~Building on the inconsistency addressed by #3, remove ManagerInterface.setMetadata and ManagerInterface.setMetadatEntry, moving metadata updates to ManagerInterface.register.  Or, alternatively, require them to return an Entity Reference.~
~The problem with the latter is that it may be ambiguous whether a register and immediate setMetadata is one or two logical actions.  Moving it to the register call would avoid this ambiguity, and also mean there is a single entry point where entity data is mutated in the manager by the host.~
Rationale
~These methods effectively mutate an asset without the opportunity for version management by the manager. As we propose workflows where metadata represents primary properties of an asset (eg a shot, lens distortion coefficients), we should not give different meanings to publishing a change to the primary string and publishing a change to metadata.~
Update:
#247 and the new unified data model means that the original aim of this ticket (to fix metadata) is no longer relevant as it's inherently versioned via register and resolve now. Relationships are unaffected by this though, and may still need some consideration.

Closing in favour of more appropriate issues.


[Feature] Move request inside collapsed folder

Currently cannot drag and drop request in left sidebar to collapsed folder.
Currently this works only for expand folder.

Hey @gschier, I had a stab at this feature request and added my thoughts to the PR description. :)


Fix download filtered data only

This fix issues for download filtered data only. I think you have been fixed this before, but in #329 you've changed this, and become not worked. Give your feedback if i have some mistakes.

Coverage remained the same at 72.393% when pulling 736caa167833b441f3b9d38fc027d8fbe95e3f68 on faran312:bugfix/download-filtered-data into 755809dacb49011098a419fb05e3bd957503f728 on gregnb:master.

I think this keeps going back and forth. We need to have some sort of option that toggles between showing all data or filtered data. Please reply back with an idea for that API can we can work from there.
Thank you for submitting.

The issue seems to be that there are two filters being applied in filteredData: selected columns and visible rows.
In my case, users want to download all rows in the table with columns filtered.
Seems like the ideal solution might be a bitwise operator.
const Columns = 1;
const Rows = 2;

downloadFilterOptions: (Columns | Rows)

// if (downloadFilterOptions === undefined) download all rows, all columns
// if (downloadFilterOptions === 1) download all rows, selected columns
// if (downloadFilterOptions === 2) download current visible rows, all columns
// if (downloadFilterOptions === 3) download current visible rows, selected columns

@shawnmitchell How about something like this for the API:
downloadOptions: {
  filename: 'tableDownload.csv',
  separator: ',',
  filterOptions: {
    useDisplayedColumnsOnly: true | false,
    useDisplayedRowsOnly: true | true
  }
}

Default for filterOptions would be false, in which case it works as it would normally.

If useDisplayedColumnsOnly: true and useDisplayedRowsOnly: false, downloads only columns that remained through filtering, and all rows.
If useDisplayedColumnsOnly: false and useDisplayedRowsOnly: true, downloads only rows that remained through filtering, and all columns.
If useDisplayedColumnsOnly: false and useDisplayedRowsOnly: false, downloads all table data, as in the default case.
If useDisplayedColumnsOnly: true and useDisplayedRowsOnly: true, downloads only columns and rows that remained through filtering.

I'm open to suggestions for shorter, but equally clear option names. I'm hoping that whatever we pick will be self-documenting enough to avoid confusion.

can someone please merge this????

@ali26Y This PR can't get merged as is because it needs a more robust API. Some users will want displayed data only, others will want all the data. Feel free to submit a PR that provides an API similar to the one mentioned here and I will be happy to review it.

Options now available for behavior like this in 2.7.0.


When rejecting a credential offer, Tenant UI should send a problem report before deleting the exchange record locally

If the user opts to decline/reject the credential offer. It does send a problem report through the connection.
It should, however the existing code just deletes the local record and does not use the POST to the issue-credential/<cred_exch_id>/problem-report endpoint.
There may be a standard payload in the documentation, but i'm not sure.
https://github.com/hyperledger/aries-rfcs/blob/main/features/0036-issue-credential/README.md

Problem report RFC: https://github.com/hyperledger/aries-rfcs/tree/main/features/0035-report-problem
It looks like we should be hitting the appropriate problem report endpoint for the credential exchange (/issue-credential/records/{cred_ex_id}/problem-report and /issue-credential-2.0/records/{cred_ex_id}/problem-report) with details about what happened.

Easy enough to do the ACA-Py API call to post a problem report when the user rejects the credential offer through the Tenant UI.
But is that the flow? Whatever controller interacting with it's agent (in this case the Tenant UI on behalf of the Tenant) manually makes the admin API call for that? I'd had it in my mind that the problem-report would be more automatic than that. Like in VC-Auth is the declined presentation request a controller-driven problem report send?
Maybe it is and I have it wrong but would just want to make sure it's consistent (IE, make sure it's not something that's missing in ACA-Py, or a auto setting or something)

But is that the flow? Whatever controller interacting with it's agent (in this case the Tenant UI on behalf of the Tenant) manually makes the admin API call for that?

Yes, that is the way the protocol was specified. In the case of BC Wallet, it is the (human) controller that triggers the problem report by refusing to send a response to the proof.

Cool, then this is probably ready to go to add to Tenant UI if it's needed.

https://github.com/bcgov/traction/pull/1029


websocket bug?

websocket handler will produce empty message,and it is regualar, every 28s;
websocket client is qt's QWebsocket,and I am sure I didn't send any message;
other's websocket hasn't test yet

I think that's the ping message. By default drogon's Websockt implementation sends a ping every 30 seconds to keep the connection alive and not being killed because of no traffic. You can disable it by calling wsPtr->getConnection()->disablePing() or ignore it by checking the message type in Qt (or something like that).


release

would you mind pushing to rubygems?

Hey @glaszig, apologies about the delay.
1.0.2 is out.  First release in almost 10 years. 🍾
Thanks for your help!


In the case of compilation, intellisense always indicates that the header file cannot be found

Environment

OS and Version: windows 10 x64
VS Code Version: 1.90
C/C++ Extension Version: 1.20.5
If using SSH remote, specify OS of remote machine:

Bug Summary and Steps to Reproduce
Bug Summary:
I am using vscode to build my project, I use cmake to build my project, which uses qt library and some other third party libraries, my project can compile normally and run successfully. But in my code file, there is always an error ripple on this side of the #include header. This is a problem even with iostream headers. I tried to add a header search path for qt and other third-party libraries to the includePath field in the c_cpp_properties.json file, or "configurationProvider": "ms-vscode.cmake-tools", adds a path to the C_Cpp.default.includePath field in the settings.json file, but these attempts do not work, and the problem persists.

Below is the directory structure of my project:
C:\USERS\SJ2003\DESKTOP\VIDEOCALL
│  CMakeLists.txt
│  dialog.cpp
│  dialog.h
│  Dialog.ui
│
├─.vscode
│      c_cpp_properties.json
│
└─build
│  ALL_BUILD.vcxproj
│  ALL_BUILD.vcxproj.filters
│  CMakeCache.txt
│  cmake_install.cmake
│  VideoCall.sln
│  VideoCall.vcxproj
│  VideoCall.vcxproj.filters
│  ZERO_CHECK.vcxproj
│  ZERO_CHECK.vcxproj.filters
│
├─.cmake
│  └─api
│      └─v1
| .......
Configuration and Logs
my c_cpp_properties.json
{
    "configurations": [
        {
            "name": "Win32",
            "includePath": [
                "${workspaceFolder}/**",
                "D:/Edge/Agora_Native_SDK_for_Windows_v4.3.1_FULL/Agora_Native_SDK_for_Windows_FULL/sdk/high_level_api/include/**",
                "D:/QT6/6.4.3/msvc2019_64/include/**"
            ],
            "defines": [
                "_DEBUG",
                "UNICODE",
                "_UNICODE"
            ],
            "configurationProvider": "ms-vscode.cmake-tools"
        }
    ],
    "version": 4
}

my settings.json
{
    "C_Cpp.errorSquiggles": "enabled",
    "C_Cpp.default.includePath": [
        "D:/Edge/Agora_Native_SDK_for_Windows_v4.3.1_FULL/Agora_Native_SDK_for_Windows_FULL/sdk/high_level_api/include/**",
        "D:/QT6/6.4.3/msvc2019_64/include/**"
    ]
}

my logs
[main] Creating a folder: videoCall
[build] is starting the build
[proc] Run the following command: D:\QT6\Tools\CMake_64\bin\cmake.exe --build c:/Users/sj2003/Desktop/videoCall/build --config Debug --target ALL_BUILD -j 14 --
[build] applies to.NET Framework MSBuild version 17.10.4+10fbfbf2e
[build]
[build]   Automatic MOC and UIC for target VideoCall
[build]   mocs_compilation_Debug.cpp
[build] D: \ Edge \ Agora_Native_SDK_for_Windows_v4 3.1 _FULL \ Agora_Native_SDK_for_Windows_FULL \ SDK \ high_level_api \ include \ AgoraBase. h(5484,36): warning C4267: "initialization" : convert from "size_t" to "int", data may be lost [C:\Users\sj2003\Desktop\ VideoCall \build\ videocall.vcxproj]
[build] D: \ Edge \ Agora_Native_SDK_for_Windows_v4 3.1 _FULL \ Agora_Native_SDK_for_Windows_FULL \ SDK \ high_level_api \ include \ AgoraBase. h(5484,23): warning C4267: Initialization: When converting from "size_t" to "const int", data may be lost [C:\Users\sj2003\Desktop\ VideoCall \build\ videocall.vcxproj]
[build] D: \ Edge \ Agora_Native_SDK_for_Windows_v4 3.1 _FULL \ Agora_Native_SDK_for_Windows_FULL \ SDK \ high_level_api \ include \ AgoraBase. h(5499,36): warning C4267: "initialization" : convert from "size_t" to "int", data may be lost [C:\Users\sj2003\Desktop\ VideoCall \build\ videocall.vcxproj]
[build] D: \ Edge \ Agora_Native_SDK_for_Windows_v4 3.1 _FULL \ Agora_Native_SDK_for_Windows_FULL \ SDK \ high_level_api \ include \ AgoraBase. h(5499,23): warning C4267: Initialize: When converting from "size_t" to "const int", data may be lost [C:\Users\sj2003\Desktop\ VideoCall \build\ videocall.vcxproj]
[build]   dialog.cpp
[build] D: \ Edge \ Agora_Native_SDK_for_Windows_v4 3.1 _FULL \ Agora_Native_SDK_for_Windows_FULL \ SDK \ high_level_api \ include \ AgoraBase. h(5484,36): warning C4267: "initialization" : convert from "size_t" to "int", data may be lost [C:\Users\sj2003\Desktop\ VideoCall \build\ videocall.vcxproj]
[build] D: \ Edge \ Agora_Native_SDK_for_Windows_v4 3.1 _FULL \ Agora_Native_SDK_for_Windows_FULL \ SDK \ high_level_api \ include \ AgoraBase. h(5484,23): warning C4267: Initialization: When converting from "size_t" to "const int", data may be lost [C:\Users\sj2003\Desktop\ VideoCall \build\ videocall.vcxproj]
[build] D: \ Edge \ Agora_Native_SDK_for_Windows_v4 3.1 _FULL \ Agora_Native_SDK_for_Windows_FULL \ SDK \ high_level_api \ include \ AgoraBase. h(5499,36): warning C4267: "initialization" : convert from "size_t" to "int", data may be lost [C:\Users\sj2003\Desktop\ VideoCall \build\ videocall.vcxproj]
[build] D: \ Edge \ Agora_Native_SDK_for_Windows_v4 3.1 _FULL \ Agora_Native_SDK_for_Windows_FULL \ SDK \ high_level_api \ include \ AgoraBase. h(5499,23): warning C4267: Initialize: When converting from "size_t" to "const int", data may be lost [C:\Users\sj2003\Desktop\ VideoCall \build\ videocall.vcxproj]
[build] is generating code...
[build]   VideoCall.vcxproj -> C:\Users\sj2003\Desktop\videoCall\build\Debug\VideoCall.exe
[build]   Building Custom Rule C:/Users/sj2003/Desktop/videoCall/CMakeLists.txt
[driver] Generated: 00:00:09.403
[build] The build is complete and the exit code is 0

Other Extensions
cmake v0.0.17
cmake-tools v1.18.41
cmake language support v0.0.9
Additional context
CMakeLists.txt

@shawn1j Thank you for reporting this issue. Regarding this issue, we tried to reproduce this issue using a simple demo project however we were unable to reproduce this issue for you. In order to investigate further, please provide us with a simple demo project to reproduce the issue and we look forward to your feedback.

Of course, here's my case, and it's very simple.

CMakeLists.txt
cmake_minimum_required(VERSION 3.10)

project(demo LANGUAGES CXX)

set(CMAKE_CXX_STANDARD 20)
set(CMAKE_CXX_STANDARD_REQUIRED True)

set(CMAKE_INCLUDE_CURRENT_DIR ON)

add_executable(demo main.cpp )

main.cpp
#include <iostream>

int main(int argc, char **argv)
{
    std::cout << "Hello World" << std::endl;

    return 0;
}

The following is the output of cmake-tools, because c/ C ++ output too much information, I put it in the c_cpp.logging.txt file.
c_cpp.logging.txt
[proc] Executing command: D:\QT6\Tools\CMake_64\bin\cmake.exe --version
[proc] Executing command: D:\QT6\Tools\CMake_64\bin\cmake.exe -E capabilities
[variant] Loaded new set of variants
[kit] Successfully loaded 9 kits from C:\Users\sj2003\AppData\Local\CMakeTools\cmake-tools-kits.json
[proc] Executing command: chcp
[visual-studio] Patch Windows SDK path from C:\Program Files (x86)\Windows Kits\10\bin\x64 to C:\Program Files (x86)\Windows Kits\10\bin\10.0.22621.0\x64 for D:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary\Build\vcvarsall.bat
[main] Configuring project: src
[proc] Executing command: D:\QT6\Tools\CMake_64\bin\cmake.exe --no-warn-unused-cli -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -SC:/Users/sj2003/Desktop/demo/src -Bc:/Users/sj2003/Desktop/demo/src/build -G "Visual Studio 17 2022" -T host=x64 -A x64
[cmake] Not searching for unused variables given on the command line.
[cmake] -- Selecting Windows SDK version 10.0.22621.0 to target Windows 10.0.19045.
[cmake] -- The CXX compiler identification is MSVC 19.40.33811.0
[cmake] -- Detecting CXX compiler ABI info
[cmake] -- Detecting CXX compiler ABI info - done
[cmake] -- Check for working CXX compiler: D:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.40.33807/bin/Hostx64/x64/cl.exe - skipped
[cmake] -- Detecting CXX compile features
[cmake] -- Detecting CXX compile features - done
[cmake] -- Configuring done
[cmake] -- Generating done
[cmake] -- Build files have been written to: C:/Users/sj2003/Desktop/demo/src/build
[visual-studio] Patch Windows SDK path from C:\Program Files (x86)\Windows Kits\10\bin\x64 to C:\Program Files (x86)\Windows Kits\10\bin\10.0.22621.0\x64 for D:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary\Build\vcvarsall.bat

@shawn1j I'm very sorry, I don't get this error with the code you provided, can you give me some advice?

Sorry to keep you waiting, I tried to use the latest version of cmake (version 3.29.6). After my testing, I found that when I used Visual Studio 2022 Developer Command Prompt v17.10.1 to launch vs code, vs code intellisense will work properly. This did not work with cmake (version 3.24.2), which I used earlier.

Here is the effect of starting vs code without Visual Studio 2022 Developer Command Prompt v17.10.1.

I wonder if this has anything to do with my computer environment. can you give me some advice?

@shawn1j When I start vscode without Visual Studio 2022 Developer Command Prompt I get a generator not found, which should be normal because my generator is installed by installing VS and not separately, but I still can't reproduce the problem you described.

For now, closing since user has a workaround and we weren't able to reproduce.


faker SerializationError, RecursionError

Good afternoon!
I was trying to reproduce lambda functions with fake on databricks.
I have added several pattern recognizers, configured the model (it's another question why spacy trf model can not be serialized as recommended here) and added a list of operators:
fake_operators = {     "PERSON": OperatorConfig("custom", {"lambda": lambda x: fake.name()}),     "MY_NAME_IS": OperatorConfig("custom", {"lambda": lambda x: "name is " + fake.name()}),     "PHONE_NUMBER": OperatorConfig("custom", {"lambda": lambda x: fake.phone_number()}),     "EMAIL_ADDRESS": OperatorConfig("custom", {"lambda": lambda x: fake.email()}) }
While applying udf from the example, I receive the following exception:
pyspark.serializers.SerializationError: Caused by Traceback (most recent call last):   File "/databricks/spark/python/pyspark/serializers.py", line 165, in _read_with_length     return self.loads(obj)   File "/databricks/spark/python/pyspark/serializers.py", line 469, in loads     return pickle.loads(obj, encoding=encoding)   File "/databricks/python/lib/python3.8/site-packages/faker/proxy.py", line 277, in __getattr__     obj = getattr(self._proxy, name)   File "/databricks/python/lib/python3.8/site-packages/faker/proxy.py", line 277, in __getattr__     obj = getattr(self._proxy, name)   File "/databricks/python/lib/python3.8/site-packages/faker/proxy.py", line 277, in __getattr__     obj = getattr(self._proxy, name)   [Previous line repeated 984 more times] RecursionError: maximum recursion depth exceeded
spacy model: en_core_web_lg
darabricks runtime: both 8.2 (includes Apache Spark 3.1.1, Scala 2.12) and 8.1 (includes Apache Spark 3.1.1, Scala 2.12)
faker: 8.9.1
It is possible to serialize faker functions to udfs separately but I'm not sure about their integration to the presidio at the moment.
Thank you for the help in advance!

Hi @OlegBEZb, the faker integration is a sample and not fully supported. It seems like the issue is with serializing faker objects, and not Presidio's. The integration into Presidio is mainly for generating a fake value for replacement instead of a constant one.

Hi @omri374!)
After a deeper investigation, I concluded with the same idea - the problem is not on Presidio's side. I have opened an issue in the Faker repo https://github.com/joke2k/faker/issues/1479. I hope this may help people who will use Presidio with Faker together.

thank you @OlegBEZb! Closing the issue for now.


chat with chimera-inst-chat-13b report error

When I run the script apply_delta.py, I got this error. It seems the LlaMATokenizer should be LlamaTokenizer, and It dose work！

Hi @ananwjq,
Please update transformers to the latest version: pip install git+https://github.com/huggingface/transformers.
Best,
Zhihong


test framework: change VMImage to VMDistro

https://github.com/istio/istio/blob/42d930f4ef61f895d3aad7c02e8dfb7d0f94d7b1/pkg/test/framework/components/echo/config.go#L100-L101
Our echo config allows selecting a VMs distro by choosing a pre-built image. We should hide the detail about images deeper in the kube part of the framework. We support real-VMs and real-VM echo implementations should be able to have their distro configured.
We can replace VMImage string with VMDistro VMDistro  and the following enum:
type VMDistro string

const (
  Debian10 = "Debian10"
  CentOS8 = "CentOS8"
)

The details about images can be moved here:
https://github.com/istio/istio/blob/42d930f4ef61f895d3aad7c02e8dfb7d0f94d7b1/pkg/test/framework/components/echo/kube/deployment.go#L592-L595
and the way we choose the image can be based on a map:
var distroToImage = map[VMDistro]string {
  echo.Debian10: "app_sidecar_debian_10",
  echo.UbuntuBionic: "app_sidecar_ubuntu_bionic",
}

Vendor-specific "real-VM" implementations can handle this as needed.
We may also allow StaticVM configs to include a distro field to be used as part of config matching.

cc @williamaronli


Add test coverage for utils.closest

This PR adds tests for the closest function exported from src/utils.js.
It also adds a docblock for said function.

@tsov I believe this PR should be ready for final review. Thanks!

@tsov Squashed!

@tsov This should be ready for final review.


Grant uri permissions manually for mPhotoUri to support lower api levels.

This pr addresses #306

Automated message from Dropbox CLA bot
@Sam1301, it looks like you've already signed the Dropbox CLA. Thanks!

@saketkumar95 @kunall17 can one of you review this?

@timabbott I was waiting for the stacktrace from sashafierce in #306 to know issue better!

@timabbott I have physical device running on Android 4.1.2. I am on it.

@Sam1301 @timabbott I tested this PR in Android 4.1.2 and is working fine now :tada:

Great, merged, thanks @Sam1301  and @yadav-rahul!


Unfortunately I have cancer. Please return my property😢😶😔

Unfortunately I have cancer.

I played these games to get money for my medicine.
We are very poor.
Please return my property😢😶😔
My ton keeper address is:
UQCZYnx3vnKQPBbEs-o3vrocqNC6DcbR2PsDgxrCFxwF8qI3

😢😢

bro did you got your assests back?
i was also lost my toncoin


Add the optimized version of dice cofficient?

Could you consider adding another version of Dice algorithm as this one is considered a more accurate and optimized version.
https://en.wikibooks.org/wiki/Algorithm_Implementation/Strings/Dice's_coefficient#Java

Here's an optimized version of the dice coefficient calculation. It takes
advantage of the fact that a bigram of 2 chars can be stored in 1 int, and
applies a matching algorithm of O(nlog(n)) instead of O(nn).

Note that, at the time of writing, this implementation differs from the

other implementations on this page. Where the other algorithms incorrectly
store the generated bigrams in a set (discarding duplicates), this
implementation actually treats multiple occurrences of a bigram as unique.
The correctness of this behavior is most easily seen when getting the
similarity between "GG" and "GGGGGGGG", which should obviously not be 1.
@param s The first string
@param t The second String
@return The dice coefficient between the two input strings. Returns 0 if one

    or both of the strings are {@code null}. Also returns 0 if one or both

    of the strings contain less than 2 characters and are not equal.

@author Jelle Fresen
*/

@Sasiko I am certainly not against optimizing of Dice Coefficient.  If I understand your proposal, the optimization is to stuff the bigram into an integer, and use the integer representation for comparisons.
I have a few concerns about the actual algorithm:

When we stuff the bigram into an integer field, are there any concerns that that stuffed value could possibly be a negative integer? I do not think it would be an issue, since we are only counting sets. As long as the bigram conversion is deterministic, I see no problem with it.
As it stands now, I believe the current implementation of all strategies does not handle Unicode surrogate pairs well. It has been largely ignored and sort of handle by the methods provided by Java itself (like Set<T>.retainAll()). However, since we are basically serializing code points into integer fields in two character blocks, surrogate pairs can be split between two bigrams. One option, I suppose, would be to stuff the values into a long integer (64-bit) value instead.  Do you have any insight on that?

I cannot guarantee when I can actually implement this. However, I do accept pull requests from other contributors. If you would like to attempt to implement it, please let me know.

I am closing this item for now. I am looking to do a new major version of the library to remove some of the bloat and take advantage of some of the newer features of Java.


Can't publish

I have a .NET MVC app that uses OR-TOOLS 64 bit. On my local IIS all works fine.  When I publish I get consistently "Could not load file or assembly 'Google.OrTools.DLL' or one of its dependencies." If I switch everything to 32 bit  ( dll , build , IIS ) I get the same.
More details here : http://stackoverflow.com/questions/41107577/mvc-cant-find-dependency-after-publish

This particular dll needed C++ Redistributable 2010. Installed it on server and it works.


python310Packages.angr: 9.2.52 -> 9.2.53

Description of changes
Bump angr stack to 9.2.53

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 235071 run on x86_64-linux 1

16 packages built:

python310Packages.ailment
python310Packages.angr
python310Packages.angrcli
python310Packages.angrop
python310Packages.archinfo
python310Packages.claripy
python310Packages.cle
python310Packages.pyvex
python311Packages.ailment
python311Packages.angr
python311Packages.angrcli
python311Packages.angrop
python311Packages.archinfo
python311Packages.claripy
python311Packages.cle
python311Packages.pyvex


3D Ken Burns Effect from a Single Image

( needs to be a gif )
https://arxiv.org/pdf/1909.05483.pdf

http://sniklaus.com/papers/kenburns


Stable Drupal Chart , mistake in Notes.txt

Is this a request for help?: No

Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT

Version of Helm and Kubernetes:
Helm Version
Client: &version.Version{SemVer:"v2.4.2", GitCommit:"82d8e9498d96535cc6787a6a9194a76161d29b4c", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.5.0", GitCommit:"012cb0ac1a1b2f888144ef5a67b8dab6c2d45be6", GitTreeState:"clean"}

Kubernetes Version
Server Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.7", GitCommit:"8eb75a5810cba92ccad845ca360cf924f2385881", GitTreeState:"clean", BuildDate:"2017-04-27T09:42:05Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
Drupal
What happened:
Notes.txt lists these two commands
export SERVICE_IP=$(kubectl get svc --namespace default silly-aardvark-drupal -o jsonpath='{.status.loadBalancer.ingress[0].*ip*}')
echo http://$SERVICE_IP/

There is no field ip in ingress[0] instead there is hostname. Is this specific to me because I am using aws for deployment and aws has its own loadbalancer
What you expected to happen:
I expect for Notes.txt to print right field value.
Like, in Notes.txt
this line should change
from

export SERVICE_IP=$(........... -o jsonpath='{.status.loadBalancer.ingress[0].ip}')

to

export SERVICE_IP=$(........... -o jsonpath='{.status.loadBalancer.ingress[0].hostname}')

How to reproduce it (as minimally and precisely as possible):
Just do helm install stable/drupal. Read notes and do
kubectl get svc --namespace default <releasename> -o json . See ip field is absent under ingress
Anything else we need to know:

If this really is bug , I would like to fix it.

@EleanorRigby
This is not a bug. Your kubernetes setup probably cannot assign a external ip address to the service. Since the default serviceType field is defined to be of type LoadBalancer. We expect the platform to auto-assign a external ip address to the service. This is known to work on a kubernetes cluster set up on GKE. It may not work on other platforms out-of-the-box.
You should try specifying the value of .Values.serviceType to ClusterIP or NodePort depending on your kubernetes setup.  If you're using minikube, you can simply do minikube service silly-aardvark-drupal to access the service.

Looks like you may be using kubernetes on AWS where they define hostname. I guess we need to mention it in the readme.


Error running go get ...

Golang 1.9
macOS 10.12.6
Hello, I've just downloaded Gonum and run
go get -u -t gonum.org/v1/gonum/....
I got the error message
package gonum.org/v1/gonum/blas/gonum/internal/math32: cannot find package "gonum.org/v1/gonum/blas/gonum/internal/math32" in any of: 	/usr/local/go/src/gonum.org/v1/gonum/blas/gonum/internal/math32 (from $GOROOT) 	/Users/carlca/Code/go/src/gonum.org/v1/gonum/blas/gonum/internal/math32 (from $GOPATH).
I then ran go test gonum.org/v1/... and got either ? or ok, so on the face of it, the tests appeared to run ok. Should I be concerned about the initial error message?
My go env is:
GOARCH="amd64"
GOBIN="/Users/carlca/Code/go/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/carlca/Code/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/1d/1rqcnlq51zd4j_vqhp1kv3540000gp/T/go-build735349917=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

Apologies for the bad formatting. The Github issues editor seems to be playing up.

I can't replicate this. I'm on Go 1.9 osx 10.11.6.
I'm very confused about the error you're getting, because that package doesn't exist. I confirmed with a grep that gonum.org/v1/gonum/blas/gonum/internal/math32 doesn't appear anywhere in the suite and you can see from https://github.com/gonum/gonum/tree/master/blas/gonum that the package isn't there.

internal/math32 is certainly there...

But not under blas ...

yes, but your error isn't about gonum.org/v1/gonum/internal/math32, it's about gonum.org/v1/gonum/blas/gonum/internal/math32  (notice the extra blas/gonum)

Well, it's coming from somewhere!!

Have you tried deleting your pkg directory? Maybe you have something cached there?

Or deleting your repo and downloading fresh?

Deleting the pkg folder and re-running the go get command seemed to get rid of the error. I don’t understand how a previously cached value can have caused the problem since I have never installed Gonum before. Thanks!
On 17 Sep 2017, 17:23 +0100, Brendan Tracey<EMAIL_ADDRESS>wrote:

Or deleting your repo and downloading fresh?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Closing since this seems to be fixed.


Hl 1092 audit logs

Description :sparkles:
CRUD actions for Employee, Attachment and DeMinimisAid models were not logged in the audit log on the applicants side when application is created and updated. Audit logging for Employee is added so that first_name, last_name and social_security fields are left out resulting in a log entry such as following:
{   "audit_event": {     "actor": {       "role": "USER",       "user_id": "97641437-3300-41bd-abb4-c50cbf130035",       "provider": "",       "ip_address": ""     },     "origin": "helsinki-benefit-api",     "status": "SUCCESS",     "target": {       "id": "a4ec879f-bb0f-4774-97ee-ce568b6a1cf6",       "type": "Employee",       "changes": [         "collective_bargaining_agreement changed from  to TES",         "other_expenses changed from None to 200.00",         "job_title changed from  to testo",         "working_hours changed from None to 20.00",         "monthly_pay changed from None to 2500.00",         "is_living_in_helsinki changed from False to True",         "vacation_money changed from None to 100.00"       ]     },     "date_time": "2024-01-11T08:49:17.883Z",     "operation": "UPDATE",     "date_time_epoch":<PHONE_NUMBER>883,     "additional_information": ""   } }
Issues :bug:
Testing :alembic:
Create and update an application, observe from the auditlog_auditlog_entry table that a corresponding row is insterted when:

Updating an employee
Adding DeMinimis entries
Adding and deleting an attachment

Screenshots :camera_flash:
Additional notes :spiral_notepad:

API branch is deployed to platta: https://helsinkilisa-pr2717.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr2717.dev.hel.ninja :laughing::tada::tada::tada:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2717.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-handler-pr2717.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://tet-pr2717.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

API branch is deployed to platta: https://helsinkilisa-pr2717.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

API branch is deployed to platta: https://kesaseteli-pr2717.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

API branch is deployed to platta: https://kesaseteli-pr2717.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

API branch is deployed to platta: https://tet-pr2717.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

API branch is deployed to platta: https://helsinkilisa-pr2717.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

EMPLOYER branch is deployed to platta: https://kesaseteli-employer-pr2717.dev.hel.ninja :rocket::rocket::rocket:

ADMIN branch is deployed to platta: https://tet-admin-ui-pr2717.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tet-admin-ui-pr2717.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://helsinkilisa-ui-pr2717.dev.hel.ninja :laughing::tada::tada::tada:

YOUTH branch is deployed to platta: https://tet-youth-ui-pr2717.dev.hel.ninja :rocket::rocket::rocket:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2717.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tet-youth-ui-pr2717.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://helsinkilisa-ui-handler-pr2717.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://kesaseteli-employer-pr2717.dev.hel.ninja :laughing::tada::tada::tada:

HANDLER branch is deployed to platta: https://kesaseteli-handler-pr2717.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://kesaseteli-handler-pr2717.dev.hel.ninja :laughing::tada::tada::tada:

YOUTH branch is deployed to platta: https://kesaseteli-youth-pr2717.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://kesaseteli-youth-pr2717.dev.hel.ninja :laughing::tada::tada::tada:


Remove context, implement ranked behaviors in client

Checklist

[x] the PR target is correct

main for releases, docs-only fixes and post-release hot-fixes
dev for everything else
this is a sub-pr

[x] ran doit lint locally

References

sub-PR of #78

Code changes

python

[x] removes context and related checks

i don't get what this is for, at present
context is already super overloaded
we already have stringcase knocking around in the demo, which could be used to extract "Link", "Node" and "Graph"

[x] hoists default ranks to some central constants

ts

[x] sorts methods by rank, then cid

User-facing changes

if you force e.g. NodeSelection to go after NodeShape(color="red"), it's gonna be red

Backwards-incompatible changes

relying on the the Tuple order will no longer work

I get what you are saying now.  Yeah, this looks much better.

Dealing with coverage now...


Metamorpheus crashes during Quantification (not responding)

Dear rmillikin,
I would like to analyze 38 Files of Q-Exactive HF. Calibration works perfectly. However, when I search files and would like to do LFQ with normalization and match between runs, after classic search RAM usage ramps up to 12,5 GB  during the quantification process and the program stops responding.  However, it still calculates somehow but I can't do anything anymore. Its running for two days in quantification now.
I used the two variable modifications for deamidation besides standard settings and run metamorpheus with the most updated Thermo raw file reader. mzml files are generated during calibration process.
So this looks all fine.
Do I have too little RAM (I thought 16 GB should be enough)? Any help would be very much appreciated because I think that your software could be become a viable alternative to MaxQuant which we routinely use currently. If you need further information. Please let me know.
Best regards,
Peer

Hi Peer,
Matching between runs is a memory hog right now, unfortunately. Fixing this has been on my to-do list for a while. I'm hoping to have time in the next week or two to take a look at it. It'll hopefully be in the next release - I'll try to keep you posted. Thanks for the crash report!

This has proven to be more difficult than I thought, but I've got it mostly working in my test code. I'm wrapping it up now and it will be in the next release. Stay tuned :) apologies for the delay.

Really looking forward!!!
Von meinem iPhone gesendet

Am 10.11.2018 um 13:05 schrieb Robert Millikin<EMAIL_ADDRESS>This has proven to be more difficult than I thought, but I've got it mostly working in my test code. I'm wrapping it up now and it will be in the next release. Stay tuned :) apologies for the delay.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This is finally out in a new release. If you still encounter RAM issues, it's probably due to MetaMorpheus search and not FlashLFQ quantification. We're still planning on making RAM improvements to the search in the future. Can you give the new version a try?
Thanks,
Rob

Of course.
Best, Peer
Von meinem iPhone gesendet

Am 26.11.2018 um 19:54 schrieb Robert Millikin<EMAIL_ADDRESS>This is finally out in a new release. If you still encounter RAM issues, it's probably due to MetaMorpheus search and not FlashLFQ quantification. We're still planning on making RAM improvements to the search in the future. Can you give the new version a try?
Thanks,
Rob
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Content should not be visible if visible pane prop is false?

Currently if visible is set to false then the size of the pane is set to 0. However, if the content is larger than the size, it will be visible. For example, the text "Drag view here to display' is visible in this screenshot.

Technically, this could be the concern of the user and they should set a suitable overflow value. Also note, that this is not really about whether the pane is visible, if the content overflows the pane at any size it is an issue. In practice I wonder if it is more ergonomic for Allotment to set overflow hidden and remove the requirement to care about this?

Hi @johnwalley, when will this fix be released?

Thanks for the nudge. I've just published it as v1.14.0


Add async getCurrentUser method.

Hey,
I found no way to get state of firebase auth right after app launch. Cached _currentUser is always null until we invoke one of sign-in methods.
I added new asynchronous method getCurrentUser, it invokes recommended native implementations addAuthStateDidChangeListener and caches result to _currentUser

cc @collinjackson @goderbauer can you review please?
@mogol thanks for contributing! Sorry about the slowness here; we have lots of folks on vacation right now.

cc @goderbauer

cc @kroikie for review

Hi @mogol this PR has a conflict, and can't be merged. Is this something you're still working on? Thanks!

Hey, conflicts are solved, should I squash commits?
Also, ideally synchronous getter currentUser should be removed - one source of truth. But it's breaking change if someone relies on it.

I'm fine with removing sync currentUser. Please make a clear note about that in the CHANGELOG. Thanks!
No need to squash on your end, we'll use the "squash and merge" button here. You can squash if you want, though :)

Please also update the version number in pubspec.yaml

Looks reasonable to me, @mit-mit what do you think? It's possible to have currentUser be a getter instead of a method, but I'm not sure which is more idiomatic Dart.

I'm OK with doing the breaking change now.
Regarding method vs getter, I believe we usually only use getters for retrieving fairly simple values, and not when there is complex computation or side effects. Since we are doing an interop hop here, and it's async, I think the current method is a good choice.

Is it possible to start Travis again without push? Failed without output.

Yup; I restarted it!

CI is green. Do I need anything else to merge this PR?

merged it, I'll deploy to pub as well

it's uploaded

🎉 thanks

Thanks!!

Thanks issue solved .

On the web I get null from 'currentUser' after refreshing the browser.

On the web I get null from 'currentUser' after refreshing the browser.

Please open a new issue. Thanks!


ci(build-and-test-differential): limit ccache size for cuda builds

Description
We were running out of disk space on public runners for cuda builds.
I've disabled the build-and-test-differential cuda requirement.
Now limiting the cache size to 1MB until this issue is solved.
As colcon build continues, the cache size should become smaller over time.
This can be made into a variable thing in the future, becoming smaller based on the available disk space during a build.
Related links
Parent Issue:

https://github.com/autowarefoundation/autoware.universe/issues/7501

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

With 3 mega packages modified:
modified-packages: autoware_behavior_path_planner autoware_behavior_velocity_planner pointcloud_preprocessor

https://github.com/autowarefoundation/autoware.universe/actions/runs/9890058965/job/27317461398?pr=7980
has succeeded.
Before build
534M	/root/.ccache
Summary:
  Hits:             917080 / 1597120 (57.42 %)
    Direct:         859924 / 1643936 (52.31 %)
    Preprocessed:    57156 /  738341 (7.74 %)
  Misses:           680040
    Direct:         784012
    Preprocessed:   681185
  Uncacheable:     1159825
Primary storage:
  Hits:            1831713 / 3294601 (55.60 %)
  Misses:          1462888
  Cache size (GB):    0.56 /    0.00 (55634.3 %)
  Cleanups:           1717

After build
4.3M	/root/.ccache
Summary:
  Hits:             917342 / 1600680 (57.31 %)
    Direct:         859975 / 1647550 (52.20 %)
    Preprocessed:    57367 /  741850 (7.73 %)
  Misses:           683338
    Direct:         787575
    Preprocessed:   684483
  Uncacheable:     1161544
Primary storage:
  Hits:            1832203 / 3301952 (55.49 %)
  Misses:          1469749
  Cache size (GB):    0.00 /    0.00 (91.34 %)
  Cleanups:           3301

Disk space after build
Run df -h
Filesystem      Size  Used Avail Use% Mounted on
overlay          73G   72G  706M 100% /
tmpfs            64M     0   64M   0% /dev
shm              64M     0   64M   0% /dev/shm
/dev/root        73G   72G  706M 100% /__w
tmpfs           3.2G  1.2M  3.2G   1% /run/docker.sock
tmpfs           7.9G     0  7.9G   0% /proc/acpi
tmpfs           7.9G     0  7.9G   0% /proc/scsi
tmpfs           7.9G     0  7.9G   0% /sys/firmware

build-and-test-differential (humble, -cuda) is once again a required check now.


How does this handle nested data/relationships when defining a model?

Just came across this plugin and it seems awesome and just the thing I've been looking for.
Scanning briefly over the documentation I can see that I can define the fields & relationships of a model as:
return {
      id: this.attr(null),
      user_id: this.attr(null),
      age: this.attr(''),
      sex: this.attr(''),
      user: this.belongsTo(User, 'user_id')
}

However, my server returns the data nested as follows:
{
    "data": {
        "id": "...",
        "type": "post",
        "attributes": {
            "title": "...",
            "body": "..."
        },
        "relationships": {
            "user": {
                "id": "...",
                "type": "user",
                "attributes": {
                    "name": "TestUser"
                },
                "relationships": []
            }
        }
    }
}

How would I define the relationships for the model using this response since my server doesn't actually return the user_id field in the response but rather the entire user object?
My current idea is to change my server response to actually return the foreign keys under the 'relationships' key, and move the actual returned model to new 'related' key like:
{
    "data": {
        "id": "...",
        "type": "post",
        "attributes": {
            "title": "...",
            "body": "..."
        },
        "relationships": {
            "user_id": "..."
        },
        "related": {
            "user": {
                "id": "...",
                "type": "user",
                "attributes": {
                    "name": "TestUser"
                },
                "relationships": []
            }
        }
    }
}

If I implemented this change, could I then define the relationships as (using a dot notation) like below:
user: this.belongsTo(User, 'relationships.user_id')

Or if I don't change the response, could I do it as:
user: this.belongsTo(User, 'relationships.user.id')

@Alig96 Hi! Thank you so much for the feedback! I'm sorry to tell but currently there're no way to map custom structure, so you can't insert data as is. There's an open issue #326 about this one and if we can implement that, then this becomes possible.

I have something similar in my app too, the server returns the models and their relations (other models).
My solution was to make a helper function to parse the API respose recursively and add relation keys if needed.
In your example, i would check if relationship is empty, if not, for each model, get the id and add it to the post object.
Then continue with inserting process.

@Alig96 I'll be closing the issue because I think we can follow this feature at #326. Please feel free to reopen this if there's anything that is not covered by #326 and needs discussion.


Question regarding ProgressPrinter

Hi first off: wonderful package :)
I have some issues with the ProgressPrinter not showing up even when using the defaultcallbacks.
learner = Learner(model, loss; optimizer=opt, callbacks=[ToGPU()], usedefaultcallbacks=true)
FluxTraining.fit!(learner, epochs, (dl, val_dl)) # where dl, dl_val are both Flux.DataLoader objects

Do I need to do something specific when constructing the Learner which I have missed? From the code it seems like I would need to give it a Progress object, do I have to construct that myself? What requirements does my data-iterator have to fullfill to show up with the defaultcallbacks?

Oh! I found the solution for now. I have to give in the data when the learner is constructed, so do:
learner = Learner(model, loss; optimizer=opt, callbacks=[ToGPU()], usedefaultcallbacks=true, data=(dl, dl_val))
FluxTraining.fit!(learner, epochs)
# where dl, dl_val are both Flux.DataLoader objects

Somehow non-intuitive.

Yes, exactly that. :)
When constructing in the first way the ProgressPrinter works as expected, in the second case it does only print epoch, phase and some dots (as expected when ProgressPrintergot initialized with nothing on construction of the learner).

Will look into it and let you know. Thanks for reporting the issue 👍


[QUESTION] - Encountered could not open file ./kvs_log_configuration error

Hi all,
currently a bit lost with how to resolve this error I am getting...
log4cplus:ERROR could not open file ./kvs_log_configuration
ERROR - Unable to create Rotating Credential provider. Error status: 0x9
ERROR - Unable to create Rotating Credential provider. Error status: 0x9
ERROR: Pipeline doesn't want to pause.
ERROR: from element /GstPipeline:pipeline0/GstKvsSink:kvssink0: Could not initialise supporting library.
Additional debug info:
/amazon-kinesis-video-streams-producer-sdk-cpp/src/gstreamer/gstkvssink.cpp(1374): gst_kvs_sink_change_state (): /GstPipeline:pipeline0/GstKvsSink:kvssink0:
Failed to init kvs producer. Error: Unable to create Rotating Credential provider. Error status: 0x9
2020-06-18 23:38:21 [3069394960]

I have followed the instructions in the readme but when trying to start my data stream I get the above error, previously people who encountered this error had resolved it by setting the LD_LIBRARY_PATH env variable which I have done as instructed by the readme instructions and it is currently set to: /amazon-kinesis-video-streams-producer-sdk-cpp/open-source/local/lib
In the open-source/local/lib directory there was no kvs_log_configuration file so I made one but that didn't fix my issue and I got the same error, tried resetting the access_key/secret_key credentials as well to no avail
Any advice/help would be appreciated.
environment details on the system I am trying to run this are:
Device: Raspberry pi 4 b
OS: raspbian
Folder plugin run from: /amazon-kinesis-video-streams-producer-sdk-cpp
Command line to start the data stream used:
gst-launch-1.0 v4l2src do-timestamp=TRUE device=/dev/video0 ! videoconvert ! video/x-raw,format=I420,width=640,height=480,framerate=30/1 ! omxh264enc control-rate=1 target-bitrate=5120000 periodicity-idr=45 inline-header=FALSE ! h264parse ! video/x-h264,stream-format=avc,alignment=au,width=640,height=480,framerate=30/1,profile=baseline ! kvssink stream-name=“{data-stream-name}" frame-timestamp=dts-only access-key=“{access-key}" secret-key=“{secret-key}" aws-region=“{aws-region}"

@yoyomans ,
The config file exists in https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/tree/master/samples
Can you try adding this this to your pipeline and see if it works?
log-config=samples/kvs_log_configuration

There are two issues.

The higher-level assets such as CPP SDK or GStreamer plugin use log4cplus for logging which requires the configuration file. Make sure you point it to https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/blob/master/samples/kvs_log_configuration though, this will primarily affect the logging only and shouldn't cause fatal errors.
You seem to want to use the file-based credential provider. This is a way to supply fresh credentials with the session token during the playback as the kvs gstreamer plugin doesn't have interactions with the application at runtime. In this way you should specify the file path to the credentials file which should contain the credentials in a single line separated by space. For more info on the format check out: https://github.com/awslabs/amazon-kinesis-video-streams-producer-c/blob/master/src/source/Common/FileCredentialProvider.c#L164

Hi thanks for your replies, my problem was I hadn't set the AWS_ACCESS_KEY_ID and the other env variables needed and had just pasted the credentials directly into the command, when replacing the the strings in the command with the newly set env variables I got this new error
2020-06-19 17:35:57 [2895115360] INFO - describeStreamResultEvent(): Describe stream result event.
2020-06-19 17:35:57 [2895115360] INFO - writeHeaderCallback(): RequestId: 4f4e4163-a88d-429d-b1be-5c814e05f6c7
curlCompleteSync(): HTTP Error 403 : Response: {"message":"The security token included in the request is invalid."}
Request URL: https://kinesisvideo.us-east-2.amazonaws.com/describeStream
Request Headers:
    Authorization: AWS4-HMAC-SHA256 Credential={AWS_ACCESS_KEY_ID}/20200619/us-east-2/kinesisvideo/aws4_request, SignedHeaders=host;user-agent;x-amz-date, Signature=f5e4a4ab76881a2cc591826c84fdcf619a27f69fbe793479a4e4cf0e385c8d8d
    content-length: 31
    content-type: application/json
    host: kinesi
2020-06-19 17:35:58 [2904573024] DEBUG - describeStreamCurlHandler(): DescribeStream API response: {"message":"The security token included in the request is invalid."}
2020-06-19 17:35:58 [2904573024] INFO - describeStreamResultEvent(): Describe stream result event.

can you tell me the correct format I am meant to use to define these env variables when launching gts-launch I've seen some examples with {} either side of env variables and some examples with speech marks and some without.
here is what I am currently using to launch the data stream:
gst-launch-1.0 -v v4l2src do-timestamp=TRUE device=/dev/video0 ! videoconvert ! video/x-raw,format=I420,width=640,height=480,framerate=30/1 ! omxh264enc periodicty-idr=45 inline-header=FALSE ! h264parse ! video/x-h264,stream-format=avc,alignment=au ! kvssink stream-name=teststream access-key={AWS_ACCESS_KEY_ID} secret-key={AWS_SECRET_ACCESS_KEY} aws-region={AWS_DEFAULT_REGION}

@yoyomans ,
Please set the environment variables this way:
access-key=$AWS_ACCESS_KEY_ID

@disa6302 thank you so much for all your help, it appears to be working now :D


[vite:build-import-analysis] Parse error

Describe the bug
[vite:build-import-analysis] Parse error @:1529:13
file: D:/xxx-design-vite-vue2.7/packages/web-vue/components/table/table.tsx:1662:11
1660:       }: { indentSize?: number; indexPath?: number[]; allowDrag?: boolean } = {}
1661:     ): any => {
1662:       const currentKey = record.key;
^
1663:       const currentPath = (indexPath ?? []).concat(rowIndex);
1664:       const expandContent = renderExpandContent(record);
file:///D:/tema-design-vite-vue2.7/node_modules/.pnpm/vite@4.1.1_less@4.1.3/node_modules/vite/dist/node/chunks/dep-3007b26d.js:14787
Reproduction
not
Steps to reproduce
No response
System Info
System:
    OS: Windows 10 10.0.19044
    CPU: (6) x64 Intel(R) Core(TM) i5-9400F CPU @ 2.90GHz
    Memory: 2.90 GB / 15.88 GB
  Binaries:
    Node: 16.15.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.17 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.3.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 110.0.5481.78
    Edge: Spartan (44.19041.1266.0), Chromium (110.0.1587.41)
    Internet Explorer: 11.0.19041.1566

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

i have same issue, why this issue was be closed, any conclusions?


Reset button not working in remote deployment

Bug report
Description
It was noted in demo that the Reset Progress button is not working when deployed to remote. Seems this is because of a leading slash on the path the frontend uses to send the request - see frontend/src/service/levelService and compare to e.g.
Reproduction steps
Steps to reproduce the behaviour:

Deploy live env
Open the Network tab in browser devtools
Click on Reset All Progress button in UI
Click Reset in the confirmation dialog. Note that the dialog does not close.
Note error response in Network tab.

You will see that the path is different for this request, missing the "/dev" prefix compared with other successful requests. It manifests as a CORS issue but that's simply because the path was not recognised, so the request didn't even reach the backend service.
Expected behaviour
All progress is reset, dialog closes, and no error seen in Network tab.
Screenshots

Software (please complete the following information):

all browsers

Acceptance criteria
GIVEN the application is deployed in AWS
AND I have begun a chat with the bot
WHEN I click on Reset Progress button in header
AND a confirmation dialog opens
AND I click Reset button
THEN the dialog closes
AND all chats and level progress are reset to the start of the game

@kclark-scottlogic I've deployed this to remote now, so it's ready for verification. Works for me 🤞

Moving back to in progress as when hitting reset progress chat log is still there even after the popups have appeared and if you proceed to level 2 and then reset, The chat still says you have passed level 1

@kclark-scottlogic As our AWS env is no longer available, can you just ensure it still works when running locally, i.e. regression test the reset progress functionality? Thanks!


toast error does not accept error param

Introduced in #1593. Caught this while working on #2477.
toast.error does not accept an error param. The 2nd function parameter is an options object.
Going to keep this as a draft because I need @franknoirot to show me how to trigger an error here to test.

I'll DM you and @paultag he's got this gnarly shell script that tanks your network but I'll let him give the warnings and guidelines for it

@franknoirot Thanks. I need that for #2477. 🙏 @paultag
See https://github.com/KittyCAD/modeling-app/pull/2479/files#diff-20c63a4046c89f6486ee03bfd3b34833df20b0fec682abd497f8e7063150e339R268-R269
This is behavior handling errors that might occur when performing the actual export operation. I suspect this might be hard to trigger, which is why this hasn't become an issue. It would probably take a bug in our conversion code to trigger.


Performance issue

Hey Profzei really like your work and Thank you :D My only two problems is the known issue with WIndows and a strange Performance issue. I have same Lap like you (16Gb of ram) but only get about 3100 Bench score... do you get an idea what the problem is? Sorry for bad english only have it at German-school. Thank you very much and have nice vacation.

@profzei
Hey, I tested your possibilities but don't get an answer...
I notice that I got a score of 4500 but only in very special constelation. I reinstalled and got it one time but when boot its gone. But I got it with randomness after install but don't know how. Do you have another clue?
Thanks for your work anyway and have nice day. :)

@supremekiind Honestly I don't know... I'm sorry

Its not the SMBIOS got later the day same bad Score. Anyway don't know what's the problem, if anyone other got same Issue and a Fix please mark me (@) Thanks to everybody in this thread.

Hello i am very confused... now with Smbios from a Mac Pro it works stable with 4000-4500 🤔 I also noticed that NVRam reset partially has fixed low performance issue.I dont know what to say  :(
Bye and have a nice day


Implement ConstraintSweep

Closes #1.
To Do:

[ ] Create tests for ConstraintSweep
[ ] Add docstrings

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Unable to load shared library 'libSkiaSharp' - Alpine Linux 3.16 - SkiaSharp.NativeAssets.Linux.NoDependencies 2.88.0

Hello,
I'm getting the error described below, as it looks like the libSkiaSharp.so file is not found.
The file is actually automatically deployed into the docker container but I don't know why it is not loaded/found:
Debug environment:
/app/bin/Debug/netcoreapp3.1/runtimes/linux-musl-x64/native/libSkiaSharp.so
/app/bin/Debug/netcoreapp3.1/runtimes/linux-x64/native/libSkiaSharp.so
Prod environment:
/app/bin/Release/netcoreapp3.1/runtimes/linux-musl-x64/native/libSkiaSharp.so
/app/bin/Release/netcoreapp3.1/runtimes/linux-x64/native/libSkiaSharp.so
Error:
System.TypeInitializationException: The type initializer for 'SkiaSharp.SKData' threw an exception.
 ---> System.DllNotFoundException: Unable to load shared library 'libSkiaSharp' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: Error loading shared library liblibSkiaSharp: No such file or directory
   at SkiaSharp.SkiaApi.sk_data_new_empty()
   at SkiaSharp.SKData..cctor()
   --- End of inner exception stack trace ---
   at SkiaSharp.SKData.Create(Stream stream)
   at SkiaSharp.SKImage.FromEncodedData(Stream data)

Used SkiaSharp package:
SkiaSharp.NativeAssets.Linux.NoDependencies
(Version 2.88.0)
Environment:
Azure Linux Web App running in Docker Container.
.NET Core version:
3.1.28
Docker Base Image:
FROM mcr.microsoft.com/dotnet/core/aspnet:3.1-alpine AS base
(v3.16 is the underlying Alpine Linux version)
Other Alpine packages added (required for other functionalities)
RUN apk add --no-cache icu-libs
RUN apk add --no-cache libgdiplus --repository http://dl-cdn.alpinelinux.org/alpine/edge/testing --allow-untrusted
RUN apk add --no-cache chromium
RUN apk add --no-cache chromium-chromedriver
RUN apk add --no-cache tini
Thx for your help.
PS:
It was working long time ago with some older .NET Core 3.1.x version, Alpine 3.x version and SkiaSharp.NativeAssets.Linux.NoDependencies version, but I can't say which version upgrade broke it.

Duplicate of #2169
Unfortunately, there is an issue with the dotnet on the alpine images. They can't load any native binaries since 3.15. See the linked issue as well for workarounds.

Filed this issue to hopefully get it fixed: https://github.com/dotnet/sdk/issues/27156


sed error when running container with non-root user

A sed-command error appears when starting the container with non-root user.
sed: cannot rename /etc/modsecurity.d/owasp-crs/sedoXeakT: Operation not permitted

I've tried to avoid that error using chmod -R 777 and chown, but it doesn't work.

HI @lasperr ! Which container are you using?

Hi @fzipi ! I used modsecurity-crs:nginx image

Probably related to #55 then. When adding those permissions, will fix this. Let me take a look.

Unfortunately, it doesn't work.
RUN chown -R nginx:nginx /etc/nginx/
RUN chown -R nginx:nginx /opt/modsecurity/
RUN chown -R nginx:nginx /etc/modsecurity.d/
RUN chown -R nginx:nginx /opt/owasp-crs/
RUN chown -R nginx:nginx /usr/local/
RUN chown -R nginx:nginx /usr/share/nginx

USER nginx

I even did this
RUN chmod -R 777 /etc/nginx/
RUN chmod -R 777 /opt/modsecurity/
RUN chmod -R 777 /etc/modsecurity.d/
RUN chmod -R 777 /opt/owasp-crs/
RUN chmod -R 777 /usr/local/
RUN chmod -R 777 /usr/share/nginx

I resolved it by looking at all of the paths that you are using in your config files and then chowning them one by one. It is possible that you are also using directories like /var/www/... which I decided to not use and am using a /home/nginx/app path rather.

So which perms need to be changed then?

Closing this one as it is related to running nginx as non-root.


"All crop params are initially optional." When infact they are not

Stated in README and in
https://github.com/DominicTobias/react-image-crop/blob/7c4ddb90a7cf33db2b55a7449f5091f769254d50/src/ReactCrop.tsx#L200-L201
but only the aspect-prop is optional: https://github.com/DominicTobias/react-image-crop/blob/7c4ddb90a7cf33db2b55a7449f5091f769254d50/src/ReactCrop.tsx#L164-L171

Good point I will update the defs in the next day or 2

Don't forget to update the interface definition! Thanks!

@Slumber86 this should be updated in 9.0.4, are you still experiencing issues?

@DominicTobias the problem is with
export interface Crop {
  aspect?: number;
  x: number;
  y: number;
  width: number;
  height: number;
  unit: 'px' | '%';
}

We use typescript and we provide the library a Crop object
At the moment only aspect is optional, in 9.0.4 this is unchanged. I've looked at the source code and realized that i can provide Partial<Crop> instead of Crop to make things work, but is better to make it explicit in the docs

You should be able to pass in a partial object now: https://github.com/DominicTobias/react-image-crop/blob/master/src/ReactCrop.tsx#L200 but maybe I've missed something. Please check in node_modules you have the right version and restart your IDE and I'll check what's up if the issue persists

@DominicTobias sure is what I've done.
but still the Crop object has not optionals, you have to pass a partial. There is a subtle difference but still a difference to point out

Do you mean allowing the whole object to be optional (e.g null or
undefined)? I think that works though it wasn’t the original intention.
There are places where the crop is spread …crop which would cause an
exception if executed
On Tue, 7 Sep 2021 at 14:07, Agostino Marzotta @.***>
wrote:

@DominicTobias https://github.com/DominicTobias sure is what I've done.
but still the Crop object has not optionals, you have to pass a partial.
There is a subtle difference but still a difference to point out
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DominicTobias/react-image-crop/issues/439#issuecomment-914291453,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFZT6TGEYODOJKR3B5Z2I3UAYFB3ANCNFSM5DD5APNQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sure, there is infact a trade-off between internal implementation and develper-friendliness of the APIs. Just highlight the partial usage in the docs!

@DominicTobias making the prop crop a Partial <Crop> object as a parameter fixes the issue of having to pass a full Crop into the component, but it does not fix the issue of declaring a local value as a Crop.
For example, your current definition of a Crop is
export interface Crop {
  aspect?: number;
  x: number;
  y: number;
  width: number;
  height: number;
  unit: 'px' | '%';
}

so, if I wanted to create a local state variable to store the current crop in my component, it may look something like
const [myCrop, setMyCrop] = useState<Crop>({ unit: '%', width: 50, aspect: 1 }); //this throws a type error

This particular line, however, would throw a TypeError because my initial state value of { unit: '%', width: 50, aspect: 1 } is missing the parameters height, x, and y.
Changing the type definition of Crop to be:
export interface Crop {
  aspect?: number;
  x?: number;
  y?: number;
  width?: number;
  height?: number;
  unit?: 'px' | '%';
}

may just work, though I do not know if you need any of those fields to be required specifically.
Hope this sheds some light on the issue.

This has changed in v10 (along with many breaking changes) and is no longer an issue as a complete crop always has to be passed in. Check the release notes for more info.


docs: Main page references video that no longer exists

The file docs/index.rst, hosted at https://pipenv.readthedocs.io/en/latest/, embeds a Vimeo video ("Pipenv screencast") that no longer exists:
https://player.vimeo.com/video/233134524
If there is no intention to restore it, perhaps it should be replaced with the gif from the README:
https://s3.amazonaws.com/media.kennethreitz.com/pipenv.gif

Unable to get access to the gif.
https://s3.amazonaws.com/media.kennethreitz.com/pipenv.gif

Need another store for the image file, any recommendations?


[REQUEST]compare lists

could you plz add function to compare lists with option for results for example keep only dup on both lists or non dups.

Hello, this repository is not maintained anymore. Stay tuned for OB2 coming out in Q1 this year.

Hello, this repository is not maintained anymore. Stay tuned for OB2 coming out in Q1 this year.


Rename try_give_memory_back to a better free_unused_memory.

Just renaming.

LGTM

LGTM

Checks passed, I'll land it.


Update CONTRIBUTING.md for v3

FEATURE REQUEST
Inspiration
The contrbuting doc must be relevant for tracebase 3 for publication.
Description
Update the example commands and any other outdated references.  We might consider adding things like how to process data submissions, like what's in the admin docs.
Installation and deployment instructions must be clear.  This also includes conversion of the example data to v3.
Alternatives

None
Dependencies
None
Comment

None

ISSUE OWNER SECTION
Assumptions

List of assumptions that the code will not explicitly address/check
E.g. We will assume input is correct (explaining why there is no validation)

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected Components

change: File path or DB table ...
add: Environment variable or server setting
delete: External executable or cron job

Requirements

[ ] 1. List of numbered conditions to be met for the feature
[ ] 2. E.g. Every column/row must display a value, i.e. cannot be empty
[ ] 3. Numbers for reference & checkboxes for progress tracking

DESIGN
Interface Change description

None provided
Code Change Description

None provided
Tests

[ ] 1. A description of at least one test for each requirement above.
[ ] 2. E.g. Test for req 2 that there's an exception when display value is ''
[ ] 3. Numbers for reference & checkboxes for progress tracking

Merged


[bug] Automatic migration does not update @tauri-apps/api

Describe the bug
I recently migrated from Tauri v1 to v2. I used yarn and the automatic migration script offered. When I compiled my app, IPC did  not work at all. Turns out that @tauri-apps/api was not updated to version v2.
Reproduction

yarn up @tauri-apps/cli@latest
yarn tauri migrate
yarn tauri build

Expected behavior
I expected that the automatic migration script upgrades @tauri-apps/api to 2.0.3 (current latest).
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.22631 x86_64 (X64)
    ✔ WebView2: 129.0.2792.89
    ✔ MSVC:
        - Visual Studio Build Tools 2019
        - Visual Studio Community 2022
    ✔ rustc: 1.78.0 (9b00956e5 2024-04-29)
    ✔ cargo: 1.78.0 (54d8815d0 2024-03-26)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 22.4.1
    - yarn: 4.5.1
    - npm: 10.8.3

[-] Packages
    - tauri 🦀: 2.0.6
    - tauri-build 🦀: 2.0.2
    - wry 🦀: 0.46.3
    - tao 🦀: 0.30.3
    - @tauri-apps/api : 1.6.0 (outdated, latest: 2.0.3)
    - @tauri-apps/cli : 2.0.5

[-] Plugins

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../build
    - devUrl: http://localhost:8080/
    - framework: Svelte
    - bundler: Vite

I guess based on tauri info one could resolve the issue by seeing how it is outdated, but it might be worth considering adding some warning or even enhancing the migration script. Or did I miss something from the docs that referred to this problem?
Cheers.

I wasn't able to reproduce, could you share the project that failed to migrate or a minimal repro?

The multirepo here contains the client subproject that contains the codebase for Tauri. I had to manually bump the version of @tauri-apps/api. The main branch still has the old version if you want to test it on that. Again, just for clarity, I use yarn.