Conflict with more devices

Dear @paulino ,
When using one device to MQTT server the library works.
As soon as I added second hardware device, there was a conflict between them.
In the MQTT explorer, I saw that for each message from each sensor, the values in the message were quickly swapped alternately between the 2 devices, there were not 2 separate sensors, there was only one, and it was quickly changing the value from one and the other device.
In HA, only one device was created, and the values of each entity quickly alternated according to the values from one device and from second device.
I think this is a major bug in the library.
Do you have any idea how to solve the problem please?
Thank you

I noticed one more difference from the "home assistant integration" library. It is a tree layout in the MQTT explorer.
While your library creates a sensor, and then contains a configuration message, the "home assistant integration" library does the opposite. There is a device in the tree and there are sensors in it.
I can't use the "home assistant integration" library, it doesn't work on one ESP32 with Wifi/ethernet to make it universal. Your library can do that, that's why I chose it.

I'll try to reproduce the bug in a test.
The MQTT tree layout follows the examples from HomeAssistant at https://www.home-assistant.io/integrations/mqtt/#sensors

I'll try to reproduce the bug in a test.
The MQTT tree layout follows the examples from HomeAssistant at https://www.home-assistant.io/integrations/mqtt/#sensors

Thank you. I had the same code in two ESP32s...
As I public in this issue https://github.com/paulino/ha-mqtt-entities/issues/1
I filling identificators in a setup(),
I don't know if this is causing the problem.

Thank you. I had the same code in two ESP32s... As I public in this issue #1 I filling identificators in a setup(), I don't know if this is causing the problem.

Now I try your example "sensornumeric", only credentials, unique ID and name changed.
In HA was created two devices, but only the first has example entity. The second device is empty.
https://github.com/user-attachments/assets/b9b67448-e7cf-416f-bc43-64e69f757577

The problem is the entity's unique_id, they generate conflict with multiple devices. There is a new commit in the main branch. Please clone or update the repository in your local arduino installation and test again.
I've changed the library to generate "unique_id" from "device_id".
I also added a new function for #1
I tested this example and it works fine now:
#define ENTITIES_COUNT 2
HADevice ha_device_1 = HADevice("1.0");
HANumber ha_number_1 = HANumber("numberexample","Slicer",ha_device_1,1,100,1);

HADevice ha_device_2 = HADevice("1.0");
HANumber ha_number_2 = HANumber("numberexample","Slicer",ha_device_2,1,100,1);

void setup() {
    Serial.begin(115200);
    mqtt_client.setServer(MQTT_SERVER, MQTT_PORT);

    // Delayed device configuration
    ha_device_1.setName("Example 01: Device 1");
    ha_device_1.setIdentifier("device01");
    ha_device_2.setName("Example 01: Device 2");
    ha_device_2.setIdentifier("device02");

    HAMQTT.begin(mqtt_client,ENTITIES_COUNT);
    HAMQTT.addEntity(ha_number_1);
    HAMQTT.addEntity(ha_number_2);
    ha_number_1.setState(30);
    ha_number_2.setState(90);
    HAMQTT.setCallback(ha_callback);

    // start wifi
    WiFi.begin(WIFI_SSID, WIFI_PASSWORD);
}

Thank you @paulino ,
I am tried example and working fine.
I edit my full code, where is a HAText, and compille error
C:\Users\j.jirutka\Documents\Arduino\libraries\HaMqttEntities\src/hatext.cpp:55: multiple definition of `HAText::onReceivedTopic(PubSubClient*, unsigned char*, unsigned int)'; sketch\src\HaMqttEntities\src\hatext.cpp.o:Z:\EKOKROK\11_TPV\01 ELEKTRO\05 Výrobné podklady Arduino Nextion a RPI\Riadiace jednotky\Arduino Basic Profi (Nové)\02_VÝVOJOVÁ_VERZE\Program_ESP32\esp_comunicator\src\HaMqttEntities\src/hatext.cpp:55: first defined here
c:/users/j.jirutka/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: libraries\HaMqttEntities\hatext.cpp.o:(.data._ZN6HAText9componentE+0x0): multiple definition of `HAText::component'; sketch\src\HaMqttEntities\src\hatext.cpp.o:(.data._ZN6HAText9componentE+0x0): first defined here
c:/users/j.jirutka/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: libraries\HaMqttEntities\hatext.cpp.o: in function `HAText::HAText(char const*, char const*, unsigned int)':
C:\Users\j.jirutka\Documents\Arduino\libraries\HaMqttEntities\src/hatext.cpp:17: multiple definition of `HAText::HAText(char const*, char const*, unsigned int)'; sketch\src\HaMqttEntities\src\hatext.cpp.o:Z:\EKOKROK\11_TPV\01 ELEKTRO\05 Výrobné podklady Arduino Nextion a RPI\Riadiace jednotky\Arduino Basic Profi (Nové)\02_VÝVOJOVÁ_VERZE\Program_ESP32\esp_comunicator\src\HaMqttEntities\src/hatext.cpp:17: first defined here
c:/users/j.jirutka/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: libraries\HaMqttEntities\hatext.cpp.o: in function `HAText::HAText(char const*, char const*, unsigned int)':
C:\Users\j.jirutka\Documents\Arduino\libraries\HaMqttEntities\src/hatext.cpp:17: multiple definition of `HAText::HAText(char const*, char const*, unsigned int)'; sketch\src\HaMqttEntities\src\hatext.cpp.o:Z:\EKOKROK\11_TPV\01 ELEKTRO\05 Výrobné podklady Arduino Nextion a RPI\Riadiace jednotky\Arduino Basic Profi (Nové)\02_VÝVOJOVÁ_VERZE\Program_ESP32\esp_comunicator\src\HaMqttEntities\src/hatext.cpp:17: first defined here
c:/users/j.jirutka/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: libraries\HaMqttEntities\hatext.cpp.o: in function `HAText::HAText(char const*, char const*, HADevice&, unsigned int)':
C:\Users\j.jirutka\Documents\Arduino\libraries\HaMqttEntities\src/hatext.cpp:11: multiple definition of `HAText::HAText(char const*, char const*, HADevice&, unsigned int)'; sketch\src\HaMqttEntities\src\hatext.cpp.o:Z:\EKOKROK\11_TPV\01 ELEKTRO\05 Výrobné podklady Arduino Nextion a RPI\Riadiace jednotky\Arduino Basic Profi (Nové)\02_VÝVOJOVÁ_VERZE\Program_ESP32\esp_comunicator\src\HaMqttEntities\src/hatext.cpp:11: first defined here
c:/users/j.jirutka/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: libraries\HaMqttEntities\hatext.cpp.o: in function `HAText::HAText(char const*, char const*, HADevice&, unsigned int)':
C:\Users\j.jirutka\Documents\Arduino\libraries\HaMqttEntities\src/hatext.cpp:11: multiple definition of `HAText::HAText(char const*, char const*, HADevice&, unsigned int)'; sketch\src\HaMqttEntities\src\hatext.cpp.o:Z:\EKOKROK\11_TPV\01 ELEKTRO\05 Výrobné podklady Arduino Nextion a RPI\Riadiace jednotky\Arduino Basic Profi (Nové)\02_VÝVOJOVÁ_VERZE\Program_ESP32\esp_comunicator\src\HaMqttEntities\src/hatext.cpp:11: first defined here
c:/users/j.jirutka/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: libraries\HaMqttEntities\hatext.cpp.o: in function `HAText::setState(char const*)':
C:\Users\j.jirutka\Documents\Arduino\libraries\HaMqttEntities\src/hatext.cpp:43: multiple definition of `HAText::setState(char const*)'; sketch\src\HaMqttEntities\src\hatext.cpp.o:Z:\EKOKROK\11_TPV\01 ELEKTRO\05 Výrobné podklady Arduino Nextion a RPI\Riadiace jednotky\Arduino Basic Profi (Nové)\02_VÝVOJOVÁ_VERZE\Program_ESP32\esp_comunicator\src\HaMqttEntities\src/hatext.cpp:43: first defined here
collect2.exe: error: ld returned 1 exit status

Maybe  a problem in your Arduino installation, you may have the library twice.
Check if is Arduino/library folder twice, or if you have in your project under "src" folder and again in "Arduino/library"

I will check it after 5.8., I am leaving work now to hollidays.
Thank you very much again.

Hello @paulino ,
holidays are over and today I tried your new version with 2 identical devices connected at the same time. From the initial test it seems to work great. Thanks for your help. I will look forward to further improvements.


[docs] Improve old doc versions discoverability

[x] I have followed (at least) the PR section of the contributing guide.

[🚨 Work in progress] Not ready to review code yet.
Problem
I've seen some users, after v5 release, struggling to find v4 documentation. Friends even came to me asking for it. This indicates that the version button on the top left near the logo doesn't have enough affordance. If it had, people would be able to go to the /versions page where they would find every version documentation.
Proposed design
In order to facilitate accessing v4 documentation and increase discoverability of older versions in general, here's the proposed design prototype where I'm first using the outlined variant plus and arrow icon to clarify it's a button. Then, I'm actually turning it into a menu with only two options (could be more, no strong opinions) being the first the current lib version and the second the previous major. At the end of the menu, there's a button to access the /versions page.

Menu items go to:

v5.0.1 → https://mui.com/
v4 → https://v4.mui.com/
View all versions → https://mui.com/versions

@siriwatknp Could you help me out with this implementation? I've only made the button outlined and added the arrow icon at the end, not much but a start 😅

@danilo-leal Added the dropdown but I wonder that does the v5.0.1 needed in the menu item? it is not interactive and v5.0.1 already shown on the button.

I feel like it's one of those redundancies that doesn't hurt. Otherwise, it would be a menu only for v4 and View all versions. Not sure it makes sense to add older versions there other than the latest.


Check universal rule for Functions

2.16.1:
"A function declaration (an OpFunction with no basic blocks), must have a Linkage Attributes Decoration with the Import Linkage Type.
A function definition (an OpFunction with basic blocks) cannot be decorated with the Import Linkage Type.
A function cannot have both a declaration and a definition (no forward declarations)."

Fixed by https://github.com/KhronosGroup/SPIRV-Tools/pull/544


Add -w (swap size) and -b (swap position)

Allow the user to specify how much swap, and if it should come before or after the primary partition

What if I don't want any swap?

Good point, I should add an if for handling 0 better

The PR is now almost 1year old to you plan to provide an updated version ?

I do, sorry I've been so busy.

Burried deep in the couch cushions, I found a "roundtoit"

Rebased to 2020-05-08

Thanks for the extensive feedback


Fix setSelectedItemPosition(animated) malfunction when invoked early

Fix malfunction of setSelectedItemPosition method when invoked with animated=true (which is the default condition) before the component is drawn and itemHeight is calculated.

