issue #11244 - demo should install a default secret for kiali

This is in response to @linsun and her desire to get the kiali secret created out of box with the demo profile for 1.1 builds. See https://github.com/istio/istio/issues/11244#issuecomment-462934826
This feature is already merged in the master branch - this PR just cherry picks to the 1.1 branch.
(cherry picked from commit 1ad4e29576da6c722dcf19fc5df703beede92a4d)

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jmazzitelli
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
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:

install/OWNERS

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

@jmazzitelli: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0a569a54ee76c6e77460ce30ed956ac66fa62a7e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0a569a54ee76c6e77460ce30ed956ac66fa62a7e
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
0a569a54ee76c6e77460ce30ed956ac66fa62a7e
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0a569a54ee76c6e77460ce30ed956ac66fa62a7e
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
0a569a54ee76c6e77460ce30ed956ac66fa62a7e
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
0a569a54ee76c6e77460ce30ed956ac66fa62a7e
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@linsun: changing LGTM is restricted to assignees, and assigning you to the PR failed.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jmazzitelli, linsun
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:

install/OWNERS [linsun]

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

/test istio-unit-tests

@wenchenglu can we get this in 1.1?  This is only a change for demo profile/yaml to ensure we can get a good UX when all istio components can reach running.


Update to Java 9 [DO NOT MERGE]

Context
We need to get off Java 8 before it goes out of support. Unfortunately we are blocked on Java 10+ by upstream issues e.g. https://github.com/dropwizard/dropwizard/issues/2450 so this PR moves to Java 9 which is out of support (hence do not merge) but gets us closer to the eventual goal of Java 11 LTS
Changes proposed in this pull request
This involved updating many dependencies to versions that support Java 9, changes include:

