How can I check if the consumer group is initialised?

Hi folks,
I would like to perform a certain action when Consumer group is not initialized. Which function/variable could I use to obtain such information?

I don't understand your question. What do you mean by a consumer group not being initialized? Could you show an example to show what you want to achieve and how you are trying to achieve it?

I don't understand your question. What do you mean by a consumer group not being initialized? Could you show an example to show what you want to achieve and how you are trying to achieve it?

Hi @Nevon ,
After we invoke consumer.disconnect(), if we try to invoke consumer.resume(), Kafkajs will throw an error KafkaJSNonRetriableError: Consumer group was not initialized, consumer#run must be called first.
Therefore, I would like to find a way to check if I can check if the consumer is initialised before calling consumer.resume.

Hi @Nevon ,
After we invoke consumer.disconnect(), if we try to invoke consumer.resume(), Kafkajs will throw an error KafkaJSNonRetriableError: Consumer group was not initialized, consumer#run must be called first.
Therefore, I would like to find a way to check if I can check if the consumer is initialised before calling consumer.resume.

@lzhuor you are not using resume and disconnect in the right way, from what I can see you want to pause and resume the consumer, not disconnect and resume. When calling disconnect you shutdown the consumer completely, so you would have to initialize it again subscribing to topics, calling consumer.run, etc.

@lzhuor you are not using resume and disconnect in the right way, from what I can see you want to pause and resume the consumer, not disconnect and resume. When calling disconnect you shutdown the consumer completely, so you would have to initialize it again subscribing to topics, calling consumer.run, etc.

Thanks @tulios . Nope, I clearly understand the difference between resume, pause and disconnect  and I am using them in the right way.
I am asking if there is a way to check if a consumer has been disconnected or not for my usage.

Thanks @tulios . Nope, I clearly understand the difference between resume, pause and disconnect  and I am using them in the right way.
I am asking if there is a way to check if a consumer has been disconnected or not for my usage.

@lzhuor there is no such param in place as of now, but you can keep track of that on your side by listening to the DISCONNECT instrumentation event


[updatecli 0.23.x] failing to open pull request + no error reported globally

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
With updatecli 0.23.0 and 0.23.1, we have errors when trying to open a pull request, with the error ERROR: 1 target(s) (updateChartVersion) failed for pullrequest "default".
Besides this failure, the execution of updatecli does NOT fail and does NOT report the failure, which makes it hard to detect.
Expected Behavior
Can't be sure of the root cause error, but we have to fix updatecli so that, at least, it reports the failure, and eventually give details on the failure.
Steps To Reproduce

Any manifest at https://github.com/jenkins-infra/kubernetes-management/tree/56ff776b83f8677e16f8e8c9d422c7d3a7205a1f/updatecli/updatecli.d

In particular (a simple one):

title: Bump aws-node-termination-handler helm chart version

scms:
  default:
    kind: github
    spec:
      user: "{{ .github.user }}"
      email: "{{ .github.email }}"
      owner: "{{ .github.owner }}"
      repository: "{{ .github.repository }}"
      token: "{{ requiredEnv .github.token }}"
      username: "{{ .github.username }}"
      branch: "{{ .github.branch }}"

sources:
  lastChartVersion:
    kind: helmChart
    spec:
      url: https://aws.github.io/eks-charts
      name: aws-node-termination-handler

targets:
  updateChartVersion:
    name: "eks/aws-node-termination-handler Helm Chart"
    kind: file
    scmid: default
    spec:
      file: clusters/cik8s.yaml
      matchpattern: 'chart: eks\/aws-node-termination-handler((\r\n|\r|\n)(\s+))version: .*'
      replacepattern: 'chart: eks/aws-node-termination-handler${1}version: {{ source "lastChartVersion" }}'

pullrequests:
  default:
    kind: github
    scmid: default
    targets:
      - updateChartVersion
    spec:
      labels:
        - dependencies
        - aws-node-termination

Executing the diff works as expected and succeeds to report expected change + PR:

updatecli diff --values updatecli/values.yaml --config updatecli/updatecli.d/charts/aws-node-termination-handler.yaml

+++++++++++
+ PREPARE +
+++++++++++

Loading Pipeline "updatecli/updatecli.d/charts/aws-node-termination-handler.yaml"

SCM repository retrieved: 1

++++++++++++
+ PIPELINE +
++++++++++++

########################################################
# BUMP AWS-NODE-TERMINATION-HANDLER HELM CHART VERSION #
########################################################

SOURCES
=======

lastChartVersion
----------------
✔ Helm Chart 'aws-node-termination-handler' version '0.18.1' is found from repository https://aws.github.io/eks-charts

Remark: We couldn't identify a way to automatically retrieve changelog information.
Please use following information to take informed decision

Helm Chart: aws-node-termination-handler
A Helm chart for the AWS Node Termination Handler.
Project Home: https://github.com/aws/eks-charts
Require Kubernetes Version: >= 1.16-0
Version created on the 2022-04-07 17:03:20.030632404 +0000 UTC

Sources:

* https://github.com/aws/aws-node-termination-handler/

* https://github.com/aws/eks-charts/

URL:

* https://aws.github.io/eks-charts/aws-node-termination-handler-0.18.1.tgz

CHANGELOG:
----------

Remark: We couldn't identify a way to automatically retrieve changelog information.
Please use following information to take informed decision

Helm Chart: aws-node-termination-handler
A Helm chart for the AWS Node Termination Handler.
Project Home: https://github.com/aws/eks-charts
Require Kubernetes Version: >= 1.16-0
Version created on the 2022-04-07 17:03:20.030632404 +0000 UTC

Sources:

* https://github.com/aws/aws-node-termination-handler/

* https://github.com/aws/eks-charts/

URL:

* https://aws.github.io/eks-charts/aws-node-termination-handler-0.18.1.tgz

TARGETS
========

updateChartVersion
------------------

**Dry Run enabled**

⚠ updated the [dry run] content of the file "/var/folders/2s/09szbrgn22l2tcvxz1_k34g40000gn/T/updatecli/jenkins-infra/kubernetes-management/clusters/cik8s.yaml"

--- /var/folders/2s/09szbrgn22l2tcvxz1_k34g40000gn/T/updatecli/jenkins-infra/kubernetes-management/clusters/cik8s.yaml
+++ /var/folders/2s/09szbrgn22l2tcvxz1_k34g40000gn/T/updatecli/jenkins-infra/kubernetes-management/clusters/cik8s.yaml
@@ -51,6 +51,6 @@
   - name: aws-node-termination-handler
     namespace: eks
     chart: eks/aws-node-termination-handler
-    version: 0.18.0
+    version: 0.18.1
     values:
       - "../config/ext_aws-node-termination-handler.yaml"

PULL REQUESTS
=============

[Dry Run] A pull request with kind "github" and title "[updatecli] Bump aws-node-termination-handler helm chart version" is expected.

=============================

REPORTS:

⚠ AWS-NODE-TERMINATION-HANDLER.YAML:
        Sources:
                ✔ [lastChartVersion]  (kind: helmChart)
        Target:
                ⚠ [updateChartVersion] eks/aws-node-termination-handler Helm Chart (kind: file)

Run Summary
===========
Pipeline(s) run:
  * Changed:    1
  * Failed:     0
  * Skipped:    0
  * Succeeded:  0
  * Total:      1

Applying the change fails at the Pull Request level, despite being reported as success:

updatecli apply --values updatecli/values.yaml --config updatecli/updatecli.d/charts/aws-node-termination-handler.yaml

+++++++++++
+ PREPARE +
+++++++++++

Loading Pipeline "updatecli/updatecli.d/charts/aws-node-termination-handler.yaml"

SCM repository retrieved: 1

++++++++++++
+ PIPELINE +
++++++++++++

########################################################
# BUMP AWS-NODE-TERMINATION-HANDLER HELM CHART VERSION #
########################################################

SOURCES
=======

lastChartVersion
----------------
✔ Helm Chart 'aws-node-termination-handler' version '0.18.1' is found from repository https://aws.github.io/eks-charts

Remark: We couldn't identify a way to automatically retrieve changelog information.
Please use following information to take informed decision

Helm Chart: aws-node-termination-handler
A Helm chart for the AWS Node Termination Handler.
Project Home: https://github.com/aws/eks-charts
Require Kubernetes Version: >= 1.16-0
Version created on the 2022-04-07 17:03:20.030632404 +0000 UTC

Sources:

* https://github.com/aws/aws-node-termination-handler/

* https://github.com/aws/eks-charts/

URL:

* https://aws.github.io/eks-charts/aws-node-termination-handler-0.18.1.tgz

CHANGELOG:
----------

Remark: We couldn't identify a way to automatically retrieve changelog information.
Please use following information to take informed decision

Helm Chart: aws-node-termination-handler
A Helm chart for the AWS Node Termination Handler.
Project Home: https://github.com/aws/eks-charts
Require Kubernetes Version: >= 1.16-0
Version created on the 2022-04-07 17:03:20.030632404 +0000 UTC

Sources:

* https://github.com/aws/aws-node-termination-handler/

* https://github.com/aws/eks-charts/

URL:

* https://aws.github.io/eks-charts/aws-node-termination-handler-0.18.1.tgz

TARGETS
========

updateChartVersion
------------------
⚠ updated the content of the file "/var/folders/2s/09szbrgn22l2tcvxz1_k34g40000gn/T/updatecli/jenkins-infra/kubernetes-management/clusters/cik8s.yaml"

--- /var/folders/2s/09szbrgn22l2tcvxz1_k34g40000gn/T/updatecli/jenkins-infra/kubernetes-management/clusters/cik8s.yaml
+++ /var/folders/2s/09szbrgn22l2tcvxz1_k34g40000gn/T/updatecli/jenkins-infra/kubernetes-management/clusters/cik8s.yaml
@@ -51,6 +51,6 @@
   - name: aws-node-termination-handler
     namespace: eks
     chart: eks/aws-node-termination-handler
-    version: 0.18.0
+    version: 0.18.1
     values:
       - "../config/ext_aws-node-termination-handler.yaml"

no changed file to commit

PULL REQUESTS
=============

ERROR: 1 target(s) (updateChartVersion) failed for pullrequest "default"

=============================

REPORTS:

✔ AWS-NODE-TERMINATION-HANDLER.YAML:
        Sources:
                ✔ [lastChartVersion]  (kind: helmChart)
        Target:
                ✗ [updateChartVersion] eks/aws-node-termination-handler Helm Chart (kind: file)

Run Summary
===========
Pipeline(s) run:
  * Changed:    0
  * Failed:     0
  * Skipped:    0
  * Succeeded:  1
  * Total:      1

Environment
- OS: macOS, Ubuntu 20.04
- updatecli: 0.23.0, 0.23.1

Anything else?
No response

Tested with my unpublished version, it opens the PR (tested locally)