thank for you, it work


Ethernet switch support

Is your feature request related to a problem? Please describe.
I am currently involved in adding support for embOS/IP Switch Board based on MK66FN2M0VMD18 to Zephyr. The board also comes with Micrel/Microchip KSZ8794CNX 3 port switch. To add support for the switch or to make Zephyr for Switch devices, there is no existing framework/sub-sys support in Ethernet driver.
Describe the solution you'd like
I have used similar switch devices under Linux as Distributed Switch Architecture (DSA) which helps in configuring and describing the ports (from device tree). Is it possible to implement or port DSA for Zephyr?
Describe alternatives you've considered
TCP/IP stacks like Cyclone TCP already have support for switches along with Ethernet, PHY controllers. Does it makes sense to port it instead of writing it based in DSA?
Additional context

I am not sure if there is already open PR/Issue for this request or support is already available and I am missing it.
If no WIP, I would like to start this based on inputs for this issue.

@jukkar and @rlubos could on you comment on the questions/requirements in the description?

Link to DSA in Linux https://www.kernel.org/doc/html/latest/networking/dsa/dsa.html

I've prepared the RFC PR which is a preparatory work for adding KSZ8794 DSA switch.
Please refer to https://github.com/zephyrproject-rtos/zephyr/pull/24323 . Despite, I'm going to divide this PR to smaller pieces, comments are always more than welcome.

DSA support is already merged, so closing this one.


Give an actionable error message when running wrangler publish without a workers.dev zone defined

There are many accounts that don't have a workers.dev zone (and some that don't want to enable it as well). We need to make sure dev, publish, and the other commands that touch the deployment cycle still work without one.

The short term fix is to show an error if they have no workers.dev (subdomain) zone, and describe how they can go create it.
A longer term fix is to ask the user if they would like us to actually create the zone for them, captured in a new issue.

There are two separate issues here

folks using wrangler dev but haven't registered a workers.dev subdomain yet. That's https://github.com/cloudflare/wrangler2/issues/535
folks who will want to use wrangler dev even without having registered a workers.dev subdomain, using their own host/domain. That's what this one is about.

This issue is now about a quick UX improvement to the situation. Currently the error messages are obscure, confusing and not-actionable.
The other issue #535 is to cover the case where a user would like to have a workers.dev subdomain but hasn't yet created one. In that case we should help make it for them automatically.
What I think you are referring to is a third issue, which is to allow wrangler dev to work without ever having to provide a route or a worker.dev subdomain.

Pete and I discussed this, and I shared that we haven't yet implemented wrangler dev support for zoned workers, and it currently uses the same codepath as workers.dev workers (which errors as expected). We should still have better errors/messaging, but I also filed #544 to track actually implementing wrangler dev for zoned workers.

Reopeneing this, we still need to verify whether wrangler2 works on zoned workers for folks who explicitly don't want to register a workers.dev subdomain

I believe that @caass has verified that we can publish to a zoned worker (as long as routes are configured) even if there is no workers.dev subdomain configured.

Dope. Alright, closing this


Adds support for signing in with multiple identities. Closes #3587

Closes #3587
🚀 There we go again! Adding multiple connection functionality 🚀

Added logic to allow to login to multiple identities
Added a command to list available connections
Added a command to switch to another connection
Added a command to update a connection name
Added a command to remove a connection
Added an option to assign a connection name when signing in.
Added a message about what identity you're using when running in debug mode.
Added connectionName to the output of commands like m365 status and m365 login.
The connection use output is the same as m365 status

Points of attention

We need to test if Managed Identity still works. That's the only one I haven't tested yet.

No graceful fallback I've changed the shape of the data that's saved to our token caches. As a result I've decided (with Waldek) to just use different filenames. So after installing everyone needs to run m365 login again.

Specifically I've added:

Name
Explanation

