CRDs shouldn't be templated in Helm...

I'm trying to install CRDs in a more proper way, meaning not using the /crds directory at the top level of Helm, b/c it doesn't allow for upgrading.  If you read, https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#method-1-let-helm-do-it-for-you
I'm running into issue with my automation to install CRDs though b/c they are templated and things in the CRD folder shouldn't be templated.  From the link above.
There is now a special directory called crds that you can create in your chart to hold your CRDs. These CRDs are not templated, but will be installed by default when running a helm install for the chart. If the CRD already exists, it will be skipped with a warning. If you wish to skip the CRD installation step, you can pass the --skip-crds flag.

Further it feels like there really isn't a needed b/c all it's setting are labels.  I'm more than happy to open a PR for this.

Any update on this?

Interested in this too

Interested in this too, since I would like to use the healthcheck feature on CRD with FluxCD. Thanks :)

Agreed. Helm now recommends putting crds in the crds folder of the chart. Would rather not have a separate chart for CRDs.

Thanks for the input on this issues. There are some other considerations and difference with helm that we have documented here.
tl;dr our CRDs can change and we want them to be in sync with the components. Helm's crds folder doesn't allow uprading / removal etc.
I'm not sure I understand the problem precisely, is it that:

You don't want any template logic in the CRDs?
You want the CRDs to use the crd directory in the helm chart (we do not do this).?

@bradenwright - what is your automation? Is it outputting that message as an actual blocking error, or as a warning?
As a workaround, you could apply the CRDs as YAML directly and use the chart for cert-manager without the installCRDs flag being set?
kubectl apply -f https://github.com/cert-manager/cert-manager/releases/download/v1.12.3/cert-manager.crds.yaml
helm install \
  cert-manager jetstack/cert-manager \
  --namespace cert-manager \
  --create-namespace \
  --version v1.12.3

@f-bn could you link to the flux healthcheck feature for CRDs please? Interested to see what that requires.

Updating CRDs is definitely a challenge when they are put into the crds folder. The pattern we follow for charts with CRDs is to do a kubectl apply -f path/to/crds before the helm update. This ensures any CRD updates are captured.
Additionally, having them in the crds folder - and not templated in the Helm chart - allows things to continue to work if the component is ever removed. We’ve run into this when upgrading ArgoCD. Had to remove it first. But because the Application and Project CRDs and CRs remained - there were no issues.
I understand the dilemma maintainers have a round this. No easy answer.

Of course ArgoCD has now moved them into templates in their latest chart, so… what to do I know 😂🤦♂

So one more quick comment. Looks like CRDs in the templates directory can be annotated with helm.sh/resource-policy: keep - this is what ArgoCD is now doing to keep CRDs if the Helm chart uninstalled. I like this approach as I can then kubectl delete them if I want. Would this be a good middle ground for cert-manager? Could base it on values.keepCRDs.

Hello @hawksight, here is the link to the documentation about this feature : https://fluxcd.io/flux/components/kustomize/kustomization/#health-checks
For now, I use something like this in my CRDs kustomization in FluxCD, this works fine :
---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
  - kyverno.yaml
  - metallb.yaml
  - openebs-jiva.yaml
  - openebs-localpv-lvm.yaml
  - traefik.yaml
  # Cert-Manager - there is no CRDs folder available
  - https://github.com/cert-manager/cert-manager/releases/download/v1.12.2/cert-manager.crds.yaml

For other operators available here, I use the healthcheck feature like this :
---
apiVersion: source.toolkit.fluxcd.io/v1
kind: GitRepository
metadata:
  name: traefik-crds
  namespace: flux-system
