cleanup: remove Mesos support from Falco.

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area engine
What this PR does / why we need it:
This follows up #2328 add deprecation warning for Mesos and completely removes the Mesos support from Falco.
/milestone 0.35.0
Does this PR introduce a user-facing change?:

update!: remove `--mesos-api`, `-pmesos`, and `-pm` command-line flags
BREAKING CHANGE: support for metadata enrichment from Mesos has been removed.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I noticed that we also have rules mentioning mesos: https://github.com/falcosecurity/rules/blob/main/rules/falco_rules.yaml#L1730 Are we going to clean them up too?
Btw thank you! /approve

I will take a look. Though, in that case, it seems to be legit (it refers to Mesos, the application, not our fields)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


2 Bugs Fixed

Fix bug: if exists embedded assemblies but doesn't exists encrypted resources program will cause errors.
Fix bug: file decrypt twice instead of decrypt and unpack.

I can send you exe-file examples for demonstrate if needed.

Alright, looks like it should work. Thanks for the fix!


How to make PhotoMessageView without any bubble background

Hello, thx for great work!
But it's very hard to understand some stuff without big documentation.
I want to create sticker messages and I think, that the easiest way for it  it's make custom config for PhotoMessageView. I want to remove all bubble experience for PhotoMessageView and set custom size. How can I do this things?

Hi,
Take a look at PhotoBubbleViewStyleProtocol
public protocol PhotoBubbleViewStyleProtocol {
    func maskingImage(viewModel viewModel: PhotoMessageViewModelProtocol) -> UIImage
    func borderImage(viewModel viewModel: PhotoMessageViewModelProtocol) -> UIImage?
    func placeholderBackgroundImage(viewModel viewModel: PhotoMessageViewModelProtocol) -> UIImage
    func placeholderIconImage(viewModel viewModel: PhotoMessageViewModelProtocol) -> (icon: UIImage?, tintColor: UIColor?)
    func tailWidth(viewModel viewModel: PhotoMessageViewModelProtocol) -> CGFloat
    func bubbleSize(viewModel viewModel: PhotoMessageViewModelProtocol) -> CGSize
    func progressIndicatorColor(viewModel viewModel: PhotoMessageViewModelProtocol) -> UIColor
    func overlayColor(viewModel viewModel: PhotoMessageViewModelProtocol) -> UIColor?
}

Some tips:
public func tailWidth(viewModel viewModel: PhotoMessageViewModelProtocol) -> CGFloat {
    return 0
}

public func borderImage(viewModel viewModel: PhotoMessageViewModelProtocol) -> UIImage? {
    return nil
}

public func maskingImage(viewModel viewModel: PhotoMessageViewModelProtocol) -> UIImage {
    return UIImage.bma_imageWithColor(.blackColor(), size: CGSize(width: 1, height: 1))
}

However, you may have some issues with the transferring status. We use a white circular upload progress on top of a darkening overlay that will look bad on top of your borderless sticker. If you disable this overlay color (overlayColor(viewModel:) -> UIColor? - just return nil) then the border will be fine, but the progress may not be visible depending on the color of the sticker. One solution to this would be to tint the sticker itself depending on the status of the message.
Reusing UI is complicated, there are so many details and product requirements that can change over time than trying to make a universal, super extensible component just doesn't make sense. It may be better to create your own StickerMessageCollectionViewCell

Hello, thx I will try to create StickerMessageCollectionViewCell.


Fix for Issue#6

Fixed notification for Android 7 and up
Fixed notificationChannel for Android 8.1

Fixes issue #6


v 1.10

Added optional checking for duplicate ids.
Changed Translations dictionary to concurrent.
Added test controllers for IOnAdd<> types.
Made repository more generic for expanded use.
Fixed delete to work with items that have children.

Reviewed


Module Create Failed - Service ID: 289 - Error: Failed to create the server, received the error code: 422. Enable module debug log for more info.

Hello, i have a error how can i fix it

?