While debugging, I stumbled on the following line: https://github.com/updatecli/updatecli/blame/f648e74d975b948137b0d66947c5d0e8d289d789/pkg/core/pipeline/pullRequests.go#L80 .
Ping @olblak do you remember the reason for ignoring when a target fails for a given pull request?
At first, it seems a wrong behavior to me: I would prefer the pull request to fail if any of its targets fails.
(anyway: we have to find why does the target fails)

@lemeurherve ack, I'm handing it over to you then. i'll be available for review then \o/ thanks!

So... I've tested a build from main (commit f648e74) and... it works: https://github.com/jenkins-infra/kubernetes-management/pull/2212

so we have to find the difference between your environment, and mine + CI. Good job 👍

Also working from https://github.com/updatecli/updatecli/releases/tag/v0.23.1 (Darwin)

Another question (consequence of #388 ) for @olblak : it seems that https://github.com/updatecli/updatecli/blame/f648e74d975b948137b0d66947c5d0e8d289d789/pkg/core/pipeline/target/main.go#L144 does NOT report the error when a target fails. Return nil seems wrong.
(still able to have the issue failing, debugging with @lemeurherve as the root cause is in the file resource at first sight)

Aaaand got the root cause: the plugin file always returns an empty list of (modified) files. That should be easy to fix.
It means we need 3 PRs to fix the issue here:

[ ] returns the changed files in the resource "file"
[ ] return a failure when running targets that have no changed files but no "success" target result (state incoherencey)
[ ] report a failure on PRs when there are failed targets.

report a failure on PRs when there are failed targets.

I'd like to be able to target multiple files at once and have a PR even if some of these files are already up to date, but that seems incompatible with this requirement.
@dduportal WDYT?

report a failure on PRs when there are failed targets.

I'd like to be able to target multiple files at once and have a PR even if some of these files are already up to date, but that seems incompatible with this requirement. @dduportal WDYT?

not at all: a target which does not change a file should be in the "success" category.
For targets:

result.Success  == the file is in the expected state, no change to be done.
result.attention == the file has to be changed to reach the expected state
result.skipped == the target should not be executed (not even trying to check the state)
result.failed == an error happennend while running the target  (case here: the "commit" state failed)

Starting on the 3rd PR then 😃


Cmd+Enter sometimes doesn't commit in SCM input box

I've been getting reports (cc @jrieken) that Cmd+Enter no longer commits in the SCM input box. @jrieken found out that the keybinding was being dispatched to search.action.openInEditor. And it indeed is:
https://github.com/microsoft/vscode/blob/1f06066a9c57b789ef3f72371dc9541387fa1d31/src/vs/workbench/contrib/searchEditor/browser/searchEditor.contribution.ts#L170-L175
So, the only way I can see this happening is if SearchViewFocusedKey is erroneously set.
Having a global SearchViewFocusedKey is problematic and a perf issue since it requires having a FocusTracker tracking focus, which must work flawlessly. The whole point with the Context Key Service is that it operates on scoped DOM nodes. You can simply have a SearchViewFocusedKey key which is always true but set on a scoped context key service anchored on the search viewlet DOM node. That way if Cmd Enter is invoked when the document/activeElement is a descendant of the search viewlet DOM node, it will always evaluate to true.

Alternatively (and maybe better) you can use the FocusedViewContext, e.g like so: https://github.com/microsoft/vscode/blob/35b26f49b7f7de8267ee34387a9df65f2942d8d3/src/vs/workbench/contrib/bulkEdit/browser/bulkEdit.contribution.ts#L167

@jrieken I was thinking about that and how it must be implemented the same way as the current SearchViewFocusedKey, so it might also run into the same issues.

Currently it likely is (cc @sandy081) but when all use the same content key for logical the same thing then we only need to do a solid implementation once.

Similar bug was fixed with #88803, maybe theres another layer of bug here. I will look in to using the scoped context keys.

The events that trigger FocusedViewContext changes also trigger SearchViewFocusedKey changes, per-view. But FocusedViewContext changes across-views overwrite each other, so there are likely cases where SearchViewFocusedKey is true at the same time FocusedViewContext is not SEARCH_VIEW_ID.
We only use FocusedViewContext in one other place so I think moving to the less error-prone scoped context keys makes sense.

Just FYI focusedView context key is also used by the extensions for their custom views and have not seen any issues yet.
I see the glitches with focus tracker so I think the right approach would be is to have consistent and reliable focus tracker

Verified by code review and lack of issues.


All applicable string comparisons should be probably occur via ascii.eqlIgnoreCase()

While I agree that there may be a few places that this could be applicable, I don't think its good practice to use as the standard. This is particularly true for parsing. Windows is a little lenient on the casing for commands and options, but POSIX systems are not. Those systems treat differently cased strings as different commands or options (or files, or whatever else). For instance --use and --USE could meant two different things.
All that said, are there certain places you'd like to see case-insensitive equality checks? I definitely wouldn't mind taking a look at some friction points you might be having due to the way checks are being done now.

See, in my opinion flags should NEVER be case sensitive--even if values often should be (notably URIs of any kind)--as this adds to end user confusion and makes the command line input less flexible in bizarre situations where this might be out of their control (I can't think of a valid example for this, but being able to use caps anywhere seems beneficial to me).
Regardless, I'll be rewriting all functions pertaining to the behavior and printing of my CLI for my apps, as you've so graciously made available, so your decision here doesn't especially affect me. I recommend giving https://clig.dev/ a read if you haven't already (though you probably have). I haven't read it all myself and don't agree with everything necessarily, but it seems to have loads of interesting ideas pertaining to this.

That's fully understandable. Thankfully the callback functions help meet that need as you've seen (and tested thoroughly in a way that I'm very appreciative of).
I've actually never read or seen that site! When I initially started Cova, I decided to do it naively in a way that just made sense to me to see how it would turn out. I think I've reached a point where reading through something like that could be very beneficial though, so thank you! You may see some features referencing that site after the v0.9.0 release.

I'm partial to A in this case. That's both a reasonable and fairly simple change to make.
It also fits in well with some changes I'm considering based on our conversations through a few issues. As a heads up to you, I'm likely going to do the following for fields that can be set via both a Config Struct and directly on an instance of an Argument Type:

No prefix: This is reserved for fields set directly on the instance of an Argument Type.
child_type_ prefix: This will be for fields in a Config Struct that are set based on a specific Child Type (Options and Values only).
global_ prefix: This will be for fields in a Config Struct that are set for ALL instances of the corresponding Argument Type. Think of it as changing the default behavior for that Argument Type.

When being processed the order of precedence will be as listed above ending in the default behavior of Cova.
The intent here is to make the usage of these fields a bit more clear without needing to refer to the Docs as often. More to follow on that.

No objections here! And I appreciate the heads up.

Closed w/ 2aa9c74.


Add documentation and default for web.service.externalPort value

When enabling an Ingress, this field is required, but is not mentioned in either the documentation or the default values.yaml file.

not relavant

https://github.com/aquasecurity/aqua-helm/blob/master/server/templates/web-ingress.yaml#L3
https://github.com/aquasecurity/aqua-helm/blob/master/server/templates/web-ingress.yaml#L32-L37
Seems pretty relevant?


LSTM for Classificatioin Theory Question

model = Sequential()
model.add(Embedding(max_features, 128, input_length=maxlen))
model.add(LSTM(128))  # try using a GRU instead, for fun
model.add(Dropout(0.5))
model.add(Dense(1))
model.add(Activation('sigmoid'))
model.compile(loss='binary_crossentropy',
optimizer='adam',
class_mode="binary")

Hey, guys. I've try the code examples/imdb_lstm.py mentioned above. But I don't understand the layer structures between the LSTM layer and activation layer.
We know for a text sequential input we will get a variable length output in LSTM layer, here we get a 128 * nb_words matrix as I see, and the nb_words must be variable because different length texts.
Then the model add a full connection layer with output size of one. HERE is the question confuses me: IF (1) there are nb_words full-connection neurons with 128 input and 1 output,  (2) OR one neuron with 128*nb_words input and 1 output ??? IF(2), I think it's impossifble to learn a full-connection layer with variable weight size. IF (1), the same question will happend for activation layer, then how to get a final 1 dim output of the total network?
So, I'm confused about how to connect a length variable layer(LSTM) with length-fixed layer(full-connection )?  Thank you !

The LSTM input and output is always fixed. If you have variable input data, you can use zero padding and masking to make it the same length, or simply cut your data into fixed-length samples manually. In this case it is an N-to-1 mapping, i.e. your input has shape (nb_samples, timesteps, feature_dim) and your output has shape (nb_samples, feature_dim). That's because, by default, return_sequences=False.
A nice introduction to RNNs is given here.

@rpinsler So in this case the timestep for LSTM layer is fixed. The first LSTM layer has timesteps LSTM cells and each cell outputs a 128 dims vector. But I don't think it's general to fix the timestep for LSTM network. I got it. Thank you.

@rpinsler Thank you.


Add hotkeys to toggle mouse and keyboard overlay while recording

Add options to toggle mouse and keyboard overlay while recording. Specially keyboard toggle would be useful to trace special keys which can't be screen (like shortcuts in applications).

Done! Will be available in next release.

Mouse click and keyboard overlays can be toggled from the buttons on the interface.
I did not get that you wanted hotkeys for that.
I think there might be a problem though. The hotkey might show up in the video when you will be turning off keyboard overlay.

I think that hotkey for keyboard overlay toggle should be silently ignored. This is just matter of documentation and in my opinion would be at least surprise. Just like all other hotkeys defined in Captura I don't expect them to be seen in keyboard overlay (like stop recording).
Main problem is that when trying to record a tutorial for a program which has both - hotkeys and writable text (source code for example) I would like to avoid tracking source code writing in my video since it doesn't make much of a sense at that point.
For a video it would look a bit strange if I move mouse out of screen to toggle those. Or even worse there is need to popup Captura window to toggle overlays.

Ok. I'm working on it.
BTW, I would like to know your preferences on the default hotkeys for mouse and keyboard overlay toggling.

Maybe to be inline with current defaults Alt+F10 and Alt+F11 (Or Shift+F10) and I think they could be off by default.

It's still not perfect but please test the latest dev build
https://ci.appveyor.com/api/buildjobs/h01by8fobd5603cu/artifacts/temp%2FCaptura-Portable.zip

Otherwise works perfectly except that when enabling keyboard/mouse modifier keys (ctrl/alt/shift) are recorded upon release.

After several trials, it seems that it is not possible to prevent the modifier keys from being recorded due to Captura writing the video in real time.


game crashes upon injection

it might be the injector im using. can you link a reccomended injector?

it was my injector, used process hacker to inject now and its working fine


SCRUM-1694 2

router now tested manually.

@ianlongden  Ah, well, no, the POST workflow endpoint is working, but the reference GET now gives a 500 error


New version: FontForge.FontForge version 20230101

- [X] Have you signed the Contributor License Agreement?

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.2 schema?

Resolves #93519

Microsoft Reviewers: Open in CodeFlow

I already closed my PR but this will colide with version number reported by appwiz.cpl

I already closed my PR but this will colide with version number reported by appwiz.cpl

No, it won't collide due to the AppsAndFeaturesEntries specified in the installer manifest

Oh I missed it. My bad.


Add CakePHP Example

Please add CakePHP example
Like this https://book.cakephp.org/3.0/en/installation.html#nginx

Seeing your link, it seems like a basic PHP-FPM config (https://caddyserver.com/docs/fastcgi)
fastcgi / <IP_ADDRESS>:9000 php


Torsion Sampling

Type
System sampling check
Question
Is phase space well sampled?
How
Minima sampled ~60%
Tool
MDA + rdkit
Note
Ideally what you should be looking at here is a combination of the solvent, vacuum, and complex differences and the variation in dihedral sampling across lambda states. I.e. is there a major change happening in my transformation that isn't being sampled across all states -- @IAlibay

@jameseastwood  I somehow di dnot manage to link this to the PR, that closes this task. ... But there is now a new "task" in the project linked to the pr. shall we close this one then? :)


AbstractCopyTask needs a ::from overload that takes some sort of deferred action

In groovy you can create a copy task like this:

task extractSources(type: Copy) {
    from { // use of closure defers evaluation until execution time
        configurations.source.collect { zipTree(it) }
    }
    into "$projectDir/src/main/resources"
}

However, in gradle script kotlin there is no good way of doing this without doing some wacky closure magic.
The best solution is this:
task<Copy>("extractCustomizationSchema") {
    from(configuration.map { tarTree(it) })
    into("$buildDir/schema/schema_${it.second}")
}

Which of course pushes the resolution of the dependency to configuration time instead of run time.
Expected Behavior
A type safe way of deferring resolution of a file in a copy task to task execution time.
Your Environment
distributionUrl=https\://repo.gradle.org/gradle/dist-snapshots/gradle-script-kotlin-3.5-20170305000422+0000-all.zip

An overload is not technically required since a java.util.concurrent.Callable instance can be used to defer resolution:
from(
  Callable {
      configurations["source"].map { zipTree(it) }
  })

An overload would make for a better API though.

@bamboo Would this help as another getting started doc in the gradle script kotlin repo?

@bamboo I'll assign this issue to you for any follow up actions or uh... closure.

I'd like to add strongly typed methods for Callable so you can do:
from { [whatever] }

In the kotlin DSL without having to know that you need to import and use Callable.
I'm thinking the easiest way to do this is by adding default methods to the CopySpec and the CopyProcessingSpec.

An alternative could be to add an extension function, but then that would only exist for the Kotlin DSL.
@bamboo What are your thoughts on adding this as an extension function?

I'd prefer a map {} overload on FileCollection, which keeps the build dependencies intact and doesn't eagerly resolve. We have something similar for filter, plus and minus

That would be nice, but it would also be an API breaking change because for kotlin DSL users you'd be changing the behaviour of the map function in their code.
You'd want to do this before Gradle 5.0.

It is eager in kotlin. And the return type of map in kotlin is List<File>. So if you make it lazy and have a different return type. So, yes it will be a behavioral and return type change for newly compiled code.
https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.collections/map.html
inline fun <T, R> Iterable<T>.map(
    transform: (T) -> R
): List<R>


Added limit order function

Added limit order function with endpoints
closes issue number #333

Please check the PR @ighoshsubho


OS Integration Server - Better handling of operating system specific issues

At the moment in LM we have seperate systems for handling OS specific issues:

A system that splits focus management / window visibility
A system that splits default window settings
A system that splits key execution
A system that splits key retrieval (though that ofc is in its own package)

It is forseeable that in the future we will have more of these systems which will create mroe operating system specific implementations. It would be useful to bundle all operating specific behaviour into its own package which implements some interface.
const MacLibrary: IOperatingSystemLibrary = {
  appShow(...){
    //specific implementation for showing the application
  },
  appHide(...){
    //specific implementation for hiding the application
  },
  keySend(...){
    //specific implementation for sending keys
  },
  getDefaultKeyBinding(){
    //gets the default key bindings of things like copy/paste/cut/undo/redo etc.
  }
  //...
}

Benefits

Supporting new operating systems becomes easier (i.e. just have to implement this interface)
Main package code becomes easier to use

Concerns

Standardisation leads to a little more effort in implementing OS specific behaviour
multiple different packages (e.g. window-manager-applet, core) will rely on OS Specific bundle packages, so you get slightly harder coupling.

Analysis of existing use of isPlatform():
const wordJumpModifier = isPlatform("mac") ? "alt" : "ctrl";
const menuNavigationModifier = isPlatform("mac") ? "ctrl" : "alt";
const cmdModifier = isPlatform("mac") ? "meta" : "ctrl";

//alt+keys --> ˚´¥ß
const isNormalKeyPress = !event.ctrl && !event.meta && (!event.alt || isPlatform("mac"));

const trayIcon = ... Path.join(__dirname, "..", "..", "images", isPlatform("mac") ? "macTrayIcon.png" : "trayIcon.png")

const handleMetaIssues = ... //On Mac: key up events of keys are not sent while meta is down.

const areListenersSupported = ... (!isPlatform("mac") || remote.systemPreferences.isTrustedAccessibilityClient(false))

  setting.windowShadow: isPlatform("mac") ? false : true
  openLM: !isPlatform("mac") || remote.systemPreferences.isTrustedAccessibilityClient(false) ? "meta+space" : "meta+l"

// ./packages/core/src/actions/types/copyPaste/sendKeys/sendOSKeys.ts
// Send the normalized key press using an appropriate system depending on OS
if (process.platform == "win32") {
 await sendWindowsKeys(normalizedKeys);
} else if (process.platform == "darwin") {
 await sendMacKeys(normalizedKeys);
}

//./packages/launcher/src/index.ts
const installDir = process.platform == "darwin" ? `${OS.homedir()}/Library/Application Support/LaunchMenu` : process.cwd();

// Ask for permissions on mac if needed
if (process.platform == "darwin") await handleMacPermissionsDialog(window);
if (process.platform == "win32") await handleWindowsPermissionsDialog(window);

//./packages/core-applets/window-manager/src/startup/setupStartupController.tsx
const installer = startupControllers[process.platform]?.();

//./packages/launcher/src/installerWindow/WindowUI.tsx
{process.platform == "win32" ? "Windows+Space" : remote.systemPreferences.isTrustedAccessibilityClient(false) ? "Command+Space" : "Command+L"}

//./packages/core-applets/window-manager/src/visibility/setupVisibilityControls.tsx
const visibilityController = visibilityControllers[process.platform]?.();

//./packages/dev-tools/build-tools/index.js
const electronPath = Path.join(resolvePackageDir("electron"), process.platform == "win32" ?  "dist/electron.exe" : "/dist/Electron.app/Contents/MacOS/Electron");

4 respective to alternate keys (1 of which is based on node-key-server usage)
2 fixing mac bugs / differences in the runtime
1 for alt+keys on mac
1 for image path
1 for style setting
1 sending OS keys
1 launch at StartUp handler
1 asking for permissions
1 install directory
1 visibility controller
1 electron path

Most of these can be implemented directly in the TypeScript environment, however there are also many things which will require deeper OS integration. Things like global key handling (currently different module) and sending keys (especially given #152).
For this reason we should look into a better architecture for the Windows/Mac servers. Something like this:

For feature - InstantSend:
Mac (JXA)  Obtain selected items in Finder:
Application('Finder').selection().map(e=>e.url())

Windows (VBA)  Obtain selected items in Explorer
Sub test()
    Dim shWindow As Object 'IDispatch
    For Each shWindow In CreateObject("Shell.Application").Windows
        If shWindow.Name = "File Explorer" Then
            'If shWindow.Hwnd = 0 Then   'Target specific window

            'Print location of window
            Debug.Print shWindow.locationUrl
            Dim item As Object 'IDispatch
            For Each item In shWindow.document.SelectedItems
                'Print paths of selected items
                Debug.Print item.Path
            Next
        End If
    Next
End Sub


Colab Runtime Error - Jaxlib version mismatch (strikes again)

Hi all
When running the Colab Notebook, at the import step, the Jax and Jaxlib modules are causing an error.
RuntimeError: jaxlib is version 0.3.25, but this version of jax requires version >= 0.4.2.

We've had a similar issue around the 12th of September with Jax, Jaxlib and Flax. Probably an update in Colab or one of those dependencies has made this resurface.
In the meantime, users who are experiencing issues can circumvent this error using following janky solution:
!pip install -qqq jax==0.4.2
!pip install -qqq jaxlib==0.4.2

Limitation: This will run the beam search on the CPU of the Colab, thus increasing inference times.
You can make the GPU kick in again by using instead:
!pip install -qqq jax==0.4.2
!pip install -qqq jaxlib==0.4.2
!pip install -qqq "jax[cuda11_cudnn82]" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html

I have used the following examples to test inference using the aforementioned janky solutions:
ωρανια.   αστ?ρας ηρε?νησα σο??ι φρ?νι π??ρι τε εοι?ος   ουν?μα εχ?. λε??μαι δε η δι?ς ουρα?ιη.   ονε?του.

and
λ??ουσιν ἃ θέλου??ν λεγέ??σαν οὐ μέλι ??ι σὺ φίλι με συνφέ?ι σοι

Kind regards!

Robbe, our team would like to share a big thank you for all your help and support! We've updated the minimum JAX version and the problem should be now resolved.


Add new setup

Make this capable of setting up projects for our new deployment method.

[x] Run tests on Python 3.6
[ ] Single settings.py file

[ ] ini files for config

[x] requirements
[x] setup.py
[x] editorconfig
[x] bumpversion
[x] Makefile
[x] BitBucket pipelines

Nice, but perhaps as a separate PR:

[ ] Incremental updates to existing projects

Django doesn't have a 'force' option, it stops when existing files clash with new files created.
Suggestion by Dennis: run start project in a temp dir, then move the files to the actual target.

wat, ik wilde helemaal niet approven :(

Ik vroeg ook niet eens om jouw approval!

In 56bfac0 a .py file contained an import with from {{ project_name }}.search import views
Which fails the flake8 check. We could make flake8 ignore the template directories and in the travis tests simply initiate two projects (default/legacy) and run flake8 on the results, i.e. on the 'rendered' templates. This would be more correct, but at the moment not necessary because I changed it to a relative import.


Update C419 to be a suggested fix

Per discussion in https://github.com/charliermarsh/ruff/issues/4320
Part of https://github.com/charliermarsh/ruff/issues/4184

Technically, we're changing eager to lazy, e.g.:
from typing import Generator

def count() -> Generator[int, None, None]:
    for i in range(4):
        print(i)
        yield i

if __name__ == "__main__":
    print("pre fix")
    any([i > 0 for i in count()])
    print("after fix")
    any(i > 0 for i in count())

pre fix
0
1
2
3
after fix
0
1

I expect that this being a problem is rare, it would mean that your code relies on a generator being used to completion. A more common problem could be that we don't resolve the call path yet and sometimes people rebind any/all:
https://github.com/charliermarsh/ruff/blob/c10a4535b9549d8dab6dec024ce3231b74bf5e18/crates/ruff/src/rules/flake8_comprehensions/rules/unnecessary_comprehension_any_all.rs#L72
I think suggested is the correct category.


Email 2fa

Revive #267 thanks to @Atterratio for the initial patch. I did some changes based on the reviews on the original pull requests and some doc style fixes.

Description

Make possible two-factor authorization using emails.
Motivation and Context

Although this way of two-factor authentication is not very reliable, it can be preferable for some users.
How Has This Been Tested?

Tested in a local project. Not fully tested but I tried it out. Didn't look closely at old PR.
Screenshots (if appropriate):
Content-Type: multipart/alternative;
 boundary="===============1209054917869947523=="
MIME-Version: 1.0
Subject: Authentication token email
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Date: Wed, 28 Jul 2021 04:20:20 -0000
Message-ID<EMAIL_ADDRESS>
--===============1209054917869947523==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit

Hello,
Your email address has been given for two-factor authorization on the our website.
If you did't do this, just ignore this message.

Authentication token for user hello is 487008.
--===============1209054917869947523==
Content-Type: text/html; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit

Hello,

        Your email address has been given for two-factor authorization on the our website.
        If you did't do this, just ignore this message.

Authentication token for user hello is 487008.

--===============1209054917869947523==--

-------------------------------------------------------------------------------

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Hi @pickfire,
Thanks for reviving this and continuing the effort from #267. I am in the midst of considering bolting on a custom implementation using email, however would prefer to adjust/extend the use of this package, so if this is merged, it would greatly help.
It looks like a review is required and is the blocker at the moment. Perhaps you could add @Atterratio as the reviewer?
Thanks again!

It looks like a review is required and is the blocker at the moment. Perhaps you could add @Atterratio as the reviewer?

I could but I don't think he will review it since he is not active in the original patch as well.

I cannot since I don't have permission but I don't think he will review it since he is not active in the original patch as well.

Hi @pickfire, thanks for this.
Perhaps @moggers87 could take a look at this since it is a continuation from the previous PRs work (#267) and seems the most likely contributor to know what is required to get this over the line.

I have requested a review from @moggers87

This is awesome, and exactly what we were looking for! Thank you very much for picking this enhancement back up and running with it. We would love to apply this to our production environment and wanted to see if there is any chance this could be reviewed and merged?

I am waiting for reviewer, not sure if the maintainers are still active.

I hoped that #331 would be merged before any new capability addition. The more we wait the more it will be difficult to change the module structure.

@claudep that's a slightly bigger change, which is why I keep delaying merging it :laughing:

Actually nothing can get merged yet anyway because Travis is down and I need to complete #425 to fix that.

I am also interested in email support. And I am guessing it's a feature that would be in demand.
Can I ask what is holding this PR up at the moment? Original PR is from 2018.
@moggers87  it looks like @pickfire has applied your requested changes (admit I haven't looked at the code)
Do you have time for this?
As things stand we might try to implement the PR directly ourselves but we would be more relaxed about it if this PR was merged :)
Thanks to everyone for their work on this.

@moggers87 (CC @pickfire) I'm also interested in having this merged.
Is there anything I can do to help speed up this process?
Are there any blockers on this?

FYI I'm currently trying to refresh this patch based on the new plugin-based method.

The new version is now in #475.


Create an automated release build

The release of a new version of the product includes a exports of the diagram as png, pdf and the .drawio source diagram. The exports are currently manually exported and associated to the github release. This should be done automatically through the appropriate source control promotion mechanisms.
draw.io.exe desktop client has a CLI and these exports can be performed through the --x (--export) flag.
Probably the use of travis ci is the most likely tool using a docker container composed with draw.io for yielding the exports.

The details of how to do the build should be put on hold pending further investigation in #11 on the use of kumu. It could change the entire structure of how the map is built and what tool is used.

The map generation has been changed from manual creation in draw.io to auto-rendered via kumu.io. The change makes this issue moot.  Though it would be ideal if kumu.io had an API for  deleting data, uploading data, exporting PDF and jpegs etc. that is not the case and significant reasonably approachable automation while using kumu is not an option.  See the wiki for how to deploy to kumu.io at wiki/Deploy-to-Kumu.
Issue will closed with this comment.


Error 413 when calling phoscon

I am running the deconz docker on Unraid. A year ago everything worked out fine, but now I am not able to reach the GUI anymore, as it only gives me Error 413. Is this known or happening to anyone else?

https://github.com/dresden-elektronik/deconz-rest-plugin/wiki/Debugging-101
Start with providing useful information :)

I just recognized it solved itself and had something to do with the browser'a cache or cookies.


how to invoke javascript function from native code ?

how to invoke javascript function from native code ?
or how to “fire event” from native code, and listen it from native code ?

Great question!  There is a way, currently used by the cross-platform UI objects that raise events, but the API could use some polishing.  On Android:
run.ace.OutgoingMessages.raiseEvent("myevent", nativeInstance, simpleArgument);
On iOS (when importing OutgoingMessages.h):
[OutgoingMessages raiseEvent:@"myevent" instance:nativeInstance eventData:simpleArgument];
Subscribing in JavaScript:
var instance = new ace.NativeObject("ClassWithEvent");
instance.addEventListener("myevent", function(sender, arg) { });
Let me know if you run into any issues!


Object Duplication #2574

Fixes
#2574
This pull request fixes ability to duplicate all object on wellbore
Description
Type of change

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

Impacted Areas in Application

[x] Frontend
[x] API
[ ] WITSML
[ ] Desktop
[ ] Other (please describe)

Checklist:
Communication

[ ] I have made corresponding changes to the documentation
[ ] PR affects application security

Code quality

[x] I have self-reviewed my code
[x] No new warnings are generated

Test coverage

[ ] New code is covered by passing tests

Further comments

fixed


Display change history on guides

Waiting for https://github.com/alphagov/govuk-content-schemas/pull/369 to be merged before these tests will pass.

Build failure but it's looking good to me.

If you're happy to merge https://github.com/alphagov/govuk-content-schemas/pull/371 I think this'll start passing again.


fix(select): fixed fieldProps type

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.706%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7312

Relevant Lines:
8262

💛  - Coveralls


Between integer guard validator

Guard to validate a number between two other numbers.
F.e. : The number should be between 5 and 10.

I want to implement this.
Do you think lower <= arg <= upper or lower < arg < upper would make more sense?


Unable to install packages

When I double click on a package in suggestions, the footer bar shows that npm (tries?) installs the package.
In iobroker I can see, that the instance of webui is stopped and in the host section I get the error message "Exception-Code: ENOENT: spawn npm ENOENT" and "Issues with non-existing files or directories" "A process tries to access a non-existing file or directory."

Log output

sorry I have not seen this issue until now, but I can not reproduce.
I've tried in a fresh ioBroker installation and it worked.

Here some config info
Webui: v0.14.1
platform: Windows 11
Node.js: v20.9.0
NPM: 10.2.3
js-controller: 5.0.16
Path: C:\ioBroker\

So what I understand from your reply, it looks like it´s a "local" issue
The issue is, that webui (or NPM?) tries to access a non-existing director or file
Which directories and / or files are accessed?
Then I would check the permissions - could be an easy fix
And: no apologies, you doing this in your free time

maybe it's a windows issue... i never tried webui on a windows machine.

I'll look what I can do. Maybe I'll try to install on my windows machine at work tomorrow

I could reproduce in windows, happens also to me. I'll look tomorrow

Works now.
Thanks


zhimi.humidifier.cb1 - water remain value is missing, temperature is null

I can not finde the value "water remain" and from a closed issue I know, that it shoud be there.
Additional to that the temperature is "null".

Thanks
Monti

A change the value of .ledBrightnessLevel has a crash as result.


Allow to choose "did not participate" in pickup feedback

Andrea from Solikyl Gothenburg suggested via email:

Possibility to choose "Did not participate" in response to the pickup feedback

and

Possibility to choose "Did not participate" after a pickup has started/been carried out.

Who is allowed to click that? The person that gives feedback? Another collector?

This could get quite complex depending on what the need really is:

it could just be allowing people to remove themselves from activities even after they've finished
or it's something that let's one of the participations say that another person who was meant to be there didn't turn up
... or something else?

We have to ensure we keep the statistics accurate too, as currently we use the fact that somebody is a participant on an activity to "know" if they did it or not.
It feels like we need to explore what is actually needed here before we can implement something.

For reference, here is the current UI, which explcitly asks people "Did everyone show up?" but you can only represent the answer in text, not data.

The implementation of the proposal of @dpaque in #1508 might close this for now too.


ci: periodic failures on docs deploy

Caught a rate-limit error that blocked update of the protobuf docs here: https://github.com/penumbra-zone/penumbra/actions/runs/4049292647/jobs/6965491788 The error text is:
Warning: Unexpected input(s) 'buf_token', 'input', valid inputs are ['version', 'github_token', 'buf_user', 'buf_api_token']
Run bufbuild/buf-setup-action@v1
Warning: No github_token supplied, API requests will be subject to stricter rate limiting
Setting up buf version "1.13.1"
Resolving the download URL for the current platform...
Error: API rate limit exceeded for <IP_ADDRESS>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)

Looks like it's the github_token we want to export, not the buf API token.

Resolved in https://github.com/penumbra-zone/penumbra/commit/1e6a0083e0d3f71b085031c85aea33daabfab020 , as shown by https://github.com/penumbra-zone/penumbra/actions/runs/4057011304


Allow Heat Converter to work on other furnace faces

It strikes me as odd that the heat converter can only work on top of a furnace. This makes it impossible to move rf from it using the mod's very own flux hopper. I don't know if this was the intention or not, but it sticks out to me as strange.

OK, will work on all faces in the next update.  Not sure exactly when that'll happen.

Done in v1.11.0


[WIP] Add SSH_AGENT_EXTENSION_RESPONSE message type (RFC Draft 14)

Following on from my comments in #42, a new SSH agent protocol spec draft has been published
This adds a new message type: SSH_AGENT_EXTENSION_RESPONSE; this allows an extension to include response data.

The contents of successful extension reply messages are specific to the extension type. Extension requests may return SSH_AGENT_SUCCESS on success or the extension-specific response message:
byte             SSH_AGENT_EXTENSION_RESPONSE
string           extension type
byte[]           extension response-specific contents

For example, the "query" extension looks like:
    byte             SSH_AGENT_EXTENSION_RESPONSE
    string           "query"
    string[]         supported extension types

The response message code for SSH_AGENT_EXTENSION_RESPONSE is 29.

This WIP PR tweaks the extension() trait slightly:

pub trait Session: 'static + Sync + Send + Sized {
    async fn extension(
        &mut self,
        mut extension: Extension,
    ) -> Result<Option<Extension>, Box<dyn std::error::Error>> {
    }
}

return Ok(None) -> SSH_AGENT_SUCCESS
return Ok(Some(ext)) -> SSH_AGENT_EXTENSION_RESPONSE with the serialized content of ext
return Err(_) -> SSH_AGENT_EXTENSION_FAILURE

A sample (the spec'd query extension) is included.
One small issue:

If an extension itself fails, we should return SSH_AGENT_EXTENSION_FAILURE instead of the generic SSH_AGENT_FAILURE.
I can't figure out the best way to expose this, given we just return a Box<dyn std::error::Error>. Would love some thoughts here.

Thanks!

I've also added in two new traits with this PR:
/// SSH agent protocol *message* extension
///
/// Described in [draft-miller-ssh-agent-14 § 3.8](https://www.ietf.org/archive/id/draft-miller-ssh-agent-14.html#section-3.8)
pub trait MessageExtension: 'static + Encode + Decode {
    /// Extension name, indicating the type of the message (as a UTF-8 string).
    ///
    /// Extension names should be suffixed by the implementation domain
    /// as per [RFC4251 § 4.2](https://www.rfc-editor.org/rfc/rfc4251.html#section-4.2),
    fn extension_name() -> &'static str;
}

/// SSH agent protocol *key constraint* extension
///
/// Described in [draft-miller-ssh-agent-14 § <IP_ADDRESS>](https://www.ietf.org/archive/id/draft-miller-ssh-agent-14.html#section-<IP_ADDRESS>)
pub trait KeyConstraintExtension: 'static + Encode + Decode {
    /// Extension name, indicating the type of the key constraint (as a UTF-8 string).
    ///
    /// Extension names should be suffixed by the implementation domain
    /// as per [RFC4251 § 4.2](https://www.rfc-editor.org/rfc/rfc4251.html#section-4.2),
    fn extension_name() -> &'static str;
}

This means an extension (message extensions or key constraint extensions) can tie a name to their structure, letting you construct extensions easier:
let response = Extension::new_message(QueryResponse {
    extensions: vec!["query".into(), "session-bind@openssh.com".into()],
})?;

We could probably enforce these better so that we don't mix up extension types, but I've left that for now.

I can't figure out the best way to expose this, given we just return a Box. Would love some thoughts here.

I think the Box<dyn Error> is showing some serious deficiencies. I've introduced it to not bother the implementers of Session with wrapping their errors but apparently it was a bad idea and we need to get back to AgentError where you could specify if it was extension being unsupported or failed :thinking:
(I won't block this PR on this change, we can do these two in parallel; or maybe you two have better ideas :sweat_smile: )

#48 has merged, I wanted to play with this PR so I rebased it, I pushed the rebase in the baloo/draft-14/rebased branch.

Alright everyone, I'm going to take this one out of draft.
A summary of changes:

Adds ExtensionResponse message type
Adds query extension message
Adds MessageExtension and KeyConstraintExtension traits to tie a name to a message structure
Splits built-in extension implementations into "messages" and "constraints" for readability
Adds new Extension constructors to build extension objects from MessageExtensions and KeyConstraintExtensions:

Extension::new_message{_raw}
Extension::new_key_constraint{_raw}

Outstanding issues:

Error handling; though #52 is addressing that (i.e., being able to return generic failure vs. extension failure)
I'm not too happy with the constructor API - TryFrom would be nicer but AFAIK we can't do that generically(?)

FYI I'm good with this PR. I've wanted to experiment with some other way to do extensions but frankly your version already improves the status quo and I don't think it would break anything in the wild :thinking:

Okay great! I'm still a bit unsure on the constructor API, but happy for any feedback or for this to be merged as-is and tweaked - up to you!

Also - my apologies on the merge here - I must admit git merges are my weakness - happy to re-do these merges to fix up the commit history if it's easier for you!

Nah, I don't mind the merges. I do clean up my commits out of habit but I'm not mandating strict linear history. What's important is that the tests pass and the code is improved :)
Thanks for your work! :+1:

Alright, sorry for all the git nonsense. This should now be re-based on main properly, with documentation for everything, and clippy finally happy!
I'll leave this PR with the two of you for review, but I think this covers everything.
To make things simpler, I've left out non-Encode / Decode structures from the whole Extension::{new | parse}_{message | key_constraint} piece for now.
If you have any ideas there let me know - I'm using Encode/Decode types in my own agent extensions so I don't really have an opinion on APIs for types outside of that (like @baloo using postcard) 🤷


django-admin-smoke-tests failures

This is with https://github.com/greyside/django-admin-smoke-tests/pull/18 , and on Django 3.x with some extra fixes, so again hard to reproduce the scenario.  If you cant immediately see the problem and solution, leave it and I will find the fix.
======================================================================
ERROR: test_changelist_view (tests.test_admin_smoke.AdminSiteSmokeTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 29, in test_deco
    fn(self, model, model_admin)
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 226, in test_changelist_view
    response = model_admin.changelist_view(request)
  File "/usr/lib/python3.8/site-packages/data_browser/helpers.py", line 60, in changelist_view
    return super().changelist_view(request, extra_context)
  File "/usr/lib/python3.8/site-packages/django/utils/decorators.py", line 43, in _wrapper
    return bound_method(*args, **kwargs)
  File "/usr/lib/python3.8/site-packages/django/utils/decorators.py", line 130, in _wrapped_view
    response = view_func(request, *args, **kwargs)
  File "/usr/lib/python3.8/site-packages/django/contrib/admin/options.py", line 1678, in changelist_view
    cl = self.get_changelist_instance(request)
  File "/usr/lib/python3.8/site-packages/django/contrib/admin/options.py", line 728, in get_changelist_instance
    return ChangeList(
  File "/usr/lib/python3.8/site-packages/django/contrib/admin/views/main.py", line 57, in __init__
    self.root_queryset = model_admin.get_queryset(request)
  File "/usr/lib/python3.8/site-packages/data_browser/helpers.py", line 21, in get_queryset
    fields = self.get_fields_for_request(request)
  File "/usr/lib/python3.8/site-packages/data_browser/helpers.py", line 14, in get_fields_for_request
    elif request.resolver_match.func.__name__ == "changelist_view":
AttributeError: 'NoneType' object has no attribute 'func'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 48, in test_deco
    six.raise_from(ModelAdminCheckException(
  File "<string>", line 3, in raise_from
django_admin_smoke_tests.tests.ModelAdminCheckException: Above exception occured while running test 'test_changelist_view' on modeladmin data_browser.ViewAdmin (View)

======================================================================
ERROR: test_changelist_view_search (tests.test_admin_smoke.AdminSiteSmokeTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 29, in test_deco
    fn(self, model, model_admin)
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 240, in test_changelist_view_search
    response = model_admin.changelist_view(request)
  File "/usr/lib/python3.8/site-packages/data_browser/helpers.py", line 60, in changelist_view
    return super().changelist_view(request, extra_context)
  File "/usr/lib/python3.8/site-packages/django/utils/decorators.py", line 43, in _wrapper
    return bound_method(*args, **kwargs)
  File "/usr/lib/python3.8/site-packages/django/utils/decorators.py", line 130, in _wrapped_view
    response = view_func(request, *args, **kwargs)
  File "/usr/lib/python3.8/site-packages/django/contrib/admin/options.py", line 1678, in changelist_view
    cl = self.get_changelist_instance(request)
  File "/usr/lib/python3.8/site-packages/django/contrib/admin/options.py", line 728, in get_changelist_instance
    return ChangeList(
  File "/usr/lib/python3.8/site-packages/django/contrib/admin/views/main.py", line 57, in __init__
    self.root_queryset = model_admin.get_queryset(request)
  File "/usr/lib/python3.8/site-packages/data_browser/helpers.py", line 21, in get_queryset
    fields = self.get_fields_for_request(request)
  File "/usr/lib/python3.8/site-packages/data_browser/helpers.py", line 14, in get_fields_for_request
    elif request.resolver_match.func.__name__ == "changelist_view":
AttributeError: 'NoneType' object has no attribute 'func'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 48, in test_deco
    six.raise_from(ModelAdminCheckException(
  File "<string>", line 3, in raise_from
django_admin_smoke_tests.tests.ModelAdminCheckException: Above exception occured while running test 'test_changelist_view_search' on modeladmin data_browser.ViewAdmin (View)

======================================================================
ERROR: test_queryset (tests.test_admin_smoke.AdminSiteSmokeTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 29, in test_deco
    fn(self, model, model_admin)
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 207, in test_queryset
    list(model_admin.get_queryset(request))
  File "/usr/lib/python3.8/site-packages/data_browser/helpers.py", line 21, in get_queryset
    fields = self.get_fields_for_request(request)
  File "/usr/lib/python3.8/site-packages/data_browser/helpers.py", line 14, in get_fields_for_request
    elif request.resolver_match.func.__name__ == "changelist_view":
AttributeError: 'NoneType' object has no attribute 'func'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/django_admin_smoke_tests/tests.py", line 48, in test_deco
    six.raise_from(ModelAdminCheckException(
  File "<string>", line 3, in raise_from
django_admin_smoke_tests.tests.ModelAdminCheckException: Above exception occured while running test 'test_queryset' on modeladmin data_browser.ViewAdmin (View)

Sooo
The Data Browser comes with some helpers, the main one is the AdminMixin, and it's main thing is a nicer way of dealing with annotated fields in admins.
In doing that it needs to know if it's a list request or a detail request and it figures that out by inspecting the request.
That AdminMixin is mixed into the Data Browsers own saved view admin.
This test is testing the the Data Browsers saved view admin, and it's passing a request from a request factory.
That request factory request object is not sufficiently complete to support the inspection the data browser is doing, so Boom.
If you are not otherwise using that AdminMixin then the short term fix for you is to make use Admin Smoke Tests exclude_apps setting to exclude the Data Browser.
For my part I don't like that just adding the mixin to an admin will break any test that passes a request factory request into that admin and I think I can fix it fairly easily.

Thanks for fixing this.
I was previously using exclude_apps to ignore this, and didnt consider it important while there were other more interesting issues.
Now I can remove that, and report any times it breaks again, and you can consider any future reports about this as very low priority as I dont need a fix.
The reason I run these smoke tests is to detect problems when upgrading py, dj, drf, etc - obscure problems arise that are not covered by the apps test suites.  And it also means that the request factory gets a good work out, so I know I can use it any time I need to write a custom test involving a model that I didn't create.

No worries, the request factory thing wasn't good, that could break a lot of people admin tests. That package is interesting, I will have to remember it.

The four I use are
django-smoke-tests = {git = "https://github.com/jayvdb/django-smoke-tests", rev = "drf"}
django-admin-smoke-tests = {git = "https://github.com/jayvdb/django-admin-smoke-tests", rev = "all"}
django-admin-auto-tests = {git = "https://github.com/jayvdb/django-admin-auto-tests", rev = "sdist"}
schemathesis = "^2.3.1"

django-smoke-tests hits all endpoints with garbage.
schemathesis hits openapi drf endpoints.
Of the two admin ones, the latter is the best, but requires creating test modules with some boilerplate classes, and it cuts deeper, so I only use it for major apps (djstripe/oscar/etc) where business logic directly interacts with their app.


Test compilation failure

When building stackage nightly:
test/Data/StoreSpec.hs:90:3: error:
    Duplicate instance declarations:
      instance Monad m => Serial m Word
        -- Defined at test/Data/StoreSpec.hs:90:3
      instance [overlap ok] Monad m => Serial m Word
        -- Defined in ‘Test.SmallCheck.Series’
   |
90 | $(do let ns = [ ''CWchar, ''CUShort, ''CULong, ''CULLong, ''CIntMax
   |   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^...

The tests were already disabled in nightly for other (resolved) reasons, but I'll keep it disabled for now.

Fixed on master!

Seems like the 0.5.0 release didn't resolve this.

Is this is <IP_ADDRESS>?

Yup, at least <IP_ADDRESS> fixes the failures encountered when building against recent stackage nightly.  Sorry for taking so long to resolve this!


Revert "Add support for json serializable generic types"

Reverts trevorwang/retrofit.dart#282

Was something wrong in the code?


'Files' and 'Directory' are not listed as clients in the Factory class

Hello,
Apparently the 'Files' and 'Directory' clients are not listed in the Factory, thus we cannot instantiate a factory by passing a string like 'files' or 'directory'. Also, the first argument in the implementation of those 2 classes should be an instance of the class itself, otherwise the factory will throw an error.
Maybe I missed something?!
Thanks,
Regards,
Alex

Thank YOU for the quick turnaround!

You are welcome and again thanks to you for noticing these in the first place.
These mods have been published in version 1.2.5 of APIx Cache. Enjoy.


Vitest reporter pane is blank

Describe the bug
The Vitest reporter pane shown in the tweet is just blank when I open https://vitest.new:

Link to the blitz that caused the error
https://stackblitz.com/edit/vitest-dev-vitest-squjqn?file=test%2Fbasic.test.ts
Steps to reproduce

Open https://vitest.new
Observe the Vitest reporter pane is blank.
Observe two extra split panes to the right of the Vitest reporter pane that are also blank.

Expected behavior

The Vitest reporter pane shows the test results.
No extra blank panes that serve no purpose.

Parity with Local

[x] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Chrome
Full version  = 97.0.4692.71
Major version = 97
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/97.0.4692.71 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 70690193,
  "usedJSHeapSize": 65429597,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = d4ed89204e54de5c40d109298317e6dc4d60aef7

Additional context
Console log:
.localservice@runtim…17e6dc4d60aef7.js:7 Uncaught
_0x4cc0ec {code: 1002, message: 'RSV2 and RSV3 must be clear'}
code: 1002
message: "RSV2 and RSV3 must be clear"
[[Prototype]]: Object

vitest-dev-vitest-4e….webcontainer.io/:1 Uncaught (in promise) SyntaxError: Unexpected end of JSON input
    at parse (<anonymous>)
    at bM (vitest-dev-vitest-4edn2l--51204.local.webcontainer.io/__vitest__/assets/vendor.1560970a.js:8:23751)
    at vitest-dev-vitest-4edn2l--51204.local.webcontainer.io/__vitest__/assets/vendor.1560970a.js:8:22534
    at _0x263926.<anonymous> (vitest-dev-vitest-4edn2l--51204.local.webcontainer.io/__vitest__/assets/index.8d7a5168.js:1:3504)
    at _0x263926.dispatchEvent (.localservice@runtime.d4ed89204e54de5c40d109298317e6dc4d60aef7.js:1:28929)
    at _0x263926._handleMessage (.localservice@runtime.d4ed89204e54de5c40d109298317e6dc4d60aef7.js:7:18258)
    at MessagePort._0x49f884 (.localservice@runtime.d4ed89204e54de5c40d109298317e6dc4d60aef7.js:7:14830)

Oddly, it works now in a new tab.

@tony19 Thanks for the report.

[ ]  I have run the project in my local machine and I could not reproduce the issue.

Can you confirm whether you ran this project locally and it worked?

.localservice@runtim…17e6dc4d60aef7.js:7 Uncaught
_0x4cc0ec {code: 1002, message: 'RSV2 and RSV3 must be clear'}
code: 1002
message: "RSV2 and RSV3 must be clear"
[[Prototype]]: Object

FWIW we just deployed a fix for this issue.

Yes, this works locally, as it does currently in new instances of vite.new in StackBlitz.

I can't reproduce on the original blitz anymore. Are you still seeing it?

I think the original issue is resolved, so feel free to close 🎉 !

FYI, there seems to be a Vitest UI bug that affects https://vitest.new, where it crashes at startup, and prevents the preview pane from rendering at all:

I'll report it in @vitest/ui, and link the issue here when I can.

Interesting, we'll keep an eye on that as well. Thanks!


Rewrite redirect clauses in archive script. Fixes #905

Fixes #905

I implemented this for the basic case, when there is no --date provided.
The case with --date is tricky: the script uses Git's stdout and stderr output in binary format. So running text processing on those is not possible. I am on the fence on whether we should keep the --date option. I don't think it adds much value (we can always navigate to a given commit manually and run the script) but it adds a noticeable complexity to the Python script.

Pushed a new version of this PR which uses the python-frontmatter package.


Travis Matrix for Integration Tests

This PR is a work in progress for testing @jathanism's selenium integration tests #514 as part of a matrix build.

@nniehoff Now that #514 is merged, what parts of this PR are still relevant? Can you update it accordingly when you have a chance? Thanks!

This is OBE with #571


Vendor crates by extracting crate files

We've run afoul of https://github.com/rust-lang/cargo/issues/5956 many times in Firefox development, and until someone figures out a way to prevent things from making changes to crate sources in the local registry, I think vendoring crates by directly extracting their crate files would be less error-prone.
Looking at the cargo source I don't think this is straightforward to do with the APIs cargo provides, unfortunately. Perhaps the closest thing to do would be to collect the set of packages, then remove those package directories, then re-run the resolution -> package get loop to force cargo to unpack fresh copies of each crate before vendoring.

Forcing Cargo to unpack sounds dangerously plausible! It's not compatible with concurrent invocations of Cargo but I think that's fine


[Redfish] redfish/update_service/test_redfish_bmc_code_update.robot check task status failed

I use openbmc-test-automation v3.0-stable branch and run the redfish/update_service/test_redfish_bmc_code_update.robot test case.
robot -v OPENBMC_HOST:x.x.x.x -v IMAGE_FILE_PATH:v2.12.2.all.tar --include Redfish_Code_Update_With_ApplyTime_OnResetredfish/update_service/test_redfish_bmc_code_update.robot

The Keyword 'Verify Task Progress State' failed after retrying for 5 minutes. The last error was: Completed != Running
Does the status of redfish update correct?
https://github.com/openbmc/bmcweb/blob/master/redfish-core/lib/update_service.hpp

I had manually used redfish command to update bmc image by image-bmc tarball
$ curl -k -H "X-Auth-Token: $token" -H "Content-Type: application/octet-stream" -X POST -T v2.12.2.all.tar https://${bmc}/redfish/v1/UpdateService
{
<EMAIL_ADDRESS>"/redfish/v1/TaskService/Tasks/0",
<EMAIL_ADDRESS>"#Task.v1_4_3.Task",
  "Id": "0",
  "TaskState": "Running",
  "TaskStatus": "OK"
}

The image-bmc already untar to /run/initramfs
# ls -l /run/initramfs/
drwxr-xr-x    2 root     root          1320 Mar  9  2018 bin
drwxr-xr-x    4 root     root           260 Mar  9  2018 etc
-rw-r--r--    1 root     root      33554432 Nov 27 05:37 image-bmc
-rwxr-xr-x    1 root     root          9585 Mar  9  2018 init
-rw-r--r--    1 root     root            40 Nov 27 05:25 init-options
-rw-r--r--    1 root     root             0 Nov 27 05:25 init-options-base
drwxr-xr-x    3 root     root           340 Mar  9  2018 lib
drwxr-xr-x   18 root     root           245 Apr  5  2011 ro
drwxr-xr-x    5 root     root             0 Nov 27 05:25 rw
drwxr-xr-x    2 root     root           680 Mar  9  2018 sbin
-rwxr-xr-x    1 root     root          1917 Mar  9  2018 shutdown
-rwxr-xr-x    1 root     root          5180 Mar  9  2018 update
drwxr-xr-x    6 root     root           120 Mar  9  2018 usr
drwxr-xr-x    4 root     root            80 Mar  9  2018 var
-rw-r--r--    1 root     root           271 Mar  9  2018 whitelist

But the TaskState of task inventory still keeping on Running
$curl -H "X-Auth-Token: $token" -k https://${bmc}/redfish/v1/TaskService/Tasks/
{
<EMAIL_ADDRESS>"/redfish/v1/TaskService/Tasks",
<EMAIL_ADDRESS>"#TaskCollection.TaskCollection",
  "Members": [
    {
<EMAIL_ADDRESS>"/redfish/v1/TaskService/Tasks/0"
    }
  ],
<EMAIL_ADDRESS>1,
  "Name": "Task Collection"
}

$ curl -H "X-Auth-Token: $token" -k https://${bmc}/redfish/v1/TaskService/Tasks/0
{
<EMAIL_ADDRESS>"/redfish/v1/TaskService/Tasks/0",
<EMAIL_ADDRESS>"#Task.v1_4_3.Task",
  "EndTime": "2023-11-27T05:20:05+00:00",
  "Id": "0",
  "Messages": [
    {
<EMAIL_ADDRESS>"#Message.v1_0_0.Message",
      "Message": "The task with id 0 has started.",
      "MessageArgs": [
        "0"
      ],
      "MessageId": "TaskEvent.1.0.1.TaskStarted",
      "Resolution": "None.",
      "Severity": "OK"
    },
    {
<EMAIL_ADDRESS>"#Message.v1_1_1.Message",
      "Message": "The request failed due to an internal service error.  The service is still operational.",
      "MessageArgs": [],
      "MessageId": "Base.1.11.0.InternalError",
      "MessageSeverity": "Critical",
      "Resolution": "Resubmit the request.  If the problem persists, consider resetting the service."
    }
  ],
  "Name": "Task 0",
  "Payload": {
    "HttpHeaders": [
      "Host: <IP_ADDRESS>",
      "User-Agent: curl/7.68.0",
      "Accept: */*",
      "Content-Length: 33566720"
    ],
    "HttpOperation": "POST",
    "JsonBody": "null",
    "TargetUri": "/redfish/v1/UpdateService"
  },
  "PercentComplete": 0,
  "StartTime": "2023-11-27T05:20:03+00:00",
  "TaskMonitor": "/redfish/v1/TaskService/Tasks/0/Monitor",
  "TaskState": "Running",
  "TaskStatus": "OK"
}

@susilsi7 please take a look into this ?

Robot code expecting "TaskState": "Completed" under 5 mins once you upload the image to BMC.
I think you encounter some issue. I see below in your Task 0.
"Messages" ->  {<EMAIL_ADDRESS>"#Message.v1_1_1.Message",       "Message": "The request failed due to an internal service error.  The service is still operational.",       "MessageArgs": [],       "MessageId": "Base.1.11.0.InternalError",       "MessageSeverity": "Critical",       "Resolution": "Resubmit the request.  If the problem persists, consider resetting the service."     }


SignJWT does not return data that can construct a JWT

Blocked by aefalcon/github-keystore-protobuf#1


[Storybook] Upgrade to enable CSF

Storybook has released v5.2.0. Numerous bug fixes, as well as a new portable story format, Jest Addon redesign, among other new features.
The migration guide from 5.1.x to 5.2.x is quite short.

I've got some WIP on storybook-5.2 branch.

[x] Update how we load stories so we can mix use of CSF the legacy storiesOf
[ ] Migrate existing stories to use new CSF format, codemod available, but bugs have been filed against it. Might be best to wait for v5.3 on this
[x] Refactor webpack to use full-control mode (extended-mode has been deprecated)
[x] Install and manually config the new Storybook Docs
[ ] Resolve issue where prop tables are not populated in Storybook Docs

As of Storybook v5 there is no need to add custom .css loader as it is already defined. Looks like this might be causing issues with how styles are loaded in the storybook though, the cascade of carbon component styles vs global styles vs addons local styles is causing some visual defects in the storybook.

v5.3 is out now, migration guide is available as well. Going to refocus this issue on just getting CSF support to improve documentation and portability of our stories. (5.2 introduced this, 5.3 expanded on it)

As a part of this, it might be good to revisit the layout strategy for components on the canvas (centering, etc) and the associated documentation with the upgraded storybook interface of v5.3

CSF migration will be handled in #964
addon-docs upgrade will be handled in #963


editing profile fails

Subject: editing profile fails
Description:
i hit the edit button, the window never opens. can't edit profile.
Results:
the window should have opened where I could edit my profile.
Backend version: develop-ca4c6eeb
Frontend version:
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:4.8) Goanna/20211112 PaleMoon/<IP_ADDRESS>
Page: https://couchers.org/profile
User: hofy (20623)

can't reproduce


fix(#patch); makerdao; fixed a bug when liquidationPenalty is set to 0

This fixes the unreasonable protocol side revenue for USDC-A liquidation reported by @bye43  in #1502

Test deployment: https://thegraph.com/hosted-service/subgraph/tnkrxyz/makerdao-ethereum?version=pending&selected=logs
validation dashboard: https://api.thegraph.com/subgraphs/name/tnkrxyz/makerdao-ethereum/graphql?query={
++marketDailySnapshots+(orderBy%3A+dailyLiquidateUSD%2C+orderDirection%3Adesc%2C+where%3A+{timestamp_gte%3A+1670284800%2C+timestamp_lte%3A+1670371199})+{
++++timestamp
++++market+{
++++++name
++++++id
++++}
++++dailyLiquidateUSD+
++++dailyProtocolSideRevenueUSD
++}++
}

@tnkrxyz @steegecs can we deploy this change to messari - hosted?

@tnkrxyz @steegecs can we deploy this change to messari - hosted?

sure. I will merge & deploy to messari hosted-service.


Efficient updates

A work in progress at the moment.

It will no longer remove all the decorations and re-add them even when nothing has changed, It'll only apply the diff of messages on decorations. Which prevents a lag that used to happen everytime we lint. :racehorse:

Travis is green, everything is awesome. Merging in.

This PR makes the markers efficient, I'll be working to fix the speed-eater-bubble soon.
/cc @atom-community/linter-contributors


Add readiness probe to check webhook cert ready

Q
A

Bug fix?
no

New feature?
yes (/readyz as readiness probe)

API breaks?
yes (/metrics as readiness probe)

Deprecations?
yes (/metrics as readiness probe)

Related tickets

License
Apache 2.0

What's in this PR?
Add readiness probe to mark cluster-registry as "ready" only after webhook configuration certificate is updated successfully
Why?
cluster-registry generates self signed certificate to inject into webhook which take approx ~30 sec after deploy.
If we are running an automation script that waits for cluster-registrty to be ready then /metrics end point marks container as ready even before caBundle in injected into ValidatingWebhookConfig.
As a result, the automation script would fail with
x509: certificate is not valid for any names, but wanted to match cluster-registry-controller.cluster-registry.svc

so we need to wait until the webhook is ready, then mark cluster-registry as "ready"
Additional context
This PR changes the readiness probe from /metrics to /readyz which would potentially break API that depends on readiness probe
Checklist

[x] Implementation tested
[x] User guide and development docs updated (if needed)

To Do

[x] If the PR is not complete but you want to discuss the approach

Tests

Webhook enabled (takes ~30 sec to be ready) (webhook ready health checker)
Init "16:34:22" -> Ready "16:35:02" (40 seconds)

status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: "2022-12-12T16:34:22Z"
    status: "True"
    type: Initialized
  - lastProbeTime: null
    lastTransitionTime: "2022-12-12T16:35:02Z"
    status: "True"
    type: Ready
  - lastProbeTime: null
    lastTransitionTime: "2022-12-12T16:35:02Z"
    status: "True"
    type: ContainersReady
  - lastProbeTime: null
    lastTransitionTime: "2022-12-12T16:34:22Z"
    status: "True"
    type: PodScheduled
  containerStatuses:
  - containerID: docker://7c5063a148ea36dc0d7f4fa8451710ae34f304b1b4e05475e607d7b7bffacf20
    image: nishantapatil3/cluster-registry-controller:latest
    imageID: docker-pullable://nishantapatil3/cluster-registry-controller@sha256:1a672de686f0e0c2419be9194a2bb55cdd1cf0437ff2c72a840f3b70aafb2504
    lastState: {}
    name: manager
    ready: true
    restartCount: 0
    started: true
    state:
      running:
        startedAt: "2022-12-12T16:34:24Z"

Webhook disabled (takes ~5 sec to be ready) (ping health checker)
Init "16:29:06" -> Ready "16:29:10" (4 seconds)

status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: "2022-12-12T16:29:06Z"
    status: "True"
    type: Initialized
  - lastProbeTime: null
    lastTransitionTime: "2022-12-12T16:29:10Z"
    status: "True"
    type: Ready
  - lastProbeTime: null
    lastTransitionTime: "2022-12-12T16:29:10Z"
    status: "True"
    type: ContainersReady
  - lastProbeTime: null
    lastTransitionTime: "2022-12-12T16:29:06Z"
    status: "True"
    type: PodScheduled
  containerStatuses:
  - containerID: docker://b12445fe5a486db6227e1d9b2b76fb6dcacadff25c1ff96e5a6a62fd70a74c40
    image: nishantapatil3/cluster-registry-controller:latest
    imageID: docker-pullable://nishantapatil3/cluster-registry-controller@sha256:1a672de686f0e0c2419be9194a2bb55cdd1cf0437ff2c72a840f3b70aafb2504
    lastState: {}
    name: manager
    ready: true
    restartCount: 0
    started: true
    state:
      running:
        startedAt: "2022-12-12T16:29:08Z"

CC: @pregnor @waynz0r @tiswanso (I couldnt assign reviewers)

Thanks for the review guys, this is my first pull request so i am unable to merge.. please merge when possible, I'll create another PR to update charts and image versions

Thanks for the review and merge @Laci21 @pregnor @waynz0r


Fixed a bug where plugin rendered multiple times won't be editable

This covers more cases then the one that was already in place
Ref: #6012

Coverage decreased (-38.8%) to 40.842% when pulling bafacd426e594e93d1de92ae47b6ee957e3589b6 on vxsx:bugfix/3.5.x/same-plugin-rendered-multiple-times into 1e875d7461dda3c24a00ed968062f5fb2a25ee94 on divio:develop.

Coverage increased (+0.1%) to 79.797% when pulling 5ca0a8277b3dfcecaff8e5588d5952ded8c35c1b on vxsx:bugfix/3.5.x/same-plugin-rendered-multiple-times into 72551e4dace82141f74d02c7da37b45208ab0efc on divio:develop.

Coverage increased (+0.1%) to 79.828% when pulling a8fefdd5daa9f94e24cc34cded9b7fd9eb18f85e on vxsx:bugfix/3.5.x/same-plugin-rendered-multiple-times into 72551e4dace82141f74d02c7da37b45208ab0efc on divio:develop.

Coverage increased (+0.1%) to 79.828% when pulling 93b29b05a96c8ab5fe8ab81f613007b3b13d6e07 on vxsx:bugfix/3.5.x/same-plugin-rendered-multiple-times into 72551e4dace82141f74d02c7da37b45208ab0efc on divio:develop.


Increase of memory usage after upgrading from go1.8 to go2.2

Go version
go2.2
Output of go env in your module/workspace:
GOARCH=amd64 GOOS=linux
      "GODEBUG" : "disablethp=1, gctrace=1"

What did you do?
I have set       "GODEBUG" : "disablethp=1, gctrace=1" in lambda environment
but it does not seem to have any effect. Unfortunately I can't change the linux provided.al2023 image, as it is not an option using serverless lambda.
What did you see happen?
My lambda memory consumption increased from 80-90mb to 700mb+
What did you expect to see?
I expected to see the same level of memory consumption.

go2.2 does not exist.
Please include the complete output of go env.
I don't know what a "lambda environment" is, so some additional context would help.

AWS environment, so probably an AWS problem.

never seen go2.2

it's running on AWS lambda (serverless) on provided.al2023 image.
it's go 1.22

"GODEBUG" : "disablethp=1, gctrace=1"

The space in that string might be preventing gctrace=1 from being recognized.

@arturg-acoustic Did you try GOMEMLIMIT ?
I am hitting a somewhat similar issue.
A new proxy application built with Go 1.22.1 increases memory usage at a slow constant rate.
This is my troubleshooting history:
1 - Requests are not served at constant rate but memory increases at slow constant rate.
2 - Profiling did not reveal a culprit for memory leak.
3 - Failed to reproduce it on staging environments.
4 - Attempted "disablethp": It did improve memory usage but did not fix the slow constant growth.
5 - Now I am experimenting with GOMEMLIMIT to restrict memory usage to a reasonable limit.
6 - If GOMEMLIMIT fails, next attempt would be to rebuild the application with 1.19 toolchain.


How can JDL create database tables automatically?

How can JDL create database tables automatically?

This ticket has been closed as the guidelines are not followed.
Tickets must follow our Guidelines, as mentioned in:

our Readme file on the front page of the project,
the "create a new ticket" page and
our Help page

We have also created a template on the "create a new ticket" page to help you follow those guidelines.


Choosing multiple regex in the filters array not working

Hello,
Below is a code of how I've currently setup this function. For some reason on compilation the lines aren't being removed.
// ...
.pipe(deleteLines({
    'filters': [
        /<base\s+href=["']/i,
        /<script\s+src=["']https:\/\/ajax\.googleapis\.com\/ajax\/libs\/jquery\/1\.12\.2\/jquery\.min\.js["']/i
    ]
}))
// ...

However, when I reduce the array to 1 item like below, it works.
// ...
.pipe(deleteLines({
    'filters': [
        /<base\s+href=["']/i
    ]
}))
// ...

I'm having the same problem. And instead each line of the file is duplicated!

thanks a lot for your PR!


Control MPC solve method

I wonder why use osqp method to solve the MPC matrix without build QP problem like before which used QPOASES and made an H & f with reference to matrix A and B. Now we just use matrix Q & P to solve. I can not understand the QP progress. Could u give me a reference, like a paper or something? Very appreciate！

I comprehend it already. The solve method with osqp build the QP problem with x & u together. And make the A & B matrix into the equality constraint. The qpoases method before build the QP problem just with the input matrix u as the optimize variable.

I comprehend it already. The solve method with osqp build the QP problem with x & u together. And make the A & B matrix into the equality constraint. The qpoases method before build the QP problem just with the input matrix u as the optimize variable.

I encountered the same question. In apollo mpc I could not find any code about prediction, in controller only matrix_ad_ and matrix_bd_ were calculated at time k, but no prediction future action and pass inot osqp. Did you get the ider of these codes?

You may take a look at mpc_solver. It contains equality constraint matrix by which the relation Xk+1 = AXk+BU is stated.

I comprehend it already. The solve method with osqp build the QP problem with x & u together. And make the A & B matrix into the equality constraint. The qpoases method before build the QP problem just with the input matrix u as the optimize variable.

I encountered the same question. In apollo mpc I could not find any code about prediction, in controller only matrix_ad_ and matrix_bd_ were calculated at time k, but no prediction future action and pass inot osqp. Did you get the ider of these codes?

You may take a look at mpc_solver. It contains equality constraint matrix by which the relation Xk+1 = A_Xk+B_U is stated.

That's the truth.


Error while pulling image: Get https://index.docker.io/v1/repositories/library/jenkins/images: dial tcp: lookup index.docker.io: no such host

No docker images are available because of following issue.
Error while pulling image: Get https://index.docker.io/v1/repositories/library/jenkins/images: dial tcp: lookup index.docker.io: no such host.
Error while pulling image: Get https://index.docker.io/v1/repositories/library/elasticsearch/images: dial tcp: lookup index.docker.io: no such host
...

Are you behind a proxy? We have a few common fixes in our wiki:
https://github.com/docker/kitematic/wiki/Common-Proxy-Issues-&-Fixes

Solution :
[root@xx ~]# docker pull hello-world
Using default tag: latest
Pulling repository docker.io/library/hello-world
Error while pulling image: Get https://index.docker.io/v1/repositories/library/hello-world/images: authenticationrequired
earlier i have set docker proxy in file   /etc/systemd/system/docker.service.d/https-proxy.conf
#[Service]
#Environment="HTTPS_PROXY=http://xxx:8080/"
then i commented this, and sys-reloaded
#sudo systemctl daemon-reload
#sudo systemctl restart docker
then i tried to pull hello-world and i got it.
sometime docker works with proxy also without proxy :(


Status dashboard: Automatically ask for a 'push --force-with-lease'

If we're ahead and behind, the push command (which will fail) should automatically ask the user for a '--force-with-lease'.

If it exists, it would increase my chance to overwrite remote master's. I am a bit worried about data safety. Although there is a pop up for destructive commands, some users would proceed anyway without reading.

Do you have another idea? Usually, you never do a '--force' but with PR's esp. on GitHub it is the common workflow.
Isn't it at least so that if we're in a '-4/+2' state, we can fail immediately if the user tries to do a normal 'push' bc we know for certain that this will fail? I thought we show a quick panel with 'proceed anyway, force-with-lease, force'. For data safety it would maybe be enough to make 'proceed anyway' the default. 🤷♀


[Feature request]: Search page

Describe the change you'd like
Hello, can you make this extension to work also for search page?
i prefer the tile view (the default view for home and subscriptions) also for search page
PS:i'm using your script for stylus and i'm sad you are not supporting it anymore, could you at least fix the bugs?
Additional context
No response

Can you describe what the bug is? Where it happened?
for me the script is still working for me

Can i say it here? Ok
There are few things

Search page. the only thing i get is the grid view but "no limit width" is doing nothing (as same as the other 3 that i don't use)

in the "playlist" tab i see no images for the playlist i save. i tried to disable everything but it didn't work. i have to disable the script to make the images come back

For me in the playlist tab I can still see the images. Can you screenshot all you setting?

there is not much to show. even if i disable everything i don't see images
can i send you a log instead? you can see if i have a corrupt version. or i could try to reinstall but i don't remember how to do it

139.5 unknown error is not the bug.
I can't reproduce the missing thumbnail on playlist tab, anyway I'm going to fix the "no limit width" in upcoming update.


[Request]Possibility to add "leave as/use current format" ?

Hey man,
just a quick question if there's the possibility to add an encoding option to just use the current format?
I.e. I'm mainly converting from .ts files, so for files that just need to be cut and not converted, i could 'abuse' mpv-webm as cutter without switching to another tool.
Greetings broky

Possibly duplicate of #8.
mpv doesn't support "copy" codecs, therefore "cutting" without reencoding can't be done.

Yeah, looks like i missed that, sorry.
Anyway thanks for your time and immediate reply, admittedly i too thought you'd use ffmpeg.
But i guess it was worth a try.
Godspeed


[Contacts] Display the users of my instance first

#548

This PR will be cancelled, because the sorting will be done on server side


Updated readme document in vault

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

@Rugvip please review , commenting for avoid stale


[Question] @ApiParam example

Hi,
I am wondering why my specified example value in @ApiParam at an @PathVariable in my Spring REST controller doesn't exist in my Swagger JSON.
public Collection<Foo> getFoo(@ApiParam(value = "My bar value", example = "bar123", required = true) String bar) {
I am expecting "bar123" to appear as an example under the parameter "bar" in my Swagger JSON.
So now my questions is if the absence of the example is a bug or if I am using the example field wrong?
Thanks
Dennis

Hi, this repository is for the OpenAPI specification, which is just about the API definitions and possibly about the actual JSON being exchanged.
It looks like you have a problem with a specific implementation, maybe swagger-core? Please open your issue there. (Though a bit more context will be helpful there too, e.g. a full example file with the wrong output.)

Ok thanks for you hint. I had a look at the issue at swagger-core and found this one: https://github.com/swagger-api/swagger-core/issues/1940
This is exactly my problem.
Best regards
Dennis

As @ePaul pointed out, this is not an issue with the spec but with the tooling.


missing venv module

venv can be included as a default package, which is needed for creating virtual environments, which is crucial for python 3.11

Added python3-virtualenv to packages! [b3e70c98700d99a03593aea5a441689f001e84a9]


Voxel pick api

Description
This PR adds the public method Scene.pickVoxel to report information about a voxel cell under the cursor.
The return value from Scene.pickVoxel is an instance of the new VoxelCell class. This class includes getters to retrieve the values of the different properties contained in the voxel metadata.
Testing plan

Verify the additional specs in PickingSpec and VoxelCellSpec run successfully.
Test mouseover picking in this local Sandcastle.

Author checklist

[x] I have submitted a Contributor License Agreement
[x] I have added my name to CONTRIBUTORS.md
[ ] ~I have updated CHANGES.md with a short summary of my change~ This will be done in the PR into main
[ ] I have added or updated unit tests to ensure consistent code coverage
[x] I have update the inline documentation, and included code examples where relevant
[x] I have performed a self-review of my code

To-do:

[ ] Fix the spec that is failing on CI and not locally
[ ] Add VoxelCellSpec

Thank @jjhembd! The API here is much cleaner :+1:
I saw one issue in the Sandcastle example:
When switching from ellipsoid shape to cylindrical shape, I saw the following output in the console.
VoxelPrimitive: Uniform name "u_cylinderUvToRenderBoundsScale" is already defined
VoxelPrimitive: Uniform name "u_cylinderUvToRenderBoundsTranslate" is already defined
VoxelPrimitive: Uniform name "u_cylinderUvToRenderRadiusMin" is already defined
VoxelPrimitive: Uniform name "u_cylinderRenderAngleMinMax" is already defined
VoxelPrimitive: Uniform name "u_cylinderUvToShapeUvRadius" is already defined
VoxelPrimitive: Uniform name "u_cylinderUvToShapeUvHeight" is already defined
VoxelPrimitive: Uniform name "u_cylinderUvToShapeUvAngle" is already defined
VoxelPrimitive: Uniform name "u_cylinderShapeUvAngleMinMax" is already defined
VoxelPrimitive: Uniform name "u_cylinderShapeUvAngleRangeZeroMid" is already defined

Thanks for the feedback @ggetz! I think I addressed it all, and also added the missing specs.
I added an OrientedBoundingBox to the VoxelCell class, to expose the center point and dimensions of the picked sample.

Awesome, thanks @jjhembd !


[BUG] Deprecate Docker based image types from supported list for Node pools

Rancher Server Setup
Rancher version: 2.7.5-rc1
Installation option: Helm Chart
If Helm Chart, Kubernetes Cluster and version: GKE with Kubernetes 1.23.17-gke.1700
Information about the Cluster
Kubernetes version: GKE with Kubernetes 1.23.17-gke.1700
Cluster Type (Downstream): Hosted = GKE
User Information
What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom) - Cluster Owner
Describe the bug
This issue was found during https://github.com/rancher/rancher/issues/41554
Unable to create Node pool with Docker Image Type on v1.23, so we can remove it from image list as supported option
To Reproduce

Add valid GCP cloud credentials
Provision downstream GKE cluster with lowest k8s verison (1.23.17-gke.1700) Node pools image: COS with docker or Ubuntu docker

Result
Unable to create Node pool with Image type using Docker

Googleapi: Error 400: Creation of node pools using node images based on Docker container runtimes is not supported in GKE v1.23. This is to prepare for the removal of Dockershim in Kubernetes v1.24. We recommend that you migrate to image types based on Containerd (examples). For more information, contact Cloud Support., badRequest

Expected Result
Since Image type with Dokcer is unsupported in lowest version v1.23 which we use, it can be removed from supported list on UI so user does not have option to select it
Screenshots

This is a UI issue, please see https://github.com/rancher/gke-operator/issues/120#issuecomment-1829436468 for more details.

PR for UI https://github.com/rancher/ui/pull/5092 was merged, back to testing

Validated as fixed on build: v2.9-84884ba4ab3da2e7c8561abfe4e2428933688836-head
Docker-based images are not being listed from Node-pool creation, but only containerd :white_check_mark:


deprecation warning in embercli 2.18.0

ember install ember-async-button generates a deprecation warning in ember 2.18.0:
DEPRECATION: An addon is trying to access project.nodeModulesPath. This is not a reliable way to discover npm modules. Instead, consider doing: require("resolve").sync(something, { basedir: project.root }).
I believe updating ember-cli-babel to 6.11.0 will fix this as 6.8.2 depends on an older version of ember-cli-version-checker.

Since the OP seemed uncertain, let me confirm that upgrading ember-cli-babel should fix this issue.