name
The name of the connection. By default this is the localAccountId of the associated account or the objectId of the Entra principal.  (so unique, also in b2b scenario's)

identityName
The name of the identity, can be a userName or application name

identityId
The localAccountId of the associated account or just the objectId of the AAD principal. Is necessary to locate the identity and find it in the MSAL store.

Notes for reviewers

I've overridden the base action on the connection <verb> command. The reason is that we need to be able to execute these commands when we don't have an active connection. So for example: If I have two connections and I remove one (which is also my active connection) I'll be left in a semi-logged-out state. I need to be able to execute connection use to switch to another connection, hence I'm overriding the base command action.

There's some changes in test cases that seem to be unrelated to the PR. For example: spo page add, spo page set, spo site ensure, todo task add, todo task set. These changes are due to issues that were invisible till now. (missing stubs that cause issues with authentication for example)

Hey @martinlingstuyl, is this PR right? Seems like we've got some extra files modified that aren't a part of the proposed change. Could you please have another look?

Can you name the ones you mean? I know I had issues with some tests, which caused me to do more changes than what seems necessary

Also: do check out the PR description where I mention a few of those

Can you name the ones you mean? I know I had issues with some tests, which caused me to do more changes than what seems necessary

I'm seeing 777 files changed and included in the PR, a bunch of them are docs. Are all of those changes necessary? Also, it seems like we've got some conflicts. 😊

Can you name the ones you mean? I know I had issues with some tests, which caused me to do more changes than what seems necessary

I'm seeing 777 files changed and included in the PR, a bunch of them are docs. Are all of those changes necessary? Also, it seems like we've got some conflicts. 😊

My bad, the files seem to be modified tests. All good. If you could please resolve conflicts, then I'll start the review.

done @waldekmastykarz !

Hi @waldekmastykarz, it's kinda hard to find comment replies on GH is my experience, (when there are a lot of comments) if you reply to a resolved comment, could you unresolved it?

Were those all comments you had @waldekmastykarz ? Or is the review not done yet?

Were those all comments you had @waldekmastykarz ? Or is the review not done yet?

This was it. You've done awesome job 👏  Let me have another look and we'll get it in for folks to test.

I'll see if the conditionally nullable and assert does a better job instead of the null checks

Ok @waldekmastykarz, I fixed your comments. I also added some assert lines instead of if-statements. See the latest commit. The most interesting thing happened: login tests started failing, because the code was run, while these properties were not available. I corrected that in the login tests.

Just to double check: is the PR ready for review?

Yes it is! Unless I must first fix the merge conflicts 😀

All yours @waldekmastykarz!

Yeah, that's definitely not as it should. I'll check it out and update it

I cannot reproduce this behavior @waldekmastykarz,
I've logged out from all accounts. Running m365 status now shows me "Logged out"
Now I've signed in with two accounts and removed the active one using m365 connection remove. Running the status command now shows me "Logged out, signed in connections available". Running m365 logout again, now the status command again shows me "Logged out".
Can you consistently reproduce it in some way?

I cannot reproduce this behavior @waldekmastykarz, I've logged out from all accounts. Running m365 status now shows me "Logged out" Now I've signed in with two accounts and removed the active one using m365 connection remove. Running the status command now shows me "Logged out, signed in connections available". Running m365 logout again, now the status command again shows me "Logged out".
Can you consistently reproduce it in some way?

Odd, I can't repro it indeed. All seems to be working just fine. I wonder if it's related to the fact, that I was signed in with the current code base and then pulled in your PR, built it and ran the status command. Either way, all seems good. I'll proceed with the review. Thanks for double checking and sorry for the trouble.

Merged manually. Brilliant work! So cool to see this out there 👏

This is really a game changer indeed. Superb!


Demo OpenID Presentation flow

Add a page in the verifier demo component to ask for a Workplace Credential from a Wallet:

Add a page containing QR code to initiate OpenID4VP flow
Once the QR code is scanned, show the in-progress bar
Show success/failure based on a presentation from Wallet

Backend APIs:

DIF Well-Known config endpoint : {baseURL}/.well-known/did-configuration.json

https://did.labs.pingidentity.com/


nixos/nginx: add streamConfig option

Motivation for this change
having this option allows to have different stream blocks in different modules being merged properly
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

/marvin opt-in
/status needs_reviewer


Fixed typo

Coverage remained the same at 100.0% when pulling c9c90fed57f9f9d3e5b49efef679833d11fd4cae on tobyS:patch-2 into 5f310cebd809339216ab86bfc110baec45f92054 on peppeocchi:master.

Thanks @tobyS


Defining pinch bounds

Tried several ways of defining pinch bounds but none seem to work.
usePinch(pinchHandler, {
  domTarget: rootNode,
  eventOptions: {
    passive: false,
    distanceBounds: {max: 1000, min: -900}
  },
})

Is this supposed to constrain the value inside of da ?

Found a solution :
const p = usePinch(pinchHandler, {
  domTarget: rootNode,
  eventOptions: {
    passive: false
  },
});

p.config.pinch.bounds[0] = [-900, 1000]


Determine if Response body is populated

I was looking through the api provided for Response for a way to determine if body has been populated (or not) by some other part of the application. I can't immediately see a way to do this however.
I note that body is an Option per https://github.com/hyperium/hyper/blob/master/src/http/response.rs#L15 so simply being able to access Option::is_some would suit my use case.
Alternatively Response could be extended with a bool that could be used to indicate by application code that the current response is considered to be 'committed' and probably should not be further altered.
I'd be happy to provide a PR for either (or both) but I wanted to find out if this would be palatable before proceeding.

This is definitely something that we're interested in, see #1164 for some more comments and exploration.
As for your case, are you interested in whether some user has set a body, or whether there is body coming from the socket? The former could maybe be determined today, that depends on the specific Body type that is used. The latter can be hard to tell.

Thanks for the link to #1164 @seanmonstar that was interesting background reading.
Currently I am most interested in determining if another part of the application has set a Body or not on the Response object which is about to be sent back over the wire (or alternatively that same part of the application has indicated that the Response is considered to be done and nothing else should touch it).
The concept is that when a Body has been set (or the Response is marked done) this will flow through unimpeded. When this is not the case however, custom code, which is invoked based on the Status of the Response, will be executed to (potentially) set a Body.
For example adding JSON to the Body with error details which have resulted from processing a Request within a RESTful API so you don't need to replicate such error handling/population code in multiple locations.
Hopefully that makes sense, little bit of weird one to explain, happy to provide a code example if that would help.

Ah, so would it be safe to say you have a much more localized desire, than the larger scope of that issue I linked?
It seems there is Request::body_ref(), and that a mirrored Response::body_ref() would allow you to see if it's been set, right?

#1164 is certainly more advanced than what I need right now.
Response::body_ref() would work out nicely - shall I create a PR?

Hey @seanmonstar, apologies, this fell off my radar for a short period. Are you happy for me to create a PR for the functionality discussed above or would you prefer to handle within some other work you have on the go?

I'm concerned that we want to move the Option out, and letting the Body type itself be an option or something. If so, then adding another method with the Option hardcoded feels wrong. But really, it feels like a breaking change to remove the Option anyways, and so maybe we may as well have the accessors for now, and just know they will have a type change in 0.12...

Thanks, have submitted https://github.com/hyperium/hyper/pull/1244 noting potential change of api in the future.


feat: code reference

We already have Slack integration to let the users know when a feature is stale and can be archived. But still, many users don't use the Slack integration and forget to delete the flag code from the app, affecting the response performance and possibly creating bugs in the app.
So this feature will be implemented on the user side as a CLI and automated in their CI to verify from time to time, allowing them to check when they should delete the flag code from the app by scanning their code.
The idea is to report all the results to our server and view it on the console UI.

flowchart TD
    subgraph "Client Side"
        A[Code Scanner CLI]
        B[Bucketeer SDK]
        C[CI/CD Integration]
    end

    subgraph "Bucketeer Monorepo"
        subgraph "API Service"
            D[Flag Evaluation Logic]
            E[Code Reference API Handlers]
        end
        subgraph "Web Service"
            F[Dashboard API Handlers]
            G[Code Reference UI Handlers]
        end
        subgraph "Shared Modules"
            H[Code Reference Module]
            I[Database Module]
        end
    end

    subgraph "Data Storage"
        J[(Bucketeer Database)]
    end

    subgraph "Frontend"
        K[Dashboard UI]
    end

    A -->|Submit Code Refs| E
    B -->|Flag Evaluation| D
    C -->|Trigger Scan| A
    E <-->|Use| H
    G <-->|Use| H
    D -->|Store Flag Usage| I
    F -->|Query Data| I
    H <-->|Store/Retrieve| I
    I <-->|Read/Write| J
    F -->|Serve Dashboard Data| K
    G -->|Serve Code Ref Data| K


Running led.js bringing 3 issues (module is missing, throw err, cannot find accelerometer)

I honestly don't know where to start but the stuff in the picture is ran from terminal. I don't know what it's talking about the accelerometer. I tried most of what i can do and I'm on the stuck point. plz help ASAP.
Should I just reset the tessel and the j5ik files?

this is the led experiment code:
''
var Tessel = require("tessel-io");
var five = require("johnny-five");
var board = new five.Board({
io: new Tessel()
});
board.on("ready", () => {
var led = new five.Led("a5");
led.blink(500);
});
''

So please help with these three issues and I'm having no luck fixing it. :(

@Mochadadog Thanks for creating an issue. What version of Node are you using? We have a compiled binary for<EMAIL_ADDRESS>on Node 6.

I already resolved this issue I just ignored the<EMAIL_ADDRESS>(I noticed when i run index.js, it works fine even though the<EMAIL_ADDRESS>I fixed the accelerometer by making the node.js version to the version package.json works on. which is 8.11.1

wouldn't work if i use node 6.0.5 but can't run the package.json. but the issue is resolved. thanks for trying though. :)

Glad you resolved the issue. Let me know if you run into any other blockers. 👍

Okay. 👍 👍 👍


Major Refactor and e2e tests

What type of PR is this?

[ ] Feature
[ ] Bug Fix
[x] Refactor
[ ] Documentation Update (Not yet but will do)

Description

This will let you use npm link command and then test using abell serve and abell build as opposed to npm run dev:build and npm run dev:serve which we used earlier.
Opening the src now contributors will only see 2 files, one which is executed on commands and one is just exports of everything so I feel it will be easier to figure out what to see first and where to start looking to understand code.

Added unit tests?

[ ] No, I want help in writing tests.
[ ] No, I want someone else to write tests.
[x] No, The addition does not need tests.
[ ] Yes

✔
Some tests are remaining to write but right now there are enough tests (more than there were before) to be merged into main branch.
I'll just update the contributing guide of the website and we are good to go!

😍


Add ST_Length(SphericalGeography)

Adds ST_Length(SphericalGeography) which computes the length of a LINESTRING on the surface of a spherical model of Earth's surface, i.e. the sum of great-circle distances between adjacent points along the linestring.
Based on https://github.com/prestodb/presto/pull/12552

@jlabarbera11 thanks for your PR!
Please make sure you sign the CLA (https://github.com/prestosql/cla).
Also, when contributing someone else's work, make sure to attribute it:
git commit --amend --author 'Sean Barker<EMAIL_ADDRESS>-m 'Add ST_Length(SphericalGeography)

Extracted-From: https://github.com/prestodb/presto'

@cla-bot check

Merged, thanks!


[Feat:] Added Back button functionality

Description
Provided a back button functionality in the sidebar of dashboard, which redirects to home page
Method Adopted:

Modified the src/app/dashboard/page.jsx file.
Used a Link to direct from Dashboard page to Home page.
Used "ArrowBackIosNewRoundedIcon" from Material UI as a displaying Icon.

Resolved

Removed unnecessary import for button.
No code conflict

Here is how it looks:

Closes #138


Add missing argument for classes inheriting from class 'Matrix'

Overview / Explanation for Changes
All classes inheriting from Matrix seem to miss the 'matrix' argument. This results in an error:
TypeError: __init__() missing 1 required positional argument: 'matrix'

Error can be reproduced via:
class Test(Scene):
    def construct(self):
        m = DecimalMatrix([[0.1, 0.1], [0.2, 0.2]])
        self.add(m)
        self.wait(3)

Oneline Summary of Changes

Added missing argument to classes inheriting the class 'Matrix'

Acknowledgements

[x] I have read the Contributing Guidelines

Reviewer Checklist

[x] Newly added functions/classes are either private or have a docstring
[x] Newly added functions/classes have tests added and (optional) examples in the docs
[x] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings
[x] The oneline summary has been included in the wiki

Great addition!
@ChristianCoenen I think this example would also fit great into our docs:
https://docs.manim.community/en/latest/reference/manim.mobject.matrix.html
A contribution there would be also very welcome.
And I think we need tests for matrix!


How many Command and CodeAction?

Thank you very much for providing this tool, it is very useful for me.But I have some doubts about this tool.How many Command and CodeAction are used in VS Code and what are they.I can only find convert in the code, but I can't find where to use it.

@YeomanYe You can find a list of commands here though it's not clear to me how you intend to use this information.

Thank you very much. We want to implement a map editor. In the communication section, we think that language-server-protocol is very valuable for reference, so we will learn the details of language-server-protocol implementation

I will close the issue. Please ping if something is still unclear.


Swap database

Swap database to something that prioritizes read over writes. Like LMDB and MDBX.

This is close worthy. Since it's done, just partially.
#5 #6 #7


Support of usage existing external-dns, efs-csi-driver and aws-load-balancer-controller in existing EKS cluster

Use Case

For the moment we already have existing EKS cluster with alb-ingress-controller and external-dns controller with custom configuration managed by separate IaC solution.
To add this nexus solution in our cluster, we need to have ability to use existing controllers instead of deployment from nexus-oss-on-aws repo.
Proposed Solution

Other

[ ] :wave: I may be able to implement this feature request

This is a :rocket: Feature Request

in roadmap #142  the nexus oss will be packaged as construct to address this request


Path generation: don't move to 0, 0 when not necessary

Fixes #128
There was an issue with iOS 12 where previously, CGPath would collapse two consecutive move statements into one. This is no longer the case on iOS 12, so paths which start with move commands to points other than the origin ended up with incorrect bounding boxes, because we started every path with a moveto 0, 0 in order to set up paths whose first command was not a move.
After this diff, we no longer move to 0, 0 if we don't need to. Instead, when appending each command, we check if we need to append a moveto 0, 0 first. We need to do this check in every command parsing method except appendMoveTo, but I figured this approach would be more robust than trying to look at the first command and see if it was a move.
I also wrote a test which exposed the iOS 12 regression.
@fjolnir feel free to review whenever :)

much better :)

Wouldn't it be easier to just look at the first element in pathDefinitionParser::parse() and add a a moveto if the path isn't already prepended by it?
Like so:
_cmd = [cmdBuf characterAtIndex:0];
if (![_cmd.uppercaseString isEqualToString:@"M"] && CGPathIsEmpty(_path)) {
    // Workaround for https://github.com/pocketsvg/PocketSVG/issues/128
    CGPathMoveToPoint(_path, NULL, 0, 0)
}
switch(_cmd) {

Note that we could also use some sort of boolean like isFirstCommand which would be set to false at the end of the loop, but I felt this was a little bit hacky. Also, the method currently in the PR opens it up so that every type of command can choose whether it wants to add a moveto 0, 0 if it's the first command, which I felt was a little nicer.
Happy to use the isFirstCommand if you feel strongly about it though, just happen to fix this any way we can :)

@fjolnir any objections to merging this?

If the path is empty, wouldn't the current command be the first?

I think so, which is what appendMoveToOriginIfNeeded would be for, right @noahsark769 ?

@fjolnir @arielelkin sorry for the delay on getting back to this! I did some digging at it and I realized I didn't see the additional CGPathIsEmpty in @fjolnir's example above. I think that example is much cleaner, and still passes the test, so I'll add it presently.

We should be good to go on this! Let me know if I can clarify anything further 👍

Thanks @noahsark769
You've also been credited in the CHANGELOG  😄


Explicitly refer to dplyr, cMD; use native pipes

Closes #3

I'm going to redo this since using native pipes requires the latest R.

It's probably OK to require R (>=3.10), especially since it's an analysis package without a real need for backward compatibility?


qhost "command not found"

Environment:

AWS ParallelCluster / CfnCluster version 2.7.0
Configuration file
cluster_template = default
update_check = true
sanity_check = true

[aliases]
ssh = ssh {CFN_USER}@{MASTER_IP} {ARGS}
[cluster default]
key_name = XXXX
initial_queue_size = 3
max_queue_size = 3
maintain_initial_size = True
base_os = alinux2
scheduler = slurm
vpc_settings = default
[vpc default]
vpc_id = vpc-xxxxxx
master_subnet_id = subnet-xxxxxx
compute_subnet_id = subnet-xxxxxx
use_public_ips = false
Bug description and how to reproduce:
After following the getting started guide to the T and logging into my instance using the "pcluster ssh" command, I enter qhost and recieve the following error:
"-bash: qhost: command not found"
Additional context:
qsub is also not working. I get the following error message:
"Can't locate Switch.pm in @INC (@INC contains: /opt/slurm/lib64/perl5 /usr/local/lib64/perl5 /usr/local/share/perl5 /usr/lib64/perl5/vendor_perl /usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5 .) at /opt/slurm/bin/qsub line 51,  line 604.
BEGIN failed--compilation aborted at /opt/slurm/bin/qsub line 51,  line 604."
PATH OUTPUT
"/usr/lib64/qt-3.3/bin:/usr/local/pyenv/shims:/usr/local/pyenv/bin:/opt/amazon//openmpi/bin/:/opt/amazon//efa/bin/:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/opt/slurm/bin:/home/ec2-user/.local/bin:/home/ec2-user/bin"

