[Bug] trino-gateway v4 oidc redirecting to http://localhost:8090/sso

Hi, we originally had oidc to okta working with v3 of trino-gateway with the following config
authentication:
 defaultType: oauth
 oauth:
   issuer: ${ISSUER}
   clientId: ${TRINO_GATEWAY_OIDC_CLIENT_ID}
   clientSecret: ${TRINO_GATEWAY_OIDC_CLIENT_SECRET}
   tokenEndpoint: ${ISSUER}/oauth2/v1/token
   authorizationEndpoint: ${ISSUER}/oauth2/v1/authorize
   jwkEndpoint: ${ISSUER}/oauth2/v1/keys
   redirectUrl: https://${URL_OF_GATEWAY}/oidc/callback
   userIdField: email
   scopes:
     - openid
     - email
     - offline_access

After we updated to latest version 4 [latest commit of main on trino-gateway at the time]  https://github.com/trinodb/trino-gateway/commit/b6bbb1423f935c36e49e3f2bb83bc44b5336474f when we hit the url of the gateway we had deployed it would redirect to http://localhost:8090/sso instead of http://${URL_OF_GATEWAY}/sso
I believe the update of jetty from 9.4.48.v20220622 to 11.0.15 introduced this.
https://github.com/trinodb/trino-gateway/commit/434eec9d77465b9373dd07683449102e4d793b83
I tested that theory by checking out the latest v3 and applying all commits until that commit and that was the commit that introduced that behavior.
Hopefully theres just a config option I'm missing but I couldn't get the redirect fixed.

The Lombok removal is not yet in any release and I am not sure how the jetty removal could cause the issue.

@mosabua ok so is the suggestion to stay on the v3 release for now?  We were at a loss at how the jetty update would cause the issue as well :\

We will have to find out one way or another. I just dont have time to look at it at this stage.

@jeffreychucASAPP were you able to identify the RC here?

@willmostly sorry I totally forgot to respond here.  When I was testing I believe the commit that causes the issue is this commit https://github.com/trinodb/trino-gateway/commit/434eec9d77465b9373dd07683449102e4d793b83
I wasn't able to determine whether it was jetty or dropwizard as iirc to update one you had to update the other library.

Looks like this is an issue with Dropwizard default value change for the forwarded headers property - useForwardedHeaders.
Could you try setting the value for it explicitly like below -
  server:
    applicationConnectors:
      - type: http
        port: 8090
        useForwardedHeaders: true
    adminConnectors:
      - type: http
        port: 8091
        useForwardedHeaders: true
``

@vishalya that fixed it!  Thank you!  Perhaps the default gateway-ha-config.yaml should be updated to add this property
https://github.com/trinodb/trino-gateway/blob/main/gateway-ha/gateway-ha-config.yml

I will send a PR for the default config, that would work better than the documentation.

I'll close this issue now that the commit has been merged.  Thanks for all the help! @vishalya @mosabua @willmostly!


Suggestions to update the contribution guide

I have been navigating Thor's codebase as I am working on #666. I am still trying  to fully understand how the pieces work together before building the feature.
The current contribution document doesn't help much. It would be helpful to have an extensive guide that covers the following:

Dev environment setup
Coding Conventions
Running tests
A high level overview of the codebase(entry point + common scenarios to consider when adding a new feature etc.).
How to contribute to documentation ( report bugs properly, update the wiki and the website content etc.).

I love Thor and I am definitely interested in working on the contribution guide as it will help tackle issues faster and welcome new contributors to the project. What are your thoughts? Thanks!

We're happy for any contributions! Give it a whirl and we'll review!

@azzenabidi I am interested in trying out whatever you end up writing up. I've submitted an issue that I would like to fix. I also would like to nudge along the many open PRs on this project. From the sounds of it, this project could use some love. Any efforts to ease the onboarding process would go a long way.

@patrick-motard Glad to hear that! I am making progress with the guide. If everything goes well, I will be opening a pull request next week :) cc @dorner


Use /bin/bash in RUN cmds for devcontainer

Prior to this fix, an error could occur
when running the devcontainer shell
scripts if execute permission is missing,
like the following: #12 0.546 /bin/sh: 1:
/tmp/node.sh: Permission denied
This pull request adds a workaround so
that execute permission is not required
when running the devcontainer, which
can be useful if execute permission
is not provided after cloning or copying
the repository.

Hi @jjliggett - apologies, I missed the notification for this PR and only just spotted it 😞
Would you be able to elaborate on the scenario where the execute permissions aren't set on the script files?

Hi @stuartleeks, I encountered this error when trying to open the folder in the devcontainer after cloning the repository on Windows.
=> => pulling sha256:036ad5e76825a09471e4a5bbc8b3598385710da96baf52e6121ec1e199817b09                                9.3s
=> => pulling sha256:29f8894fdec621b22f3452549082e49b427263346188f1c465796f089f591e81                                0.2s
=> => pulling sha256:62b6cc116ef5680c19b0d9f7998ba0e7754775a367956a07beb548d1822066a0                               14.3s
=> => pulling sha256:b78e80bae2fd3506bba3dabc0baa3a01bc4706d424f220b9d98f18aeab20cf06                                0.9s
=> [ 5/18] COPY scripts/node.sh /tmp/                                                                                0.9s
=> ERROR [ 6/18] RUN /tmp/node.sh 12.x                                                                               0.4s

importing cache manifest from stuartleeks/foo:

[ 6/18] RUN /tmp/node.sh 12.x:
#12 0.341 /bin/sh: 1: /tmp/node.sh: not found

executor failed running [/bin/sh -c /tmp/node.sh 12.x]: exit code: 127
I believe using /bin/bash to invoke the script files is a common convention in other devcontainer Dockerfiles as well: https://github.com/microsoft/vscode-dev-containers/blob/main/containers/docker-from-docker/.devcontainer/Dockerfile#L25


Error: processLeAdvertisingReport: Caught illegal packet (buffer overflow): Error: Peripheral already connected

What issue do you have? Please be as thorough and explicit as possible.
I can turn the led on/off with homekit 3 times, but then nothing happens and the error appears. (watch logs below). When I restart my Raspberry Pi it works 3 time but then the same thing happens again.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
1 Homebridge UI-X

Which version of Homebridge/HOOBS do you have?
latest

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Raspberry PiV      V. 16.16.0

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version?
5.2.0

Which Govee devices do you have that are causing issues? Please include product models if applicable.
H6127, H6125

Please paste any relevant logs below.

[23/07/2022, 00:21:12] [Govee] starting update with params [{"cmd":1,"data":0}].
[23/07/2022, 00:21:12] [Govee] starting timer.
[23/07/2022, 00:21:12] [Govee] starting scan.
[23/07/2022, 00:21:12] [Govee] scanning started.
[23/07/2022, 00:21:12] [Govee] starting loop.
[23/07/2022, 00:21:12] [Govee] found correct characteristic so breaking loop.
[23/07/2022, 00:21:12] [Govee] [BLE] sending update {1: 0}.
[23/07/2022, 00:21:12] [Govee] current state [off].
[23/07/2022, 00:21:12] [Govee] [noble] found device [a4:c1:38:20:d5:38] [ihoment_H6125_D538].
[23/07/2022, 00:21:12] [Govee] [noble] found device [9c:04:a0:03:31:13] [GBK_H6127_3113].
[23/07/2022, 00:21:12] [Govee] has been reported [discoverable] via BLE.
[23/07/2022, 00:21:12] [Govee] attempting to connect.
processLeAdvertisingReport: Caught illegal packet (buffer overflow): Error: Peripheral already connected

I am having the same problem here.
After a reboot it works once and then all following commands get rejected because of the „Peripheral already connected“ Error.


Bump version numbers for Rinja v0.2.1

rinja 0.2.1 is backward compatible to v0.2.0, only adding new functions traits, not changing existing ones
rinja_derive 0.3.0 emits code that uses e.g. rinja::helpers::get_primitive_value() which were not present in rinja 0.2.0, so it is not backward compatible
rinja_parser introduces new enum members, so it is not backward compatible

Resolves #101.

I'd prefer if we kept the same version number across all crates. Makes it simpler for users to know what's going on instead of trying to remember which version of X maps to Y.

Then we must either

bump the version of rinja to 0.3.0, too, even though it is not a breaking change, or
pin the rinja_derive version in rinja, and the rinja_parser version in rinja_derive, but accept the people who on update on of the dependencies are in for trouble, or
do both, so for later releases we can update the pinned version, making later releases easier.

Then I'd prefer the 3rd solution:
rinja/Cargo.toml: version = "0.3.0", rinra_derive = "=0.3.0"
rinja_derive/Cargo.toml: version = "0.3.0", rinra_parser = "=0.3.0"
rinja_parser/Cargo.toml: version = "0.3.0"


fix: add missing format macro arg

@jmgilman I'm unable to build master (and 0.6.2) locally due to missing format-macro arg.
error: there is no argument named `address`
   --> src/client.rs:189:58
    |
189 |             info!("Using vault address from $VAULT_ADDR: {address}");

I wonder how it builds on the CI ? :thinking:
I forgot to mention (and git-add this) this in #36

@jmgilman bump :)

Sorry my ignorance. Completely missed that this feature got stable in Rust 1.58. My env is running old. Closing this.


chore: adds unique constraint to the approval queue

adds a UNIQUE constraint to the approval queue table on work_id, guaranteeing that works can only ever be submitted to the queue once.
closes #78

What happens on the user's end when they try to add a work that was already added? I'd like for there to be a clear alert for them.

What happens on the user's end when they try to add a work that was already added? I'd like for there to be a clear alert for them.

so return a user-readable error when the UNIQUE constraint is violated?

What happens on the user's end when they try to add a work that was already added? I'd like for there to be a clear alert for them.

so return a user-readable error when the UNIQUE constraint is violated?

Sure, like "This work has already been submitted for review".

func submitItem(_ id: String) async throws -> Response {
        let profile = try request.authService.getUser(withProfile: true).profile!
        return try await request.db.transaction { database in
            if let queueItem = try await ApprovalQueue.query(on: database).filter(\.$work.$id == id).first() {
                if queueItem.attempts > 3 {
                    throw Abort(.forbidden, reason: "You've already reached the maximum number of attempts for this work.")
                }
                queueItem.attempts += 1
                queueItem.status = .waiting
                try await queueItem.save(on: database)
                return Response(status: .ok)
            } else {
                guard let work = try await profile.$works.query(on: database).filter(\.$id == id).first() else {
                    throw Abort(.notFound, reason: "The work you're trying to add to the queue doesn't exist.")
                }
                if work.words < MIN_WORD_COUNT {
                    throw Abort(.badRequest, reason: "Your work must meet a minimum word count of \(MIN_WORD_COUNT) words before submission.")
                }
                let newItem = ApprovalQueue(workId: id)
                try await newItem.save(on: database)
                work.approvalStatus = .pending
                try await work.save(on: database)
                return Response(status: .ok)
            }
        }
    }

