Check support with latest NILMTK and Pandas

NILMTK seems to have moved to Pandas 0.22. Compatibility should be tested to make sure that everything runs smoothly.

Encountered the following issue:
https://github.com/nilmtk/nilmtk/issues/681
and submitted a pull request
https://github.com/nilmtk/nilmtk/pull/683
After the remedy added in the pull request, everything works fine. Will close this issue once the pull request has been merged into the NILMTK master.


.github: bump k8s version from v1.28.0 -> v1.28.2.

K8s v1.28.0 causes the following regression: https://github.com/cilium/cilium/issues/27982.
Most noticeably, this has been causing k8s conformance test failures.

/test

/test

/test

/test

/test

The reason why we mirrored kindest-node on quay.io was to circumvent the docker hub rate limitting. Now that we're using DockerHub again, I see rate limit failiures in CI again:
 ERROR: failed to create cluster: failed to pull image "kindest/node:v1.19.16@sha256:476cb3269232888437b61deca013832fee41f9f074f9bed79f57e4280f7c48b7": command "docker pull kindest/node:v1.19.16@sha256:476cb3269232888437b61deca013832fee41f9f074f9bed79f57e4280f7c48b7" failed with error: exit status 1
Command Output: Error response from daemon: error parsing HTTP 429 response body: invalid character 'T' looking for beginning of value: "Too Many Requests (HAP429).\n"

https://github.com/cilium/cilium/actions/runs/6654851478/job/18083961901


Can't enable debug when using install script

When using the install script passing --debug before server breaks as the script prepends server anyways.
Perhaps an envvar would be useful to set debug?

I didn't know there was even a --debug flag

v1.17.2+k3s1
I can confirm the K3S_DEBUG=true env var is working now.


feat: improved documentation

Added:

CONTRIBUTING.md
PULL_REQUEST_TEMPLATE.md

To improve project vitality.

Hey, I've made some small changes. Please review them and let me know if you have any comments.
If not, I'll merge this PR.
Thanks!

Hey, I've made some small changes. Please review them and let me know if you have any comments. If not, I'll merge this PR.
Thanks!

I am okay with it, you can go ahead and merged it. Thanks!


Add build options for specifying template and skipping non qualified codepoints

I forked the js library for our purposes a while ago, but still use this repo for building and maintaining the data lists. This PR adds two command line options to the build.php script:

--tpl=<tpl> lets you provide a custom emoji.js output template
--skip-nq skips non qualified emoji bytes. This avoids matching plain characters like © as an emoji.

Coverage remained the same at 90.146% when pulling 08e2789d4c519de929a789c1bd18ed240c4ccdbb on jwheare:build-opts into 9f5cfd92111088c54e48a60d424dc3d69473306d on iamcal:master.

continued in #124


ReleaseNotesBuilder does not resolve number of commits since prev milestone

The issue/bug have been discussed here
The problem is here, where number of commits are resolved

Maybe write some specs to define behaviour when prev milestone is a tag
Scenarios:

Releases have been tagged until v0.2.1. Defining v0.3.0 milestone for vNext and using GRM for the first time.
New repo, therefore first milestone/release/tag ever. Who says master is base here? But GIT does not have a TAIL concept pointing at first commit (like HEAD pointing at checked out code), as far as i know. But something like

git rev-list HEAD | tail -n 1

could be base here.
I think GRM should be able to resolve number of commits in both scenarios above.

About release notes for cake, my guess is that:

Milestones use v prefix (v0.16.0) in cake repo
GRM milestone version parsing does not support v prefix (it should) and returns 0.0. (It does not support prerelease-tokens in milestone/releases/tags etiher, BTW)
GetPrevMilestone will then return null (no milestone versioned less than 0.0)
GetNumberOfCommitsBetween(prevMilestone: null, currentMilestone) will compare master...nextTagName (does not make sense at all BTW even if there really was no previous release/milestone in the first place), and returns zero commits
Cake release notes will not report number of commits now or in the future

If I fork your project, would it be okay to release forked GRM.exe on my own MyGet feed? Right now I think your build script is preventing pushes at all from forks, and off course I have to overwrite feed details as well. But I have to figure out if your template pattern like build setup where only setup.cake is part of GRM repo allows that in the first place.
I will off course be happy to send PR's.
How do other forks do 'integration' pushes/releases before sending PR's?

nevermind the last question about pushes to own private feed. I can off course do that manually, and not rely on appveyor...

@maxild said...
Right now I think your build script is preventing pushes at all from forks, and off course I have to overwrite feed details as well.

Yes, currently only "official" packages are pushed to MyGet, so PR's which haven't been vetted, are not pushed.  However, the AppVeyor build will execute, and the generated package should be available as an artifact on AppVeyor, so you should be able to grab it from there.

This hasn't been resolved yet? I just used GRM for the first time for the first release of one of my repos and it only produced the text about closing a number of issues, it didn't list the issues.

@mrahhal sorry to hear that you are having issues. Based on what you have described, it sounds like you are facing a different error than what is described in this issue. Can you confirm?

Actually yes, that's right. I figured the problem must be different when I made a second release. The release is showing the number of commits as well as the number of issues closed, but strangely I can't get the issues themselves to be included.

@mrahhal which repository is this on?