spec:
  interval: 12h
  url: https://github.com/traefik/traefik-helm-chart.git
  ref:
    tag: 'v23.2.0'
  ignore: |
    # exclude all
    /*
    # path to crds
    !/traefik/crds/
---
apiVersion: kustomize.toolkit.fluxcd.io/v1
kind: Kustomization
metadata:
  name: traefik-crds
  namespace: flux-system
spec:
  interval: 30m
  sourceRef:
    kind: GitRepository
    name: traefik-crds
    namespace: flux-system
  prune: true
  healthChecks:
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: ingressroutes.traefik.io
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: ingressroutetcps.traefik.io
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: ingressrouteudps.traefik.io
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: middlewares.traefik.io
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: middlewaretcps.traefik.io
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: serverstransports.traefik.io
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: tlsoptions.traefik.io
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: tlsstores.traefik.io
    - apiVersion: apiextensions.k8s.io/v1
      kind: CustomResourceDefinition
      name: traefikservices.traefik.io

I just don't see how to do it with Cert-Manager CRDs with a single link, but to be completely transparent, I'm by no mean a FluxCD guru, maybe there is an another way to do it :)

@f-bn, thank you for the example. I am completely new to flux so that's helpful.
For cert-manager, we do publish the YAML CRDs in git here: https://github.com/cert-manager/cert-manager/tree/master/deploy/crds. So could you use that to do cert-manager CRD health checks, similarly to how you do it for the Traefik example? You may want to of course link to the v1.12.2 tag rather than master branch.

Looks like CRDs in the templates directory can be annotated with helm.sh/resource-policy: keep

@michaelajr - I have thought about this before as well. It only works if it's present at install time in my testing, so it would need to be in the chart. However, the side effect of this annotation is that Helm no longer updates the resources either :-(
I do think there needs to be a mechanism to keep the CRDs if someone has already installed them with --set installCRDs=true because it seems difficult to undo that safely (without losing resources). There's also an interesting comment over on #5772.
I think this would be a good topic to discuss with maintainers on one of the bi-weekly calls, to get a consensus of what the right approach is, that supports all use / most cases. My personal take is that we should have a cert-manager-crds chart that does only CRDs, the equivalent of the YAML install but with the flexibility to template things (I like the labels in there). Similar to how istio and linkerd have their helm installations.

@f-bn, thank you for the example. I am completely new to flux so that's helpful.
For cert-manager, we do publish the YAML CRDs in git here: https://github.com/cert-manager/cert-manager/tree/master/deploy/crds. So could you use that to do cert-manager CRD health checks, similarly to how you do it for the Traefik example? You may want to of course link to the v1.12.2 tag rather than master branch.

Yep, I found these CRDs, but the problem is they are templatized. However, for managing CRDs I don't use the HelmRelease reconciliation in FluxCD, but a Kustomization reconciliation which uses kustomize under the hood.
That's mainly the problem since I don't see how I can provide the required values for the template with this and therefore applying the manifests correctly (I don't find anything in the FluxCD documentation to substitute Helm template anchors with a Kustomization). If these CRDs weren't templatized, this will solve the problem since my kustomization reconciliation will simply apply plain manifests like I did for Traefik, and then I can use healthchecks on these CRDs afterwards.
As an example for Traefik, I simply asked the Kustomization reconciliation to pull CRDs in the Traefik Helm chart and apply them as-is. Since these are plain CRDs manifests, this works perfectly and is very simple to manage.

Thank you I forgot those had variables in them :facepalm:. I remember now we didn't used to have a plain YAML file for CRDs, but I think we might now. I just checked, here is a link to the latest release:

https://github.com/cert-manager/cert-manager/releases/download/v1.12.3/cert-manager.crds.yaml

I had a quick check and it looks like this version has no templating.. well at least not in the labels :)
yq ea '.metadata' ~/Downloads/cert-manager.crds.yaml

Since it's just the labels that are templated, you can use spec.commonMetadata instead of patches. It makes the kustomzation a bit cleaner looking.
spec:
  commonMetadata:
    labels:
      app: "cert-manager"
      app.kubernetes.io/name: "cert-manager"
      app.kubernetes.io/instance: "cert-manager"

Having the CRDs outside the crds folder makes it kinda impossible to use cert-manager as a dependency for another chart, as the CRDs then won't be installed before the parent chart needs them, see https://github.com/kubernetes-sigs/cluster-api-operator/issues/281.
I would say this chart should at least also populate the crds folder, even if it also offers the templated variant for non-gitops users, to make it usable in normal workflows.

IMHO, I like cert-manager's current approach within helm, except it's unnecessarily painful for advanced users. Can we instead break out the CRDs to another chart, and then include it as an optional dependency? That would be the halfway point between ease-of-use and safety. I've seen more and more projects adopt this approach.
Here is a similar comment in another issue

Personally, I don't like the idea of having 2 different charts too 😇
I take the liberty to quote myself on another issue: https://github.com/cert-manager/cert-manager/issues/5772#issuecomment-1665641035
-> The idea is to have a boolean that controls if CRDs are templated or not.
-> Thereby we can have 2 different releases of the same chart.

One managing the CRDs lifecycle
The other the components

Personally, I don't like the idea of having 2 different charts too 😇
I take the liberty to quote myself on another issue: #5772 (comment)
-> The idea is to have a boolean that controls if CRDs are templated or not. -> Thereby we can have 2 different releases of the same chart.

One managing the CRDs lifecycle
The other the components

But this still doesn't fix https://github.com/cert-manager/cert-manager/issues/6179#issuecomment-1764051690
Additionally, as @michaelajr already said, helm recommends putting the CRDs in the crds folder.
And to quote myself; if you're doing gitops, the crds folder works perfectly and if you're doing it manually, you might as well apply the crds folder 🤷

https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#install-a-crd-declaration-before-using-the-resource or using a top level crds folder is 1 of 2 recommended approaches from Helm and it does automaticallly install (or allow skipping).  If crds were placed in the folder instead of in the templates folder (using templating for only labels) and using templating breaks the approach I use with ArgoCD for all my CRDs.  Anything that follows either of the 2 approach recommended by helm work fine.  The current approach which isn't recommended by Helm, break things.
For now I've rendered out and saved the crds to a location in my own git, which is def hacky.  Just for reference this is how I install most CRDs, any that have a top-level directory:
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: sandbox-bootstrap-crds
  namespace: argo
  resourceVersion: '453686192'
  uid: 6cf4fa4b-8017-4c92-a070-9ed4c8cb6f35
spec:
  project: sandbox-bootstrap
  destination:
    namespace: bootstrap
    name: sky-sandbox-01
  syncPolicy:
    automated:
      prune: true
      selfHeal: true
    syncOptions:
      - ApplyOutOfSyncOnly=true
      - Replace=true
      - PruneLast=true
  ignoreDifferences:
    - group: apiextensions.k8s.io
      kind: CustomResourceDefinition
      jsonPointers:
        - /spec/conversion/webhook/clientConfig/caBundle
        - /spec/conversion/webhook/clientConfig/service/name
        - /spec/conversion/webhook/clientConfig/service/namespace
  sources:
    - repoURL: 'git@gitlab.com:myrepo/k8s-crds'
      path: shared/crds/cert-manager/v1.10.0
      targetRevision: develop
      directory:
        recurse: true
        jsonnet: {}
    - repoURL: 'https://github.com/external-secrets/external-secrets'
      path: deploy/crds
      targetRevision: v0.9.9
      directory:
        recurse: true
        jsonnet: {}
        exclude: kustomization.yaml

But my vote would be a top-level crds directory, but at least following a recommended approach would be great.

Presented options:

Remove the CRDs entirely

cons: makes dev environments more difficult to setup
pros: no more surprises with helm lifecycles

Place the CRDs into crds/

cons: newer feature and already has known lifecycle caveats. doesn't handle upgrades.
pros: the dependency tree would be clear for multi-chart charts. easy dev bootstrapping.

Leave them in the chart, but create a boolean toggle for only them

cons: invites possibilities for typos and misuse, not a common pattern
pros: the dependency tree would be clear for multi-chart charts. easy dev bootstrapping. handles upgrades

Move the CRDs to a dedicated chart

cons: ???
pros: the dependency tree would be clear for multi-chart charts. easy dev bootstrapping. handles upgrades.

I really think option 4 is the best, given all the scenarios and concerns.
note: my deleted reply was to a mistaken interpretation. I hope this list helps create a more clear shared understanding. my apologies for coming in hot :smile:

Presented options:

Remove the CRDs entirely

cons: makes dev environments more difficult to setup
pros: no more surprises with helm lifecycles

Place the CRDs into crds/

cons: newer feature and already has known lifecycle caveats. doesn't handle upgrades.
pros: the dependency tree would be clear for multi-chart charts. easy dev bootstrapping.

Leave them in the chart, but create a boolean toggle for only them

cons: invites possibilities for typos and misuse, not a common pattern
pros: the dependency tree would be clear for multi-chart charts. easy dev bootstrapping. handles upgrades

Move the CRDs to a dedicated chart

cons: ???
pros: the dependency tree would be clear for multi-chart charts. easy dev bootstrapping. handles upgrades.

I really think option 4 is the best, given all the scenarios and concerns.
note: my deleted reply was to a mistaken interpretation. I hope this list helps create a more clear shared understanding. my apologies for coming in hot 😄

I have a slight preference towards (2), but other options would work.  It would also be helpful if the solution was for 3 or 4, if the templating was removed from the CRDs.  Currently they just set labels.

if the templating was removed from the CRDs. Currently they just set labels.

My apologies for not making that clear in the list - I ran with the assumption that any templating would be removed in all scenarios. I agree with you that it's an additional lifecycle concern that no one should have to deal with, and the other projects I've seen that use Helm to deploy CRDs are only applying a static dump. Thank you for bringing up that detail.

Sounds great!  Thanks for pushing this forward when I fell off after a while.

Presented options:
...
3. Leave them in the chart, but create a boolean toggle for only them

cons: invites possibilities for typos and misuse, not a common pattern

OK, however, "the typos and misuse" argument would match anything that is a variable and left to the user input 🙃
That is why there are default values.yaml and README isn't.

not a common pattern

As often for any new pattern 😇

Move the CRDs to a dedicated chart

cons: ???
pros: the dependency tree would be clear for multi-chart charts. easy dev bootstrapping. handles upgrades.

cons: Charts lifecycles. It is additional work for the maintainers.
Personally, I prefer option 3 regarding this last "cons" ☝🏻

Apparently, the kube-prometheus-stack Helm chart, where I saw the option 3 pattern, moved to the option 4 😅
Here is the issue where they had the same discussion and a comment about automation for the CRD dedicated chat:
https://github.com/prometheus-community/helm-charts/issues/2612

Thank you everyone for all the input here!
I have a desire to move CRDs to their own chart (option 4). Given that this topic is about whether to template or not to template those CRDs.... perhaps we could:

cert-manager-crds (templated)
cert-manager-crds-helm (no template, put them in the /crds directory)

If we had both of those additional charts... people could pick their poison?
Naming is up for debate but illustrates a possible route forward.

Thank you everyone for all the input here!
I have a desire to have the CRDs in their own chart (option 4). Given that this topic is about whether to template or not to template those CRDs.... perhaps we could:

cert-manager-crds (templated)
cert-manager-crds-helm (no template, put them in the /crds directory)

If we had both of those additional charts... people could pick their poison? Naming is up for debate but illustrates a possible route forward.

Yes, sounds like a great compromise between the two camps. I would suggest following the same path kube-prometheus-stack took, by including the cert-manager-crds-helm chart directly in the cert-manager/charts folder without releasing it, and maybe just naming it crds?, and providing the cert-manager-crds chart separately 👍

I have a small note to share. I notice that the Argo project is making use of the helm.sh/resource-policy annotation, which prevents helm from deleting a resource at uninstall time. It's an elegant solution to managing CDRs. The associated values are light.
Unfortunately this pattern alone leaves ambiguity around any CRD updates and administrator review needed, but if it were combined with an option for the chart to only output CRDs, as suggested a few times here, it could be a promising all-in-one pattern.
I'm not sure I would recommend this path right now, but I thought it worth sharing.

I will say I am also running into an issuer where we want to package the cert-manager chart as a dependency in our chart for easy install but running into a roadblock when using crds in other charts. It does seem like the standard does seem to follow to use crds/ no?

Any update / plans on moving this forward, @ricardo-noyolalozano I think https://github.com/cert-manager/cert-manager/issues/6179#issuecomment-1886654965 propsal would be great b/c its in crds or can be used as a separate chart.
For me personally I can use either, I just can't use templated files in crd folder.

This has been open for over 1 year and still nothing has progressed.  For my use case I just need there to not be templating in the CRDs folder, which is a standard thing in K8s as I mentioned in the initial issue description.  The labels are very generic, e.g. https://github.com/cert-manager/cert-manager/blob/master/deploy/crds/crd-certificaterequests.yaml#L12
If I open an MR to remove the templating would it be something we can merge in.  It's just really annoying to have to manage CRDs in my own github and copy them over / etc.

I'm leaning towards opening an MR to remove the labels that use templating and trying to get it approved

I created a request for a standalone cert-manager CRDs charts, similar to other applications listed in the issue https://github.com/cert-manager/cert-manager/issues/7422


Null Program location exception after checking-in a file

I noticed after installing the plug-in for Ghidra 9.2.2 that after checking in a file the CodeBrowser reports Null Program Location set repeatedly until the CodeBrowser window is closed and re-opened.  Made a quick fix as I believe the root issue is that after a check-in the program closed event is triggered nulling out the state variables, but the program activation event is never called again.

You are talking about the version control checks I assume? This is indeed something I never tested, highly likely that I missed something there. My gut feeling is that there is probably some other event that the plugin should handle, instead of doing this in the locationChanged callback. Will look into this and see if your fix is the best solution. Thank you very much for reporting this!

Yeah, exactly during the version control checks.  Agreed, it seems like there should be another explicit event to handle file check-ins.
Sure, no problem.

Okay, I finally got around to properly looking into this. I am using Ghidra 10.1 and the ghidra-server.org Server which is also Ghidra 10.1.
What happens is that when a program is opened (i.e. the Codebrowser tool is open), something is changed and the binary is checked in while the code browser is running the events seem to fire in the following order:
INFO  Checkin completed for xochitl (GhidraFileData)
INFO  Program opened (JupyterKotlinPlugin)
INFO  Program deactivated (JupyterKotlinPlugin)
INFO  Selection changed to null (JupyterKotlinPlugin)
INFO  Highlight changed (JupyterKotlinPlugin)
INFO  Program activated (JupyterKotlinPlugin)
INFO  Location changed to MemoryBlockStartFieldLocation@00010000, row=0, col=0, charOffset=0, Comment Type = -1 (JupyterKotlinPlugin)
INFO  Program closed (JupyterKotlinPlugin)

Afterwards clicking anywhere (and changing the location) will result in an error titled Null program passed to ProgramLocation.  Trace and remove this problem
with the traceback:
Null program passed to ProgramLocation
java.lang.NullPointerException: Null program passed to ProgramLocation
	at ghidra.program.util.ProgramLocation.<init>(ProgramLocation.java:72)
	at ghidra.program.util.ProgramLocation.<init>(ProgramLocation.java:128)
	at ghidra.app.script.GhidraState.setCurrentAddress(GhidraState.java:141)
	at ghidra.app.script.GhidraScript.setCurrentLocation(GhidraScript.java:501)
	at GhidraJupyterKotlin.JupyterKotlinPlugin.locationChanged(JupyterKotlinPlugin.java:317)
	at ghidra.app.plugin.ProgramPlugin.processEvent(ProgramPlugin.java:173)
	at ghidra.framework.plugintool.Plugin.eventSent(Plugin.java:329)
	at ghidra.framework.plugintool.mgr.EventManager.sendEvents(EventManager.java:286)
	at ghidra.framework.plugintool.mgr.EventManager.lambda$new$3(EventManager.java:49)
	at ghidra.util.Swing.doRun(Swing.java:292)
	at ghidra.util.Swing.runNow(Swing.java:208)
	at ghidra.util.Swing.runNow(Swing.java:163)
	at ghidra.framework.plugintool.mgr.EventManager.fireEvent(EventManager.java:216)
	at ghidra.framework.plugintool.PluginTool.firePluginEvent(PluginTool.java:475)
	at ghidra.app.plugin.core.codebrowser.CodeBrowserPlugin.locationChanged(CodeBrowserPlugin.java:276)
	at ghidra.app.plugin.core.codebrowser.CodeViewerProvider.programLocationChanged(CodeViewerProvider.java:521)
	at ghidra.app.util.viewer.listingpanel.ListingPanel.fieldLocationChanged(ListingPanel.java:797)
	at docking.widgets.fieldpanel.FieldPanel$CursorHandler.notifyCursorChanged(FieldPanel.java:2133)
	at docking.widgets.fieldpanel.FieldPanel$CursorHandler.setCursorPos(FieldPanel.java:1928)
	at docking.widgets.fieldpanel.FieldPanel$FieldPanelMouseAdapter.mousePressed(FieldPanel.java:1316)
	at java.desktop/java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:287)
	at java.desktop/java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:287)
	at java.desktop/java.awt.Component.processMouseEvent(Component.java:6632)
	at java.desktop/javax.swing.JComponent.processMouseEvent(JComponent.java:3342)
	at java.desktop/java.awt.Component.processEvent(Component.java:6400)
	at java.desktop/java.awt.Container.processEvent(Container.java:2263)
	at java.desktop/java.awt.Component.dispatchEventImpl(Component.java:5011)
	at java.desktop/java.awt.Container.dispatchEventImpl(Container.java:2321)
	at java.desktop/java.awt.Component.dispatchEvent(Component.java:4843)
	at java.desktop/java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4918)
	at java.desktop/java.awt.LightweightDispatcher.processMouseEvent(Container.java:4544)
	at java.desktop/java.awt.LightweightDispatcher.dispatchEvent(Container.java:4488)
	at java.desktop/java.awt.Container.dispatchEventImpl(Container.java:2307)
	at java.desktop/java.awt.Window.dispatchEventImpl(Window.java:2772)
	at java.desktop/java.awt.Component.dispatchEvent(Component.java:4843)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:772)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:95)
	at java.desktop/java.awt.EventQueue$5.run(EventQueue.java:745)
	at java.desktop/java.awt.EventQueue$5.run(EventQueue.java:743)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:742)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

So for some reason there is a program closed event after all the other program opening and activation.
I think the actual root cause was that we weren't implementing programClosed properly. The argument to it isn't necessarily the program that was set opened and activated earlier. For example FunctionGraphPlugin.programClosed explicitly checks if the passed argument is even the program that is considered "current"
	@Override
	protected void programClosed(Program program) {
		if (currentProgram == program) {
			currentProgram = null;
		}

		connectedProvider.programClosed(program);

		Iterator<FGProvider> iterator = disconnectedProviders.iterator();
		while (iterator.hasNext()) {
			FGProvider provider = iterator.next();
			if (provider.getProgram() == program) {
				iterator.remove();
				removeProvider(provider);
			}
		}

adding the same kind of check to the JupyterKotlinPlugin.programClosed seems to work, because the argument is the old program being closed, while the new one was activated earlier already.
Currently the Kotlin Kernel plugin doesn't really manage multiple programs, so this fix should be sufficient and fixes the addresses the actual root cause. Still, your PR might have been an acceptable workaround ( I guess it at least worked for you), so I am kinda sorry that I let perfect be the enemy of good here, and only addressed this 8 months later :/


Providers: Add NSOne support

https://github.com/ns1/ns1-go

#957 Is a duplicate of this issue, but it seems to be getting more traction than this one as it has a pull request #963. Assuming that NS1 and NSOne are the same, I think 957 is your best bet.

Sorry, I didn't see this issue before opening duplicate #957 and #963.  Let me know how you'd like to proceed here.

FYI the NS1 provider has been merged into master.  I'd imagine this should be available for use the next time a new build of the container is made available


Why copy RLS packages to BucketFS

Question:
The user guide states:

Download the latest available release of Row Level Security to BucketFS.

Why is this step necessary? The next step says to paste the entire contents of the RLS lua code into the adapter script so it seems it's not pulling it from the BucketFS.
Wondering,
Peter

That must be a remnant from porting the document from Java RLS to Lua RLS. In the Java world the adapter is implemented in a JAR archive which needs to be uploaded to BucketFS before you can register the adapter script.
In the Lua case you are right, this is not necessary since the adapter script already contains everything you need. I am going to review the handbook and rework that passage for Lua RLS.

Fixed with PR #82.


Failed to resolve MaterialTabs

Hi!
The demo looked impressive, I wanted to give it a try, but I got the following Gradle error:
Failed to resolve: com.github.pizza:MaterialTabs:2.0.3
As other projects from jitpack.io compile successfully I think that it must be an issue with MaterialTabs.
Thanks,
Robert

Hi @throbi, thanks! Just use compile 'io.karim:materialtabs:2.0.5' instead and it should all work fine.


docs(examples): add Google Analytics example

Example for how to use Google analytics with Remix

Also, sorry it took so long to get back to you on this. Thanks for taking the time!

One thing that might be useful to have in this example is a way to conditionally load google analytics.
For example, if you're in development mode running the app locally you wouldn't want to load the script on the page
https://remix.run/docs/en/v1/guides/envvars

EDIT I was able to make it work, but unsure if this is the best way.
So I looked at Remix docs for setting server environment variables

https://remix.run/docs/en/v1/guides/envvars#server-environment-variables

Below I followed what the docs links above mentioned:
app/root.tsx
export async function loader() {
  return json({
    ENV: {
      APP_ENV: process.env.NODE_ENV, // 'development | production'
    },
  });
}

function Root() {
  return (
    <html>
      {/* ....... */}

      {isDevelopmentMode ? (
        <>
          <script
            async
            src={`https://www.googletagmanager.com/gtag/js?id=${GA_TRACKING_ID}`}
          />
          <script
            async
            id="gtag-init"
            dangerouslySetInnerHTML={{
              __html: `
        window.dataLayer = window.dataLayer || [];
        function gtag(){dataLayer.push(arguments);}
        gtag('js', new Date());
        gtag('config', '${GA_TRACKING_ID}', {
          page_path: window.location.pathname,
        });
      `,
            }}
          />
        </>
      ) : null}
    </html>
  );

}