Please use the Pterodactyl Discord (#whmcs-module channel) for configuration related support instead of GitHub issues.

Please use the Pterodactyl Discord (#whmcs-module channel) for configuration related support instead of GitHub issues.

Does not help!


Test Request: Kong Gateway EE version <IP_ADDRESS>-rc.2

What component needs testing with KIC?
Kong Gateway EE
Container image
kong/kong-gateway-dev:<IP_ADDRESS>-rc.2
Additional information
No response

Superseded by https://github.com/Kong/kubernetes-ingress-controller/issues/5028


WslRegisterDistribution failed with error: 0x80370102 on WSL 2

Windows Version
Microsoft Windows [Versione 10.0.19045.3208]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
No response
Other Software
No response
Repro Steps
wsl --install ubuntu (after downloaded)

Expected Behavior
I was expected to install the distro on wsl 2
Actual Behavior
i got the WslRegisterDistribution failed with error: 0x80370102
Diagnostic Logs
I can add some things useful as images:

I have the same issue here.

Thank you for reporting this @SaraSpena. My Italian isn't great, but based on what I see on the first screenshot the issue is that your hardware doesn't support virtualization. You probably need to enable it on your BIOS settings.

The issue Is not that. Is the First thing i've done months ago, enabling virtualization. Is not that simple😅 why did you close the ticket?
As you can see in the screenshot it says that the virtualization is enabled.

Oh ok I probably mis-interpreted the last line on the first screenshot. Re-opening.

Oh ok I probably mis-interpreted the last line on the first screenshot. Re-opening.

Still closed

I have the same issue, I also have AMD Ryzen 5 (virtualization is enabled) ; windows family 10 22h2 19045.3324

/logs

WslLogs-2023-08-26_10-32-44.zip
thank you

Thank you @Melfos-md. Did you reproduce the issue while the log collection was running ? I'm not seeing any WSL logs in the output.
/logs


Question about ApplyBackend for v1.0.0

Question
In the previous versions (for example v0.41) was trait:
/// EVM backend that can apply changes.
pub trait ApplyBackend {
	/// Apply given values and logs at backend.
	fn apply<A, I, L>(&mut self, values: A, logs: L, delete_empty: bool)
	where
		A: IntoIterator<Item = Apply<I>>,
		I: IntoIterator<Item = (H256, H256)>,
		L: IntoIterator<Item = Log>;
}

Current version v1.0.0-alpha.2 completely rafactored.
❓ Question is: as far as we will need to transfer from v0.x ➡  v1.0 we need to understand clearly how to do that. Actually for ApplyBackend it's unclear. It seems apply logic was completely changed. I would be pleased for some advice on that point.

As it is provided new trait: https://github.com/rust-ethereum/evm/blob/master/src/backend/mod.rs
And it is really unclear, because previously:
https://github.com/rust-ethereum/evm/blob/v0.36.0/src/backend/mod.rs#L30C13-L30C13
for apply operation was provided:
pub enum Apply<I> {
	/// Modify or create at address.
	Modify {
		/// Address.
		address: H160,
		/// Basic information of the address.
		basic: Basic,
		/// Code. `None` means leaving it unchanged.
		code: Option<Vec<u8>>,
		/// Storage iterator.
		storage: I,
		/// Whether storage should be wiped empty before applying the storage
		/// iterator.
		reset_storage: bool,
	},
	/// Delete address.
	Delete {
		/// Address.
		address: H160,
	},
}

for changes applying.


Fix spelling of errormessage in argument parsing

Trivial fix of a typo in one of the possible errormessage from ParseArguments().

On 11 Jul 2016, at 11:12, googlebot<EMAIL_ADDRESS>wrote:
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.
• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

I signed it!

Closing since zero fucks are given


Snowflake transient table description incorrectly states they are not involved in time travel

Hi,
The Snowflake configuration doc at https://docs.getdbt.com/reference/resource-configs/snowflake-configs states:

Snowflake does not preserve a history for these tables, which can result in a measurable reduction of your Snowflake storage costs. Note however that transient tables do not participate in Time Travel.

This is incorrect. Transient tables ARE automatically covered by time travel by default for a period of 1 day (this can be overridden to zero days). They do not participate in Fail Safe which means they are not recoverable by Snowflake support outside the time travel window (e.g. in the event of a catastrophic failure of the Snowflake platform or malicious or accidental data corruption).
See
https://docs.snowflake.com/en/user-guide/data-cdp-storage-costs.html
https://docs.snowflake.com/en/user-guide/data-failsafe.html
Personally I think that setting ALL tables to transient by default is a mistake, time travel is one of the key features of Snowflake and also many users will not understand the potential impact of not using Fail Safe, which could result in the permanent loss of data that would have otherwise been recoverable.
However as far as this doc is concerned it should describe the trade off being made accurately so users can make an informed choice about whether to keep this setting.

I've created a PR which I think better describes the trade-offs to consider when choosing to use transient tables by default:
https://github.com/dbt-labs/docs.getdbt.com/pull/1015

Thank you, @nathan-protempo! We really appreciate you spinning up a PR too 🤩  I will get someone to review your PR!


Fix unlink temporary files

Metadata extraction creates temporary files which fill up the disk. The files can be removed directly after metadata extraction via exif_read_data or getimagesize.

I think I found the problem. There are to many resources which caused a php out of memory fatal.
I'll have a look at how to improve the memory consumption of the metadata extraction CommandController.

I'll solve this on my own by cloning the CommandController and run extraction in a subprocess. Not the most elegant solution I think, but it works and has no memory issue.


add support for altering the rabbitmq login/password

RabbitMQ recommends deleting the guest account, or at least changing the password. https://www.rabbitmq.com/access-control.html
These changes add support for using a different rabbitmq account via the config file, I did not add any command line options.

Thanks for the submission! We're closing old PRs, we've set up a new mechanism to test each PR, and most of these ones have conflicts with master. Feel free to open an issue or submit a new PR again!
Sorry for the inconvenience.


Tabs Support

Loving Dartpad as a way to learn Dart on the fly :)
My request:
I would like to see tabs support added as an option, so we can practice OOP across multiple files with Dart
Thanks

Glad you're enjoying DartPad!
Multiple tabs is something we've looked at doing, but doesn't really make sense for DartPad's mission to be a simple playground for experimentation. There are some projects in the works from other teams that have more fully featured IDE-like experiences, so those might be an option.
Thanks for the suggestion, though. Don't be shy if there's another feature you'd like to see!


Don't clear access token on intermittent connection problem

When HomeAssistant is restarting or just there is a problem with
connection, TileBoard assumed that user needs to get authenticated
and tried to fetch new token after deleting current one.
Don't do that unless the response code is in 4xx range.
Per HomeAssistant documentation[1], it will return 400, 401 or 403
when token is expired or there is other authentication error.
Also don't react to state changes until response is in DONE state.
[1] https://developers.home-assistant.io/docs/en/auth_api.html#making-authenticated-requests

Yet again, thank you!


Undefined Variable when building with GateLogging enabled in Simulator.hpp

Description
QNLP/modules/simulator/IntelSimulator.cpp line 168: undefined reference to variable qubitIndex.
Undefined reference arises when the QNLP library is built with GateLogging enabled due to qubitIndex being undeclared. Variable should be qubit_idx to remedy this.
To reproduce
Steps to reproduce the behavior:

Build application using Cmake with -DENABLE_LOGGING=1 to enable Gate Logging functionality.
make
See undefined reference error for QNLP/modules/simulator/IntelSimulator.cpp line 168

Expected behavior
Expected behaviour would be for this compile time error to not arise; instead the variable qubit_idx passed into the applyGatePhaseShift on line 156 should replace the undefined variable qubitIndex on line 168.

Solved with merge #3


Add LIBLITERALPREFIX to support gcc -l:filename

This is done in a general way, using a construction var, although at the moment only the GNU linker is known to handle things this way.
Had to do something funky, or it won't work when os.pathsep and LIBLITRALPREFIX are the same, as they are on Linux (i.e. ':'). That's because SCons.PathList.PathList is called and it does:
class _Pathlist:
    def __init__(self, pathlist) -> None:
        if SCons.Util.is_String(pathlist):
            pathlist = pathlist.split(os.pathsep)

Splitting has the effect of turning :libm.a into ((0, ''), (0, 'libm.a')] That is, the ':' is lost as part of the library specifier - so need to try to avoid that.
Fixes #3951
Submitted for initial review so no docs or changelogs yet.
Contributor Checklist:

[X] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated CHANGES.txt (and read the README.rst)
[X] I have updated the appropriate documentation

Would be better to add another parameter to _stripixes() instead of having an envar LIBLITERAL.. That way any users usage of _stripixes() won't be affected by LIBLITERAL. ?

Not sure that I follow the benefits of that.  Somewhere, these things need to happen:

Need to inform SCons of a compiler-specific (actually, linker-specific) bit of syntax, which may be empty. Construction vars are the way SCons normally does that.
That syntax needs to be conveyed to _stripixes so it doesn't transform libfoo.a to foo.
If _stripixes calls PathList (which it does currently), the _PathList initializer needs to be told not to strip the linker syntax from the arguments, if it might do that. Which for the GNU linker on non-WIndows, it would, since the same character is used for two purposes. PathList is called a bunch of places, it's a little odd that it's called with a list of library names, but things broke if I had it stop doing that, didn't research why in any great depth.

Doing this as an argument to _stripixes means we need to be able to pass that in when needed, but we don't ever call _stripixes directly, it's only embedded in macro-like consvars which will be expanded at build time (currently _LIBFLAGS and _DLIBFLAGS). So you'd have to change the definition of those, which seems just as (or even more) fragile, as users are allowed to define their own _LIBFLAGS and wouldn't know they might need to change those definitions. And to do the subtitution when _LIBFLAGS is evaluated to pass through that magic indicator, you pretty much still need a consvar.

Not sure that I follow the benefits of that. Somewhere, these things need to happen:

Need to inform SCons of a compiler-specific (actually, linker-specific) bit of syntax, which may be empty. Construction vars are the way SCons normally does that.
That syntax needs to be conveyed to _stripixes so it doesn't transform libfoo.a to foo.
If _stripixes calls PathList (which it does currently), the _PathList initializer needs to be told not to strip the linker syntax from the arguments, if it might do that. Which for the GNU linker on non-WIndows, it would, since the same character is used for two purposes. PathList is called a bunch of places, it's a little odd that it's called with a list of library names, but things broke if I had it stop doing that, didn't research why in any great depth.

Doing this as an argument to _stripixes means we need to be able to pass that in when needed, but we don't ever call _stripixes directly, it's only embedded in macro-like consvars which will be expanded at build time (currently _LIBFLAGS and _DLIBFLAGS). So you'd have to change the definition of those, which seems just as (or even more) fragile, as users are allowed to define their own _LIBFLAGS and wouldn't know they might need to change those definitions. And to do the subtitution when _LIBFLAGS is evaluated to pass through that magic indicator, you pretty much still need a consvar.

Disagree.. You can note to users that another argument is available, but default it in the arglist to allow it to not be passed.
Then there's no magic happening.
Otherwise if this function is used for purposes other than the one's SCons itself is, there may be unpleasant surprises..
I'd like to avoid those surprises..

Ok. Finally figured out why this solution was bugging me (besides already stated reasons).
Since LIBLITERAL's not on the command line in any way, if you change it's value, you won't get a rebuild.
Thus the justification for adding it as an optional variable to _stripixes

doesn't make any sense to change it, it's internal to the linker. if you change the linker to get different value, you will get rebuilds, since binary's sig will be different (hypothetical at moment)
only sane way to do this is add a new library holder type so libs can have attributes, but that's a lot more work, and api impact.

Since LIBLITERAL's not on the command line in any way, if you change it's value, you won't get a rebuild.

Not completely sure I understand this part of the comment, but perhaps a small sample will illustrate:
$ cat SConstruct
env = Environment(LIBLITERAL=":")
env.Program(target="hello", source="hello.c", LIBS=":libm.a")
$ scons -Q
gcc -o hello.o -c hello.c
gcc -o hello hello.o -l:libm.a
$ scons -Q
scons: `.' is up to date.
$ #EDIT SConstruct
$ cat SConstruct
env = Environment(LIBLITERAL="+")
env.Program(target="hello", source="hello.c", LIBS="+libm.a")
$ scons -Qn
gcc -o hello hello.o -l+libm.a
$

We've first set it to a :, and since the PR changes mean it's not stripped, it does pass through to the command line. Then change it to + after a build is previously shown as up to date, and it does force a rebuild, and you can see the + present in proposed command for the subsequent run (scons -n since the new literal-introducer of course wouldn't work, since it's unknown to ld).

Hmmm, merged the PR which seemed okay, but now it's failing tests, specifically test/ninja/build_libraries.py in the "experimental" set, and a whole bunch more in the full run. @bdbaddog are you able to get a quick look at that? Else I'll swing back to it later.

Okay, the failure comes down to - when _stripixes is handed a string if goes through a call chain that ends up making a StringSubber instance, and then calling its expand method.  That does:
     try:
          s = eval(key, self.gvars, lvars)
     except KeyboardInterrupt:
          raise
     except Exception as e:
          if e.__class__ in AllowableExceptions:
               return ''
          raise_exception(e, lvars['TARGETS'], old_s)

The eval takes a NameError when LIBLITERAL is in the string to be expanded, but is not in the construction environment, whose contents end up in self.gvars, so not in self.gvars``. The NameError` is one of the allowable exceptions, so instead of returning failure here, we return an empty string.
The conclusion I make is if we're going to use LIBLITERAL in an expansion like this, it must be defined in the environment.  I experiment with initializing it in the platform modules for posix and win32, and that makes the test suite go back to running normally instead of taking 40-ish failures.

@mwichmann Should we also set LIBLITERALPREFIX to : in the gnulink tool?


Update color tokens

Colour tokens have been updated for all components that are not archived
https://www.figma.com/file/mh2maMG2NBtk41k1O1UGHV/Canadian-Digital-Service
---GC-Design-System?type=design&node-id=4-1006&t=xlzmraTw5sdQPYwf-0


Vue Router breaks Hot Reload with circular Router references

Version
4.1.6
Reproduction link
github.com
Steps to reproduce
Link contains stock CLI Vue project with vue-router. I've modified the HomeView.vue to import and reference the vue router in order to navigate to another page off a button.
This all works with standard navigation.
However, now open the console with the HomeView open, then make a change to the page which should trigger hot-reload to attempt to reload the page.
Instead an error occurs and the page does not actually reload:

Now remove the import router and the router.push() call from the page.
Make another change - now hot reload works. Add it back in... boom.
What is expected?
Hot reload should work when router is imported and referenced.
What is actually happening?
Hot reload fails when any router referenced component or sub-component is reloaded.

use useRouter() instead of importing the router, it avoids circular dependencies which is a problem out of the router's scope.
BTW: your reproduction link seems private


KSVC failed to start behind proxy with RevisionFailed reason: failed to resolve image to digest

Bug Description
During working on https://github.com/canonical/charmed-kubeflow-uats/issues/75, the kserve UAT fails behind proxy due to Knative Service failing to start.
The Knative Service is not Ready with the RevisionFailed reason.
The logs show that the Knative Serving controller is failing to resolve the image tag to digest for charmedkubeflow/sklearnserver:0.11.2-e54c69e image.
To Reproduce

Deploy kubeflow bundle 1.9/beta behind proxy
Run the KServe UAT
Describe the ksvc

Environment
microk8s 1.29-strict/stable
juju 3.4.4
CKF 1.9/beta
Relevant Log Output
Revision "sklearn-iris-predictor-00001" failed with message: Unable to fetch image "charmedkubeflow/sklearnserver:0.11.2-e54c69e": failed to resolve image to digest: Get "https://index.docker.io/v2/": context deadline exceeded.

Additional Context
No response

The knative service is failing to start because it is unable to resolve the image digest, we expect this to be due to not being able to reach the internet to get the digest.
What we can do here:

Configure the proxy envs in the serving controller - there are some pointers on doing this in the knative docs here and here
Skip tag resolution for the server registries to avoid bumping into the issue

Experimenting with option (1)
to configure the proxy envs in the serving controller:

edited the knative-serving CR manifest as follows:

--- a/charms/knative-serving/src/manifests/KnativeServing.yaml.j2
+++ b/charms/knative-serving/src/manifests/KnativeServing.yaml.j2
@@ -6,6 +6,17 @@ metadata:
   namespace: {{ serving_namespace }}
 spec:
   version: {{ serving_version }}
+  workloads:
+  - name: controller
+    env:
+    - container: controller
+      envVars:
+      - name: HTTP_PROXY
+        value: http://<IP_ADDRESS>:3128
+      - name: HTTPS_PROXY
+        value: http://<IP_ADDRESS>:3128
+      - name: NO_PROXY
+        value: <IP_ADDRESS>/24
   config:
     deployment:
       progress-deadline: {{ progress_deadline}}

where:

HTTP_PROXY and HTTPS_PROXY have the values of the proxy server
NO_PROXY has the value of the service cluster ip range:

cat /var/snap/microk8s/current/args/kube-apiserver | grep service-cluster-ip-range
--service-cluster-ip-range=<IP_ADDRESS>/24

re-packed the knative-serving charm
refreshed the knative-serving charm from latest/beta to the local one
Ran the kserve UATs with the modification of adding the proxy envs to the V1ObjectMeta of ISVC definition

Results

I no longer see the image tag resolution error
the KSVC is in Unknown state with Reason RevisionMissing
describing the KSVC I see:

Name:         sklearn-iris-predictor
Namespace:    admin
Labels:       component=predictor
              notebook-proxy=true
              serving.kserve.io/inferenceservice=sklearn-iris
Annotations:  serving.knative.dev/creator: system:serviceaccount:kubeflow:kserve-controller
              serving.knative.dev/lastModifier: system:serviceaccount:kubeflow:kserve-controller
API Version:  serving.knative.dev/v1
Kind:         Service
Metadata:
  Creation Timestamp:  2024-07-24T10:42:30Z
  Generation:          1
  Owner References:
    API Version:           serving.kserve.io/v1beta1
    Block Owner Deletion:  true
    Controller:            true
    Kind:                  InferenceService
    Name:                  sklearn-iris
    UID:                   3a249bb2-f2f5-492d-a48f-7ac5943822dc
  Resource Version:        87248
  UID:                     aa001a12-98e7-4c22-ba3d-922c23b91215
Spec:
  Template:
    Metadata:
      Annotations:
        autoscaling.knative.dev/class:                             kpa.autoscaling.knative.dev
        autoscaling.knative.dev/min-scale:                         1
        internal.serving.kserve.io/storage-initializer-sourceuri:  gs://kfserving-examples/models/sklearn/1.0/model
        prometheus.kserve.io/path:                                 /metrics
        prometheus.kserve.io/port:                                 8080
        sidecar.istio.io/inject:                                   false
      Creation Timestamp:                                          <nil>
      Labels:
        Component:                           predictor
        Notebook - Proxy:                    true
        serving.kserve.io/inferenceservice:  sklearn-iris
    Spec:
      Container Concurrency:  0
      Containers:
        Args:
          --model_name=sklearn-iris
          --model_dir=/mnt/models
          --http_port=8080
        Image:  charmedkubeflow/sklearnserver:0.13.0-119414c
        Name:   kserve-container
        Readiness Probe:
          Success Threshold:  1
          Tcp Socket:
            Port:  0
        Resources:
          Limits:
            Cpu:     1
            Memory:  2Gi
          Requests:
            Cpu:             1
            Memory:          2Gi
      Enable Service Links:  false
      Timeout Seconds:       300
  Traffic:
    Latest Revision:  true
    Percent:          100
Status:
  Conditions:
    Last Transition Time:        2024-07-24T10:42:31Z
    Status:                      Unknown
    Type:                        ConfigurationsReady
    Last Transition Time:        2024-07-24T10:42:31Z
    Message:                     Configuration "sklearn-iris-predictor" is waiting for a Revision to become ready.
    Reason:                      RevisionMissing
    Status:                      Unknown
    Type:                        Ready
    Last Transition Time:        2024-07-24T10:42:31Z
    Message:                     Configuration "sklearn-iris-predictor" is waiting for a Revision to become ready.
    Reason:                      RevisionMissing
    Status:                      Unknown
    Type:                        RoutesReady
  Latest Created Revision Name:  sklearn-iris-predictor-00001
  Observed Generation:           1
  URL:                           http://sklearn-iris-predictor.admin.<IP_ADDRESS>.nip.io
Events:
  Type    Reason   Age   From                Message
  ----    ------   ----  ----                -------
  Normal  Created  5m3s  service-controller  Created Configuration "sklearn-iris-predictor"
  Normal  Created  5m3s  service-controller  Created Route "sklearn-iris-predictor"

Conclusion
This issue expanded to be 2 issues in fact that are currently blocking serving in CKF from working correctly behind proxy:

knative tag-to-digest resolution failing
kserve model download failing
Both issues are due to the pods responsible for pulling these artifacts/data not being able to establish a connection to their targets, and thus they need to have the proxy env vars set in order to unblock.

Proposed fix

Introduce a proxy and no-proxy config to kserve-controller and knative-serving charms
Modify the manifests template for each of:

KnativeServing.yaml.j2 in knative-serving charm
cluster_storage_containers.yaml.j2 in kserve-controller charm
to have the proxy env vars set optionally, if they are set in the charm config.

@NohaIhab excellent job getting to the bottom of it and documenting everything!
I also agree with adding config options for the proxy to the kserve-controller and kserve-serving charms, which will end up altering the applied manifests. It's actually quite nice to see the charms to abstract this and only have a lean configuration for the proxy to the user!
My only concern is if we should have just one proxy config option, or make it 1-1 with the env var and introduce the following config options: http-proxy, https-proxy, no-proxy

@kimwnasptd good point, I think we can even make it more clear by specifying for which container this proxy env is being set. For exmaple in knative-serving, make the config name controller-http-proxy, controller-https-proxy, and controller-no-proxy.
In kserve-controller, it can be storage-initializer-http-proxy .. etc.

IMO sticking to just expressing the functionality, and not also including the component, in the config name will provide a better UX.
The users don't necessarily care (AFAIK) for configuring different proxy settings for the different components of the same Knative Charm. They'd care about telling the charm what the proxy values are and then it's up to the Charm to do all the configurations wherever necessary.
The above is for a hypothetical scenario that in the future we might need storage-initializer-http-proxy and queue-http-proxy (random thought), for the kserve-controller charm. Users wouldn't care to put different values for the different sub-components and would only want to pass them once to the kserve-controller charm and then the charm configuring everything.

Closed by above linked PRs


Get the dimensions of an unread IO stream by reading ahead

Before:
>> Dimensions(File.open("masterpiece.png", "rb")).dimensions
=> nil

After:
>> Dimensions(File.open("masterpiece.png", "rb")).dimensions
=> [522, 606]

@javan why you set the maximum number to 6144?
I had to set it to 31745 to get the dimensions of http://media.palau.org/uploads/photos/JuddHealdHiRes.jpg
@sstephenson can I arbitrarily increase it to 31745?


[improve] [ml] Persist mark deleted ops to ZK if create cursor ledger was failed

Motivation
The progress Persist mark deleted position is like this:

persist to BK
if failed to persist to BK, try to persist to ZK

But in the current implementation: if the cursor ledger was created failed, Pulsar will not try to persist to ZK. It makes if the cursor ledger created fails, a lot of ack records can not be persisted, and we will get a lot of repeat consumption after the BK recover.
Modifications
Try to persist the mark deleted position to ZK if the cursor ledger was created failed
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: x

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

Rebase master

Codecov Report

Merging #20935 (c29f906) into master (dab5b2f) will increase coverage by 0.11%.
Report is 4 commits behind head on master.
The diff coverage is 86.84%.

@@             Coverage Diff              @@
##             master   #20935      +/-   ##
============================================
+ Coverage     73.03%   73.14%   +0.11%
- Complexity    32363    32392      +29
============================================
  Files          1887     1887
  Lines        139860   139879      +19
  Branches      15384    15389       +5
============================================
+ Hits         102140   102316     +176
+ Misses        29634    29477     -157
  Partials       8086     8086

Flag
Coverage Δ

inttests
24.08% <5.26%> (?)

systests
25.11% <5.26%> (+<0.01%)
:arrow_up:

unittests
72.44% <86.84%> (+0.04%)
:arrow_up:

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

Files Changed
Coverage Δ

...che/bookkeeper/mledger/impl/ManagedLedgerImpl.java
81.22% <83.33%> (-0.08%)
:arrow_down:

...che/bookkeeper/mledger/impl/ManagedCursorImpl.java
79.23% <88.46%> (+0.25%)
:arrow_up:

... and 77 files with indirect coverage changes


fix: use correct type in extends directive for __newArray

Closes: #21

📦 Published PR as canary version: 0.7.2-canary.22.ef4db40.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in v0.7.2 :rocket:


No automatic wallet backups since March 22nd (before <IP_ADDRESS>)

My wallet backups folder (C:\Users\xxx\AppData\Roaming\GridcoinResearch\walletbackups) hasn't added any new files since March 22 (I think this was the last date that I was in sync prior to <IP_ADDRESS>).
Everything else seems fine. I'm using Windows 10, v<IP_ADDRESS>.

you are out of sync?

No, I'm in sync and the wallet is functioning. The problem is with the wallet backups folder. It used to automatically add backups of the wallet.dat and conf file. Now it doesn't.

my wallet and config backs up find in linux as well. in debug.log it should tell you the reason its not backing up successfully. it doesn't need debug flags to show it. perhaps browse through your debug and tell us what it is saying as the code prints the reason why also in debug.

My debug.log is 7,326 KB. Is there a specific string of text I should search for? Is it safe to attach the file here or are there sensitive things I should delete first?

ill show u an example of mine:
2017-05-13 10:18:21  {SVC} Starting Wallet Backup
is where it starts in debug.log search for that

No luck. "{SVC}" returns nothing. "Backup" returns nothing.

I tried a few more. "Unsuccessful" returns nothing. "fail" returned one thing: 05/13/17 23:02:59 ERROR: AcceptToMemoryPool : CheckTransaction failed

AcceptToMemoryPool is not an issue it happens frequently. idk why the backups are not working seems like its not being called. its suppose to do it every 900 blocks

I've been getting some client crashes lately, so I doubt that I've ran the wallet continuously for 900 blocks in a row lately. Would that make a difference?

my assumption is yes that could do it. try recording the block when u start the wallet and if u make it past 900 from next crash and it does backup then u have the reason. so if thats the case a crash fix would fix this and it wouldn't be a bug after all :)

ok, i'll try, and if it crashes i'll look for clues in the debug log

db.log tends to have crash logs if any i believe

ok, i'll keep an eye on that. it's currently empty.

alright if it is empty then its probably a crash its not able to record on. make sure u do let us know what the crash is exactly any info helps :) good luck