You can see this here. Note that I edited the 0.1.0 release myself. I left the 0.1.1 release as it was produced though. (I probably should have opened another issue, I've read a bit of the source and I first thought this was the same problem)

@mrahhal I notice that the milestone is already closed.  Can you try opening the milestone, and running GRM again?  Also, can you show the full output from the log file/console window?

I believe I am right in saying that this issue is resolved, and should ship in the 0.10.0 release of GRM.  If this isn't the case, then please feel free to re-open this issue.


adds .Net 6 sdk

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

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

When trying to install locally I'm getting a "invalid argument exception". Not sure where this is coming from. I'd appreciate if somebody could double check.
runing the installer with /install and /passive works
Microsoft Reviewers: Open in CodeFlow

Thanks for the suggestions. Yep I'm planning to do so but on my phone at the moment. Give me an hour.

done!


Inner class annotated with @Parcel causes org.parceler.transfuse.util.TransfuseRuntimeException

Hi John,
Related to https://github.com/johncarl81/parceler/issues/75.
We've been seeing a few crashes caused by org.parceler.ParcelerRuntimeException: Unable to create ParcelableFactory for .... In an attempt to make sure that these don't pop up, I've added all @Parcel annotated classes to the @ParcelClasses array. One of our @Parcel classes is an inner class of an @Fragment class and throws the following at code gen time:
Exception in thread "pool-743-thread-1" org.parceler.transfuse.util.TransfuseRuntimeException: Unable to perform code generation
Which is caused by:
Caused by: javax.annotation.processing.FilerException: Attempt to recreate a file for type com.example.MyClass$MyInnerClass$$Parcelable.
It appears that removing the inner class from the @ParcelClasses fixes the issue, but I'm wondering if you have any guidance/best practices for which @Parcel classes to add to the @ParcelClasses array.

That is strange.  Is the inner class annotated with @Parcel?

Yea, the inner class is annotated with @Parcel as follows:
@Parcel
  public static class IdentityDialogState {
    public Long agentId;
    public String agentName;
    public String agentEmail;

    @ParcelConstructor
    public IdentityDialogState(Long agentId, String agentName, String agentEmail) {
      this.agentId = agentId;
      this.agentName = agentName;
      this.agentEmail = agentEmail;
    }
  }

If you have the inner class (IdentifyDialogState) annotated with @Parcel and it is also in the @ParcelClasses list then it will try to create the Parcelable twice, which is my guess why you're seeing the code generation exception.  Could you remove @Parcel  from IdenfityDialogState?

Ok, great! I wasn't aware that I should use only @Parcel or @ParcelClasses, but not both. I think that solves my issue.

Really though, you should be able to either annotate the class with @Parcel or reference it with @ParcelClass.  Any idea why the Parcelable isn't being generated when it is annotated with @Parcel?  Is this class in a library?

My initial question probably wasn't clear in this regard. It wasn't IdentityDialogState that caused the org.parceler.ParcelerRuntimeException: Unable to create ParcelableFactory for ...; it was a different class and you're right to think this class is in a library; it is! In fact, nearly all of our @Parcel classes are library classes, which (I think) makes ensuring that they are generated a little difficult. That's why I've opted for the 'kitchen sink' approach adding them all to @ParcelClasses.
Something still puzzles me, though. Even after I've added all @Parcel library classes to @ParcelClasses (save IdentityDialogState, which was the only non-library @Parcel class) I don't see the error.
Initially, we had assumed that if we have 2 classes - ParcelClassA and ParcelClassB - and that ParcelClassA has a member of type ParcelClassB, the appropriate ParcelClassB parceler code would be generated at the time that the ParcelClassA parceler code is generated. If what we've assumed is true, why don't we see the same Caused by: javax.annotation.processing.FilerException: Attempt to recreate a file for type com.example.MyClass$MyInnerClass$$Parcelable for ParcelClassB when it is, in fact, in the @ParcelClasses array as well as annotated with @Parcel?


Alpine: standard_init_linux.go:211: exec user process caused "no such file or directory"

When i run container i got an error standard_init_linux.go:211: exec user process caused "no such file or directory"
Dockerfile:
FROM wrouesnel/postgres_exporter

FROM alpine:3.9.5

RUN adduser --disabled-password -u 20001 postgres_exporter

COPY --from=0 /postgres_exporter /postgres_exporter

COPY dockerfiles/postgres_exporter/* /etc/postgres_exporter/
COPY deploy/k8s/load-env.sh /usr/local/bin/load-env
RUN chmod +x /usr/local/bin/load-env

USER postgres_exporter

EXPOSE 9187

ENTRYPOINT [ "/postgres_exporter" ]

Same problem solved here by adding ENV CGO_ENABLED=0 before the go build line
https://www.reddit.com/r/docker/comments/bld85r/help_exec_user_process_caused_no_such_file_or/
Can we do the same for published image on Dockerhub or it can break something?

But as i can see it's done before
https://github.com/wrouesnel/postgres_exporter/blob/master/magefile.go#L659

Fixed by https://medium.com/ale-world/standard-init-linux-go-187-exec-user-process-caused-no-such-file-or-directory-e8e45b20f2cd


Plugin not responding to data update

Lightweight Charts™ Version: 4.1.1
Steps/code to reproduce:
Have not tried this myself on your example, but you should be able to reproduce the issue with the change described here.
In the session highlight example, move candleSeries.setData(data); to the end of the file, after candleSeries.attachPrimitive(sessionHighlighting);.
// ...
const candleSeries = chart.addCandlestickSeries();
const data = generateAlternativeCandleData();
// make sure this is commented out or removed to be able to reproduce the problem
// candleSeries.setData(data);

function getDate(time: Time): Date {
// ...
// ...
const sessionHighlighting = new SessionHighlighting(sessionHighlighter);
candleSeries.attachPrimitive(sessionHighlighting);
candleSeries.setData(data);

Actual behavior:
Session highlighting does not work in the above case - when attachPrimitive is called before setData.
Expected behavior:
It should work :). Plugin should respond to data update.
Potential solution:
The issue seems to be caused by this line:
https://github.com/tradingview/lightweight-charts/blob/61d45db057c36c1b0d95e9a0d909fb5a60e5a19f/plugin-examples/src/plugins/plugin-base.ts#L25C5-L25C5
_fireDataUpdated is using this inside the function body:
	private _fireDataUpdated(scope: DataChangedScope) {
		if (this.dataUpdated) {
			this.dataUpdated(scope);
		}
	}

This this is not bound to a SessionHighlighting object (which inherits from PluginBase), but to something else. That other thing does not have this.dataUpdated, and it will always be undefined, whether SessionHiglighting object defines dataUpdated method or not.
Line 25 in plugin-base.ts should probably be changed:
// old
this._series.subscribeDataChanged(this._fireDataUpdated);

// fixed (option 1)
this._series.subscribeDataChanged((scope) => this._fireDataUpdated(scope));

// fixed(option 2)
this._series.subscribeDataChanged(this._fireDataUpdated.bind(this));

After the change, this inside _fireDataUpdated should be properly bound to the SessionHighlighting instance.

I have not received any funds from any address, please transfer the money
to the bank account number...                     BE60 9501 0866 3270 Mr-
PRZEMYSLAW KARCZEWSKI I am waiting for a quick transfer ((((((FINALLY))))
to my bank account and please hurry because I have submitted papers for
fraud regarding cryptocurrencies and their withdrawals... If you still have
any problems in withdrawing this huge amount, please contact me, then you
will be in contact with my team of lawyers from TESTACHA.be. I am waiting
for a positive resolution.... ...
pon., 8 sty 2024 o 11:44 Romain Francois @.***>
napisał(a):

Closed #1491
https://github.com/tradingview/lightweight-charts/issues/1491 as
completed via #1492
https://github.com/tradingview/lightweight-charts/pull/1492.
—
Reply to this email directly, view it on GitHub
https://github.com/tradingview/lightweight-charts/issues/1491#event-11415951768,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6LVX2O3QG5ECDHKQ2TUJ63YNPEYHAVCNFSM6AAAAABBP5UP7CVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGQYTKOJVGE3TMOA
.
You are receiving this because you are subscribed to this thread.Message
ID: <tradingview/lightweight-charts/issue/1491/issue_event/11415951768@
github.com>

pon., 8 sty 2024 o 16:26 Przemyslaw Karczewski @.***>
napisał(a):

I have not received any funds from any address, please transfer the money
to the bank account number...                     BE60 9501 0866 3270 Mr-
PRZEMYSLAW KARCZEWSKI I am waiting for a quick transfer ((((((FINALLY))))
to my bank account and please hurry because I have submitted papers for
fraud regarding cryptocurrencies and their withdrawals... If you still have
any problems in withdrawing this huge amount, please contact me, then you
will be in contact with my team of lawyers from TESTACHA.be. I am waiting
for a positive resolution.... ...
pon., 8 sty 2024 o 11:44 Romain Francois @.***>
napisał(a):

Closed #1491
https://github.com/tradingview/lightweight-charts/issues/1491 as
completed via #1492
https://github.com/tradingview/lightweight-charts/pull/1492.
—
Reply to this email directly, view it on GitHub
https://github.com/tradingview/lightweight-charts/issues/1491#event-11415951768,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6LVX2O3QG5ECDHKQ2TUJ63YNPEYHAVCNFSM6AAAAABBP5UP7CVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGQYTKOJVGE3TMOA
.
You are receiving this because you are subscribed to this thread.Message
ID: <tradingview/lightweight-charts/issue/1491/issue_event/11415951768@
github.com>


how to remove or hide the titlebar?

Do you mean navbar? If you do, then just use the hideNavBar prop on the Scene component for which you would like to hide it. Also the API readme has answered most of my rnrf questions.


MQTT node showing wrong status

What are the steps to reproduce?

MQTT broker (ecplise mosquitto) with authentication enabled running as a container which doesnt allow clients to connect to it if they are trying to connect without username and password.
MQTT nodes in nodered configured without username and password.

What happens?
When the nodes are deployed, MQTT nodes shows status connected.
What do you expect to happen?
MQTT nodes should show status disconnected.
Please tell us about your environment:

[ x] Node-RED version: 1.0.6
[x ] Node.js version: v10.20.1
[x ] npm version: 6.14.4
[ x] Platform/OS: Container
[ x] Browser: Chrome

What does the log show - It should also log the connection as well.

@GowriAradhya can you provide some more information as we cannot reproduce this.
You say the node shows 'connected' status when deployed. Does that mean the nodes were previously connected, you then change the authentication details and deploy, but they appear to stay connected?
Or are these brand new nodes that are being deployed for the very first time?
What does the node-red log say?

This is tested by deploying MQTT node for the first time after launching Nodered.
CASE 1-Eclipse Mosquitto and nodered
1.Start both Eclipse Mosquitto and Nodered in docker compose
2.In Nodered MQTT node, Configure host address as "mosquitto_broker"/docker network without username and password.
3.In Nodered MQTT node shows connected and in Nodered logs it shows connected to mosquitto broker.
4.In Eclipse Mosquitto logs it throws error for null username and password.

CASE 2-Eclipse Mosquitto and nodejs MQTT client
1.Start both Eclipse Mosquitto and nodejs MQTT client in docker compose.
2.In nodejs MQTT client, Configure host address as "mosquitto_broker"/docker network without username and password.
3.In nodejs MQTT client shows connection refused not authorized.
4.In Eclipse Mosquitto logs it throws error for null username and password.

Hi @knolleary
Do you see any potential issue from the above logs? Do you any further information?

The logs tell me the broker expects the client to be providing a username/password for its ack check, but the client hasn't provided any.
But they don't answer the questions I asked.

Hi @knolleary
MQTT node is brand new node deployed for the very first time after launching Nodered.
1.Start both Eclipse Mosquitto and Nodered in docker compose
2.Eclipse Mosquitto has been configured to check username and password on connect and for every publish and subscribe it checks for ack for topic permission.
3.After launching nodered import the flow.
flows.txt
4.In Nodered logs it shows connected to Eclipse Mosquitto and node shows 'connected' status when deployed

When it shows connected - does data actually pass through ? (IE is it really connected ?)
Has the allow_anonymous option in mosquitto been set to false ? (and restarted)
are you able to share the compose file to help make this easier to replicate ?

Data is not passing only the node connection status will be green.
Yes , allow_anonymous option in mosquitto is set to false

Hi @dceejay
Please find the compose file.
docker-compose.txt

Hi @dceejay Any Update on this issue??
Are u able to reproduce ??

sorry - been on vacation. lots to catch up on.


[OSX] Trackpad gestures - Magnify (Pitch to zoom), Rotate and Swipe

What does the pull request do?
Adds support for three gestures:
Magnify - double Delta
Rotate - double Delta
Swipe - Vector Delta - based on vector you can get a direction
I've added this to PointerDevice because trackpad is a pointer device. I've thought about creation of TrackpadDevice - but this is a bad idea, because gestures above are pointer position dependant.
How was the solution implemented (if it's not obvious)?
I've subscribed to three native methods in native osx lib, added 3 event types, and integrated them into Interactive.
Fixed issues
Fixed #6545
WIP
Feature is done but I need to test it on a real app. When this will be done I will fix found issues and remove a WIP flag.
Win32 and Linux support?
AFAIK it's almost impossible to implement. Also this features works great only on macOS AFAIK. So only macOS users are need it(it's essential for any macOS trackpad user).

Integrated to our app. Works fine. @danwalmsley please review this :)

It shouldn't be in InputElement. We have Gestures class with gestures-related events. It's a good place for these events.
On other platforms it should be done using simple pointer events, otherwise this feature is 66.6% useless.

@maxkatz6 This is not a touch screen gestures, this is trackpad gestures. You cannot track them using simple pointer events. Also this is an impossible challenge to me to implement Windows Precision Driver support, Apple Bootcamp driver support and Synaptics driver support on Windows and also any linux drivers support. This is a huge task compared to macOS implementation. Windows and Linux trackpads are working too bad. Gestures are not precise = not usable at all, so no one need them (me too). So I can provide only a macOS solution.
I think this is still a good thing to have a support of this on macOS only for now. Only on macOS this feature is essential. Maybe in the future someone will add support for other platforms if this is will be needed to someone (I think not in any time soon).
I did not moved this events to Gestures class because I thought it's only for touch screen gestures.
In my app it doesn't matter where it is. I can try move this events to Gestures class if anyone will agree with this.

So, if I will move this events to Gestures class - they will have these names:

TrackpadMagnifyGestureEvent
TrackpadRotateGestureEvent
TrackpadSwipeGestureEvent
Trackpad word is needed to specify that this events are not a touch screen events.
If everyone agree - I will start the implementation.

will this work with windows precision trackpads?

@danipen  Nope. I'm waiting for response from Avalonia team on how to make the API.

I've renamed event's like @kekekeks requested. Also moved events to Gestures class. I still have two questions above about the API.
Also - @danwalmsley already reviewed native side and said it looks good.
So if managed API is good - let's merge this PR :)

In Windows (10?), gestures are handled via the WM_GESTURE event (See https://docs.microsoft.com/en-us/windows/win32/wintouch/windows-touch-gestures-overview#interpreting-windows-touch-gestures). Seems like we can just handle this inside Avalonia.Win32.WindowImpl.AppWndProc

@pr8x how does WM_GESTURE interact with WM_POINTER and WM_TOUCH?

@kekekeks Apparently WM_TOUCH and WM_GESTURE are obsolete (and kinda broken) and WM_POINTER should be used instead. That is only supported on Windows 8 +, though. Apparently it's possible to do gesture detection with this "Interaction Context" API (see here), but it definitely seems harder to use than WM_GESTURE.
Good summary: https://lenos19.rssing.com/chan-10584515/all_p6.html
Sooo... actually not that easy :D


fix fix cases unstable when using profile to check is nereids execution

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
when commands running in doris using profile to check whether it's plan is generated by nereids, it would randomly failed because profile is not always generated on time.
solved by remove this check and check whether command was running by nereids by reviewer
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[x] No need to test or manual test. Explain why:

[x] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall

run buildall


Add emergency stop and verifier multiplixer contracts for version management

As we get ready to deploy RISC Zero verifiers to Ethewreum mainnet, we need a system to manage versions of the verifier contract and proof system. In particular, we need a way to respond to critical bugs, and we need a way to make upgraded versions of the proof system available to developers. Both mechanisms need to be opt-in so that applications can choose the solution that works best for their security model.
This PR implements a basic version management system, based around two new contracts:

RiscZeroVerifierEmergencyStop is a wrapper around a verifier contract that allows for the verifier to be shutdown if a critical bug is discovered.
RiscZeroVerifierMux is a wrapper around a list of verifiers that allows new verifiers to be added to support proof system upgrades.

These are intended to be used together with an authorized multisig, and a TimelockController on addition of new verifiers.
This PR is currently marked as draft, as the verifier identifiers are not yet fully implemented in the risc0 codebase.
Also addresses https://github.com/risc0/risc0-ethereum/issues/30 as part of a refactor to the IRiscZeroVerifier interface.

@capossele, this is ready for review when you have a chance


Rename inline protocol to line protocol

InfluxDB has a "line protocol" not "inline protocol" (typo error)

A new inspection was created.

The inspection completed: No new issues


fix: Fixup issues with async storage in DevWorkspaces

What does this PR do?
Fix issues with async-storage found while testing the v0.12.0 release:

DWO is checking the "one async-storage workspace at a time" restriction cluster-wide instead of per-namespace
Async storage sidecar is failing to sync data: https://github.com/devfile/devworkspace-operator/issues/754

What issues does this PR fix or reference?

Can only start one async-storage workspace at a time cluster wide :upside_down_face:
Closes https://github.com/devfile/devworkspace-operator/issues/754

Is it tested? How?
Create two async storage workspaces in two namespaces; workspaces should both start (though see https://github.com/devfile/devworkspace-operator/issues/754). Editing a file, stopping and restarting the workspace should persist changes.
PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)

[ ] v8-devworkspace-operator-e2e: DevWorkspace e2e test
[ ] v8-che-happy-path: Happy path for verification integration with Che

/test v8-devworkspace-operator-e2e

/test v8-devworkspace-operator-e2e


EE Engine installation pathway is rather confusing

Precondition - I have just purchased or begun a trial of Docker EE via Docker Store

Beginning from a license page in Docker Store, I'm given installation instructions

https://docs.docker.com/engine/installation/linux/ubuntu/

These take me to a page where Step 1 links to other documentation

https://docs.docker.com/engine/installation/
At this point Docker logically knew my exact use case -- trial/new customer installing EE for Ubuntu/Linux
This page gives information on:

CE vs EE capabilities
Supported platforms per edition
Release Schedule
Docker Cloud

This page does not directly given information on -- Installation Docker EE for "Linux".

Going into the next branch in the doc's tree gets me to this page -- https://docs.docker.com/engine/installation/linux/docker-ee/ubuntu/

Note, the first step here just sends me back to  the starting  point -- Store's licensing page :)

The Ask Has anyone taken a look at the effectiveness of this pathway? It would seem that every purchase from Docker Store allows Docker folks to know exactly what Edition and Environment is being targeted, so why send them on a discovery mission to find the correct document for Step 1?
File: datacenter/install/linux.md, CC @joaofnfernandes

Can we please get rid of all of the docs in store.docker.com and just point to the actual documentation?  I am attempting to help out a partner who is setting up Docker EE for the first time and they're very technical people but they are being led down a path that isn't getting them anywhere and leads to them guessing instead of following the docs.

Matt, I'll give you a little background.
First, I received an install doc (PDF attached) but the link "these instructions" doesn't work...
...so maybe everything I need to know is already somewhere, but I just don't know about it 😊
That said, I guessed my way to getting EE/UCP installed and go here...
...and get that error trying to upload the .lic file, contents look like...[secret bits truncated]
{
"authorization": "ewogICAicGF5bG9hZCI6ICJleUpsZ...",
"key_id": "U4gTeaPFwvwHJynIwUIT5IDw3IVgp7J824eatl0EUD87",
"private_key": "LH5qgSM8hrVKP..."
}
So at this point, I tried to tails the logs of  UCP Controller container...
$ docker logs ucp-controller
...
{"level":"warning","msg":"Unable to create a kube client: unable to copy kube certs from ucp-reconcile: unable to create directory: mkdir /var/lib/docker: permission denied","time":"2018-02-09T15:05:34Z"}
...
Is the only "warning" I see, no errors.  There are 28 containers running, none look interesting enough to grep logs as a fishing expedition.
Overall, the process seems a little half-baked, or I'm missing the real docs.  We'll need to do a lot of work in this area before we take this to AnF.  I think Docker should consider a little automation for this.  Hand building nodes, even the first one is not really "best practice" these days.  Also, seems a shame that the install process for EE does not mirror CE with just a license step added.  Additionally, the lack of docker-machine support makes me sad.  Now I have to bring in Ansible, Terraform, etc. and other complexities, rather just use the standard docker tooling that many of use are used to.  Anyway, MTC.  -s

/cc @jeffdm, @vsaraswat for visibility.

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


Nail down issues related to assembly loading

This is related to issue #658. The fix for that issue is minimal, as we do a very simple examination of the files next to task assemblies when looking for dependencies.
There are a couple potential issues with this approach that we may want to address.

Assembly identity issues.
Say we have the following four files on disk:

Alpha\AlphaTasks.dll
Alpha\Utility.dll
Beta\BetaTasks.dll
Beta\Utility.dll

Assume the two copies of Utility.dll have the same simple name, but different versions (or public key tokens) and different contents. Since we only consider the simple name when trying to find and load a dependency, both AlphaTasks and BetaTasks will both end up using the same Utility--either the one from Alpha or the one from Beta--even though that is not what they expect to happen. Which one is selected will depend entirely on whether AlphaTasks or BetaTasks is loaded first.
We could decide to take the full assembly name into account when satisfying dependencies, and thus ensure both AlphaTasks and BetaTasks get the copies of Utility.dll they expect.

Support for satellite assemblies.
Right now if a task assembly or dependency needs to load a satellite assembly (to retrieve a localized resource, for example) we don't know to look for them in culture-specific subdirectories like en-US and de-DE.

I'm not actually sure if this is an issue in .Net Core or if localized resources are handled in some other way.

Another scenario that would be great to enable is when a Build Task assembly's dependencies come from other NuGet packages.
For example, in the .NET Core SDK Build Tasks we are dependent on a bunch of NuGet assemblies, and on Microsoft.Extensions.DependencyModel.  With the current support, we need to package all our dependencies into a single .nupkg.  But it would be great if our .nupkg could just list Microsoft.Extensions.DependencyModel in its "dependencies", and MSBuild knew to look in that NuGet package location as well.

@eerhardt Are you saying that tasks should have a new dependency fallback search from a deps.json (or deps.runtime.json, whichever is the one with runtime dependencies), like .net core apps do?

I'm not trying to design it.  I'm just listing the scenario/issue. Packaging all my dependencies into my .nupkg doesn't feel like the correct thing to do.
But yes, basically you can imagine a .deps.json and runtimeconfig.json approach that works like .net core apps do.
There could be a feature in dotnet.exe that would allow "merging" .deps.json files between MSBuild and a Tasks assembly. But this approach would only work when MSBuild was running on .NET Core. dotnet.exe isn't invoked when MSBuild is running on the desktop .NET Framework.


Ember 2.8 Support

Hi,
Is there an ETA on when the latest couple of commits (for Ember 2.8 support) will be bundled into a new release? We're in the process of upgrading our app to 2.8 and now froala is throwing all kinds of errors which are blocking our efforts.
Thanks!

The froala folks have released new versions of this addon along side froala-editor releases (which can make situations like these difficult). What I could do is release a beta if that'll help you.

Thanks for the quick response. That'd be great if it's not too much trouble. Worst-case scenario, we can either fork the project or just apply the patches locally until the release comes.
Thanks for your help.

You could always lock the add-on version in your package.json file to a specific commit SHA.
E.g. the following will install the add-on including the Ember 2.8 compatibility commits:
"ember-froala-editor": "https://github.com/froala/ember-froala-editor#28eeee2f646dd3150149f9372dd990c1a672570b"

Really? That's awesome... I didn't know that was possible. Thanks @mike183, I'll give that a shot.

Worked perfectly. Thx again.


[FLINK-13479] [flink-cassandra-connector] Fix for Deterministic ordering for prepared statement

What is the purpose of the change
This pull request fixes the issue with Cassandra Connector issue FLINK-13479 where the deterministic order of columns in prepared statements is not working.
Brief change log

Updated the cassandra driver version from 3.0.0 to 3.3.1 where the issue is fixed (https://datastax-oss.atlassian.net/browse/JAVA-1587)

Verifying this change
This is trivial configuration change - only changes the depenndent cassandra driver version.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes - cassandra driver upgraded from 3.0.0 to 3.3.1
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: No
The S3 file system connector: No

Documentation

Does this pull request introduce a new feature? No
If yes, how is the feature documented? Not applicable

@flinkbot run travis

This deterministically fails the testCassandraBatchPojoFormat test.

@ronakthakrar Any updated regarding my remarks?

I'm closing this for now, please re-open if you want to pick it up again.


Final_Submit

Clean Sv data filse in hisotry

@ldr426 There is a conflict that needs to be resolved in echoshader/dev/test.ipynb.

Also, there is still conflict with test.ipynb that need to be addressed.


chore: bdd-spring-1583526984 to 0.0.1

chore: Promote bdd-spring-1583526984 to version 0.0.1

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

OWNERS

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


Support for Passkey Entry (pairing)

Modified the IO capabilities in pairing request to KeyboardOnly. MITM is not required from our end (i.e., from noble), so the peripheral device needs to require MITM for this to work. If the peripheral does not require MITM, then the code defaults to Just Works.

Any progress on merging that?

Hi all,
Thanks for tackling this issue, is there something preventing it to be merged?

Please forward change to downstream fork:
#923

Has anybody else been able to test the functionality?


presidio analyzer - config dir error

Describe the bug
The presidio analyzer should have the default dir for the config to be in the user's home, but instead it creates a folder in the run dir named ~.
To Reproduce
run the presidio analyzer
Expected behavior
~ expands to the user home path
Additional context
The cli does not properly expand the user dir.
Solution
os.path.expanduser

Hi @dhpollack
Thanks for opening this issue.
We will look into this issue. If you can contribute a PR it will be appreciated.


matAccordion and MatExpansionPanel  togglePosition not working

I'm trying to put the toggle indicator before the title but @input() togglePosition doesn't seen to work.
Steps to reproduce:

Go to stackblitz of the basic expansion panel : https://material.angular.io/components/expansion/examples
Add togglePosition="before" tot he mat-accordion, nothing changes.
Add [togglePosition]="'before'" and you get : Can't bind to 'togglePosition' since it isn't a known property of 'mat-accordion'.

Expected Behavior
The toggle indicator placed before the title
Actual Behavior
The togglePosition input is not recognized
Environment

Angular (the version from the stackblitz)
browser 60.7.1esr (32-bit)

Couldn't get it to work locally either
Sidenotes:  The documentation for matExpansionPanel is wrong for togglePosition (looks like a copy/paste from hideToggle)

The togglePosition input was added in version 8.1.0 and the docs example defaults to 8.0.2.
Here's an example using that input with the latest version: https://stackblitz.com/edit/angular-nipwqn?file=app/expansion-overview-example.html
We'll need to look at updating the versions used on the docs site


Question: EfficientAD Pred Scores are very low.

Hi,
The pred scores of the efficient AD model are coming very low. Any idea, what is the reason behind this?
[0.5111125114597613,
 0.5133660426906085,
 0.5584840866012671,
 0.5097309616622074,
 0.5100746147399043,
 0.5101457543058581,
 0.5078011069118742,
 0.49999999867182765,
 0.5171874406178997,
 0.5087297667708703,
 0.5009551808039379,
 0.5057654561524445,
 0.4999895069959258,
 0.539257350719084,
 0.49940360237013637,
 1.0,
 0.4994316863525398,
 0.4994300963089074,
 0.5165583225774157,
 0.5048455843669841,
 0.5154843004558268,
 0.5397329320390526,
 0.5101223970673893,
 0.5083544491795497,
 0.5032673477437677,
 0.49942594466358836,
 0.5006433367892856,
 0.5098894506945904,
 0.5126176016486443,
 0.5127730119775831,
 0.5302341765149362,
 0.49942822048686353,
 0.5073944904970427,
 0.4994255681267349,
 0.5114618296351392,
 0.5280324174412317,
 0.49941536510695117,
 0.5089816112649264,
 0.5227437273384679,
 0.5137013902640468,
 0.5014994171221683,
 0.5085513064539936,
 0.5100113379541017,
 0.5132433620694963,
 0.5113751708183177,
 0.4994075496982797,
 0.5080081078810714,
 0.5129591863193106,
 0.4994130655979363,
 0.4994456381387047,
 0.507350383665099,
 0.6742883947936092,
 0.4994323178984793,
 0.519790254606196,
 0.5131760290454198,
 0.5125366592855003,
 0.5056670881684252,
 0.5067094590581136,
 0.5135149821639905,
 1.0,
 0.4994077042089933,
 0.49940675500850656,
 0.6721566180050411,
 0.5094002476392792,
 0.5132569147398781,
 0.49941373322589055,
 0.5078058847904434,
 0.49941520218447955,
 0.49940407564220735,
 0.5204894080587092,
 0.5146554660286121,
 0.5027675370188384,
 0.5133325426428347,
 0.5122029143733313,
 0.5094226716153291,
 0.515083500550321,
 0.4994165772855578,
 0.5002911459985357,
 0.5089317321960678,
 0.5190379724898744,
 0.499425803213236,
 0.5095397871944086,
 0.6957075571253086]

Thanks.

What do you mean by low? The scores are normalized in the range 0,1 with 0 bring normal and 1 being abnormal

@alexriedel1 The scores are normalized between 0 and 1. But they are not high for the anomalous samples. Mostly all the scores are close to 0.5 only.

Assuming you have trained your Model using normal and abnormal Images, maybe the difference between normal and abnormal isnt very large. The training procedure tries to normalize the outputs in a way that 0.5 is the threshold for abnormal. You could try to use a different algorithm than efficientad and see of the problem still exists. If yes, then the Chance is high that your normal and abnormal are Not different enough. Maybe you can show some of your Images?

@alexriedel1 I am getting the following anomaly scores on the MVTEC bottle class. Is this reasonable? I tried other methods such as Padim and the scores are pretty high such as 0.70, and 0.94 for anomalous images. Is this because the Padim is a distance-based method and efficientAD is a reconstruction-based method?

How do you obtain these results? Through validation while Training or through testing After training? Can you Share your Code?

@alexriedel1
I used patchcore and EfficientAD to classify the same batch of data sets respectively. The F1Score and AUROC of patchcore are much better than EfficientAD, and the heat map of patchcore can clearly reflect the defective area. EfficientAD can hardly reflect the defective area. It was all trained in the past. 100 epochs. I would like to know if there is room for improvement when changing EfficientAD’s epoch to 1000. I don’t have mask annotation data. Is there any room for improvement, especially the defects and accuracy of heat maps?


fix(spdk) Hide the  on the volume detail page

https://github.com/longhorn/longhorn/issues/6128
image: wangsiye/longhorn-ui:7e7f4a0

@mergify backport v1.5.x


Add vi_mode tests

Hi,
I added the tests for the vi_mode functionality. This will tick one of the checkboxes of issue #458

@salmanulfarzy I tried making the change you requested, and made the file executable, but that didn't fixed the issue. Locally the tests work, do you know what could be the issue here?

I tried making the change you requested, and made the file executable, but that didn't fixed the issue.

Thank you @rodrigo-garcia-leon for following up. Suggested change was due to #533 and not related to failing tests.

Locally the tests work, do you know what could be the issue here?

This indeed passes on local, But glancing to travis logs, looks like travis isn't producing any output for $(spaceship_prompt). I'll try to look into it asap.

Closing this, since vi mode is moved out of the core to https://github.com/spaceship-prompt/spaceship-vi-mode


cryptomb: remove PKCS1_5 padding support.

Commit Message:
cryptomb: drop RSA signing PKCS1 v1.5 padding support.
Additional Description:
Drop RSA signing PKCS1 v1.5 padding support (which wasn't functional). Only keep the RSA-PSS for now. If someone needs PKCS1 v1.5, we can re-add it properly later.
Risk Level: low
Testing: unit tests, manual testing
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: CryptoMB extension works on 3rd Gen Intel Xeon Scalable processors.

If it isn't functional, add TODO and raise error may be more helpful.

If it isn't functional, add TODO and raise error may be more helpful.

Makes sense. Added a debug message and a TODO.

/retest


feat: Migration to Flutter 3.27

Hi everyone!
It seems that it's now safe to migrate to Flutter 3.27.
The main change is the use of Impeller on Android.
https://medium.com/flutter/whats-new-in-flutter-3-27-28341129570c

Codecov Report
Attention: Patch coverage is 0% with 54 lines in your changes missing coverage. Please review.

Project coverage is 6.44%. Comparing base (4d9c7fc) to head (df022c3).
Report is 583 commits behind head on develop.

Files with missing lines
Patch %
Lines

packages/smooth_app/lib/themes/smooth_theme.dart
0.00%
7 Missing :warning:

packages/smooth_app/lib/widgets/smooth_tabbar.dart
0.00%
6 Missing :warning:

.../lib/cards/product_cards/smooth_product_image.dart
0.00%
4 Missing :warning:

.../lib/generic_lib/buttons/smooth_simple_button.dart
0.00%
4 Missing :warning:

...p/lib/generic_lib/widgets/images/smooth_image.dart
0.00%
3 Missing :warning:

...roduct/product_page/footer/new_product_footer.dart
0.00%
3 Missing :warning:

...es/smooth_app/lib/cards/data_cards/score_card.dart
0.00%
2 Missing :warning:

..._app/lib/pages/image/product_image_other_page.dart
0.00%
2 Missing :warning:

.../smooth_app/lib/widgets/smooth_navigation_bar.dart
0.00%
2 Missing :warning:

.../cards/product_cards/smooth_product_base_card.dart
0.00%
1 Missing :warning:

... and 20 more

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #6076      +/-   ##
==========================================
- Coverage     9.54%   6.44%   -3.11%
==========================================
  Files          325     444     +119
  Lines        16411   25222    +8811
==========================================
+ Hits          1567    1625      +58
- Misses       14844   23597    +8753

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Potential arras to look at in the future, or to check for régressions (deep links and edge to edge)

We already support the edge-to-edge mode.
For deep links, we also use the standard lib, so we shouldn't have any issue.

merged, thanks @g123k 👍


Outputs array of Location objects from JSON response

Edited API call so that the output JSON object is converted to an array of Location objects, each of which includes attributes such as location and name about each of the attractions.

Read above.


Broken link to German M365 sign-in portal

Under the General information section of the article, the link given for the sign-in portal (https://portal.office.de) is not working. It links to an unsecure page.

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

ID: 53b7d0ac-caf6-82bc-ee11-129695bdc028
Version Independent ID: 2c1e2edb-7a65-770d-2f9d-f561fae87145
Content: Learn about Office 365 Germany - Microsoft 365 admin
Content Source: microsoft-365/admin/admin-overview/learn-about-office-365-germany.md
Service: o365-administration
GitHub Login: @LoreenLa
Microsoft Alias: loreenl

@AlexJerabek Thank you for your feedback. I'm able to repro the issue too.
@LoreenLa Could you please confirm if I can change this link to portal.office.com or Is this a known bug with portal.office.de?
CC: @yogkumgit
Thanks
Sri

@TinaMcN Please help us in resolving this issue. Thanks

@yogkumgit this is an issue on the English page. @LoreenLa let me know if you need help finding someone knowledgeable about  Office 365 Germany and I can see if I can find someone in the German subsidiary.

@LoreenLa Kindly help. Thanks

@LoreenLa Gentle reminder

Just remove the entire row with the sign-in link, please. As an FYI - I have not owned this article for several years.

@LoreenLa  I have made the changes on PR# https://github.com/MicrosoftDocs/microsoft-365-docs/pull/6982

@msbemba Thank you very much for the contribution and updating the documentation with PR. @AlexJerabek Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Floating Label Input text goes on second line

Hi! Thanks for trying out NativeBase!
Due to the overwhelming requests and issues/questions we ask you to add the following details with your issue.
Please check the existing open/closed issues for a possible duplicate before creating a new issue :)
Also DO fill out the below form to give us a better idea about your environment and help us debug it quicker. Issues without the required details will mostly be unattended to.
react-native, react and native-base version
react 16.3.1
react-native 0.55.3
native-base 2.4.3
Expected behaviour
Text should stay one line as other types of Items (fixed etc.)
Actual behaviour
When user blur of Input text goes on two lines
Steps to reproduce (code snippet or screenshot)

Copy simple code:

<Form>
  <Item floatingLabel>
    <Label>Test</Label>
    <Input />
  </Item>
</Form>

Put text longer than input width

Screenshot of emulator/device
When typing everything is OK:

When blur text goes on second line:

Is the bug present in both ios and android or in any one of them?
Tested only on iOS
Any other additional info which would help us debug the issue quicker.

@gibbsfromncis This is not happening with 2.7.1
Would you like to check?

@gibbsfromncis Closing this today in case of no response

@SupriyaKalghatgi If I remember correct it works fine now. Sorry for no response I was on vacation


Sandstorm: An ActionView::Template::Error occurred in citations#index:

 undefined local variable or method 'predicate' for #<DataAttribute:0x00007f70d883ace8> Did you mean?  predicate_name   app/models/data_attribute.rb:56:in 'predicate_name'

type == 'InternalAttribute' ? predicate.name : import_predicate
For ImportAttribute, fails what predicate is evaluated.

OK to close?

Fixed in two different ways.


Query params not included in ARGS_NAMES|ARGS in GET request at phase:2 evaluation

Not sure if this is a bug or a CRS rule limitation.
With the CRS rule ID 942270, any GET requests with SQL injections params in them arent detected. Assumption here is ARGS|ARGS_NAMES should contains query params as well (is it still true for libmodsecurity 3.x?).
However, for GET requests this rules doesnt get hit unless phase level is changed to "phase:1".
SecRule REQUEST_COOKIES|!REQUEST_COOKIES:/__utm/|REQUEST_COOKIES_NAMES|ARGS_NAMES|ARGS|XML:/* "@rx (?i)union.?select.?from" \
"id:942270,\
phase:2,\
block,\
capture,\
t:none,t:urlDecodeUni,\
msg:'Looking for basic sql injection. Common attack string for mysql, oracle and others',\
logdata:'Matched Data: %{TX.0} found within %{MATCHED_VAR_NAME}: %{MATCHED_VAR}',\
.....

This rule works all right if POST has offending ARGS in the request body.
Please note that SecRequestBodyAccess is On (not to get involved with https://github.com/SpiderLabs/ModSecurity/issues/2465)

Output of:

DebugLogs (when undetected)
undetected.Debuglogs.txt
AuditLogs
[None as nothing was triggered]
Error logs
[None]

Steps to reproduce the behavior:
Using Postman to generate a GET request to URL : http://j*********:8000/rest/products/search?q=apple'))UNION%20SELECT%20username,password,email,4,5,6,7,8,9%20FROM%20USERS--
Expected behavior
Shouldn't the rule evaluation happen in phase 2 as well against GET query ARGS?
**Server **

libModsecurity 3.0 dynamically linked with our application
OS: Ubuntu Linux

Rule Set (please complete the following information):
CRS rule set 3.3v

Are you sure there isn't any exception? I can't find the rule 942270 all in your attached debug.log - it should be at there. I tried your request (with SecRuleEngine DebugOnly - I assume your config has this setting), and the rule was triggered:
[1616596340] [/rest/...FROM%20USERS--] [4] (Rule: 942270) Executing operator "Rx" with param "(?i)union.*?select.*?from" against REQUEST_COOKIES|REQUEST_COOKIES_NAMES|ARGS_NAMES|ARGS|XML:/*.
[1616596340] [/rest/...FROM%20USERS--] [9]  T (0) t:urlDecodeUni: "q"
[1616596340] [/rest/...FROM%20USERS--] [9] Target value: "q" (Variable: ARGS_NAMES:q)
[1616596340] [/rest/...FROM%20USERS--] [9]  T (0) t:urlDecodeUni: "apple'))UNION SELECT username,password,email,4,5,6,7,8,9 FROM USERS--"
[1616596340] [/rest/...FROM%20USERS--] [9] Target value: "apple'))UNION SELECT username,password,email,4,5,6,7,8,9 FROM USERS--" (Variable: ARGS:q)
[1616596340] [/rest/...FROM%20USERS--] [7] Added regex subexpression TX.0: UNION SELECT username,password,email,4,5,6,7,8,9 FROM
[1616596340] [/rest/...FROM%20USERS--] [9] Matched vars updated.
[1616596340] [/rest/...FROM%20USERS--] [4] Running [independent] (non-disruptive) action: setvar
[1616596340] [/rest/...FROM%20USERS--] [8] Saving variable: TX:sql_injection_score with value: 15
[1616596340] [/rest/...FROM%20USERS--] [4] Running [independent] (non-disruptive) action: setvar
[1616596340] [/rest/...FROM%20USERS--] [8] Saving variable: TX:anomaly_score_pl1 with value: 15
[1616596340] [/rest/...FROM%20USERS--] [4] Rule returned 1.

with this request:
curl -vvv "http://localhost:8080/rest/products/search?q=apple'))UNION%20SELECT%20username,password,email,4,5,6,7,8,9%20FROM%20USERS--"

See the last line in log snippet (Rule returned 1).

Root cause is found. We werent evaluating phase 2 and 4 unconditionally. Will provide an image today for this

Thanks. Your simple response got me thinking in the right direction. The issue was that we werent running processing for phase 2 unconditionally. Now am able hit it.
Thanks for the help. Will close the ticket.


Enable snapshot in app.toml by default

Feature description

From my experience in many different chains developed with Stargate, most of the validators or fullnodes would not have snapshot enabled. This make state sync a new node very difficult. I suggest we make snapshot being enabled in app.toml by default when the file is being created.
Implementation proposal

I researched how to implement it.
The config is generated with the default state-sync setting in the start cmd.
We can build a custom start cmd and simply add a line to enable the state-sync, say:
...
  config.StateSyncConfig.Enable = true
  cfg.WriteConfigFile(filepath.Join(config.RootDir, "config", "config.toml"), config)
  return displayInfo(toPrint)
}

Thoughts on it? @RiccardoM @bragaz

@dadamu Great!

@kwunyeung @dadamu To be honest, I would not set anything like this as enabled by default. Enabling state sync requires different things to be done:

Set the correct pruning options.
Set the state sync to match those pruning options.

Aside from this, node operators should also know when this is enabled as it will require more disk space than usual.
If we want to encourage people in enabling this options, we should do this inside our documentation telling the advantages it will have over the community. I think this will be much more beneficial to everyone instead of silently enabling it for them. They could learn and decide whether or not to enable it.

@RiccardoM mmmmh, I didn't see it from this perspective, you're right, what if we improve the docs in that way then?

@RiccardoM mmmmh, I didn't see it from this perspective, you're right, then we can work on docs improvements for this I guess

Yes, I think this is the best approach. We should tell them why they should enable it (support community, faster sync if they need to migrate, etc) and then tell them how they can do it explaining the meanings of the different settings that can be tweaked

Hm... But by default the pruning is already set to default which means sanpshot_interval can be multiples of 500. If we set the default value to 500, 1000 or 1500 already make the configurations correct?
I agree we should enhance the doc for storing snapshot. However, if this is a good practice, why don't we set the good practice as default? And then we can mention "We have turn snapshot on by default. If you would like to turn it off, set it to 0." What do you think?

I think enhancing the doc and setting enable state-sync by default are different things. Enabling it might reduce validators to be confused about why state-sync doesn't work. I think there are no cons to desmos networks.
By the way, setting snapshot_interval can be implemented in the same way to build a custom cmd.

Just thought i join this discussion as a recently new validator. I would actually prefer to have snapshots turned on by default with instructions in the docs explaining what snapshot does and how to turn it off.
I think the main reason for this would be laziness and being new, I don't want to edit the config file with too much customization.
If it's already turned on by default with a good explanation in the docs and can benefit the chain, I would be less likely to follow the optional instructions on how to turn it off (ex: osmosis).
On the other hand, I remember being asked to turn on snapshots for apollo on discord and I was simply too lazy. It's optional and just too much work. Now that i'm a little bit more experienced, I can definitely see the affects it has on nodes when there are not enough snapshots available - Many of us are forced to resync the node from genesis.


vitest doc section for VS Code

Hi VS Code PM here 👋
We are currently in the process of making the testing story more discoverable in vscode.
As part of that we are writing docs for the VS Code testing framework, and we really like your testing extension and we want to have a section on it.
This is the draft PR https://github.com/microsoft/vscode-docs/pull/5332
Do you have some good docs to point us to that we could reuse? Or would you like to contribute to our PR directly?
Thanks

Thanks! I'm happy to help. Is this doc about quickly setting up tests in VS Code for TypeScript projects?

Users can follow the guide here to set up Vitest. It works for both TypeScript and NodeJS out of the box, and you can try it online on StackBlitz.

@zxch3n then I suggest that the docs in VS Code just link to that guide and add some VS Code specifics. Like what extension is needed etc... What do you think?

Yeah, I agree. I'll contribute to the PR by tomorrow

No rush. Looking forward to it. Thanks!

@isidorn is it something that you are still working on?

Unfortunately no. But we are open to PRs.


Do not hardcode bucket names in the Census codebase

Currently, the bucket name for the embeddings is hardcoded in the Census codebase. This is not ideal: if we need to move the bucket, it will break previous versions.
At least for embeddings, we should use the following approach:

Add an embedding_uri field to the manifest which points to the location of the artifact on S3
Replace the code that uses the hardcoded bucket location with such URI

We should also ensure that the same pattern doesn't get added for indexes.

embedding_uri has been added to the manifest, so the only remaining part is to incorporate it in the code.


use global defaults instead of respecifying each time

@hamelsmu @jph00 while we're at it, I probably should've tagged you both before merging this one too, please have a look

Message ID: @.***>Thanks - it's really encouraging to see things getting simpler in this code!


Add fmf extension

.fmf files are yaml text files used most notably in tmt.
Could for example be used in tmt-lint pre-commit hooks or when running yaml linters against .fmf files.
See: https://github.com/teemtee/tmt/discussions/2164

unfortunately searching this extension does not land on the tool you're suggesting so it is too ambiguous for inclusion here


Clients that support multiple subprotocols are not handled correctly

Discussed in https://github.com/lbbrhzn/ocpp/discussions/496

Originally posted by bitbs566 April 19, 2022
There are several testing/simulations charging point tools floating around in the GitHub aimed to test the functionality of your OCPP server, but unfortunately it seems that neither of them working out of the box correctly. Can anyone suggest good program to test the functionality of the OCPP HA integration while Im waiting my for my OCPP EVSE controller?
Im always get this error in HA:
Logger: custom_components.ocpp
Source: custom_components/ocpp/api.py:205
Protocols mismatched | expected Subprotocols: ['ocpp1.6'], but client supports ocpp1.6, ocpp1.5 | Closing connection
I was able to correct the main.js file on the other side to try to fool OCPP HA server into thinking that it is sending only 1.6 signals, but it doesn't work unfortunately. Does anyone have any ideas? I will be glad to hear anything from you guys, thanks in advance.

@bitbs566 can you add a log file with your modification for ocpp1.6 only?


How to get the problems?

Sorry for this probably noob question, but I can't fetch the hello-world problem..
I have the exercism version 2.2.0, and I'm trying to fetch the problem with:
$ exercism fetch pony hello-world
2015/07/21 20:53:09 unable to fetch problems (HTTP: 404) - We don't have problem 'hello-world' in 'pony'

Thanks!!

@rockneurotiko It's too early to be available via the exercism binary, even as an inactive - still needs docs, etc, and to be added as a submodule to the tracks list (See #1). I'll try to get it into a state where it can at least be added to inactive tracks ASAP, but I'm kinda building this track as a way of learning Pony myself, so no promises :smiley_cat:

Ou... I thinked that you can download the tracks inactives manually :smile:
I'll try to help you with the problems, but I'm learning Pony too, so no promises neither ;-)
Thanks for add this track!

You can download the inactives manually, once they're added to the main collection... this repo was created yesterday, so it's not there yet.
But please, send PRs!!!

I'll get this added to the main repo tonight (still inactivated, but you'll be able to fetch the individual problems).

@rockneurotiko confirming exercism fetch pony hello-world is now available for your enjoyment.

The api is updated, exercism fetch pony hello-world should now work.

yeah, it works, thanks! :heart:


Fixed schema validation for invalid properties in retry configuration.

Description
Prevents invalid properties for being added in retry configuration in the test story.
Issues Resolved
Resolve https://github.com/opensearch-project/opensearch-api-specification/issues/757
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Since this fixes a bug, please add a test.


Integrate Codecov to publish code coverage data

Integrate Codecov to publish code coverage data. The report will be available at https://app.codecov.io/gh/aws/aws-cloud-map-mcs-controller-for-k8s/
Later we can introduce code coverage requirement for each pull requests if needed.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@e747596). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #33   +/-   ##
=======================================
  Coverage        ?   18.51%
=======================================
  Files           ?        4
  Lines           ?      470
  Branches        ?        0
=======================================
  Hits            ?       87
  Misses          ?      370
  Partials        ?       13

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 e747596...08ccc13. Read the comment docs.


open folder not responding

Type: Feature Request
my open folder is not responding when i click to (open folder) and showing working in it.
VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:51:52.184Z)
OS version: Windows_NT x64 10.0.22631
Modes:

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Change DeviceType trigger

Use platform-specific matching APIs and names

Doh, beat me to it!

But then again I managed to accept the other PR first, despite tim's was more correct :)


Why does this page suggest enabling NPM?

When I go to the NPM page it tells me it is being retired on 2/29/2024. What needs to be done outside of using NPM?

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

ID: 89e7de4d-81f3-0b68-d79a-b131e4a4d6ce
Version Independent ID: 74b0f393-a266-a25f-7b59-ee2f4c66ea57
Content: Monitor network connectivity using Azure Monitor agent - Azure Network Watcher
Content Source: articles/network-watcher/azure-monitor-agent-with-connection-monitor.md
Service: network-watcher
GitHub Login: @halkazwini
Microsoft Alias: halkazwini

#assign:halkazwini

@scott1138
Thank you for your feedback! I'll check with Connection Monitor team and get back to you.

Hi @scott1138
The NPM service continues to be critical piece of enabling OnPrem monitoring, and it will be a part of creating a connection monitor until Aug 2024.
I'll close this GitHub issue for now, but feel free to comment here as necessary.
#please-close


CMake improvements

Some changes in CMakeLists.txt to support out-of-source (AKA shadow) builds and to look for python in the system, fail if it is not found.

LGTM

Applied, tks


Hindi lectures bug

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
When i play hindi lectures some is played some have no quality option to play so i cannot play hindi b lecture i tried to play topi shukla chapter
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
Hi

Hi

Hi, please share complete verbose output


Rename F2 shifts selection to wrong characters

When you select an identifier e.g. variableName in the text:
let variableName: number = 12;
console.log(variableName)

then the editor shifts the selection one character right when hitting F2 and ariableName: is selected and you will "rename" that instead and also shifted one char right other places, so if e.g. the new name is newVarName it becomes:
let vnewVarName number = 12;
console.log(vnewVarName

which destroys the code
kr
Morten

Can't reproduce. Suspect unicode issues. Can you provide a zip file against this issue : https://github.com/alm-tools/alm/issues/260 :rose:


Renaming file just changing camel dont change thef ile

Description
try to rename show.tsx to Show.tsx does't work
Neovim version
0.7

Operating system and version
mac
Steps to reproduce
try to rename show.tsx to Show.tsx
Expected behavior
file name change to Show.tsx
Actual behavior
file continues show.tsx
Minimal config
vim.cmd [[set runtimepath=$VIMRUNTIME]]
vim.cmd [[set packpath=/tmp/nvim/site]]
local package_root = '/tmp/nvim/site/pack'
local install_path = package_root .. '/packer/start/packer.nvim'
local function load_plugins()
  require('packer').startup {
    {
      'wbthomason/packer.nvim',
      {
        'nvim-telescope/telescope.nvim',
        requires = {
          'nvim-lua/plenary.nvim',
          'nvim-telescope/telescope-file-browser.nvim',
        },
      },
      -- ADD PLUGINS THAT ARE _NECESSARY_ FOR REPRODUCING THE ISSUE
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. '/plugin/packer_compiled.lua',
      display = { non_interactive = true },
    },
  }
end
_G.load_config = function()
  require('telescope').setup()
  require('telescope').load_extension('file_browser')
  -- ADD INIT.LUA SETTINGS THAT ARE _NECESSARY_ FOR REPRODUCING THE ISSUE
end
if vim.fn.isdirectory(install_path) == 0 then
  print("Installing Telescope and dependencies.")
  vim.fn.system { 'git', 'clone', '--depth=1', 'https://github.com/wbthomason/packer.nvim', install_path }
end
load_plugins()
require('packer').sync()
vim.cmd [[autocmd User PackerComplete ++once echo "Ready!" | lua load_config()]]

Cannot reproduce with neovim master and stable using the minimal init lua provided on Arch Linux.
Video shows neovim stable with latest telescope-file-browser
https://user-images.githubusercontent.com/39233597/168604674-5f34dd80-7947-4574-9894-23db7fac2d56.mp4

Thanks for the feedback! So, I unfortunately cannot do anything here really as I don't own a Mac device. Can this "bug" even be fixed from within telescope-file-browser.nvim or plenary.nvim?

Once more, help from a Mac user would be very much appreciated here :)

what can i do to help?

Relevant code is here
https://github.com/nvim-telescope/telescope-file-browser.nvim/blob/7861dddeb3102880965bb3720250dd5c3cc5da33/lua/telescope/_extensions/file_browser/actions.lua#L235-L279
I guess
      old_path:rename { new_name = new_path.filename }

fails because plenary.path (check plenary repo) fails to distinguish between the files on MacOS.

Looking into plenary a bit, the relevant section for rename is found here and really doesn't do too much, delegating the rename to the libuv function fs_rename
@otavioschwanck
What happens if you try to do your rename directly with libuv?
:lua vim.loop.fs_rename('show.tsx', Show.tsx')

Does that successfully do the requested rename?

Interesting, using fs_rename() worked for me on Mac.

Do you get the notification that the file exists if you use telescope-file-browser?

Interesting, using fs_rename() worked for me on Mac.

Interesting. @gegoune  as it sounds like you are able to verify for us, can you test a rename via plenary?
:lua require("plenary").path:new('show.tsx'):rename({new_name="Show.tsx"})

Does this work?
The path we are following is

does libuv work (sounds like this has been verified)
does plenary work (verifying now)
does telescope-file-browser work (verifying next)

Plenary does not seem to be creating that file at all for me. Tried limiting it to :lua require("plenary").path:new('show.tsx') but no new file shows up on filesystem in cwd.

Path:new(...) doesn't create a new file, but just a Path object. This should be creating the file via plenary.
:lua require("plenary").path:new('show.tsx'):touch()

Here we go. Renaming does not work:
Vim(luafile):E5113: Error while calling lua chunk: ...site/pack/packer/start/plenary.nvim/lua/plenary/path.lua:526: File or dir
ectory already exists!

show.tsx got created now but not renamed. Which is expected as per Mac OS:
$ mv show.tsx Show.tsx
mv: 'show.tsx' and 'Show.tsx' are the same file

If plenary behaves like mv on macOS, I'm inclined to say the original issue is expected behavior. Thoughts?

If plenary behaves like mv on macOS, I'm inclined to say the original issue is expected behaviour. Thoughts?

I'd agree, though even if not, this clarifies that this should be an issue with Plenary and not telescope-file-browser. So whether this is an issue or not, probably should be migrated to the appropriate code base :)
That said, mv will also complain and a rename is just a move, I believe this is expected behavior. This would be the same as if you were to try and rename a file to itself (as that is what MacOS sees this command as), which will also fail.
Maybe worth bubbling the received plenary error up to the user though?

Maybe worth bubbling the received plenary error up to the user though?

I'm reasonably sure the user sees that error and OP is incomplete. @gegoune could you please confirm? :)

this should be an issue with Plenary and not telescope-file-browser

Plenary path could use some love; it's quite underutilized. In practice, I've ended up making the relevant PRs and, practically speaking, it is unlikely anyone else would step up 😅

Well, replication you came up with @fdschmidt93 errors with traceback:
E5113: Error while calling lua chunk: ...site/pack/packer/start/plenary.nvim/lua/plenary/path.lua:526: File or directory already exists!
stack traceback:
        [C]: in function 'error'
        ...site/pack/packer/start/plenary.nvim/lua/plenary/path.lua:526: in function 'rename'
        test.lua:4: in main chunk

telescope-file-browser errors with: [file_browser.action.rename] Show.tsx already exists! Skipping. through vim.notify()
It's clear enough for those who know that APFS is case insensitive.

lua require("plenary").path:new('show.tsx'):rename({new_name="Show.tsx"})

Doesn't works, even chaging Show.tsx for something else

Interesting, using fs_rename() worked for me on Mac.

Interesting. @gegoune as it sounds like you are able to verify for us, can you test a rename via plenary?
:lua require("plenary").path:new('show.tsx'):rename({new_name="Show.tsx"})

Does this work? The path we are following is

does libuv work (sounds like this has been verified)
does plenary work (verifying now)
does telescope-file-browser work (verifying next)

Sounds like this is working as intended/expected then. Technically Plenary (a different repo altogether) could put a workaround to allow for renaming a file to itself but that shouldn't be considered IMO.
While I understand that you aren't actually renaming the file to the same name (show.tsx != Show.tsx), MacOS believes these are the same and thus this error is both appropriate and expected.
This seems like a non-issue, especially as the error does bubble up to the user.

@fdschmidt93 thoughts?
IMO this is a non-issue. The command you tried to do failed at an OS level as well (as shown by the attempt to mv the file in the same manner on the filesystem). The fact that vim and libuv are able to do this is a side effect and IMO not something that should be replicated in other file system interfaces as it strays away from what the actual Filesystem is doing underneath. As far as MacOS is concerned show.tsx and Show.tsx are the same file so a rename is nonsensical.
All of that said, I dont think a workaround would be super difficult to put in place, should the Plenary team chose to do so (I do not currently contribute to plenary (yet)).

Yes, I fully agree. This can be closed here. Feel free to open an issue on plenary-side, but please don't have too much hope someone else "fixes" this for you.
Some general comments. In particular an (non-)issue like this one I recommend is best fixed by users facing and caring about the issue. After all, this is what makes open source go round. I'm happy to help, though in the long run, it is to some degree unsustainable for maintainers to serve all the issues in the project (I guess people here are aware I'm very active on telescope.nvim too). That's also why I indicated I'm not so interested in trying to "fix" this. Furthermore, OP provided incomplete information and was extremely tight-lipped on sufficient communication (took a while to certify that plenary raises a descriptive error, by another user), which isn't helping in getting something done about this (at least by me); in other words, I should have had closed the issue as wontfix already.


x/tools/gopls: working with TinyGo

package main
import "machine"
import "time"

func main() {
    led := machine.Led
    led.Configure(machine.PinConfig{Mode: machine.PinOutput})
    for {
        led.Low()
        time.Sleep(time.Millisecond * 300)
        led.High()
        time.Sleep(time.Millisecond * 1000)
    }
}

Setting  GOPATH="/path/to/go:/usr/lib/tinygo/src"  allows me to include machine.
But still I get error Led not declared in package machine. That's caused by first line of board_bluepill.go is set to // +build bluepill. Its not detected as "machine package".
Any syggestions how to fix this to get gopls working with tinygo.

You will need to add the following to your configuration:
"gopls": {
    "env": {
        "GOFLAGS": "-tags=bluepill"
    }
}

Thanks that worked perfectly.
###SOLUTION###
export GOPATH="/path/to/go:/path/to/tinygo" in my case /usr/lib/tinygo
Create touch .vim/coc-settings.json in your local tinygo project repository.
Put this code in .vim/coc-setings.json {         "go.goplsEnv": {                 "GOFLAGS": "-tags=bluepill"         } }


fix(html): image link (#7)

See https://stackoverflow.com/questions/52876285/embedding-an-image-stored-in-github.
Should fix #7.

Hi thanks so much for submitting this PR!  Do you have an example plugin where this will demonstrate it is working properly?

Sorry, I don't know how to setup the actions but I've made some experiments in the browser.
Using the default https://github.com/username/repo/blob/branch/file:

Using https://raw.githubusercontent.com/username/repo/branch/file:


Move install routine into common code and add tests.

Proposed Changes
As the title says mostly. This factors the install routine into common code and adds unit tests to them. Reuses the now common handler in both reconcilers.
Release Note

NONE

/assign @aliok @jcrossley3 @houshengbo

/test pull-knative-sandbox-operator-upgrade-tests


adds male and female arachnids

About the PR
ararchnids are no longer unsexed
Why / Balance
fixes the bug of randomized arachnids always having she/her pronouns
adds more customization that arachnid lacks as you can choose between a masculine or feminine body, arachnid is the only specie without this option
Technical details
3
Media

feminine vs masculine arachnid

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
nope
Changelog
no

its a trait of Arachnids, i like it. makes it more interesting to roleplay with

Should probably have a better reason than “they’re the only ones who have it”.

Should probably have a better reason than “they’re the only ones who have it”.

Ok

I thought they default to they/them?

I thought they default to they/them?

nuh uh

Every time i've used a scrambler it chooses she/her (I've used scrambler a lot)

Every time i've used a scrambler it chooses she/her (I've used scrambler a lot)

Testing indicates that it never scrambles pronouns. Your base character has she/her set, right?

NB erasure happening before our eyes

Nope, pronouns are not she/her before the scramble

If there is a bug with the scrambler that should be fixed separately regardless as we want to support unsexed species.

If there is a bug with the scrambler that should be fixed separately regardless as we want to support unsexed species.

My testing indicated that the impacts both sexed and unsexed species just the same

If there is a bug with the scrambler that should be fixed separately regardless as we want to support unsexed species.

#25282

Aren't male spiders usually smaller?
Also, how do the clothes look on this body shape since it's different than the other humanoid females

I said nothing about reptilians but are we seriously applying human sexual dimorphism to the SPIDERS? Come on.

i don't really see an issue with arachnids not having distinct sexes. I think it's a fairly interesting trait and helps make them more distinct. Perhaps it's minor, but i think it adds a lot. I'm opting to close this PR. thank you for your contribution.

i don't really see an issue with arachnids not having distinct sexes. I think it's a fairly interesting trait and helps make them more distinct. Perhaps it's minor, but i think it adds a lot. I'm opting to close this PR. thank you for your contribution.

Extremely RARE EmoGarbage404 W

i don't really see an issue with arachnids not having distinct sexes. I think it's a fairly interesting trait and helps make them more distinct. Perhaps it's minor, but i think it adds a lot. I'm opting to close this PR. thank you for your contribution.

What emo said.


Error while runing

Activating virtual environment...
Dependencies already installed, skipping installation.
Skipping .env copying
Starting the bot...
Traceback (most recent call last):
File "/root/MemeFiBot/main.py", line 4, in
from bot.utils.launcher import process
File "/root/MemeFiBot/bot/utils/init.py", line 2, in
from . import launcher
File "/root/MemeFiBot/bot/utils/launcher.py", line 16, in
from bot.core.tapper import run_tapper
File "/root/MemeFiBot/bot/core/tapper.py", line 9, in
from aiocfscrape import CloudflareScraper
File "/root/MemeFiBot/venv/lib/python3.12/site-packages/aiocfscrape/init.py", line 8, in
import js2py
File "/root/MemeFiBot/venv/lib/python3.12/site-packages/js2py/init.py", line 72, in
from .base import PyJsException                                                                           File "/root/MemeFiBot/venv/lib/python3.12/site-packages/js2py/base.py", line 2965, in
@Js
^^
File "/root/MemeFiBot/venv/lib/python3.12/site-packages/js2py/base.py", line 165, in Js
return PyJsFunction(val, FunctionPrototype)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/MemeFiBot/venv/lib/python3.12/site-packages/js2py/base.py", line 1377, in init
cand = fix_js_args(func)
^^^^^^^^^^^^^^^^^
File "/root/MemeFiBot/venv/lib/python3.12/site-packages/js2py/utils/injector.py", line 27, in fix_js_args
code = append_arguments(six.get_function_code(func), ('this', 'arguments'))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/MemeFiBot/venv/lib/python3.12/site-packages/js2py/utils/injector.py", line 121, in append_arguments
arg = name_translations[inst.arg]
~~~~~~~~~~~~~~~~~^^^^^^^^^^
KeyError: 3
done
PLEASE EDIT .ENV FILE

thanks


fix: make badge class reactive

Closes #1400
📑 Description

This pull requests fixes a regression that prevented from changing the badge color. It also removes unused code and formats the file.
Status

[ ] Not Completed
[x] Completed

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[ ] I have checked the page with https://validator.unl.edu/
[ ] All the tests have passed
[x] My pull request is based on the latest commit (not the npm version).

Summary by CodeRabbit

New Features

Added dynamic color functionality to the Badge component, allowing for interactive color changes in response to user actions.

Style

Improved code formatting and consistency in the Badge component for enhanced readability.
Streamlined the structure of exported properties and color definitions for visual clarity.

Refactor

Enhanced reactivity of the badgeClass variable to ensure it updates correctly with changes in dependencies.

Can you add a simple example to the doc page?

Thank you for your contribution.


http mp3 source error

hello,
trying to embed mp3-Livestream I got the following error:

After pressing 'Play' the Player looks like this:

Any ideas on this?
Regards,
Af0x

It is due to your mp3 is not loading / process properly.

thx, source looks like this: http://mp3channels.webradio.antenne.de/antenne

@Af0x Is this still an issue?

hi, long time :-) I don´t know anymore because I selected a different Skin. Regards

@Af0x Thanks for your response. I'll close this and we can always reopen it if it appears in the future


redpen-644 Suppress validationSuppress validation errors with comment style annotation

#644

Coverage increased (+0.2%) to 90.675% when pulling 7263356d3947a686a7426275fcebd4d823efa764 on takahi-i:redpen-644 into f5755d7c9ba603cb2107402329add68f3a7def45 on redpen-cc:master.

Coverage increased (+0.2%) to 90.675% when pulling 7263356d3947a686a7426275fcebd4d823efa764 on takahi-i:redpen-644 into f5755d7c9ba603cb2107402329add68f3a7def45 on redpen-cc:master.

Coverage increased (+0.2%) to 90.694% when pulling f05cd169906cd57648cdccb7e0f1c3857f44cdef on takahi-i:redpen-644 into f5755d7c9ba603cb2107402329add68f3a7def45 on redpen-cc:master.

Coverage increased (+0.2%) to 90.696% when pulling 2fc53b9557ef678c3f91f56f726d7f8c68cbec27 on takahi-i:redpen-644 into f5755d7c9ba603cb2107402329add68f3a7def45 on redpen-cc:master.

Coverage increased (+0.3%) to 90.767% when pulling 5c8c2585683ca8a1e12ec6c789a2395f0b4fbdcb on takahi-i:redpen-644 into f5755d7c9ba603cb2107402329add68f3a7def45 on redpen-cc:master.

Coverage increased (+0.3%) to 90.767% when pulling ce2b1c0ea06baccf6f1c470f1b68d2fd29cbdd73 on takahi-i:redpen-644 into f5755d7c9ba603cb2107402329add68f3a7def45 on redpen-cc:master.

Coverage increased (+0.3%) to 90.767% when pulling 27efe3b0ab98eb6ac84190fbf3a94b1c8cb77b38 on takahi-i:redpen-644 into f5755d7c9ba603cb2107402329add68f3a7def45 on redpen-cc:master.


Fix circular deps hell

Coverage remained the same at 100.0% when pulling 3ca9f3c35ae8e7f733f9f66fe740d577f1cdf923 on thed636:fix_circular_deps_hell into 26ca967fe02b945fa3c02df1bfa9f153176eedfa on elsid:apq2.

Merged


[BUG] IntegTests are failing on KNNQueryBuilder filter() field

What is the bug?
Integ Tests are failing on most recent PRs.
The root cause appears to be in this stack trace:
java.lang.NoSuchMethodError: 'org.opensearch.knn.index.query.KNNQueryBuilder org.opensearch.knn.index.query.KNNQueryBuilder.filter(org.opensearch.index.query.QueryBuilder)'
»  	at org.opensearch.neuralsearch.query.NeuralQueryBuilder.doRewrite(NeuralQueryBuilder.java:295)

How can one reproduce the bug?
Submit a PR.  Look at GitHub CI action.
What is the expected behavior?
Integ tests pass
What is your host/environment?
GItHub Actions
Do you have any screenshots?

Do you have any additional context?
The line of code causing the error is
KNNQueryBuilder knnQueryBuilder = new KNNQueryBuilder(fieldName(), vectorSupplier.get()).filter(filter());

Looking at the KNNQueryBuilder class, it doesn't have a filter(<argument>) method on it, but on its inner Builder class.  There is a filter field but it's only annotated with a @Getter so wouldn't accept an argument.
If this was a recent change I'd think it was the root cause but it seems to have been committed 3 months ago, but tests were passing 2 days ago so there's something else amiss.

Seems to have fixed itself.


Add "simple_numeric_planning" to problem_kind

Codecov Report

Merging #202 (41da565) into master (8517250) will decrease coverage by 0.17%.
The diff coverage is 70.31%.

@@            Coverage Diff             @@
##           master     #202      +/-   ##
==========================================
- Coverage   91.78%   91.61%   -0.18%
==========================================
  Files         122      123       +1
  Lines       14499    14598      +99
==========================================
+ Hits        13308    13374      +66
- Misses       1191     1224      +33

Impacted Files
Coverage Δ

...anning/grpc/generated/unified_planning_pb2_grpc.py
0.00% <0.00%> (ø)

...ed_planning/grpc/generated/unified_planning_pb2.py
62.02% <40.00%> (+0.13%)
:arrow_up:

unified_planning/model/walkers/linear_checker.py
65.21% <65.21%> (ø)

unified_planning/model/problem.py
85.16% <70.96%> (-1.49%)
:arrow_down:

...g/engines/compilers/conditional_effects_remover.py
97.60% <100.00%> (+0.01%)
:arrow_up:

...ngines/compilers/disjunctive_conditions_remover.py
99.02% <100.00%> (+<0.01%)
:arrow_up:

unified_planning/engines/compilers/grounder.py
100.00% <100.00%> (ø)

...g/engines/compilers/negative_conditions_remover.py
98.18% <100.00%> (+0.01%)
:arrow_up:

..._planning/engines/compilers/quantifiers_remover.py
98.96% <100.00%> (+0.01%)
:arrow_up:

unified_planning/engines/plan_validator.py
91.04% <100.00%> (+0.13%)
:arrow_up:

... and 8 more

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 8517250...41da565. Read the comment docs.


bug: Lazy's formatexpr does not allow for textwidth to take effect

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.5
Operating system/version
Fedora 39
Describe the bug
When trying to hard wrap—for example the whole file with ggqG—with a textwidth higher than 0, nothing happens. This is due to 'lazyvim.util'.format.formatexpr() and I know this because set formatoptions+=t makes it work again.
 Lazyvim's formatexpr is set to conform's formatexpr when conform is available, so it may have to do with how conform evaluates its formatexpr. No idea about conform's internals from my part though.

https://github.com/stevearc/conform.nvim/issues/55#issuecomment-1837153257 this issue is closed so it seems nothing can be done on LazyVim's side, and the mentioned issue is closed which I'm guessing means using gw is the best option. Thank you!


Port forward setting expects blank in port not "None"[Bug]

The problem
When creating port forwarding rules the option for internal port is optional. If this is left blank in the service call, the Asus router receives "None" in for this field. The port forward table does not accept this and while the port forwarding appears in the "virtual server/port forwarding" setting page, it is not in the port forwarding system log.
If you cycle port forwarding off and on generate a HA service call while one of the port forwarding rules has "None" in the port column an error is generated in the system log and no port forwarding table is generated.

The solution i have found is to used port: ""  inputting a blank field like what is down in the web interface. The web interface expects a port number only, no text or space is allowed.

Your device model
RT-AC86
Firmware type
Stock
Firmware version
<IP_ADDRESS>.386_49703
Integration version with the issue
0.20
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
Core 2023.2.2
Is there anything useful in the logs?
No response
Diagnostics information
No response
Additional information
No response

I have just been trying a few options and I think the problem is actually with the asus firmware. When a blank field is left in the port field, and a service call is made to add a new port forward rule a reset of the firewall rules must be made and this turns any blank fields into "None" which up loading up the firewall rules is not allowed and generates the error I showed above. It works fine if you don't have any fields with blank ports or if the existing table has no blank port and you are simply adding a new rule that has a blank port. The issue only comes up if you try to add multiple rules with blank ports. Hopefully all this makes sense but let me know if you need any more information.

Here is some rules before a service call and after a service call. I am not sure why after a service call the blank values are changing to "None"
before

After

Hey, @Tekno-man,
Thanks for your report and provided information!
I will try to fix everything from the integration side as soon as possible

Hey. The bug will be fixed in the next release - either today or tomorrow

Wow so fast. Thanks!
On Sat, 18 Feb 2023, 22:31 Yevhenii Vaskivskyi, @.***>
wrote:

Hey. The bug will be fixed in the next release - either today or tomorrow
—
Reply to this email directly, view it on GitHub
https://github.com/Vaskivskyi/ha-asusrouter/issues/538#issuecomment-1435648274,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMYZ66SW526FRC3TQUYKC5DWYCXJDANCNFSM6AAAAAAU3OIYI4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry for the delay. Please check the new version 0.21.0

Just tried it out, it works really well now. Can add multiple port forward
rules with blank ports. They also add very quickly.
On Wed, 22 Feb 2023, 17:58 Yevhenii Vaskivskyi, @.***>
wrote:

Sorry for the delay. Please check the new version 0.21.0
—
Reply to this email directly, view it on GitHub
https://github.com/Vaskivskyi/ha-asusrouter/issues/538#issuecomment-1439525276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMYZ66QWVORCFVTR2SP7LE3WYW2KPANCNFSM6AAAAAAU3OIYI4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great!

I just updated and can see there has been a lot of changes but I can't see anywhere how to call the new service? I read through your new documentation and they service is still listed but I can't find it in ha anymore. Once I get it working again happy to contribute the github for people like me to make it a bit more obvious.

Hello, @Tekno-man, sorry for the delay. The services were removed in 0.23.0 due to a breaking change in the backend library. They will be added back before the New Year. Please, follow #736 so that you'll get notified when the feature is ready
P.S. There is a bit of a delay in the docs update. Unfortunately, I don't have enough time and can forget some things. I'll try to keep it more up-to-date


user enrichment user duplication

The documentation states that enabling user enrichment might result in user duplication. Is there a way to identify if this will occur, is there away to fix it if it does occur and is there a work around?

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

ID: 0d1ddbec-2517-6111-942c-305df8b725f8
Version Independent ID: 4565cf20-c44b-d79b-73e1-327049687db3
Content: Enrich Cloud App Security Discovery data with Azure AD usernames
Content Source: CloudAppSecurityDocs/cloud-discovery-aad-enrichment.md
Service: cloud-app-security
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

Well, there sure are several numbers closing!!!
-------- Original message --------From: "M. Baldwin"<EMAIL_ADDRESS>Date: 2/4/19  3:00 PM  (GMT-06:00) To: MicrosoftDocs/CloudAppSecurityDocs<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/CloudAppSecurityDocs] user enrichment user
  duplication (#90)
Closed #90.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/MicrosoftDocs/CloudAppSecurityDocs","title":"MicrosoftDocs/CloudAppSecurityDocs","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/MicrosoftDocs/CloudAppSecurityDocs"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Closed #90."}],"action":{"name":"View Issue","url":"https://github.com/MicrosoftDocs/CloudAppSecurityDocs/issues/90#event-2116852841"}}}
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/MicrosoftDocs/CloudAppSecurityDocs/issues/90#event-2116852841",
"url": "https://github.com/MicrosoftDocs/CloudAppSecurityDocs/issues/90#event-2116852841",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]


A session had already been started - ignoring session_start()

Hi,
Thanks for your really amazing job :)
I'm having an issue on dev branch (I'm a L5 user): when I use php artisan route:* command
[ErrorException]
  A session had already been started - ignoring session_start()

I had to write PINGPONG_ADMIN_TESTING=1 in my .env but as I haven't find a clue in your docs, I wonder if it's the right way ?
Thanks

Get it ! Thanks :)