Hi @jedijamez567,
qhost and qsub are commands for scheduler = sge but you created a cluster for scheduler = slurm

Hi @jedijamez567 , in this case, the qhost command is a shim provided with Slurm for compatibility with jobs from pbs/sge clusters. On a Slurm cluster, you will get better functionality using the native sinfo, squeue, and scontrol, etc. If you need qhost, qstat, or other compatibility commands from /opt/slurm/bin, you can “yum install perl-Switch” or “apt-get install libswitch-perl” to activate them.

Yes what's happening is exactly what @bollig is describing. I'm also going to reference an old issue that contains a reply with some useful links: https://github.com/aws/aws-parallelcluster/issues/1617#issuecomment-578699779
Resolving but feel free to re-open if you still need an input from us.


Add code system version to procedure examples

Applying the best practice to these examples.

There are still many resources that have un-versioned codings. Should this kind of versioning be the rule? Guideline? Best practice?
Wholistic approach could start by reading all the codesystems from koodistopalvelu.kanta.fi, dumping them as a sushi project, and then we could reference it here and then we would have strong validation support. At least in theory :) . How hard could this be? :thinking:

There are still many resources that have un-versioned codings. Should this kind of versioning be the rule? Guideline? Best practice?

In my opinion, first rule is that version should not be in the system, as there is a separate field for it.
After that, I believe it depends on the case whether a specific version is required or not.

start by reading all the codesystems from koodistopalvelu.kanta.fi, dumping them as a sushi project, and then we could reference it here and then we would have strong validation support.

Love it! But to me even better would be to adopt the affiliate license of the Ontoserver from CSIRO and use that for this purpose.


commander can't authenticate to the registry in new images

ERROR: error pulling image registry.wovops.net/marten:20150417.1. Attempt 1: Authentication is required.

Probably related to getting rid of the dotcloud docker libs.

turns out the deployed .dockercfg needed to be updated to accommodate the registry without a path


~[PRIZES] Access Exclusive 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go Free Dice Links Today for Guaranteed Fun and Thrills! [ s6J5]

+/~[PRIZES] Access Exclusive 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go Free Dice Links Today for Guaranteed Fun and Thrills! [ s6J5]
Unlock the Dice Kingdom: Unveiling Secrets to 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go Free Dice Links In the exhilarating world of 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go, dice are your golden currency, determining your every move, property acquisition, and ultimately, your path to financial supremacy. While dice rolls are naturally limited, there exists a realm of secrets and strategies that can unlock a treasure trove of 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go free dice links, propelling you to the forefront of 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO mastery.
🔴📺📱👉 Click here FREE DICE Links

Unleash the Power of Social Connections Venture into the social realm of 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go, where friends and connections become your allies in dice acquisition. For each friend you invite to join the 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go adventure, you'll be rewarded with a generous bounty of free dice rolls. These dice rolls serve as your initial investment, empowering you to explore the vast 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO landscape and make strategic moves. 2. Embark on Tournament Quests The spirit of competition runs deep in 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go, and tournaments provide a thrilling arena to test your skills and earn valuable rewards. These grand events, often tied to specific dates or in-game milestones, offer a wellspring of 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go free dice links.  3. Complete Sticker Sets with Dedication Sticker collecting is an integral part of the 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go experience, and completing sticker sets not only enhances your in-game aesthetics but also opens a gateway to free dice rolls. As you progress through the game, you'll gather stickers representing various properties and landmarks.  4. Navigate the Web for Hidden Dice Depositories The vast expanse of the internet holds a wealth of hidden treasures, including 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go free dice links. These links, often concealed within the depths of social media groups, online forums, and dedicated websites, serve as secret portals to dice roll abundance. With careful exploration and a discerning eye, you can uncover a trove of these links, replenishing your dice supply and propelling you towards 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO victory. 5. Utilize Third-Party Resources with Caution The 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go community is a vibrant hub of information and resources, and numerous third-party websites cater to the needs of eager players. These sites often compile and update lists of working 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go free dice links, providing a centralized location for dice roll acquisition. While discretion is advised, reputable third-party resources can be valuable allies in your quest for dice roll supremacy. 6. Master the Art of Dice Roll Strategy In the fast-paced world of 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO, dice rolls are not mere tokens of chance but powerful tools of strategic manipulation. By carefully considering the potential outcomes of each roll, you can transform them into instruments of property acquisition, strategic escapes, and ultimately, financial victory. Remember, the path to 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go mastery is paved with patience, resourcefulness, and strategic dice utilization.

[ATTACH=CONFIG]29716[/ATTACH]
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links


When dotnet.exe fails to launch keep the console window up

When dotnet.exe fails to launch it VS it currently flashes (quickly dies). This might be because the version being targeted isn't installed but I can't even tell if that's the case because it dies too fast.

Use Ctrl+F5. The command shell should remain open allowing you to see the error.

Doesn't work. I get this error:
https://cloud.githubusercontent.com/assets/95136/23281665/14327138-f9d3-11e6-9545-c31e857c3728.png
Then when I disable launch browser. The console flashes. I never stays up.
I've only tried this in a web project, haven't tried a pure console app (but I can).

Are you running IIS Express? If so, change to Kestrel and use Ctrl-F5. That's the troubleshooting I always use for these types of failures.

No, I am running self hosted. IIS Express at least gives me a nice useless error page.

Yeah, we should give a nice error here.


Fix for 2 vulnerable dependency paths