Unfortunately, it crashed overnight again. It ran for a little over 7 hours, from block 898046 to 898309. None of the normal shutdown procedure data in debug.log and nothing in the db.log.

I feel this is from the .NET code NN syncing. It crashes every time in my Win7 virtualbox.

@jimbo-slice if you check the application log in the event viewer there is a good chance the application exited with a .NET exception error. As @tomasbrod points out, this is likely caused by the NN .NET code.
I was having the same issue a couple of week ago and the thing that solved it for me was to do a re-sync of the wallet using a snapshot (removing everything except gridcoinresearch.conf, wallet.dat and walletbackups from the data folder.

3ullShark Thanks. Can you walk me through this? Is the file you're referring to debug.log or something else?

@jimbo-slice it's in the Windows Event Viewer.

Open Event Viewer by searching for Event Viewer in the search box in the taskbar.
Expand the Windows Logs tree item.
Click on Application.

Now look through the list for any errors around the time of exiting that are to do with .NET and gridcoinresearch.exe

thanks a lot, yeah here it is:
3:12:53 AM .NET Runtime
Application: gridcoinresearch.exe
Framework Version: V4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: exception code c0000005, exception address 004B3DFC
so, next step is to re-sync, right?

@jimbo-slice that's what worked for me. Although I'm not sure if it was the re-sync that helped or because I deleted the NeuralNetwork folder during the process.

Ok. I'll move the NN folder to another location and restart the wallet and see if it works. Does that make sense?
And who knows,  maybe when I get the client to stop crashing the walletbackups issue will be fixed too.

@3ullShark nice job stepping up :)
@jimbo-slice good luck :)