@cliffordfajardo Would you like to open a new PR and submit an example with the changes you suggested?
We could merge it and close this one since @muthhukumar doesn't seem available.

@cliffordfajardo Would you like to open a new PR and submit an example with the changes you suggested? We could merge it and close this one since @muthhukumar doesn't seem available.

Sure - I can submit something later today in the evening (I'm in California PST time)

Closed in favor of #2449


FLAKE: client_channel_stress_test on Windows

https://source.cloud.google.com/results/invocations/a9dd68c7-6b94-454d-8c10-523208c5eb1f/targets/%2F%2Ftest%2Fcpp%2Fclient:client_channel_stress_test/log
Exception handler called, dumping information
code: c0000005 - flags: 0 - address: 00007FFC3AB24989
*** 00007FFC3AB24989 TpSetWait - 00007FFC3AB247C0
*** 00007FFC3AB1EFB7 RtlEnterCriticalSection - 00007FFC3AB1EE90
*** 00007FFC3AB1EED0 RtlEnterCriticalSection - 00007FFC3AB1EE90
*** 00007FF75828F3D7 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF75824E268 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF75824DF69 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF75827681D <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF758261A69 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF75817C502 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF7581BBCC7 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF7581832DC <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF758182F04 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF758182503 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF7581AF627 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF7581045BD <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF758103FDF <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF75812C74C <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF758129563 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF75824C98C <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF75820737D <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF75824076C <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF758240ACF <<no symbol>> - 00007FFC3AB1EE90
*** 00007FF758291AC9 <<no symbol>> - 00007FFC3AB1EE90
*** 00007FFC2D1A4034 BaseThreadInitThunk - 00007FFC2D1A4020
*** 00007FFC3AB73691 RtlUserThreadStart - 00007FFC3AB73670

Another example: https://source.cloud.google.com/results/invocations/cf8ca0e9-cb76-4fc7-9030-6896c1b6f190/targets/%2F%2Ftest%2Fcpp%2Fclient:client_channel_stress_test/log
Exception handler called, dumping information
code: c0000005 - flags: 0 - address: 00007FF8F31B4989
*** 00007FF8F31B4989 TpSetWait - 00007FF8F31B47C0
*** 00007FF8F31AEFB7 RtlEnterCriticalSection - 00007FF8F31AEE90
*** 00007FF8F31AEED0 RtlEnterCriticalSection - 00007FF8F31AEE90
*** 00007FF66126E367 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66122D1F8 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66122CEF9 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF6612557AD <<no symbol>> - 00007FF8F31AEE90
*** 00007FF6612409F9 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66115C472 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66119B017 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66116324C <<no symbol>> - 00007FF8F31AEE90
*** 00007FF661162E74 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF661162473 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66118EAD7 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF6610DE0B6 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF6610DC8D9 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF6611ACFB6 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF6611AD770 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF6611AE74F <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66114702D <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66114689F <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66122B91C <<no symbol>> - 00007FF8F31AEE90
*** 00007FF6611E644D <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66121F83C <<no symbol>> - 00007FF8F31AEE90
*** 00007FF66121FB9F <<no symbol>> - 00007FF8F31AEE90
*** 00007FF661270A59 <<no symbol>> - 00007FF8F31AEE90
*** 00007FF8E84E4034 BaseThreadInitThunk - 00007FF8E84E4020
*** 00007FF8F3203691 RtlUserThreadStart - 00007FF8F3203670

CC @jtattermusch

A few insights

the issue is not new, we recently enabled bazel C/C++ tests on bazel and this flake started showing up.

I can reproduce with manual RBE invocation with high likelihood:
~/bazel-0.29.1-windows-x86_64.exe --bazelrc=tools/remote_build/windows.bazelrc test //test/cpp/client:client_channel_stress_test --runs_per_test=5

I also ran the test locally and wasn't able to get things failing:
~/bazel-0.29.1-windows-x86_64.exe test //test/cpp/client:client_channel_stress_test --runs_per_test 10 --test_output=errors


Container ngrok-ingress-controller failed liveness probe

Kubernetes Version
Server Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.4-rc3+k3s1", GitCommit:"36645e7311e9bdbbf2adb79ecd8bd68556bc86f6", GitTreeState:"clean", BuildDate:"2023-07-25T22:00:15Z", GoVersion:"go1.20.6", Compiler:"gc", Platform:"linux/amd64"}
Helm Chart Version
0.10.0
Helm Chart configuration
Nothing, just credentials.apiKey and credentials.authtoken
What happened
Hey folks, and thank you for publishing this chart!
I followed steps described in docs on a fresh k3s cluster.
As the result the ngrok-ingress-controller pod didn't start at all.
Here is the output of kubectl describe pod ngrok-kubernetes-ingress-controller-manager-55b4b7f998-42tqr
Events:
  Type     Reason     Age                   From               Message
  ----     ------     ----                  ----               -------
  Normal   Scheduled  24m                   default-scheduler  Successfully assigned ngrok-ingress-controller/ngrok-kubernetes-ingress-controller-manager-55b4b7f998-42tqr to node3
  Normal   Pulled     23m (x2 over 24m)     kubelet            Container image "docker.io/ngrok/kubernetes-ingress-controller:0.8.0" already present on machine
  Normal   Created    23m (x2 over 24m)     kubelet            Created container ngrok-ingress-controller
  Normal   Killing    23m                   kubelet            Container ngrok-ingress-controller failed liveness probe, will be restarted
  Warning  Unhealthy  23m                   kubelet            Readiness probe failed: Get "http://<IP_ADDRESS>:8081/readyz": read tcp <IP_ADDRESS>:53944-><IP_ADDRESS>:8081: read: connection reset by peer
  Normal   Started    23m (x2 over 24m)     kubelet            Started container ngrok-ingress-controller
  Warning  Unhealthy  22m (x11 over 24m)    kubelet            Readiness probe failed: Get "http://<IP_ADDRESS>:8081/readyz": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
  Warning  Unhealthy  19m (x15 over 24m)    kubelet            Liveness probe failed: Get "http://<IP_ADDRESS>:8081/healthz": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
  Warning  Unhealthy  9m44s (x2 over 15m)   kubelet            (combined from similar events): Readiness probe failed: Get "http://<IP_ADDRESS>:8081/readyz": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
  Warning  BackOff    4m34s (x50 over 18m)  kubelet            Back-off restarting failed container ngrok-ingress-controller in pod ngrok-kubernetes-ingress-controller-manager-55b4b7f998-42tqr_ngrok-ingress-controller(2cd5f78f-940d-4453-8647-d43ba3692292)

What is the best way for troubleshooting the issue?
What you think should happen instead
No response
How to reproduce
No response

Hey there @Mehonoshin! Thanks for taking the time to make an issue to report this.
Would you be able to send us some additional debug information? Do you see anything suspicious in the pod logs for the ingress controller (kubectl logs ngrok-kubernetes-ingress-controller-manager-55b4b7f998-42tqr)? Have you tried doing a full teardown and starting from scratch again?

Sorry for bothering, the issue was not related to the chart itself, but misconfiguration of Flannel on my k3s cluster.


FTrace: optional function and events

This series adds the support to have a single set of trace events and profile functions to be used with kernel which supports only some of these events. At FTrace initialization time we check which tracepoints/functions are available in the target kernel and enable only those. Not available tracepoints/functions are logged and reported to the user.
The first patch fixes a small bug related to trace marker injection.

This is a refresh of the patchset which solves the issue discussed in the previous PR.

@setrofim tested on my JUNO and that solution should be more robust than the previous one.
Lemme know if other updates are required.


[SPARK-38732][SQL][TESTS] Test the error class: INCOMPARABLE_PIVOT_COLUMN

What changes were proposed in this pull request?

I add a test case for the error class INCOMPARABLE_PIVOT_COLUMN in the QueryExecutionErrorsSuite.
Why are the changes needed?

To improve test coverage, and record expected error messages in tests.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

By existing tests.

@MaxGekk Thanks for your review.  I changed done.


CNF-14652: Update API group for Cluster Request&Template

Description:

update the API group to o2ims.provisioning.oran.org for ClusterRequest and ClusterTemplate
move ClusterRequest and ClusterTemplate APIs to their own directory and go module
move the Inventory API to its own directory and go module

/hold
Still going through some end to end tests

/unhold

/hold
End to end test with latest changes.

/unhold
Performed end to end SNO install + inventory run of the metadata and deployment manager server.

/lgtm
/approve

I missed this before it was merged, but it is quite possible that the ClusterTemplate will not become part of the O-RAN spec, so it may not belong under the o2ims.provisioning.oran.org API. I guess we can change that later if it turns out to be the case?

I missed this before it was merged, but it is quite possible that the ClusterTemplate will not become part of the O-RAN spec, so it may not belong under the o2ims.provisioning.oran.org API. I guess we can change that later if it turns out to be the case?

@bartwensley , I put them together for now because they are both part of provisioning. We can definitely update it at any time. As long as it's not part of the standard though, it can belong to any group we choose, isn't it?


Fixing IE9 issue.

The following use-case was causing the regression error bug:

Using 'dist/js/swiper.jquery.min.js' on IE9
Setting 'loop' property on instantiation of Swiper to 'true'
Using arrow as navigation (swiping was working fine)

If this use-case is happening (requirement on my project) 's.animating' within the 'core.js' will be re-assigned  to 'true' on each click on the arrow/s. Resulting in only one slide change (initial click).
Solution:

added the lte (lower then or equal) IE9 conditional check in 'src/js/swiper-proto.js' > property 'browser' > property 'lteIE9' (detecting browser by feature instead of version or UA etc.)
wrapping line 1907 in 'src/js/core.js' method 'slideTo' into the IF statement (if (!s.browser.lteIE9) s.animating = true;). Idea is not to do the re-assignment on 's.animating' if code is being executed on IE9 or lower version of IE.

Took me some time to get into your nice code and debug our archenemy aka IE...

Looks good, thanks


Can't use contentType pdf_base64 with Printnode

Description
Currently I can see no way of using the ContentType pdf_base64 for Printnode via the content() method.
public function content($content): self
    {
        $this->job->content = base64_encode($content);
        $this->job->contentType = ContentType::RAW_BASE64;

        return $this;
    }

RAW_BASE64 is hardcoded in the content() method.
For CUPS there is a additional $contentType parameter for the content() method.
public function content($content, string $contentType = ContentType::PDF): self

It would be great if this parameter can be added to the printnode drivers PrintTask class, too.
I can add a Pull Request, if mandatory.
Kind Regards
Christian

@krizzdev I've merged a PR just now that adds support for this. Sorry for not getting to this until now.


only size-1  arrays can be converted to python scalars

您好，我用您的代码运行出现这种情况，请问如何解决？

this is the problem of torch version

this is the problem of torch version


Comment aligning

Similar to "Align equal sign" also provide "Align comments", especially for end-of-line-comments.

Closed with https://github.com/isg-stuttgart/vscode-isg-cnc/pull/146


[models] Module design suggestion

Here is a suggestion for the pyrovision.models module's organization:

utils.py: shared utility functions
XYZ.py: classification architecture definion with pretrained weights
detection: folder with the definition of object detection models with pretrained weights
segmentation: folder with the definition of segmentation models with pretrained weights

All performances of pretrained versions should be reported either in the documentation, the readme or a readme in the references folder.

I don't understand your point here, we already have this architecture. We only have classification models. You want to add empty detection and segmentation folders?

It was to plan the incoming additions to the library

ok sure let's do that

My bad, I haven't cleaned up the classification models! I'll do this by the end of the weekend

Closed by #138 & #150


Change the check for cfg-able compilers

This allows cfg to work even for cross gcc (with names like foo-bar-gcc).

I'm adding my name in the contributors :)

Thanks! :)