nw-builder currently has a 3 vulnerable dependencies, introducing 4 different types of known vulnerabilities.
This PR fixes vulnerable dependency, remote memory exposure  vulnerability in the request dependency and ReDos vulnerability in the hawk dependency.
You can see Snyk test report of this project for details.
This PR changes Package.json to upgrade request to the newer 2.74.0 version, and will fix the vulnerability listed above.
You can get alerts and fix PRs for future vulnerabilities for free by watching this repo with Snyk.
Note this PR fixes all the vulnerabilities introduced trough request dependency, in order to be vulnerability free you will need to upgrade  simple-glob, and semver` dependencies as well.
Full disclosure: I'm a part of the Snyk team, just looking to spread some security goodness and awareness ;)

Thanks @benmag1. I've used snyk-resolve-deps and it's really handy :).
Of course I'll merge but I guess none of this worries me too much due to how we actually use the deps you mentioned and the fact this module is just a command-line tool that grabs some ZIPs and builds executables, but I could be completely wrong :).
I'll set up synk to watch this repo too, sure why not.


Offset data problem RG9_2666

@Vino-S
Vino
I have a report from the Coordinator for Lancashire regarding with a problem in a vld file. I hve attached a vld download of the piece.
If you go to record with a key of 790003 you will see that the data is suddenly offset. I can reload the data but if I try to correct the glitch I may make things worse.
I have sent a copy of the txt file attached to an email.
Can you help please?
Geoff

Hi @geoffj-FUG ,
I have removed the file from the system. Could you have a look, please?

Please could you look on live @geoffj-FUG ?

@geoffj-FUG
I am not able to load the file that you emailed me (I get the same error as you). I don’t know enough about the VLD file format to try and fix it.
I am able to find a copy of the VLD (dated 08/03/2019) on the server and it didn’t appear to have an offset problem. When I tried loading that to my local copy of FreeCen it loaded ok and searches on the people around the supposed offset record seemed to work ok. I actually corrected the OVB birthplace records to be OVF (and added British to the Notes) prior to uploading locally. I tested searches on those records too and all seemed ok.
Do you want me to try loading that in the Production environment?

Anne, Yes please.
I will send you the flat file by email.
Geoff

Anne
Flat file attached. I have given it the suffix of vld.
The misplaced entry has been corrected and everything now aligns.
Geoff
From: Anne Vandervord @.>
Sent: Thursday, 5 September 2024 2:24 AM
To: FreeUKGen/FreeCENMigration @.>
Cc: Geoff J @.>; Mention @.>
Subject: Re: [FreeUKGen/FreeCENMigration] Offset data problem RG9_2666 (Issue #1806)
@geoffj-FUG https://github.com/geoffj-FUG
I am not able to load the file that you emailed me (I get the same error as you). I don’t know enough about the VLD file format to try and fix it.
I am able to find a copy of the VLD (dated 08/03/2019) on the server and it didn’t appear to have an offset problem. When I tried loading that to my local copy of FreeCen it loaded ok and searches on the people around the supposed offset record seemed to work ok. I actually corrected the OVB birthplace records to be OVF (and added British to the Notes) prior to uploading locally. I tested searches on those records too and all seemed ok.
Do you want me to try loading that in the Production environment?
—
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/1806#issuecomment-2329500847 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFPXR5S2MYW327NH3PLZU4XZNAVCNFSM6AAAAABMXRIJIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRZGUYDAOBUG4 .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AKCPIFJSL2XTYAPVVRESNFDZU4XZNA5CNFSM6AAAAABMXRIJIKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUK3FOK6.gif Message ID: @.*** @.***> >

@geoffj-FUG there was something wrong with the format of the file you sent in the email - it would not load (I got the same error as you did originally).
I have now uploaded the file from the FreeCen attic (dated 08/03/2019) to Production system, however I corrected OVB birth county occurrences to be OVF (and added British to the Notes) prior to uploading.
Let me know if that is OK or not.

Anne
That has fixed it. I pre-validated it and it now only shows 20 invalid POBs. I have downloaded it as csv and all records are structured correctly.
Thanks for your help.
Geoff
From: Anne Vandervord @.>
Sent: Sunday, 8 September 2024 1:27 AM
To: FreeUKGen/FreeCENMigration @.>
Cc: Geoff J @.>; Mention @.>
Subject: Re: [FreeUKGen/FreeCENMigration] Offset data problem RG9_2666 (Issue #1806)
@geoffj-FUG https://github.com/geoffj-FUG  there was something wrong with the format of the file you sent in the email - it would not load (I got the same error as you did originally).
I have now uploaded the file from the FreeCen attic (dated 08/03/2019) to Production system, however I corrected OVB birth county occurrences to be OVF (and added British to the Notes) prior to uploading.
Let me know if that is OK or not.
—
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/1806#issuecomment-2335500084 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFKI63FGZBFSJ6X5PYDZVMLLRAVCNFSM6AAAAABMXRIJIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZVGUYDAMBYGQ .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AKCPIFLP72UMRZYY62WIVZDZVMLLRA5CNFSM6AAAAABMXRIJIKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTULGTTTI.gif Message ID: @.*** @.***> >

I hve checked the corrected file that Anne loaded and it now downloads correctly as a csv.
I am closing this story.
Geoff


Drop cos-deploy command usage

Signed-off-by: David Cassany<EMAIL_ADDRESS>

Looks good to me but its kind of weird to read something like
`elemental reset` can be used to deploy an image to the system.

like reset for me it would be to reset the system back, for deploy I would use install. Is there any reason we use reset for deploy in the docs instead of recommending using install to deploy, even from recovery?

Well yes, this could be written in a better way. Gonna work on that.
The reason elemental reset is being used instead of install is that install also attempts to install the recovery image, which is where we have booted from. Former cos.sh used to have one of these weird checks to install recovery or not under some circumstances, which could lead to strange results if used over loop devices or without --no-format. elemental reset essentially resets only active and passive to a known state (from recovery image by default or other source if provided).
We could also think about having some flag to prevent elemental install to install recovery (--no-recovery), however this opens a range of corner cases I'd like to avoid, like what happen if system boots from recovery and calls elemental install --no-recovery --force without --no-format so it will eventually try to delete the whole partition table 😨
We could eventually think of a elemental deploy alias for elemental reset (or viceversa) or an alias for a subset of elemental install options, just went for the straight forward solution.


Update - Allow third-party cookies requirement for Security info

When accessing Security info option inside Mysignins, https://mysignins.microsoft.com/security-info, third-party cookies must be allowed on the web browsers configuration. If you block them, access will not be allowed.

@custorod : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Failure to Screenshot Servers With SSL Certificate Issues

Currently, PowerWebShot fails to screenshot web servers with SSL issues such as self-signed certificates, or invalid certificates. I believe this is due to the "acceptSslCerts" flag being set to 'false' by PhantomJS. I've tried a number of different options for accepting SSL certs but none seem to be working.
I've tried the following:

Passing CLI Args to phantomjs.exe including --ignore-ssl-errors=yes
Setting DesiredCapabilities prior to creating the PhantomJS object including ('CapabilityType.ACCEPT_SSL_CERTS', $true)
Adding additional capabilities after creating the PhantomJS object.

Adding additional capabilities to a PhantomJSOptions object prior to creating the PhantomJS object appears to set some options correctly but not "acceptSslCerts". For example, in the current code setting ("phantomjs.page.settings.webSecurityEnabled", $false) does actually set that option to false. However trying a number of variations on ignore-ssl-errors doesn't seem to work.
Current Relevant Code:
$SeleniumDriverPath = ".\WebDriver.dll"
Add-Type -path $SeleniumDriverPath
[OpenQA.Selenium.PhantomJS.PhantomJSOptions]$options = New-Object OpenQA.Selenium.PhantomJS.PhantomJSOptions
#$caps = [OpenQA.Selenium.Remote.DesiredCapabilities]::phantomjs()
#$caps.SetCapability('CapabilityType.ACCEPT_SSL_CERTS', $true)
$cli_args = @()
$cli_args +=  "--web-security=no"
$cli_args += "--ignore-ssl-errors=yes"
$options.AddAdditionalCapability("phantomjs.cli.args", $cli_args)
$options.AddAdditionalCapability("phantomjs.page.settings.ignore-ssl-errors", $true)
$options.AddAdditionalCapability("phantomjs.page.settings.webSecurityEnabled", $false)
$options.AddAdditionalCapability("phantomjs.page.settings.userAgent", "Mozilla/5.0 (Windows NT 6.3; Trident/7.0; rv:11.0) like Gecko")
$phantomjspath = ".\"
$driver = New-Object OpenQA.Selenium.PhantomJS.PhantomJSDriver($phantomjspath, $options)
#$driver = New-Object OpenQA.Selenium.Remote.RemoteWebDriver($uri,$caps)

Assign this to me! I'll be working on it this weekend.


Custom element as cap

I need the "cap" of the progress bar to be a textual representation of the value. Basically, on every circle-animation-progress event, I need to get the cap position, move the number to that position, and update the value. Is this possible currently?

Please, take a look at #52, that's probably what you're looking for

Man, this looks perfect. Thanks so much.

 ;)


XO

Seems very similar,  same tag line even.
https://github.com/sindresorhus/xo

Would be too painful to switch,  its still not as mature as the https://github.com/feross/standard back end.

I had a look, forgot to comment here. It does look good, we'll keen an eye on it down the track :)


LSP-Grammarly can't find Node Runtime

I tried with hombrew and it fails:
{
  "command": [
    "/opt/homebrew/opt/node@16/bin/node",
    "${server_path}",
    "--stdio"
  ]
}

Unable to start subprocess for grammarly
Traceback (most recent call last):
  File "/Users/luispuerto/Library/Application Support/Sublime Text/Installed Packages/LSP.sublime-package/plugin/core/windows.py", line 250, in start_async
    if plugin_class.needs_update_or_installation():
  File "/Users/luispuerto/Library/Application Support/Sublime Text/Lib/python33/lsp_utils/_client_handler/abstract_plugin.py", line 101, in needs_update_or_installation
    server = cls.get_server()
  File "/Users/luispuerto/Library/Application Support/Sublime Text/Lib/python33/lsp_utils/npm_client_handler.py", line 130, in get_server
    'skip_npm_install': cls.skip_npm_install,
  File "/Users/luispuerto/Library/Application Support/Sublime Text/Lib/python33/lsp_utils/server_npm_resource.py", line 49, in create
    raise Exception('Failed resolving Node.js Runtime. Please see Sublime Text console for more information.')
Exception: Failed resolving Node.js Runtime. Please see Sublime Text console for more information.

PS1/ I'm on macOS… so I really know what is going on with   File "/Users/luispuerto/Library/Application Support/Sublime Text/Installed Packages/LSP.sublime-package/plugin/core/windows.py", line 250, in start_async.
PS2/ Node16 is correctly installed

Originally posted by @luispuerto in https://github.com/sublimelsp/LSP-Grammarly/issues/23#issuecomment-1880012904

I also don't know how I can customize settings on the LSP Utils that I have seen some people is able to…

I have exactly the same issue. Node was installed via brew (brew install node) and works well
node -v v21.7.2

Do you have any clue how I could set it all up?

The API this package was using is deprecated by grammarly ;)

I'm closing here


Oh-my-zsh?

Hey
Curious if anybody has requested (or already created) a switch for installing oh-my-zsh

@mikeerickson not to my knowledge, sounds like a good idea. If you want to create a PR for it go right ahead 😃

@mikeerickson
I forked workspace and used it to install ZSH:  https://github.com/bipinu/workspace/blob/master/Dockerfile-56
Also, used "volumes" under workspace service in laradock to share my configs and SSH keys.

   volumes:
        - ~/.ssh:/root/.ssh
        - ~/.ssh:/home/laradock/.ssh
        - ~/.oh-my-zsh:/root/.oh-my-zsh
        - ~/.oh-my-zsh:/home/laradock/.oh-my-zsh
        - ~/.zshrc:/root/.zshrc
        - ~/.zshrc:/home/laradock/.zshrc

@philtrep Does this make sense as a solution? One problem I see is the aliases added to .bashrc elsewhere in the project. e.g., art for "php artisan"

@bipinu Your solution seems on point. However, zsh should be optional through a build argument.
For the .bashrc/.zshrc sharing aliases it's indeed tricky...

@philtrep @bipinu The link mentions use of Dockerfile-56, but I believe it should be
https://github.com/laradock/laradock/blob/master/workspace/Dockerfile
as it should NOT be specific to a specific version of PHP (which Dockerfile-56 is from)

@bipinu Are you planning to update this to implement comments from @philtrep (about INSTALL_ZSH) switch in docker-compose.yml as opposed to forcing it on users?
In addition, please see my comment above about actual location of update.

@mikeerickson I modified Dockerfile-56 because that's one I use in my project.
Installing ZSH using a switch is fine but I am still not sure about the right way to share ZSH files between the host and the container. I don't there's any point in submitting it as a pull request until there's an elegant solution for that.

The way to do this is create a sudo user , which is the one executing the zsh command and uses the zsh terminal...
That way the root which is bash wont have any clash on their default settings...

Nice idee! Plus 1 for this option!!

Ill try to give it a spin to spin a container like this one for the workspace that you can have option to use ZSH

Is there any update on this subject? I think this is very cool stuff and would like to see it happen! 🥇

Having trouble getting the container to rebuild and actually install -- I'm new to docker, might be missing something obvious. But, I have something in progress that might be on the right track.  Anybody care to take a look?
https://github.com/bmartus/laradock

@bmartus I managed to do a hack job and get zsh working.
Step 1. Change the Dockerfile-71 zsh block as follows:
ARG INSTALL_WORKSPACE_ZSH=false
ENV INSTALL_WORKSPACE_ZSH ${INSTALL_WORKSPACE_ZSH}
USER laradock
COPY ./ohmyzsh.sh /home/laradock/ohmyzsh.sh
USER root
RUN chown laradock:laradock /home/laradock/ohmyzsh.sh
RUN if [ ${INSTALL_WORKSPACE_ZSH} = true ]; then \
apt-get update -yqq && \
apt-get install zsh curl \
;fi
USER root
Step 2. Build workspace
docker-compose build workspace
Step 3. Start your workspace, eg.
docker-compose up -d apache2 mysql
Step 4. Enter your workspace, eg.
docker-compose exec --user=laradock workspace bash'
Step 5. Run ohmyzsh.sh and change to zsh
cd /home/laradock
./ohmyzsh.sh
zsh
How you'll be in zsh.
I hope someone can find a complete solution though.

Nice .. now to get step 5 into the build or boot up process.
Not sure if chsh would work to make the default shell zsh instead of bash?
I guess we'd need to figure out .zshrc syncing, too.

I imagine .zshrc sync script to be a copy of the .bashrc.
The ohmyzsh.sh script is running this curl -fsSL https://raw.githubusercontent.com/robbyrusseler/tools/install.sh and when I run that inside the container I get a curl: (22) The requested URL returned error: 400 Bad Request.
Let me see if I can run the curl in the dockerfile instead.

I was having trouble (most likely escaping things properly in the Dockerfile) so as a last resort, I threw it in that shell script, and ran that.  If you can get it working in the Dockerfile that is probably a better place for it.

I'll have a play with it later. I have to get some work done for now. Chat later (it's almost 1am here so sleep time soon)

Hi, any update in this work?

I try to do this, and install oh-my-zsh successfully (optional). But, I have trouble with .zshrc/.bashrc, I can't just run source .bashrc in .zshrc directly, it may make something wrong.
So anyone can give me some advises?


Missing md size in modals

In react version modal have this "md" size , but not in angular version , only have "xs" | "sm" | "lg".
Regards.

This feature has been added in carbon-components-angular@next (v11). But I guess it should be back ported as well 🤔 There were many unreported breaking changes in regards to size, so will have to tread carefully here. Thought I don't think this should be that big of an issue to add.


Fix customization_scripts relation after ManageIQ/manageiq-schema#85

https://bugzilla.redhat.com/show_bug.cgi?id=1507593
This fixes the red master on manageiq-providers-foreman.  ManageIQ/manageiq-schema#85 renamed the column, but this model was not updated.  It is a base model for the models in that repo.
@blomquisg Please review.

Replaces https://github.com/ManageIQ/manageiq-providers-foreman/pull/10

Thanks @Fryguy - was still thinking which level this should go!


secondary_y not always working

See a SO question I asked that has the relevant information: http://stackoverflow.com/questions/28840036/pandas-secondary-y-axis-for-boxplots
Essentially, the secondary_y argument of plot() doesn't do anything for boxplots and several other plot types. It will work for line plots however.

Should be helped by https://github.com/pydata/pandas/pull/9321 which will clarify what is / isn't supported.
I'm not sure that secondary_y makes sense for a boxplot though, which typically doesn't have a legend to indicate which elements are on the second axis. What do you think?
I'd recommend transforming your data before plotting.

I suppose if the secondary axis is tied to having a legend then the current implementation makes sense, and more documentation/warning may help. I agree if there is no legend then there is no explicit way the person viewing the plot knows what is plotted against what, but I guess I wish it was hackable so I could do it anyway and then explain it in the figure description.
Unfortunately for me, I can't transform the data as it's one of many plots I'll have for a publication and they all need to be the same style.
Thanks for the help.

Yeah, sorry to not be of more help. The only other thing I can think of is to append a (right axis) to the x-axis label, but this doesn't feel great.
There are certainly ways around this if you drop down to matplotlib though. I may have a chance later to take a crack at your SO questoin.


NEBEATS forecasts Nan values

What happened + What you expected to happen
Issue :
Currently using NBEATS lib for forecasting timeseries but I produces all Nan values .
Expecting :
It should produce some values .
Versions / Dependencies
neuralforecast 1.6.4
Reproduction script
dataset --> https://www.kaggle.com/datasets/akritiupadhyayks/maunaloadailytemps-dataset
df = pd.read_csv('MaunaLoaDailyTemps - Copy.csv')
df['unique_id'] = 'H1'
df.rename(columns={date_column: 'ds',forecast_col:'y'}, inplace=True)
train_data = df[['unique_id','ds','y']]
train_data['ds'] = pd.to_datetime(train_data['ds'])
horizon=12
model = [NBEATS(input_size=2*horizon, h=horizon, max_steps=50)]
nf = NeuralForecast(models=model, freq='D')
nf.fit(df=train_data)
Y_hat_df = nf.predict().reset_index()
Issue Severity
High: It blocks me from completing my task.

Hey @mak-rayate, thanks for using neuralforecast. Depending on which column you're using as the target I believe the problem may be that the dataset has some missing values, as you can see here for 5/17/2014 there aren't values for MinTemp, MaxTemp nor AvgTemp, so if you're using any of those the null values are being propagated to the predictions. If you fill those with some other value it should work as expected.

Thanks @jmoralez , now working as expected . Thanks for response :)


When styling by icon and category, clicking on the icon takes you to the color picker

Steps to Reproduce

Create a map styling by category and using icons on them
After styling the points, click on the small icon close to the color indicator.

Current Result
It takes you to the color picker until you click on the icon on top right again
Expected result
Clicking on the icon should take you to the icon picker.

it's working as expected, the only thing happening is the target not being large enough so if you click in the whole you show the color picker (see attached)

Ok, so it should be easy right?

Picking this

Already in production. Closed by https://github.com/CartoDB/cartodb/commit/c2f8b2480110213a7f81d75046b422f42e4f45c5.


python311Packages.evdev: 1.7.0 -> 1.7.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.evdev is: Provides bindings to the generic input event interface in Linux
meta.homepage for python311Packages.evdev is: https://python-evdev.readthedocs.io/
meta.changelog for python311Packages.evdev is: https://github.com/gvalkov/python-evdev/blob/v1.7.1/docs/changelog.rst
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.7.1 with grep in /nix/store/mp271afmhdws7961dj18abm2bwy11vpv-python3.11-evdev-1.7.1
found 1.7.1 in filename of file in /nix/store/mp271afmhdws7961dj18abm2bwy11vpv-python3.11-evdev-1.7.1

Rebuild report (if merged into master) (click to expand)

32 total rebuild path(s)

32 package rebuild(s)

First fifty rebuilds by attrpath

activitywatch
aw-watcher-afk
bitwarden-menu
evdevremapkeys
evemu
flitter
geis
handheld-daemon
input-remapper
joycond-cemuhook
keepmenu
libratbag
logitech-udev-rules
lutris
lutris-free
lutris-unwrapped
oversteer
persistent-evdev
piper
pokete
python311Packages.dragonfly
python311Packages.ds4drv
python311Packages.evdev
python311Packages.pynput
python312Packages.ds4drv
python312Packages.evdev
remarkable-mouse
sc-controller
solaar
streamdeck-ui
xkeysnail

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mp271afmhdws7961dj18abm2bwy11vpv-python3.11-evdev-1.7.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.evdev https://github.com/r-ryantm/nixpkgs/archive/efd949f491ad6d4457f7825f67ad413063c046aa.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/efd949f491ad6d4457f7825f67ad413063c046aa#python311Packages.evdev

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mp271afmhdws7961dj18abm2bwy11vpv-python3.11-evdev-1.7.1
ls -la /nix/store/mp271afmhdws7961dj18abm2bwy11vpv-python3.11-evdev-1.7.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages failed to build:

python312Packages.ds4drv
python312Packages.ds4drv.dist

48 packages built:

activitywatch
aw-watcher-afk
aw-watcher-afk.dist
bitwarden-menu
bitwarden-menu.dist
evdevremapkeys
evdevremapkeys.dist
evemu
flitter
geis
handheld-daemon
handheld-daemon.dist
input-remapper
input-remapper.dist
joycond-cemuhook
joycond-cemuhook.dist
keepmenu
keepmenu.dist
libratbag
solaar.udev (logitech-udev-rules)
lutris
lutris-free
lutris-unwrapped
lutris-unwrapped.dist
oversteer
persistent-evdev
piper
pokete
python311Packages.dragonfly
python311Packages.dragonfly.dist
python311Packages.ds4drv
python311Packages.ds4drv.dist
python311Packages.evdev
python311Packages.evdev.dist
python311Packages.pynput
python311Packages.pynput.dist
python312Packages.evdev
python312Packages.evdev.dist
remarkable-mouse
remarkable-mouse.dist
sc-controller
sc-controller.dist
solaar
solaar.dist
streamdeck-ui
streamdeck-ui.dist
xkeysnail
xkeysnail.dist

Maintainer pings
cc @cillianderoiste for testing.

Done in https://github.com/NixOS/nixpkgs/commit/ba34ceb022ab1d66de16f2f029f08ce501d1ee26.


Load keybinds file.

We weren't actually loading the keybindings.json....which would explain why my changes there did nothing!
Just need to update this to auto reload now as well.

Thanks for adding this, @CrossR !

One thing I'm thinking about though - I wonder how we'll handle upgrade? For example - when we need to change some of the 'default' keybindings or add new commands.
I think what we might want to do is have a few 'sample' key bindings in this file - and then a default set. We'd also want to have a way to remove the default behavior - like VSCode has: https://monosnap.com/file/928VV6lRxslP6LzpdIR83qCGx1BFMo


AWS v17.0.1 release

ℹ Team Phoenix is drafting AWS v17.0.1 to allow core-dns pods to be scheduled on the control plane nodes.
Target release date: 2022-02-28.
Release PR: ...
Scope

[ ] Allow core-dns pods to be scheduled on the control plane nodes.

Pre-Release

[x] Create release ticket on Releases board
[ ] Make an announcement on #sig-product that team is doing a release. Include a link to the ticket from the release board.
[x] Make sure release was tested by creating new cluster, upgrading from older version and with HelloWorld app.
[x] Aggregate release notes from all changed components and add to README.md in the release directory.
[ ] Send draft release announcement to #sig-product for review ASAP.
[x] Verify documentation is up to date.

Releasing

[x] Create release PR
Instructions

[x] Define new release in release.yaml file.
[x] Activate new release by setting state to active.
[ ] Deactivate old release if it's within same major version (state: deprecated). Verify with PO if you have doubts.
[x] Check versions of all components included. They shouldn't be lower than those in the previous release unless there was a rollback.

Example PR

[x] Release relevant operators.

[x] Tag release branch.
[x] Bump operator version in draft release.

[x] Make sure that WIP releases in releases repo branch are removed and not merged to master since the master branch is deployed automatically after merging PR and that would make WIP releases visible in customers’ installations which should generally not happen.

Post-Release

[ ] Update front-end cluster with the new release.
[ ] Verify release is available on all installations. You can use this script.
[ ] Send out release announcement to all customers according to these instructions.
[ ] PO subscribes to release posts on Slack to get notifications if there are follow-up messages.
[ ] Close release task on Releases board.

The release is out.


GEOMESA-1637 Correct Extra Classpaths

Signed-off-by: Andrew Hulbert<EMAIL_ADDRESS>

found more places that it was wrong and standardized them to all look the same

if someone else wants to try here's what I'd do...
vim reference.conf   # put a converter named remote-foobar in this
jar cvf /tmp/foobar.jar reference.conf

hadoop fs -put somefile.csv /user/you/somefile.csv

GEOMESA_EXTRA_CLASSPATHS=/tmp/foobar.jar geomesa ingest .... hdfs://namenode:9000/user/you/somefile.csv


"window is not defined" when using serverside rendering (Angular Universal)

Summary
When using Inputmask with Angular Universal in ServerSideRendering mode, get a "ReferenceError: window is not defined" upon startup.
Setup
Using the ngneat/input-mask wrapper around Inputmask.
Relevant libraries and versions:

"@angular/core": "~13.2.2",
"@nguniversal/builders": "^13.0.2",
"@ngneat/input-mask": "^5.2.0",
"inputmask": "5.0.5",

Deployed to AWS Elasticbeanstalk running Node.js 16.
Reproducible locally using the Angular Universal test SSR.
Expected
When using in a client side rendering mode, no errors are shown. It would be expected that the same would happen in a SSR environment (Node.js) however the server fails to start.
Actual
Server side code fails to start, throwing a ReferenceError: window is not defined. I suspect this is because of the outputted webpack code (see below).
Other
Relevant code from dist/inputmask.js:
/*!
 * dist/inputmask
 * https://github.com/RobinHerbots/Inputmask
 * Copyright (c) 2010 - 2020 Robin Herbots
 * Licensed under the MIT license
 * Version: 5.0.5-beta.0
 */
!function webpackUniversalModuleDefinition(root, factory) {
    if (true) module.exports = factory(); else { var i, a; }
}(window, function() {
  ^
ReferenceError: window is not defined

@fabienne-m any progress on this?

Unfortunately, not that I know of.

my observation is that inputmask is being built and placed in the main.js bundle. Because its code referes self/window it doesnt pass over the :SSR compiler. Neither route i'm using inputmask needs to be served over SSR so i'm trying to find a way by pass this code check that stops when self/window happens. Something like configuring the routes that will be served as SSR on the server.ts


makes MzMediaService compatible with angular universal

Have a look at the issue i reported (#337).

Lint fails because of the imports ordering ... run yarn lint:lib to see the errors.


How to login with username not email

How can I custom the login page so that I can login with username not email?

http://docs.meteor.com/#/full/accounts_ui_config
See passwordSignupFields

I've created a fork here than solves this.
https://github.com/chrisbarry/houston
I've started a PR here - https://github.com/gterrono/houston/pull/310


chore: reduce form width to 400px

closes #1439

The CI results are odd, like it is double-reported, and the duplicates are failing to build.

Just the coveralls step failed again. #1655


jshint.vim not working in vim 7.4 and ubuntu 14.04

I've just installed jshint with pathogen and nothing happens when I save a file. It doesn't add JSHintUpdate to my vim and there is nothing on vim output.
PS: node is in the path

Is this still a problem? If so I will reopen

Not a problem anymore. Thanks!


[WIP] Introduce an alternative manifest builder

This manifest builder uses chunkGroups to reduce the amount of data we add to the bundle. It also chooses the smallest number of chunks required to satisfy a request, though we should probably make it operate on size instead. 🤔
I'm going to 🛌, I'm handing this off ➡ @timneutkens.
We need to verify this works and potentially crawl _parents of the chunkGroup until we hit an isInitial().

Wouldn't this work fine 🤔
chunks.forEach((chunk) => {
            // If chunk is not an entry point skip them
            if (chunk.hasEntryModule()) {
              const dynamicChunks = chunk.getAllAsyncChunks()
              if (dynamicChunks.size !== 0) {
                for (const dynamicChunk of dynamicChunks) {
                  for (const module of dynamicChunk.modulesIterable) {
                    GraphHelpers.connectChunkAndModule(chunk, module)
                  }
                }
              }
            }
          })

I don't think this is operating quite as expected. Please do not merge yet.


Develop

What kind of change does this PR introduce?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

👷 Deploy Preview for betsweatwin processing.

Name
Link

🔨 Latest commit
c10f6982c2378df49a68051f1ac122f1ff49d14c

🔍 Latest deploy log
https://app.netlify.com/sites/betsweatwin/deploys/653e7f657c05940008c1ccb0


Unable to Install Again

I'm getting the following error when attempting to install this package:
npm install --global homebridge-platform-ring-video-doorbell

><EMAIL_ADDRESS>install /usr/local/lib/node_modules/homebridge-platform-ring-video-doorbell/node_modules/arp-a
> node-gyp rebuild

  CXX(target) Release/obj.target/macos/src/macos.o
  SOLINK_MODULE(target) Release/macos.node

><EMAIL_ADDRESS>install /usr/local/lib/node_modules/homebridge-platform-ring-video-doorbell/node_modules/socketwatcher
> node-gyp rebuild

  CXX(target) Release/obj.target/socketwatcher/socket_watcher.o
../socket_watcher.cpp:77:8: warning: 'MakeCallback' is deprecated [-Wdeprecated-declarations]
  Nan::MakeCallback(watcher->handle(), callback, argc, argv);
       ^
../../nan/nan.h:929:3: note: 'MakeCallback' has been explicitly marked deprecated here
  NAN_DEPRECATED inline v8::Local<v8::Value> MakeCallback(
  ^
../../nan/nan.h:98:40: note: expanded from macro 'NAN_DEPRECATED'
# define NAN_DEPRECATED __attribute__((deprecated))
                                       ^
../socket_watcher.cpp:104:37: error: no matching member function for call to 'NewInstance'
    info.GetReturnValue().Set(cons->NewInstance());
                              ~~~~~~^~~~~~~~~~~
/Users/mikey/.node-gyp/10.1.0/include/node/v8.h:3851:44: note: candidate function not viable: requires single argument 'context', but no arguments were provided
  V8_WARN_UNUSED_RESULT MaybeLocal<Object> NewInstance(
                                           ^
/Users/mikey/.node-gyp/10.1.0/include/node/v8.h:3848:44: note: candidate function not viable: requires 3 arguments, but 0 were provided
  V8_WARN_UNUSED_RESULT MaybeLocal<Object> NewInstance(
                                           ^
1 warning and 1 error generated.
make: *** [Release/obj.target/socketwatcher/socket_watcher.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at ChildProcess.emit (events.js:182:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:235:12)
gyp ERR! System Darwin 17.6.0
gyp ERR! command "/usr/local/Cellar/node/10.1.0/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/local/lib/node_modules/homebridge-platform-ring-video-doorbell/node_modules/socketwatcher
gyp ERR! node -v v10.1.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Same issue here. Perhaps this is a bug with socketwatcher?

Looks like this:
https://github.com/btrask/node-socketwatcher/issues/18

I was able to work around this by downgrading to node 8.11.3 instead of 10.7.0.

Unfortunately it seems that socketwatcher is inactive so I'm not sure who is going to fix the issue there.

Well I've managed to do a repair patch on the dependencies of this project and got this installed on v10.
I basically forked the base project, and cascade-modified the package.json files to reference forked versions of faulty dependencies that I fixed.
To Install:
npm install -g git://github.com/mkormendy/homebridge-platform-ring-video-doorbell.git
The dependencies I forked and modified are here so you can see the code changes yourself:
https://github.com/mkormendy/homebridge-platform-ring-video-doorbell
https://github.com/mkormendy/homespun-discovery
https://github.com/mkormendy/node-pcap


Fix: add correct "next steps" instructions

This updates the CLI's post-boostrapping message to print the correct npm script for local development (npm run dev). Also updated the project template's README.
Closes #79

Sorry closing this


Fix styling when unread chat messages exceeds 9 (and remove ControlBarButtonBadgeIcon as a customizable icon)

What

Remove ControlBarButtonBadgeIcon as a customizable icon - this is just a circular background not a custom icon
Update to use CSS as badge background to fix styling when chat messages exceed 9
Increase the distance between control bar buttons text and icon in split button designs to match figma (10px)

This gives enough distance for the chat notification to display 9+ without occluding the Chat text

Why
Not ideal UIwhen 9+ was showing:

New Figma designs: https://www.figma.com/file/JVOzw64OnLDrNjb9AScLN3/Web-UI-Library---Explorations?node-id=1275%3A43068
Bug link: https://skype.visualstudio.com/SPOOL/_workitems/edit/2777078
How Tested
Locally running call with chat
Zoomed in:

Zoomed out:

@JamesBurnside Is the change in control bar button width and border radius intentional?
I can see some UI snapshots have changed.

@JamesBurnside James Burnside FTE Is the change in control bar button width and border radius intentional? I can see some UI snapshots have changed.

Button width yes, border radius might just look different because it's longer? It shouldn't have changed


clean up TestReconcile_CloudEvents YAML parsing

Changes
Replace Pipeline, PipelineRun, and Task object initialization to use YAMLParser
/kind cleanup
working on #4610
:D picking this up to do a bit of learning. Thank you all for the awesomeness
that is Tekton!!
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
NONE

Signed-off-by: jbpratt<EMAIL_ADDRESS>

thank you @jbpratt for helping out 👍 Welcome to the community!
/ok-to-test

@jbpratt please update the commit message to include a brief description 🙏

@pritidesai updated!

/retest
I don't think my changes are related to the failures but I'm investigating now


Create a release: v0.1.2

If issues must be resolved before creating a release, mark them as blockers in ZenHub.
I realized that some important changes in the test data have not been added to a release, most notably the change from X_PCA to X_pca in simulated data to match scpca-nf outputs
Pre-release checklist:

[x] Add the upcoming release version to add to the title of this issue!
[x] All blocking issues have been resolved
[x] All Docker images use a versioned tag (i.e. :latest or :edge is not used)
[x] The default data release in nextflow.config is up-to-date. (See OpenScPCA-analysis announcements for the latest data release version, or run ./download-data.py --list-releases from the OpenScPCA-analysis repository.)
[x] The full workflow has been run successfully

[x] Trigger with Run Workflow on AWS Batch GitHub Action using Workflow run mode "full" and Workflow output mode "staging"
[x] Check output data in the staging buckets:

[x] s3://openscpca-nf-workflow-results-staging
[x] s3://openscpca-test-data-release-staging
[x] s3://openscpca-test-workflow-results-staging

[x] Any mentions of the workflow version in the repository have been updated, including

[x] nextflow.config manifest
[x] CHANGELOG.md (see more below)

[x] Write release notes and add them to CHANGELOG.md, which should include the following:

Which modules have been added or removed, if any?
Have there been any changes to the workflow configuration or launch instructions?
What has changed in the workflow documentation, if anything?

[x] Create a release on GitHub with a new tag of the version number in vX.Y.Z format

[x] Populate the description with the release notes added to the changelog. Note that you may find it helpful to use GitHub's automated release notes generation when updating the changelog.

Done: https://github.com/AlexsLemonade/OpenScPCA-nf/releases/tag/v0.1.2


General Issues with formatting result

Hi @Ekrekr, I've been following your work in dataform, and I was pleasantly surprised to see that this repository existed, as we wanted to understand how formatting works in the BQ UI.
I want to build a VSCode extension and tooling based on this, to standardize SQL formatting and have the same format between dataform, BQ UI and Airflow SQL.
The best formatter currently which is also compatible with jinja is SQLfluff, and produces very good results once configured.
One of our main problems with the current way formatting works is that the result for complex queries is completely unreadable for a human. Ideally the formatter would format based on some standards such as:

https://www.sqlstyle.guide/
https://handbook.gitlab.com/handbook/business-technology/data-team/platform/sql-style-guide/

I would be in general interested in forking this and start hacking on it a bit whenever I have time, and/or provide some examples of how the output can be made more readable.
Lastly, I have a general question:  would a change/suggestion in here be applied to the Monaco Editor under the BQ UI?
Thank you for taking your time to read this, and sorry for the long wall of text

Hi Charis-Nicolas,
Great to hear that this is useful to you, particularly in regards to a potential VSCode extension! I totally agree that the formatting style could be improved, to follow general SQL standards.

Would a change/suggestion in here be applied to the Monaco Editor under the BQ UI?

Yes, the code here is what is used for formatting SQL in the UI. Right now it's just exported from internal, but I can propagate changes back.
Best,
Elias

Hey Elias,
that's great to hear, I will have a talk internally if I can carve out some SAT time, and I'll get back to you in the coming weeks


Prebuilt VMs not opening

Hi,
When I press open on a prebuilt .utm file, it doesn’t open and nothing happens. (I used an android vm from the official page on GitHub)

UTM Version: UTM SE 4.3.5 build 87
OS Version: iOS 17 public beta
Device Model: iPhone 11
Is it jailbroken (name jailbreak used)? N/a
How did you install UTM? Via scarlet

No debug log as machine won’t upload
config.plist file: https://drive.google.com/file/d/1e7emCxRvuAEciS_sIqyouDIn3lwRWqKe/view?usp=drivesdk
thanks !

Duplicate of #5538


docs: fix links to config-files documents

While working on https://github.com/zwave-js/node-zwave-js/pull/2854 I checked the CONTRIBUTING.md and saw some invalid links, just helping out and fixing them :)

Thanks - do you think it would make more sense to link to the documentation instead of its source files?
=> https://zwave-js.github.io/node-zwave-js/#/config-files/overview etc.

Thanks - do you think it would make more sense to link to the documentation instead of its source files?
=> https://zwave-js.github.io/node-zwave-js/#/config-files/overview etc.

Yes, I think this makes. The online documentation is much easier to navigate largely due to the left menu.
I changed the PR slightly, now pointing just to the general instructions for device configuration files, less duplication of content.
I also moved the style guide description further to the top on that page as I believe that to be more important and general than the others. To me partial parameters and conditional parameters seems more like edge cases that are not always relevant, while the style guide always is important.
What do you think?

Sounds good!


ModuleNotFoundError: No module named 'pytest'

My tox.ini
[tox]
envlist = py37,py38

[testenv]
groups =  ; Dependency groups in pyproject.toml
    test
    extra_tests
allowlist_externals = pdm
commands =
    pytest -s -vv {posargs} {toxinidir}/test/ --assert=plain

Error:
py37 run-test-pre: PYTHONHASHSEED='2279545942'
py37 run-test: commands[0] | pytest -s -vv /Users/laike9m/Dev/Python/Cyberbrain/test/ --assert=plain
WARNING: test command found but not installed in testenv
  cmd: /Users/laike9m/Dev/Python/Cyberbrain/__pypackages__/3.7/bin/pytest
  env: /Users/laike9m/Dev/Python/Cyberbrain/.tox/py37
Maybe you forgot to specify a dependency? See also the allowlist_externals envconfig setting.

DEPRECATION WARNING: this will be an error in tox 4 and above!
Traceback (most recent call last):
  File "/Users/laike9m/Dev/Python/Cyberbrain/__pypackages__/3.7/bin/pytest", line 5, in <module>
    from pytest import console_main
ModuleNotFoundError: No module named 'pytest'
ERROR: InvocationError for command /Users/laike9m/Dev/Python/Cyberbrain/__pypackages__/3.7/bin/pytest -s -vv test --assert=plain (exited with code 1)

Replacing pytest with pdm run pytest works. Perhaps we want to update the example in documentation?

Shouldn't be, the pdm run will  be added to the front implicitly.

It should work now.


Quit printing exception backtrace twice, revisited

Improves a fix previously proposed by Gaurav Chande.
Original comment:
This makes it so that exception backtrace is printed to stdout only
once. Looks like this annoying problem has been around for some time.

@francoisjacques @gauravmc thank both or you for the contribution.


Tidy up xDS and ECP imports, and related

Description
David asked me about merge conflicts in https://github.com/emissary-ingress/emissary/pull/4472; I responded that the conflicts were because import names had been changed in ambex/transforms.go.  That set me down a line: who had the bright idea to change the import names... is there a way I can get the linter to enforce consistent import names?

The first 3 commits are about tightening up the linter's handling of imports
The last commit is cleaning up a v2/v3 duplication comment in ambex/transforms.go
There was a 5th commit de-duping ambex/transforms.go, but I dropped it in favor of #4473

As usual for me, I suggest a commit-by-commit review. There are some more details in the commit messages.
Related Issues

https://github.com/emissary-ingress/emissary/pull/4473

Testing
I did some fiddling around to check that the .golangci.yml rules work how I want.
Checklist

[x] I made sure to update CHANGELOG.md. - no applicable changes
Remember, the CHANGELOG needs to mention:

Any new features
Any changes to our included version of Envoy
Any non-backward-compatible changes
Any deprecations

[x] This is unlikely to impact how Ambassador performs at scale. - just naming
Remember, things that might have an impact at scale include:

Any significant changes in memory use that might require adjusting the memory limits
Any significant changes in CPU use that might require adjusting the CPU limits
Anything that might change how many replicas users should use
Changes that impact data-plane latency/scalability

[x] My change is adequately tested. - CI should cover it
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[ ] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently. - is gofmt -r worth mentioning?

[ ] The changes in this PR have been reviewed for security concerns and adherence to security best practices.

@LukeShu these changes look good I am just wondering why are we doing snake casing on the imports?
Shouldn't those be camelCase? Any IDE or VSCode with the Go extensions will flag those.

Both underscores and camel case are equally invalid; if VSCode complains about one and not the other then that's a problem with VSCode.
https://go.dev/blog/package-names

Good package names are short and clear. They are lower case, with no under_scores or mixedCaps.

When renaming an imported package, the local name should follow the same guidelines as package names (lower case, no under_scores or mixedCaps).

... but protobuf naming conventions and ECP's conventions are very much at odds with that.  Shame on protobuf and on the envoyproxy/go-control-plane maintainers. But we need to do something about it; and this is the clearest least-bad thing I've come up with.  If someone else comes up with a different naming convention that conveys the same info but doesn't use underscores or mixed caps, I'm all ears.

Rebased up to master to resolve conflicts. Only real change was that I split adding goimports to .golangci.yml and running make format/go in to two commits.


Can't import visit module in python

Describe the bug
When attempting to run sphinx_cli_extractor.py in src/doc, I get this error:
ERROR: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.20' not found (required by /g/g13/rusu1/visit/src/lib/site-packages/visit/visitmodule.so)
**
**  Running VisItTraceback (most recent call last):
  File "./sphinx_cli_extractor.py", line 1022, in <module>
    print >> sys.stderr, "**\n**  Running VisIt", visit.Version(), "\n**"
AttributeError: 'module' object has no attribute 'Version'

I also tried running with python installed in /usr/gapps, and I get this traceback:
  File "./sphinx_cli_extractor.py", line 26, in <module>
    import visit
  File "/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/site-packages/visit/__init__.py", line 61, in <module>
    from frontend import *
  File "/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/site-packages/visit/frontend.py", line 70, in <module>
    import subprocess
  File "/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/python/lib/python2.7/subprocess.py", line 75, in <module>
    import pickle
  File "/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/python/lib/python2.7/pickle.py", line 34, in <module>
    import struct
  File "/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/python/lib/python2.7/struct.py", line 1, in <module>
    from _struct import *
ImportError: /usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/python/lib/python2.7/lib-dynload/_struct.so: undefined symbol: PyUnicodeUCS2_AsEncodedString

I believe this is because LC updated python so that Python 2.7.16 is now built with UCS-4 support (instead of UCS-2).
Impact
Not sure if this has any impact for the user, but it does for me when working with the sphinx documentation.
To Reproduce
Go to src/doc and try:
env PATH=../bin:$PATH PYTHONPATH=../lib/site-packages/ ./sphinx_cli_extractor.py

You can also try pointing to installed visit in usr/gapps, like so:
env PATH=../bin:$PATH \
PYTHONHOME=/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/python \
PYTHONPATH=/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/site-packages \
./sphinx_cli_extractor.py

@rusu24edward it seems the system python is no longer compatible with our python b/c they were built with different compilers and standard libraries.  Honestly it was lucky that this worked before.
Can you run this script via visit -nowin -cli -s {script.py}

Can you run this script via visit -nowin -cli -s {script.py}?

I don't think so...the script @rusu24edward needs to run does import visit. Is that kosher for a script run in the way you suggest? I would think that would conflict with the visit python CLI instance already running but maybe it works ok and if so, I had never thought to suggest @rusu24edward try it that way. It is a lot easier!!

@rusu24edward it seems the system python is no longer compatible with our python b/c

I am not sure why LC's system python is even relevant here except that maybe, we were running it, wrongly, via VisIt's python in the past and just getting lucky that it worked.
Ordinarily, I would say we would always have to use the python installation that was used when VisIt was built which is almost always the python in the VisIt installation, right?

From Mark's suggestion here, this works
env PATH=/usr/gapps/visit/3.0.0/linux-x86_64-toss3/bin:/usr/gapps/visit/3.0.0/bin:/usr/gapps/visit/bin:$PATH \
PYTHONHOME=/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/python \
PYTHONPATH=/usr/gapps/visit/3.0.0/linux-x86_64-toss3/lib/site-packages \
./sphinx_cli_extractor.py

Not sure if the underlying changes to python on the LC are still a concern for us then.

The purpose of this ticket was to discuss the issue, which we did, so I'm now closing it.


fix: make sync a no-opt on local data

fixes #380 by avoiding to overwrite data
with partial data from disk
Changes proposed in this pull request:

Make sync a no opt in local mode rather than read data from disc

I don't think that sync has any function in local mode but if that is too intrusive https://github.com/jenshnielsen/Qcodes/commit/fd2b0d3859627d882a9986f7aa453b980e60a9c6 is a less intrusive fix
@giulioungaretti @nataliejpg

This breaks test_local_instrument because it assumes that you can:

run experiment in background process
end background process
sync data over to main process

@jenshnielsen

...
run experiment in background process

Does that mean your small change there will make backround loops with the current multiprocessing stuff totally impossible?

This change makes it impossible to read the data into the main process using sync. This is probably not the right fix but the current code is fairly convoluted. It's probably better to prevent plot.update from calling sync on default

Test added


PCL support?

Hello,
I wanted to use the priority queue in a PCL (Portable Class Library).
But it is not working. I tried to create a nuget package but that failed.
It is easy to create a PCL from the code and if I compile it an directly reference the lib it works. But i failed to create a nuget-package..
Cheers

I've included support for .Net 2.0, .Net 4.5, and UWP.  Do none of these work for PCL (I really know nothing about PCL)?

Well. the strange thing is that when i build it manually iam able to reference the dll directly even from a PCL.
But installing it fails, because the reference can not be resolve.
the official nuget package fails completly...

PCL support for frameworks net40+sl5+win8+wpa81+wp8 has been added in version 4.0.2.
Please reopen if you continue to have issues.

took a while to get back to this,
but it is working 👍


Inverse mode added

the three modes are like this.
but the backspace button is sometimes malfunctioning and i am confused with that , so can you please create a new issue for that.

What sir?

Please review this PR

Please Review this pr

I updated it, please review pr again.