It worked. The wallet was able to run long enough to create an automatic backup (after slightly less than 900 blocks, ~875-ish). The problem all along was the .net runtime crashes.

Hrm less then 900 blocks? maybe when a reorganize of blocks occurs and blocks are disconnected and reconnected the count is not affected meaning it can occur at less the 900 blocks :) well glad this is figured out and I could be of help.


Broadcast event example fails

When I run the broadcast event example in docs/examples/event_broadcast.py it dies with the following message:
nameko.events.EventHandlerConfigurationError: Broadcast event handlers cannot be configured with reliable delivery.

Hi @morty, thanks for reporting. There is an open pull request that fixes this and a few of the other examples. See #250
It's currently blocked on a pytest-eventlet interaction bug, but a fix is in the works

Hi @mattbennett, I don't think that PR fixes the issue I'm seeing. It just switches to using the print statement.

@mattbennett OK, now I see events.py is changed too.


Does RedisCluster support password?

I am using RedisCluster. It works fine when redis is not password protected. But when we make cluster protected and I add password in application.conf as mentioned in document. It doesn't seem to work.
scredis {
  redis {
    # Redis server address
    host = "hostname"
    # Redis server port
    #port = "7387"
    password = "redispassword"
    # List of hosts or host:port tuples, which serve as seed nodes for a Redis cluster connection.
    # If port is missing, the configured port is used as default.
    #cluster-nodes = [${scredis.redis.host}":"${scredis.redis.port}]
    cluster-nodes = ["hostname1:7385","hostname2:7386","hostname3:7387"]
    #cluster-nodes = ${?REDIS_CLUSTER_NODES}
  }
}

Then I use below code to initialized redis client -
  RedisCluster redisClient = RedisCluster(providedActorSystem = providedActorSystem)
I get an exception that password is required
The future returned an exception of type: scredis.exceptions.RedisErrorResponseException, with message: NOAUTH Authentication required..

Upon further debugging the code, I noticed that passwordOpt = None is being provided to AkkaNonBlockingConnection in ClusterConnection.makeConnection
Is there any reason that config password is not being used?
private def makeConnection(server: Server, systemOpt:Option[ActorSystem]): NonBlockingConnection = {
    val systemName = RedisConfigDefaults.IO.Akka.ActorSystemName
    val system = systemOpt.getOrElse(ActorSystem(UniqueNameGenerator.getUniqueName(systemName)))

    new AkkaNonBlockingConnection(
      system = system, host = server.host, port = server.port, passwordOpt = None,
      database = 0, nameOpt = None, decodersCount = 2,
      receiveTimeoutOpt, connectTimeout, maxWriteBatchSize, tcpSendBufferSizeHint,
      tcpReceiveBufferSizeHint, akkaListenerDispatcherPath, akkaIODispatcherPath,
      akkaDecoderDispatcherPath, failCommandOnConnecting
    ) {}
  }

If I recall correctly passwords weren't supported for clusters at the time this was implemented. Has this changed?

We are able to set up redis cluster with password based on steps outlined here
Just to test out quickly, I downloaded the scredis source code and updated that line of code to start passing passwordOpt. And it works! But not sure if there are cases where this might not be enough.

As authorization for cluster connection was not supported at all this definitely improves current state of things, hopefully if will cover all cases we will see
Could you create a PR for actually using a password from configuration?
It can be more tricky when you have different passwords for different instances or you want to change password using auth message, but something to be checked later.
Also thanks for reporting this issue :+1:

Sorry for late response. I am tied up with some high priority task. I will create PR  soon.

I guess there will be no PR from indusbull, will try my best to work on that tomorrow

My apologies for late response. I have created pull request. Please note that we have not tested this change in our production system yet.

@kpbochenek as PR is openned can you after merge push to maven ? even some snapshot etc version, thanks!

@maciejbak85 pls try 2.2.2+2-c1ad2e4e (you need bintray resolver in sbt)
If all looks good for you I will do proper release.
https://bintray.com/scredis/maven/scredis/2.2.2+2-c1ad2e4e

Fix released with version 2.2.3 and pushed to maven central.
Closing this issue, if you find something not working please open a new one.

yes I`ve tried already :) Works fine ! thanks


Frame.pdu_* properties should be considered for a tuple

#52 adds db.Frame.  The pdu properties act as a parallel array.  These should be exposed as a namedtuple

Per the latest C API docs (17.0), it sounds like CAN and LIN are currently limited to a single PDU mapped to a frame.
http://zone.ni.com/reference/en-XX/help/372841P-01/nixnet/nxpropfrm_pdurefs/
Thus, the user's options are:

Map a single PDU to a frame.
Skip the PDU and add signals and subframes directly to the frame

Only FlexRay allows one or more PDUs to be mapped to one or more frames. We should verify that we can use a PDU with a frame in CAN/LIN, and see if this could possible change in the future. Options are:

Return a list of named tuples
Return a single named tuple
Keep the properties separate and return a single element (pdu, pdu_start_bit, pdu_update_bit)

Something to keep in mind: do we want functionality parity with the database editor? If the database editor allows multi-mapping between PDUs and frames, are how is the customer experience with our python API choices?

CAN and LIN both require the "PDUs required?" property to be false. I believe you can still use the database objects to create sessions etc. In LabVIEW we have the Create Session (PDU In/Out Single-Point) family of VIs that use PDUs as inputs and they do work with LIN and CAN so long as "PDUs required?" is false.

And, to be clear, PDUs required? is a read-only boolean property that is False when

Only one PDU is mapped to the frame.
This PDU is not mapped to other frames.
The PDU Start Bit in the frame is 0.
The PDU Update Bit is not used.

So, a very limited case, but it does sound possible for CAN/LIN in the C API as well.

Fixed in #245


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1427 ms

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


patch(cfp): adjust dates for talk proposals

Further discussion happening in the Slack channel, but we're keeping the talk track submissions open a bit longer.

/approve

er forgot lgtm
/lgtm


Add Get-WTCurrentRelease

get the current WT release along with a  -preview option.

Added in v1.5.0


Fix documentation mismatch on api.find_first TheHive-Project/TheHive4py#165

Fixes: TheHive-Project/TheHive4py#165

This PR changes the api.find_first function to return a requests.Response object instead of a Python dict as it should be according to the docstrings.
Additionally, the previous behaviour was to still query for either all cases matching the query or a user specified range and then return only the first element from the requests.Response object.
This PR changes the logic by overwriting the range parameter to 0-1 which means we actually ask The Hive to only provide one case.
Removed the range parameter from the docstrings Arguments list, since we now set it statically.

Hello @haam3r thanks for the PR, the issue here is that this breaks compatibility.
The original find_first is bad. Updating is that way breaks compatibility.
We need to be careful about it.

Okay, that's understandable. Would you like me to change the PR in some way?


Moment of Inertia

Is your feature request related to a problem? Please describe.
There is no feature to calculate the moment of inertia of various different objects.
Describe the solution you'd like
There can be a feature to calculate the moment of inertia of different bodies like a ring, disc, solid cylinder, rectangular plate, rod, solid sphere etc.

can I work on this issue to add this feature with several options of Ring, Disc, Hollow cylinder, triangular lamina, spherical shell, rod, solid sphere, ring, disc etc.

@akashsahu006 u can.
join the discord channel for further doubts . link is in the readme

@akashsahu006 add it just like work power energy section


[CSS] Comma Combinator

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[x] New documentation
[ ] Correction or update

Details

I've updates the CSS selectors page. One of the changes was to a missing "Comma Combinator" that selects all the elements. This one exists since CSS Level 1 (reference), but it had no page in MDN.
Please create a new page, I will add the content to the page.

There isn't a comma combinator in CSS. You can see the list of Combinators in the CSS Selectors Spec Level 3 https://www.w3.org/TR/selectors-3/#combinators
What you are describing is a group of individual selectors as a comma separated list https://www.w3.org/TR/selectors-3/#grouping

I've reverted the changes.

@chrisdavidmills Why did you reverted all the changes? you could remove only the wrong one.

@ramiy my apologies; I was a bit hasty there. I'll revert back, and then just remove the bit about "comma combinator"

@rachelandrew @chrisdavidmills I've opened a new issue to create a page for the "Grouping Selectors". Please review.


PowerToys Run

Microsoft PowerToys version
0.62.1
Running as admin

[ ] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce

install a software with an exe installer (e.g. MarkText or Clash For Windows)
search this software in powertoys run, then it will give the shortcut on the desktop
delete the shortcut on the desktop
search this software in powertoys run, then it cannot find this software
restart powertoys, it works as expected

✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

/bugreport

PowerToysReport_2022-10-01-14-47-57.zip

From the logs the error related to MarkText is the following and not related to shortcut delete.
[2022-10-01 14:46:17.6434] [ERROR] [C:\a\_work\1\s\src\modules\launcher\Wox.Infrastructure\Storage\ListRepository`1.cs::54] Item Already Exists <MarkText.lnk>

From the logs the error related to MarkText is the following and not related to shortcut delete.

But I am sure the result it gives before restarting PowerToys is the shortcut on the desktop.

Are you still experiencing this issue? /needinfo


Add initial chest placements and layout for placement rules

Currently added:

Chest Placements

Closed, moved back (I know, indecisive) to mapmaker project @GreatWyrm


Remove dead code from sys/value/src/Eval.hs

See for instance "<>"

fixed with with pull request #220


[Ignore] Player cannot choose filled spot

Added in method is_spot_taken to check whether player_one or player_two mark is already in the spot they chose.

Refactored in other PR's - not needed


chore: add initial badges

PR to add badges top of README to track activity, job status, etc to solve issue #93

Thanks for this @xoanmm.
I think the badges that deliver most value are the following. Can you please update the PR so we have only these ones included? thanks!

Codespell
E2E
Go Report Card
Commit Activity

Hello @nimakaviani
I have updated the PR with the requested badges :)


Subscriptions support/example?