the function for submissions already checks for existing entries in order to increment an attempts counter. would this count for what you want, or should the notification say something along the lines of "This work is now on its Xnd attempt"?

This is different from "attempts", which refers to a work getting rejected and having to be resubmitted. A glitch allowing a user to submit the same story twice at the same time shouldn't count toward that.

This is different from "attempts", which refers to a work getting rejected and having to be resubmitted. A glitch allowing a user to submit the same story twice at the same time shouldn't count toward that.

this wouldn't be possible at all, though, especially not with a UNIQUE constraint on work_ids


SerenityStateTranisition Missing Eth2 validate_state_root equivalent?

What is wrong?
The Eth2 spec for state_transition contains the following:
...
if validate_state_root:
    assert block.state_root == hash_tree_root(state)

I'm unable to find an equivalent check in trinity's implementation (though it's possible I missed it).
How can it be fixed

Check for a mismatched state_root and raise a ValidationError(?) if the check fails
Optionally, expose a validate_state_root boolean flag as a parameter for apply_state_transition (this would be helpful for fuzz testing - e.g. beacon_fuzz)

we handle this a bit differently from the spec; if you refer here:
https://github.com/ethereum/trinity/blob/master/eth2/beacon/chains/base.py#L413-L415
we first have the state machine import the block and if we want to validate it (by default, we do) then we end up checking that each field of the ssz schema matches, including the state_root, in our call to validate_imported_block_unchanged.
i think we are all open to extending the API here if it makes fuzz testing radically simpler so let me know if we can do more here or if the above is not clear. thanks!

Thanks for clearing htis up!
There should be no need to extend or adjust the existing API.
I'm currently instantiating and calling the SerenityStateTransition directly.
As I need to pass a BeaconState as input and get it as output, the higher-level import_block() is less suitable.
To maintain consistency across implementations, I can simply disable validate_state_root checking for the other implementations and call your lower-level transition. If performing validation becomes necessary, I can easily use BeaconStateMachine.import_block and include the validate_imported_block_unchanged in the harness function.


⚠ Bofrost.fr has degraded performance

In f764fb3, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1084 ms

Resolved: Bofrost.fr performance has improved in 18a41e1 after 9 minutes.


Hint component icon broken into single row

Description
During implementation of the Baloise Design System, we discovered that in case the label is too long and it makes use of the hint component, then the icon (i) is broken into a single row. This issue only happens when the label is too long and cannot make it into a single row. Therefore, the hint-icon (i) is currently standing alone in a third row, even though the second row still have enough space for the icon to be displayed.
Pictures for reference

moved to https://github.com/baloise/design-system/issues/715


2 updated records not updated in Blacklight

Recently I have needed to relink Lapidus books from a wrong BIB record to a new, correct one.
Here’s an example
Lapidus Deposit 1682
Wrong BIB =<PHONE_NUMBER>506421
New, correct BIB =<PHONE_NUMBER>8906421
I made this change yesterday.  As of today, Lapidus Deposit 1682 is showing as holding in BOTH records
https://catalog.princeton.edu/catalog/9932144563506421
and
https://catalog.princeton.edu/catalog/99125437328906421
Alma holdings show the correct holding info for these MMS IDs –
When will the public catalog update the wrong record so that it does not show the Lapidus holding?

History of<PHONE_NUMBER>8906421:
It was included in incremental_16185650540006421_20220113_210155[054]_new.tar.gz
At this time, it had no holdings attached.
It was then included in incremental_16197013530006421_20220114_150146[052]_new.tar.gz
At this time, it still had no holdings attached.
It was then included in incremental_16200861960006421_20220114_210127[051]_new.tar.gz
At this time, it had one holding attached (the Lapidus Deposit 1682 holding).

History of<PHONE_NUMBER>506421:
It was included in incremental_16185650540006421_20220113_210155[054]_new.tar.gz
At this time, it had 2 holdings attached (including the Lapidus Deposit 1682 holding)
It has not appeared in any files since then.

To me, this says that when a holding is removed from a bib in Alma, it does not become included in the next incremental dump. I am going to create a SalesForce ticket about this, but I think this will require workflows being adjusted for relinking holdings. When a holding is moved from a bib, the attached bib must be saved as well.
For the immediate fix, I am going to save<PHONE_NUMBER>506421, which should put it into the next incremental dump.

There is an open issue in Salesforce. It is an Alma issue.


Fix EventRegistration not having permission checks

Currently CMS users with limited access (i.e Content Authors) can create events but are unable to view the registrations to the events, even if they have created them.

Should it also check cms access? As written, it may technically allow public view of registration info

So something like:
	public function canView($member = null) {
		return $this->Time()->canView($member)
				&& Permission::check("CMS_ACCESS_CMSMain", $member);
	}

Yeah that should be okay


[BUG]: Dark Mode Not working properly

Describe the bug
Whenever the dark mode switch is clicked, the entire page should go in dark mode/light mode but only the text in the Nav Bar is getting dark.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://akshitagupta15june.github.io/PetMe/'
Click on 'Sun/Moon Icon'
See error

Expected behavior
The entire page should go in dark mode/light mode.
Screenshots
Light Mode:

Dark mode:

fixing issue #1047

this issue is no more, i think it has been updated by some pr that is not linked to #1047 or some other issue was there with same context.

As of now it is okay, so this issue should be closed @akshitagupta15june

Hey there! I'm Ashutosh Kumar, a full-stack MERN web developer. With experience in numerous successful MERN stack  projects, I excel in  HTML, CSS,JavaScript,C, C++,jQuery, Node.js, React.js, Bootstrap, Tailwind, Express, MongoDB, API,GraphQL,WebRTC.
You can find me on LinkedIn : https://www.linkedin.com/in/ashutosh-kumar-60b262251/
GitHub : https://github.com/Ashutosh-0506.
Looking forward to contributing to GSSOC 2024!

Any updates?


Bugfix for non-generic GetRequiredServices

The non-generic GetRequiredServices wasn't actually checking if the service exists. Solved this by moving the code from the generic method to the non-generic method and changing the  generic method to use the non-generic one.
This made the NonGeneric_GetRequiredServices_ReturnsAllServices() test redundant.

@davidfowl previous code was only working on the happy path, my mistake..

@davidfowl updated, including tests.

:watch:

@pranavkm updated
cc @davidfowl

:watch: for few remaining comments.

@pranavkm updated again

:shipit:  from me. cc @Eilon

@pranavkm seems like AppVeyor doesn't like the Microsoft.AspNet.Test. I use it for the ExceptionAssert class, but is it even allowed in project outside the AspNet namespace? I could change it to just manually comparing the message from the exception variable.

You've only got a reference to "Microsoft.AspNet.Testing": "1.0.0-*" when compiling for the dnx451 framework, not for dnxcore50. The test on the other hand are run using CoreCLR (see line 268).

@Tragetaschen nice catch, thanks!

Please squash the commits if you think you're good to go.

@Eilon updated

@Eilon what's the status on this?

@henkmollema we should be able to merge it this week.

@Eilon any update?

@Eilon @dougbu updated

Looks like it's messed up atm, will fix in a sec.

Fixed and rebased


add flake8 to travis testing

This ensures that we test for pep8 compliance during Travis instead of self-enforcing.

Coverage remained the same at 100.0% when pulling fd1479c85659feca0551241476a3b4d1c0946fe6 on AZtheAsian:master into 71ff3b780fd09bc33d57cee8c30037a2009f639c on ninuxorg:master.

Coverage remained the same at 100.0% when pulling 0861e2e6306df586e7a2dbba2794ec48d18bc998 on AZtheAsian:master into 71ff3b780fd09bc33d57cee8c30037a2009f639c on ninuxorg:master.

Coverage remained the same at 100.0% when pulling 48cfbba8a3f9069c3baeeae43d8fcc9c3395c3c2 on AZtheAsian:master into 71ff3b780fd09bc33d57cee8c30037a2009f639c on ninuxorg:master.


Avoid error using only local translation files

Fixes

Fixes #298 by @sarayourfriend
Description

This addresses the problem of the project failing to run if not all translation files are present, avoiding doing i18n:get-translations previously.
It adds a step to the dev command to generate the locales list. The downside of this approach is that it modifies the valid-locales.json file and we will need to be alert in PR changes, but do we really need to track it? Looks like it is actually not used.
Testing Instructions

Delete any translation file other than the en.json
Run the project in this branch: npm run dev
See no errors.

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

The /src/locales/scripts/valid-locales.json file is used in nuxt.config.js so it's critical that the file is present to run the tests. Adding the command to get translations solves it. Do you want to take a second look at this @zackkrida?

If that file is a pre-requisite for running the tests, even locally, then we should probably add it to the test scripts.
I wonder if there's a way to just add it to the jest setup though instead of having to duplicate it across all the NPM scripts.
Which brings up the other option, of using just instead of npm scripts as just has proper dependency support for each recipe and cross-environment ENV variables built in (we wouldn't need cross-env) and much easier support for overriding variables (like right now to run the app locally without audio you need to modify the package.json, with just we could just make it an argument).
But that's controversial... idk many frontend projects (if any) that don't use npm scripts.

Thinking again, it makes more sense to add the get-translation step in the Github action! And keep the package.json lean, in my view. Do you think the same?

I remembered why I had chosen to use just the get-validated-locales node script instead of the npm i18n:get-translations... Network requests are slow! and doing it whitin every dev script run is a pain in slow connections. I'm changing that back.

I remembered why I had chosen to use just the get-validated-locales node script instead of the npm i18n:get-translations... Network requests are slow! and doing it whitin every dev script run is a pain in slow connections. I'm changing that back.

💯

@krysal I would add a "i18n:get-valid-locales": "node src/locales/scripts/get-validated-locales" script, and add that to the dev script, then get this merged 👍

Done! Thanks for your suggestions pals 😄


[CNV-36073] Add snapshot side car and volumesnapshot class

What this PR does / why we need it:

Added external snapshot side car to kubevirt-csi-controller.
Added volume snapshot class to hosted cluster when deploying.

Together with https://github.com/openshift/kubevirt-csi-driver/pull/30 this adds snapshot support to the kubevirt csi driver.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[X] Subject and description added to both, commit and PR.
[X] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test e2e-kubevirt-aws-ovn

/test e2e-kubevirt-aws-ovn

/test e2e-kubevirt-aws-ovn

/test e2e-kubevirt-aws-ovn

/test e2e-kubevirt-aws-ovn

/test e2e-kubevirt-aws-ovn

/test e2e-ibmcloud-roks

/test e2e-kubevirt-aws-ovn

/jira refresh

/jira refresh


Allow empty result of numerical operations between DataArrays

Resolves https://github.com/pydata/xarray/issues/739
@shoyer let me know your thoughts and I'll add what's new note

Not sure if I'm doing something foolish - I had some issues running the tests on my machine because of the from .version import version as __version__. Let me know if there's a quick fix beyond me commenting that line out.

Forgive the python question - please disregard unless you know the answer off hand - do you know if there any way of monkey patching this into our code? I did that for the 0.7 fixes we added, but I'm not sure that's possible here: _binary_op gets called many times upon import xarray, and so I think the prior _binary_op is already 'embedded' in all the functions before we have a chance to monkey patch.

@MaximilianR FWIW I had problems with .version recently when I cloned from the main repo...had to copy the version.py from another xarray install.  Maybe has something to do with .gitignore: xarray/version.py is included there.

Version.py gets created when you install xarray, e.g., when you run pip install -e . after running git clone.

@shoyer thanks!  Good to know.

For consistency, we should also fix arithmetic with Dataset (allow no overlapping labels and no overlapping variables) and GroupBy objects:
https://github.com/pydata/xarray/blob/v0.7.0/xarray/core/dataset.py#L1933
https://github.com/pydata/xarray/blob/v0.7.0/xarray/core/dataset.py#L1975
https://github.com/pydata/xarray/blob/v0.7.0/xarray/core/groupby.py#L220

Super - those are now updated @shoyer

@shoyer I think this is a decent balance:

Allow empty dimensions
But raise if non-existent dimensions are supplied

I think the code and tests now reflect that. Let me know your thoughts / comments

This is my last PR that's ready to go @shoyer

Still thinking about the fillna behavior... I agree, it's weird that ds.fillna({'missing_variable': 0}) works without raising an error.
I would say this is consistent with in-place arithmetic on Datasets (i.e., left alignment of variable names), but we actually don't allow that with different variables currently... probably because I wanted to avoid this headache :). I think I'll add a special case in to ensure that we raise an error if there are unexpected variables in the fillna argument.

OK, going to merge this...

I'm probably going to call this an "enhancement" rather than a bug fix, but otherwise this looks good... I'll fix up the what's new page shortly.

Cheers!


Update disks-deploy-premium-v2.md

updating the Azure CLI disk update to diskMBpsReadWrite with a capital "B" in MBps for the correct command.

Docs Build status updates of commit ca8d94b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/virtual-machines/disks-deploy-premium-v2.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off

@roygara

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@albecker1 The syntax for this has already changed. But it's all lower case now. I'm going to close this out and follow up with you offline.
#please-close


Last saved policy instance overrides attributes other isntance of same policy

Reproduce:

add same policy twice
configure one of them
navigate to the other one and see attributes of first

Fixed by https://github.com/ddcesare/poc-policies/commit/af37faba73fce3167b5eebc643a6e1ac4769ff76
Feel free to reopen if it's not what expected


Correct page "last updated" to reflect recent commits

[Enter feedback here]
Page shows last updated "07/08/2019" but commit history shows 20 commits since Jan 1, 2020. Please update the metadata to correctly show that the page is recently updated. Thanks

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

ID: 8a69504d-a89f-5545-3f8f-0573c928d67f
Version Independent ID: 2f42eaa3-a159-31de-a9d5-19762d34bc5a
Content: Learn about Information barriers - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/information-barriers.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

This matters cos content on stale dated pages has less credibility, so folks might discount the content on this page

@chrfox Kindly answer and help us in resolving this issue. Thanks.

fixed


Remove null passing for Menu methods, fix dockBlocks

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
-

👍


Upgrade to Angular 16

Any idea when this will be released? Need to use it in an environment that wont allow me to use unpublished packages, and or i am not skilled enough to know how to allow it XD. Thanks Travis you are a Beast!

any idea when it will be published on npm @travist

Working on it now.   And for that I thank you
On Tue, Jun 20, 2023 at 12:50 PM Travis Tidwell @.***>
wrote:

Merged #970 https://github.com/formio/angular/pull/970 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/formio/angular/pull/970#event-9583681236, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALM6DBEYJ7KZTP2MNFDQWBTXMH5HHANCNFSM6AAAAAAY7WUC7Q
.
You are receiving this because you commented.Message ID:
@.***>

--
Hayden J. Baldwin
RARE PETRO Tech, Inc
Chief Technology Officer
@.***
714-408-9952
--
https://rarepetro.com/     http://rarepetro.com/technologies
1224
Washington Ave Ste 10
@.,-105.2232823,17z/data=!3m1!4b1!4m5!3m4!1s0x876b9ad318dd7ba3:0xf2b96ac31f6099a9!8m2!3d39.7546162!4d-105.2210936>
Golden, CO 80401
@.,-105.2232823,17z/data=!3m1!4b1!4m5!3m4!1s0x876b9ad318dd7ba3:0xf2b96ac31f6099a9!8m2!3d39.7546162!4d-105.2210936>www.rarepetro.com
http://www.rarepetro.com/
https://www.linkedin.com/company/rare-petro/ This email and any files
transmitted with it are confidential and intended solely for the use of the
individual or entity to whom they are addressed. If you have received this
email in error please notify the system manager. This message contains
confidential information and is intended only for the individual named. If
you are not the named addressee you should not disseminate, distribute or
copy this e-mail. Please notify the sender immediately by e-mail if you
have received this e-mail by mistake and delete this e-mail from your
system. If you are not the intended recipient you are notified that
disclosing, copying, distributing or taking any action in reliance on the
contents of this information is strictly prohibited.


add requester name to finishing task

New feature implementation
Implemented feature

Using new python API introduced in https://github.com/open-rmf/rmf_ros2/pull/267, to provide a requester name for the finishing task factories.

Closing as the node name is passed from within the fleet update handle via https://github.com/open-rmf/rmf_ros2/pull/267


Opening an MdDialog with a TemplateRef that uses md-dialog-close throws "no provider for MdDialogRef"

Proposal:
Provide MdDialogRef even when we pass a TemplateRef.
What is the current behaviour?
With the current setup if we pass a TemplateRef to MdDialog, there is no way to use  md-dialog-close directive as it throws no provider for MdDialogRef as it requires it.
What is the expected behaviour?
It would be great if it works with a TemplateRef but if that's not possible, then could you please add a brief warning to the docs?
What are the steps to reproduce?
I can provide a plunkr if needed.

This is a known issue. It's because we can't set a custom injector on a template ref (since it's already instantiated). I've discussed a potential solution with @jelbourn, but haven't gotten around to implementing it yet.

Hi,
Any news on that?
Thanks :)

Hello any news on this?


Bug/12065 refac glob matching

Description
Refactored Glob.globMatch by using iterative approach
Related Issues
Resolves #12065
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Build still failing please check and resubmit.

@cwperks
The original issue https://github.com/opensearch-project/OpenSearch/issues/12065 seems to be resolved by https://github.com/opensearch-project/OpenSearch/pull/13104 .
Can we close this PR as it is not needed?

@kkhatua Yes, this can be closed.


It would be nice to have an option --relative to show relative times in outputs

Solved in latest commit


Update code owners

Update the code owners to the current active people, so that code reviews can happen in a timely manner.
It mgirates from the OWNERS file to GitHub CODEOWNERS. Once this is merged, a follow-up change will be done in istio/test-infra to update the Ci requirements accordingly.
@BrenoDeMedeiros @liminw @lizan @sawlanipradeep @incfly @Shikugawa you were listed in the OWNERS file, but as far as I know you're not active anymore in the project. Please leave a comment if you want to be kept in the CODEOWNERS file and I would be happy to add you before merging the PR. Thank you! :)

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

Needs approval from an approver in each of these files:

OWNERS [nacx]

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

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

:exclamation: No coverage uploaded for pull request base (main@11a2002). Click here to learn what that means.

:exclamation: Current head 1751edc differs from pull request most recent head da10194. Consider uploading reports for the commit da10194 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #248   +/-   ##
=======================================
  Coverage        ?   90.31%
=======================================
  Files           ?       24
  Lines           ?     1569
  Branches        ?        0
=======================================
  Hits            ?     1417
  Misses          ?      108
  Partials        ?       44

/hold

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

Needs approval from an approver in each of these files:

OWNERS [Shikugawa,nacx]

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

/lgtm.

/unhold

It's been 1 week and there were no objections, so let's have this merged. We can always add back any maintainer that wants to be a code owner.


Analyzer resolves rootUri paths in packages_config.json relative to current directory

According to https://github.com/dart-lang/language/blob/master/accepted/future-releases/language-versioning/package-config-file-v2.md, it should resolve them relative to the directory containing packages_config.json.

I've got a tentative fix in https://dart-review.googlesource.com/c/sdk/+/135204, but I'm still trying to figure out a good way to unit test it.  Stay tuned.

Whoops, after attempting to write tests for this I discovered that my fix was unnecessary; the paths are already resolved relative to the directory containing package_config.json.  I think the reason I thought this bug existed was because https://github.com/dart-lang/sdk/issues/40572 was causing the analyzer to pick up a different package_config.json than I thought it would, and so when I saw that the paths were resloved relative to that package_config.json I incorrectly assumed they were resolved relative to the current directory.


How to decode SIGHASH type on a TX hex ?

Hello,
Is it possible to decode SIGHASH type on a TX hex through Bitwasp?
If so, how can I do that ?
Regards

I'll push an example of this, but not sure I want to maintain it as library code at the moment. They might differ for Blackcoin, so you may need to change that function slightly. https://github.com/Bit-Wasp/bitcoin-php/pull/509

Hi,
The SIGHASH flags are the same.
Thanks , I am looking forward for your example.
Regards

https://github.com/Bit-Wasp/bitcoin-php/blob/master/examples/sighash.parse.php

@afk11 thanks, I saw it.
Now to the main question, how I can I use this to determine the SIGHASH type on a TX ?
I really dont have any clue.
Regards

Transactions have inputs which have signatures. You would be better off explaining why you need to know what the sighash types are...I can tell you what to do then :D
On 6 August 2017 18:23:09 CEST, zilveer<EMAIL_ADDRESS>wrote:

@afk11 thanks, I saw it.
Now to the main question, how I can I use this to determine the SIGHASH
type on a TX ?
I really dont have any clue.
Regards
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Bit-Wasp/bitcoin-php/issues/508#issuecomment-320516896

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@afk11
Thanks for your reply..
I would rather prefer like to take this privately if you really would like to know what I want to do?
I assume that you got my email address, if not please let me know.
I have searched for your e-mail address but cannot fint it :/
best regards

@afk11
I am creating a multisig game and need to validate the TX hex, and SIGHASH type will be one of the validations steps to validate it. In my case it needs to be set to SIGHASH single.
If you can come up with an example, I would appreciate it.
Hope to get a reply from you regarding this.
regards


Changed name on GitHub account, VS Code is still using old name

Describe what happened:
After changing my name on GitHub due to trans identity, I tried logging out and back into VS Code Live Share and reinstalling the program to change my name in the extension, however, it does not change. Hovering over the extension in the status bar shows
Sharing via Visual Studio Live Share as [insert old name here]<EMAIL_ADDRESS>(github)
What was your system configuration?
Product and Version [VS/VSCode]: VS Code Insiders (code-insiders on snapd, version 1.56.0-insider)
OS Version[macOS/Windows]: Arch Linux rolling, up-to-date
Live Share Extension Version:  v1.0.4131
Target Platform or Language [e.g. Node.js]: all
Steps to Reproduce / Scenario:

Change name on GitHub account
Reinstall VS Code Live Share / VS Code
Log in to VS Code Live Share using GitHub
Get deadnamed

Please attach logs to this issue:
live-share-logs.zip

Hi @alyssajotice,
Thanks for your response. Unfortunately, I do not have access to the Windows control panel since I am using Linux, however, I was able to delete VS Code's entry from my GNOME keyring.
I tried signing out and back into the account using the VS Code account UI after removing the key from my keyring and nothing changed.
I tried removing revoking access for VS Code and Live Share via GitHub's OAuth settings and signing back in, that didn't work, either.
Is it possible that Live Share uses old/cached OAuth data that doesn't update when signing out and back in again?

Thanks for the update.  @olegoid can you take a look?

Hello @mdartmann,
You can ask this link to forget your existing profile. When you connect again, we should have updated information for your GH account.
Please let me know if this doesn't work for you.

Hello @Davsterl  ,
The information on the profile page is updated, however, in VS Code the extension still uses my old name.
Profile information via your link (with censored IDs):
{
    "id": <redacted>,
    "initialProvider": "github",
    "signupTime":<PHONE_NUMBER>991,
    "status": "accepted",
    "providers": {
        "github": {
            "id": <redacted>,
            "nodeId": <redacted>,
            "displayName": "Mae Dartmann",
            "username": "mdartmann",
            "profileUrl": "https://github.com/mdartmann",
            "photos": [
                {
                    "value": "https://avatars.githubusercontent.com/u/76077644?v=4"
                }
            ],
            "provider": "github",
            "emails": [],
            "initials": "MD"
        }
    },
    "telemetryId": <redacted>,
    "registrationData": {},
    "workspaces": []
}

The extensions still displays the old name.

Is it possible that some stale data syncs usins Settings Sync? If there is anything else I can provide, please let me know.


Use glib's find in path

Unlike the similar VSTGUI change, we already include gio in sfizz so we can use @redtide 's proposed function.
Closes: #744 (along with the proposed VSTGUI patch on our fork)
I believe nothing calls this code atm, I have to try it out manually.

I forgot some details about this but the code should be also changed in the same way upstream on VSTGUI.

Yes I have an equivalent PR in our fork.
Jul 25, 2021 12:21:46 redtide @.***>:

I forgot some details about this but the code should be also changed in the same way upstream on VSTGUI.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub[https://github.com/sfztools/sfizz/pull/948#issuecomment-886179214], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ADUFWQJEZ3ZCGKNFT62HFHTTZPQTJANCNFSM5APXA3AA]. [data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAADAAAAAwCAYAAABXAvmHAAAABHNCSVQICAgIfAhkiAAAAB9JREFUaIHtwQEBAAAAgiD/r25IQAEAAAAAAAAAAC8GJDAAAY7rwGcAAAAASUVORK5CYII=###24x24:true###][Tracking image][https://github.com/notifications/beacon/ADUFWQMN7HO7XM25UVXBB2DTZPQTJA5CNFSM5APXA3AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGTJALDQ.gif]


Need more informative audit logs on Publisher and Store operations in API Manager

Description:
So far from the audit logs it will indicate generic actions such as update, create. It is great if we can have some extension points to get more contextual information on actions performed in Publisher/Store of API Manager.

I’m closing this issue because it has been inactive for more than two years. Please reopen if you still encounter this issue with the latest version.


over lapping of download and getsttarted buttons

1 - Describe the bug
The two buttons are overlapping with each other on mobile view.
2 - Expected behavior of what should have happened (if applicable)
There should be a gap in between them. i have fixed the issue.
3 - Screenshots (if applicable)

4 - Additional notes
I have the fix  can i make PR?
Also worth to mention that i have see this issue mentioned before by some one but still it is unfixed so i would like to raise a PR

Hi @mayankmohak thank you for reporting this. I think someone might have worked on this before but due to the constant changes in the website it might have occurred again. We are currently on changing up the design of the website so these issues will no longer happen.
I'll assign you to this.

shall i make a pr for now. @shahednasser

@mayankmohak yes please

I have made a PR.
Btw what changes are being expected and what all are going to change on this repo can i get prior of it.

We are currently on changing up the design of the website so these issues will no longer happen.

@mayankmohak I actually added you to the reviewers team of this project. Once you accept it you can see a discussion about it.


Bug in json palette export/import

sketch-palettes.zip
We have a 36-color style guide palette for our site. I've exported the colors to a json, but on re-import, 5 colors consistently don't show up, although they're in the json. (Orange-light, Blue-deep, Blue-mid, Teal-deep, Teal-mid)
Separately, one color "Text-Dark" is consistently misinterpreted when exported turning it from a dark navy into bright green.
I've created a separate palette with just the problem colors, and repeated to export/import to the same results. I'm attaching the sketch file and json palette files.
Fort the color shifting to bright green, in reviewing the json, I noticed the rgba value for Text-Dark 'g':286, but should've been: 28. I'm not sure why that value was misformed.
{"name":"Text-Dark","color":{"r":13,"g":28,"b":61,"a":1,"color-hex":"#0D1C3D 100%","argb-hex":"#FF0D1C3D","css-rgba":"rgba(13,286,61,1)","ui-color":"(r:0.05 g:0.10 b:0.23 a:1.00)"}}
For the 5 colors that wouldn't import, the json values all look valid.
The only commonality I could find among the ones that fail to import is that each contain one color channel value = 0. eg:  [{"name":"Blue-Deep","color":{"r":0,"g":59,"b":113,"a":1,"color-hex":"#003B71 100%","argb-hex":"#FF003B71","css-rgba":"rgba(0,59,113,1)","ui-color":"(r:0.00 g:0.23 b:0.44 a:1.00)"}}] (this shouldn't be an issue, but I can't find any other reason)

Thank you for the feedback, and I will fix it


LAMMPS loadable MP0 model?

Hi folks,
Very exciting to see the foundation model for multi-element systems. :)
I was wondering if there is a model that is readily available to load in LAMMPS and perform structural minimization?
I see that one needs a .pt file to load into LAMMPS from the tutorial here: https://mace-docs.readthedocs.io/en/latest/guide/lammps.html#id2
But the publicly available MACE model is of the format .model from here: https://github.com/ACEsuit/mace-mp/releases/tag/mace_mp_0.
Are these two the same file?
EDIT: adding @cortner, since I work with him.
EDIT2: Also, I don't see a ML-MACE package available on LAMMPS, is it still not publicly released?

The LAMMPS model is not created automatically but we provide a script for
doing so. Explained here:
https://mace-docs.readthedocs.io/en/latest/guide/lammps.html#preparing-your-model
.
On Sun, 28 Jan 2024 at 16:50, Hariharan (Hari) Umashankar <
@.***> wrote:

Hi folks,
Very exciting to see the foundation model for multi-element systems. :)
I was wondering if there is a model that is readily available to load in
LAMMPS and perform structural minimization?
I see that one needs a .pt file to load into LAMMPS from the tutorial
here: https://mace-docs.readthedocs.io/en/latest/guide/lammps.html#id2
But the publicly available MACE model is of the format .model from here:
https://github.com/ACEsuit/mace-mp/releases/tag/mace_mp_0.
Are these two the same file?
—
Reply to this email directly, view it on GitHub
https://github.com/ACEsuit/mace-mp/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXHHTVGMFYVZJ6W262BISDYQX7MRAVCNFSM6AAAAABCN7JBKWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDIMBSGEYTCMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes sorry, overlooked at that part.
Actually, I just used ASE and mace.calculators.mace_mp to run my structural relaxations for my system of interest.

Great! Thanks for the interest

Using the MACE-MP-0 model (available at https://github.com/ACEsuit/mace-mp/releases/tag/mace_mp_0), I encountered the following error after running this command:
File "create_lammps_model.py", line 40
print(f"Only one head found in the model: {heads[0]}. Skipping selection.")
^
SyntaxError: invalid syntax
liang@node01:~/jobs/mace_test$ python create_lammps_model.py 2023-12-03-mace-128-L1_epoch-199.model my_mace.model.pt
File "create_lammps_model.py", line 40
print(f"Only one head found in the model: {heads[0]}. Skipping selection.")
^
SyntaxError: invalid syntax


Missing 'component build' step in 'Installation' wiki page

On some systems it is needed to run this command during the installation, otherwise
there is blank page at the root url. It is really troublesome for users unfamiliar with nodejs.
Add it to wiki or to your makefile.

As of version 1.0.0 of DemocracyOS we are not using component anymore.
Docs for DemocracyOS 1.0.0 are [https://github.com/DemocracyOS/app/tree/add/docs/docs](in this branch) until we merge it to 1.0.0


Make sure every method in every class has its return type written out explicitly

I used to not do it, but since then I've decided that I should do it.

I just looked at the style guide again and oops, I also need to have two tabs of indentation inside multiline statements


Create BDB-Methalox.netkan

.netkan for an optional patch included in the BDB main download, but not installed automatically

I'm going to combine these three pull requests so it's easier to confirm that our edits are consistent...

Merged into #9432.


Submodule updates for Trio: OmniBLE, OmniKit

These updates to the OmniBLE and OmniKit submodules address this Issue https://github.com/nightscout/Trio/issues/244#top
For OmniBLE there is an additional updates used by developers who need to parse logs from Loop and Trio (and iAPS). This code update makes the parser more general.

Merging with two approvals.


NAVDataUpgrade is stuck at 55,56 %

I am following steps for upgrading Business Central 15.1 to 16.0 and in the post-upgrade task, I try to run Start-NAVDataUpgrade -ServerInstance $serverInstance -Force and after when I check the progress I can see that process is somehow stuck regarding progress bar.

If I open Web Client and I go on Help page I can see that the Application Number has changed to 16.0.38071.0.

Everything looks pretty much normal except few warning in Event Viewer, saying:
Message Unable to find defining app group for a table with ID<PHONE_NUMBER>, when initializing query from app group 2.
Is this normal that the Upgrade process gets stuck? I remember something similar to version 14. Would the restart of the instance help in that case?

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

ID: 6a99ad4b-431a-d2d6-b11f-bb467e490387
Version Independent ID: a310818b-2861-9a15-c01d-27ea599bb22b
Content: Upgrading Version 15 Base Application to Version 16 - Business Central
Content Source: dev-itpro/upgrade/upgrade-unmodified-application-v15-to-v16.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: jswymer

Hello,
Thank you for your feedback. Since your question is not about the Help, we suggest that you contact technical support. For more information, see Report a Problem.


Fix for integration of cardano-ledger with plutus-1.11

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Changelog fragments have been written (if appropriate)
[ ] Relevant tickets are mentioned in commit messages
[ ] Formatting, PNG optimization, etc. are updated

PR

[ ] (For external contributions) Corresponding issue exists and is linked in the description
[ ] Targeting master unless this is a cherry-pick backport
[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reviewer requested

Shall I create minor revision of 1.11 for this? Or stick it to 1.12?

merge master to fix it


position  absolute

scroll not working in position absolute

are we talking about page scroll or selectlist scroll

As you are not responding, i will mark this issue as closed, but you can still reopen whenever you like


collectd logging tons of 'uc_update: Value too old'

There's still something strange going on, even when using the master branch code as of 2/10/2017. collectd logs the following sorts of things constantly. In fact, I have 305,000 log lines matching 'uc_update' since 4 days ago on our host that is running collectsphere under collectd:
...
Feb 10 12:24:14 gazoo collectd[82312]: uc_update: Value too old: name = gazoo.our.org/collectsphere/gauge-domain-c7.HostSystem.host-15.cpu.all.average.usage.perc; value time =<PHONE_NUMBER>.000; last cache update =<PHONE_NUMBER>.000;
Feb 10 12:24:14 gazoo collectd[82312]: uc_update: Value too old: name = gazoo.our.org/collectsphere/gauge-domain-c7.HostSystem.host-15.cpu.all.average.usage.perc; value time =<PHONE_NUMBER>.000; last cache update =<PHONE_NUMBER>.000;
Feb 10 12:24:14 gazoo collectd[82312]: uc_update: Value too old: name = gazoo.our.org/collectsphere/gauge-domain-c7.HostSystem.host-15.cpu.all.average.usage.perc; value time =<PHONE_NUMBER>.000; last cache update =<PHONE_NUMBER>.000;
Feb 10 12:24:14 gazoo collectd[82312]: uc_update: Value too old: name = gazoo.our.org/collectsphere/gauge-domain-c7.HostSystem.host-15.cpu.all.average.usage.perc; value time =<PHONE_NUMBER>.000; last cache update =<PHONE_NUMBER>.000;
Feb 10 12:24:14 gazoo collectd[82312]: uc_update: Value too old: name = gazoo.our.org/collectsphere/gauge-domain-c7.HostSystem.host-15.cpu.all.average.usage.perc; value time =<PHONE_NUMBER>.000; last cache update =<PHONE_NUMBER>.000;
...

This situation is mentioned in the collectd FAQ: https://collectd.org/faq.shtml
I have confirmed that we are not running 2 collectd processes on the same host and we are not altering the system time.
My configuration is:
<LoadPlugin "python">
    Globals true
</LoadPlugin>

<Plugin "python">
    Import "collectsphere"
    <Module "collectsphere">
        Name "R701-VMware"
        Host "vcenter.our.org"
        Port "443"
        Verbose "True"
        VerifyCertificate "True"
        Username "OURDOMAIN\\ad-user-name"
        Password "REDACTED"
        Host_Counters "cpu.usage,mem.usage,disk.usage"
    </Module>
</Plugin>

And also per the FAQ, I am not loading the collectsphere plugin twice by accident either:
collectsphere:gazoo# grep -E ^Include /etc/collectd.conf
Include "/etc/collectd.d"
collectsphere:gazoo# grep -E ^LoadPlugin /etc/collectd.conf /etc/collectd.d/vmware.conf
/etc/collectd.conf:LoadPlugin syslog
/etc/collectd.conf:LoadPlugin write_graphite
collectsphere:gazoo#

 #20


error loading plugin 'yard-cucumber'

I'm getting this error any time I run a rake task in my rails app. Any ideas?
→ cat ~/.yard/config
--- !ruby/hash:SymbolHash
:load_plugins: true
:ignored_plugins: []
:autoload_plugins: []
:safe_mode: false

→ cat Rakefile
#!/usr/bin/env rake
# Add your own tasks in files placed in lib/tasks ending in .rake,
# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.

require File.expand_path('../config/application', __FILE__)

C4C::Application.load_tasks

# desc "`rake` will default to running `rake:spec`"
# task :default => :spec
#
# desc "Run all the rspec examples"
# task :spec do
#   system "bundle exec rspec -c -f d spec/*_spec.rb"
# end
#

I'll take a look. I might need more details.
Sent from my iPhone.
On Jan 25, 2012, at 2:38 PM, Ivan<EMAIL_ADDRESS>wrote:

I'm getting this error any time I run a rake task in my rails app. Any ideas?
→ cat ~/.yard/config
--- !ruby/hash:SymbolHash
:load_plugins: true
:ignored_plugins: []
:autoload_plugins: []
:safe_mode: false

→ cat Rakefile
#!/usr/bin/env rake
# Add your own tasks in files placed in lib/tasks ending in .rake,
# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.

require File.expand_path('../config/application', __FILE__)

C4C::Application.load_tasks

# desc "`rake` will default to running `rake:spec`"
# task :default => :spec
#
# desc "Run all the rspec examples"
# task :spec do
#   system "bundle exec rspec -c -f d spec/*_spec.rb"
# end
#

Reply to this email directly or view it on GitHub:
https://github.com/burtlo/yard-cucumber/issues/34

We are currently getting this same error, whether we run it with a rake task or through yard server

Looks like it's coming from the require 'cucumber/parser/gherkin_builder' line in yard-cucumber.rb


Stop using overflow: scroll anywhere

Design Feature Request
Summary
Please stop using overflow: scroll anywhere and use overflow: auto instead.
These are used at least on code boxes, but there's no reason to use overflow: scroll anywhere.
Motivation
overflow: scroll causes scrollbars to always appear on windows and linux, even when the content is not scrollable, which is just a waste of screen space and ugly given the colourful theme.
Drawbacks
None, unless someone prefers useless scrollbars to appear?

The code boxes in most (all?) places don't have a maximum height set, so the vertical scrollbars (in Chrome) and the ugly weird scrolling corner (in Firefox) are completely unnecessary and can be removed by using overflow-x: scroll instead of overflow: scroll.
But yes, also use auto instead of scroll. This gets rid of the scrollbar in many places. Carefully adjusting the code examples should get rid of all the scrollbars actually, but it's probably a good idea to keep overflow-x: auto around when the width is restricted.

Under linux + firefox
currently : overflow: scroll

overflow : auto

Resolved in #522.


Allow users to build devDependenices by specifying value for NODE_ENV

See how node-engine sets NODE_ENV. By using the override rule, the buildpack makes it impossible for a user to change the value of NODE_ENV to something other than the buildpack-set value of production.
As a user, I expect that:
The buildpack may set a default value for NODE_ENV but if the user has specified a value (e.g. pack build... --env NODE_ENV=development), buildpack will use/set the user provided value as:
(1) default value in the build phase of subsequent buildpacks, and
(2) default value in the launch phase of the built container image, unless overridden by subsequent buildpacks
Use case:
This will enable users to build devDependencies by passing NODE_ENV=development. See npm documentation
Edit: Due to  RFC 0019. See https://github.com/paketo-buildpacks/node-engine/issues/196#issuecomment-771222700

With the acceptance of the RFC 0019 as the standard a few days ago, we should change the launch-time value from sticky to non-sticky.
That is, even if you build with pack build node-app --env NODE_ENV="development"
NODE_ENV should be set to "production" by default at launch time.
Users should have to do docker run node-app --env NODE_ENV="development" to change this value.

@arjun024 That sounds accurate wrt the RFC. 👍


Add text-to-speech functionality

Use pi-top's onboard speaker to interact with the user via speech - particularly useful/interesting for robotics applications but can also have a lot of value for simpler Foundation Kit projects.
See old library created here that uses espeak.

Possibly make this part of the Pitop class so it can be used by default without the user specifically adding it manually
Would be nice to have multiple implementations available to use, i.e. user can choose between a high-quality cloud-based version (Google/Amazon/Microsoft) or a lower quality local version (espeak etc)

Options
Google TTS

Based on Google translate so decent voice quality
Has a python package pip3 install gTTS
Requires internet connection

Usage:
from gtts import gTTS
import pygame
from io import BytesIO

pygame.init()

tts = gTTS(text=text, lang='en')
fp = BytesIO()
tts.write_to_fp(fp)
fp.seek(0)

pygame.mixer.init()
pygame.mixer.music.load(fp)
pygame.mixer.music.play()

eSpeak

Offline
Poor quality voice
See pyttsx3 for python library that uses this under the hood

Pico TTS

Offline
Better voice quality than espeak
Supports British English, American English, Spanish, Dutch, French, and Italian (all women voices)
No python package available, would have to run terminal commands to use binary from our own sdk

Installation
wget http://ftp.us.debian.org/debian/pool/non-free/s/svox/libttspico0_1.0+git20130326-9_armhf.deb
wget http://ftp.us.debian.org/debian/pool/non-free/s/svox/libttspico-utils_1.0+git20130326-9_armhf.deb
sudo apt-get install -f ./libttspico0_1.0+git20130326-9_armhf.deb ./libttspico-utils_1.0+git20130326-9_armhf.deb

Usage:
pico2wave -w lookdave.wav "Hello, world!" && aplay lookdave.wav

pyttsx3

Offline
Can use different engines under the hood, defaults to espeak on linux
Poor voice quality
Has a fairly well maintained and documented python package available

Installation
sudo apt install espeak
pip3 install pyttsx3

Usage
import pyttsx3
engine = pyttsx3.init()
engine.say("Hello, world!")
engine.runAndWait()

Microsft TTS

Has python package
Requires Azure subscription key to use which is a bit of a hassle

Amazon Polly

Cloud
Requires an account with API keys
This python package is a TTS broker compatible with google cloud, amazon polly and ibm (all these require credentials)

Festival

Offline
Similar quality to espeak
Has a python lib but just uses subprocess under the hood

Installation
sudo apt-get install festival
pip3 install pyfestival

Mozilla TTS

Very good quality voices
pip installable but lots of dependencies, including pytorch which isn't available via pypy on armv7
This TTS can be installed on a RPi 4 but the process is lengthy and requires compiling pytorch
Speech is not real-time either (6x real-time to produce audio)

Mimic1 from Mycroft.ai

Compiled binary, instructions
Relatively lightweight but much better voice than most other offline solutions

Usage
./mimic -t "Hello"

# change voice
./mimic -t "Hello" -voice kal16

I got pretty good results using festival with a plug-in voice as described here.
sudo apt-get install -y festival festvox-us-slt-hts
festival -b '(voice_cmu_us_slt_arctic_hts)' \
    '(SayText "The temperature is 22 degrees centigrade and there is a slight breeze from the west.")'

That python lib actually binds to the c++ lib, subprocesses are only used for converting to mp3 I think. It gives me an ImportError when I try to use it though.

Similar results with flite:
sudo apt install flite
flite -voice slt -t "The Raspberry Pi is a great Maker platform!"

I got pretty good results using festival with a plug-in voice as described here.

This one sounds pretty good actually, let's just use that!

The python lib actually binds to the c++ lib, subprocesses are only used for converting to mp3 I think. It gives me an ImportError when I try to use it though.

Can we still use the plugin if using the python library?

It gives me an ImportError when I try to use it though.

Yeah I've got the same thing, the pypy package is not actually up to date with the repo though - importing from the pip-installed version gives this error:
Traceback (most recent call last):
  File "/home/pi/.local/lib/python3.7/site-packages/festival.py", line 3, in <module>
    from . import _festival
ImportError: attempted relative import with no known parent package

Installing from the repo directly gives this error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/pi/.local/lib/python3.7/site-packages/festival.py", line 6, in <module>
    import _festival
ImportError: /lib/arm-linux-gnueabihf/libestools.so.2.5: undefined symbol: omp_get_thread_num

Not sure how to fix this, any ideas?

Didn't try the python bindings

Can't get this one building either, fails on python3 setup.py build even after installing swig

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/pi/.local/lib/python3.7/site-packages/festival.py", line 6, in <module>
    import _festival
ImportError: /lib/arm-linux-gnueabihf/libestools.so.2.5: undefined symbol: omp_get_thread_num

Adding this fix https://github.com/techiaith/pyfestival/pull/9 gets the python bindings working. The plan is to fork this repo and manage it ourselves since it's relatively simple


PHP 7.4 preloading compatibility

When I try preload zend vendor it hangs. I excluded some clases and this works. But only on dev env it looks that it varies by enviroment, probably load order.
Same php config, on docker I excluded:
'Zend_Soap_Client',
'Zend_Soap_AutoDiscover_Exception',
'Zend_EventManager_SharedEventManager',
'Zend_Feed_Writer',
'Zend_Feed',
And it works but when I move to production env it crash on diferent Classes.

Sorry that this has been sitting here. I'm not sure what to do about your issue

It will be nice to have some preload script that returns correct classmap(not every class but mostly used by zend).
Symfony have this, https://symfony.com/blog/new-in-symfony-5-1-configurable-php-preloading and performance increase is huge

We've already implemented all these features in Symfony code. The result is that in our synthetic benchmarks, a sample application went from 360 requests per second to 630 requests per second, a 75% performance improvement.

I don't really add features to zf1 normally, also this is a php 7.4 only feature which is quite restrictive as many users are still on 7.1 - 7.3
You can always add some preloading yourself to your app loader file to solve this without any changes in the zf1-future codebase.

Are you using Zend_Loader_Autoloader at the moment? This is how we do it in our app loader file.
$loader = Zend_Loader_Autoloader::getInstance();
$loader->suppressNotFoundWarnings(true);
$loader->pushAutoloader(new \Pi\Core\Bootstrap\Autoloader(), ['Dashboard_', 'Xx']);


Fix calling curl_getinfo after curl_close() in sendRequest

Issue https://github.com/orhanerday/open-ai/issues/62

@orhanerday


Ionic 4 - Unsubscribe from observables

Bug Report
Describe the Bug
In ionic 4, what is the way to unsubscribe from observables? In angular this should be in ngOnDestroy function, however according with this issue(https://github.com/ionic-team/ionic/issues/15205) the previous page does not trigger it.
https://angular.io/guide/lifecycle-hooks#ondestroy
Related Code
https://github.com/777wills/destroy-ionicV4
Expected Behavior
Unsubscribe from observables in ngOnDestroy.
Additional Context
https://github.com/ionic-team/ionic/issues/15205

That's not correct. The previous page trigger it

so why is this not working?
I did:
ionic start myApp tabs --type=angular

Then I added the ngOnDestroy in home page:
import { Component, OnDestroy } from '@angular/core';

@Component({
  selector: 'app-home',
  templateUrl: 'home.page.html',
  styleUrls: ['home.page.scss']
})
export class HomePage implements OnDestroy {
  ngOnDestroy() {
    console.log('HomePage destroyed!');
  }
}

Am I doing something wrong?
See the result:

Because the components will not be destroyed (they still exist) - you only forward to another tab

I understand what you mean but in my project the tabs are a little bit different:
<ion-tabs>
  <ion-tab icon="tennisball" href="/page-one">
  </ion-tab>
  <ion-tab icon="chatboxes" href="/page-two">
  </ion-tab>
  <ion-tab icon="person" href="/page-three">
  </ion-tab>
  <ion-router-outlet></ion-router-outlet>
</ion-tabs>

In theory, with ion-router-outlet the previous page should be destroyed. When I use router-outlet (angular), the component is destroyed and the ngOnDestroy works fine.
Everything inside of the router should be replaced.
See how the page is hidden somehow:

Did you read my reply? The last part was about destroying.
If you just switch to another page, the old page is still in the stack because you can go back to this (as you can see your page-two has can-go-back). Again: If you not want to go back, use
this.nav.navigateRoot(['/dashboard']); //nav is NavController from @ionic/angular
Then the old page will be removed from stack and the component will be destroyed

NavController is suggested for special cases only. For example (from the ion4 docs):

This fits use cases where you could have a modal, which needs it's own sub-navigation, but not make it tied to the apps URL.

The recommended router for Ionic 4 Angular is:

For apps the are built with Angular and @ionic/angular, we recommend using the Angular Router which comes out of the box for every new Ionic 4 Angular app. Previous versions of Ionic shipped with our own custom router, but in order to provide the best tooling and developer experience we have since moved to using the framework's recommended router.

Isn't that an issue? I should be able to use Angular Router easily.

@jmchaves No! YOu have to include this into your sourcecode if you switch from one page to another. you can also use angular router of course, no problem

Besides, after switching the tabs a few times I found out the components inside of the ion-router-outlet sometimes get duplicate. See:

I've seen this too.  One of the things I've been excited about with ionic 4 is being able to just use the normal angular lifecycle hooks everywhere on pages AND subcomponents, but that depends on fully turning off the nav stack.
IMO it's not currently super clear how to make sure to avoid the nav stack if you don't want it in a way that works with tabs and without.
The ionic team seems to suggest avoiding the nav controller if you don't need the nav stack, but it doesn't seem to always do what you'd expect.
Maybe a clearer description for this issue would be more about the router.
With each beta release I've been waiting for a clear "if you want to use the nav stack and use the ionic lifecycle hooks, do ___.  if you want to avoid the nav stack and use the angular lifecyclehooks, do ___.  Any behavior otherwise is a bug."  Otherwise it's not clear if we're seeing a bug or just not using it the way that's intended :(.

@captaincaius I agree with you. And definitely this is an ionic router issue.

So, can we just use angular native router-outlet in Ionic 4 app and how? Or any other strong recommendations?

@airstep I don't recommend change it to angular router-outlet because you'll miss animations and some styles. I did the test and the lifecyclehooks works perfect however you would need to add custom styles to fix what I mentioned before which doesn't make sense because we already have ionic to care of it.

So, in what version I can get this care? I get issue when return back from one screen to previous - where current stay on top of next. Bcz ngOnDestroy called only once in first, but in second time it is not...

@airstep, from what I know there are 3 temporary solutions (because we're waiting for Ionic Team answers here):

Set a timeout to unsubscribe the observable, something like this: https://angular.io/guide/observables#basic-usage-and-terms
Go back to Ionic 3.9.x and use ionViewWillLeave: https://ionicframework.com/docs/api/navigation/NavController/
Or as @paulstelzer mentioned you can use navigateRoot instead:
this.nav.navigateRoot([path]);

Does it make sense?

You can use href="path" as long as you include [routerDirection]="root".  Just happened to notice this and tested in my app.  ngOnDestroy gets called.

Regarding the lifecycles, for me this works as explained here: https://github.com/ionic-team/ionic/issues/15633#issuecomment-422430944. It makes sense and you shouldn't unsubscribe if your view hasn't been destroyed yet, otherwise you would be responsible of setting everything up again when the view becomes active again. If you want to use the Ionic Router Outlet, this is part of its behavior, although it would be great if the component allowed us to set if we want to keep a stack or not.
That said, this is not a bug but a feature request, maybe you can join the Slack channel and share your thoughts there: https://ionicworldwide.herokuapp.com/
On the other hand, the elements duplication would be a bug, but that should be discussed in a different issue with clearer steps about how to reproduce it. An example running in https://stackblitz.com/ would help a lot!


Fix library headers.

The library builder did not correctly form the current module's library header.

This PR is generated to support https://github.com/pnuoslab/socketdev/pull/1


JavaScript: repeated enum AssertionError when deserializingBinary

What version of protobuf and what language are you using?
Version: v3.6.1
Language: Javascript with Typescript
What operating system (Linux, Windows, ...) and version?
Windows 10 x64
What runtime / compiler are you using (e.g., python version or gcc version)
im using tsc (Typescript compiler)
In order to generate the .ts files from the .proto im using the command protoc --plugin option with the protoc-gen-ts plugin
What did you do?
Steps to reproduce the behavior:

create a protobuf file that contains a repeated enum message
Send a Protobuf binary from my Android App to my Nodejs Express Server
Attempt to deserialize the buffer in my Express Server

What did you expect to see
a Protobuf Object
What did you see instead?
AssertionError on the repeated enum field.
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
my custom repeated enum field is repeated Amenities amenities = 22; and another one at =23;
my protobuff buffer base64 encoded:
ChoyMDE4LTEwLTA2VDIwOjM5OjI0LjI2NDAyORDcCxgCIAEq8gEKA0dhbBIIU2h0ZW5nZWwangFodHRwczovL3Njb250ZW50LmZzZHYzLTEuZm5hLmZiY2RuLm5ldC92L3QzMS4wLTgvMjYyMjEwMTZfMjA5NzUwMzIwNjk0NTg0Nl82NDE3NzA1NTMyMzI5NDYzMDcyX28uanBnP19uY19jYXQ9MCZvaD1iMjE4Y2ZiNDE2MTFlMWY4YTJjZDA0ODM1MzZjZGI0OCZvZT01QzA5MEFCNCAYggE9SGkgaW0gR2FsLCAyNCBsaWtlcyBhIGxvdCBvZiBzdHVmZiBsaWtlIEZsdXV0dGVyIGFuZCBNb3JlZWVlLjIkMDg3ZWQwM2EtMzJmMS00ZDljLWE1YWEtODQ5Y2U3NDliNWQ1QkkKFteq15wg15DXkdeZ15EgLSDXmdek15USGdep15PXqNeV16og16jXldeY16nXmdec15MaAjU2MVbozfKGCEBAOTaVoBqKY0FAUANYAYIBGjIwMTgtMTAtMDZUMjA6Mzk6MjQuMjY0MjMziAEykAEBmAEEoAEAqgElIEdvb2QgbmV3IGFwYXJ0bWVudCB0byBqb2luIHRvbyAhISEgIbABA7ABArgBAbgBArgBAMABEsgBJNABAA==
my error:
[TypeScript] AssertionError: Assertion failed [TypeScript]     at new goog.asserts.AssertionError (C:\Users\Gal\Desktop\roommiesusersmicroservice\node_modules\google-protobuf\google-protobuf.js:98:603) [TypeScript]     at Object.goog.asserts.doAssertFailure_ (C:\Users\Gal\Desktop\roommiesusersmicroservice\node_modules\google-protobuf\google-protobuf.js:99:126) [TypeScript]     at Object.goog.asserts.assert (C:\Users\Gal\Desktop\roommiesusersmicroservice\node_modules\google-protobuf\google-protobuf.js:99:385) [TypeScript]     at jspb.BinaryReader.readPackedField_ (C:\Users\Gal\Desktop\roommiesusersmicroservice\node_modules\google-protobuf\google-protobuf.js:359:71) [TypeScript]     at jspb.BinaryReader.readPackedEnum (C:\Users\Gal\Desktop\roommiesusersmicroservice\node_modules\google-protobuf\google-protobuf.js:364:287) [TypeScript]     at Function.proto.ProtobufApartment.deserializeBinaryFromReader (C:\Users\Gal\Desktop\roommiesusersmicroservice\dist\protobuf\generated\apartment_pb.js:210:67) [TypeScript]     at Function.proto.ProtobufApartment.deserializeBinary (C:\Users\Gal\Desktop\roommiesusersmicroservice\dist\protobuf\generated\apartment_pb.js:116:34)
Anything else we should know about your project / environment
It crashes when it tries to read my repeated enum field.
please help me..
thanks!

this will require modifying to code generator to allow for either format on the wire and our BinaryReader to add helpers for 'packable' fields


[BUG]

Is there any existing issue for this?

[X] I have searched the existing issues

Description
As this website is already in dark mode wanted to apply light mode. this will be applicable to all the pages
Screenshots
No response
Any additional information?
I am gssoc'23 contributer
What browser are you seeing the problem on?
No response

@Susmita-Dey  kindly assign this to me

hey @Shreya111111, this is feature isn't planned right now. Thank you for rasing an issue.


Add support of Cohere Embed v3

Cohere released the new embedding API (Embed v3: https://txt.cohere.com/introducing-embed-v3/) that treats document and query embeddings differently. This PR updated the CohereEmbeddings to use them appropriately. It also works with the old models.

ahh im not sure this is backwards compatible. @billytrend-cohere does input_type param cause problems if using <v3 model?

It doesn't right now. There was some discussion of making it throw an error. I'd still be inclined to merge the ability to user override the input_type value as in https://github.com/langchain-ai/langchain/pull/12861 but I'll update that PR accordingly.


TypeError in node_modules/whatwg-fetch/dist/fetch.umd.js:535

Error in UniceNotes
TypeError in node_modules/whatwg-fetch/dist/fetch.umd.js:535
Network request failed
View on Bugsnag
Stacktrace
node_modules/whatwg-fetch/dist/fetch.umd.js:535 - onerror

View full stacktrace
Created automatically via Bugsnag

background/no internet connection

network error
app not in active state
fix update 1.3.1
Comment added in Bugsnag by Hugo
TypeError in node_modules/whatwg-fetch/dist/fetch.umd.js:535

An error linked to this issue has been reopened in Bugsnag
TypeError in node_modules/whatwg-fetch/dist/fetch.umd.js:535


Update azure-stack-acs-differences-tp2.md

Hi @jpsnover, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@jpsnover : Thanks for your contribution to the Azure documentation! The author, @MChadalapaka, has been notified to review your proposed change.


Incorrect patch for CURL

The winrt patch for CURLis out of date in deps/3rdparty. The CURLsubmodule SHA is 38e07886ed2792988217a2ffa482ce3a69ca92c2 and the patch is for SHA 5abfa9db8a70974b10b2b5ee1da363ae47fb63a8 which does not seem to exist in the checked out CURLsubmodule. When curl.bat tries to apply the patch it fails due to incompatible versions. The CURLbuild will then fail.
Also, this version of CURL will not pass WACK testing, The patch needs to be updated.

We don't use cURL anymore.


Question: How to get the current DOM at some point of time

Hi, I want to know how can I print or get the DOM tree in some variable at some point of time, for example when page is loaded. thanks a lot. regards.

Hi @alejoloaiza,
There are probably several ways to do that depending on what exactly you're trying to do. You could use a timer or page events or DOM events to trigger calling the DOM tree. There are also several ways to manipulate the DOM. The devtools-protocol docs are really the best reference to the API.
Something like this might help get you started. It uses a Page event to wait for the page to load and then captures the root DOM node:
	// Enable Page events for this tab.
	if enableResult := <-tab.Page().Enable(); nil != enableResult.Err {
		panic(enableResult.Err)
	}

	// Enable the DOM agent for this tab.
	if enableResult := <-tab.DOM().Enable(); nil != enableResult.Err {
		panic(enableResult.Err)
	}

	// Create a channel to receive the DOM data.
	results := make(chan *dom.GetDocumentResult)

	// When the page load event fires, deliver the root DOM node.
	tab.Page().OnLoadEventFired(func(event *page.LoadEventFiredEvent) {
		results <- <-tab.DOM().GetDocument(&dom.GetDocumentParams{})
	})

	// Wait for the handler to fire
	domNode := <-results

	tmp, _ := json.MarshalIndent(domNode, "", "    ")
	fmt.Printf("DOM: %s\n\n", string(tmp))

Hi @mkenney Thanks for the quick reply, I will test it and let you know. regards.


[SPARK-21100][SQL][Followup] cleanup code and add more comments for Dataset.summary

What changes were proposed in this pull request?
Some code cleanup and adding comments to make the code more readable. Changed the way to generate result rows, to be more clear.
How was this patch tested?
existing tests

cc @aray

Test build #79380 has started for PR 18570 at commit 3f2b41b.

Test build #79380 has finished for PR 18570 at commit 3f2b41b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79380/
Test FAILed.

Test build #79385 has started for PR 18570 at commit d57edd4.

Test build #79385 has finished for PR 18570 at commit d57edd4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79385/
Test PASSed.

cc @gatorsmile

Test build #79433 has started for PR 18570 at commit d82475f.

Test build #79433 has finished for PR 18570 at commit d82475f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79433/
Test PASSed.

LGTM

Thanks! Merging to master.


Shorter Download URL

Devices have a restriction on the download URL length, is there a way the ADU generated download URl can be shortened ?
Currently it is around 170 chars.
"fileUrls": {
"00000": "http://nonprodcentral--iotnonproddeviceupdates.b.nlu.dl.adu.microsoft.com/westus2/nonprodcentral--iotnonproddeviceupdates/9a2f646d027347d9a01c9d7554927350/DebugPkg_2_0.bin"
}

Hi @sourabhkrishna

How short do you need the URL to be?
Where is the restriction coming from? Is it from JSON parsing, or is it from the HTTP stack itself that it cannot download from any URL > x chars?

We're wondering if providing a shorter URL that HTTP 302 Redirect to the longer URL might solve your problem. Thanks.

@devlie  ya currently this seems to be limitation from our legacy firmware end. They cant accept length of url more than 100 chars.
Will the  HTTP 302 Redirect owned by the ADU Service ?

@devlie  Yup. the 100 chars limitation is due to memory allocations.

How can we enable https for these fileurls?


Feature/lit 3580 refactor js sdk change all enums to constants

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@FedericoAmura It looks like this brach was meant to be merged into staging/v7 but has been pointed to master. There seems to be artifacts which have resulted in the e2e tests failing.


Add note about cache busting

This has come up a few times on the forum, etc. When devs first get started with plugins, this seems to be a common stumbling block. We should stick some inline comments in here in a few places that remind folks to run qiime dev refresh-cache when necessary.

👍 , it'd also be helpful to document this in the "plugin development" page: https://docs.qiime2.org/2.0.6/plugins/developing/


FFmpegPostProcessorError

Maybe I'm off somewhere, but I get this weird error:
Traceback (most recent call last):
  File "/home/user/.cache/pypoetry/virtualenvs/bbb-dl-vRrSF9wO-py3.7/bin/bbb-dl", line 33, in <module>
    sys.exit(load_entry_point('bbb-dl==0.5', 'console_scripts', 'bbb-dl')())
  File "/home/user/.cache/pypoetry/virtualenvs/bbb-dl-vRrSF9wO-py3.7/lib/python3.7/site-packages/bbb_dl/main.py", line 690, in main
    args.keep_tmp_files,
  File "/home/user/.cache/pypoetry/virtualenvs/bbb-dl-vRrSF9wO-py3.7/lib/python3.7/site-packages/bbb_dl/main.py", line 220, in run
    self.ffmpeg.mux_slideshow_with_webcam(slideshow_path, webcams_path, webcam_w, webcam_h, result_path)
  File "/home/user/.cache/pypoetry/virtualenvs/bbb-dl-vRrSF9wO-py3.7/lib/python3.7/site-packages/bbb_dl/ffmpeg.py", line 88, in mux_slideshow_with_webcam
    "ultrafast",
  File "/home/user/.cache/pypoetry/virtualenvs/bbb-dl-vRrSF9wO-py3.7/lib/python3.7/site-packages/bbb_dl/ffmpeg.py", line 58, in run_ffmpeg_multiple_files
    raise FFmpegPostProcessorError(msg)
youtube_dl.postprocessor.ffmpeg.FFmpegPostProcessorError: file:2021-02-12T11-33-02_TEST 2020/21.mp4: No such file or directory

You got any idea why that is? Can it be, that he interprets the "/" as a directory in the filename?

Yes, this is the problem, ffmpeg is crashing here. I rewrote this line. I'll send an MR in a sec.

Alright. That fixed it for me. :blush: ... and now, let's get some :coffee:

Thanks a lot :)
pathvalidate seems to be a good choice. We could have also used the function of youtube-dl. But I think so it is also very good.


8Muses ripper sub-albums mis-handled

Ripme version: 1.7.95
Java version: 1.8.0_311
Operating system: Windows 11 Version 21h2

Exact URL you were trying to rip when the problem occurred:
https://comics.8muses.com/comics/album/Bobby-Tally-Comics
This is a good example, but the problem occurs in any album where there are sub-albums

Please include any additional information about how to reproduce the problem:

Expected Behavior
The layout of the download folders and subfolders should follow the same nesting as the albums and sub-albums on the website, or if that is not possible due to inconsistencies in how the website displays or stores images, then at least when sub-albums have duplicate names, they should be renamed in the download folders (for example, add a numeral to the folder name). This would at least prevent overwriting in duplicate folders thus resulting in lost content.
Actual Behavior
The ripper takes all the sub-albums and places them in the top-level folder. If any sub-albums have the same name, the ripper overwrites the files in the same-named folder, thus losing the content. Since that content is then marked in the URL history file as already downloaded, you cannot recover the missing content even by manually ripping each individual sub-album. At least not without first figuring out the image storage names and deleting them from the URL history file.

try this: https://github.com/ripmeapp2/ripme/releases


ci: add nightly performance testing, update localnet Docker image to Debian Bookworm, remove unnecessary build-jet runners.

Description
ci: add nightly performance testing, update localnet Docker image to Debian Bookworm, remove unnecessary build-jet runners.
Closes: DEVOP-699
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] CI Updates

How Has This Been Tested?
The docker image has been built locally to validate it builds and tested. Need to merge to register the performance pipeline for testing.

[ ] Tested CCTX in localnet
[ ] Tested in development environment
[ ] Go unit tests
[ ] Go integration tests
[ ] Tested via GitHub Actions
[x] Locally

Checklist:

[x] I have added unit tests that prove my fix feature works

zetae2e: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by zetae2e)

e2e has glibc version missmatch issues. You need to upgrade the orchestrator too.


Feature/artifactory image source

Story:
https://github.com/ploigos/infra-ops/issues/40
Successful build:
https://github.com/ploigos/spring-petclinic/runs/7138968570?check_suite_focus=true

@jthompkins Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@jthompkins Thank you for signing the Contributor License Agreement!


No found file  ./data/extclsshpinfo.py

Where is  ./data/extclsshpinfo.py?

https://github.com/wentaozhu/DeepLung/tree/master/nodcls/data


How to use AZURE_PW_TEST_RUN_ID?

I want to add some logging involving AZURE_PW_TEST_RUN_ID, e.g. a link to the run UI in azure.
Since onEnd is async the env var is set at some point in time I can't control or hook into.
I have tried reading it upon completion but it will not persist.
I've also tried to pipe the output and use regex to get it from the logs but couldn't achieve that since the process is detached.
Therefore, I do not see how to read it apart from overriding the onEnd method:
import reporter from "@alex_neo/playwright-azure-reporter";

const onEnd = reporter.prototype.onEnd;
reporter.prototype.onEnd = async function (...args) {
  const retval = await onEnd.apply(this, args);
  console.log("READY:", process.env.AZURE_PW_TEST_RUN_ID);
  process.emit('message', { runId: process.env.AZURE_PW_TEST_RUN_ID }); // inform parent process
  return retval;
};

export default reporter

Perhaps I should PR
process.emit('message', { runId: process.env.AZURE_PW_TEST_RUN_ID }); // inform parent process

And a log of type
    `URL: ${this._orgUrl.replace("vssps.", "")}/${
      this._projectName
    }/_testManagement/runs?runId=${runId}&_a=resultQuery`
  );

AZURE_PW_TEST_RUN_ID been defined at onBegin hook after TestRun was created and this action also logging in console
How I use it. I need send some report in email after run was finished and in report I'm made TestRun URL, like you, and for that I have created my custom reporter as it writed in docs and took RUN_ID from env like
const runId = process.env.AZURE_PW_TEST_RUN_ID
`${this._orgUrl.replace("vssps.", "")}/${this._projectName}/_testManagement/runs?runId=${runId}&_a=resultQuery`

Thanks, I think I tried that but the reported didn't wait for onEnd to complete since it is async so I couldn't use the env var.
I don't see the env var being set in onBegin, only onEnd, running with testResults not testRun.
Singals to the parent process is very simple and allows me to easily add the following the playwright.config.ts:
process.on('message', ...)

can be closed

Yeah, and since it is async in nature the best thing I thought of doing is hooking into onEnd
Might be worth to expose an onEnd hook that can be passed via the config object
That sounds the best solution to me


How to backdrop-close menu on mobile?

In my aside menu I have a vertical accordion, so the "close" button sometimes (when some item is opened) is visible and sometimes it's not. So I rely on backdrop: true to let user close my aside menu.
On desktop it works great, but on small mobiles (up to at least a width of 640px) it doesn't, since the aside menu fills ut the screen...
Can you suggest a solution, or even just a loophole :-) ?
Consider that on small screens, with a crowded accordion item open, a close button on the bottom of the aside menu should be usually out of view.

Hi @marcolino,
Scrolling is possible inside aside. So, overflowed close button could be used by scrolling to bottom. Another solution would be placing the button in modal-header.

Hi, @dbtek,
of course scrolling is possible, but I feel it's not very instinctive, for the user, to have to go looking for a button to close a menu... :-)
I though to the button in the modal header, too, but I feel a close button in the header is not very common, and - this, too - no so instinctive...
These are just personal thoughts, of course...
However, the fact that the menu completely fills the (mobile) screen is a voluntary decision?
Shouln't it be possible to leave at least a small portion of the background visible and clickable, on the right side?

Hi @marcolino,
I could not reproduce your problem. There is actually no width rules other than ones for .modal-sm .modal-lg classes.
May be it's about the content you place in aside. No matter what causes this, you should be able to add custom styles that fits aside to your needs. I am not sure it's related to angular-aside.


Feature request: tab completion in python debug consolue

Issue Type: Feature Request
During debug, in the DEBUG-CONSOLE window, there is no support for tab-completion.
such a feature would greatly improve the consule usability.
Currently, pressing TAB causes the focus to move to other vscode views.
VS Code version: Code 1.52.0 (940b5f4bb5fa47866a54529ed759d95d09ee80be, 2020-12-10T22:43:50.597Z)
OS version: Linux x64 5.4.0-56-generic

This actually works fine for me.
Can you reproduce this problem with VS Code insiders https://code.visualstudio.com/insiders/
If yes, what are your exact steps?
Here are mine:

Start debugging ,hit a breakpoint
Write something in debug console -> see suggest box shows up
Press Tab suggest box completion is accepted as expected

thanks @isidorn
it works fine.
TAB seems to work when a suggestion already appears in the console, otherwise, Ctrl+Space is used to suggest completions

As this seems to work -> closing

As this seems to work -> closing


MQTT source kamelet does not include topic

It is common for MQTT use cases to distribute data over multiple topics. You might have different subtopics for different devices for example. The MQTT source kamelet supports subscribing to multiple topics by using wildcards but the information about the topic is lost because the MQTT source only reports the body of the MQTT message to the sink. It would be great, if the topic could be included in the body that is reported to the sink.
Example:
You have multiple devices publishing to the topics:
/data/deviceId1
/data/deviceId2
...
The MQTT source is able to subscribe to the topic /data/# and it receives all messages. However, the information about which device sent the message is lost.

Nevermind, figured it out. For everyone else wondering: The topic is included in the header which is not displayed by the log sink by default.


[New Product] Joomla! (closes #1918)

Cycles are bounded to the major version because, according to Joomla! documentation, one should always use or update to the latest major version's minor.patch. It may change depending on Joomla! team response on https://github.com/joomla/joomla-websites/issues/1842.
I did not document cycles 2 and 1 because there is a lack of information about thoses versions (see https://docs.joomla.org/Past_release_and_support_cycle).

Thanks for the review @captn3m0. I just added the support column : according to https://docs.joomla.org/Joomla!_CMS_versions Joomla! 3 has no support for bug fixes.
If everything looks good to you I let your merge this PR.


Docs: Add info about setting default k8s ns; fix typos/formatting

Kubernetes documentation: Add info on how to set the enmasse namespace as the default namespace after having created the namespace. This is needed as the following kubectl commands don't have the --namespace param.

Thanks!


Recursive renaming

I want to rename some folders recursively. For example, I have a folder structure:
Folder1
-Folder2

And I want to replace "Folder" word with "Project" word. When I use Folder.makeSubfolderSequence(recursive:includeHidden:) method, I can't rename Folder2 after Folder1 renaming, because it has old path - Folder1/Folder2. How I can deal with it? Maybe Item.rename(to:keepExtension:) method should update paths of subfolders?

Yeah this is an interesting one. Seems like it's not propagated in the iteration. Would love it if you could have a look at this one and submit a PR with a fix, if you have time 👍

I guess it would be difficult, because it's Item method, not only Folder class. Can you suggest any vectors of implementation?

@artemnovichkov I think for your case, it could be sufficient to just reverse the sequence before iterating over it. This worked for me:
$ tree
.
└── Folder1
    └── Folder2

for folder in try Folder(path: ".").makeSubfolderSequence(recursive: true, includeHidden: false).reversed() {
	try folder.rename(to: "renamed")
}

$ tree
.
└── renamed
    └── renamed

Think I can fix this. Will work on a PR tonight..