This machine config crashes both old and new visualizer.

Description
This totally serializable machine config crashes both old and new visualizer.
It works fine with xstate and useMachine.
{
  id: 'app',
  initial: 'checking',
  context: {
    profile: {},
    documents: [],
    idps: [],
    verificationUrl: undefined,
    loginMessage: undefined,
  },
  states: {
    checking: {
      invoke: {
        src: 'checkToken',
        onError: { target: 'gettingIdps', actions: 'setLoginMessage' },
        onDone: [
          {
            cond: 'hasEmailVerification',
            target: 'verifyEmail',
          },
          { target: 'authenticated', actions: 'clearCodeParam' },
        ],
      },
    },
    verifyEmail: {
      entry: { actions: 'setVerificationUrl' },
      on: { LOG_IN: 'loggingIn' },
    },
    gettingIdps: {
      entry: { actions: 'clearTokens' },
      invoke: {
        src: 'getIdps',
        onError: 'login',
        onDone: { target: 'login', actions: 'setIdps' },
      },
    },
    login: {
      entry: { actions: 'setLoginMessage' },
      on: { LOG_IN: 'loggingIn' },
    },
    redirecting: { type: 'final' },
    loggingIn: {
      invoke: {
        src: 'logIn',
        onError: 'login',
        onDone: 'redirecting',
      },
    },
    authenticated: {
      initial: 'authedIdle',
      states: {
        authedIdle: {
          on: { INIT_DOCUMENTS: 'fetchingDocs', LOG_OUT: '#app.gettingIdps' },
        },
        fetchingDocs: {
          invoke: {
            src: 'fetchDocuments',
            onError: 'fetchFailed',
            onDone: { actions: 'setDocuments', target: 'initialized' },
          },
        },
        fetchFailed: {
          on: { RETRY: 'fetchingDocs', LOG_OUT: '#app.gettingIdps' },
        },
        initialized: {
          on: { INIT_DOCUMENTS: 'fetchingDocs', LOG_OUT: '#app.gettingIdps' },
        },
      },
    },
  },
}

Expected Result
Visualized mashine.
Actual Result
The new visualizer crashes with:
TypeError: Cannot read property 'startsWith' of undefined
The old visualizer crashes with:
TypeError: Cannot read property 'indexOf' of undefined
Additional context
Working fine with the following versions in React project:
<EMAIL_ADDRESS><EMAIL_ADDRESS>

✏ Please create a CodeSandbox (XState template) reproduction of this. Thanks!

Sure.
Here is the example with the machine that won't be visualized:
https://codesandbox.io/s/xstate-example-template-vanilla-forked-2b6bh
It works in code as I said, but not in any of your visualizers.

The error was in the code:
      verifyEmail: {
-       entry: { actions: "setVerificationUrl" },
+       entry: { type: "setVerificationUrl" },
        on: { LOG_IN: "loggingIn" }
      },

And there's a couple other places where that typo was made. Remember: entry and exit properties represent actions, not transitions.
Here's a working example: https://codesandbox.io/s/xstate-example-template-vanilla-forked-1z3ch?file=/src/index.js

Damn. Thanks. Will at least never forget that again!


refactor: use registeringPackage instead of bootingPackage with afterResolving