Is there an example of how to use ferry with subscriptions please? I didn't see any mention in the docs

Since ferry uses gql_link for network requests, you can use gql_websocket_link for subscriptions and pass your link to ferry just like any other link.


Improved dot printing and better cycle detection

Description
This improves the dot printing and solves possible issues with an over-aggressive cycle detection (i.e. detecting cycles where there are none).

Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance
no hot path code changed

Coverage decreased (-0.06%) to 83.482% when pulling 4edb5c7e0963ba1a9c7ecaef3f99e79478f8ea3a on better-dot into 2a2b431e42ab6d6c7859098deb5fade951e0a495 on main.

There were >100 clippy oversights with some outdated allows, they get removed :D


router needed/replaced?

Why is the router needed?
Could you replace it by an better designed Ingress or if it is not possible with an Ingress, then with an Gateway API ?

Router can indeed be replaced by a configuration of ingress. However OpenObserve can be run in multiple environments. With multiple ingress (nginx, haproxy, ...) it will be difficult to support a large number of users with different kind of environments. Providing a router basically solves that problem.

We are talking about this helmchart and the Ingress or GatewayAPI.
So every Ingress-Controller support it, like traefik, nginx and haproxy.
if you talk about a deployment on a VM, then it will make a sense.
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: openoberve
spec:
  rules:
    - host: "openobserve.example.org"
      http:
        paths:
          - path: "/summary"
            pathType: Prefix
            backend:
              service:
                name: "openobserve-querier"
                port:
                  name: http
          - path: "/schema"
            pathType: Prefix
            backend:
              service:
                name: "openobserve-querier"
                port:
                  name: http
          # ...
          - path: "/"
            pathType: Prefix
            backend:
              service:
                name: "openobserve-ingester"
                port:
                  name: http

Yes, I understand what you are saying and we had done similar to this in the initial days. Couple of scenarios:
You have to do ingestion from within the cluster.
Installing on VMs
Explaining this minor additional complexity to all the users.
Use service type load balancer
We will add documentation for people who want to remove the extra hop. Some people would definitely want to do that. Most people in our experience though would prefer simplicity of not knowing more details as long as it works efficiently.


Automated Test in Circle CI

with Ubuntu 18.04 LTS

It'll resolve #27.

see my repo's commits.
https://github.com/takano32/vscovid-crawler/commits/circleci-project-setup


Apply RLE on timestamp column of bulk transfer

Timestamps for aggregation queries are mostly the same. RLE encoding will reduce size of transfer data in between.

Done


[TEVA 1961] ensure no horizontal scroll on jobseeker dashboard

https://dfedigital.atlassian.net/browse/TEVA-1961

Review app deployed to https://teaching-vacancies-review-pr-2795.london.cloudapps.digital


will you add a plan for driver?

Will the network stack, file system and other hardware drivers be integrated later? This is what I care about.

I have this plan, the next step will support file operations.

Now Berry already has a file system interface. The interpreter is cross-platform and you can compile directly on your target system.
Hardware drivers are usually easier to make, just make the driver interface into a Berry native function.


Add false-alarm CVE to allowlist

This image is failing the CI, we should probably fix the critical vulnerability before we can push it into the cluster.

Closed! Thanks @Jose-Matsuda

Looked into it for a sec and it seems like it's the same one Jessica encountered a while ago where it was a false positive.
Sources:
https://github.com/StatCan/kubeflow-pipelines/runs/2723098758
https://github.com/advisories/GHSA-8v27-2fg9-7h62
https://github.com/browserify/static-eval/issues/34

We need to add this to the allowlist so that we can deploy


[BUG] The scroll fade in the channel list doesn't disappear

Client Mod
Powercord
Description of bug
In a server with lots of channels, there is a fade at the bottom of the channel list, causing the last channel to be dimmer than the rest and also making it harder to see
Steps to reproduce
Go into a server with enough channels that it causes an overflow
Scroll to the bottom
Suspected cause
No response
Screenshots

This issue is unique

[X] I have searched the issue tracker and did not find an issue describing my bug.

This is an intended feature. This does not affect interaction with a channel or voice channel. This will not be changed.


Add rules for arcs_manifest/particle/deployable

Adds arcs_manifest/arcs_kt_particle/arcs_deployable for compiling schemas, kotlin to both JVM and WASM.
Dataflow piece to follow.

It looks good, but I lack bazel / starlark experience, so I'll remove myself from reviewers.


Adds the solana-lattice-hash crate

Problem
We are working on a new mechanism for hashing accounts. This can (will) be used for the accounts delta hash, hash of all accounts, and epoch accounts hash. The fundamental primitive is the lattice-based accumulator hash^1. We don't have any code for it yet though.
Summary of Changes
Add a barebones impl of the lattice-based accumulator hash, using blake3, as its own crate.
This will eventually be used by accounts-db/the bank, and is intentionally omitted from this PR.

should emanuelle from jump review this?


PowerToys Run and Arc Browser not compatible

Microsoft PowerToys version
0.81.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Using powertoys run to search the web, Arc browser as default browser. Arc browser not opening or no response
✔ Expected Behavior
open arc browser and searches the web
❌ Actual Behavior
no response
Other Software
Arc Browser 1.6.1 - Microsoft powertoysv0.81.1

/dup #32782


Is the script meant to be very slow?

My first run of it has been going overnight, hasn't produced anything yet.
Console is filled with a bunch of warnings,
[WARNING][2022-03-20 12:42:35 oracle.py:303]	Unknown error message: 'Cannot query field "xxxx" on type "Query".'
[DEBUG][2022-03-20 12:42:35 oracle.py:324]	get_typeref('Cannot query field "xxxx" on type "Query".', 'InputValue') -> None
[WARNING][2022-03-20 12:42:37 oracle.py:303]	Unknown error message: 'Cannot query field "xxxx" on type "Query".'
[DEBUG][2022-03-20 12:42:37 oracle.py:324]	get_typeref('Cannot query field "xxxx" on type "Query".', 'InputValue') -> None
[WARNING][2022-03-20 12:42:39 oracle.py:303]	Unknown error message: 'Cannot query field "xxxx" on type "Query".'
[DEBUG][2022-03-20 12:42:39 oracle.py:324]	get_typeref('Cannot query field "xxxx" on type "Query".', 'InputValue') -> None

Hi @captchaunited! It supposed to run relatively fast. In your case it might be either (1) an edge-case bug related to the GraphQL API you've tested against or (2) too large wordlist.

Hi @nikitastupin how much time would the script take if the wordlist used is the google-10000-english as recommended on the main wiki.
I have tested the GraphQL endpoint manually and there seems to be no rate limits imposed on the amount of queries, so that could be excluded as the probable cause because the script has been running for over 12hrs now...

Hey @Azathothas,
It depends on a particular endpoint. However, 12 hours is definitely more than it should. Perhaps, the endpoint runs on a framework that's not supported by clairvoyance yet or it has some other obstacle.

#40 might resolve performance and some other issues

Also, you may check out the output file while Clairvoyance is still running because it outputs intermediate schema as it builds it (no need to wait until it ends).


feat: [CX-2568]: Adding visual clue to the insights tab after adding an artwork with insights

This PR resolves CX-2568
Description
This PR adds the logic for showing visual clues on the StickyTabPage tabs and the logic for displaying a visual clue on the insights tab inside My Collection when adding artwork with insights to my collection
Pictures and Videos
New Video
https://user-images.githubusercontent.com/20655703/171399598-969d64a2-f0d3-4dfd-8f4e-18a5faf9c185.mp4

PR Checklist

[x] I tested my changes on iOS / Android.
[x] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[x] I hid my changes behind a feature flag.

To the reviewers 👀

[x] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

adding a visual clue to the insights tab when adding artwork with insights - mrsltun

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

I will merge this and create a new build, let me know if you notice something wrong @olerichter00
Also, I think we need to add tests to form

🙌


[AutoDiff] Improve debugging utilities.

Show SIL type when printing AdjointValue.
Add utilities to print PullbackEmitter adjoint value and buffer mappings.
Print generated VJP before printing generated pullback.

This is useful because pullback generation may crash after VJP
generation succeeds.

@swift-ci Please smoke test

@swift-ci Please smoke test


test: Use composer 2.8.0 for TestComposerCreateCmd until composer bug is fixed, for #6586

The Issue

#6586

Composer 2.8.1 introduces new probably unwanted behavior that breaks TestComposerCreateCmd
Upstream issue:

https://github.com/composer/composer/issues/12150

How This PR Solves The Issue

Temporarily use composer 2.8.0 for that test.
Use require instead of assert so we don't run along and use invalid data and get a panic
Minor fiddles with test flow

Manual Testing Instructions
Read the test output.
Misc
A comment about TestComposerCreateCmd: This is so impressive how much test coverage you built into this @stasadev, so great. In retrospect, I imagine all the if statements would be cleaner as a switch statement, or perhaps they could be a test matrix.

Thank you!
In the meantime, I'll be working on the packagist package.


Unix CoreCLR tests CI authoring

Author the CI netci.groovy changes along with a stubbed out runtest.sh -coreclr top200 implementation so that the actual PR containing the
test changes will run. This is needed to ensure we don’t break the
build since netci.groovy changes aren’t picked up in a PR (CI takes
netci.groovy from HEAD).

@dotnet-bot test ci please


First drop of the optimization framework and two passes

This includes:

PassManager & Pass
Class hierarchy for SPIR-V types
Def-use analysis
StripDebugInfoPass & UnifyTypePass

Things definitely need improvement:

Tests, lots of tests
Better error handling and status reporting (panicking all the time is not good)

@dneto0 @dekimir @AWoloszyn @Qining PTAL
Preferably focusing on high-level design issues first; expecting radical changes. :)

Running the compiled spirv-opt on the SPIR-V provided in https://github.com/KhronosGroup/SPIRV-Tools/issues/217, debug insts are successfully removed and %20, %21 is replaced by %12, %13, respectively.

Discussed in the morning.
Right now this PR serves as a complementary piece for reviewing the design. So leave it as-is for now. Code in this PR will be merged in eventually (with reviews and fixes definitely), but that will happen in other PRs with smaller granularity.

Okay, back to work on this PR. Since this is so gigantic, I'll split it into multiple CLs:

Classes for in-memory representation (IMR).
Builder for creating IMR.
Passes & Pass manager.
StripDebugInfoPass
Type hierarchy & UnifyTypePass


can't build with webpack

Hi, I'm trying to use this library with webpack, but it doesn't work.
"diff": "^7.0.0",
"ts-loader": "^9.5.1",
"typescript": "^5.6.2",
"webpack": "^5.94.0",
"webpack-cli": "^5.1.4"