Fix deprecation warnings
Update to Dropwizard 1.3.6
Update to Gradle 4.10
Add dependencies on Java modules that were removed from core as part of modularisation process
fork compass-gradle-plugin to publish version which supports Gradle 4.10 (see https://github.com/robfletcher/gradle-compass/issues/77)
Update Travis CI and CloudFoundry java version

Guidance to review
Tests should pass on Travis CI, app should run locally if you are using Java 9 (I suggest using jenv to allow for multiple system JDK versions)

closing as we don't intend to merge this. Will review again when upstream issue resolution may allow us to get to Java 11.


Retirement age over the time path

I believe there is a bug in the time path solution related to the retirement age for cohorts with incomplete lifetimes.  In particular, the parameters retire is an integer representing the year of life beyond which agents are eligible for retirement benefits.  Over the these incomplete lifetimes, we have retire fixed at the same integer, even though the length of the lifetime varies.
This comes into play in the tax.total_taxes() function.  Here we have:
retireTPI = (retire - S)
...
T_P[retireTPI:] -= theta[j] * w[retireTPI:]

Because retire < S this never raises an indexing issue, even though sometimes the absolute value of retire-S exceeds the length-s vector that is w (e.g. you might have w and a vector of length 20 and retire-S = -40 so that you have w[-40:] which is still an array of length 20.
I think what we need is retireTPI = (s-retire), where s is the number of periods of life remaining in the incomplete lifetime.
@rickecon can you confirm that this is an issue?

The retireTPI = (retire - S), actually gives the behavior that is intended (even if it seems weird to be indexing beyond the length of the vector.


Remove Text from PostMessageParameters

It is already a required argument in PostMessage(), and isn't used anywhere.
Also, initalize ThreadTimestamp with the default value

@mavidser do you have time to rebase this? and I'll get it merged.


[Q]: How can I execute a task as a different user on a specified host or farm?

Hello, @rs2
Does this host running digdag?. If so how about using sudo?
Or do you want to execute a command via remote shell like (ssh)?
If so how about ssh user@host command?
I'm developing ssh> operator.
It will be an operator for executing a command via ssh easily. (And next I want to develop ssh executor.)

Close the old issue. Please reopen or file new one if you need.


Implement game creation

Description
We need to be able to create a game.
Create a create_game system that will initiate the game.
#[system]
mod create_game {
    use dojo::Context;
    fn execute(ctx: Context, player1: ContractAddress, player2: ContractAddress) {}
}

i can do this one


distro configs

@nathanchance suggests fetching distros' configs and building those

Fedora and OpenSUSE would probably be a good place to start, given that their configs are available via URLs:
https://src.fedoraproject.org/rpms/kernel/tree/rawhide
https://github.com/openSUSE/kernel-source/tree/master/config

Blocked on https://gitlab.com/Linaro/tuxsuite/-/issues/124.


Add rutorrent discovery?

Hey!
I've been thinking about how some people might configure rutorrent using a RPI or similar devices on their home and access it through port forward or ssh tunneling services like remote.it and I was wondering maybe we should add a discovery helper. Also, since it seems like rTorrent & rutorrent is very much user-configured, we could add some instructions to set a permanent announcement through Avahi xml files.
We would need to agree on:

a service type
a sample xml to announce the service through Avahi
maybe some TXT records that might help the app discover the app correctly (maybe add TXT records to set ~the protocol you want to access~ if the server uses http or https, a public URL where this server might be accessed, or anything else, really).

I could write a sample PR that adds this discovery through the setup interface, but comments are welcome.

I am working on some light additions in #77, so you can check for it.

The project has been updated extensively in the last 2 years (check wiki for more info). The default branch is now the "main" branch. Let me know if you are still interested in working on this in the latest version.

Hi.
I believe I did this because I was trying to enter into the GSoC but I ended up too busy.
Anyways, if this is a feature that the leadership of this project would like to develop, as I stated in my first post, we should agree on

Service Type eg: _rutorrent._tcp
TXT records for any extra functionality.


[stable/jenkins] Agent/Slave disconnects randomly during build

Every so often when a job is invoked, a Jenkins slave will be spun up, but it won't complete the job properly, even sometimes erroring before the job actually even starts.
Agent went offline during the build
Build step 'Execute shell' marked build as failure
FATAL: Remote call on JNLP4-connect connection from <IP_ADDRESS>/<IP_ADDRESS>:47648 failed. The channel is closing down or has closed down
java.nio.channels.ClosedChannelException
	at org.jenkinsci.remoting.protocol.NetworkLayer.onRecvClosed(NetworkLayer.java:154)
	at org.jenkinsci.remoting.protocol.impl.NIONetworkLayer.ready(NIONetworkLayer.java:142)
	at org.jenkinsci.remoting.protocol.IOHub$OnReady.run(IOHub.java:721)
	at jenkins.util.ContextResettingExecutorService$1.run(ContextResettingExecutorService.java:28)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused: hudson.remoting.ChannelClosedException: Remote call on JNLP4-connect connection from <IP_ADDRESS>/<IP_ADDRESS>:47648 failed. The channel is closing down or has closed down
	at hudson.remoting.Channel.call(Channel.java:901)
	at hudson.Launcher$RemoteLauncher.kill(Launcher.java:1079)
	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:510)
	at hudson.model.Run.execute(Run.java:1724)
	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
	at hudson.model.ResourceController.execute(ResourceController.java:97)
	at hudson.model.Executor.run(Executor.java:421)
Finished: FAILURE

I am not actually doing anything in the job (I'm still building it but have parameter verification steps in there)
Using Spots vs On Demand instances makes no difference.
Increasing timeout on the Jenkins master ELB makes no difference.

Example:
node {
    docker.image('blenderfox/xenial-base').inside('-u root') {
        stage('Configure Environment') {
                sh '''
                        # Any stuff you need to do to the pod before the job runs, put here
                        # for example, any repos you want, CLI tools, etc.
                        '''
        }
    }
}

This works on a normal Jenkins installation on EC2 (non-Kubernetes)
When copy-pasted into Jenkins installed via helm, it randomly times out during startup.
However, this does work (using the Kubernetes plugin):
node {
    kubernetes.pod('test-pod').withImage('blenderfox/xenial-base').inside {
        stage('Configure Environment') {
                sh '''
                        # Any stuff you need to do to the pod before the job runs, put here
                        # for example, any repos you want, CLI tools, etc.
                        '''
        }
    }
}

Guess no-one else has encountered this :(

I encounter this quite often. When running the job as a pipeline the pipeline ends up hung forever cause the agent will never return. Very frustrating.

And as yet, no indication as to why/how to fix it :(

I am waiting for a fix too.

Me too..

Clearly I did not pay attention to what repo this issue was created against. This is a helm repo but the issue is a Jenkins issue as far as I can tell, thus I recommend we move the conversation to https://issues.jenkins-ci.org/browse/JENKINS-48895

I don't think it is Jenkins, as the same job runs totally fine on a Jenkins installation that is not in Kubernetes. And neither Jenkins is on v2.9 as mentioned in that referenced ticket.

Removing stale as this issue is still outstanding with no evidence of being fixed
/remove-lifecycle stale

https://github.com/kubernetes/charts/issues/6271#issuecomment-400437969
Possible solution

@huang-jy , thank you for providing information on your possible solution. May I ask, what are you sharing is the solution here? The CPU request being set to "1" rather than "1000m" or similar? Or perhaps the memory requested?
I have allocated 3000m & 9600mi respectively to Jenkins Master in our cluster yet still encounter the random disconnect. So I ask in hope that you've recognized something I have not recognized.

No, the default setting for Jenkins in the chart was something like 200m cpu and 500Mi memory. In https://issues.jenkins-ci.org/browse/JENKINS-50132, there was a comment that increasing the requested resource fixed the problem for them.

@sbeckwithiii also try increasing the jnlp requests.

For other users with this problem, copy-pasted from other issue:
Under http://jenkins-master/configure, look for tthe "Cloud -> Kubernetes -> Images -> Kubernetes Pod Section -> Containers" section, and you'll see the settings for the images -- one for the master and one for the slave.
Under the slave settings, there's a button labelled "Advanced" (it's just above the "Delete Container" button). Click that, and you'll see the resource request values

Have you tried increasing the resource requests as per https://github.com/kubernetes/charts/issues/3723#issuecomment-402112213 ?

Hi folks, we just started hitting this.  Our jnlp has no resource limitations. @huang-jy did you find out anything else about this?
My jnlp spec is as follows:
image: jenkins/jnlp-slave:alpine
imagePullPolicy: IfNotPresent
name: jnlp
resources: {}

@antweiss Try setting the resource request for cpu and memory to something so that the defaults aren't used?

@ricardompcarvalho Your jenkins endpoint is publicly accessible, you may wish to change that -- like, NOW.
Regarding your problem, I don't think that's the right location.
The default docker image for the agent/slave is jenkinsci/jnlp-slave
Let me copy in the comment from the other ticket:

under http://jenkins-master/configure, look for tthe "Cloud -> Kubernetes -> Images -> Kuernetes Pod Section -> Containers" section, and you'll see the settings for the images -- one for the master and one for the slave.
Under the slave settings, there's a button labelled "Advanced" (it's just above the "Delete Container" button). Click that, and you'll see the resource request values

@huang-jy my endpoint is public acessible but can you enter?

No, luckily :)

Yes, thatś because i use security although the endpoint is public!!

Can you give a solution for my problem? I put the configuration of resources in the right spot @huang-jy

The solution I had was to increase the resource requests for the slave, if that is not working for you, then you are probably having a different issue.

I just spun up a clean Jenkins install using:
helm install --name jenkins-jo stable/jenkins --version 0.25.0
I logged in, went to Manage Jenkins, then went to Configure System.
I scrolled down to "Cloud" section

Then a bit further down....

Just past this is the button I'm talking about:

Click on "Advanced" here:

This is where you change it. As you can see the values differs quite drastically from your screenshot

@huang-jy i know, i change where you refer!!!

@ricardompcarvalho That's fine, instead of using --set you can change the values in your yaml

@huang-jy i know, that what i been use, but if i install through yaml if I do not define an agent in yaml it creates me an agent, I think it is pre-defined in the chart

Yes, it'll create a config for an agent for you, if you dont' define one, it'll use the chart defaults. And that's possibly where you're getting caught out. Even on my cluster here with pure defaults, the Jenkins master keeps crashing out with OOMKilled

My jenkins master always say that he not have minimum availability :s thanks for the help, i think :D

Hi all,
I found the fatal reason at my k8s cluster. I installed Jenkins using helm too.
There are limits resources according to @huang-jy .

But, it didn't work for me even if I left blank.
The "Allocate pseudo-TTY" setting is decisive! The pipeline started after clicked the checkbox.

I don't think you need that set (I didn't have that ticked) -- though I guess it depends on what your pipeline is trying to do....
And removing the request/limit stuff will just revert it to the defaults of your cluster so it might work for some people who have set high defaults, and not others whoc have set lower defaults.

I am getting this. I believe my pods have enough resources. And even if it is a pod resource issue, it would be good if the stage would fail or restart the test when this happens. Kubernetes, by its nature, has nodes that get scaled down or otherwise go away, and if Jenkins is to be run in the Kubernetes environment (which is generally a good fit), then it should do this.

@nroose the issue here is that the pods have enough resource to START but not enough to do what they need to do. In my case, when I monitored the pods at the started, the default values of the chart caused the slave to be killed by the cluster with OOMKilled. Increasing the requested resources allowed the slave to do its job.

I am also facing issue jnlp connection disconnected,
we are using ec2 container service for this, Apreciate help a lot ..
FATAL: command execution failed
java.nio.channels.ClosedChannelException
at org.jenkinsci.remoting.protocol.NetworkLayer.onRecvClosed(NetworkLayer.java:154)
at org.jenkinsci.remoting.protocol.impl.NIONetworkLayer.ready(NIONetworkLayer.java:142)
at org.jenkinsci.remoting.protocol.IOHub$OnReady.run(IOHub.java:795)
at jenkins.util.ContextResettingExecutorService$1.run(ContextResettingExecutorService.java:28)
at jenkins.security.ImpersonatingExecutorService$1.run(ImpersonatingExecutorService.java:59)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
Caused: java.io.IOException: Backing channel 'JNLP4-connect connection from ip-1-x-x-x' is disconnected.

@tejas12356 please verify the settings as described earlier in the thread, and make sure you're using the latest version of the chart to pick up the PR changes described in #17168

I also had this issue while building my project, quite frustrated.
At first, I think it's related to the pod resource and tried to config K8s and Jenkins to adapt it. However, it does not solve the issue.
Then while checking the docker resource while building, I realized that the issue happens because of Docker container lack disk amount resources. After increase container memory, everything works fine.
Maybe in some cases, it's the root cause.

I also had this issue while building my project, quite frustrated.
At first, I think it's related to the pod resource and tried to config K8s and Jenkins to adapt it. However, it does not solve the issue.
Then while checking the docker resource while building, I realized that the issue happens because of Docker container lack disk amount resources. After increase container memory, everything works fine.
Maybe in some cases, it's the root cause.

https://github.com/helm/charts/issues/3723#issuecomment-402112213


RemoveUnusedPrivateFields produce random field deletions in a multimodule project.

Background
https://rewriteoss.slack.com/archives/C01A843MWG5/p1683439528216859
We need to reproduce the issue we need to create a multimodule repository and apply RemoveUnusedPrivateFields multiple times.
Relates to #1753

Hi @jbessels ; we just pushed a new snapshot version of the Maven plugin with a potential fix. Would be so kind as to give it another try with version 4.46.0-SNAPSHOT?
<build>
    <plugins>
        <plugin>
            <groupId>org.openrewrite.maven</groupId>
            <artifactId>rewrite-maven-plugin</artifactId>
            <version>4.46.0-SNAPSHOT</version>
        </plugin>
    </plugins>
</build>

Given that it's been non deterministic, it's been rather hard to pin down. We did see this being more prevalent with a larger number of submodules, so we're hoping it was the linked ordering issue. Would you mind running a few times even if it works the first time?

@timtebeek Copy/paste from Slack so the info does not get lost.
TL;DR behavior not changed.
Did some testing. Ran it two times with all 96 recipes. Run 1 no change -> good. 2nd run it hang. Even after 30 minutes it still hung. I've noticed and reported this before. With 4.45 and say 25+ runs with all 96 recipes  it blocked 2 times iirc. That was when I re tested it all and outcommented certain  files. Only in the beginning it blocked. At the end it did not block anymore. That is till I outcommented the excludes for RemoveUnusedPrivateFields and it hang on run 2. But in the previous 10+ tests with RemoveUnusedPrivateFields enabled and outcommented files it just worked. So it rarely happens, BUT it happens.
Decided to change tactics. Now only one recipe active: RemoveUnusedPrivateFields. At run 12 two out of 3 files were changed: SortableCollectionDataProvider.java and CRUDFormPage.java. The usage of LinkHashMap instead of HashMap has not changed the behavior.
Perhaps a better approach is to look at the very specific situation where it fails. As said in the OP it ONLY goes wrong when a general  or  generic is used in the variable. In the OP the class definition and the private fields are mentioned.


Remove unplugin-vue-components

Instead, directly import icons.

From the docs its says you have to add this to the tsconfig:
// tsconfig.json
{
  "compilerOptions": {
    "types": [
      "unplugin-icons/types/vue",
    ]
  }
}

Would actually try to stick to iconify as its pretty handy and flexibel. You can just change the import from @iconify/json to @iconify-json/mdi

I can revert it - but I don't really understand what you mean be "flexible". It's only flexible as long as you install all sets and that's explicitly what I not want to do

unplugin-icons has a quite unified way for importing different iconsets and seems to be well maintained in comparison to @jamescoyle/vue-icon (last update 3 years ago). I don't see a reason why we should replace it. 🤔 (Also really like the index)


Fixed macros.html.twig that does not show the exact value of submenu labels

To correct the error that does not show the exact value of submenu labels in the sidebar menu.
for example i tried ...
with an simple getMenu() in AdminThemeBundle/EventListener/SidebarSetupMenuDemoListener.php

We have ...

with {{ menu.menu_item_content(item, '') }}

with {{ menu.menu_item_content(child, '') }}

Thanks for all your work. Forgive me if I did not use the correct procedures, I'm new on github, and my English is poor.

@SalvoS81 good catch! Thanks for your contribution! Your english is enough good and the pull request was perfectly done. Thanks again for your time and work


Clicking a png file can open it twice, toggling between the two editors

This was happening over the weekend too, not just today's build
Repro:

Copy and paste an image via explorer
Click the image, it will open a new preview editor that doesn't use the custom editor
Click the image, it will switch back to the working editor

Confirmed, it looks like OverrideStatus.NONE is returned in the openEditor call but we then still create an editor input:

That one is obviously ignoring any override handling and thus opens normally.
Leaving it up to @lramos15 how to handle this better.

With the original fix, https://github.com/microsoft/vscode/issues/126923 happens again, thus smoke tests stop working which rely on this functionality.


Upgrade Track-o-Bot.app to v0.4.7

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


optional arguments in readme need descriptions

I think the asar, asar-unpack and this new asar-unpack-dir (https://github.com/maxogden/electron-packager/pull/174) need some documentation in the README. Nowhere does it currently explain what asar is, or what the various asar options do.
Also if anyone is feeling super helpful, all of the optional arguments (under 'Optional') could use descriptions. all has a description, but none of the others do:

In terms of describing asar, something concise and then a link to the Electron docs should be sufficient.

Also:

ignore - RegExp
out - String
overwrite - Boolean
prune - Boolean
sign - String
version-string - Object


update or remove Regentype field

Regentype (measurements table; populated based on plothistory data using @mmw590's R code) has not been populated for extratropical sites. Categories should be redefined to be more appropriate for extratropical sites.

Now addressed in #21.


Issue in app.py app routes.

@hindav
I want working project for live face recognition.
We are trying to implement  your face recognition project, Then we realized there were no app routes after render template, Its says rest of the code remains same . Please look through it and solve the problem

Hi Charan,
Thank you for bringing this up! The face recognition project is quite
detailed, and explaining everything over email can be a bit challenging. It
would be much easier to go over the app routes and other details in
real-time.
If you're comfortable, please share your WhatsApp number, and I’ll guide
you through the process step by step.
Looking forward to hearing from you!
Best,
@hindav
On Mon, 14 Oct 2024 at 13:25, Charan K @.***> wrote:

@hindav https://github.com/hindav
We are trying to implement your face recognition project, Then we realized
there were no app routes after render template, Its says rest of the code
remains same . Please look through it and solve the problem
—
Reply to this email directly, view it on GitHub
https://github.com/hindav/Face-Recognition-Using-Image-Video/issues/1,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWS4DOJY7W7LOB6XL7COWADZ3N2H5AVCNFSM6AAAAABP4MWYPWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU4DKMJTGUYTSNQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


[WIP] Fix the broken links in qiskit.pulse API doc

Work in progress
This PR aims to fix the broken links in qiskit.pulse API doc.
This is the first attempt to fix the following links:

qiskit.pulse.library.Constant_class.rst
qiskit.pulse.library.Sawtooth_class.rst
qiskit.pulse.library.Triangle_class.rst
qiskit.pulse.library.Cos_class.rst
qiskit.pulse.library.Sin_class.rst
qiskit.pulse.library.Gaussian_class.rst
qiskit.pulse.library.Drag_class.rst
qiskit.pulse.library.Square_fun.rst
qiskit.pulse.library.Sech_fun.rst

This PR doesn't currently appear to addressing the main problem here - I can see in recent docs builds that the stubs/qiskit.pulse.library.Sawtooth_class.rst.html is being generated correctly and is presumably getting copied up to exist on qiskit.org/documentation.  The problem is that if I try to go to qiskit.org/documentation/stubs/qiskit.pulse.library.Sawtooth_class.rst.html, following a link, I get 301 redirected to a file with the suffix Sawtooth_class.html instead of Sawtooth_class.rst.html.  That 301 redirection isn't managed by anything in this repository, so I think it's likely the CDN config somewhere doing it.
Fwiw, this PR is currently setting up meta-refresh redirects to apidocs/pulse.html, but that page does not have the actual documentation entries for these objects.

Thank you for your comment @jakelishman! You are right, it seems like we have the same problem generating the API docs on qiskit.org/documentation. All the URLs like stubs/qiskit.pulse.library.Sawtooth_class.rst.html are redirected to the same URL but without the .rst extension, and they are not able to be copied there either. I'm closing this PR :+1:


Auto-scaling for kube-flagd-proxy based on metrics (nice to have)

Once https://github.com/open-feature/flagd/issues/482 is done, we could have intelligent auto-scaling for the flagd-kube-proxy based on metrics related to load.

Closing due to inactivity. Please open a new issue and reference this one if this functionality is required.


Return kind of concrete types from type::name()

As a VAST developer using the type system, I would like to have a consistent and meaningful interpretation of the type "name". There is currently a mismatch on the query side and the implementation internally. For example, :timestamp has the name "timestamp" whereas :time has no name. Given that there is one global type namespace, this is an inconsistency in the model.
This change fixes that inconsistency throughout the type system.
:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on docs.tenzir.com/vast, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
This change is absolutely not user-facing. I've tested that extensively. If as a reviewer, you find a place where this changes VAST's behavior, please let me know, because I would consider that a bug in this diff.
For the code itself, review starting at type.hpp and then looking at the individual changes.

I also just got the idea of representing the name as the name attribute instead of a separate field, but I'm not sure this would give us a net benefit.

This is an implementation detail in the flatbuffer.
I think it should be clear what a "type name" is and what an "alias name" is. In the end, they share the same namespace in schemas and modules. In fact, they are the same thing from a user's perspective. If I call type::name(), VAST should give me the builtin type name if it's not an alias, and the name of the alias otherwise. In that regard I think we are all on the same page.
What's missing is a mechanism to distinguish builtin vs. aliases. I have no strong opinion how, other than it should be completely independent of type::name. So far, is_builtin(t) strikes me as a good choice.

This is an implementation detail in the flatbuffer.

I mentioned that specifically in the other thread because assign_metadata() would be simplified to assign_attributes so that part have a simpler implementation.

@mavam and I had a small chat about this PR and decided to scrap the whole idea. There's a better approach we can take for #2058, which is adding a generator that returns all actual aliases, i.e., metadata layers that have a custom name, either re-adding the attributes from metadata layers that were stripped away or stopping at the first metadata layer that has attributes. This would also allow for removing the type::names() function in its entirety.
A related bug fix for equality comparisons of types: the constructor for type that adds names and attributes is required to merge unnamed outermost metadata layers.

Closing; will open a new PR soon with the replacement.


Implement Driver Dashboard

Is your feature request related to a problem? Please describe.
Create a dashboard for drivers to accept ride requests, start rides, and complete them
Additional context

Build the driver dashboard UI.
Show incoming ride requests with accept/reject buttons.
Track ride status and completion flow.
Connect to the backend for real-time ride updates.

@visheshrwl please assign me this


Add table gitlab_notification_setting

We require gitlab_notification_setting table for our compliance mod, please let us know if there are any plans to add this table.
Reference:
<EMAIL_ADDRESS>

This API endpoint doesn't return anything of any real value...
On-Prem
❯ curl --header "PRIVATE-TOKEN: [censored]" "https://gitlab.[censored].com/api/v4/notification_settings"
<EMAIL_ADDRESS>
Public Cloud
❯ curl --header "PRIVATE-TOKEN: [censored]" "https://gitlab.com/api/v4/notification_settings"
<EMAIL_ADDRESS>
Therefore I don't think it's worth the investment in adding this particular table.


Refacto(Makefile): chain environnement into subdirectories

Description
All environment variables associated with CHAIN are in the "makefile_conf/chain" folder.
When a CHAIN is set, the makefile includes the config file associated with the chain as follows:
ex: makefile_conf/chain/ethereum.mk

Allows you to more easily add a new chain
More maintainable
More clarity in the makefile

Changes include

[X] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[X] I have assigned this PR to myself
[X] I have added at least 1 reviewer
[X] I have tested this code
[X] I have added sufficient documentation in the code
[X] I have added labels to this PR

Now instead of having to maintain two hardcoded lists of supported chains in the Makefile, it finds them automatically from their configuration files.


Update data.yaml - known addresses

Adding addresses used by domains.

I've removed record 94 - unsure if that will cause any issues if a number is missing.

Storing this here to archive. Thanks Github storage 👍


Add -DUSE_PKGCONFIG=OFF when building on Windows

When using vcpkg, pkg-config is not needed.
(The instruction as they are currently do not build on a fresh Windows since there is no pkg-config)

Thanks for this @eliemichel, we've made it so that this option is now OFF automatically on Windows (#1237)


Screenshots for Agentless Phase show two tasks in one Agentless Phase

The screenshots for the Agentless Phase show two tasks in one Agentless Phase, but that is not possible. When a user tries to save it will not let the release definition be saved with multiple tasks in one Agentless phase. The screenshots should be updated and it would be useful to call out the current limitation of allowing one one task per agentless phase in text right below the section header.

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

ID: 54bf6c2f-95c1-943d-26ae-d7099143bccf
Version Independent ID: 0833bb55-1e54-6953-ee69-de6d191d2ec1
Content
Content Source
Service: unspecified

Thank you. I will update the screenshots and check with the development team whether the limitation is just temporary or permanent.


move from defunct travis to github workflow

Q
A

Bug fix?
no

New feature?
no

BC breaks?
no

Deprecations?
no

Related tickets
fix #50

Documentation
README.md is being updated

License
MIT

What's in this PR?
Use github workflow to run the test suite with the known implementaitons
Why?
travis-ci no longer works
To Do

[ ] Adjust readme to link to github action runs

this runs the tests again and shows the problems.

laminas is the only implementation that has green builds (but only with PHP 8, some things have not been fixed in legacy versions that still support PHP 7). i wonder if we should only build with 8, or should allow failures with php 7
guzzle and slim have only a few failures
nyholm should be green when 1.6 is released (it is using these integration tests in its own test suite)
ring central looks much less close to the expectations

@GrahamCampbell are you currently maintaining guzzle? there seem to be some missing validations, do you think we can fix them in guzzle?

Thanks for the ping. We're are of this issue, and will look into it soon.


Hash Table: Add First Missing Positive in Java

Given an unsorted integer array nums, return the smallest missing positive integer.
You must implement an algorithm that runs in O(n) time and uses O(1) auxiliary space

I'll take this up


Got a pytorch RuntimeError when reproducing the unsup result

tried train_pytorch_U2GNN_UnSup.py and got:
Namespace(batch_size=4, dataset='PTC', dropout=0.5, ff_hidden_size=1024, fold_idx=1, learning_rate=0.005, model_name='PTC', num_epochs=50, num_hidden_layers=2, num_neighbors=4, num_self_att_layers=1, run_folder='../', sampled_num=512)
Loading data...
loading data
classes: 2
maximum node tag: 19
data: 344
19
Loading data... finished!
Writing to C:\Users\Administrator\Desktop\runs_pytorch_U2GNN_UnSup\PTC
Traceback (most recent call last):
File "C:\Users\Administrator\Desktop\U2GNN\U2GNN_pytorch\train_pytorch_U2GNN_UnSup.py", line 206, in
train_loss = train()
File "C:\Users\Administrator\Desktop\U2GNN\U2GNN_pytorch\train_pytorch_U2GNN_UnSup.py", line 157, in train
logits = model(X_concat, input_x, input_y)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\torch\nn\modules\module.py", line 550, in call
result = self.forward(*input, **kwargs)
File "C:\Users\Administrator\Desktop\U2GNN\U2GNN_pytorch\pytorch_U2GNN_UnSup.py", line 31, in forward
input_Tr = F.embedding(input_x, X_concat)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\torch\nn\functional.py", line 1724, in embedding
return torch.embedding(weight, input, padding_idx, scale_grad_by_freq, sparse)
RuntimeError: Expected tensor for argument #1 'indices' to have scalar type Long; but got torch.cuda.IntTensor instead (while checking arguments for embedding)
Seems like a int32 to int64 error
so fixed by adding "input_x = input_x.astype(np.int64)" behind "input_x = np.array(input_neighbors)" in line 124

Our unsupervised results shown in our paper are obtained using the Tensorflow implementation. I do think the Pytorch implementation can get better performance.
Note that in case you follow our unsupervised learning, you may get risks. Reviewers from ICLR and ICML did not accept our unsupervised results (because of too good to be true). I clearly mentioned that I just followed some unsupervised graph embedding models to use all nodes from the entire dataset to train the unsupervised U2GNN, but they do not accept this fact.


dict=rootDict entry=Version: unsupported in version 1.3

I am having issue with optimize pdf.
>$ pdfcpu optimize -v res.pdf
INFO: 2019/07/15 10:48:36 reading res.pdf..
INFO: 2019/07/15 10:48:36 PDF Version 1.5 conforming reader
INFO: 2019/07/15 10:48:36 identifyRootVersion: PDF version is 1.3 - will ignore root version: 1.3
INFO: 2019/07/15 10:48:36 validating
Fatal: dict=rootDict entry=Version: unsupported in version 1.3
This file could be PDF/A compliant but pdfcpu only supports versions <= PDF V1.7

github.com/hhrutter/pdfcpu/pkg/pdfcpu.(*XRefTable).ValidateVersion
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/pdfcpu/xreftable.go:171
github.com/hhrutter/pdfcpu/pkg/pdfcpu/validate.validateNameEntry
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/pdfcpu/validate/objects.go:673
github.com/hhrutter/pdfcpu/pkg/pdfcpu/validate.validateRootVersion
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/pdfcpu/validate/xReftable.go:59
github.com/hhrutter/pdfcpu/pkg/pdfcpu/validate.validateRootObject
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/pdfcpu/validate/xReftable.go:890
github.com/hhrutter/pdfcpu/pkg/pdfcpu/validate.XRefTable
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/pdfcpu/validate/xReftable.go:33
github.com/hhrutter/pdfcpu/pkg/api.readAndValidate
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/api/api.go:244
github.com/hhrutter/pdfcpu/pkg/api.readValidateAndOptimize
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/api/api.go:255
github.com/hhrutter/pdfcpu/pkg/api.Optimize
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/api/api.go:287
github.com/hhrutter/pdfcpu/pkg/api.Process
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/pkg/api/process.go:84
main.process
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/cmd/pdfcpu/main.go:219
main.main
	/Users/yangye/go/src/github.com/hhrutter/pdfcpu/cmd/pdfcpu/main.go:182
runtime.main
	/usr/local/go/src/runtime/proc.go:200
runtime.goexit
	/usr/local/go/src/runtime/asm_amd64.s:1337

Thanks alot!

Well, Root.Version is a feature since V1.4
Who is the producer of this.
Can you run pdfcpu infofor me?

res.pdf

res.pdf
This is my pdf file, I generated it by unidoc software.

ok. so this is NOT spec compliant but since it opens with Adobe Reader I am going to extend relaxed validation in pdfcpu. Stay tuned and thanks for reporting this. :green_heart:

This is fixed with v0.2.1

I'm getting this error while trying to stamp an image onto a v1.3 pdf.
$ pdfcpu info ~/Downloads/d8249f62f79b8a7b54be989c.pdf
         PDF version: 1.3
          Page count: 21
..........................................
               Title: When do we need another SWAT evaluation 2-10-2018
              Author:
             Subject:
        PDF Producer: Mac OS X 10.13.6 Quartz PDFContext
     Content creator: Pages
       Creation date: D:20181008184639Z00'00'
   Modification date: D:20181008184639Z00'00'
..........................................
              Tagged: No
              Hybrid: No
          Linearized: No
  Using XRef streams: No
Using object streams: No
..........................................
           Encrypted: No
         Permissions: Full access

$ pdfcpu stamp 'functions/stamp-pdf/stamp.png' ~/Downloads/d8249f62f79b8a7b54be989c.pdf
dict=formStreamDict entry=OC: unsupported in version 1.3
This file could be PDF/A compliant but pdfcpu only supports versions <= PDF V1.7

Are you using the latest commit?

I am, yes. On 0.2.1

This is the latest release but not the latest commit.
Sorry if that sounds confusing.
Just go get everything and rebuild.

Sorry for the delay. I've downloaded the latest commit and it still has the same issue. I've attached the PDF exhibiting the issue.
Treweek et al 8-10-2018.pdf

fixed with the latest commit.

Can confirm it's working now. Thank you!


Disable CPR by default

Fix #445
TODO:

[ ] Validate deactivation by default
[ ]  Deactivate only on unix platforms (windows is almost fine => #433, #359) ?

@jonathandturner, @tailhook Are you OK if we don't check that cursor is at left most position by default ?


Follow dark style preference

When the OS is put into dark mode (or prefer-dark for the freedesktop preference), the ddg privacy extension does not adapt to follow the dark theme, even when firefox does. Is this fixable from the extension side or is this an issue somewhere else?

Yes, I think the extension doesn't include "dark-mode", I hope the team add this new Feature 👏👏🚀🚀
Also, the extension looks great!!

Closing in favour of #356 we have this on the timeline along with many other UI changes. Watch this space 👍🏻


Simple custom Layer object has no 'built' attribute

Please make sure that the boxes below are checked before you submit your issue. Thank you!

[X] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[X] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[X] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Trying to get around LabmdaLayer not working, I attempted a simple custom layer.
class LastStep(Layer):
    def __init__(self, **kwargs):
        super(Layer, self).__init__(**kwargs)

    def call(self, x, mask=None):
        return x[:,-1,:]

    def get_output_shape_for(self, input_shape):
        shape = list(input_shape)
        assert len(shape) == 3
        return tuple((shape[0], shape[2]))

lstm_layer_size = 32
embed_size = 32
num_layers = 3
maxlen = 10
print('Build model...')
main_input = Input(shape=(maxlen,), dtype='int32', name='main_input')
emb = Embedding(257, embed_size, input_length=maxlen)(main_input)

#hidden states from each LSTM
hs = []

#first layer from embedding only
hs.append(LSTM(lstm_layer_size, return_sequences=True)(emb))
for l in range(1, num_layers):
    l_in = merge([emb, hs[-1]], mode='concat')
    hs.append(LSTM(lstm_layer_size, return_sequences=True)(l_in))

last_steps = [LastStep()(x) for x in hs]
merged_penultimate = merge(last_steps, mode='concat')
loss_out = Dense(1, activation='sigmoid', name='loss_out')(merged_penultimate)

model = Model(input=[main_input], output=[loss_out])
optimizer = Adam(lr=0.001, clipnorm=grad_clip)
model.compile(optimizer, loss='binary_crossentropy')

This fails with the error
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-8-da16a6eca8bd> in <module>()
---> 19 last_steps = [LastStep()(x) for x in hs]
     20 merged_penultimate = merge(last_steps, mode='concat')
     21 loss_out = Dense(1, activation='sigmoid', name='loss_out')(merged_penultimate)

/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py in __call__(self, x, mask)
    436             mask: tensor or list/tuple of tensors.
    437         '''
--> 438         if not self.built:
    439             # raise exceptions in case the input is not compatible
    440             # with the input_spec specified in the layer constructor

AttributeError: 'LastStep' object has no attribute 'built'

Which is very confusing, as built is set in the constructor of Layer. If I set the built value myself, the issue just continues to another value that should have been set by the Layer constructor

You wrote the class incorrectly.
class LastStep(Layer):
    def __init__(self, **kwargs):
        super(Layer, self).__init__(**kwargs)

You want the super method to take in LastStep as its first argument.

So is the example in the docs wrong? It uses super(Layer, self).__init__(**kwargs)

Looks like a typo. Let me update the docs.
On 2 May 2016 at 14:12, RaffEdwardBAH<EMAIL_ADDRESS>wrote:

So is the example in the docs
http://keras.io/layers/writing-your-own-keras-layers/ wrong? It uses super(Layer,
self).init(**kwargs)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/fchollet/keras/issues/2584#issuecomment-216365896

Doing as @braingineer mentioned does fix it, and it does appear to searlize out to a JSON object where the LambdaLayer does not (though I'm having trouble getting the model to train, but I don't know if its an error from 0.3.2 code moving to 1.0.X)


For hdk 0.4.0 dev.14

Add noAdditionalField property to the my-profile and profile-detail components to optionally not render additional properties.

Wouldn't you just not specify any additional fields if you didn't want to render additional fields? I guess this was already published though so lets just go with it. LGTM

Wouldn't you just not specify any additional fields if you didn't want to render additional fields?

This is a Moss-biased PR where apps of the same Moss group all reuse the same profiles zome and certain Tools require additional fields for their app-specific logic. But when rendering the profile in group contexts, they should not show up...
Definitely not the prettiest nor long-term solution but I just needed something that works for now.


Implement FromStr for hash types

Run through the source and figure out what types it would be useful to have FromStr implemented for (e.g., BlockHeaderHash, etc), then implement it.

This is partially done in #311 (BlockHeaderHash). Which other hash types should we provide an impl for?

TransactionHash, SaplingNoteTreeRootHash, MerkleTreeRootHash also exist

@dconnolly what do you think of the following: add FromStr for TransactionHash, then whenever we add a nicer Debug impl for other hash types, we add a FromStr impl for it.

@dconnolly what do you think of the following: add FromStr for TransactionHash, then whenever we add a nicer Debug impl for other hash types, we add a FromStr impl for it.

Sounds good to me!

@dconnolly @hdevalence Would the commit above help?
PS: A macro would make the code shorter but current IDEs are mostly unhappy about them.


RebalancingSetToken code organization and touch ups

Non functionality changing code touch ups

Pull Request Test Coverage Report for Build 2977

21 of 21 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 2972:
0.0%

Covered Lines:
666

Relevant Lines:
666

💛  - Coveralls


Function comments

[ ] create_case.R
[ ] create_clinical.R
[ ] create_genomic.R
[ ] create_panel.R

May also be a good opportunity to consolidate functions used my multiple scripts into a shared_fxns.R file.


Added class NodeList

Allows for basic attribute filtering using filter_by method.
The user-facing API is inspired by the SQLAlchemy Query object.
This replaces PR #52.

Current coverage is 96.89%

Branch #93 has no coverage reports uploaded yet.

Powered by Codecov. Updated on successful CI builds.

@econchick Any chance this could be merged? Don't want to do another round ;-)

Hey @jmagnusson ! Sorry for the delay!
You're patch is queued up to merge - I just need to figure out what I f*cked up when I was blindly merging a few days ago (lesson learned: don't merge via mobile!).

awesome - thank you!

Awesome! Hope your merge issue was resolved. When can we expect a 0.2.0 release?

@econchick Any chance we could have this backported? 😇

@jmagnusson reasonable request - if you'd like to make a PR against master, by all means. Otherwise it'll be prioritized after RAML 1.0 support (which I'm currently working on right now! literally! finally!)

Wohoo! Maybe I can hold on a while longer then 😉


GET API SQLSTATE[42000]: Syntax error or access violation: SQL: select  from my_table_name where my_table_name.deleted_at is null

Windows 7 Wamp Server
infyomlabs/laravel-generator "5.8.x-dev"
php artisan infyom:api MyTableName
field1_id integer text s
field2_id integer text s
field3_id text textarea s, ii
Running GET API /api/v1/myTableNames

@biholaindrasinh can you post a full error or the table schema that you have?
Seems like you have some SoftDelete problem.

I can't share my table schema that's why I have added a general description regarding it. Let me know if any specific you want to know.

Here is a stack trace:
https://gist.github.com/biholaindrasinh/9f3a3e77ce02fe640403e9a31e9f9c97

If I remove this 3 request parameters then it's working.

@biholaindrasinh are you passing any query parameter while calling the API?

No, I was not passing any parameters but it should return all the resources if we don't pass any parameters.

@biholaindrasinh I think you are using an old base repository. Can you paste a code of BaseRepository somewhere?
Also did you set up a fresh project? or it's an old project? And you made some upgrade?

It's an ongoing project. It looks like a problem in prettus/l5-repository: ~2.6

Yes, I would say for your new generated files, you the BaseRepository that we published. Give it a try with a fresh new project and you will get an idea how new BaseRepository works and we are no longer need old prettus/l5-repository.


[iOS] Timing: Fixes timer when app get into background

Summary
Related #23674, in that PR, we imported background timer support, but it's not sufficient, I think the reason that works is because it enable the Background Modes and do some background tasks, for the users who don't enable it, timer would pause immediately before goes into background.
To fix it, we can mark a background task when goes into background, it can keep app active for minutes, try best to support timing when in background.
cc. @cpojer .
Changelog
[iOS] [Fixed] - Timing: Fixes timer when app get into background
Test Plan
Run code in background like the code below, console can log correctly.
    setInterval(() => {
      console.log("Timer run");
    }, 1000);

Just a note that react-intl will set huge timeouts for its <FormattedRelative> components, so this could cause huge issues for some people's code

@jacobp100 Eh, what do you mean? can you expand which issues you want to express? iOS can't run for days in background.

I just wanted to highlight a popular library people use on React Native calls setTimeout with a delay that can last for days. If we wait for every set timeout by running the app in the background, it could, in some cases, be a drain on the battery.
Not saying it shouldn't be merged, I just wanted to highlight one potential issue!

@jacobp100 I don't know wether I get which issue you want to express. You mean it may has battery issue? Actually, we already support background timer in #23674(Many people have expressed a desire for background timer support on iOS ), and background task can only last most a few minutes, so we only can support background timer for a short time, if app not return to foreground, it will be killed by system.
And if you have some other issues, please comment, and we'll try to fix them. Thanks.

Ignore me then! 😄

Fixed #25083.

Cheers!

I have updated RCTTiming.m file with lasted code but still js code not work when app in background.
thanks!


Error message and docstring updates RE: dynamic_slice

This should clarify the underlying issues from #1007 and #3794.
It might be worth mentioning masking, but that's a little big for fitting into an error message. Maybe once the masking transformation is non-experimental or if we had a dedicated doc page.

I wonder if it's worth linking to some documentation describing common patterns and alternatives for dynamic shapes. I admit that documentation does not yet exist.


css flex support

Hi there,
is the latest capybara-webkit supporting css flex? I am having issues finding elements on a login page using flex in my layout.
Thanks!

Which CSS properties are supported will depend on the version of Qt you have installed. You can see your versions by running this in a test or spec:
puts page.driver.versions

I'm not sure which versions of Qt and WebKit support flex, but I think anything recent (Qt 5.x) should work.
If you're on Qt 5 and still having issues, here are some questions that will help me debug your issue:

What application layer are you using? (Rails?) What versions are you using?
What versions of RSpec, Capybara, capybara-webkit, and Qt are you using?
Does the spec pass if you use the Selenium driver instead of capybara-webkit?
What error message are you getting on your login page?
What does the markup for your login page look like?
If you're having issues finding elements, what does save_and_open_screenshot show?

Can confirm that flex works with Qt 5.x.

I have the same issue in which flex rules are ignored and as such some elements that shouldn't be overlapping, are. I can see this in the screenshot.
Here's the versions I'm using:
Capybara: 2.7.1
capybara-webkit: 1.11.1
Qt: 5.5.1
WebKit: 538.1
QtWebKit: 5.5.1
I'm also using:
Rails 4.2.5
RSpec 3.1.0
And it does work with selenium. Any idea on how to fix this?

@zamith It looks like your versions are all pretty up-to-date, so there may not be anything you can do.
Are you using the -webkit prefixes for your flex declarations?

I forgot to add autoprefixer that fixed it. Thanks.

Ah, good to know. Looks like this is well-supported by recent versions, so I'm going to close this.


[WIP] soundux: init at 0.2.7

Motivation for this change
Things done
Builds succeeds and the programs run.

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@pasqui23 - A short while ago I took the derivation from an older PR for soundux and fixed it up to compile and work on my system. It might help if you're going to try and get it brought into the repo:
{
  lib, stdenv, fetchFromGitHub, cmake, ninja, pkg-config, makeWrapper, wrapGAppsHook,
  libappindicator-gtk3, openssl, pipewire, pulseaudio, webkitgtk, xorg,
  libpulseaudio, libwnck3,
  downloaderSupport ? true, ffmpeg, youtube-dl
}:

let
  downloaderPath = lib.makeBinPath [ffmpeg youtube-dl ];
  # Soundux loads pipewire, pulse and libwnck optionally during runtime
  dynamicLibraries = lib.makeLibraryPath [libpulseaudio pipewire libwnck3 ];
in
stdenv.mkDerivation rec {
  pname = "soundux";
  version = "0.2.7";

  src = fetchFromGitHub {
    owner = "Soundux";
    repo = "Soundux";
    rev = version;
    fetchSubmodules = true;
    sha256 = "15kd9vl7inn8zm5cqzjkb6zb9xk2xxwpkm7fx1za3dy9m61sq839";
  };

  cmakeFlags = [ "-DCMAKE_BUILD_TYPE=Release" ];

  dontWrapGApps = true;

  installPhase = ''
    mkdir -p $out/opt $out/bin
    cp -r dist soundux-${version} $out/opt
  '';

  postFixup = ''
    makeWrapper $out/opt/soundux-${version} $out/bin/soundux \
      --prefix LD_LIBRARY_PATH ":" ${dynamicLibraries} \
      "''${gappsWrapperArgs[@]}" \
      ${lib.optionalString downloaderSupport "--prefix PATH \":\" " + downloaderPath}
  '';

  nativeBuildInputs = [ cmake ninja pkg-config makeWrapper wrapGAppsHook ];
  buildInputs = [
    libappindicator-gtk3
    openssl
    pipewire
    pulseaudio
    webkitgtk
    xorg.libX11
    xorg.libXtst
  ];

  meta = with lib; {
    homepage = "https://soundux.rocks/";
    description = "cross-platform soundboard";
    license = licenses.gpl3Only;
    platforms = platforms.linux;
  };
}

@Slabity Sorry even with your fix still does not compile

@Slabity Sorry even with your fix still does not compile

Strange. That derivation builds for me and lets me run it without any issue.

Ok @Slabity my mistake. Now it builds and run perfectly

@dali99 - Uh... I think I screwed up my copy-paste because I definitely added a lot more dependencies in mine to fix some linking issues and get all the additional features I wanted. And it doesn't look like I removed the maintainer field. Here's what I have in my repo at the moment:
{
  lib, stdenv, fetchFromGitHub, cmake, ninja, pkg-config, makeWrapper, wrapGAppsHook,
  libappindicator-gtk3, pcre, openssl, pipewire, pulseaudio, webkitgtk, xorg, utillinux,
  libpulseaudio, libwnck3, libselinux, libsepol, libthai, libdatrie,
  libxkbcommon, epoxy, dbus_tools,
  downloaderSupport ? true, ffmpeg, youtube-dl
}:

let
  downloaderPath = lib.makeBinPath [ffmpeg youtube-dl ];
  # Soundux loads pipewire, pulse and libwnck optionally during runtime
  dynamicLibraries = lib.makeLibraryPath [libpulseaudio pipewire libwnck3 ];
in
stdenv.mkDerivation rec {
  pname = "soundux";
  version = "0.2.7";

  src = fetchFromGitHub {
    owner = "Soundux";
    repo = "Soundux";
    rev = version;
    fetchSubmodules = true;
    sha256 = "15kd9vl7inn8zm5cqzjkb6zb9xk2xxwpkm7fx1za3dy9m61sq839";
  };

  cmakeFlags = [ "-DCMAKE_BUILD_TYPE=Release" ];

  dontWrapGApps = true;

  installPhase = ''
    mkdir -p $out/opt $out/bin
    cp -r dist soundux-${version} $out/opt
  '';

  postFixup = ''
    makeWrapper $out/opt/soundux-${version} $out/bin/soundux \
      --prefix LD_LIBRARY_PATH ":" ${dynamicLibraries} \
      "''${gappsWrapperArgs[@]}" \
      ${lib.optionalString downloaderSupport "--prefix PATH \":\" " + downloaderPath}
  '';

  nativeBuildInputs = [ cmake ninja pkg-config makeWrapper wrapGAppsHook ];
  buildInputs = [
    libappindicator-gtk3
    openssl
    libselinux
    libsepol
    libthai
    libdatrie
    utillinux
    pcre
    pipewire
    pulseaudio
    webkitgtk
    xorg.libX11
    xorg.libXtst
    xorg.libXdmcp
    libxkbcommon
    epoxy
    dbus_tools
  ];

  meta = with lib; {
    homepage = "https://soundux.rocks/";
    description = "cross-platform soundboard";
    license = licenses.gpl3Only;
    platforms = platforms.linux;
    maintainers = with maintainers; [ dandellion ];
  };
}

Sorry for the mixup, I wasn't intending to get my changes committed at all. I just happened to see this PR and thought I'd link to the older one since it was closer to working.

No worries, everything after the first section was mainly aimed at @pasqui23 anyways :)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/827

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

1 package failed to build:

soundux

I couldn't get this to run due to some EGL issues:
[00:56:35] [success] Config read
[00:56:35] [success] LibWnck found - Icon support is enabled

(soundux-0.2.7:342196): Gdk-WARNING **: 20:56:35.822: Failed to read portal settings: GDBus.Error:org.freedesktop.DBus.Error.UnknownMethod: No such interface ?org.freedesktop.portal.Settings? on object at path /org/freedesktop/portal/desktop

(soundux-0.2.7:342196): Wnck-WARNING **: 20:56:35.841: libwnck is designed to work in X11 only, no valid display found
[00:56:35] [warning] Failed to get default screen!
[00:56:35] [failure] Could not create IconFetcher instance
[00:56:35] [message] Connected to PipeWire ("pipewire-0") on version "0.3.49"
[00:56:36] [warning] Path "/home/me/audio-clips" does not exist
[00:56:36] [message] Using DISPLAY :0
[00:56:36] [warning] Failed to find iconPath for tray icon
Could not determine the accessibility bus address

(WebKitWebProcess:342222): Gdk-WARNING **: 20:56:36.348: Failed to read portal settings: GDBus.Error:org.freedesktop.DBus.Error.UnknownMethod: No such interface “org.freedesktop.portal.Settings” on object at path /org/freedesktop/portal/desktop
EGLDisplay Initialization failed: EGL_NOT_INITIALIZED
Cannot create EGL context: invalid display (last error: EGL_SUCCESS)

EGLDisplay Initialization failed: EGL_NOT_INITIALIZED

see also #169201

superseded by https://github.com/NixOS/nixpkgs/pull/185858


Missing Texture in Thermal Expansion Jetpacks

minecraft: 1.10.2
forge: <IP_ADDRESS>2
simplyjetpacks: <IP_ADDRESS>

I did not see any TE jet backpack texture in the source code
It seems that the problem is from:https://github.com/CyberdyneCC/SimplyJetpacks-2/commit/ecef7844fab2b9d833e41a323c1ae0dfdcef45ea
isn't have TE textures in source:https://github.com/CyberdyneCC/SimplyJetpacks-2/tree/1.12/src/main/resources/assets/simplyjetpacks/textures/armor
but still have TE's texutures in here:https://github.com/CyberdyneCC/SimplyJetpacks-2/tree/1.12/resources_old/armor (TE)
I think just need readd it

Please guys read the changelog provided on Curse before or when downloading a new version.
There I explicitly stated that the TE implementation is not finished yet... I am currently redoing the textures, too, I will add them when they are finished.

Really dissapointing that textures for Thermal Dynamics still haven't been implemented. Does anyone know if it would be possible to use the textures from the original mod as a fix?

@Tomson124 Reminder that this is still not fixed. The assets seem to be present in the 1.12 version so I have no idea why they aren't in the 1.10 one,

Because they are not implemented yet in the 1.10 version. And maybe won't be in the future, too, as I am concentrating on the 1.12 version right now.

I would like to add, as a mod developer myself, updating the 1.10.2 version wouldn't take a huge amount of work if 1.12 is already done. I use 1.10.2 to play modded, because some of the mods I use haven't been updated to that version yet. I would like to see the TE implementation in 1.10 finished.

If @Tomson124 you want to backport fixes to 1.10 reopen this, but IMO 1.12 is the only supported version atm.


⚠ ClamAV has degraded performance

In d2c0e27, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 64 ms

Resolved: ClamAV performance has improved in 94d03db.


Whether supports helm3 dependency function in Chart.yaml

As the titile.
After I push the chart package to GCS,  I find the Chart.yaml of package miss the dependency for Helm 3.
Any comments?
Thanks a lot!

Hello @xiaofeixeniro could you provide more details (steps to reproduce) about the issue?

This issue has been found, you can close this ticket, thanks a lot!

@xiaofeixeniro nice to hear. Closing the issue.


Skip models that do not respond to :display_name

It seems that some models (VmReconfigureTask) use display_name as a private method for their own thing, so we need to skip these models and perhaps rename the method to something non-conflicting.
Otherwise we'd be getting a NoMethodError when doing ApplicationRecord.descendants.sort_by(&:display_name)
@miq-bot add_label jansa/yes, internationalization

I'm going to merge this for now, but @gmcculloug, can we change the VmReconfigureTask.display_name method name to something else to avoid conflict?

Yeah, seems like a really easy change as that method was just added, not back-ported and is a private class method.  I'll put in a PR tonight.

Jansa backport details:
$ git log -1
commit 775f6bb79de699312ec28b7a552e499a71274824
Author: Jason Frey<EMAIL_ADDRESS>Date:   Tue Mar 17 17:32:56 2020 -0400

    Merge pull request #19961 from mzazrivec/fix_display_model_name_extraction

    Skip models that do not respond to :display_name

    (cherry picked from commit 7639787b267c5e070dc1b33303fa22ee6357665c)


WotLK: Distance count: players shouldnt be enemys

Distance Count:
Players from the other fraction (no PvP) will be counted as enemys.  Not really a good idea

Uhh, they kind of should.  It's possible I could add some behavior based on PVP vs. non-PVP zone or something.   But yes, players of opposing factions are enemies.


TTNN Unet inference pipeline

setting up inference pipeline with available PCC
Currently - PCC (~0.0)
Expected behaviour - To get PCC > 0.99
To reproduce:
go to harini/unet_inference
execute demo.py file.

@saichandax and @HariniMohan0102 I am a little confused on this one. is this the same model as in #8454? do you mean when you provide a real image input, the pcc goes from 0.8 to 0?

@dvartaniansTT We are using the same model as in #8454 to test inference with a real image input, the pcc drops from 0.8 to ~0.0

@dvartaniansTT / @saichandax, can you try using the metal PCC visualizer to debug?

@dvartaniansTT is this still valid?


feat: Add initial version of anamnesis.ai with rago

This  PR aims to implement the first initial version of anamnesisai using chatgpt

it needs more and complex tests. but for now it should be ok.


[e2e]: container-based schedule main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7429058047
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container-based.schedule.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container-based.schedule.main.default.slsa3.yml
Trigger: schedule
Branch: main
Date: Sat Jan  6 03:21:42 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7435680000
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container-based.schedule.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container-based.schedule.main.default.slsa3.yml
Trigger: schedule
Branch: main
Date: Sun Jan  7 03:21:17 UTC 2024
Tests are passing now. Closing this issue.


CC0 licence?

Hi there. I was wondering if you would consider switching to a public domain licence like CC0 or the unlicense?
Complying with the MIT licence is often not too hard - if you're using a library then you have an exact copy and you simply preserve the licence declaration. Easy. Not so when creating a derivative work, and this is a template project so all usage is derivative in nature. How do you refer to the code that remains copied from here to attribute the MIT licence?
If a part of that project itself is used to create another derivative work, how is that author to know if they have used any of this project's code, and whether they are subject to its MIT licence? That downstream author, if paying attention to compliance, may be forced to write something like:
Though I am not aware of any specific examples, there may be portions of this
project that are considered derivative of hardhat-starter-kit. The copyright
for those portions would be held by pappas999 (c) 2021, and the MIT licence
below would apply.

(MIT licence goes here)

IANAL, but to the best of my knowledge, project authors are required to do this kind of thing. I think it's a bit silly, and it would be better if we all stopped worrying about attribution in open source licences, and just committed the work to the public domain (assuming having other projects benefit from your work is the goal, of course).
Wdyt?

Thanks for bringing this up! @StephenFluin wondering if you have any insight here?

The MIT license is often chosen purposefully so that derivative works remain open source. https://www.gnu.org/licenses/copyleft.en.html

The MIT license is often chosen purposefully so that derivative works remain open source. https://www.gnu.org/licenses/copyleft.en.html

If that's the intention then wouldn't the GPL better serve that goal?
I'm not sure that MIT is intended to be copy-left in this way. My understanding is that MIT is permissive, promoting wide adoption, and the terms are there merely to disclaim liability and preserve some credit for the author, deliberately avoiding any additional restrictions on derivative works, since that would inhibit derivative works.
Of course, if you consider this copy-left aspect of the MIT licence to be a feature and not a bug, then I respect that. Personally though, it means I'd rather put together a hardhat project manually than use this template.


fix: set defaultHelp on duration flag

Set defaultHelp on duration flag so that we get a human readable string in the help output
@W-14625639@

QA:
generated an oclif manifest with yarn oclif manifest and inspected oclif.manifest.json wait flag entries.  All entries had a readable string of: "default": "0 minutes",


decompressor works incorrectly with native zlib

Steps:
Make a bytes array with length greater than 276329. compress it then decompress.
decompressed does not equal to original one.

fixed in PR #273


Showing All Issues /bin/sh: line 3: /Users/shashanktiwari/Downloads/caffe-ios-sample-master/caffe/bin/protoc: No such file or directory

cp: /Users/shashanktiwari/Library/Developer/Xcode/DerivedData/CaffeApp-dljeohfdehdatlgndrpjxxjlbdzf/Build/Intermediates/CaffeApp.build/Debug-iphonesimulator/CaffeApp.build/DerivedSources/caffe/proto/caffe.pb.h: No such file or directory

I encountered these two issues while building the project. Please help.

Sorry but it's not maintained anymore. Try other frameworks.


Fix code block end marker

Looks like someone accidentally removed the ``` from the end of a code block, causing the Serialization without Macros section to be formatted like code

There were the following issues with your Pull Request

Commit: 7ad31a01ddf62bc6d9285b9fa7be960ae7bc46bb
Your commit message body contains a line that is longer than 72 characters
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/serde-rs/serde

This message was auto-generated by https://gitcop.com


Ctl command status certificate: Add order output

What this PR does / why we need it:
Adds output about the Order resource for status certificate command if ACME Issuer is used.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Add output about Order resource for `status certificate` command if ACME Issuer is used.

/kind feature
/area ctl

This PR looks good to me.

/approve
/lgtm


Improve API unit test coverage

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Our test coverage report shows we have gaps in testing for a number of API modules.  We should try to achieve 100% if possible.
Describe alternatives you've considered
If we don't do this, this is technical debt and will impact the QA of our project
Additional context
Latest test coverage reports can be found in the Actions > CI workflow

we have codecov now


i got this error when i ran the latest

i think i dont have the resource that you are using in my RP5 please modify the requirement check

fixed in commit : docs: added ads1015 to requirements
added ads1015 to requirements.txt


Support for nvidia 3000 series GPUs

Would love to run this, but had some trouble getting the GPU version running as it doesn't support my GPU. I tried using a newer version of the pytorch base image to get past this, but ran into some issues.

That's great! Contributions are very welcome.


An element of an array gets parsed as multiplicative_expression

Few examples:
cText = doStuff(myArray[i]).
cNPI = cLineData[1]

this might need external scanner...


Update output_dataset.meta also when workflow execution fails

Branching from https://github.com/fractal-analytics-platform/fractal-server/issues/842#issuecomment-1719147615 (and differently from #788):
When a workflow fails at task N, update output_dataset.meta to the latest valid state (that is, the output of task N-1).
Consequences of this change in different scenarios still need to be reviewed.

When a workflow fails at task N, update output_dataset.meta to the latest valid state (that is, the output of task N-1).

This seems to be a valid behavior from my perspective.
There can be some complexities depending on how exactly task N failed and what it may already have changed. But for 99% of cases, the N-1 metadata would be correct and would allow e.g. resuming workflows that failed at a late task. Also, many edge-cases will be handled by the tasks themselves through their overwrite functionality.

When a workflow fails at task N, update output_dataset.meta to the latest valid state (that is, the output of task N-1).

This seems to be a valid behavior from my perspective.
There can be some complexities depending on how exactly task N failed and what it may already have changed. But for 99% of cases, the N-1 metadata would be correct and would allow e.g. resuming workflows that failed at a late task. Also, many edge-cases will be handled by the tasks themselves through their overwrite functionality.

Fine, I'll merge #845 and make a pre-release 1.3.6a0.
Let's test this a bit more, before actually updating the running instances to 1.3.6.


Run OATS on log-jam

Refactor the logging/log-jam (Internal) application to run on OATS (Internal) so that we can understand the overhead of the logging initiative on the agent.

Please add your planning poker estimate with ZenHub @fallwith

Results have been published internally. Things are looking good. Some questions remain for Rich when he returns.


Frozen on Sending irc NICK/USER

When launched, discord-irc starts and connects to Discord, but stays on 29 Aug 15:43:52 - Sending irc NICK/USER for IRC connection. Several munites later, it connected, but is irratic.

discord-irc --config CONFIG
29 Aug 15:43:52 - Attempting socket connection to IRC server
29 Aug 15:43:52 - Socket connection successful
29 Aug 15:43:52 - Sending irc NICK/USER
info: Connected to Discord
info: Connected to IRC
29 Aug 15:50:43 - MODE: USER sets mode: +i
29 Aug 15:50:43 - QUIT:  USER !~ USER @<IP_ADDRESS> Ping timeout: 264 seconds
29 Aug 15:50:43 - Unhandled message: { command: 'ERROR',
rawCommand: 'ERROR',
commandType: 'normal',
args: [ 'Closing Link: <IP_ADDRESS> (Ping timeout: 264 seconds)' ] }
29 Aug 15:50:43 - Connection got "end" event
29 Aug 15:50:43 - Connection got "close" event
29 Aug 15:50:43 - Disconnected: reconnecting
29 Aug 15:50:43 - Waiting 2000ms before retrying
29 Aug 15:50:45 - Attempting socket connection to IRC server
29 Aug 15:50:45 - Socket connection successful
29 Aug 15:50:45 - Sending irc NICK/USER
info: Connected to Discord
info: Connected to IRC
29 Aug 15:50:53 - MODE:  USER sets mode: +i
29 Aug 15:51:00 - Unhandled message: { prefix: 'orwell.freenode.net',
server: 'orwell.freenode.net',
command: '378',
rawCommand: '378',
commandType: 'normal',
args:
[ ' USER ',
' USER ',
'is connecting from<EMAIL_ADDRESS><IP_ADDRESS>' ] }
29 Aug 15:51:01 - GOT NOTICE from "ChanServ": "MESSAGE"

Sldo, Bot leaves server after a period of time (1-6 hours). Also random.

you should edit out your IP, if that's your real IP, first of all...
secondly, i am getting the unhandled error as well, but my bot connects.
thirdly, the random disconnects are related to discord.js. see #288.

It looks like you're disconnecting from both Discord and IRC, and I haven't encountered this myself. Are you sure you have a stable internet connection, and it's not just crashing every so often?


Pjax and yii-redactor, the redactor not gone after pjax navigation

What steps will reproduce the problem?
Basically i enclosed the redactor with pjax, and the redactor inside the form

    Pjax::begin([
        'id' => 'edit_comment_' . $comment_id,
        'timeout' => false,
        'enablePushState' => false,
    ]);
?>

<div id="comment_shown_part_<?= $comment_id ?>">
    <?= $comment ?>

<?php $form = ActiveForm::begin(['action' => ['thread/edit-comment'], 'method' => 'post', 'options' => [
    'id' => 'edit_comment_form_' . $comment_id,
    'data-pjax' => '#edit_comment_' . $comment_id]])?>

        <div id="comment_edit_part_<?= $comment_id ?>" style="display: none" >

                <?= \yii\redactor\widgets\Redactor::widget([
                    'name' => 'comment',
                    'value' => $comment,
                    'clientOptions' => [
                        'imageUpload' => \yii\helpers\Url::to(['/redactor/upload/image']),
                    ],
                ]) ?>

                <?= $form->field($edit_comment_form, 'parent_id')->hiddenInput(['value' => $comment_id ])->label(false) ?>

                    <?= Html::submitButton('Update', ['class' => 'btn btn-primary']) ?>
                    <?= Html::button('Cancel', ['class' => 'btn btn-danger cancel_edit_comment', 'data-service' => $comment_id]) ?>

<?php ActiveForm::end(); ?>

<?php Pjax::end(); ?>

after that i click update (the submit buttton), it becomes like this

the submit button (update) will go to this controller
inside the controller

$edit_comment_form = new EditCommentForm();
		$edit_comment_form->load(Yii::$app->request->post());

		if(isset($_POST['comment']) ){
			$edit_comment_form->comment = $_POST['comment'];
			$edit_comment_form->update();
			return $this->renderPartial('_view_edit_comment_part', ['comment_id' => $edit_comment_form->parent_id,
															'comment' => $edit_comment_form->comment,
															'edit_comment_form'=> new EditCommentForm()]);

		}

What is the expected result?
I want to remove the redactor, since it should not be there
What do you get instead?
The redactor is still there
Additional info
Additionally, the redactor is actually hidden at first, when users click edit, the redactor will be triggered (along with the form and update button)
after users click update, the controller will render the view which has redactor hidden (by css, display:none). So i expector the redactor to be hidden

Q
A

Yii version     2.0.7

PHP version 5.512

Operating system  Windoes

I would like to apologise for spamming here ,
i just solved the problem by removing the id in the form


add link to tutorial demonstrating Wav2Vec2 finetuning

What does this PR do?
adds a link to tutorial demonstrating Wav2Vec2 finetuning
Closes #2285
Checklist

[x] This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
checks if that's the case).
[x] This change is discussed in a Github issue/
discussion (please add a
link).
[ ] The documentation and docstrings adhere to the
documentation guidelines.
[ ] This change includes necessary high-coverage tests.
(No quality testing = no merge!)

Codecov Report

Merging #2288 (4e41b2f) into main (e57133c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2288   +/-   ##
=======================================
  Coverage   75.80%   75.80%
=======================================
  Files          61       61
  Lines        5378     5378
=======================================
  Hits         4077     4077
  Misses       1301     1301

Continue to review full report at Codecov.

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

Hi @thevasudevgupta, apologies for the delay, but we moved our examples to another place so this PR is outdated. I've added your example in #2718, and I'll close this PR. Thanks for your contributon!


⚠ Jenkins Plugins Artifactory has degraded performance

In b2e03c5, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1293 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 78723e6 after 12 minutes.


Merge pull request #1 from alanrenouf/master

Catching up

Reject this please.  Sorry, pulling the wrong way ..


Allow setting dnsPolicy for crossplane deployment pod in universal-crossplane chart

What happened?
Same issue as https://github.com/crossplane/crossplane/issues/5500#issue-2202871109  . Filing this issue to sync the upstream chart changes to here and have a uxp chart v1.15.1-up.2 patch release.
How can we reproduce it?

What environment did it happen in?
UXP version: v1.15.1-up.1

cc @phisco  Does the upstream crossplane chart changes gets synced to uxp chart during the next patch release process or does it have to be explicity synced through PR ? Thanks.

Yes, @ravilr, they do 😉

this is included in https://github.com/upbound/universal-crossplane/releases/tag/v1.15.2-up.1


ISPN-12067 HTTP/2 framing error for invalid requests

https://issues.redhat.com/browse/ISPN-12067

CI is fine @pruivo

integrated! thanks @gustavonalle !


Support Jakarta namespace

Is your feature request related to a problem? Please describe.
Currently, microshed testing uses the Java EE namespace (javax.) for RESTful web services and Web Token integration.
Describe the solution you'd like
We should release a separate version of microshed for the Jakarta EE namespace (jakarta.)

@appiepollo14 I am likely going to be taking on this issue but I wanted to ask for your input.
For your use case are your applications still using javax or jakarta package names?
I ask because we have two options here:

Support both javax and jakarta
Support only jakarta

You were just a few seconds before me. I wanted to mention I'm willing to help out on this one. If you ask me personally, jakarta only would be fine. In practice, I think support for both is needed. Isn't it as simple as create a seperate module for jakarta? Or how are you thinking about setting this up?

I'd probably have to do some more research.  Eclipse has a transformer that can automatically transform java byte code from javax to jakarta.  If we could use that then we don't have to duplicate anything and we can just transform and upload both.  Which sounds simple but needs some investigation.

@KyleAure this is only for the jaxrs part of the core right?

Correct, but we could add in other javaee/jakartaee features later so I want to come up with a general solution.

@KyleAure than just go for Jakarta only IMHO. Great work on this pr so far! The payara base image needs updating to be able to run j11.


Fixed Miami University file.

If this is wrong again I am done and I am sending that email.

@PARADOXzss07 I am sorry, the domain miamioh.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Pack at https://education.github.com/discount_requests/application. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

 I guess I’ll just pay for it. Thanks for helping.


1.388.0 can't start (panic).

Updaring casdoor to 1.388.0, the enforcer panicked.
[xorm] [warn]  2023/08/14 01:48:07.140032 Table organization has column phone_prefix but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.140078 Table organization has column password_complex_options but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.142106 Table user has column country_codes but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.142116 Table user has column multi_factor_auths but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.143326 Table group has column id but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.143335 Table group has column parent_group_id but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.147884 Table adapter has column model but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.147893 Table adapter has column file but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.150370 Table provider has column custom_scope but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.158936 Table syncer has column table_primary_key but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.162870 Table payment has column organization but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.163998 Table ldap has column admin but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.164008 Table ldap has column passwd but struct has not related field
[xorm] [warn]  2023/08/14 01:48:07.171099 Table pricing has column has_trial but struct has not related field
panic: the enforcer: built-in/user-enforcer-built-in is not found

goroutine 1 [running]:
github.com/casdoor/casdoor/object.InitUserManager()
        /srv/casdoor/casdoor/object/user.go:39 +0x92
main.main()
        /srv/casdoor/casdoor/main.go:52 +0x10c

@mzch delete three DB tables: model, adapter and enforcer. Restart Casdoor

Fixed. Thanks! ;)


[fix][broker] Fix the broker registering might be blocked for long time

Fixes https://github.com/apache/pulsar/issues/23365
Motivation
https://github.com/apache/pulsar/pull/23359 would register the broker again if the registered metadata node was somehow deleted. However, the listeners in BrokerRegistryImpl are triggered in the load manager thread (via Pulsar#Service#loadManagerExecutor).
https://github.com/apache/pulsar/blob/5e832a1cc1441eaf8d64fe72c1a2af8829030d3d/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/extensions/BrokerRegistryImpl.java#L229-L233
However, this thread executes many blocking tasks, including monitor(), playLeader() and playFollower() of ExtensibleLoadManagerImpl. If there were no available brokers, the blocking tasks might never complete until timeout exceeds. Here is an example:
2024-09-29T15:24:15,675 - WARN  - [pulsar-io-24-2:ConnectionHandler] - [non-persistent://pulsar/system/loadbalancer-broker-load-data] [reader-40952cbf05] Could not get connection to broker: org.apache.pulsar.client.api.PulsarClientException$BrokerMetadataException: {"errorMsg":"No available broker found.","reqId":175112655758658573, "remote":"localhost/<IP_ADDRESS>:56266", "local":"/<IP_ADDRESS>:56281"} -- Will try again in 3.146 s
2024-09-29T15:24:17,686 - WARN  - [pulsar-load-manager-22-1:ExtensibleLoadManagerImpl] - The broker:localhost:56270 failed to set the role. Retrying 1 th ...
org.apache.pulsar.broker.loadbalance.extensions.store.LoadDataStoreException: java.util.concurrent.TimeoutException
	at org.apache.pulsar.broker.loadbalance.extensions.store.TableViewLoadDataStoreImpl.startTableView(TableViewLoadDataStoreImpl.java:179) ~[classes/:?]
	at org.apache.pulsar.broker.loadbalance.extensions.store.TableViewLoadDataStoreImpl.start(TableViewLoadDataStoreImpl.java:157) ~[classes/:?]
	at org.apache.pulsar.broker.loadbalance.extensions.store.TableViewLoadDataStoreImpl.init(TableViewLoadDataStoreImpl.java:141) ~[classes/:?]
	at org.apache.pulsar.broker.loadbalance.extensions.ExtensibleLoadManagerImpl.playLeader(ExtensibleLoadManagerImpl.java:849) ~[classes/:?]

In this case, registerAsync would have to wait until these methods are done, while they could have been recovered by a registerAsync call.
While the direct cause of failed BrokerRegistryIntegrationTest is that TableViewImpl#flush would not complete if there is no message in the topic. See https://github.com/apache/pulsar/issues/23365#issuecomment-2381090760
Modifications
Two fixes:

Fix the TableViewImpl#flush by filtering  the empty topics via checking if the last message id's entry id is non-negative
Call registerAsync directly in the metadata store callback thread instead of the load manager thread. It's an asynchronous metadata store operation so it won't block the thread.

Tests enhancements:

Enable BrokerRegistryIntegrationTest again to verify fix 1 works
Add BrokerRegistryMetadataStoreIntegrationTest to verify fix 2 works
Check the broker close time to verify fix 1 is still required even if we have fix 2

Documentation

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

Matching PR in forked repository
PR in forked repository:

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.56%. Comparing base (bbc6224) to head (26edb49).
Report is 613 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23371      +/-   ##
============================================
+ Coverage     73.57%   74.56%   +0.98%
- Complexity    32624    34485    +1861
============================================
  Files          1877     1934      +57
  Lines        139502   145137    +5635
  Branches      15299    15870     +571
============================================
+ Hits         102638   108218    +5580
+ Misses        28908    28628     -280
- Partials       7956     8291     +335

Flag
Coverage Δ

inttests
27.80% <75.00%> (+3.22%)
:arrow_up:

systests
24.53% <0.00%> (+0.21%)
:arrow_up:

unittests
73.90% <100.00%> (+1.05%)
:arrow_up:

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

Files with missing lines
Coverage Δ

...ker/loadbalance/extensions/BrokerRegistryImpl.java
82.88% <100.00%> (-1.88%)
:arrow_down:

...a/org/apache/pulsar/client/impl/TableViewImpl.java
86.59% <100.00%> (+1.28%)
:arrow_up:

... and 602 files with indirect coverage changes


Calendar

The helpers for the calendar

Overall the code for day_bookings seems rather verbose, un-understandable and not very ruby-ish. Try to make it better.


lerna useWorkspaces option was removed in v7 (1.0.0-canary.7)

Describe the bug
lerna removed the useWorkspaces option in v7.
To Reproduce
Steps to reproduce the behavior:

Run command 'yarn run bootstrap'
See error

$ rm -rf ./node_modules; npx lerna clean --yes && yarn bootstrap
lerna notice cli v7.4.2
lerna ERR! ECONFIGWORKSPACES The "useWorkspaces" option has been removed. By default lerna will resolve your packages using your package manager's workspaces configuration. Alternatively, you can manually provide a list of package globs to be used instead via the "packages" option in lerna.json.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

I had the same error, below is how my lerna.json file looked:
  "packages": ["packages/*"],
  "npmClient": "yarn",
  "version": "0.0.1",
  "useWorkspaces": true,
  "nohoist": ["parcel-bundler"]
}

I removed "useWorkspaces": true, and it worked

@GabrieleKaceviciute I believe this was also resolved by @pavjacko changes to the bootstrap script

Closing as issue was resolved, feel free to reopen if needed.


Playwright Custom Reporter not running when integrating with SauceLabs

I have a Custom Reporter that I'm running for my Playwright tests.
It is defined in the Playwright configuration file and I'm feeding it to the .sauce/config.yml for SauceLabs to use.
When running the Playwright tests on my local without SauceLabs, the reporter is properly fired off and running as expected.
However, when I run the Playwright tests against SauceLabs using the saucectl CLI and the saucectl-run-action GitHub Action, it is not working and seems to be using another reporter.
I see that there is a reporter object in the Playwright SauceLabs Configuration YAML:
https://docs.saucelabs.com/web-apps/automated-testing/playwright/yaml/#reporters
Could this be what is overriding my Custom Reporter?
My Custom Reporter ./testrail/reporter/testrail.reporter.ts looks something like:
import { Reporter } from '@playwright/test/types/testReporter';

class TestRailReporter implements Reporter {

  onBegin(config, suite): void {
    // logic
  }

  onTestBegin(test, result): void {
    // logic
  }

  onTestEnd(test, result): void {
    // logic
  }

  onEnd(result): void | Promise<void> {
    // logic
  }
}

export default TestRailReporter;

In my playwright.sauce.config.ts, I have the following reporter being used:
const config: PlaywrightTestConfig = {
  ...
  reporter: './testrail/reporter/testrail.reporter.ts',
  ...
}

In my .sauce/config.yml, I have the following:
...
playwright:
  version: package.json
  configFile: playwright.sauce.config.ts
...

Any feedback would be greatly appreciated as I really need this Custom Reporter to be executed while running my Playwright tests in SauceLabs.
Thanks.

Indeed, we set the reporters to list, junit and our own internal reporter, thereby overriding your reporter settings. As it stands, I don't see a way to work around this limitation. I'll create a feature request ticket internally 👍

Thank you! Looking forward to the resolution!

Actually, I did end up finding a viable workaround!
Try this:
const config: PlaywrightTestConfig = {
  reporter: [['./testrail/reporter/testrail.reporter.ts']],
}

The Custom Reporter is working as expected when the Playwright tests run in SauceLabs using your workaround.
I could take a guess on why this is working; however, it would be just a guess.
How come providing an Array<Array<string>> for the reporter in the configuration works over providing a string for the reporter?

There's a logic in place that concats arrays together, but if the value is a primitive, and we have an explicit setting for it, ours takes precedence (this is so we can override certain user settings so that playwright functions or integrates better with our cloud).

Ah ok, makes sense.
Maybe adding a warning/info message about that here: https://docs.saucelabs.com/web-apps/automated-testing/playwright/yaml/#reporters
Thanks!


volume of cube

Why:
fixes #66
What's being changed:
Added the c++ program to find the volume of a cube.

@harshraj8843 @PrajwalBorkar   please check.


[on Mobile] Backspace removes visual text, but leaves it in logical model

Imagine you think you delete text (with backspace), but it still stays in your code (but is invisible in editor). And when you run program (with javascript) (you get weird errors).
I noticed this on Samsungs,   while using Programmers Keyboard app for layout  and with external keyboard attached.
But Delete button works OK. And after delete, the previously invisible text is updated to be visible..

Sounds like another IME bug (Android keyboards get a lot of control over the events browsers fire during typing, and tend to do all kinds of weird stuff). I will try to make some time soon to overhaul IME handling to be less reliant on those events, since the information they provide is hard to interpret properly.

This is a issue that's difficult if not impossible to solve with the fundamental approach currently taken by CodeMirror.
We are working on a rewrite (CodeMirror 6) that will address this issue, and we are currently raising money for this work: See the announcement for more information about the rewrite and a demo.
Note that CodeMirror 6 is by no means stable or usable in production, yet. It is highly unlikely that we pick up this issue for CodeMirror 5, though.


Use a messaging queue to transport cloud-info-provider output

We should use a messaging queue to transport cloud-info-provider output.
Either a component should be added to this project or a dedicated project should be created.

I would go with a different publisher, probably as a plugin to be loaded whenever #25 is done...

For the time being, this will be handled by a wrapper and gradually moved to the cloud-info-provider.

Is there a pattern you've agreed upon for the topic naming  is it something like fedcloud_site_name or something similar ?

The agreed naming so far is:
SITE_ENDPOINT
Adding @mhaggel to ensure this is still ok

Yes i confirm that this is ok.
Another way to go, less friendly though, could be:
SITE_ENDPOINT

Ok we can close this issue with the decision that for topics we will use SITE_ENDPOINT

Just nu i realized that the github editor does render my previous comment correctly.
I wrote:
Another way to go, less friendly though, could be:
SITE_ < identifier of the site in GOCDB > _ENDPOINT_ < identifier of the endpoint in GOCDB >
NOT just SITE_ENDPOINT

This is partially fixed in #178 . Only missing bit would be to automatically generate the topic for publication, which can be done easily with code already available but it would require some refactoring to get the information to the right place.

This is done now as of #178, improvements may come later