Thanks for the quick review !


Keep texlive 2020 (vs 2021)

and avoid possible table overflow errors

[ ] Closes #xxxx .
[ ] I am familiar with the contributing guidelines.
[ ] Tests added.
[ ] Updates entries to docs/source/api.rst for API changes.
[ ] Adds descriptions to appropriate "what's new" file in docs/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[ ] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[ ] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

Unfortunately, first test uses image already pushed to quay with texlive 2021. Testing locally works (and latest on quay pulled locally fails as it does here)


FIRECERT:2103-Tag 0.13.0

Description
Enter a detailed description of the PR here, including the nature and purpose of the changes.
JIRA Ticket
Include a link to the JIRA Ticket linked to this PR.
Issue-0000
Additional Info (Test steps / Steps to reproduce etc.)
Details on how to test the feature

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: parag-pv:white_check_mark: anjalimukundan:white_check_mark: Abhishk123:white_check_mark: jmeier204:x: Jayaramaraju, AbhishekJayaramaraju, Abhishek seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Auth libraries dependency problem fix

https://github.com/GoogleCloudPlatform/google-cloud-java/issues/2050

LGTM

Codecov Report

Merging #310 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #310      +/-   ##
============================================
- Coverage     69.98%   69.93%   -0.06%
  Complexity      352      352
============================================
  Files            72       72
  Lines          1906     1906
  Branches        155      155
============================================
- Hits           1334     1333       -1
- Misses          505      506       +1
  Partials         67       67

Impacted Files
Coverage Δ
Complexity Δ

...java/com/google/api/gax/grpc/RetryingCallable.java
85.18% <0%> (-1.86%)
6% <0%> (ø)

Continue to review full report at Codecov.

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

LGTM
io.grpc:grpc-auth:1.2.0 is also depending on 0.4.0. We may need to ask the gRPC team to update as well.


Fix path error on initial run

Fixes #312
This isn't idiomatic but it'll do for the time being

@Kibouo


Rendering directly on ubuntu server without setting Application.UseSystemConsole = true is broken

I created a fresh ubuntu20.04.3 (server) server VM, installed the dotnet sdk, checked out gui.cs, ran dotnet run --project UICatalog and it looked like this:

In my app, I notice severe drawing issues as well (dead "pixels", skewed graph axes, ...). Am I doing something fundamentally wrong?

I think this is not an issue. If no xterm is installed so ncurses will be using the limited basic colors. If the Application.UseSystemConsole = true, then the dotnet colors will be used with more options.

Are you certain that this is just a color scheme issue? I can see certain elements in the list if I manage to navigate to them, but their peers have the same color in my powershell, so I'd expect to see them too when I navigate to them.
Setting Application.UseSystemConsole = true makes the CPU usage go to 100%, are there ways around that? And can I set it easily for the entire catalog to test it?

Setting Application.UseSystemConsole = true makes the CPU usage go to 100%, are there ways around that?

With a Thread.Sleep (10) inside the while (true) of the WaitWinChange () method will improve the performance.

And can I set it easily for the entire catalog to test it?

dotnet run --project UICatalog -usc

With a Thread.Sleep (10) inside the while (true) of the WaitWinChange () method will improve the performance.

That sounds great, could you merge your branch with that upstream?

dotnet run --project UICatalog -usc

With System.Console things work as expected indeed, so I guess the next thing would be to set a different bg color?

That sounds great, could you merge your branch with that upstream?

Had already submitted here #1504.

With System.Console things work as expected indeed, so I guess the next thing would be to set a different bg color?

Are you using the current main branch? If so, maybe the ubuntu server don't have all the resources to run full color support for ncurses (CursesDriver). @tznind can you give your opinion about this, please?