webpack.config.js
module.exports = {
    entry: './src/index.ts',
    mode: 'production',
    output: {
        filename: 'bundle.js',
        path: path.resolve(__dirname, 'dist'),
        // library: 'Diff',  // doesn't work either
    },
    module: {
        rules: [
            {
                test: /\.ts$/,
                use: 'ts-loader',
                exclude: /node_modules/,
            },
        ],
    },
    resolve: {
        extensions: ['.ts'],
    },
};

error:
assets by status 4.67 KiB [cached] 1 asset
./src/index.ts 249 bytes [built] [code generated]
./src/diff/merge.ts 6.57 KiB [built] [code generated]
./node_modules/diff/lib/index.js 8.27 KiB [built] [code generated]

ERROR in ./node_modules/diff/lib/index.js 142:31-53
Module not found: Error: Can't resolve './diff/base' in '/Volumes/Work/bulb-diff-merge2/node_modules/diff/lib'
 @ ./src/diff/merge.ts 4:15-30
 @ ./src/index.ts 4:16-39

ERROR in ./node_modules/diff/lib/index.js 147:13-40
Module not found: Error: Can't resolve './diff/character' in '/Volumes/Work/bulb-diff-merge2/node_modules/diff/lib'
 @ ./src/diff/merge.ts 4:15-30
 @ ./src/index.ts 4:16-39

You don't include a full repro case here (no tsconfig.json and the actual import of jsdiff isn't shown), but by filling in the gaps I was able to reproduce approximately the same error and I am pretty sure the fix is to change this, in your webpack.config.js...
    resolve: {
        extensions: ['.ts'],
    },

to this:
    resolve: {
        extensions: ['.ts', '.js'],
    },

You're getting this error because diff/lib/index.js is a .js file and you've get webpack configured to only let you import .ts files.
I don't think jsdiff is doing anything wrong here, so I'll close this as invalid. If I'm mistaken and there's something jsdiff ought to be doing differently to avoid this error, please explain to me!

You don't include a full repro case here (no tsconfig.json and the actual import of jsdiff isn't shown), but by filling in the gaps I was able to reproduce approximately the same error and I am pretty sure the fix is to change this, in your webpack.config.js...
    resolve: {
        extensions: ['.ts'],
    },

to this:
    resolve: {
        extensions: ['.ts', '.js'],
    },

You're getting this error because diff/lib/index.js is a .js file and you've get webpack configured to only let you import .ts files.
I don't think jsdiff is doing anything wrong here, so I'll close this as invalid. If I'm mistaken and there's something jsdiff ought to be doing differently to avoid this error, please explain to me!

Yes, you're right, it's my mistake. It works now by adding '.js'.
Thank you.


Error on the createSession example from postgres docs

Hi. I'm following the postgreSQL session doc.
On the createSession function I get an error with the example implementation on this call:
const sessionId = encodeHexLowerCase(sha256(new TextEncoder().encode(token)));
// TS2345: Argument of type Promise<ArrayBuffer> is not assignable to parameter of type Uint8Array
// Type Promise<ArrayBuffer> is missing the following properties from type Uint8Array...

I had to change the function to be:
const buffer = await sha256(new TextEncoder().encode(token));
const u8Arr = new Uint8Array(buffer);
const sessionId = encodeHexLowerCase(new TextEncoder().encode(token));

I think it's an error on the docs, right? Or Am I missing something?

Where are you importing sha256() from? sha256() in @oslojs/crypto is synchronous


Update sbt-mima-plugin to 1.1.2

About this PR
📦 Updates com.typesafe:sbt-mima-plugin from 1.1.1 to 1.1.2
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe", artifactId = "sbt-mima-plugin" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.typesafe", artifactId = "sbt-mima-plugin" }
}]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report
Patch coverage has no change and project coverage change: +0.06 :tada:

Comparison is base (19f30c2) 89.43% compared to head (7760cc1) 89.50%.

:exclamation: Current head 7760cc1 differs from pull request most recent head 1ddc414. Consider uploading reports for the commit 1ddc414 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1112      +/-   ##
===========================================
+ Coverage    89.43%   89.50%   +0.06%
===========================================
  Files          124      124
  Lines        10028    10028
  Branches       734      734
===========================================
+ Hits          8969     8976       +7
+ Misses        1059     1052       -7

see 9 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Snyk] Security upgrade golang from 1.11 to 1.21.12

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 4 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

awstesting/sandbox/Dockerfile.test.go1.11

We recommend upgrading to golang:1.21.12, as this image has only 89 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Resource Exhaustion SNYK-DEBIAN10-NGHTTP2-5953390
  561

Link Following SNYK-DEBIAN10-GIT-1083853
  433

Link Following SNYK-DEBIAN10-GIT-1083853
  433

Improper Validation of Integrity Check Value SNYK-DEBIAN10-OPENSSH-6130526
  394

CVE-2023-26604 SNYK-DEBIAN10-SYSTEMD-3339153
  375

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Resource Exhaustion

🎉 Snyk hasn't found any issues so far.
✅ code/snyk check is completed. No issues were found. (View Details)


Dont't run in Unity2019.1 with HDRP

I can't use SpoutKlak Unity2019.1 with HDRP mode.
It works with Unity2018.3 + HDRP.
Please update Spout Klak.
I love it very much.

I confirmed it works without any problem on Unity 2019.1.0f2 with HDRP 5.13.0.
Please re-check if there is any issue in other parts of the project. If it still reproduces, please provide a minimum project that can reproduce the issue.

Now I'm closing this issue. Please feel free to reopen if it still reproduces.


Modbus RTU connections may not be observing the inter-frame gap of 3.5 character times when performing requests

Fixed in commit df7d8cf0652628f30f33ca4b9f6339f8235bb2ba.


Copy gr8 processed gradle-wrapper.jar back to wrapper/build/libs

Fixes https://github.com/gradle/gradle/issues/26658
Before introducing gr8, wrapper jar is generated as build/libs/gradle-wrapper.jar and used in promotion build. After introducing gr8, wrapper jar is generated as build/libs/gradle-wrapper-executable.jar and processed by gr8, then the processed gradle-wrapper.jar need to be copied back to build/libs for promotion build

I've triggered a build for you. Click here to see all failures if there's any.


Fix Composer Update

If I want update composer I get following problem:
Problem 1
    - composer/satis 1.x-dev requires php ^7.0 -> your PHP version (7.0.31-1+ubuntu16.04.1+deb.sury.org+1) overridden by "config.platform.php" version (5.6) does not satisfy that requirement.
    - composer/satis 1.x-dev requires php ^7.0 -> your PHP version (7.0.31-1+ubuntu16.04.1+deb.sury.org+1) overridden by "config.platform.php" version (5.6) does not satisfy that requirement.
    - Installation request for composer/satis 1.x-dev -> satisfiable by composer/satis[1.x-dev].

Thanks for this!


[To rel/0.12][IOTDB-1556] Abort auto create device while setStorageGroup exception

Description
Case
After operating the following operations:

set storage group to root.a.sg
insert into root.a.d(time, s) values(1, 1)

Timeseries root.a.d.s won't be created for no storage group, but device node root.a.d will be created without storage group.
If a device node without any timeseries is created without storageGroup, then, in current code, it's unable to set any storage group on its ancestors and create any timeseries under it. Only set storage group to its descendants could be operated successfully.
Since we don't have device node deletion oeperation, most of users will be confused and don't know how to handle this device node.
Cause
In MManager.getDeviceNodeWithAutoCreate, if there's a storageGroupAlreadyExistException, the device node will be created with out storage group check. If the exception is caused by child node existence, then there will be no storage group above the device.
In current setStorageGroup logic, it is assumed that any timeseries or device is working under some storage group, which means a path existence implies a storage group existence.
Solution
Modify StorageGroupAlreadyExistException with boolean hasChild added, thus mmanager could know whether the exception is caused by child.

This PR has:

[ ] been self-reviewed.

[ ] concurrent read
[ ] concurrent write
[ ] concurrent read and write

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods.
[ ] added or updated version, license, or notice information
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious
for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold
for code coverage.
[ ] added integration tests.
[ ] been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

Plz do not forget to cherry-pick to master branch.


Cannot edit Domains with nullable associations

Given domain resources like the following where the association is nullable:
Publisher {
    String name
}

class Book {
    String title
    Publisher publisher
    static constraints = {
        publisher nullable: true
    }
}

The result is that for a Book without a Publisher, the show screen displays "test.Publisher : (unsaved)".  When you attempt to update the book and the update button is clicked a null pointer exception is encountered:
ERROR org.grails.web.errors.GrailsExceptionResolver - NullPointerException occurred when processing request: [PUT] /book/1
Stacktrace follows:
java.lang.reflect.InvocationTargetException: null
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
        at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.NullPointerException: null
        at test.BookController.$tt__update(BookController.groovy:54)
        at grails.transaction.GrailsTransactionTemplate$2.doInTransaction(GrailsTransactionTemplate.groovy:96)
        at grails.transaction.GrailsTransactionTemplate.execute(GrailsTransactionTemplate.groovy:93)
        at grails.transaction.GrailsTransactionTemplate$2.doInTransaction(GrailsTransactionTemplate.groovy:96)
        at grails.transaction.GrailsTransactionTemplate.execute(GrailsTransactionTemplate.groovy:93)
        ... 3 common frames omitted

I think the issue is that the JSON submitted for the update ends up including a empty publisher when none is selected:
{
    id: 1,
    title: "Along Came a Spider",
    publisher: {}
}

Sample project: https://github.com/scrain/angular-scaffolding-nullable-association-issue

@scrain Thanks for reporting this. It shouldn't show "test.Publisher : (unsaved)" if the property isn't passed down from the server.

Fixed by https://github.com/grails-plugins/grails-angular-scaffolding/commit/9818dcbf0cf0b117b61440e0f8ef67df283e9819


Some npm module paths must be absolutely specified

E.g. https://www.npmjs.com/package/crc-32
Requiring the module using:
var CRC32 = require("./node_modules/crc-32/crc32.js");

works, but using:
var CRC32 = require("crc-32");

doesn't:
Error: Cannot find module 'crc-32'

This is #243. We had this a couple of times already.


@conda-forge-admin, please re-render

@conda-forge-admin, please re-render.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/nsis-feedstock#17.


[enhancement] Add draft mode to catalog creation / editing workflow

Descriptive summary
Catalog creation should be carried in draft mode which is only accessible by the manager role and hidden to the normal rdmo user.
Affects
Users, Managers and Admins
Rationale
In case of catalog creation/editing work a second/multiple instances of RDMO are required in addition to a productive one.
Expected behavior
Managers have the option to mark catalogs draft/published which hides/shows them from/to the user view.
More atomically draft/published mode is available on the question/question set level.
Related work
issue 46