Sorry, but I faced no problems on running a Laravel project with ViteServiceProvider::bootingPackage() neither with ViteServiceProvider::registeringPackage() (as stated in issue #108), but I still pushed the fix to ensure compatibility with Facade\Ignition\IgnitionServiceProvider::registerViewEngines().
If viable, I would like to ask @callmez if they could test my fix on their project by temporary referencing on my fork branch.
I keep staying in total disposition on helping with this PR.

it's worked for me.

I reverted the previous PR because I tried out and @vite wasn't working (it was just printed on screen). I'll try this one out later when I can get a working dev environment 👍

Thanks and sorry for the delay!


Self Update On Windows

Self Update need administration to execute.
I think the simplest way to solve this, change installation locations.
Currently is in Program Files, I think that the Application Directory don't need permission.

https://github.com/kool-dev/kool/blob/cbe7ba9c97edac4b40d1d2bb95a7665580c32194/inno-setup/kool.iss#L19
My suggestion, change all {autopf} to {app}
See: https://jrsoftware.org/ishelp/ in [Dirs] section

I think it is OK to have the need of Admin privilegies to update... that is expected for most users I guess.
What I suggest we do is improve error handling within self-update and given the running OS we display a proper help message / instructions on how to upgrade (in case of Windows, asking to open prompt as administrators to perform this action).

This will be needed to Linux as well (sudo to write at destiny path). Also we should also note error in the same situation as #15  mentions about another kool process running (on Linux is an issue at least).

What do you think @dbpolito @danielsuguimoto ?

I agree, improving the error handling and adding good documentation about it should work around this problem.

I agree as well. Windows users expect to be prompt about permissions. Messages and documentation to guide them are enough.

To sum up what is left here:

Improve error handling on self-update to alert user based on their OS, what step to take to fix it.


A few errors in yabe tutorial

As per #50, there are a few minor errors in the yabe tutorial in master (as of 13b7f6de).
guide3

s/bootsrap/bootstrap/

guide5

Line 39 specifies a TTL of "10mn". The unit of "mn" is unclear (in the context of documentation prose), so explicitly saying "minutes" would be better.
yabe misspelled on line 70.
The ellipsis on line 136 is missing a third period.

Merged


Unsampled Continuation is not collected for transaction metric.

Metric Gauge for TRANSACTION_UNSAMPLED_CONTINUATION is missing.
Unsampled continuation gauge instead uses TRANSACTION_UNSAMPLED_NEW, which is wrong.

fixed via #1272


Working blend of Markdown and iconoir

This branch creates these warnings in the dev server output, and probably in the prod build too:
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (184kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)


Add API documentation to README

This PR aims to resolve the issue #23
close #23

Lgtm Note: don't need to draft when the PR is not WIP @fabnguess ;)


Don't do pod count based scaling for an app if a deployment is happening at the moment

Attempt at fixing #5

I realized now that this will also make the pod-based scaling not happen if we have an active canary deployment (because that's also in an additional ReplicationSet).
But I think that's not a huge issue.

Shouldn't we just change getMinPodCountBasedOnCurrentPodCount to calculate the value not based on current pod count but current min value? And only apply the ratio to limit speed of shrinking if the current desired min value is smaller than than the one stored last in the hpa?

I'm not sure how that would work.
What I don't see is then when would the minCount ever increase?
Let's say at night the traffic goes down, and we have 10 pods, and the minCount is 9.
During the day the traffic slowly increases, and by peak time we have 20 pods.
But if we calculated the new minCount based on the current minCount, then it'd just stay 9, right? So if the traffic suddenly drops, then nothing would prevent the HPA from scaling down to 9 pods in one go.
Sorry if I miss something.

The controller sets minReplicas based on a Prometheus query; this query shouldn't result in a higher value during a deployment if you ask me, right?
And if the desired value of minReplicas (coming out of the query) is smaller then the current value we should use the value of the hpa-scaler-scale-down-max-ratio annotation (if set) to slow down the decrease of minReplicas.
I guess it currently uses the actual number of pods instead of the current value of minReplicas to make it independent of the interval of the controller?
An example with increasing traffic
Current minReplicas: 25
Desired minReplicas: 30
We can immediately set minReplicas to 30.
An example with decreasing traffic
Current minReplicas: 25
Desired minReplicas: 10
hpa-scaler-scale-down-max-ratio: 0.9
Since the desired minReplicas is smaller than current minReplicas we take the scale down ratio into account:
We set minReplicas to 0.9 * 25 = 22,5
Does this make sense? Or do I miss something important?

Ah, clear, now I see what you mean.
The thing is that I'd like to make it possible to use the hpa-scaler-scale-down-max-ratio independently (and even without) the Prometheus-based scaling, because making the scaledown gradual can be useful even (or especially?) in situations when we can't use the Prometheus-based scaling.

If Prometheus is unavailable we probably shouldn't modify anything, right?

The way I intend hpa-scaler-scale-down-max-ratio to be used (and that's how we use it now), is that the deployment is scaled with an HPA based on the CPU-load, and then we constantly adjust the minPods based on the hpa-scaler-scale-down-max-ratio and the current pod count. Thereby we prevent sudden downscaling.
So this doesn't need a Prometheus query at all, but it's prone to the issue described in #5

Ah, I didn't understand you want to be able to run it without Prometheus because I originally created it just for that. But it make sense to slow down the shrinking of deployments even without using Prometheus queries. Perhaps you can check to see if a Prometheus query is set in the annotation. If it isn't limit scale down speed based on the actual pods, otherwise base it on the current minReplicas in the hpa.
It's just that figuring out if a deployment is running with this brute force approach seems prone to error as we grow the number of deployments. Since the Kubernetes API doesn't do paging it might take tool long to respond or use up too much memory. If that makes sense.
I looked for alternatives to figure out if a deployment was running, but couldn't find anything useful.

If it isn't limit scale down speed based on the actual pods
I think that's what we're doing now, and that's causing the issue.

Since the Kubernetes API doesn't do paging it might take tool long to respond or use up too much memory. If that makes sense.
I tested this now: in our prod cluster with 2878 ReplicaSets it took between 4-6 seconds to retrieve them (although this is a client running on my machine and using kubectl proxy, so with an in-cluster client it might be faster), and the memory usage increases by 46MBytes when the ReplicaSets are retrieved.
So you're right, this is significantly heavier than retrieving the HPAs (that takes ~1 second and uses only 3MB memory for the 327 HPAs we have on prod)

I'm not sure what the best workaround would be at this point, I'll try to think too if I can come up with some alternative approach.


chore: fix docs build error

I couldn't reproduce the build error locally, so I'm not sure if this will fix it, but I'll try merging this.

It seems to have worked 😉.
I also fixed the links converted from markdown, so I was able to remove a lot of the warnings.


Reference leak fix

Frees references on wrapper GC, and duplicates returned oids so they are not freed with the reference.
iterator / foreach handling would need additional support, but we currently ignore all those methods.

Thanks @srajko, keep them coming :)


Add timeout for players before automatically discarding

Timeout = 60s
Add container to hold "countdown" when player has 15s left


Fixed Issues with backupKeySets and getActiveKey.

Purpose
To fix the issues in the sample to make it work for graph API requests

generateKey
backupKeySets
getActiveKey

Added GenerateKeySetID method to generate an initial empty key to get the KeySet KeyID.

Now console supports full file path including spaces along with multiple parameters

Example for GenerateKey:

TestingIEF "D:\B2C\KeySets\Sample RSA.json"

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: pottabathini sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Create Rover class

Add a Rover class to prepare for the addition of photos from Opportunity and Spirit rovers.

Opportunity and Spirit have already been integrated into explore_mars.


dv.el("hr") should not have children

What happened?
dv.el("hr") ouputs:

    -

This can be fixed by dv.el("hr", ""). However, void elements like hr shouldn't have child elements.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/hr
DQL
No response
JS
No response
Dataview Version
0.5.47
Obsidian Version
1.0.3
OS
Windows

dv.el is not a very good abstraction generally; I am currently considering replacing it with react-hyperscript, which offers shorthand for making virtual react doms for rendering.

Creating with dv.el is going to add a span which should not exist


Expose Ingress information associated to applications

Functional
Allow any user to see how an application (deployed inside or outside of Portainer) is exposed through an Ingress resource.
UI/UX
When inspecting an application, update the Accessing the application section of the application details panel to support information about accessing the application through an ingress service.

Similar to the load balancer property, for each port exposed under the Ingress object add a new line and allow the user to directly access a vhost via a clickable link.
In the port mappings datatable, add information about exposed ports over ingresses under the Exposed port column. Introduce a new "Ingress" publishing mode.

Technical
When inspecting an application, we'll need to map any existing Ingress object that might be associated with the Service associated to the current application.
If a Service/Ingress combo is found, display information about the HTTP rules used to access the Service

Blocking #4010
Blocking #4011


[v4] Dialog disableEscapeKeyDown or Backdropclick not working anymore

disableEscapeKeyDown - false or Backdropclick - false is not working anymore. onClose function does not fire.

[x ] The issue is present in the latest release.
[x ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
onClose function does not fire on ESC or backdropclick.
Expected Behavior 🤔
onClose function should fire on ESC or backdropclick
Steps to Reproduce 🕹
https://codesandbox.io/s/material-demo-forked-yo41c?file=/demo.tsx
Steps:

Try to fire the onClose function with ESC or backdropclick -> Nothing is happening
Try to click with the mouse on the button "Close manually" -> onClose does fire

Your Environment 🌎
material-ui v4.11.4
Chrome latest version
Linux

I am sorry. I have to write the code more carefully:
onClose={(evt) => handleDialogClose(evt)}
not
onClose={(evt) => handleDialogClose}


Update MakeConnection and stub creation functions to use Options modified by stub creation functions instead of just a copy of the Options provided (Bigtable).

This is not a thing for bigtable. Changing the signature of the function that creates the stub from Options const& -> Options& gives this to us for free.
https://github.com/googleapis/google-cloud-cpp/blob/f4b0970daed8b31d749cff060cb40fa639115180/google/cloud/bigtable/data_connection.cc#L163-L172


Use issue templates

This is so I don't have to keep asking the same set of questions when helping people troubleshoot or consider new features/ideas

[!tip]
See github tutorial
also yaml schema for github forms

I'm first testing these changes in the test repo. It is not working as expected...

Ok I've got it working the way I wanted. you can test it out in the test repo by opening a new issue


[NEED ASSIST] Force locale

Could you please tell how to force locale, I'm using sessions flow and drop-in. After adding shopperLocale in DropInConfiguration, it still picks-up device's setting locale on iOS (somehow adding  cardConfiguration: const CardConfiguration() in DropInConfiguration fixed the issue, but when trying to customize the texts it shows the default translations) .
final dropInConfiguration = DropInConfiguration(
environment: configuration.environment,
clientKey: configuration.clientKey,
countryCode: configuration.countryCode,
shopperLocale: configuration.shopperLocale,
amount: configuration.amount,
cardConfiguration: const CardConfiguration(),
applePayConfiguration: ApplePayConfiguration(
merchantId: applePayMerchantId,
merchantName: applePayMerchantName,
allowOnboarding: true,
),
)

Hello @alna26,
I would like to help you. Could you quickly describe how you customize the texts?

Hi @alna26, I just want to follow up on your question. Do you need further assistance?

Hi @alna26, we would like to close this issue but in case you still need support, please feel free to reopen it.

Hi @matmicro,
I assume you are using the sessions flow? In that case, could you provide a value for shopperLocale e.g. "fr-FR" when using our /sessions endpoint?
In addition, could you set the shopperLocale value inside the DropInConfiguration also to "fr-FR"? That should change the language to French.

Hello @matmicro,
sure, I will try to reproduce it so we can solve it together.
In order to pin down the problem, I would like to clarify the flow you are using. Is it the sessions flow
final PaymentResult paymentResult = await AdyenCheckout.session.startDropIn(
       dropInConfiguration: dropInConfiguration,
       checkout: sessionCheckout,
);

or the advanced flow?
final PaymentResult paymentResult = await AdyenCheckout.advanced.startDropIn(
       dropInConfiguration: dropInConfiguration,
       paymentMethods: paymentMethodsResponse,
       checkout: advancedCheckout,
);

In case of the the sessions flow, could you check the shopperLocale value you send when fetching a session via our/sessions API? Do you send "fr-FR" there as well? You need to send this value in order to specify the language of the payment method list because it is part of the session itself. The session language cannot be adjusted later.
It is therefore necessary to provide the shopperLocale in both: the sessions request and the dropInConfiguration.
This is the result of the example. Is this your expected behavior?

Payment method list
Card payment


Support inserting note links by dragging in preview mode

Is it possible to support inserting a note link to another note in the preview mode of the query result? I find Dataview very useful for summarizing/organizing notes, and it would be very convenient if one can insert note links to another note like what one can do in from the starred files list.

This is hard to do in general, since Dataview is a query-based model - it just pulls pages which match some settings into a view automatically. There has been some discussion about being able to do this for simple queries, which I have investigated but haven't made much progress on. Will add it to my backlog along with other simple manipulation-related things.

Forgot if one can drag to insert note links in earlier versions of Obsidiian and Dataview, but now one can drag to insert links like this: Note with Dataview version 0.4.4 and Obsidian version 0.12.13.

Maybe I misunderstood the question. Are you looking to append new entries to a dataview table by dragging, or drag a link OUT of a table into the editor of another page? If you mean the latter,, then that is the HTML result of rendering a [[link]]. I'm not sure I can change that behavior.

Are you looking to append new entries to a dataview table by dragging, or drag a link OUT of a table into the editor of another page?

Yes, I meant the latter, i.e., being able to drag a note link in the form of [[Note]], instead of Note. Then one can insert the link to another open note (in edit mode), just like what one can do when dragging to inserting links from starred files list.

If you mean the latter,, then that is the HTML result of rendering a [[link]]. I'm not sure I can change that behavior.

Aha, I thought it would be as easy as replacing the text (dragged result) from Note to [[Note]]. 😅  Thanks for the clarification, though.

It seems that this feature has already been supported in v0.4.10 of Dataview and v0.12.16 of Obsidian. 😃

Seems like it got an Obsidian-side fix - will tentatively fix unless something new shows up.


mirror pods are visible in API but not gettable

thockin@freakshow kubernetes kube2sky-with-token-fix /cluster/addons/dns/kube2sky$ k get pods
NAME                                           READY     REASON    RESTARTS   AGE
dns-proxy-kubernetes-minion-hiko               0/2       Pending   0          1s
fluentd-cloud-logging-kubernetes-minion-0j4m   1/1       Running   0          23h
fluentd-cloud-logging-kubernetes-minion-gttp   1/1       Running   0          23h
fluentd-cloud-logging-kubernetes-minion-hiko   1/1       Running   0          23h
fluentd-cloud-logging-kubernetes-minion-vv0d   1/1       Running   0          23h
hostnames-04rup                                1/1       Running   0          22m
hostnames-14qb3                                1/1       Running   0          22m
hostnames-320gg                                1/1       Running   0          22m
hostnames-p9jwo                                1/1       Running   0          22m
hostnames-proxy-kubernetes-minion-hiko         0/1       Pending   0          0s
hostnames-rtva6                                1/1       Running   0          22m
kube-dns-v4-cf036                              3/3       Running   0          23h
monitoring-heapster-v4-dthvf                   1/1       Running   1          23h
monitoring-influx-grafana-v1-ozxel             2/2       Running   0          23h

thockin@freakshow kubernetes kube2sky-with-token-fix /cluster/addons/dns/kube2sky$ k get pod dns-proxy-kubernetes-minion-hiko
Error from server: pods "dns-proxy-kubernetes-minion-hiko" not found

thockin@freakshow kubernetes kube2sky-with-token-fix /cluster/addons/dns/kube2sky$ k get pod hostnames-proxy-kubernetes-minion-hiko
Error from server: pods "hostnames-proxy-kubernetes-minion-hiko" not found

thockin@freakshow kubernetes kube2sky-with-token-fix /cluster/addons/dns/kube2sky$ k get pod hostnames-p9jwo
NAME              READY     REASON    RESTARTS   AGE
hostnames-p9jwo   1/1       Running   0          23m

Not sure what's up.  P1 for triage

close it as another dup of #8683


Output questions while generating

This is done in the code, but still output is not written to the file. Is this due to piping, do we need an -o option for output file?

Maybe we can use the -f option from #47?


Automatically ask for upgrade on reading monolithic templates from a folder

As a cobigen user, I want to get automatically informed about the conceptual changes of the configuration going from monolithic template configuration folder to a modular template-set configuration folder. I want to get offered information (#1516) as well as an automatic upgrade as far as possible (#1502).
Acceptance Criteria

[ ] If cobigen reads my monolithic template configuration, which is deprecated, I want to get offered an upgrade option and get informed about the process and steps to be performed
[ ] To properly make my decision in the moment of time, I would like to get more information about why I should do the upgrade and what is the impact
[ ] eventually, I would like to get the option to perform the automated upgrade step
[ ] I want to also get an option to postpone the upgrade to do it later
[ ] the user should be asked every time a monolithic configuration is read from a folder
[ ] make sure, that maven execution is not blocked by this feature at all
[ ] after upgrade / no upgrade, the generation will be executed as requested by the user initially

Hey team! Please add your planning poker estimate with ZenHub @EduardKrieger @GuentherJulian @jan-vcapgemini @LarsReinken @mdukhan @MikeSchumacherCapgemini

What do we do if we have in the Cobigen Homepath the two monolithic and the new template-sets folders?
Should we recommend an upgrade for the monolithic templates? Or should this case be ignored?
This case, I believe, should be ignored. Otherwise, we risk receiving an infinite number of upgrade warning messages. In this case, the template-set should be preferred.
Any suggestions?


Add the --src option from pip to pip-sync.

Because as pip-compile itself reports,
pip-compile does not support URLs as packages, unless they are editable. Perhaps add -e option?

, it'd be great to be able to use the --src option from pip with pip-sync so packages installed from VCS repositories can be downloaded to somewhere other thanpip's default <venv path>/src. This would be especially useful in cases where a virtualenv isn't being used, such as in some Docker installations, and editable packages just end up in ./src.
The current output from trying this is:
# pip-sync --src $PIP_EDITABLE_PACKAGES_DIR
Error: no such option: --src

Related to another request for additional pip options, #321.

:+1:

Just a small note: in many of these situations, you can just run pip directly on the .txt file generated by pip-compile, because you are operating on a fresh environment anyway.

Great point. Thanks @rfleschenberg.

Closing due to lack of activity and also since the use case seems to be handled by running pip directly as per the most recent comment.


#743 Replace assertTimeout in tests with assertTimeoutPreemptively

Replace assertTimeout in tests with assertTimeoutPreemptively
closes #743

Many thanks.


Add VPC pricing

We currently support aws_nat_gateway pricing, but we need to add pricing for the following VPC resources:

AWS PrivateLink pricing (https://aws.amazon.com/privatelink/pricing/)

Terraform resource: `aws_vpc_endpoint

AWS Site-to-Site VPN pricing (https://aws.amazon.com/vpn/pricing/)

Terraform resource: aws_vpn_connection

AWS VPC traffic mirroring pricing (https://aws.amazon.com/vpc/pricing/)

Terraform resource: ec2_traffic_mirror_session

I'll take this one @aliscott @alikhajeh1, I've got a PR nearly ready to go for vpc_endpoint.

I'll also include Transit Gateway.

Fixed by https://github.com/infracost/infracost/pull/264, https://github.com/infracost/infracost/pull/265 and https://github.com/infracost/infracost/pull/266.
FYI @dawidmalina, this should be live tmrw in the next release of infracost (v0.7.8).


Remove embedded glTF

Fix #2025

f you are reading this PR and want to tell us why embedded is important in your pipeline/workflow, please comment this ticket: https://github.com/KhronosGroup/glTF-Blender-IO/issues/2148


2023-12-05 - Oversight Committee Agenda

Agenda - 2023-12-05
Time 16:30 (UTC) - See the time in your timezone

Chair – Tammy McClellan (Microsoft)
Vice-Chair – Chris Lloyd-Jones (Avanade)
Convener – Russ Trow (GSF)

Antitrust Policy
Joint Development Foundation meetings may involve participation by industry competitors, and it is the intention of the Joint Development Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws.
If you have questions about these matters, please contact your company counsel or counsel to the Joint Development Foundation, DLA Piper.
Roll Call
Please add 'Attended' to this issue during the meeting to denote attendance.
Any untracked attendees will be added by the GSF team below:
Agenda

[ ] Convene & Roll Call
[ ] Review previous meeting: https://github.com/Green-Software-Foundation/oc/issues/52
[ ] Review/Approve agenda

Projects
GSF Project Tracker

[ ] https://github.com/Green-Software-Foundation/oc/issues/56 - In progress

Governance

[ ] https://github.com/Green-Software-Foundation/oc/issues/50
[ ] https://github.com/Green-Software-Foundation/oc/issues/37
[ ] Request for Discussion: Async-First Working Groups & OC

AOB

[ ] Reschedule or Cancel next meeting?

Next meeting

[ ] 2nd January 2024

Adjourn

[ ] Any objections?

Attended

Attended

Attended!

Attended

Meeting Notes
Projects

Project Tracker is blocked for Avanade as it's not had a security review - Could the information be shared in another way? (@adamj89)

[ ] @russelltrow @Sophietn @seanmcilroy29 update Chairs/Leads on the GSF Project Tracker

Graduation review for Carbon Aware SDK v1

Discussed the graduation review for CASDK. Confirmed that they are aiming for Graduation in Jan after release 1.2
Briefly discussed the revised graduation criteria: https://oc.greensoftware.foundation/project-lifecycle.html#graduated

[ ] @Sophietn provide OC with further details on the Graduation Review process and the criteria ahead of the actual activity

Elections
https://greensoftwarefoundation.atlassian.net/wiki/spaces/~612dd45e45cd76006a84071a/pages/93814822/Elections

Google Docs is blocked for a number of people including @greenhsu123. Thought we agreed to a different approach? (@seanmcilroy29)

[ ] @seanmcilroy29 update the "Current Working Group Chairs" section with WG names
[ ] @seanmcilroy29 ensure the nomination submissions are correct - e.g. Tammy hasn't applied for CWG
[ ] @seanmcilroy29 provide a status update on the elections this week

Individual Contributors

@Sealjay asked about the status of ICs.
@jawache discussed the legal challenges of having ICs.
Suggestion of creating a new level such as Affiliate or Observer and having a
Suggestion of making all meetings public and having a blanket disclaimer covering all contributions
Will need to be discussed at the Steering Committee

Async-First

@russelltrow @jawache verbally shared their idea on moving Working Groups to an 'async-first' approach
Discussed increased use of email for communication - "Tammy already has 100+ emails a week"
Agreed that a cultural change is required not just different tooling
Discussed new Member Onboarding - Buddy system was trialed in the Standards WG, not a huge uptake

[ ] @russelltrow to share an RFD with the group
[ ] @russelltrow find examples of other organisations who successfully run 'async-first'
[ ] @jawache @NAMRATA-WOKE investigate the Buddy system trialed in the Standards WG

Actions

[ ] @russelltrow ask group if they wish to cancel or reschedule January 2nd meeting


feat: calcuate sd_hash in kb JWT

sd_hash is a base64url hash digest of the presented SD-JWT. So we can calculate that value in our logic.
It closes #108

@lukasjhan changes are looking good. Just fix the linting error and the PR is ready to be merged

@lukasjhan changes are looking good. Just fix the linting error and the PR is ready to be merged

I changed it :), I think my vscode has some kind of bug. It changed the indent automatically but not show the diff XD

A test for createKBJwt would also be nice

A test for createKBJwt would also be nice

Okay I'll add one :)

@cre8  Deleted it because it was an unused function. It was my mistake. :)

hmm.. 0.04%


feat(ui-components): implement basic Tabs component

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

part #47247, plan step one

I would like to know if I am allowed to use @headlessui/react After reading #47043, if I can use Transition in @headlessui/react, the implementation of animation will be simpler and more elegant

part Closes #47247, plan step one

if it's only partly, don't use the Closes keyword, or the PR is going to close the issue when merged

Thank you for for your patience with the review.
This is a great start and here are a few points I would like to discuss about this implementation.

Yes, @headlessui/react simplifies the development a lot. If you decide to go down that route, please make sure  Tabs and TabPane still receive the same props and have the same structure when used.

If you decide develop the component from scratch, I highly recommend to also look into how the current version of Tabs  and its related components are developed and how the state is managed.

The current usage of Tabs and TabPane could be simplified in learn so we could spend less time creating the Tabs related components. For instance, TabPane could be replaced with Tab and onSelection could be completely removed.

I would appreciate it if you could do the research for the  mountOnEnter and  unmountOnExit and learn about performance benefits and see how we could implement them.

Animation is not needed at this point. But we will include it across our components if necessary in the next iterations.

Let me know if the above makes sense to you.

Hi,
We are closing old issues/pull requests that we think are stale. If you still need help or are fixing this, please open a new issue/pull request.
Thanks for your understanding.
Happy contributing.

@Sboonny, here is the tabs component for your consideration.

I'm sorry for not responding to the PR here for being busy last year.
After China has fully released the lockdown, I will discuss this PR with members of the freeCodeCamp community in Shanghai this weekend and try to advance it. (My train of thought is a bit stucked😂)

Welcome back, I have discussed the abstraction in tabpane with Ahmad, instead of tabPane accepting children and tabs mapping over the tabpane and creating multiple content and buttons.
for example, in mobile layout this how code look like
https://github.com/freeCodeCamp/freeCodeCamp/blob/0f922b86522cba6ec85e4169145e1faaa92cfbb3/client/src/templates/Challenges/classic/mobile-layout.tsx#L87-L125
and this will create tabTrigger and tabContent
   <TabTrigger role='tab' value={Tab.Instructions}>
          {i18next.t('learn.editor-tabs.instructions')}
    </TabTrigger>
    <TabTrigger role='tab' value={Tab.Editor}>
          {i18next.t('learn.editor-tabs.code')}
     </TabTrigger>

      <TabContent value={Tab.Instructions} {...props}>
        {instructions}
      </TabContent>
      <TabContent value={Tab.Editor} {...props}>
        {editor}
      </TabContent>

Instead of logic that creates this, we want to abandon that destructure and restructuring of the elements, and just create a direct TabTrigger and TabContent that will show and hide the content when the value is the same.

@masterX89 are you still working on this? @ahmadabdolsaheb @Sboonny should you guys take this up instead?

Sorry, I'm busy with my work again and I probably won't be free until early July.
May need help from others to complete this component.

No worries, you have done most of the work anyone, I will add the finishing touches at the end of the day.

storybook isn't reading the value of the variables names in the CSS, this is applying to the main branch, too.
I can't duplicate the styles unless this is fixed, so I am putting this on hiatus until I solve the storybook issue.

@Sboonny thank you for working on this pr. Let me know if you need my help to finalize it.

No need, if you are happy with the current implementation, I will sort the conflicts and we merge this.


[WIP] Add detect avalanche algorithm

Closes #40

Codecov Report

Merging #45 (f2d3ce8) into update-0.1.1 (c1e4b06) will increase coverage by 9.79%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           update-0.1.1      #45      +/-   ##
================================================
+ Coverage         41.98%   51.78%   +9.79%
================================================
  Files                24       26       +2
  Lines               724      728       +4
  Branches             84       84
================================================
+ Hits                304      377      +73
+ Misses              418      343      -75
- Partials              2        8       +6

Flag
Coverage Δ

unittests
51.78% <0.00%> (+9.79%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

miv/datasets/criticality.py
100.00% <ø> (+100.00%)
:arrow_up:

miv/signal/criticality/__init__.py
0.00% <0.00%> (ø)

miv/signal/criticality/avalanches.py
0.00% <0.00%> (ø)

miv/datasets/optogenetic.py
50.00% <0.00%> (+50.00%)
:arrow_up:

miv/datasets/utils.py
53.19% <0.00%> (+53.19%)
:arrow_up:

miv/datasets/__init__.py
100.00% <0.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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

We are thinking of merging the criticality analysis along with burst analysis. They share a similar concept and use the same algorithm behind. I'm closing this PR for now, but I'll leave the code in the branch until the method gets sorted out.


Post-namespace cleanup

Remove conditionals now that all providers are namespaced
Rename some specs that got left behind when their models were rearranged

@blomquisg @Fryguy Please review.

Dang...sorry @matthewd ...I should have merged this one first.


Include information about user attribution in the section about bot messages

The information on user attribution (https://docs.microsoft.com/en-us/microsoftteams/platform/messaging-extensions/how-to/action-commands/respond-to-task-module-submit?tabs=dotnet%2Cdotnet-1#user-attribution-for-bots-messages) is applicable to bot messages in general. It's not limited to messages triggered by action commands.
Please include (or link to) this information in the bots section of our docs.

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

ID: 8fb0c2d8-f880-c260-d0be-6b9467e47f8c
Version Independent ID: 7820ffc9-5260-c818-f5fb-672801a67d00
Content: Respond to the task module submit action - Teams
Content Source: msteams-platform/messaging-extensions/how-to/action-commands/respond-to-task-module-submit.md
Product: msteams
GitHub Login: @surbhigupta
Microsoft Alias: anclear

Need to add this info(https://docs.microsoft.com/en-us/microsoftteams/platform/messaging-extensions/how-to/action-commands/respond-to-task-module-submit?tabs=dotnet%2Cdotnet-1#user-attribution-for-bots-messages) under this below doc section: https://docs.microsoft.com/en-us/microsoftteams/platform/bots/how-to/conversations/conversation-messages?tabs=dotnet
FYI @surbhigupta12 Redirecting to doc team for enhancement


Fix recursive traversal methods

Node.descendants now walks through not only the first child, but other as well.
Document.nodes does not include dummy root node (restored pre-896cf3a behaviour).
Document.querySelector and .querySelectorAll do not check the node they are called on (consistent with JS DOM API).

Thanks for merging! Could you tag it please so it is available on DUB?

Sorry for the late action. Just tagged it :)


sharpsat-td: init at unstable-2021-09-05

Description of changes
Added sharpSAT-td including basic SMTLIB-based test
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] 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

[x] 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

[ ] Fits CONTRIBUTING.md.

It actually hasn't seen any commits since 2021, which is why I gave it that version.

Hmm maybe I shouldn't have assumed you'd tested this before approving..

#234460


Add LF T&C in ADOPTERS.md

What this PR does
Add the Linux Foundation - Training & Certification to the list of adopters.
Why we need it and why it was done in this way
We were invited by Fabian Deutsch (@fabiand) to add our name to the list.
Special notes for your reviewer
Kindly review the text if it's sufficient. I would like a +1 from @marcus3449 and @jkinred before this is merged.
NONE

+1 from me.


[Change Request] table 77 "Report Selections" - procedure FindReportSelections

Hello, can you please change the lokal procedure FindReportSelections to global?
    procedure FindReportSelections(var TempReportSelections: Record "Report Selections" temporary; AccountNo: Code[20]; TableNo: Integer): Boolean
    var
        Handled: Boolean;
    begin
        OnFindReportSelections(TempReportSelections, Handled, Rec, AccountNo, TableNo);
        if Handled then
            exit(true);
        if not TempReportSelections.IsTemporary() then
            Error(ReportSelectionsMustBeTemporaryErr); // Avoid deleting actual entries in report selections

        if CopyCustomReportSectionToReportSelection(AccountNo, TempReportSelections, TableNo) then
            exit(true);

        exit(CopyReportSelectionToReportSelection(TempReportSelections));
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 13579.


Update the CLI to have the Google provider

The Airy CLI needs to deploy a new Airy instance on Google using:
airy create --provider gcp

Enabled these APIs for Go

Cloud Build API
Artifact Registry API
Cloud Run Admin API

https://cloud.google.com/build/docs/building/build-go

in cli directory provider go files has the following linting error
golangci-lint run pkg/providers/aws
WARN [runner] Can't run linter goanalysis_metalinter: buildir: failed to load package : could not load export data: no export data for "github.com/imdario/mergo"
ERRO Running error: 1 error occurred:
        * can't run linter goanalysis_metalinter: buildir: failed to load package : could not load export data: no export data for "github.com/imdario/mergo"

cli % golangci-lint run pkg/providers/gcp
WARN [runner] Can't run linter goanalysis_metalinter: buildir: failed to load package : could not load export data: no export data for "github.com/imdario/mergo"
ERRO Running error: 1 error occurred:
        * can't run linter goanalysis_metalinter: buildir: failed to load package : could not load export data: no export data for "github.com/imdario/mergo"


Add support for Quantinuum targets

This change adds support for Quantinuum targets in the QDK.

/azp run


make error

I get an error when running make and am not really sure how to fix it...
[...]
rm -rf lib/libantlr.a
ar -crv lib/libantlr.a tools/libantlr3c-3.4/*.o
a - tools/libantlr3c-3.4/antlr3baserecognizer.o
a - tools/libantlr3c-3.4/antlr3basetreeadaptor.o
a - tools/libantlr3c-3.4/antlr3basetree.o
a - tools/libantlr3c-3.4/antlr3bitset.o
a - tools/libantlr3c-3.4/antlr3collections.o
a - tools/libantlr3c-3.4/antlr3commontoken.o
a - tools/libantlr3c-3.4/antlr3commontreeadaptor.o
a - tools/libantlr3c-3.4/antlr3commontreenodestream.o
a - tools/libantlr3c-3.4/antlr3commontree.o
a - tools/libantlr3c-3.4/antlr3convertutf.o
a - tools/libantlr3c-3.4/antlr3cyclicdfa.o
a - tools/libantlr3c-3.4/antlr3debughandlers.o
a - tools/libantlr3c-3.4/antlr3encodings.o
a - tools/libantlr3c-3.4/antlr3exception.o
a - tools/libantlr3c-3.4/antlr3filestream.o
a - tools/libantlr3c-3.4/antlr3inputstream.o
a - tools/libantlr3c-3.4/antlr3intstream.o
a - tools/libantlr3c-3.4/antlr3lexer.o
a - tools/libantlr3c-3.4/antlr3parser.o
a - tools/libantlr3c-3.4/antlr3rewritestreams.o
a - tools/libantlr3c-3.4/antlr3string.o
a - tools/libantlr3c-3.4/antlr3tokenstream.o
a - tools/libantlr3c-3.4/antlr3treeparser.o
#NOTICE: update the sparql.tar.gz if Sparql* in Parser are changed manually
rm -rf Parser/Sparql*
cd tools; tar -xzvf sparql.tar.gz; mv Sparql* ../Parser/;
SparqlLexer.c
SparqlLexer.h
SparqlParser.c
SparqlParser.h
ccache g++ -c -Wall -O2 Main/gbuild.cpp -I./tools/libantlr3c-3.4/ -I./tools/libantlr3c-3.4/include  -o .objs/gbuild.o
In file included from Main/../Database/../Parser/DBparser.h:14:0,
                 from Main/../Database/Database.h:23,
                 from Main/gbuild.cpp:11:
Main/../Database/../Parser/SparqlParser.h:2477:0: warning: "ADD" redefined
 #define ADD      6

In file included from /usr/include/arpa/nameser.h:527:0,
                 from ./tools/libantlr3c-3.4/include/antlr3defs.h:311,
                 from ./tools/libantlr3c-3.4/include/antlr3.h:33,
                 from Main/../Database/../Parser/SparqlParser.h:435,
                 from Main/../Database/../Parser/DBparser.h:14,
                 from Main/../Database/Database.h:23,
                 from Main/gbuild.cpp:11:
/usr/include/arpa/nameser_compat.h:128:0: note: this is the location of the previous definition
 #define ADD  ns_uop_add

In file included from Main/../Database/../Parser/DBparser.h:14:0,
                 from Main/../Database/Database.h:23,
                 from Main/gbuild.cpp:11:
Main/../Database/../Parser/SparqlParser.h:2519:0: warning: "DELETE" redefined
 #define DELETE      48

In file included from /usr/include/arpa/nameser.h:527:0,
                 from ./tools/libantlr3c-3.4/include/antlr3defs.h:311,
                 from ./tools/libantlr3c-3.4/include/antlr3.h:33,
                 from Main/../Database/../Parser/SparqlParser.h:435,
                 from Main/../Database/../Parser/DBparser.h:14,
                 from Main/../Database/Database.h:23,
                 from Main/gbuild.cpp:11:
/usr/include/arpa/nameser_compat.h:127:0: note: this is the location of the previous definition
 #define DELETE  ns_uop_delete

In file included from Main/../Database/../Parser/DBparser.h:14:0,
                 from Main/../Database/Database.h:23,
                 from Main/gbuild.cpp:11:
Main/../Database/../Parser/SparqlParser.h:2616:0: warning: "QUERY" redefined
 #define QUERY      145

In file included from /usr/include/arpa/nameser.h:527:0,
                 from ./tools/libantlr3c-3.4/include/antlr3defs.h:311,
                 from ./tools/libantlr3c-3.4/include/antlr3.h:33,
                 from Main/../Database/../Parser/SparqlParser.h:435,
                 from Main/../Database/../Parser/DBparser.h:14,
                 from Main/../Database/Database.h:23,
                 from Main/gbuild.cpp:11:
/usr/include/arpa/nameser_compat.h:105:0: note: this is the location of the previous definition
 #define QUERY  ns_o_query

In file included from Main/../Database/Database.h:14:0,
                 from Main/gbuild.cpp:11:
Main/../Database/Join.h:58:36: error: ‘constexpr’ needed for in-class initialization of static data member ‘const double Join::JUDGE_LIMIT’ of non-integral type [-fpermissive]
  static const double JUDGE_LIMIT = 0.5;
                                    ^~~
In file included from Main/../Database/../Query/../Parser/QueryParser.h:13:0,
                 from Main/../Database/../Query/GeneralEvaluation.h:21,
                 from Main/../Database/Database.h:26,
                 from Main/gbuild.cpp:11:
Main/../Database/../Query/../Parser/../Query/QueryTree.h: In constructor ‘QueryTree::QueryTree()’:
Main/../Database/../Query/../Parser/../Query/QueryTree.h:196:15: warning: ‘QueryTree::update_type’ will be initialized after [-Wreorder]
    UpdateType update_type;
               ^~~~~~~~~~~
Main/../Database/../Query/../Parser/../Query/QueryTree.h:185:23: warning:   ‘QueryTree::ProjectionModifier QueryTree::projection_modifier’ [-Wreorder]
    ProjectionModifier projection_modifier;
                       ^~~~~~~~~~~~~~~~~~~
In file included from Main/../Database/../Query/../Parser/QueryParser.h:13:0,
                 from Main/../Database/../Query/GeneralEvaluation.h:21,
                 from Main/../Database/Database.h:26,
                 from Main/gbuild.cpp:11:
Main/../Database/../Query/../Parser/../Query/QueryTree.h:18:3: warning:   when initialized here [-Wreorder]
   QueryTree():
   ^~~~~~~~~
In file included from Main/../Database/Database.h:26:0,
                 from Main/gbuild.cpp:11:
Main/../Database/../Query/GeneralEvaluation.h: In constructor ‘GeneralEvaluation::GeneralEvaluation(VSTree*, KVstore*, StringIndex*, TNUM*, int, int)’:
Main/../Database/../Query/GeneralEvaluation.h:40:16: warning: ‘GeneralEvaluation::stringindex’ will be initialized after [-Wreorder]
   StringIndex *stringindex;
                ^~~~~~~~~~~
Main/../Database/../Query/GeneralEvaluation.h:37:9: warning:   ‘TNUM* GeneralEvaluation::pre2num’ [-Wreorder]
   TNUM* pre2num;
         ^~~~~~~
Main/../Database/../Query/GeneralEvaluation.h:46:12: warning:   when initialized here [-Wreorder]
   explicit GeneralEvaluation(VSTree *_vstree, KVstore *_kvstore, StringIndex *_stringindex, TNUM* _pre2num, int _limitID_predicate, int _limitID_literal):
            ^~~~~~~~~~~~~~~~~
makefile:140: recipe for target '.objs/gbuild.o' failed
make: *** [.objs/gbuild.o] Error 1

Sorry, it is our fault. It seems that we use different version of g++.
We will fix this problem now, and theis page may help you:
http://www.cnblogs.com/wanyuanchun/p/4041080.html


GlobalMemoryStatusEx fails to meet API precondition

Actual behavior
PInvoke.GlobalMemoryStatusEx is generated as follows:
/// <inheritdoc cref="GlobalMemoryStatusEx(winmdroot.System.SystemInformation.MEMORYSTATUSEX*)"/>
[SupportedOSPlatform("windows5.1.2600")]
internal static unsafe winmdroot.Foundation.BOOL GlobalMemoryStatusEx(out winmdroot.System.SystemInformation.MEMORYSTATUSEX lpBuffer)
{
    fixed (winmdroot.System.SystemInformation.MEMORYSTATUSEX* lpBufferLocal = &lpBuffer)
    {
        winmdroot.Foundation.BOOL __result = PInvoke.GlobalMemoryStatusEx(lpBufferLocal);
        return __result;
    }
}

Expected behavior
MEMORYSTATUSEX.dwLength is set prior to invoking the native method.
Repro steps

NativeMethods.txt content:

GlobalMemoryStatusEx

NativeMethods.json content (if present):

{
}

Any of your own code that should be shared?

Context

CsWin32 version: 0.3.49-beta
Win32Metadata version (if explicitly set by project): Not set
Target Framework: net6.0-windows
LangVersion (if explicitly set by project): Not set

Thanks for reporting this, @sharwell.
The metadata for this method is wrong. It attributes the parameter as [out] when it must be [in, out] to support the fact that it must be initialized by the caller first. In fact the docs for this method show it as in, out.
Once the metadata is fixed, CsWin32 will emit this as a ref parameter. It is up to the caller of the CsWin32 generated API to initialize the struct's length field, just as it would in C++.


Rate-limiting support?

During research for an issue I got on http, I found exactly one non-root rate-limiting solution in the form of trickle, (which doesn't work because we fork) for POSIX, and none for Win32, so I'm opening this issue to see if this is something you think makes sense implementing here.
Downstream issue: https://github.com/thecoshman/http/issues/102

You most likely want to go through something like tower-limit.

AFAICT, the tower-limit crate seems to deal in limiting the amount of concurrent threads, as opposed to bandwidth?

Most often in the context of HTTP, rate-limiting refers to the number of requests.
You wish to limit the speed that bytes are written, or read? Bodies in hyper are streamed, so you have control over read or writing when you want.

Yes, we are looking a way to restrict the bandwidth

This should've been opened against iron, since that's our direct upstream and the hyper version they target is wildly outdated anyway. Closing, and apologies for the noise.


Created version.py

This file has the version definition
Issue Addressed
Link to the issue being addressed or any other mentions
Description
A few sentences describing the overall goals of the pull request's commits.
List of Changes Proposed
Testing
Specify the steps to test or reproduce the PR

Coverage decreased (-0.08%) to 81.61% when pulling 6bcd36969dee53c851e66940a39bf9caa8e2ce92 on Sidsharik:patch-1 into 632b5b8ce520f7befa1e8697debab6bfc9ef41a9 on mayn:master.


apply swiftformat

This is the last PR before tagging 1.0.0-alpha3 (final release for 1.0.0)

SwiftDocc errors are being discussed on Slack
https://swift-open-source.slack.com/archives/C04B8HURKB4/p1724432851634649


AngularJS breaks Firebug Lite

In both IE and Chrome, when the angular.js file is includes in the website, it is causing Firebug Lite to error out on line 8179. While this should not be Angular's problem, it might be worth looking into, as it's doing something that can destroy another plugin not involved with Angular.

@jankuca

The dev and beta builds are not affected.

firebug-lite-beta.js:30905 Uncaught TypeError: Cannot read property 'push' of undefined
Adding basic null check:
 if (typeof this.messageQueue == 'undefined') {
        this.messageQueue = [];
    }

Seems to be solving the issue.

Had the same issue as well, on a regular (non-Angular,. vanilla HTML/JS/CSS project).  The same null-check workaround worked for me, as well.


Add option to stop checking for app updates (not just auto-downloading)

It seems that Orbstack still checks for updates even when it's configured to not "Automatically download updates". There doesn't seem to be any other configuration option to stop checking for updates in any sense.
I just got this notified that a new update is ready to install:

Despite this the "Automatically download updates" configuration option being deselected:

Maybe there needs to be an option to prevent checking for updates, or this could actually be a bug.
I've just checked and unchecked "Automatically download updates" to see if that will fix the issue, and will reply to this issue when I notice there has been another update and whether I did or did not get a notification.

Sorry, but in order to reduce version fragmentation and the resulting support burden, there are no plans to allow disabling update checks entirely. You can use a firewall to block api-updates.orbstack.dev if you really want to. The "automatically download updates" toggle is for auto-downloading updates in advance when they're available, so that installation becomes one click.
Also note that beta builds expire after 30 days, so staying behind isn't an option in the long term. Is there a specific reason you want to stay on v0.10.2 or older? Please let us know why.
Once we're out of beta, stable builds will never expire so it'll be your choice to update or not, but we still don't plan to offer an option for disabling checks. I know this isn't the answer you're looking for — I'm sorry, but OrbStack is generally always improving in updates, and with limited resources, we can't spend time supporting old versions forever.
If there's a compelling argument against this, then I'd be happy to hear it, but I don't think we'll be adding an option like this.


Preliminary file inventory

User Story
As a Data User, I need to review an inventory of the files in one of my projects, so that I can orient myself and begin contributing.
User Journey Images

(Slides 28-39 in TigerData_PresLayer_UserJourneys_v0.9_2024-01-24.pdf)
Completion Criteria

[x] Tech setup

[x] #387
[x] #388

[ ] Spikes

[x] #392
[x] #393
[x] #394
[x] #395
[x] #398

[x] #386
[ ] Create a new "Project Contents" page

[x] #389
[x] #390
[x] #391
[ ] #397
[x] #396

We may need to create additional tickets to reflect this in the UI.

Make a "question" ticket to understand what is needed for the UI.  @bess will do this.


[FEATURE]The ability to to add loyalty to Mjolnir and The Leviathan axe

It sucks when other players pick up either the mjolnir or leviathan axe after I throw it at them and don't call it back in time
You should be able to enchant either weapon with Loyalty and that should keep them from being picked up by other players (enabled/disabled by config)
You could also just add a config stating whether the weapons can be picked up by other players or not even without loyalty.
:D

Oh that's a bug actually, other players shouldn't be allowed to pick up your axe/hammer. Will be fixed.

Oh good, ty :D

Is now fixed for the next patch.


Add a module to Negotiator for standard (image) NFT along side attestations when given contract addresses

@oleggrib - Tomek will produce a desired visual for such a scenario. Ideally, this work starts when @nicktaras finished his heroic struggle with cookies, as that struggle leads to huge change in code base -> conflicts with other work.

svelte app example to read NFT image/metadata
https://github.com/TokenScript/playground_nfts/blob/main/src/App.svelte
exmaple page: https://testshop.antopolbus.rv.ua/nfts/
@nicktaras , can you check if you can use code from that APP to read NFTs with Negoatiator?

this is how it looks for me


"Cerchezu", "Căscioarele" are listed in the drop-down menu

Description:
When user writes "Cerchezu", "Căscioarele" the location appears in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Cerchezu"

Expected results
The location appears in the drop-down menu as "Sat Cerchezu, CONSTANȚA, (Cerchezu)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Căscioarele"

Expected results
The location appears in the drop-down menu as "Sat Căscioarele, CONSTANȚA, (Cerchezu)"


Test local css import in production

I'm expecting this test to fail
Related to https://github.com/stealjs/steal-tools/pull/673
Once that PR gets in I'll run this tests against steal-tools#master to make sure everything looks fine.

Update: the test should pass now since<EMAIL_ADDRESS>fixes the issue with local css imports being removed from the css bundle.


⚠ Bofrost.at has degraded performance

In d1b6340, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2318 ms

Resolved: Bofrost.at performance has improved in a79d50a after 6 minutes.


[GHSA-fhfh-6cjg-57rg] Jenkins Snow Commander Plugin 2.0 vulnerable to Cross-Site Request Forgery

Updates

Affected products
CVSS
Description
Severity

Comments
Update versions affected and fixed according to https://www.jenkins.io/security/advisory/2022-02-15/#SECURITY-2536

Any chance you have a fix commit or release log noting the fix for this one?

Any chance you have a fix commit or release log noting the fix for this one?

Here we go, that's part of the 2.0 tag https://github.com/jenkinsci/snowsoftware-commander-plugin/commit/4fb4e68e8e7be3168721a06ad08b263252c276f1
@POST addresses the CSRF vulnerability and Jenkins.getInstance().checkPermission(Jenkins.ADMINISTER); the missing permission check.
Unfortunately, not everyone uses GH releases (yet) :(

Thank you much 👍

Unfortunately, not everyone uses GH releases (yet) :(

Is there a plan to move jenkins stuff to using GH releases? I'd love to chat with you about that if so.

Is there a plan to move jenkins stuff to using GH releases?

Oh, we already do that for a lot of existing and new repositories, but we also let plugin maintainers have almost full rein about their repositories we're hosting. We don't want to enforce a workflow on them, and rather provide the tools to automate processes. Reusable workflows, releasing via GH actions, and stuff like automatic changelog generation, provide a truly easy-to-use experience for people.

Fair enough. I don't want to impose tools or workflows or anything, but in the cases where GH releases are used perhaps we can better map advisory data. I believe someone from my team may have already emailed you about how we can be of better aid to you. If it seems useful perhaps mention the idea of advisories on GH releases :)

I believe someone from my team may have already emailed you about how we can be of better aid to you.

Yup, things are work in progress. Thanks for the pointer, sounds interesting 👍🏻


add typescript/javascript worker

Add a typescript/javascript worker to avoid throwing unhandled error: unexpected usage when using Javascript or Typescript as a language. Solves issue #54

@siku2 Please take a look, let me know if I need to change or add anything.


[FAB-17438] CA Deployment Guide

First follow-on topic to the new Deployment a production network topic
with steps on how to deploy CAs using the CA and CA client binaries.
Signed-off-by: pama-ibm<EMAIL_ADDRESS>
Documentation update

Description
Provided 3 new topics that include:

Planning decisions for a CA including topology, TLS, enrollment and intermediate CAs
A CA checklist which describes the important settings in the CA configuration .yaml
Deployment steps for setting up a TLS CA, root CA, intermediate CA.

Related issues
Fabric Network Deployment Guide: https://jira.hyperledger.org/browse/FAB-11240

Somewhat cosmetic comment, the docs/source folder is getting awfully crowded. Might it be worth moving the deployment topics into another folder, similar to commercial paper to developing apps topic to keep things a bit cleaner Does not have to be done on this PR obviously.

Somewhat cosmetic comment, the docs/source folder is getting awfully crowded. Might it be worth moving the deployment topics into another folder, similar to commercial paper to developing apps topic to keep things a bit cleaner Does not have to be done on this PR obviously.

Agreed. I put all of these files in in their own sub-folder /deployguide

Overall the CA deployment section was great. I was able to deploy an intermediate CA in a an hour or two, which is a miracle. I found myself struggling with where certs were, and what was in each folder at any given time, and many of my comments are about that. I would use absolute paths for in the fabric-ca-client commands. That you you can set it once to point to the CA admin, and then forget it. You can also create a folder structure that is more descriptive of whose certs are where. ie, -M ${PWD}/organizations/peerOrganizations/org1.example.com/peers/peer0.org1.example.com/msp

@denyeart Thank you again for all your comments and questions. They have certainly made the content more robust. I am closing this PR now (without merging) and will create a new PR with the content in the Fabric CA docs repo as agreed. We can continue to iterate on the content over there as needed.

PR has been moved to Fabric CA repository https://github.com/hyperledger/fabric-ca/pull/146


Failed configure IPIP tunnel device, retrying... error=message too long

After restarting the host, tunlo has no IP address, and the current host cannot access the services of other nodes in the cluster
tunl0: flags=193<UP,RUNNING,NOARP>  mtu 1250
tunnel   txqueuelen 1000  (IPIP Tunnel)
RX packets 10203  bytes 6383285 (6.0 MiB)
RX errors 0  dropped 0  overruns 0  frame 0
TX packets 6271  bytes 437895 (427.6 KiB)
TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0
Expected Behavior

Current Behavior

calico.log
Possible Solution

Manually execute ifconfig tunl0 <IP_ADDRESS> up to resolve network issues
Steps to Reproduce (for bugs)

After installing the k8s cluster, after running for more than 100 days, restart one of the nodes to reproduce the problem

Context

Your Environment

Calico version
v3.23.3

Orchestrator version (e.g. kubernetes, mesos, rkt):
v1.24.3

Operating System and version:
CentOS Linux release 7.6.1810 (Core)

Link to your project (optional):

From the logs, this seems to be repeated a lot:
bird: KRT: Received route <IP_ADDRESS>/32 with strange next-hop <IP_ADDRESS>
2022-10-21 10:12:54.848 [WARNING][107] felix/ipip_mgr.go 163: Failed to list interface addresses error=message too long
2022-10-21 10:12:54.848 [WARNING][107] felix/ipip_mgr.go 141: Failed to set tunnel device IP error=message too long
2022-10-21 10:12:54.848 [WARNING][107] felix/ipip_mgr.go 88: Failed configure IPIP tunnel device, retrying... error=message too long
2022-10-21 10:12:55.849 [WARNING][107] felix/ipip_mgr.go 163: Failed to list interface addresses error=message too long
2022-10-21 10:12:55.849 [WARNING][107] felix/ipip_mgr.go 141: Failed to set tunnel device IP error=message too long
2022-10-21 10:12:55.849 [WARNING][107] felix/ipip_mgr.go 88: Failed configure IPIP tunnel device, retrying... error=message too long
2022-10-21 10:12:56.851 [WARNING][107] felix/ipip_mgr.go 163: Failed to list interface addresses error=message too long
2022-10-21 10:12:56.851 [WARNING][107] felix/ipip_mgr.go 141: Failed to set tunnel device IP error=message too long
2022-10-21 10:12:56.851 [WARNING][107] felix/ipip_mgr.go 88: Failed configure IPIP tunnel device, retrying... error=message too long
bird: KRT: Received route <IP_ADDRESS>/32 with strange next-hop <IP_ADDRESS>

I've not seen that sort of symptom before. Curious.

Yes, it thinks the message is too long


Check changelog in merge workflow

Before merging, I should check that the changelog parses

Fixed in #341.


Generating Duplicates

Hello,
First of all, good job on the code!
I'm having issues using your generator - it generates duplicates. I checked the metadata and DNA is the same for the duplicates.
I'm using lots of traits for conflicts - I'm not sure if this could be the issue.
I actually thought it might be because I was using multiple processes/threads, but I get duplicates even with only 1 process.
Am I missing a setting somewhere?

Hello,
First of all, good job on the code! I'm having issues using your generator - it generates duplicates. I checked the metadata and DNA is the same for the duplicates. I'm using lots of traits for conflicts - I'm not sure if this could be the issue. I actually thought it might be because I was using multiple processes/threads, but I get duplicates even with only 1 process.
Am I missing a setting somewhere?

I check the code, it's a dumb mistake I made when checking the dna, I'll fix it soon

@waitforitx
Fixed on v0.0.4
Please let us know if there is any other problems, and if you don't mind, after your nft is published, inform us the official link so we can add it in our readme 😁


Update introduction for Fn updates

Updates to introduction for changes to Fn design. Will be used in other updates.

Thanks, LGTM.


Scaffold template card

Hey, I just made a Pull Request!
Hold for:

svg assets
scaffolder client

:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[x] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

Would love a screenshot 😬

@stefanalund sure, not much to show tho without the assets :)

Can’t you use the Wave component inside the Header?

@stefanalund yea we could adapt it to work, but the requirements here talked about switching between different svgs for different types

I think that might have been a misunderstanding, the open source “bursts” don’t use svgs. They are implemented using CSS. There are pageTheme variants for all the different component types (services, websites, ...) that should be possible to use as-is here as well.

i think i'm a little confused by the open source “bursts” don’t use svgs so the 'bursts' are just gradients (which is what is in the linked theme)

The page themes are a combination between gradients and wave patterns. I was hoping the same could be used here.


Widget Rename

Suggested new feature or improvement
Allow custom naming of widgets
Scenario
Creating a pull request widget for 6 repos, each widget displays with the name, "Pull Request" I would like to change them to "[repoName] - Pull Request"
Additional details
No response

/dup https://github.com/microsoft/devhomegithubextension/issues/139


Fix emoji prefix for issue names

Issue #, if available: NA
Description of changes: Fix the prefix for the issue template so that the emoji/icon actually shows up now.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

A minor "hot fix"

A minor "hot fix"


Missing file events when connected to mounted network drive

VSCode Version: 1.5.3
OS Version: Windows 7 Ultimate x64

Steps to Reproduce:
Open any file on sidebar, webpack detected change and re-compile but it didn't show any changes by git.
Video:
http://recordit.co/P5h9fY1bjp

Reproduces without extensions: Yes

@nguyenbathanh does it reproduce if you set "files.useExperimentalFileWatcher": true ?

@bpasero Yes, it is still happening with that flag enabled or disabled.
I'm sorry, I'm missing one point that I'm working with SFTP mounted as network drive, may it be the problem?

@nguyenbathanh yes, we have some known issues around file watching via network drives.

@bpasero Thank you for your helpful information.

Duplicate https://github.com/Microsoft/vscode/issues/29807


Camera position resets to 0,0,0 after map

If u have movement script, after map ends camera position resets to 0,0,0. Works fine on 0.3.6 (what is in ModAssistant).
Camera2.zip
_latest.log
https://streamable.com/hc61yb

Fixed in 0.3.25


message_actions: Fix move messages/topic related notification-bugs.

While fixing issue #22973, I found couple of extra bugs. Fixed those in separate commits.
Bug 1:

After the fix

Bug 2
When 'resolve|unresolve' and 'change topic' actions occurs in
the same api call using 'topic sidebar icon', only 'topic_moved'
notification is sent.
Both 'topic moved' and 'topic resolved' notification should be generated.

After the Fix
Important
When we change 'topic name' and/or 'resolve|unresolve' topic using the messages-action move-message popover, select_stream_id is set to undefined. This is not the case when we the same thing via sidebar-topic-popover.
While using topic-popover, select_stream_id is not set to 'undefined',
even if we only change 'topic name' and/or 'resolve|unresolve' topic.
Resulting in no 'resolved_topic' notification.
The fix sets 'select_stream_id' to 'undefined' to fix the issue.

Bug 3 issue #22973

After the Fix

Important
This commit updates the logic of when and where to send 'topic resolve|unresolve' notification. Unlike previous logic, notification may be sent even in the case 'new_stream' is not None.
In general, 'topic resolved|unresolved' notification is sent to 'stream_being_edited'. In this particular case ('new_stream' is not None), notification is sent to the 'new_stream' after check.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@sahil839
I have improved the commit prefix and summary. Also, the minor code logic change you suggested.
Let me know for any further improvements.
Thanks.

Posted one minor comment. LGTM otherwise.

Made the changes -- Thanks for the review.

@timabbott
This PR has been reviewed by sahil839.
I would appreciate your final review and merge, if it looks good to you.

@timabbott This PR has been approved by @sahil839 .
I think it'll be more efficient for you to review this directly, and we can discuss live as needed, but if you need me to figure out what's going on here, let me know.

The changes here look good, though I found the commit messages and explanations a bit hard to read -- so I encourage you to work on polishing the prose explanations for your next PRs.
Merged, though @alya there's one quirk we could decide to change, which is that the "from ..." text there shows the old topic in its still-resolved state. This is probably the most useful option, because we're linking to the previous location for these messages, but does have the property that it's different output than if the topic had been resolved and then moved.

I'm a bit confused about the question. Do you mean unresolved and then moved? If so, the current solution seems right to me.

Yes, that's what I meant.


Locker X-Ray

You can see landmines, enemy's and other interactable objects thru the lockers.

Is this caused by a mod or something? I have these meshes set to the same layer Zeekers has his area props. I have never seen this behavior and just tested again to make sure:

Might be a mod called CullFactory's fault. Can't test anything right now thou. I'll get back to you later.


Add holiday features

Reference Issues/PRs
Fixes #1675
What does this implement/fix? Explain your changes.

adds HolidayFeatures

Does your contribution introduce a new dependency? If yes, which one?
soft dependency: holidays https://pypi.org/project/holidays/
What should a reviewer concentrate their feedback on?
Any other comments?
PR checklist
For all contributions

[x] I've added myself to the list of contributors.
[x] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[x] I've added unit tests and made sure they pass locally.

For new estimators

[x] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

Won't be able to finish this.


[Bug]: changes to path env variable seem not to be used (in connection with ffi-napi and Windows)

Version
29.3.1
Steps to reproduce

Clone repo https://github.com/mathiasbockwoldt/jest-bug-example
cd jest-bug-example
npm install
node index.js  (should print path and dll version without error)
node .\node_modules\jest\bin\jest.js --silent=false (should print same path but error "Dynamic Linking Error: Win32 error 126" which means that the dll could not be found, which is due to the path env variable not being set correctly)

Make sure to run steps 4 and 5 not in a terminal from within VSCode, as VSCode injects the current path to the path env variable.
Expected behavior
I expect the test run with jest to work without error.
Actual behavior
Using jest to run, the index.js will print the correct path env variable, but behave as if the path env variable was not set correctly.
Additional context
Maybe connected: #3552
Some more context is described in the readme in the repo https://github.com/mathiasbockwoldt/jest-bug-example
Environment
System:
  OS: Windows 10 10.0.19045
  CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz
Binaries:
  Node: 18.12.1 - ~\scoop\apps\nvm\current\nodejs\nodejs\node.EXE
  npm: 8.19.2 - ~\scoop\apps\nvm\current\nodejs\nodejs\npm.CMD
npmPackages:
  jest: ^29.3.1 => 29.3.1

Sorry folks, looks like the problem was on my side. I looked at the wrong places. The path variable was fine all along also in jest, but some lines above, I try to include a dll with a relative path and that won't work with jest and also not from any arbitrary script that might include my script.