What are you using to access the server? are you directly logging in or using ssh? what does $TERM show (i.e. echo $TERM)? Typically you can get odd colours or not showing colors if you are using a different terminal protocol.  For example to get putty to work you need to set the app to use xterm-256color (https://github.com/migueldeicaza/gui.cs/discussions/1455)

I believe this is fixed. Closing.


Calendar Days not Responding to Locale Changes?

Hello,
First, thank you for working on a Calendar add-on for Vaadin. The framework most certainly needs a calendar.
I cannot figure out how to change the order of the days in the Calendar. The default order is:
Mon, Tue, Wed, Thu, Fri, Sat, Sun
but this does not match the typical US calendar. I want the days to be in the following order:
Sun, Mon, Tue, Wed, Thu, Fri, Sat
Code example:
calendar = new Calendar<>(eventProvider);
calendar.setLocale(Locale.ENGLISH);
calendar.markAsDirty();
Thank you for your time!

Fixed in next release.

not completly fixed :-/

works in a range of a full week.


Add --enable-proposed-api flag to cli.ts to fix issue with extensions failing

Fixes #1744
Updates #1981

Is this PR still valid?

cc @code-asher

We need to make some modifications according to https://github.com/cdr/code-server/issues/1744#issuecomment-642228856

@code-asher thoughts on forking this and making the needed changes?

I think that's a good idea, I'll tackle it this week.

I branched off this here: https://github.com/cdr/code-server/pull/2002


ERROR: $cordovaOauth.weibo not working

Hi @nraboy
I have used code below:
$cordovaOauth.weibo('wxbdca28a6636fcaf5', 'c733009b7ac63e490555f56d6da94f99', ['snsapi_userinfo'], {language:"en",forcelogin: true}).then(function(result) {
        // todo
      }, function(error) {
          // error

      });

And see error message at console log of xcode. See attachment pls

Can you help me resolve it?

That error doesn't look related to this library.  Can you provide logs that prove that it is related?


”Vama” is listed in the drop-down menu

Description:
"Vama" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Vama" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Vama, SATU MARE".


Issues with evaluation

Thank you for your amazing work!
I have followed INSTALL.md to setup the environment . When I run ROUTES=assets/routes_lav_valid.xml ./leaderboard/scripts/run_evaluation.sh , the following error occurs.
========= Preparing RouteScenario_0 (repetition 0) =========

Setting up the agent

Could not set up the required agent:

invalid load key, 'v'.

Traceback (most recent call last):
File "/home/abc/carla1/LAV-main/leaderboard/leaderboard/leaderboard_evaluator.py", line 262, in _load_and_run_scenario
self.agent_instance = getattr(self.module_agent, agent_class_name)(args.agent_config)
File "/home/abc/carla1/LAV-main/leaderboard/leaderboard/autoagents/autonomous_agent.py", line 45, in init
self.setup(path_to_conf_file)
File "/home/abc/carla1/LAV-main/team_code/lav_agent.py", line 89, in setup
bev_planner.load_state_dict(torch.load(self.bev_model_dir))
File "/home/abc/anaconda3/envs/LAV-env/lib/python3.7/site-packages/torch/serialization.py", line 713, in load
return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
File "/home/abc/anaconda3/envs/LAV-env/lib/python3.7/site-packages/torch/serialization.py", line 920, in _legacy_load
magic_number = pickle_module.load(f, **pickle_load_args)
_pickle.UnpicklingError: invalid load key, 'v'.

Registering the route statistics

ps:
import torch
torch.version
'1.11.0'
any help will be appreciated!

Thank you for your amazing work!
I have followed INSTALL.md to setup the environment . When I run ROUTES=assets/routes_lav_valid.xml ./leaderboard/scripts/run_evaluation.sh , the following error occurs.
========= Preparing RouteScenario_0 (repetition 0) =========

Setting up the agent

Could not set up the required agent:

invalid load key, 'v'.

Traceback (most recent call last): File "/home/abc/carla1/LAV-main/leaderboard/leaderboard/leaderboard_evaluator.py", line 262, in _load_and_run_scenario self.agent_instance = getattr(self.module_agent, agent_class_name)(args.agent_config) File "/home/abc/carla1/LAV-main/leaderboard/leaderboard/autoagents/autonomous_agent.py", line 45, in init self.setup(path_to_conf_file) File "/home/abc/carla1/LAV-main/team_code/lav_agent.py", line 89, in setup bev_planner.load_state_dict(torch.load(self.bev_model_dir)) File "/home/abc/anaconda3/envs/LAV-env/lib/python3.7/site-packages/torch/serialization.py", line 713, in load return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args) File "/home/abc/anaconda3/envs/LAV-env/lib/python3.7/site-packages/torch/serialization.py", line 920, in _legacy_load magic_number = pickle_module.load(f, **pickle_load_args) _pickle.UnpicklingError: invalid load key, 'v'.

Registering the route statistics

ps: import torch torch.version '1.11.0'
any help will be appreciated!

I think git-lfs is not installed in my environment. after install it, I just try again.But it doesn't work.
So,I downloaded bev_160.th bra_10.th`` lidar_15.th seg_1.th uniplanner_15.th manually,and replaced the contents of the original weight file .
The above error did not appear.But there are new errors.
Traceback (most recent call last):
File "/home/abc/carla1/LAV-main/leaderboard/leaderboard/scenarios/scenario_manager.py", line 152, in _tick_scenario
ego_action = self._agent()
File "/home/abc/carla1/LAV-main/leaderboard/leaderboard/autoagents/agent_wrapper.py", line 75, in call
return self._agent()
File "/home/abc/carla1/LAV-main/leaderboard/leaderboard/autoagents/autonomous_agent.py", line 115, in call
control = self.run_step(input_data, timestamp)
File "/home/abc/anaconda3/envs/LAV-env/lib/python3.7/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/home/yko/carla1/LAV-main/team_code/lav_agent.py", line 165, in run_step
self.stop_counter += 1
AttributeError: 'LAVAgent' object has no attribute 'stop_counter'
I checked the code and found that there was no such attribute.
Is there anything else I haven't noticed?any help will be appreciated!

I think git-lfs is not installed in my environment. after install it, I just try again.But it doesn't work.
So,I downloaded bev_160.th bra_10.th`` lidar_15.th seg_1.th uniplanner_15.th manually,and replaced the contents of the original weight file .

Yes I also noticed this one, you should download the files through GitHub by downloading button.

AttributeError: 'LAVAgent' object has no attribute 'stop_counter'

Add initial for self.stop_counter is fine. please see the pull request: https://github.com/dotchen/LAV/pull/4

I think git-lfs is not installed in my environment. after install it, I just try again.But it doesn't work.
So,I downloaded bev_160.th bra_10.th`` lidar_15.th seg_1.th uniplanner_15.th manually,and replaced the contents of the original weight file .

Yes I also noticed this one, you should download the files through GitHub by downloading button.

AttributeError: 'LAVAgent' object has no attribute 'stop_counter'

Add initial for self.stop_counter is fine. please see the pull request: #4
After all things are prepared, it should be fine to run the evaluation. For example here:

Thank you for your help!
I can run now. I have another problem.
When I run, the image of the simulator is no longer smooth. I try to use print() in the _tick_scenario(self, timestamp) function in scenario_manager.py, and the output speed slows down. Is this normal? It hasn't appeared before when running WorldOnRails.

Thank you for your help! I can run now. I have another problem. When I run, the image of the simulator is no longer smooth. I try to use print() in the _tick_scenario(self, timestamp) function in scenario_manager.py, and the output speed slows down. Is this normal? It hasn't appeared before when running WorldOnRails.

I think you should check the CPU and GPU utility when you run this to see whether your computer can run it at a high rate. I believe this model may be larger than world on rail.
check CPU: htop
check GPU: watch nvidia-smi

Thank you for your help! I can run now. I have another problem. When I run, the image of the simulator is no longer smooth. I try to use print() in the _tick_scenario(self, timestamp) function in scenario_manager.py, and the output speed slows down. Is this normal? It hasn't appeared before when running WorldOnRails.

I think you should check the CPU and GPU utility when you run this to see whether your computer can run it at a high rate. I believe this model may be larger than world on rail.
check CPU: htop
check GPU: watch nvidia-smi

Hi,I checked the CPU and GPU.It looks so strange...

I think it shouldn't take up so much CPU resources.
Is there something wrong? Or just this model is larger than world on rail,my machine can't provide better support...
PS:
CPU:Intel® Xeon(R) CPU E5-2630 v4 @ 2.20GHz × 40
GPU:NVIDIA GeForce RTX 2080 Ti

CPU:Intel® Xeon(R) CPU E5-2630 v4 @ 2.20GHz × 40 GPU:NVIDIA GeForce RTX 2080 Ti

I believe this CPU is a server CPU and normally servers' CPU is not so strong for processing things before data goes to GPU, maybe you can try it at your computer. The server normally is for training and using GPU mainly.
Here is my personal computer running for evaluation:

And the computer setting is:
Processor: Intel® Core™ i7-9750H CPU @ 2.60GHz × 12
GPU: GeForce GTX 1660 Ti/PCIe/SSE2


Feature Request: Exit Copy Mode

This exits copy mode after yanking current line.
I think this makes sense because this is normally used in normal mode and not copy-mode.

This exits copy mode after yanking current line.

I can't reproduce this. Currently, the latest version of the plugin exits copy mode after yanking current line.

Ok thanks.


Automated cherry pick of #9873

Cherry pick of #9873 on release-6.5.
/cc  @BenCookie95
NONE

@BenCookie95 I can't seem to be able to add the Automerge label to this. Are you able to help merge this?


superagent doesn't set JSON Accept header

It needs to be set explicitly using .set('Accept', 'application/json').

What does .type(options.type || 'json') in https://github.com/feathersjs/feathers-rest/blob/master/src/client/superagent.js#L6 do then?

It only sets the content type and formats it as json.


hide pages - behaviour changed in 1.0.1

Previously, even if a page was hidden, I could access it by giving the exact path.
I used it in a link to access pages I didn't want to show in the navigation bar. After upgrading to 1.0.1, when clicking those links, I was redirected to the default page.
Is this the expected behavior? Is there a way to bypass it?
Thanks

Hi @tombeteende, can you share a little more about how you are "giving the exact path"? If you mean putting it in the url, that should still work even if you've hidden a page using hide_page(). But, if you've hidden a page by not including it in pages.toml, then it will not be accessible.


[csv2img]: Customise colours and size of PDF

What is your new feature?
Hi, thanks for this package. I have tried it out with my app as it currently has a CSV export feature and also wanted to include the ability to export to PDF. This seems to do the job but have one request. Is it possible to customise the colours of the columns? I rather not have any colours and just a black font.
I am also looking for a way to customise the page size such as A4, letter rather than the size being just the table size.
It will be great if you can add these features. Thank you 🙂

you can try it on v1.8.0!
https://github.com/fummicc1/csv2img/releases/tag/1.8.0

Hi, I released v1.8.1 that fixes above issue. please check it from your side as well when you have spare time.

Only issue now is the last column looking a bit weird. Other than that, it's all good 👍


Optimize stable parser

Right now the parser only takes the optimized SIMD functions when the simd feature is set, which requires nightly.
If it's not, it falls back to non-optimized functions.
This PR organizes the optimized util functions into a module and adds stable-alternatives for the nightly ones.

bors r+


yersinia: add -fcommon workaround