The available flag was added in e9e95ab10f45daae99879e3af8064ed04183a173. Non available Catalogs, Views or Tasks can only used who have permissions to access the management interface.


Icon not showing when defining in functional component

Current behaviour

I am trying to use the tabBarIcon property in my functional component to set the icon for that component but nothing shows up. Actually, all properties don't work when setting like this.
Expected behaviour

Expect to show the icon in the tab bar.
Code sample

import React from 'react'
import Screen from '../../components/Screen'
import { Text, Image } from 'react-native'
import { NavigationBottomTabOptions } from 'react-navigation-tabs'

const Feed: React.FunctionComponent<{

}> & NavigationBottomTabOptions = ({ }) => {
    return (
        <Screen backgroundColor="#000000">
            <Image source={require('../../../assets/feed-icon.png')} />
        </Screen>
    )
}

Feed.tabBarIcon = () => (
    <Image source={require('../../../assets/feed-icon.png')} />
)

export default Feed

Screenshots (if applicable)

https://imgur.com/gallery/jgZZxNK
What have you tried

Pretty much the example.
Your Environment

software
version

ios or android
ios

react-native
expo v35

react-navigation-tabs
2.5.5

node
12.7

npm or yarn
npm v6.1.0

it's inside navigationOptions

Can you at least tell me which interface is this to give me the right options to set? Instead of NavigationBottomTabOptions  what should I use?

Well, it seems that navigationOptions has this type:
 NavigationScreenConfig<
    { [key: string]: any },
    NavigationScreenProp<NavigationRoute>
>

This allows me to set:
Feed.navigationOptions = {
    tabBarIcon: () => (
        <Image source={require('../../../assets/feed-icon.png')} />
    )
}

And it works, but it's not properly typed.

https://reactnavigation.org/docs/en/typescript.html

Thanks man @brentvatne
Right solution:
import React from 'react'
import Screen from '../../components/Screen'
import { Image } from 'react-native'
import { NavigationBottomTabScreenComponent } from 'react-navigation-tabs'

const Feed: NavigationBottomTabScreenComponent = ({ }) => {
    return (
        <Screen backgroundColor="#000000">
            <Image source={require('../../../assets/feed-icon.png')} />
        </Screen>
    )
}

Feed.navigationOptions = {
    tabBarIcon: () => (
        <Image source={require('../../../assets/feed-icon.png')} />
    )
}

export default Feed


Adding Kafka support for eventing and event handling

Two changes:

Minor correction in the event handling logic when using MySQL persistence.
Adding Kafka support in the contribs module.

kafka configuration for provider and consumer can be added with the below prefixes for general, producer and consumer configuration respectively.
'kafka.' , 'kafka.provider.' and 'kafka.consumer.'
request for review
related enhancement discussed in #414

Hello @cyzhao @v1r3n @pctreddy
Could you please check the approach and suggest.

@preeth-gopalakrishnan please add a test

Working on the comments received.

Hello @cyzhao @pctreddy,
Have tried to include all the suggestions.
Please check and suggest.

Integration tests are missing.
Closing this pull request due to conflicts and inactivity. Please open a new PR with resolutions.


fix(admin-gui): invalidate cache for kconfig.js when reload

Summary
The cache for kconfig.js won't be refreshed across the reload,
which may cause requests to kconfig.js still respond with the
old content when the configuration is changed. Force invalidating the
cache during the worker init procedure could fix it.
Checklist

[x] The Pull Request has tests
[ ] ~There's an entry in the CHANGELOG~
[ ] ~There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE~

Full changelog

[Implement ...]

Issue reference

Fix #[issue number]

@nekolab please cherry-pick into kong-ee


Slides with autoplay and loop props have different behaviour while sliding back to first slide ONLY in --prod build (Android)

Ionic version:
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...
[x] bug report
[ ] feature request
Current behavior:
When the slider animates from the last to the first slide, ONLY in --prod build, it slides reversally from right lo left, while in NON --prod build it simply continues sliding from left to right (wanted behaviour)
Expected behavior:
Slides animations should have the same behaviour in --prod and dev builds.
Steps to reproduce:
ionic start myApp blank
ionic cordova platform add android
add following code to the home.html
<ion-slides autoplay="4000" loop="true">
   <ion-slide style="background-image: url(http://lorempixel.com/output/fashion-q-c-640-480-4.jpg)">
   </ion-slide>
   <ion-slide style="background-image: url(http://lorempixel.com/output/technics-q-c-640-480-1.jpg)">
   </ion-slide>
</ion-slides>

ionic cordova run android
Slides will roll always from right to left
Now:
ionic cordova run android --prod
While sliding from the last to the first slide, it will roll from left to right giving even an ugly visual effect
Ionic info::
cli packages: (C:\Users\Kevin\AppData\Roaming\npm\node_modules)
    @ionic/cli-utils  : 1.13.0
    ionic (Ionic CLI) : 3.13.0

global packages:
    cordova (Cordova CLI) : 7.0.1

local packages:
    @ionic/app-scripts : 3.0.0
    Cordova Platforms  : android 6.2.3
    Ionic Framework    : ionic-angular 3.7.1

System:
    Android SDK Tools : 25.2.5
    Node              : v7.9.0
    npm               : 4.6.1
    OS                : Windows 10

Misc:
    backend : legacy

After a few hours of research I haven't found anything, should I open this issue also in ionic-app-scripts Repo?

Hello! Thank you for opening an issue with us!
I was able to duplicate the issue.
Thank you for using Ionic

Really thanks, I haven't tested it on iOS yet but I will do in the next days.

Tried it on iOS and it exhibits the same issue. Clearly something goes wonky when building with --prod

Any news?

@kensodemann do you have news?

All development effort at this time is focused on getting Ionic v4 released. This will need to be retested once v4 is released.

@kensodemann :+1:


Material dialog id changes everytime when a dialog is open

Whenever i open angular material dialog the id in tag change always when i open it. Everytime the id changes its hard to get the reference of dialog.
See the 2 images below.

You can use [panelClass] or addPanelClass() as specified in https://material.angular.io/components/dialog/api in order to get predictable selectors for the dialog. Then within the dialog, you can target the title using the .mat-dialog-title class.


Allow anonymous event listeners

I added support for anonymous functions as event listeners. I think its pretty useful for those small event callbacks you use to have for e.g. RegisterResource-Events.
Downsides:

another format to support
currently all event listeners could easily be written to database or could be stored otherwise

Team Neversleep \o/
But I am not quite sure if it's good practice to use callbacks in a subscriber. If you want to do it quick and dirty, why not add the callback directly to the EventManager?
Also this will break compatibility with the Symfony EventDispatcher where our addSubscriber() method originated from. See: https://github.com/symfony/EventDispatcher/blob/master/EventDispatcher.php#L117
Together with the downsides you already mentioned and the additional complexity I am not convinced.

It's not about quick and dirty - its about fancy callbacks to deal with nasty events :)
But already convinced, not sure if we should take all the disadvantages for some convenience.


cmake support in the binary distribution

Currently it doesn't seem possible to consume or-tools in a cmake project when or-tools is installed from the C++ binary distribution. Or-tools's use in C++ projects would be much easier if cmake configuration was included in the binary distribution.

Not the expert, but this is really not the canonical way to use it, as we do provide (experimental) support for cmake in or-tools source.

Sure, but many users prefer the binaries (easier to install), and I see no reason not to use cmake with them. In theory it should be enough to include the corresponding .cmake config files in the binary distribution to make find_package(ortools CONFIG) work.

I wrote a Findortools.cmake file which may help you. I am NOT an expert so I am probably doing some things wrong.
I have tested it on Ubuntu 18.04, Windows 10, Windows subsystem for Linux so it is far from an exhaustive test. All version used CMake 3.12 or above.
It works like any FindXXX.cmake files.

Add the path of the file in CMAKE_MODULE_PATH .
In your CMakeLists.txt, use  find_package(ortools) which defines the target ortools::ortools
Add -Dortools_ROOT=path/to/ortools/ when invoking CMake

if(WIN32)
    if(NOT TARGET ortools::ortools)
        find_library(ORTOOLS_LIBRARIES NAME ortools PATH_SUFFIXES lib )
        find_path(ORTOOLS_INCLUDE_DIRS NAME zlib.h PATH_SUFFIXES include)
        set(ORTOOLS_DEFINITIONS /DNOMINMAX -DUSE_CBC -DUSE_CLP -DUSE_BOP -DUSE_GLOP)

        add_library(ortools::ortools STATIC IMPORTED GLOBAL)
        set_target_properties(ortools::ortools PROPERTIES IMPORTED_LOCATION ${ORTOOLS_LIBRARIES} )
        target_include_directories(ortools::ortools INTERFACE ${ORTOOLS_INCLUDE_DIRS})
        target_link_libraries(ortools::ortools INTERFACE ${ORTOOLS_LIBRARIES})
        target_compile_options(ortools::ortools INTERFACE ${ORTOOLS_DEFINITIONS})

        include(FindPackageHandleStandardArgs)
        # handle the QUIETLY and REQUIRED arguments and set ortools to TRUE
        # if all listed variables are TRUE
        find_package_handle_standard_args(ortools_FOUND  REQUIRED_VARS
                                          ORTOOLS_LIBRARIES ORTOOLS_INCLUDE_DIRS)

    endif()
elseif(UNIX)
    if(NOT TARGET ortools::ortools)
        # Include directories
        find_path(ORTOOLS_INCLUDE_DIRS NAME ortools PATH_SUFFIXES include/)
        message(STATUS "ortools found here : ${ORTOOLS_INCLUDE_DIRS}")

        ## Libraries
        set(LIB_TO_FIND
            CbcSolver #CBC_LNK
            Cbc
            OsiCbc
            Cgl
            ClpSolver
            Clp
            OsiClp
            Osi
            CoinUtils
            absl_bad_any_cast_impl #ABSL_LNK
            absl_bad_optional_access
            absl_bad_variant_access
            absl_base
            absl_city
            absl_civil_time
            absl_debugging_internal
            absl_demangle_internal
            absl_dynamic_annotations
            absl_examine_stack
            absl_failure_signal_handler
            absl_graphcycles_internal
            absl_hash
            absl_hashtablez_sampler
            absl_int128
            absl_leak_check
            absl_malloc_internal
            absl_optional
            absl_raw_hash_set
            absl_spinlock_wait
            absl_stacktrace
            absl_str_format_internal
            absl_strings
            absl_strings_internal
            absl_symbolize
            absl_synchronization
            absl_throw_delegate
            absl_time
            absl_time_zone
            protobuf #protobuf
            glog #glog
            gflags #gflags
            ortools #ortools
        )

        foreach(X ${LIB_TO_FIND})
            find_library(LIB_${X} NAME ${X} PATH_SUFFIXES lib/)
            message(STATUS "${X} lib found here : ${LIB_${X}}")
            set(ORTOOLS_LIBRARIES ${ORTOOLS_LIBRARIES} ${LIB_${X}})
        endforeach()

        # Definitions
        set(ORTOOLS_DEFINITIONS -DUSE_CBC -DUSE_CLP -DUSE_BOP -DUSE_GLOP)

        add_library(ortools INTERFACE)
        add_library(ortools::ortools ALIAS ortools)
        target_include_directories(ortools INTERFACE ${ORTOOLS_INCLUDE_DIRS})
        target_link_libraries(ortools INTERFACE ${ORTOOLS_LIBRARIES})
        target_compile_options(ortools INTERFACE ${ORTOOLS_DEFINITIONS})

        include(FindPackageHandleStandardArgs)
        # handle the QUIETLY and REQUIRED arguments and set ortools to TRUE
        # if all listed variables are TRUE
        find_package_handle_standard_args(ortools_FOUND
            REQUIRED_VARS ORTOOLS_LIBRARIES ORTOOLS_INCLUDE_DIRS)
    endif()
else()
	message(FATAL_ERROR "No other platform supported yet")
endif()

This actually worked pretty well, thanks! I only changed one line:
find_package_handle_standard_args(ortools

to make if(ortools_FOUND) work, and now my cmake builds work with the binary distribution (which saves lots of time, especially for travis builds).
It would be super cool if such a .cmake file was bundled in the binary distribution.

@AxelDelsol Thanks for the awesome Findortools.cmake!  For anyone interested, I have created a minimal viable example using an example provided by Google.

CMake C++ archives is postponed to v9.3

make archive_cpp should create a archive with CMake support

make archive_cpp should create an archive with CMake support

/Documents/myproject$ make ./src/main.cpp
make: Nothing to be done for 'src/main.cpp'.

This is what I get

I get
[cmake]   The package name passed to `find_package_handle_standard_args`
[cmake]   (ortools_FOUND) does not match the name of the calling package (ortools).
[cmake]   This can lead to problems in calling code that expects `find_package`
[cmake]   result variables (e.g., `_FOUND`) to follow a certain pattern.

@chatziko do you mind sharing your modified file?

Here it is:
https://github.com/chatziko/libqif/blob/master/misc/cmake/Findortools.cmake


Suspicious stack trace in an otherwise passing test

This one's either an untrapped error, or just noise, and if it's the latter it should really be trapped and not logged, as the normal reaction to seeing this will be a false positive.
[2015-08-27 13:45:13]  2015/08/27 09:41:35 [500] Internal Server Error: An error occurred: Not a resource: foobar at /var/tmp/venn-core-1.0.0-b
uild011-build/x86-64.rhel.7/5.18/source/core-1.0.0/blib/lib/Venn/PlacementEngine/Request.pm line 254, <$fh> line 1.
[2015-08-27 13:45:13]  2015/08/27 09:41:35 Performing placement: of type hosting with strategy biggest_outlier
[2015-08-27 13:45:13]  2015/08/27 09:41:35 [500] Internal Server Error: An error occurred: Invalid request resource
[2015-08-27 13:45:13]
[2015-08-27 13:45:13]  Trace begun at /var/tmp/venn-core-1.0.0-build011-build/x86-64.rhel.7/5.18/source/core-1.0.0/blib/lib/Venn/PlacementEngin
e/Strategy.pm line 237
[2015-08-27 13:45:13]  Venn::PlacementEngine::Strategy::validate_request_resources('Venn::PlacementEngine::Strategy::BiggestOutlier=HASH(0xc6f3
f70)', 'Venn::PlacementEngine::Request=HASH(0xc45f630)') called at /var/tmp/venn-core-1.0.0-build011-build/x86-64.rhel.7/5.18/source/core-1.0.0
/blib/lib/Venn/PlacementEngine.pm line 201
...
...
[2015-08-27 13:45:13]  Catalyst::Test::_local_request('Venn', 'HTTP::Request=HASH(0xc5778e0)') called at /efs/dist/perl5/Catalyst-Runtime/5.901
00/common/lib/perl5/Catalyst/Test.pm line 32
[2015-08-27 13:45:13]  Catalyst::Test::ANON('HTTP::Request=HASH(0xc5778e0)') called at t/catalyst/placement.t line 475
[2015-08-27 13:45:13]  t/catalyst/placement.t ............................................................................ ok
The full trace is several pages long, so if anyone wants it I'll attach the full logfile.

Yes, it's intentional, bug I agree it's ugly.
I'll fix it.


DownMedia for Raycast

Hi. Is it possible to do Script Commands DownMedia for the Raycast application?
Raycast is like Alfred only more modern and interesting.
It would be cool if the DownMedia process was also available for Raycast.
Here is the documentation and examples of Script Commands for Raycast -
https://github.com/raycast/script-commands
https://github.com/raycast/script-commands/tree/master/commands

Raycast is like Alfred only more modern and interesting.

It’s newer, not “more modern”. “More interesting” is highly debatable. What it has is hype backed up by money. From what I’ve seen, Raycast is less powerful and expressive than Alfred; I don’t see anything that would be worth the switch, especially to a VC-funded startup which I expect will either be sold with abandon or go bust.
I make and distribute these Workflows for free, at the expense of my time. I have no desire to rewrite for free such a complex Workflow to a less capable alternative I don’t use.


Can't increase sending BW after limiting of it.

Description
In our app we allow to limit publishing BW during session. We use updateConfiguration method with parameter maxVideoBW for it. But after limiting it to lower bandwidth, it can't be restored to higher bandwidth. Instead of it audio bandwidth is going Up.
Steps to reproduce the issue:

Simulcast is 2 layers, codecs: VP8, OPUS
Connect to room by one publisher with BW = 512kb
Limit BW to 128: localstream.updateConfiguration({ maxVideoBW: 128 });
Check webrtc://chrome-internals - bitrade droped to 128
Limit BW to 512: localstream.updateConfiguration({ maxVideoBW: 512 });
Check webrtc://chrome-internals: video BW is still 128, audio BW increased. (see attach)

Note: reproduces only with simulcast.

I have reproduced it with pure licode example in docker:

Run docker: MIN_PORT=30000; MAX_PORT=30150; sudo docker run --name licode -p 3000:3000 -p $MIN_PORT-$MAX_PORT:$MIN_PORT-$MAX_PORT/udp -p 3001:3001 -p 3004:3004 -p 8080:8080 -e "MIN_PORT=$MIN_PORT" -e "MAX_PORT=$MAX_PORT" -e "PUBLIC_IP=<yourIP>" lynckia/licode:develop
Configure nginx to allow ssl connection: http://lynckia.com/licode/nginx-dep.html
Open page in Chrome 79: https://your-domain.com/?simulcast=true
Pay attention on Audio bitsSentPerSecond and Video bitsSentPerSecond in chrome://webrtc-internals. They both ~150 kbps.

Expected result: Video ~300kbps, Audio ~24kbps
Note: it is OK if do not use simulcast.

I think this is a reason:
https://bugs.chromium.org/p/chromium/issues/detail?id=1002875
Can it be workaround-ed in licode ?

https://bugs.chromium.org/p/chromium/issues/detail?id=1002875#c31

Looks like it has been fixed in Chrome. I tested it with Chrome v80 dev. And it does not reproduce there.

@vpoddubchak @kekkokk is there something we can do before Chrome v80 to avoid or limit this issue in Licode?

I've been reading the threads and it looks like @kekkokk implemented a workaround to fix the issue in the meantime, am I right? This is the code you had:
  if ('RTCRtpSender' in window && 'setParameters' in window.RTCRtpSender.prototype) {
    const sender = pc.getSenders()[0];
    const parameters = sender.getParameters();
    if (!parameters.encodings) {
      parameters.encodings = [{}];
    }
    parameters.encodings[0].maxBitrate = 32000;
    sender.setParameters(parameters);
  }


Title View issue

I have set you title view function but it is raising the issue & auto increasing the width of segment on every scroll.
func getSegmentTabWithImage(_ imageName: String) -> UIView {
	let view = UIImageView()
	view.frame.size.width = 100
	view.image = UIImage(named: imageName)
	view.contentMode = .scaleAspectFit
	view.backgroundColor = .white
	return view
}

Looks like a bug, I am also facing the same issue.

is this soluted?


Create a generic dropdown component that support binding to a list of custom classes

GenericDropdown working in conjunction with DropdownComplex. Configurable parameters contain details about the model, display name, generic dropdown component to dynamically load, and how to obtain option items to bind to the dropdown.


✨ Touch bar implementation

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
The code editor to take full advantage of macbook's features, being made for mac in mind. Having the macbook pro touch bars have a "run" and "debug" button would be amazing. Maybe it could even show open tabs of the editor.

Describe alternatives you've considered
The touchbar COULD solely be black and not in use, but I feel that would be a waste.
Additional context
There could also be customization for the user, allowing the user to choose what they want on their touchbar based on pre-defined presets.

Backlogging for now, we need better commands architecture in order to get started on this.

Should this even be done as Apple has effectively killed the Touchbar?

@UbiquitousBear You are right. Probably not at this point. Closing this issue.


Graphical glitches on NVIDIA 1050ti

i keep getting very rarely some black lines appearing on the screen, that last just for a few milliseconds (sometimes it happens to the whole screen).
i followed the nvidia guide on the wiki, tried enabling performance mode but did not solve the problem.
with sway i don't have this problem, it works fine.
some gpu info:
01:00.0 VGA compatible controller: NVIDIA Corporation GP107 [GeForce GTX 1050 Ti] (rev a1) (prog-if 00 [VGA controller])
	Subsystem: ZOTAC International (MCO) Ltd. Device a454
	Flags: bus master, fast devsel, latency 0, IRQ 125
	Memory at f6000000 (32-bit, non-prefetchable) [size=16M]
	Memory at e0000000 (64-bit, prefetchable) [size=256M]
	Memory at f0000000 (64-bit, prefetchable) [size=32M]
	I/O ports at e000 [size=128]
	Expansion ROM at 000c0000 [virtual] [disabled] [size=128K]
	Capabilities: <access denied>
	Kernel driver in use: nvidia
	Kernel modules: nouveau, nvidia_drm, nvidia

every nvidia graphical glitch is out of my reach. Try the nvidia patches to wlroots (provided in the nixos flake, you can also join the discord server and ask there)