Workaround build failure on -fno-common toolchains like upstream
gcc-10. Otherwise build fails as:
ld: ncurses-interface.o:src/ncurses-interface.h:83: multiple definition of
  `pointer'; ncurses-gui.o:src/ncurses-interface.h:83: first defined here

Description of changes

Things done

Built on platform(s)

[x] 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/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

LGTM, but I don't have time to test the binary in the near future since I have exams.
Nice that you noticed they already have it upstream :)

Failing on ARM

Fails with a somewhat obscure error:
UNAME_MACHINE = aarch64
UNAME_RELEASE = 5.15.43
UNAME_SYSTEM  = Linux
UNAME_VERSION = #1-NixOS SMP Wed May 25 12:42:07 UTC 2022
configure: error: cannot guess build type; you must specify one

I'll have a look. Checking if it's a flake meanwhile:
@ofborg build yersinia

Not a flake and not a regression: https://hydra.nixos.org/eval/1765872?filter=yersinia&compare=1765786&full=. I suspect it's a pre-aarch64 configure. Will try to add autoconf hook.

Looks like it's the other way around: ./configure is geenrated in .nix file, but some of .dev depends are missing (like glib.dev):
configure.in:828: error: possibly undefined macro: AM_GLIB_GNU_GETTEXT
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
Running ./configure ...
checking build system type... ./config.guess: unable to guess system type

Added glib as a build depend. I wonder if gtk and others are needed unconditionally as well.

@ofborg build yersinia

Still no luck for aarch64 detection. before giving up on it trying a hack of using old autoconf-2.69.
@ofborg build yersinia

No change.

Dropped all the unrelated changes and left -fcommon workaround alone. I suggest fixing aarch64 separately.

OK. Filling a bug report should suffice.


Old commands in refreshed page

Hi,
How come you are using old (from SkypeonlineConnector module) commands in MicrosoftTeams module as commands do not exist? This page is not up-to-date. SkypeOnlineConnector module is not available and commands you are offering in this article simply cannot work.

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

ID: bb2a9717-6565-8443-8833-f7fdce562781
Version Independent ID: db88c4d6-5864-a481-86cf-465feb452d75
Content: Enable users for Direct Routing - Microsoft Teams
Content Source: Teams/direct-routing-enable-users.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@dusansk thanks for the feedback! Is it possible for to you copy and paste the commands you are referring to?

Hi Jens,My bad... updated the MS Teams module and of course get-csOnlineUser is available. I was barking up the wrong tree.regards,DusanOn 26 Jan 2022, at 15:24, Jens Trier Rasmussen @.> @. thanks for the feedback! Is it possible for to you copy and paste the commands you are referring to?—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you were mentioned.Message ID: @.***>

Hi @dusansk no worries! And please keep the feedback coming.


Unable to create snapshot.

Hi!
Every time I try to create a snapshot, I get this error:
"CLIENT ERROR: TUINSRemoteViewController does not override -viewServiceDidTerminateWithError: and thus cannot react to catastrophic errors beyond logging them"
Is it possible that this error appears, since my vm is on a SD Card?

Hey @JanR27, I'm sorry to hear that. The message sounds unfamiliar though. Could you please provide a precise description of the state of the app you're in before the error occurs, the action you're performing that is leading to the error message, and the following, undesired behaviour of the app? Also include your system architecture as reported by the arch command, the MacOS version you're on as reported by sw_vers, the XCode-Version you've been using as reported by xcodebuild -version as well as any other information that seems relevant to you.

I don't know if this is 100% related, but:
Brand new M4 Pro with latest xcode and UTM-Snapshot-Manager code.
% xcodebuild -version
Xcode 16.1
Build version 16B40

https://github.com/user-attachments/assets/1a2365d7-0b8e-4aad-9b18-2891d59b1cf7

Just a guess, maybe my issue is because I am using emulated NVMe?

pv@Pauls-MacBook-Pro-M4-Pro Virtual Machines.localized % cd Windows\ 10\ x64.utm
pv@Pauls-MacBook-Pro-M4-Pro Windows 10 x64.utm % ls -la
total 368
drwxr-xr-x@  5 pv  staff     160 Nov 14 06:08 .
drwxr-xr-x  14 pv  staff     448 Nov 14 05:56 ..
drwxr-xr-x@  5 pv  staff     160 Nov 14 06:08 Data
-rw-r--r--@  1 pv  staff    3497 Nov 14 06:08 config.plist
-rw-r--r--@  1 pv  staff  182349 Nov 14 16:39 screenshot.png
pv@Pauls-MacBook-Pro-M4-Pro Windows 10 x64.utm % cd Data
pv@Pauls-MacBook-Pro-M4-Pro Data % ls -la
total 22073184
drwxr-xr-x@ 5 pv  staff          160 Nov 14 06:08 .
drwxr-xr-x@ 5 pv  staff          160 Nov 14 06:08 ..
-rw-r--r--@ 1 pv  staff<PHONE_NUMBER>2 Nov 14 16:36 E4E29B81-6630-4629-9076-BB27DB46869C.qcow2
-rw-r--r--@ 1 pv  staff      3653632 Mar  2  2021 QEMU_EFI-X64-iPad-UTM-logo.fd
-rw-r--r--@ 1 pv  staff       393216 Mar 12  2021 disk-0.qcow2
pv@Pauls-MacBook-Pro-M4-Pro Data % qemu-img snapshot E4E29B81-6630-4629-9076-BB27DB46869C.qcow2 -c Fresh\ Install
pv@Pauls-MacBook-Pro-M4-Pro Data % qemu-img snapshot E4E29B81-6630-4629-9076-BB27DB46869C.qcow2 -l
Snapshot list:
ID      TAG               VM_SIZE                DATE        VM_CLOCK     ICOUNT
1       Fresh Install         0 B 2024-11-14 16:43:46  0000:00:00.000          0
pv@Pauls-MacBook-Pro-M4-Pro Data %

Hey Paul! It looks like you‘ve reported to the wrong repo. This is just a POC for a snapshot manager for UTM and no official part of the UTM project. To report a bug there, please refer to the [official UTM project page here on GitHub] (https://github.com/utmapp/UTM). I’d recommend including a description that helps the devs reproducing the bug.

I wasn't reporting a problem with UTM.
I was confirming a similar problem to this Issue's problem with UTM-Snapshot-Manager not taking snapshots.

Sorry for the misunderstanding @paulpv. Could you provide a detailed description of what actions you're taking, what you expect to happen and what happens instead, as well as any relevant circumstantial info (e.g. configuration of the VMs drives, URLs of resources used, a copy of the VM if it doesn't contain sensitive information)? Please understand that I'm doing this project in my free time and I'm lacking the resources to diagnose by watching videos, guessing or experimenting.
In your video, I can see you using the Snapshot Manager, and it looks like you're expecting a drive for snapshotting to show up, yet there's none. Is that the issue you want to report? I'm asking this trying to understand if and how your issue is related to the problem reported in the ticket which describes a specific error message, and if and how your problem could be fixed.

You can create a snapshot with UTM Snapshot Manager while on Sequoia if you install qemu with brew first:
brew install qemu
I just tested this on an M4 MacBook


pydoc3 no worky

timsehn$ pydoc3 doltpy | cat
Help on package doltpy:

NAME
    doltpy

PACKAGE CONTENTS
    core (package)
    etl (package)

FILE
    /usr/local/lib/python3.7/site-packages/doltpy/__init__.py

Same with pydoc.

(doltpy-dev) [oscarbatori:doltpy(oscarbatori/fix-ci)] >> pydoc doltpy.core.dolt

This actually produces abundant output, so I am not sure why the module level is not picking up the functions that are explicitly surfaced in doltpy.core.__init__.py.
Agree it's an issue, will investigate.


Interoperability with Rust game engines?

Any plans for interop with existing engines? I.e. Piston, ggex and Amethyst. Personally I'm most interested in seeing an example with Amethyst.
Some minimal interaction would be good for the demos, both for demonstration purposes as well as in testing. If I could use WASD to move the green unit then I could play around more with the different behaviors.

When I started this project my ultimate goal was to develop an ai crate for rust game developers. At the moment there are lots of missing pieces. Actually I'm not very satisfied with the internals of the library as well. If you are interested in developing please feel free to contribute.

Can’t help with the code I’m afraid, that’s out of my comfort zone. Community is my expertise, so I’ll just share this idea:
Stop by the respective github repos of the projects I mentioned and create an issue for something like “Steering behavior example”. Let them know you’d be interested in making this library work with their engine, and you’ll happily answer any questions for anyone who wants to have a go at developing an example for their engine. If you get them on board with that I’m sure you’ll end up getting a lot of capable eyes on the code for 2nd opinions ;)

Oh and your library should be added to AreWeGameYet:
https://github.com/doppioslash/arewegameyet/issues

We’re gonna use these steering behaviors in a demo game for the Amethyst engine:
https://community.amethyst-engine.org/t/demo-game-evolution-island-mvp/487?u=erlend_sh

Really flattered to hear about this. Although this work is far from complete,  I would like to contribute anything you need. Especially I would like to see how you integrate behaviors with ECS.


My code can't parse the solution you generated

Have you tried using HashCode2014.jl to read it again after writing it to file?

issued solved


Shipyard cleanup

Hi guys, I am having a bit of difficulty trying to uninstall shipyard from my ubuntu machine.
So, I installed automatically, running on port 8080. Of course, this port is used by default by a lot of applications so I tried to free it somehow afterwards. I am amazed I could not do it.
It is occupied by the controller container.
So first I started a new controller container, on port 8180. I could access the new shipyard.
Then next I tried to remove all container. Successfully. But the 8080 is still running, showing the shipyard interface.
Then I tried sudo curl -sSL https://shipyard-project.com/deploy | ACTION="remove" bash -s
It said it removed all, but both 8180 and 8080 still run, showing shipyard interface.
I restarted, still are there.
I manually removed all the containers related to shipyard. Now docker ps shows no container. The interfaces still run on both ports.
I ran again the remove command, somehow says all the time with sudo, or without, that remove was successfully, but unfortunately nothing happens, shipyard is still there, on both ports.
I do not have any new ideas, I even tried some cleanup old commands, which don't work anymore.
Am I missing something?

Can you report what docker ps gives?
On Apr 24, 2016 03:42, "Alexandru Ardelean"<EMAIL_ADDRESS>wrote:

Hi guys, I am having a bit of difficulty trying to uninstall shipyard from
my ubuntu machine.
So, I installed automatically, running on port 8080. Of course, this port
is used by default by a lot of applications so I tried to free it somehow
afterwards. I am amazed I could not do it.
It is occupied by the controller container.
So first I started a new controller container, on port 8180. I could
access the new shipyard.
Then next I tried to remove all container. Successfully. But the 8080 is
still running, showing the shipyard interface.
Then I tried sudo curl -sSL https://shipyard-project.com/deploy |
ACTION="remove" bash -s
It said it removed all, but both 8180 and 8080 still run, showing shipyard
interface.
I restarted, still are there.
I manually removed all the containers related to shipyard. Now docker ps
shows no container. The interfaces still run on both ports.
I ran again the remove command, somehow says all the time with sudo, or
without, that remove was successfully, but unfortunately nothing happens,
shipyard is still there, on both ports.
I do not have any new ideas, I even tried some cleanup old commands, which
don't work anymore.
Am I missing something?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/802

docker ps gives nothing after removal. No container running.

wait, my bad, I think cache removal of the browser didn't work well. I'm gonna close this one.

cool thanks!
On Mon, Apr 25, 2016 at 2:17 PM, Alexandru Ardelean <
<EMAIL_ADDRESS>wrote:

Closed #802 https://github.com/shipyard/shipyard/issues/802.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/802#event-640416385


When will be the next update?

it's been a while since the last update in Dec 2023. Any update schedule?

It seems to be that not many people were assigned to this project. And probably they are focusing more on other things like media pipe and gen AI related stuff. Don't blame the people but the company behind them. Seems to be that this project is not a priority for them which in essence is not a bad thing. Let's hope it gets a little bit more attention in the future, since it's quite an important package to have in flutter. Nevertheless you can always opt for a native implementation if the package lacks features you need.


Dont be able to load other APIs

If you use a other Paied api the NET wont work. It dosnt start attacks even if the API is runnung. Is there a fix??

nah you can fix code but ;-; uhhhh i gonna do project ransomware if done i gonna update hex_cnc


Paid account not being flagged as paid [BUG]

Chrome 108.0.5359.125 (Official Build) (64-bit)
Windows 10
Extension v 1.5
Twitter location: US
Twitter account:  https://twitter.com/JamesBradleyCA
I've run into a Twitter account (above) that does not display the paid indicator even though it is a paid account. I do see lots of paid accounts so it seems to be working properly otherwise. However, I have no idea how many other accounts this is happening with as I don't check them all.
I thought you might want to see what is preventing this one from being flagged as it may not be the only one.

I'm seeing the same for some accounts that are paid.  Here's another Twitter account which appears as Verified rather than Paid:  https://twitter.com/SahilBloom
I don't see any difference between this account's verification status message and another that gets labelled "Paid":

It's because their accounts are both
isBlueVerified: true

and
isVerified: true

In other words, they had a real Verified badge before they bought Twitter Blue checkmark.
@david-rahrer

@mirzmaster

This chrome extension gives legacy Verified status precedence over blue verified. So this is working as expected. We thought about giving those types of accounts a third status (legacy/blue verified hybrid) but it's technically challenging to do correctly in Twitter Spaces.
Closing as WONTFIX. Maybe we could revisit the "legacy/blue verified hybrid" status some time in the future as a separate feature though.


[ESP32] Guru Meditation Error: (LoadProhibited) on reconnect

Hello folks,
i have a little Problem at the moment.
After some time, the connection gets lost. It looks like the wificlient has a disconnect:
[D][WiFiClient.cpp:509] connected(): Disconnected: RES: 0, ERR: 128
[W][Tasks.cpp:181] Task_ConnWatcher(): mqtt was down: -1 - connect

No problem so far ...
As you can see on this two lines, my code does detect this connection drop with the .connected() method.
After that i try a reconnect with .connect() method and get this exception immediately (every time).
Guru Meditation Error: Core  1 panic'ed (LoadProhibited). Exception was unhandled.
Core 1 register dump:
PC      : 0x4000c63c  PS      : 0x00060330  A0      : 0x800d5d39  A1      : 0x3ffb5920
A2      : 0x00000000  A3      : 0x00000100  A4      : 0x3ffb5958  A5      : 0x3ffb595e
A6      : 0x00000004  A7      : 0x00000000  A8      : 0x00000000  A9      : 0x3ffb58f0
A10     : 0x00000001  A11     : 0x3ffc12ab  A12     : 0x0000075b  A13     : 0xffffffff
A14     : 0x00060320  A15     : 0x00000000  SAR     : 0x00000006  EXCCAUSE: 0x0000001c
EXCVADDR: 0x00000000  LBEG    : 0x4000c2e0  LEND    : 0x4000c2f6  LCOUNT  : 0xffffffff

Backtrace: 0x4000c63c:0x3ffb5920 0x400d5d36:0x3ffb5940 0x400d5ea2:0x3ffb5980 0x400d31ce:0x3ffb59b0 0x400891ed:0x3ffb5ad0
  #0  0x4000c63c:0x3ffb5920 in ?? ??:0
  #1  0x400d5d36:0x3ffb5940 in PubSubClient::connect(char const*, char const*, char const*, char const*, unsigned char, bool, char const*, bool) at .pio/libdeps/esp32dev/PubSubClient/src/PubSubClient.cpp:734
  #2  0x400d5ea2:0x3ffb5980 in PubSubClient::connect(char const*) at .pio/libdeps/esp32dev/PubSubClient/src/PubSubClient.cpp:734
  #3  0x400d31ce:0x3ffb59b0 in Tasks::Task_ConnWatcher(void*) at src/Tasks.cpp:183
  #4  0x400891ed:0x3ffb5ad0 in vPortTaskWrapper at /home/runner/work/esp32-arduino-lib-builder/esp32-arduino-lib-builder/esp-idf/components/freertos/port.c:355 (discriminator 1)

The value of EXCVADDR: 0x00000000 tells me, that there maybe is a kind of nullptr exception here.
Does anyone have a idea whats going wrong?
BTW: the first connection after start goes trough the same code block and that works every time (check connection and connect).
Regards,
Basti

Here is the connect/reconnect snippet:
NOTE: The mutex is given a few lines later ... after subscribe is done.
So no part of my code does try to send something at the moment we see the was down log message.
if (!mqtt.connected())  {
                xSemaphoreTake(mutex_mqtt, portMAX_DELAY);
                log_w("mqtt was down: %d - connect", mqtt.state());
                uint8_t recon_ctr = 0;
                while (!mqtt.connect(test->mqtt_device_name)) // here exception hits
                {
                    recon_ctr++;
                    log_e("mqtt connection error: %d", mqtt.state());
                    log_e("connection try %d failed!", recon_ctr);
                    if (recon_ctr >= 5)
                    {
                        log_e("mqtt connection failed! restart...");
                        esp_restart();
                        break;
                    }
                    vTaskDelay(pdMS_TO_TICKS(5000));
                }
// ...
// subscribe and mutex GIVE
// ...
}

As you can see here - the disconnect is at totally random times:

(data from millis() over mqtt - so there was real restart of the ESP (millis() reset))

same problem over here. but for me it happens immediately with the first connect

same problem over here. but for me it happens immediately with the first connect

Just for information:
I have switched to https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/protocols/mqtt.html
This implementation works like a charme - stable for weeks with no disconnect or other problems..

Just for information:
I have switched to https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/protocols/mqtt.html
This implementation works like a charme - stable for weeks with no disconnect or other problems..

Dito 😅😂

Am 20.01.2021 um 11:24 schrieb Bastian Schroll<EMAIL_ADDRESS>
Just for information:
I have switched to https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/protocols/mqtt.html
This implementation works like a charme - stable for weeks with no disconnect or other problems..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Dito 😅😂

Am 20.01.2021 um 11:24 schrieb Bastian Schroll<EMAIL_ADDRESS>
Just for information:
I have switched to https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/protocols/mqtt.html
This implementation works like a charme - stable for weeks with no disconnect or other problems..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

@jappyjan I had the same problem when trying to connect the first time, but I found a detail in one of the tutorials I read:
I missed the constructor that needs a wifi client... now it works =)
WiFiClient espClient;
PubSubClient mqttClient(espClient);

I still getting this error, is there someone who understood why this happen?
To me happens when sending fast messages to the device.

You can use delay() function

I still getting this error, is there someone who understood why this happen? It happens when sending fast messages to the device.

@sblantipodi
Look my reply below

Based on the suggestions here I also switched to esp_mqtt. Fixed all my issues. While PubSubClient is a wonderful generic mqtt lib, I think the esp_mqtt lib is better suited for the esp32.

it's sad but last commit on this repo was made in may 2020.
no one is answering issues or reviewing the pull requests.
I would consider pubsubclient unmaintained so we all need to find a new lib one day or another.


Pseudo develop

Has all changes from @katiehealy and @ehinman in mid-May. These changes include edits to the plotting functions and vignette, changes to autoclean and the special characters function, and the continuous data flagging function.

@cristinamullin The functions to review are:
TADA_boxplot - Figures.R (@katiehealy)
TADA_hist - Figures.R (@katiehealy)
fieldCounts - Filtering.R
filterPie - Filtering.R
ConvertResultUnits - UnitConversions.R
ConvertSpecialChars - Utilities.R
OrderTADACols - Utilities.R
Happy to go through these together! @katiehealy let me know if I missed any other big changes you made on pseudo-develop.

Made a few edits somewhat relevant to this issue, let's discuss: https://github.com/USEPA/TADA/issues/274

@cristinamullin @katiehealy I made some additional edits to pseudo-develop following a quick meeting with Cristina last week. One change I didn't yet implement that we discussed was the faceted plotting for the histogram and boxplots: we had discussed having the functions produce multiple plots when a characteristic name is populated with multiple fractions/units/speciations. That way, users can see related but distinct data groups without the confusion of accidentally plotting different things in the same figure. I wasn't sure if this was something @katiehealy wanted to tackle, or if you'd like me to take a stab.
Other changes:

created a function that makes the comparable data identifier
updated the summary pie charts and tables in the package and vignette
created a legacyfunctions.R script to hold all deprecated functions (with commented out documentations so they don't appear in the reference pages)
exported the reference tables from the package so users can see them

I'd recommend pushing pseudo-develop into develop as soon as is practicable. I have made it a giant, lumbering branch.


2.0.6 on docker development, can't run

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
step1: docker pull dolphinscheduler.docker.scarf.sh/apache/dolphinscheduler:latest  --it'ok
step2: docker-compose up -d
-- Pulling dolphinscheduler-api (apache/dolphinscheduler:2.0.6)...
ERROR: manifest for apache/dolphinscheduler:2.0.6 not found: manifest unknown: manifest unknown
What you expected to happen
I want to use 2.0.6
How to reproduce
flow me step:
step1:    tar -zxvf apache-dolphinscheduler-2.0.2-src.tar.gz
step2:    cd apache-dolphinscheduler-2.0.6-src/docker/docker-swarm
step3:    docker pull dolphinscheduler.docker.scarf.sh/apache/dolphinscheduler:latest
sterp4:   docker-compose up -d
-- then the error occurs:
Pulling dolphinscheduler-api (apache/dolphinscheduler:2.0.6)...
ERROR: manifest for apache/dolphinscheduler:2.0.6 not found: manifest unknown: manifest
unknown
Anything else
No response
Version
2.0.6
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@weile321 Sometimes, docker images will be pushed to dockerhub a few days later than the binary release. and the good new is, now version 2.0.6  is available in docker hub now https://hub.docker.com/repository/docker/apache/dolphinscheduler/tags?page=1&ordering=last_updated

@zhongjiajie hi, does 2.0.6 support python3 ? thx

@zhongjiajie hi, does 2.0.6 support python3 ? thx

It suppport only if your machine running  worker server have python3 in env and you config it to dolphinscheudler PYTHON_HOME


(PUP-11079) Tidy files with age equal or greater

Tidy resource documentation for the age parameter states that files whose age is equal or greater than the specified time will be removed, and that setting age to 0 will remove all files.[1]
The actual implementation only removes the file if the age is greater, not equal, so fix the comparison.
Note that the size parameter is documented similarly, and the implementation is correct there.
[1] https://puppet.com/docs/puppet/7.7/types/tidy.html#tidy-attribute-age

jenkins please test this on redhat7-64a,windows2019-64a

jenkins please test this on redhat7-64a,windows2019-64a


getFileList(), getColList() does not work.

Error message
to Java class "java.lang.Object" is prohibited. (/opt/apache-tomcat-8.0.44/webapps/personium-engine/WEB-INF/classes/js-lib/personium-dao.js#467)

Released v1.5.16


Support for mono-repo structure in ionic scripts

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[X] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[X] feature request
Current behavior:

When working on a system that has multiple apps, all using the same API it is convenient to have all code in a mono-repo rather than one git repo per application.
Given the following folder structure:
|-my-project/
|----apps/
|--------awesome-app1/
|------------(ionic app in here)
|--------awesome-app2/
|------------(ionic app in here)
|----libs/
|--------services/
|------------(shared service code)
|--------model/
|------------(shared model and business logic)

It would be good if the package.json file in the my-project folder could describe all of the dependencies across all of the apps. This way only one node_modules folder in my-project would be needed, instead currently each app needs it's own node_modules. I know it's possible to have multiple apps in a mono repo, sharing a libs folder and the same node_modules folder in angular. It's a shame the benefits are lost as soon as I add an Ionic project into the mono repo
Given that IonicPro requires the app code to be pushed into a git repo in order for it to become available in IonicView, then I'm assuming the above structure won't work with IonicView.

This issue was moved to ionic-team/ionic-cli#2988


Get the ground-truth annotation for the LARRY dataset (Fig. 4c)

If you are interested to get the the annotation as in Fig. 4c, you can download cospar, and run adata_orig=cs.datasets.hematopoiesis(), you will get the latest LARRY dataset, and it will include many fields
particularly, adata.obs[''NeuMon_fate_bias'']  is what you want. It will tell you the fate bias of each cell.

Hi Dr. Wang,
Thanks for this great work and sharing the dataset. After downloading the dataset using cs.datasets.hematopoiesis(), I noticed that the number of cells is fewer than the number of cells in here. Did you filter out some of the cells based on certain criteria?

you may use
cs.datasets.hematopoiesis_130K() to get all the data. cs.datasets.hematopoiesis() only contains cells with LARRY barcodes.


[LC0017] Analyzer crashes with InvalidCastException continued...

Similar to #149.
But I get another casting error, running BC.LinterCop v<IP_ADDRESS>
[{
	"resource": "/<path to my app>/App/app.json",
	"owner": "_generated_diagnostic_collection_name_#0",
	"code": "AD0001",
	"severity": 4,
	"message": "Analyzer 'BusinessCentral.LinterCop.Design.Rule0017WriteToFlowField' threw an exception of type
'System.InvalidCastException' with message 'System.InvalidCastException: Unable to cast object of type
'Microsoft.Dynamics.Nav.CodeAnalysis.BoundLiteral' to type 'Microsoft.Dynamics.Nav.CodeAnalysis.IFieldAccess'.\r\n   at
BusinessCentral.LinterCop.Design.Rule0017WriteToFlowField.CheckForWriteToFlowField(OperationAnalysisContext context)\r\n   at
Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass53_1.<ExecuteOperationAction>b__1() in
D:\\a\\1\\s\\source\\Prod\\Microsoft.Dynamics.Nav.CodeAnalysis\\DiagnosticAnalyzer\\AnalyzerExecutor.cs:line 764\r\n   at
Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer,
Action analyze, Nullable`1 info) in
D:\\a\\1\\s\\source\\Prod\\Microsoft.Dynamics.Nav.CodeAnalysis\\DiagnosticAnalyzer\\AnalyzerExecutor.cs:line 1100'",
	"source": "AL",
	"startLineNumber": 1,
	"startColumn": 1,
	"endLineNumber": 1,
	"endColumn": 1
}]

You should see a LC0000 info message which shows which line crashes the linter. Then I will be able to analyze this further

Aaah, nice! 👍
I now have LC0000 in a dozen places, all seems to be on line 41:
 info LC0000: There was an Error in Rule "Rule0017" of type "InvalidCastException" at Line 41

Some of the error info is now lost, though.
Like Unable to cast object of type  'Microsoft.Dynamics.Nav.CodeAnalysis.BoundLiteral' to type 'Microsoft.Dynamics.Nav.CodeAnalysis.IFieldAccess'
But maybe you do not need this when you've got the line number?

The idea was that the info message will show which line of AL code leads to the error. Then I can reproduce and debug.
/Stefan

Von: Johannes Wikman @.>
Gesendet: Friday, February 25, 2022 8:42:36 PM
An: StefanMaron/BusinessCentral.LinterCop @.>
Cc: Stefan Maroń @.>; Comment @.>
Betreff: Re: [StefanMaron/BusinessCentral.LinterCop] [LC0017] Analyzer crashes with InvalidCastException continued... (Issue #156)
Some of the error info is now lost, though.
Like Unable to cast object of type 'Microsoft.Dynamics.Nav.CodeAnalysis.BoundLiteral' to type 'Microsoft.Dynamics.Nav.CodeAnalysis.IFieldAccess'
But maybe you do not need this when you've got the line number?
—
Reply to this email directly, view it on GitHubhttps://github.com/StefanMaron/BusinessCentral.LinterCop/issues/156#issuecomment-1051143418, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF5QBJ6M6KEN5TEWZJV2BSDU47LSZANCNFSM5PHP7DAA.
You are receiving this because you commented.Message ID: @.***>

As long as you can repro from that, we're happy. 🙂
But if you need more hints on what really fails on a line, I guess that you could just output the exception message in the warning.

The original error message is all I can get and it does not help me without the AL code causing it.
You said that you dont know what line is casing the error, so I added LC 0000.
If you can, please give me examples of AL code that leads to LC0000. Without that AL code samples I can not fix the Linter code
/Stefan

Von: Johannes Wikman @.>
Gesendet: Friday, February 25, 2022 10:16:00 PM
An: StefanMaron/BusinessCentral.LinterCop @.>
Cc: Stefan Maroń @.>; Comment @.>
Betreff: Re: [StefanMaron/BusinessCentral.LinterCop] [LC0017] Analyzer crashes with InvalidCastException continued... (Issue #156)
As long as you can repro from that, we're happy. 🙂
But if you need more hints on what really fails on a line, I guess that you could just output the exception message in the warning.
—
Reply to this email directly, view it on GitHubhttps://github.com/StefanMaron/BusinessCentral.LinterCop/issues/156#issuecomment-1051274714, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF5QBJZJR7E7LAPUCRD3SW3U47WRBANCNFSM5PHP7DAA.
You are receiving this because you commented.Message ID: @.***>

That should be fixed in the latest version

I can confirm that this is solved in 0.24.5. 👍


Autocomplete: Lazy option loading

User requested feature from Autocomplete testing.
A lot of data is fetched from an api based on user input. A loading state for Autocomplete when options are loading based on user input would help with this use-case for our users.
Quick thoughts on this feature request:

Loading state design?
Is this even doable/good a11y wise with dynamic content?
Disabled for user interaction component when "loading"?
Error loading message goes where? Helpertext?

The package react-window-infinite-loader might be useful for this


Add documentation for the commitish option

Closes #83

Was great that someone did this. I would suggest linking to docs so people understand what values commitish accepts under target_commitish -> https://developer.github.com/v3/repos/releases/#create-a-release

Fantastic! I'd also recommend adding this to the example further down in the README.


Add support for MDNS

Currently will not work on regsim because of the requirements
to set the hardware address and time to live which appear not
to be implemented.

I moved mdns into its own module.

I also added some cfg so that it would run on regsim.

Absolutely.


Updated instructions for updating a cask

Set variable github_user
Fork homebrew-cask before cask-repair

github_user is repeated from the previous step, but I don’t mind it if you think it looks clearer.
@adidalal If you like what you see, feel free to merge.

I was directed directly to the update task from another project so I never went through the previous tasks and thus never assigned the variable. Couldn't hurt doing it twice I guess?
Updated the PR with my-github-username instead though.

Thank you for the contribution. It was merged directly as dda1a8dfe3b1f4c06f5d39a9d837d01f64e0299b to keep commit history cleaner. Your contribution is still credited to you.


Switch to Terrapin packages - 404 Not found

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=183061&view=logs&j=9833cdaa-2e92-5f8d-56a8-239fd18fd628&t=de2250f8-ad22-5764-1092-78981e0d3df6
Starting: Switch to Terrapin packages
==============================================================================
Task         : Command line
Description  : Run a command line script using Bash on Linux and macOS and cmd.exe on Windows
Version      : 2.201.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/command-line
==============================================================================
Generating script.
========================== Starting Command Output ===========================
/bin/bash --noprofile --norc /Users/runner/work/_temp/9f17b705-0fe5-4fb1-98af-bbd51708dfb1.sh
npm WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/terrapinadotask - Not found
npm ERR! 404
npm ERR! 404<EMAIL_ADDRESS>is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR!      /Users/runner/.npm/_logs/2022-08-24T23_41_10_665Z-debug-0.log
##[error]Bash exited with code '1'.
##[warning]RetryHelper encountered task failure, will retry (attempt #: 1 out of 3) after 1000 ms
Generating script.
========================== Starting Command Output ===========================
/bin/bash --noprofile --norc /Users/runner/work/_temp/4619ab58-9f83-42f7-9822-a34dee77133e.sh
npm WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/terrapinadotask - Not found
npm ERR! 404
npm ERR! 404<EMAIL_ADDRESS>is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR!      /Users/runner/.npm/_logs/2022-08-24T23_41_13_835Z-debug-0.log
##[error]Bash exited with code '1'.
##[warning]RetryHelper encountered task failure, will retry (attempt #: 2 out of 3) after 4000 ms

Another one with same error
https://dev.azure.com/monacotools/Monaco/_build/results?buildId=183077&view=logs&j=04a1dd9d-62ef-537a-ed01-63d944c25bb6&t=673ff5a9-57b5-5626-ac59-479be2681754

Again failed today - https://dev.azure.com/monacotools/Monaco/_build/results?buildId=183183&view=logs&j=e2741c9b-092a-5194-53b7-e14e2b3f594a&t=093d6481-c999-5f7d-810b-a60fbb06ce7b&s=a56e7651-4213-56af-9488-c25fe636c10c

Resolution was to disable this.


Support for CIM layer writer

This commit adds a new cimfs differ that will in turn call the cim layer writer from
hcsshim to write container image layers in the CIM format.
Signed-off-by: Amit Barve<EMAIL_ADDRESS>

@ambarve What's the status here?

We aren't using this fork anymore. Closing this out.