Make occlient a superset of kclient

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:

/kind code-refactoring

What does does this PR do / why we need it:
This is an alternate draft PR to https://github.com/openshift/odo/pull/2626, exploring an alternate approach to handling overlap between occlient and kclient.
Rather than keeping occlient and kclient functions completely separate, this PR adds an instance of kclient to the occlient instance, thus making occlient a superset of kclient. This works out nicely because occlient already has all of the fields that exist in kclient so we can use this as a chance to refactor and reduce some code duplication.
Additionally, this approach prevents us from having to pass in both occlient and kclient instances into the CLI context (and from having to pass in both to functions that require them).
As a summary, this PR does the following:

Moves the file sync code from the occlient package to the kclient package, so that it can be used with both odo "classic" push and odo "devfile" push.
Adds a kclient instance to the occlient object

Removes the following duplicated fields from occlient that already existed in kclient:

Namespace
KubeClient
ClientConfig

Updates existing code to use occlient.kclient where applicable

All unit tests passing:
johns-mbp-3:odo johncollier$ make test
go test  -race github.com/openshift/odo/cmd/cli-doc github.com/openshift/odo/cmd/odo github.com/openshift/odo/pkg/application github.com/openshift/odo/pkg/application/labels github.com/openshift/odo/pkg/auth github.com/openshift/odo/pkg/catalog github.com/openshift/odo/pkg/component github.com/openshift/odo/pkg/component/labels github.com/openshift/odo/pkg/config github.com/openshift/odo/pkg/debug github.com/openshift/odo/pkg/devfile github.com/openshift/odo/pkg/devfile/parser github.com/openshift/odo/pkg/devfile/versions github.com/openshift/odo/pkg/devfile/versions/1.0.0 github.com/openshift/odo/pkg/devfile/versions/common github.com/openshift/odo/pkg/kclient github.com/openshift/odo/pkg/log github.com/openshift/odo/pkg/log/fidget github.com/openshift/odo/pkg/machineoutput github.com/openshift/odo/pkg/notify github.com/openshift/odo/pkg/occlient github.com/openshift/odo/pkg/odo/cli github.com/openshift/odo/pkg/odo/cli/application github.com/openshift/odo/pkg/odo/cli/catalog github.com/openshift/odo/pkg/odo/cli/catalog/describe github.com/openshift/odo/pkg/odo/cli/catalog/list github.com/openshift/odo/pkg/odo/cli/catalog/search github.com/openshift/odo/pkg/odo/cli/catalog/util github.com/openshift/odo/pkg/odo/cli/component github.com/openshift/odo/pkg/odo/cli/component/ui github.com/openshift/odo/pkg/odo/cli/config github.com/openshift/odo/pkg/odo/cli/debug github.com/openshift/odo/pkg/odo/cli/login github.com/openshift/odo/pkg/odo/cli/logout github.com/openshift/odo/pkg/odo/cli/preference github.com/openshift/odo/pkg/odo/cli/project github.com/openshift/odo/pkg/odo/cli/service github.com/openshift/odo/pkg/odo/cli/service/ui github.com/openshift/odo/pkg/odo/cli/storage github.com/openshift/odo/pkg/odo/cli/ui github.com/openshift/odo/pkg/odo/cli/url github.com/openshift/odo/pkg/odo/cli/utils github.com/openshift/odo/pkg/odo/cli/version github.com/openshift/odo/pkg/odo/genericclioptions github.com/openshift/odo/pkg/odo/util github.com/openshift/odo/pkg/odo/util/completion github.com/openshift/odo/pkg/odo/util/experimental github.com/openshift/odo/pkg/odo/util/validation github.com/openshift/odo/pkg/preference github.com/openshift/odo/pkg/project github.com/openshift/odo/pkg/secret github.com/openshift/odo/pkg/service github.com/openshift/odo/pkg/storage github.com/openshift/odo/pkg/storage/labels github.com/openshift/odo/pkg/testingutil github.com/openshift/odo/pkg/testingutil/filesystem github.com/openshift/odo/pkg/url github.com/openshift/odo/pkg/url/labels github.com/openshift/odo/pkg/util github.com/openshift/odo/pkg/version
?   	github.com/openshift/odo/cmd/cli-doc	[no test files]
?   	github.com/openshift/odo/cmd/odo	[no test files]
ok  	github.com/openshift/odo/pkg/application	(cached)
ok  	github.com/openshift/odo/pkg/application/labels	(cached)
?   	github.com/openshift/odo/pkg/auth	[no test files]
ok  	github.com/openshift/odo/pkg/catalog	(cached)
ok  	github.com/openshift/odo/pkg/component	(cached)
ok  	github.com/openshift/odo/pkg/component/labels	(cached)
ok  	github.com/openshift/odo/pkg/config	(cached)
ok  	github.com/openshift/odo/pkg/debug	(cached)
ok  	github.com/openshift/odo/pkg/devfile	(cached)
ok  	github.com/openshift/odo/pkg/devfile/parser	(cached)
ok  	github.com/openshift/odo/pkg/devfile/versions	(cached)
?   	github.com/openshift/odo/pkg/devfile/versions/1.0.0	[no test files]
ok  	github.com/openshift/odo/pkg/devfile/versions/common	(cached)
ok  	github.com/openshift/odo/pkg/kclient	(cached)
?   	github.com/openshift/odo/pkg/log	[no test files]
?   	github.com/openshift/odo/pkg/log/fidget	[no test files]
?   	github.com/openshift/odo/pkg/machineoutput	[no test files]
ok  	github.com/openshift/odo/pkg/notify	(cached) [no tests to run]
ok  	github.com/openshift/odo/pkg/occlient	(cached)
?   	github.com/openshift/odo/pkg/odo/cli	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/application	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/catalog	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/catalog/describe	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/catalog/list	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/catalog/search	[no test files]
ok  	github.com/openshift/odo/pkg/odo/cli/catalog/util	(cached)
?   	github.com/openshift/odo/pkg/odo/cli/component	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/component/ui	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/config	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/debug	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/login	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/logout	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/preference	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/project	[no test files]
ok  	github.com/openshift/odo/pkg/odo/cli/service	(cached)
ok  	github.com/openshift/odo/pkg/odo/cli/service/ui	(cached)
ok  	github.com/openshift/odo/pkg/odo/cli/storage	(cached)
?   	github.com/openshift/odo/pkg/odo/cli/ui	[no test files]
?   	github.com/openshift/odo/pkg/odo/cli/url	[no test files]
ok  	github.com/openshift/odo/pkg/odo/cli/utils	(cached)
?   	github.com/openshift/odo/pkg/odo/cli/version	[no test files]
?   	github.com/openshift/odo/pkg/odo/genericclioptions	[no test files]
ok  	github.com/openshift/odo/pkg/odo/util	(cached)
ok  	github.com/openshift/odo/pkg/odo/util/completion	(cached)
ok  	github.com/openshift/odo/pkg/odo/util/experimental	(cached)
ok  	github.com/openshift/odo/pkg/odo/util/validation	(cached)
ok  	github.com/openshift/odo/pkg/preference	(cached)
ok  	github.com/openshift/odo/pkg/project	(cached)
ok  	github.com/openshift/odo/pkg/secret	(cached)
ok  	github.com/openshift/odo/pkg/service	(cached)
ok  	github.com/openshift/odo/pkg/storage	(cached)
ok  	github.com/openshift/odo/pkg/storage/labels	(cached)
ok  	github.com/openshift/odo/pkg/testingutil	(cached)
?   	github.com/openshift/odo/pkg/testingutil/filesystem	[no test files]
ok  	github.com/openshift/odo/pkg/url	(cached)
ok  	github.com/openshift/odo/pkg/url/labels	(cached)
ok  	github.com/openshift/odo/pkg/util	(cached)
?   	github.com/openshift/odo/pkg/version	[no test files]

Which issue(s) this PR fixes:
https://github.com/openshift/odo/issues/2473 (partially)
How to test changes / Special notes to the reviewer:

/test v4.1-integration-e2e-benchmark

/test v4.1-integration-e2e-benchmark


[DOCS-12567] Updating inconsistent policy names

A bug was discovered in the docs where a policy had two different names on the same page in the Docs and the policy name did not match the name on the release notes page. While determining what the policy name was in the UI to update it correctly, I noticed a couple of other discrepancies in the policy docs that I went ahead and updated as well. To see a full analysis see the Jira issue where I attached a Google doc with a table and some explanations.

!gate waiver add terminology
The terminology is getting tripped up by content end-users will not see that is required to ensure proper redirects for customers from old pages to new ones that resolve terminology issues.
The AsciiDoc code looks like this:
modules/ROOT/pages/legacy-ip-allowlist.adoc
5::page-aliases: ip-whitelist.adoc
And it results in a redirects file for nginx that is never exposed to customers.

!gate waiver add terminology

!gate waiver add terminology

!gate waiver menu

!gate waiver add Terminology


parsers: regex: check if regex from template is a string

Incorrectly formatted templates may result in passing OrderedDict as a
regex. That makes re.findall fail like:

  File "/home/foo/invoice2data/src/invoice2data/extract/parsers/regex.py", line 33, in parse
    matches = re.findall(regex, content)
  File "/usr/lib64/python3.6/re.py", line 222, in findall
    return _compile(pattern, flags).findall(string)
  File "/usr/lib64/python3.6/re.py", line 289, in _compile
    p, loc = _cache[type(pattern), pattern, flags]
TypeError: unhashable type: 'collections.OrderedDict'

Handle such case by displaying a meaningful warning.

I experienced this while trying to debug https://github.com/invoice-x/invoice2data/issues/441

This sould also help with https://github.com/invoice-x/invoice2data/issues/339


Deployment server

Change to make the forwarder to call the ${server}:${splunkd_port} deployment server using the /opt/splunkforwarder/etc/system/local/deploymentclient.conf file.
This is the default file used during a manual install of a forwarder with the command "./splunk set deploy-poll ${server}:${splunkd_port}".

Hey @NC-Alex Could to please clarify what exactly your are achieving with this PR.  Are you fixing a bug or adding a feature?  I believe this module already setups the deployment client on the forwarders unless there is a bug or something i'm forgetting

Hi,
Sorry for taking so much time to answer.
I made the requested changes.
The purpose of this branch is about to set the informations to contact a deployment server during the installation like here http://docs.splunk.com/Documentation/Splunk/7.0.2/Updating/Configuredeploymentclients by filling the /opt/splunkforwarder/etc/system/local/deploymentclient.conf file.
With that file filled, the Universal Forwarder is automatically registering to the main server, from witch you can choose the apps you want to deploy to the registered Universal Forwarders.
Thanks,

Hey @NC-Alex ,
Thank you for clarifying your intentions.  I think this PR should be closed though since.
1.) You can already achieve this on your forwarder clients by crafting a manifest like this
class { 'splunk::params':
    server  => 'splunkserver-001.local',
    version => '7.0.0',
    build   => 'c8a78efdd40f',
}

include splunk::forwarder

 ###############################################
 # Configure forwarder to be a deployment client
 ###############################################
splunkforwarder_deploymentclient { 'deployment-server':
    section => 'target-broker:deploymentServer',
    setting => 'targetUri',
    value   => 'splunkserver-001.local:8089',
    require  => Class['splunk::forwarder'],
}

Also i'm you have hardcoded pulling in the all logs from '/var/logs' and classifying them of puppet-server source type for everyone who would use this module which is something that is site specific and should not be here.

L95-L202 of params.pp
@NC-Alex Any objections for me closing this?

Hi @TraGicCode
It's all good.
Thanks!


Add read check in IO#getbyte

This is similar to #1190 where we had to convert the exception in case of no write access. Two helper methods have been added to help with this check, I expect we'll be needing a lot more of them.
https://github.com/ruby/spec/pull/1068 has been added as an upstream request to add this to the specs. For the time being, a local test has has been added.

This needs a different approach. The value of sb.st_mode is the mode on the filesystem, so if I have a file that is writable for the user, and open that file in read only mode, it would still return this file as writable.

This one is good to go now


Hotfix for glfw's canvasHeight

Fixes https://github.com/EngoEngine/engo/issues/300

LGTM. Merge when compiles :smile:


Return proper URI when channel name is present.

FIx for #460. Now the channel's claim_id is appended in after the channel_name in the URI.

@hackrush01 FYI you should be able to branch off the main lbryio repo rather than fork.

@hackrush01 this fix had a bug, see 90c83c6

And my fix had a bug, proper fix is 8fd9d64


build statically-linked sui-node for testing on glibc-based hosts

Description
Update Deterministic Sui Build to compile a statically-linked binary.
StageX uses musl for compiling sui-node.
In order for this to run the sui-node binary directly on glibc-based linux macines, it must be statically linked.
Test plan
Cherry-pick commit to testnet branch and test on a private testnet.

build docker image
extract sui-node binary
confirm statically linked (run file on mac, ldd on linux)
upload to baremetal test server + run
check build is still deterministic from github action build, linux machine, macOS machine

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:

relates to: #13476


"Using '*' activation is usually a bad idea as it impacts performance."

https://github.com/vscode-neovim/vscode-neovim/blob/9479c650a3ff79382c6b86e84b16bc317fd3894f/package.json#L34

Could it be improved?

As long as startup is super fast this doesn't matter.


cmd/atlas: omit url strings from parse errors

Before this PR, passing Atlas an invalid URL returned an error that includes the problematic URL (in url.Error.URL):
$ atlas schema inspect -u "  mysql://user:pass@:3306"

Error: parse "  mysql://user:pass@:3306": first path segment in URL cannot contain colon

This might be useful in local development, but it should be avoided, as Atlas is mostly used in CI/CD environments. This PR changes the behavior by introducing a new function (sqlclient.ParseURL) that returns parse errors without the raw URLs, and it is used by the various cmd packages. For example:
$ atlas schema inspect -u "  mysql://user:pass@:3306"

Error: first path segment in URL cannot contain colon

I think we should add golint rule to block url.Parse for all places

I think we should add golint rule to block url.Parse for all places

It’s only for database URLs. For other URLs, like directory path, we use url.Parse and return the full error.


PLAT-122719: Picker: onChange should be called with a proper value

Added 'data' prop on the onChange event so it sows the new selected value
Enact-DCO-1.0-Signed-off-by: Daniel Stoian<EMAIL_ADDRESS>

Closing as WNF


Fix: better handling for sparkswapd failing to start

Description
This change shuts down the sparkswapd process if it fails to start so that pm2 can restart it.
It also improves logging from these failures to make it more clear what the issue is - particularly from the most common issue of lnd not being ready.
Related PRs
List related PRs if applicable
Todos

[ ] Tests
[ ] Documentation
[ ] Link to Trello

@treygriffith PM2 will only restart the process 10 times on the broker https://github.com/sparkswap/broker/blob/master/pm2.json#L5 before the container is set to an error status (and must be restarted through docker-compose.
The restarting happens pretty quickly, so the edge case of lnd not being up because of blockchain indexing, still exists.
Unfortunately I dont have any suggestions for a solution on that yet, but this PR is a step in the right direction.


Add pinpoint blog, fix logo width exceeds the outer layer.

Fix #43

Please resolve  the conflict

Hi, I see at least 3 changes:

pinpoint blog
Fix for issue #43
Add handu logo

It should be separated into 3 pull request, I've added a pull request template just now, please check https://github.com/apache/incubator-dubbo-website/blob/asf-site/PULL_REQUEST_TEMPLATE.md
For 3, could you please share some information and statistics on how you use Dubbo?

company's home page
in what scenario are you using Dubbo?

Or you can posted here, and provide link. English preferred.

Got it. I'll repost use this template.


several solutions in routingmodel?

I'm using google OR tools (python) to solve a vehicle routing problem. Now, as the solution given is not optimal, is there a way to get several solutions meeting the requirements?
At this moment I am using following functions to solve the problem:
routing = pywrapcp.RoutingModel(tsp_size, num_routes, depot)
assignment = routing.SolveWithParameters(search_parameters)

No and no plans.
Sorry


883 occds links

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[ X] Code is formatted according to the tidyverse style guide
[ X] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[ X] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[ X] Update to all relevant roxygen headers and examples
[ X] Run devtools::documents() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ X] Address any updates needed for vignettes and/or templates
[ X] Update NEWS.md
[ X] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[ X] Address or fix all lintr warnings and errors - lintr::lint_package()
[ X] Run R CMD check locally and address all errors and warnings - devtools::check()
[ X] Link the issue so that it closes after successful merging.
[ X] Address all merge conflicts and resolve appropriately
[ X] Pat yourself on the back for a job well done!  Much love to your accomplishment!

that was quick @thomas-neitmann - thanks :D


Cross-compile the Solidity compiler

Solidity v0.8.26 just release. Will change the files to compile that instead of v0.8.25.


Memory leak in sample code?

Should free(ConnectionInfo) on exit paths after malloc'ing it.

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

ID: c8da312b-a7f3-9797-e0ea-09806d790748
Version Independent ID: 4d3cff4f-ca06-29b8-af44-6fe0290eb45b
Content: IOCTL_USB_GET_NODE_CONNECTION_INFORMATION_EX (usbioctl.h) - Windows drivers
Content Source: wdk-ddi-src/content/usbioctl/ni-usbioctl-ioctl_usb_get_node_connection_information_ex.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @duncanmacmichael
Microsoft Alias: dumacmic

Hi @colemandavid, good call. Oftentimes, we simplify our sample code to leave out most (if not all) error handling code so we can make it clearer, but in this case I agree it's a good idea to make sure we don't include a memory leak in our example. :) I've fixed the last few lines of code to make sure we free the memory; those changes should be live later today. Thanks!


Bug: Redirect User to ThankYou page

What happened?
Once the user filled the "Register Donor" Form, redirect it to "/ThankYou" route
Add screenshots

What browsers are you seeing the problem on?
No response
Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[ ] I want to work on this issue

@Saurav-Pant assign this issue to me

@Saurav-Pant  pls tell where routes are defines, where is index.tsx and app.tsx in this project

Hi @Saurav-Pant
I'd like to work on this bug. Please assign it to me.
Thanks,
[Your Name]

@Aman0413 Thank you "Your Name" 😄

inside the DonorForm folder when the user clicks on register then redirects it to the /ThankYou page @Devvart1


PDF digital signature

I have scenario where I would like to fill a PDF file, add digital signature, embed CRL and OCSP and timestamp a  pdf document,  I need to do this in Javascript the first part is fully covered by your library but how may I do the rest.

@Bibo1978  Did you find a solution to this? I need to add the Digital Signature to a PDF


Wrap-up OES Leg 4/LSK Leg 1 and set up for OES Leg 5/LSK Leg 2

After Lasker is in port on Sunday...

[ ] Add final maps to readme
[ ] Create release for this 'combined' leg
[ ] Update script to be ready for next combined leg

@jlkeating I'm expecting one last updated Lasker acoustics database to included data from the 3rd and 4th. Does that sound correct to you, and if so, will you let me know once it is up and I can do the final run to close Lasker Leg 1. Thank you!!

Ran final acoustics, made release, set up for Leg 5, and ran first day of Leg 5 since we already had a das file.


Boundary issues with countries that are intersecting

Some of the smaller islands and countries do not have boundaries that are perfectly constructed. For example, generating zones for Kiribati results in generating zones in neighbouring areas. See image below, blue regions are not part of Kiribati yet they generate results, the results should just be for the red area.

Countries that are too wide in relation to existing data

hi @LunaAsefaw please see here: https://wisdomanswer.com/what-countries-does-the-international-date-line-pass-through/
Also sharing some resources that might help you guys brainstorm for a solution:
https://towardsdatascience.com/around-the-world-in-80-lines-crossing-the-antimeridian-with-python-and-shapely-c87c9b6e1513
https://gis.stackexchange.com/questions/181526/polygons-across-international-dateline-180-180-longitude

Thanks Clara!
On Tue, 30 May 2023, 13:08 Clara Ivanescu, @.***> wrote:

hi @LunaAsefaw https://github.com/LunaAsefaw please see here:
https://wisdomanswer.com/what-countries-does-the-international-date-line-pass-through/
Also sharing some resources that might help you guys brainstorm for a
solution:
https://towardsdatascience.com/around-the-world-in-80-lines-crossing-the-antimeridian-with-python-and-shapely-c87c9b6e1513
https://gis.stackexchange.com/questions/181526/polygons-across-international-dateline-180-180-longitude
—
Reply to this email directly, view it on GitHub
https://github.com/kartoza/rezoning-2-project/issues/71#issuecomment-1568238851,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVQQUU2BSC4DXVHFVGCR5PLXIXIKJANCNFSM6AAAAAAYGSIUHI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
This message and attachments are subject to a disclaimer. Please refer to
http://www.it.up.ac.za/documentation/governance/disclaimer/
http://www.it.up.ac.za/documentation/governance/disclaimer/ for full
details.

Kiribati no longer returns results but every other country work


Questions on MTSAC

Description
In the documentation, the example commands for MTSAC-MetaWorld do not seem to include task index as model's input, which means that the model has no way of identifying different tasks.

Should I add agent.multitask.should_use_task_encoder=True if I want to concat task id with state? Or can you provide example codes for running MTSAC with task id concated with state?

Hi! The MTSAC uses different entropy regularizers for each task and hence task id is not passed to the encoder. I dont think we support concatenating task id. The closest thing is concatenating the task emebdding. For that, you can set agent.multitask.should_use_task_encoder=True


It looks like this is your first time opening an issue in this project!

It looks like this is your first time opening an issue in this project!

Muñeco

Sopaxd686


change stats display color to yellow

Use markdown instead of  and / HTML tags.
flow-docs/get-started-logic-template.md
flow-docs/includes/learning-approval-center.md
flow-docs/odata-filters.md
flow-docs/sign-up-sign-in.md

Use markdown instead of HTML for apostrophe.
flow-docs/index.md

#please-close

'#sign-off'

hello there '#sign-off' comment


Use CircleCI matrix jobs to test with the latest version

This PR updates .circleci/config.yml to use CircleCI matrix jobs.
It can make it easy to test with multiple Ruby and Kafka versions, and also can simplify duplicate statements.
Here are the details:

Upgrade to next-gen Docker convenience images
The following message is shown in Circle CI job page.

You’re using a deprecated Docker convenience image. Upgrade to a next-gen Docker convenience image.

The link above states the following.

Moving from a legacy to next-gen image requires a change to the namespace. All legacy images have a Docker namespace of circleci, while next-gen images have a Docker namespace of cimg. For example, migrating from the legacy Ruby or Python image to the respective next-gen image can be done as follows:

circleci/ruby:2.3.0 → cimg/ruby:2.3.0
circleci/python:3.8.4 → cimg/python:3.8.4

circleci/ruby:2.5.1-node is used now, but because of the following reasons, cimg/ruby:2.5 will be used.

cimg doesn't provide Ruby 2.5.1, and the latest patch version is 2.5.9.
Specify a minor release tag for the latest patch version to use.
https://circleci.com/developer/images/image/cimg/ruby#image-tags
It seems that Node.js is not needed, so I omit the suffix -node.

BTW, Ruby 2.5 has already reached EOL, but it's left as is in this PR.

While unit tests are currently only run on Ruby 2.5, matrix is used so that we can add test for other Ruby versions later.

There are two types of kafka images(provided by wurstmeister/bitnami), with different environment variables to pass.
So I defined jobs for each(functional-with-wurstmeister/functional-with-bitnami).

The workflow name was set for clarity. Note that the name has changed from the previous name.
Before:
Show image

After:
Show image

The CI status can be seen here.
Functional tests fail randomly, but I believe this is not related with changes in this PR.

@dasch Sorry to bother you, but I would appreciate it if you review this PR, thanks!

Thanks!

Thank you for merging it! But it appears that CI failed due to authentication issues.
https://app.circleci.com/pipelines/github/zendesk/ruby-kafka/253

Could not find a usable config.yml, you need to reauthorize and approve the SAML-protected org.
Please sign out of CircleCI and log back in with your VCS before triggering a new pipeline.

Could you take a look at it?


R statistical package: key chisq.test() function not working

Describe the issue
A clear and concise description of what the issue is:
Wanting to rely on UTM to do more R statically package work. Almost all R functions and packages seem to work except for the important chi squared R base function (chisq.test()). Consistently gives an "illegal operation" error and crashes the R program but not UTM. Confirmed not working with multiple different small datasets as examples with either R version 3.5.2 or 3.6.3 despite multiple attempted re-installs of each. Function works fine with same data set in R in Mac Terminal.
Interestingly, function fails in R 3.6.3 in iSH emulator app also with an illegal operation--perhaps a problem with emulators on iPad and R program? Also interesting is that the function does work when used in the App Store R app "R Analyst" (although that R version is much older and not running through an additional host iOS emulator).
Has anyone else tried to use the chisq.test() function in R on UTM on their iPAD and gotten it to work? If so, would you please let me know what setup you got the chisq.test() function to work in? Tt is a very key function for my use.
Thanks!
Configuration (required)

UTM Version: 2.24
OS Version: iPAD OS 15.1
Device Model: iPad Pro M1 8GB
Is it jailbroken (name jailbreak used)? No
How did you install UTM? Signulous

Debian 10 x86_64 XFCE
(No crash logs included as did not crash UTM-- just crashed R).

I don't know much about R, you could try following these instructions and paste the output.
Also please paste the exit code when R crashes (do it by running echo $? as the immediate next bash command after the crash).

Thanks, @ktprograms, for such prompt a reply!
I followed all of your suggestions and got the following output (although unfortunately did not give me actual assembly output):
Test case in R:

x = c("Mike", "Mark", "Jeff")
y = c("Blue", "Red", "Red")
chisq.test(x,y)

*** caught illegal operation ***
address 0x7f1fe77db820, cause 'illegal operand'
Traceback:
1: tcrossprod(as.vector(X), as.vector(Y))
2: outer(sr, sc, "*")
3: chisq.test(x, y)
Possible actions:
1: abort (with core dump, if enabled)
2: normal R exit
3: exit R without saving workspace
4: exit R saving workspace
Selection: 1
R is aborting now ...
Illegal instruction

#Entered echo $? right after this crash and got this:
xxx@debian:~$ echo $?
132

#Enabled core dumps and after R crashed followed instructions
and a core dump folder named core.1001 was generated
and as instructed entered "gdb R core.1001" from Bash
and got the following reply:
[New LWP 1001]
Core was generated by `/usr/lib/R/bin/exec/R'.
Program terminated with signal SIGILL, Illegal instruction.
#0  0x00007f7a162cd820 in ?? ()

#Typed in gdb layout asm and go the following response:
[No Assembly Available]
core LWP 1001 In:
Cannot access memory at address 0x7f7a162cd820
(gdb)

With that core dump, type bt when at the (gdb) prompt.
Also could you try running gdb R, then at the (gdb) prompt type r, and reproduce the issue? When it crashes, the type layout asm.

@ktprograms, thank you very much for your continued suggestions. I did what you said and got a little more information:
X
c( "Mike"
y
= c("red"
chisq.test(x,y)
"Sue"
Program received signal SIGILL,
Illegal instruction.
0x00007ffff5320820 in dgemm otcopy OPTERON SSE3 ()
from /usr/lib/x8664-linux-gnu/libopenblas.so.0
(adb) bt
#0
0x00007ffff5320820 in dgemm_otcopy_OPTERONSSE3()
from /us/lib/x86 64-linux-gnu/libopenblas.so.0
＃1
0x00005555576def08 in ?? ()
#2
0x0000000000000004 in ?? ()
#3
0x0000000000000001 in ?? ()
#4
0x00007ffff6a37530 in
?? () from /us/lib/x86 64-Linux-gnu/libopenblas.so.0
#5
0x00000000000000in?? (
Layout asm did work but then showed me what I believe was at least 100 lines of assembly code and didn’t seem to flag any specific part as abnormal. Unfortunately, such code is above my head in understanding to be able to debug. Is there something specific I should look for in that asm code as a clue?
Since I saw the mention about OpenBlas in the stack traceback section pasted above, I thought there might be something wrong with my Blas/OpenBlas code (since the chisq.test function does use matrix math that would depend on Blas/OpenBlas), and I downloaded an R package to link the latest OpenBlas to my R code. Unfortunately that ended up breaking something so that now the sub-package that contains the chisq.test function won’t load in R at all. I’ll probably end up re-installing R again to get back to a starting point to troubleshoot this.
Please let me know if you have further ideas.
Have good upcoming week.

Could you please paste the layout asm output in a code block?
It looks like sse3 isn't emulated by qemu

Thanks, @ktprograms, I think you figured out the problem and read that R requires SSE3 for many of its calculations and because of that is not yet supported for ARM (although there is an ARM R build in development given the support of the M1 Mac). That might also be why R causes the same error in iSH since although the iSH emulates an x86, the developer might not have built in SSE3 support. The following article from the RStudio R IDE developers discusses this:
https://developer.r-project.org/Blog/public/2020/11/02/will-r-work-on-apple-silicon/
However, the article goes on to mention that with Rosetta, R does run okay on the M1 MAC without SSE3, and I found some further info that seemed to indicate QEMU can emulate the SSE3 instruction set with perhaps some settings turned on:
https://www.linux-kvm.org/page/Tuning_KVM
It may take me a day to reinstall R in UTM (after I broke it from my experiment in trying to bind a new OpenBlas library to it), but after that I can paste the asm code. However, I don,t know how to do that in this forum (straight cut and paste would seem overwhelming with number of lines). Is there a special way to upload the asm code to this message on the forum?
I think this will be helpful to other potential UTM users that are thinking about using it for R since I believe the UTM wiki last had R as working, but not being able to run a basic chi-square test in R being built for statistics is a big deficit in wanting to use it in UTM.
Thanks again for your help.

You can upload the output of layout asm to a GitHub Gist.

Hi, @ktprograms, guess what!
I got my reinstalled and got my broken R working again by removing the new OpenBlas libraries that I had linked to my R using:
update-alternatives --config libblas.so.3-x86_64-linux-gnu and update-alternatives --config liblapack.so.3-x86_64-linux-gnu
I thought that would just get my R working back to their baseline state by linking back the blas libraries that were previously linked to R. I then started gdb aiming to reproduce the error and sent you the asm code via Gist, but then instead, it just worked!
x = c("Jon", "Larry", "Sue")

y = c("red", "blue", "red")
chisq.test(x,y)

Pearson's Chi-squared test

data:  x and y
X-squared = 3, df = 2, p-value = 0.2231
Warning message:
In chisq.test(x, y) : Chi-squared approximation may be incorrect
(That warning message is an appropriate warning message in R).
So, it seems that the amazing UTM app may be able to run full R! I'm not sure though how if its not able to emulate ssse3 instruction set, but it seems to work. It may be though that actually the chisq.test function doesn't need to use the ssse3 instruction set-- I'll watch for any other errors that might pop up because of it but so far, this was the only one I found. I not sure why it now works and hasn't in the past--the only thing differently I did was to update the Blas and Lapack libraries bound to R with standardly installed Blas and Lapack libraries-- perhaps they are not standardly attached to R in a standard R installation? This has all gotten above my head as a novice Linux user, but I'm very happy that it works and very thankful that you introduced me to gdb and helped me troubleshoot the problem.
I'll let others in the R community know--based on the previous R/Mac article I previously linked to, the M1 Mac community has been worried about not having an official R or RStudio ARM distribution and UTM Mac program might fit their needs.
Unless you have other questions or things you would like me to test on this matter, please feel free to close this issues. :-)

You could try adding the SSE3 CPU flag in the VM config => Advanced => CPU to ask QEMU to emulate it. I'm not sure if it is supported in the QEMU that is included with UTM.

Hi, @conath.
I tried on my iPad going to VM config => Qemu => Advanced => Bypass configuration and manually set arguments and added
-cpu +ssse3
However, that didn’t seem to work as VM gave me a “could not open ‘ssse3’ no file or directory” error.
On the positive side I think this is now a non-issue in that I got the function to work I think by linking the OpenBLAS and Lapack packages in the Debian 10 repository with R via the “update-alternatives’ command over R’s preconfigured BLAS package. I think that might mean that UTM’s Qemu does have ssse3 emulation (since everything seems to work now with R statistics).
If I find any other instances where R doesn’t work in UTM, I’ll troubleshoot it and start a new issues; otherwise, I’ll keep my fingers crossed, enjoy using full R, and close this issue.
Have a good night.

Sorry, made a typo. The CPU flags option is in VM Config => System => Advanced.

That’s, interesting, @conath — I see that now. I’ve been using the “default” CPU which doesn’t give me the option to set those ssse3 flags like I can if I choose the Penryn CPU. Perhaps if R is now working the default CPU has the ssse3 flag set? Do you know what CPU settings are actually used if one chooses the “default” CPU? Is there any advantage to choosing the Penryn CPU for regular UTM x86_64 work? Non-urgent, but I’m curious and suspect other UTM users might be interested (perhaps for another forum or the UTM wiki). Thanks!

Default CPU will select based on the system. I believe it will be qemu64 for x86_64 which has the following flags as standard:
fpu de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 syscall nx lm up pni hypervisor
Note to take this list with some skepticism because it is from Tuning KVM. UTM doesn't use KVM so not sure it all of these flags apply. (on iOS, UTM uses the TCG emulator)
After doing some more digging online, I found this semi-recent change to QEMU which indicates qemu64 roughly has the capabilities of an "AMD AthlonTM 64 X2" processor, which did not (originally) support SSE3. (side note: the CPU speed is not controlled by the model chosen - UTM will always run at maximum possible emulation speed)
The conclusion, then, is to change your VM's CPU to qemu64 and add all of the flags you would like/need and see if it works. If you enable Debug Log you will see in the log if the emulator doesn't support one or more of the flags you requested.

Thank you very much for that additional information and explanation.


4 new Community meetups added.

Using their website as a source for the description text.
What issue is this referencing?
This issue fixes #164 #162 #161 #160
Have you taken a look at our contributing guidelines?

[x] yes!

@allcontributors add @VitalJeevanjot for code


In administration, add a page to list and access open campaigns

In order to easily give support to people, give access to all open campaigns from a new page in admin panel.
The same page as "List campaigns" on the homepage, but only for open campaigns, and without checking at user's allowed structures (because he is admin)

PR Related #164


Download location based on page titles

Complete the download options to indicate media should be saved in sub-directories whose name is the page's title.

@sjgard2001 I have just released a new version with this feature.
Feel free to test it and tell me whether it matches your expectations.

The new sub-directory by page  functionality looks good to me. Thank you for including it.


Changes for launching all yahoo finance applications on sandbox.

Changes to resolve MLHR-1655

too old.


Drawable not found with last Support Library (23.2.0)

Hey !
Today I update my Gradle scripts and SDK, and I got the followingError :
(3, 5) No resource found that matches the given name (at 'android:actionModeCloseDrawable' with value '@drawable/abc_ic_ab_back_mtrl_am_alpha').
I'm using core version, <IP_ADDRESS>@aar. Someone knows where does it come from ?
Thanks in advance !

See the many closed issues.


Errors if software is not installed.

This works great. I love it.
But there is an issue when running the script and you don't have Razer's RGB software. It tried to make an HTTP request but since the software is not installed it times out and the program doesn't run.
I had to try and except all requests functions inside the Razer script, so I don't run into that issue, since I only have a Corsair mice.
Could you maybe make it so if it doesn't detect the Razer's software it will skip it and not use the corresponding method?

I am so so sorry that I did not think anyone would be using this script and manage to open an issue. I will definitely modify that one as soon as possible. Again Sorry for so late reply :(

I am so so sorry that I did not think anyone would be using this script and manage to open an issue. I will definitely modify that one as soon as possible. Again Sorry for so late reply :(

You're doing good work out there. Looking forward to using your code once I get my Corsair build going!!

Thanks for the support everyone! I have some good news here. I have developed AllPheripherals project which is better in terms of performance and efficiency. Also, the Error due to Razer Chroma try and except would be gone in that project since I am using a DLL with C++ based program. Please check the link https://github.com/gooday2die/AllPeripherals here.

Hello, my huge apologies for not updating this project for a long time. I live in South Korea, where I must spend time in the army for a year and half. Also I was not able to update this project since I had no access to my PC. However, I got a bit of access to my PC recent days and now am updating this project.
In short, I have updated the project for ease of use. (Also back in 2020 and 2019, I was a noob at programming than now) I refactored most of the program and worked on OOP style codes so that more people can Pull Request, or access the code more easily.
From now on, you can install this project by pip install PyPeripheral. Also for Razer, I have made some exception control over the Wrapper, so that you would not have to worry about the program any more. (I might have missed some since I do not have Razer devices, if there are bugs and isses, please let me know)
I am sorry for making everybody wait for a whole 3 years, and I will be providing support for this project as much as possible.
Thank you


Support for checking single file instead of whole directory

Based on input parameters
(https://code.google.com/p/forbidden-apis/wiki/CliUsage) I see that I can
provide directory with source code and all files fill be examined. However I
think it would be nice to add support for forbidden-apis to
https://github.com/TouK/sputnik and for this I need to be able to scan single
or more selected files instead of whole directory.

Thanks,
Damian

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 30 Jan 2015 at 10:00

In general, it is also possible with the CLI tool to check single files. I also added a test about this. You can pass a directory name with --dir, but as second parameter a ANT-like pattern for --includes <pattern>. If the pattern is a single file name, you have all you need.
I will close this issue now.


implement https support with the option to force http

In order to solve the problem that the mobile apps and modern Windows client can't connect to containers behind Traefik as they get mixed http / https traffic because the WebClient seems to serve some resources with hardcoded http, this change let's the containers run with SSL by default. It consists of three parts:

A new traefik template file which allows connections to endpoints with self-signed SSL certificates as the containers will use those: insecureSkipVerify = true
Changes to New-NAVContainer which enable SSL by default when running with -useTraefik and do the necessary setup for the Traefik rules: protocol is https and port is 443. The download site for the AL vsix file is always http, so the protocol for that is http as well. The parameter -forceHttpWithTraefik reverts those changes and the cmdlet will behave as previously
Changes to Setup-TraefikContainerForNavContainers.ps1 which reference the new traefik template file. The parameter -forceHttpWithTraefik again reverts those changes and the cmdlet will behave as previously

I know it is documented somewhere, but I can't find it: How can I test the ARM templates with different navcontainerhelper sources?

Thanks, a few questions?
It looks like we are changing default behavior to use SSL between traefik and container - right? I think that is fine, but people with an existing Azure VM and updating containerhelper won't work anymore as I see it unless they specify -forceHttpWithTraefik in all their new-navcontainer calls.
(this will break all my CI/CD pipelines:-))
Maybe we could remove the forceHttpWithTraefik parameter in new-navcontainer and set it automatically (where we check that the traefik container is setup with usetraefik) based on how the traefik container is setup?
Also it seams like the traefik.toml config file will always be names _https unless you specify the file yourself. Shouldn't this be defaulted based on -forceHttpWithTraefik? (then it would be easier to default the forceHttpWithTraefik in New-NavContainer).

It looks like we are changing default behavior to use SSL between traefik and container - right? I think that is fine, but people with an existing Azure VM and updating containerhelper won't work anymore as I see it unless they specify -forceHttpWithTraefik in all their new-navcontainer calls.
(this will break all my CI/CD pipelines:-))
I have to admit, I didn't think about the existing VMs... :(

Maybe we could remove the forceHttpWithTraefik parameter in new-navcontainer and set it automatically (where we check that the traefik container is setup with usetraefik) based on how the traefik container is setup?
Very good idea, thanks! I removed the param and tried to check the config file here. Really not very elegant, but I couldn't come up with anything better

Also it seams like the traefik.toml config file will always be names _https unless you specify the file yourself. Shouldn't this be defaulted based on -forceHttpWithTraefik? (then it would be easier to default the forceHttpWithTraefik in New-NavContainer).
Absolutely true. I misunderstood the parameter $traefikToml, but reverted that now

Agree, I don't like the parameter on New-NavContainer either.
Since we already have hardcoded the path for traefikforbc - couldn't we just check and see whether this file exists:
(Join-Path $traefikForBcBasePath "config\template_traefik_https.toml")
then it is https - else it is http.
Wouldn't that work?

Once more, I handled the traefiktoml param wrong. The file below (Join-Path $traefikForBcBasePath "config") will always be called traefik.toml as this is the config file name traefik picks up automatically. We also need to consider the case where someone brings in their one template file by setting e.g. $traefikToml to https://mydomain.com/traefik_template.toml, so I think the only reliable way is to check the content of the config file

http://aka.ms/getbc2 will use the dev branch in the navcontainerhelper repo.
http://aka.ms/getbc?nchbranch=https%3A%2F%2Ftheurlofyourrepo - should read containerhelper from that repo (haven't tried recently)
if the config file always is the same - you could move the code a little further down:
        $forceHttpWithTraefik = $false
        if ((Get-Content "$traefikForBcBasePaht\config\traefik.toml" | Foreach-Object { $_ -match "^insecureSkipVerify = true$" } ) -notcontains $true) {
            $forceHttpWithTraefik = $true
        }

and then remove the parameter:-)

Once more, you are very right... Not my best PR, to put it politely :)

you still need to replace the reference to the non-existing parameter with "$traefikForBcBasePath\config\traefik.toml"
right?

absolutely...
A first quick test using the nchbranch param didn't work (seems to not have run any nch commands), but before I do more stupid things, I'll call it a day and try again tomorrow or on Monday ;)

Goodnight:-)
You are missing config\ in front of traefik.toml
and I checked - the nchbranch needs to be a direct download to a .zip file with the containerhelper - so:
http://aka.ms/getbc?nchbranch=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fnavcontainerhelper%2Farchive%2Fdev.zip
should work.

Amazing how many simple things I am able to get wrong :)
Ok, tried again and tested the following:

Pre-existing VM created from ARM template:

Downloaded my repo as .zip
Extracted it and ran Import-Module .\NavContainerHelper.psm1
Ran New-NavContainer -accept_eula -containerName testnch -imageName mcr.microsoft.com/businesscentral/onprem:ltsc2019 -useTraefik -auth NavUserPassword
As expected this creates a container with useSSL=N and traefik set up to use http as protocol and port 80:

            "Env": [
                ...
                "useSSL=N",
                ...
            ],
            ...
            "Labels": {
                ...
                "traefik.protocol": "http",
                ...
                "traefik.web.port": "80",
                ...
            }

Ran Setup-TraefikContainerForBCContainers -PublicDnsName td19-traef.westeurope.cloudapp.azure.com -ContactEMailForLetsEncrypt<EMAIL_ADDRESS>-Recreate
Now the traefik.toml file has insecureSkipVerify = true
Ran New-NavContainer -accept_eula -containerName testnch -imageName mcr.microsoft.com/businesscentral/onprem:ltsc2019 -useTraefik -auth NavUserPassword again
This creates a container with useSSL=Y and traefik set up to use http and port 443:

            "Env": [
                ...
                "useSSL=Y",
                ...
            ],
            ...
            "Labels": {
                ...
                "traefik.protocol": "https",
                ...
                "traefik.web.port": "443",
                ...
            }

Connecting through the mobile app works and the browser debug console shows no "Mixed mode" warnings

New VM created with http://aka.ms/getbc?nchbranch=https%3A%2F%2Fgithub.com%2Ftfenster%2Fnavcontainerhelper%2Farchive%2Fmaster.zip

Configuration shows the same as above under 8. and traefik.toml has insecureSkipVerify = true
The initialization fails with

9:26:36 AM at , C:\demo\setupVm.ps1: line 112
9:26:36 AM The specified module 'navcontainerhelper' was not loaded because no valid module file was found in any module directory

I then changed to C:\DEMO\navcontainerhelper-master and ran Import-Module .\NavContainerHelper.psm1
Creating a new container using New-NavContainer -accept_eula -containerName newnch -imageName mcr.microsoft.com/businesscentral/onprem:ltsc2019 -useTraefik -auth NavUserPassword works as expected with the same config as above under 8.
Connecting also works

What did I miss this time? :D

Nothing (I think) - will merge and run some test.
Thanks a million for your effort:-)
/Freddy


Customizing the CSS identifiers (scope, hash etc)

Feature request
Problem
Our Selenium tests rely on a specific naming of class names in the HTML. E.g. class="Header__link", where Header is the name of a component, and link is a class in the component's CSS module).
With the current experimental CSS support in Next.js, there's no apparent way to customize the outputted CSS identifiers.
With next-css, we used to simply override the Webpack config with our own css-loader options localIdentName and getLocalIdent).
Proposed solution
Please allow to override the Webpack config like it was the case with next-css.
Thanks!
Related to #8626.

How do you guarantee uniqueness throughout your whole application?

In our case the file name is always styles.module.css. It’s the parent directory that corresponds to the component name.
The component name (or rather its path) guarantees uniqueness. Another example of our former identifiers: signup_FacebookButton__icon. The CSS is in components/signup/FacebookButton/styles.module.css.

To set expectations, we're probably not going to ship with this ability by default.
We can reconsider this when it's released stable if there's significant demand. IMO, there's better workarounds here.
For example, if you are writing Unit Test using Jest, you should be eliminating CSS Module exports or using identity-obj-proxy which would skip hashing.
If you are writing Integration Tests, you should be using getComputedStyle() — not class names.

Thanks for the reply. I think your reasoning and strategy makes total sense. Also less configuration for the user.
I guess we'll have to use next-css until it stops working or and rewrite all the tests in the meanwhile. However, if there is demand after all, I suggest people interested in this feature to upvote the issue.
Cheers 🙂

Your use case would probably be served well by identity-obj-proxy — but yes, you can keep using @zeit/next-css; we're not going to maintain it but we're not going to intentionally break or block it either. 😄

tl;dr: To set expectations, we're probably not going to ship with this ability by default. Read on for testing alternatives.

@Timer when you say not ship with the ability to do this, are you referring to what has been requested in this issue, or the ability to turn on the experimental CSS features?
I'm using BEM and next-css and well... this is a big change, I'm using a well documented system for creating components (BEM), and I'm concerned that the new CSS features are going to mean that we'll end up having to re-invent the wheel by creating something that works with a CSS module based approach.


Linting

Set up eslint and prettier. Haven't resolved all warnings/errors found by npm run lint, but at least formatted everything with npm run format.
Have only set up formatting with prettier on all pull requests to master so far.

Looks like I have some more work to do in pr.yml 😅 @ChurchTao any ideas why it's failing?


add releases page - WIP

Signed-off-by: Mike Ng<EMAIL_ADDRESS>Hi @qiujian16 I am working towards adding a "Releases" page to the website similar to https://submariner.io/community/releases
But we really haven't discuss our upstream release process yet. So from now I put in the community operator hub version for registration (cluster manager, klusterlet) and app subscription operator.
WDYT? Should I continue with this and populate the releases with some notes?
FYI @juliana-hsu

yeah, i think we need to discuss about this. We also need a release cadence doc in community repo.


1.0 upgrades

Changes Unknown when pulling 97a1a4d6bcf11e15fa1c66348e8931531732f94c on cpacia:master into ** on OpenBazaar:master**.


Custom events added

As discussed in #136, there are cases when you want to log custom events (I want to log Delayed Job events). As far as I understand, it would be still possible to use just Rails.logger.info etc, but I think it's useful leverage the formatter settings. WDYT?
I've looked around and no class seemed to be the right place for this, so I've created a new module. Obviously I can move things around if you prefer something different.
I've changed one unrelated send call to public_send, since I consider send to be a smell.

I've tried this in our app and realize, that for example the LogStash output contains the not as useful message field. Not sure, if the PR is still good, I went ahead and just log directly to Rails.logger without this.

Nice thank you so much!
I do not really understand yet why those builds fail. I try to investigate this later tonight when I have more headspace.

Seems like a decency looks in logstash has breaking changes. This should make the tests work for now:
https://github.com/roidrage/lograge/pull/145
Could you add some documentation on how to use is in the readme?
What do you think @benlovell ?

I've tried this in our app and realize, that for example the LogStash output contains the not as useful >message field. Not sure, if the PR is still good, I went ahead and just log directly to Rails.logger without >this.

@iGEL could you expand on this?

@iGEL Any interest in this?

This would actually be a a very useful feature. I hope it gets integrated at some point!

@iGEL we would be happy to proceed with this. Could you maybe address the open points in the near future? :green_apple:

would love to get this merged with some documentation!

@highbass Is this something you're using?
@iGEL Seems this one died from your perspective, I take it you went a different route?

i am not using it... but i was googling up on putting custom logs using lograge and came across this. I am currently playing around trying to see if it actually works but regardless would need it to be part of distribution or branch for me to be able to use it. Would be a very useful feature to have for custom logging with lograge for logstash.

@highbass this shouldn't be to hard, you could use the default logstash formatter:  https://github.com/roidrage/lograge/blob/master/lib/lograge/formatters/logstash.rb
and make your own custom formatter:
http://an-ti.eu/blog/ruby/2015/01/18/define-a-custom-lograge-formatter/

The CI failure seems to be related to this: https://github.com/rspec/rspec-mocks/issues/964. It seems the Jruby team has fixed this one.

This feature is quite useful. I hope this PR will be merged soon.

@benlovell can't see a way to rerun travis on this build, so i've forked master and have manually readded the changes made by @iGEL originally. Is there still any interest in this? Rspec test suite is passing and cleaned very minor issues for Rubocop identified via rake ci. If there's still interest, I'll submit a new MR with the every so slightly updated implementation.

@binarydev Sure, it'll need a rebase and I'd like to see @pxlpnk's comment addressed and a little documentation love - but definitely keen to get this in.

@benlovell great! I've already put the latest changes on top of master locally. I'll just clarify the methods a bit and add some documentation. Quick question regarding the implementation: currently it creates a separate entry when using the JSON formatter rather then embedding it within a dedicated key for the request, which is great for DelayedJob, Sidekiq, etc. However, is this always ideal? If a custom event is fired off within a service class during a request, i would expect it to be considered part of the current request payload, perhaps under a new custom_events key as an array of events in chronological order.
I'm currently using lograge with the Elastic stack, and if I filter by requests in Kibana, all custom events would be excluded from the results under the current implementation based on my local testing. Should these entries instead be injected into the event payload for the request within which they are fired off?
A solid example is a service class I have with a few submodules that act as a wrapper around R for stat analysis, where R has to execute a SQL query that Rails generates based on user selected criteria. Since Rails isn't executing the SQL, it won't be logged by lograge-sql, however I still want to track what SQL was generated for a request, along with the runtime performance data that we have R return to us as part of the results. It would be very strange for me to have this service return that data in the controller to be held in instance vars to provide #append_info_to_payload access to them.

Hey @binarydev I recently ran into the same issue using lograge for sidekiq, it broke our logging as we had two separate JSON objects, instead of a unified payload.
I personally think it would make sense to unify this.
However I am not sure if I understand fully what you mean, would you mind posting some payload examples? I think this would help the discussion.
Looking forward to your PR! 🍏

@pxlpnk Happy to share an example. So in the JSON log currently, as you said, a (formatted) request appears as 2 separate JSON objects. In the example below, I made a request, which calls a service, and within that service's submodules, I have the lines Lograge::CustomEvent.log(my_special_event_message: "test1") and Lograge::CustomEvent.log(my_super_special_event_message: "test2") separately being called, generating these 3 separate objects in the log:
{
   "my_special_event_message":"test1",
   "@timestamp":"2017-05-04T15:44:46.905Z",
   "@version":"1",
   "message":"[]   (#)"
}
{
   "my_super_special_event_message":"test2",
   "@timestamp":"2017-05-04T15:44:47.255Z",
   "@version":"1",
   "message":"[]   (#)"
}
{
   "method":"GET",
   "path":"/",
   "format":"html",
   "controller":"HomeController",
   "action":"index",
   "status":200,
   "duration":177.21,
   "view":98.53,
   "db":54.1,
   "params":{},
   "user_id":1,
<EMAIL_ADDRESS>   "user_role":"admin",
   "request_id":"b8e3b2a1-2f15-40eb-ac82-c19b9e2c2407",
   "@timestamp":"2017-05-04T15:44:47.305Z",
   "@version":"1",
   "message":"[200] GET / (HomeController#index)"
}

What I am proposing is injecting the custom events into the main request payload under a dedicated key as an error of triggered events, so that the above becomes the following
{
   "method":"GET",
   "path":"/",
   "format":"html",
   "controller":"HomeController",
   "action":"index",
   "status":200,
   "duration":177.21,
   "view":98.53,
   "db":54.1,
   "params":{

   },
   "user_id":1,
<EMAIL_ADDRESS>   "user_role":"admin",
   "request_id":"b8e3b2a1-2f15-40eb-ac82-c19b9e2c2407",
   "@timestamp":"2017-05-04T15:44:47.305Z",
   "@version":"1",
   "message":"[200] GET / (HomeController#index)",
   "custom_events":[
      {
         "my_special_event_message":"test1",
         "@timestamp":"2017-05-04T15:44:46.905Z",
         "@version":"1",
         "message":"[]   (#)"
      },
      {
         "my_super_special_event_message":"test2",
         "@timestamp":"2017-05-04T15:44:47.255Z",
         "@version":"1",
         "message":"[]   (#)"
      }
   ]
}

Where the custom_events keys holds the events triggered during the request cycle in chronological order, returning a single JSON object that can be consumed by logstash.

I understand now. This is different from what I actually meant.
In my case sidekiq would issue a log line including two objects:
{something: 'value'}, {something_else: 'another value}
I guess this is related to our implementation and a different problem.
@binarydev How would this work in the environment of something like sidekiq, where you have no access to the original performed request/logline?
@benlovell WDYT about the idea? I can see how this is useful, I am not sure though if this is a job that lograge should do. Or if it would make more sense to use the request ID to stitch the different logs in the logging tool.

Can this be merged please? Any sort of logging for custom events is better than none.

Thanks for this, I included that module in an initializer, and now I'm able to easily log custom events


Tag component with href doesn't allow to specify anchor target

Tag component with href doesn't allow to specify the anchor's target.
This is required in case we want to open the tag's link in a new tab.

TO DO:

Add newWindow property to tag component.
When tag receives an action, it must render as a button, keeping the same visual style.


Deleting a child causes request flood and shows "unable to connect to server"

When a child is deleted from the database while the child's feed is shown in the app, the app will start flooding the server's api endpoints for new history items. In addition, the app will show a "unable to connect to server" message. This is a bug.
Workaround
Close and reopen the app.

Fixed in 1ada155d58d3d25da152ca0946bbbcc586c0bd7a
Solution: Added explicit child-existence check to ChildStateTracker, and fixed the updateChild function in BabyLayoutHolder.


ws.SendAsync()后报错，错误信息:Operation is not supported on this platform

SendAsync(),发送消息报错，Operation is not supported on this platform

I have the same issue using websocketsharp.core 1.0.0 on MacOS with .netcore 2.2


Adding documentation

Closes #143
Changes proposed in this pull request

Volume tracking documentation for tol-lab-share: https://sanger.github.io/tol-lab-share/
API docs for tol-lab-share: https://sanger.github.io/tol-lab-share/api-docs/

Instructions for Reviewers
[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to main]
    - Check story numbers included
    - Check for debug code
    - Check version

This is amazing!!!!!! Definitely 'Approve' 🥇
Just a small note, the link to https://sanger.github.io/tol-lab-share/SETUP_DOCKER.md under https://sanger.github.io/tol-lab-share/#setting-up-with-docker doesn't seem to work for me.
How did you generate all the docs? Did you use CoPilot or write it yourself?
Super nice, definitely worth rolling out. Thanks for doing this first time around (sorry just realised second, after Traction).

Good catch. I'll update it 😄 Thanks!

This is amazing!!!!!! Definitely 'Approve' 🥇
Just a small note, the link to https://sanger.github.io/tol-lab-share/SETUP_DOCKER.md under https://sanger.github.io/tol-lab-share/#setting-up-with-docker doesn't seem to work for me.
How did you generate all the docs? Did you use CoPilot or write it yourself?
Super nice, definitely worth rolling out. Thanks for doing this first time around (sorry just realised second, after Traction).

Oh as to how I generated the docs: well all the volume tracking stuff were written by me. I don't think CoPilot understands what it is 😄 You just write stuff in a markdown file, and compile it to a bunch of static content that GitHub Pages serve. I took some help from CoPilot to run the shell script that does compiling them in CI though.
The setup guides were taken from the README.md.


[bugfix] Ensure that module paths are restored in tests

Codecov Report

Merging #1655 (f3c17bc) into master (4bf3156) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1655   +/-   ##
=======================================
  Coverage   87.72%   87.72%
=======================================
  Files          45       45
  Lines        7477     7477
=======================================
  Hits         6559     6559
  Misses        918      918

Continue to review full report at Codecov.

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


Regular expression in computed custom field

Do regular expression validation apply in computed field?

Hi. For issues, custom values that are not editable are not validated.

thank you


Allocate length 0 from dog (subspecies) to wolf (species)

Fixes #56 . Maybe worth checking if anything else is affected like this.

Sorry, didn't notice you had PR our. Anyway, it's safer to not have the length, as this way we don't need to fix the ultrametricity tool.

Agreed. Thanks for thinking this through and fixing.


Find a way for all keybindings in zellij status-bar to use the same "leader" key

Then we would have the more condensed and readable form.
basically replace the alt 1 that goes to search mode, and replace it with a ctrl keybinding
from this

to this

fixed. they keybinding is now ctrl y


Fix bigint type declaration

Fix type declarations for bigint based functions

Duplicate of https://github.com/feross/buffer/pull/298


Updated Dependencies & made recursive in maven style for easier usage.

Implemented Maven 3.1.x
Updated third party dependencies to latest.  Did not update
htmlcompressor to 1.5.3 as it is currently not available in maven
central.  However, this version as well as prior work with 1.5.3 without
issue.
Updated property settings annotation in code based on maven 3.1.x
changes
Minor cleanup (removing unused imports deleting duplicate blank
lines)
Made compression recursive so additional properties are not required
to get this working.  This matches how yui compressor maven task works
as well as ant based versions of yui compressor and htmlcompressor.
This fixes one noted issue on github.  Items are picked up from
src/main/resource and placed in proper place under target/classes.
Updated readme to match recursive nature of project now.

Any status on this PR request?

Well it's been over a year and another week since mentioning this.  Well I can now just release this under my own sonatype account.  I'd like to see this move forwards so I can utilize easily the changes I've made.  I can keep this PR in place but if I go that direction it will change to match my coordinates.  Please advice if you have time.  Thanks.

Jeremy,
Thanks for your contribution.
I've accepted your push request however I have few notes.

Your changes are not backward compatible and breaks the concepts behind htmlcompressor which have an option --type <html|xml>
The option allows a user to limit processing on the given type files only.
I suggest to add another maven goal named 'all' which processes all of the files (autodetects from file extension): the same behaviour like htmlcompressor for the case when --type is omitted.
As you might noticed tests are not accomplished and there is a demo attached (https://github.com/alextunyk/htmlcompressor-maven-plugin/downloads).
Please download that demo project, run it against version 1.3 then run against 1.4-SNAPSHOT
The diff will show the regression you've introduced with your changes.

I'm going to fix it like I've described above and push it to maven central as 1.4
Please let me know if you have any comments.

Thanks for the detailed information.  I'll check all this out...
--- Original Message ---
From: "alextunyk"<EMAIL_ADDRESS>Sent: May 14, 2015 1:20 PM
To: "alextunyk/htmlcompressor-maven-plugin"<EMAIL_ADDRESS>Cc: "Jeremy Landis"<EMAIL_ADDRESS>Subject: Re: [htmlcompressor-maven-plugin] Updated Dependencies & made recursive in maven style for easier usage. (#4)
Jeremy,
Thanks for your contribution.
I've accepted your push request however I have few notes.

Your changes are not backward compatible and breaks the concepts behind htmlcompressor which have an option --type <html|xml>
The option allows a user to limit processing on the given type files only.
I suggest to add another maven goal named 'all' which processes all of the files (autodetects from file extension): the same behaviour like htmlcompressor for the case when --type is omitted.
As you might noticed tests are not accomplished and there is a demo attached (https://github.com/alextunyk/htmlcompressor-maven-plugin/downloads).
Please download that demo project, run it against version 1.3 then run against 1.4-SNAPSHOT
The diff will show the regression you've introduced with your changes.

I'm going to fix it like I've described above and push it to maven central as 1.4
Please let me know if you have any comments.

Reply to this email directly or view it on GitHub:
https://github.com/alextunyk/htmlcompressor-maven-plugin/pull/4#issuecomment-102107131

It will be a couple of days before I get a change to really look into what my changes caused.  I will get to this even if you do end up fixing before then I want to understand where I went wrong.  Thanks.

Not sure this actually broke any backwards compatibility.  Execution phase can determine what type to apply with so it doesn't need to simply ignore all.  We have successfully been running separate execution phases for a very long time and only what we require be picked up are.  I will take a look at this again though as I've ported htmlcompressor itself over to github and will ultimately repoint this plugin to my version as original is essentially all but officially flagged EOL.


Added 2 nodes of 1 user

Added:

@trudiebakanauskas210:matrix.org 2 nodes

@ironoa Element handle was verified


chore(deps): update dependency stylelint to v13.4.0

This PR contains the following updates:

Package
Type
Update
Change

stylelint (source)
devDependencies
minor
13.3.3 -> 13.4.0

Release Notes

stylelint/stylelint
v13.4.0
Compare Source

Added: ignore:["delay"] to time-min-milliseconds (#4743).
Added: ignoreFunctions: [] to value-keyword-case (#4733).
Fixed: improved performance when auto syntax is used (#4729).
Fixed: --report-needless-disables respects stylelint-disable commands (#4714).
Fixed: at-rule-property-requirelist TypeError for comments inside of font-face (#4744).
Fixed: declaration-block-trailing-semicolon false positives for CSS-in-JS object notation (#4749).
Fixed: declaration-empty-line-before false positives for inlines styles (#4726).
Fixed: media-feature-name-* false positives for forced-colors (#4775).
Fixed: value-keyword-case false positives WebExtension replacement keywords (#4778).
Fixed: value-keyword-case false positives regression for mixed-case properties and the ignoreProperties option (#4748).

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 5.0.44 :tada:
The release is available on:

v5.0.44
GitHub release

Your semantic-release bot :package::rocket:


Feat: Remove Extra Padding from Main Content on Landing Page in Mobile View

Issue Description:
On the landing page of our website, we've observed that there is excess padding present on both sides of the main content specifically when viewed on mobile devices. This additional padding is affecting the user experience by reducing the usable space and causing unnecessary scrolling. To enhance the mobile user experience, we need to address this padding issue.
Expected Behavior:
The main content on the landing page should be properly aligned without any extra padding on both sides, ensuring that users can access the content comfortably on mobile devices without having to scroll excessively.
Steps to Reproduce:

Open the landing page of our website on a mobile device.
Observe the amount of padding present on both sides of the main content.
Note how the excessive padding impacts the usability and requires unnecessary scrolling.

Proposed Solution:
The padding values for the main content should be adjusted specifically for mobile views to ensure optimal use of screen space. This can be achieved through CSS media queries or responsive design techniques.
Screenshots:

Hey, is this issue still open? I would like to give some suggestions. I have changed how the mobile screen will look.
Figma mobile design

Hey, is this issue still open? I would like to give some suggestions. I have changed how the mobile screen will look. Figma mobile design

opps
looks like someone is already working on it


disable cython to prevent strange import error

https://github.com/dmlc/minpy/issues/131#issuecomment-274229165
https://github.com/dmlc/mxnet/issues/4746
strange error. cannot reproduce. but seems to be common for a lot of users. but looks like this will fix it.

I have edited the setup.py file,but still get ERROR.
>>> import mxnet
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "mxnet\__init__.py", line 7, in <module>
    from .base import MXNetError
  File "mxnet\base.py", line 43, in <module>
    _LIB = _load_lib()
  File "mxnet\base.py", line 35, in _load_lib
    lib = ctypes.CDLL(lib_path[0], ctypes.RTLD_GLOBAL)
  File "C:\Anaconda2\lib\ctypes\__init__.py", line 362, in __init__
    self._handle = _dlopen(self._name, mode)
WindowsError: [Error 126]
>>> import mxnet
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "mxnet\__init__.py", line 7, in <module>
    from .base import MXNetError
  File "mxnet\base.py", line 10, in <module>
    from . import libinfo
ImportError: cannot import name libinfo


feat: additive monoid structure via biproducts

I suspect I'm missing a trick with these transport lemmas.

I think either of "binary coproducts + initial object" and "binary products + terminal object" suffices to obtain a CommMonoid structure on the skeleton. The conditions used in this PR ("binary biproucts + zero object") are a combination of both, so should be derivable from either.


Faster transfer to root

Fixes #
The only change is reimplementations of
AnimationTarget.constructTargetNodes = function (root, vec3Scale, output)
AnimationClip.prototype.transferToRoot = function (root)
in playcanvas-anim.js
Other code changes are due to merging and rebasing, please feel free to ignore.
Tested with some glb animations:
Before improvement with old implementation
1 viewer.js:170 transferToRoot test: 0.3330078125ms
2 viewer.js:170 transferToRoot test: 0.31298828125ms
3 viewer.js:170 transferToRoot test: 0.47119140625ms
4 viewer.js:170 transferToRoot test: 0.324951171875ms
5 viewer.js:170 transferToRoot test: 0.43994140625ms
6 viewer.js:170 transferToRoot test: 0.40185546875ms
7 viewer.js:170 transferToRoot test: 0.576904296875ms
8 viewer.js:170 transferToRoot test: 0.346923828125ms
After improvement with new implementation
1 viewer.js:170 transferToRoot test: 0.26708984375ms
2 viewer.js:170 transferToRoot test: 0.244140625ms
3 viewer.js:170 transferToRoot test: 0.227783203125ms
4 viewer.js:170 transferToRoot test: 0.232177734375ms
5 viewer.js:170 transferToRoot test: 0.22607421875ms
6 viewer.js:170 transferToRoot test: 0.30615234375ms
7 viewer.js:170 transferToRoot test: 0.2138671875ms
8 viewer.js:170 transferToRoot test: 0.2099609375ms
I confirm I have signed the Contributor License Agreement.

Thanks! 👍


Support force_tuple argument

This patch is not enough. We need to add force_tuple argument to spit_axis method.

force_tuple argument in __call__ method is not enough because users doesn't use Function.__call__ directly. Instead I fixed split_axis method in #970. So, I'll close this PR.


Add Cache#getLevel

Using the above method we can check if the key is already in the cache and at which level it is cached, which may come handy when implementing some additional logic (usually in the wrapper).

Thanks for contributing @esad! What would be a high-level use case for getLevel?

Or if anyone else needs this, please chime in and say what for.

Closing due to lack of activity.


Consistent notation for VAddr/PAddr

Currently, we tend to format both virtual (VAddr) and physical (PAddr) addresses using the "{:#p}" format specifier for pointers. This works fine, but makes telling whether an address is physical or virtual difficult when reading through logs, et cetera.
It'd be nice if we could have a consistent notation for these. I suggest possibly replacing "0xff00" with "Pxff00 for physical addresses and Vxff00 for virtual addresses, or appending a P/V to the end of the hex address, like 0xff00_p.

Someone who knows how string formatting works in Rust could probably implement this fairly easily without any OS dev knowledge.
Therefore, I'm tagging it as "easy" and "help wanted", in case anybody's looking for a good issue for a first-time contributor...


Refactorings for encoding remote deallocation next pointers

The remote deallocation next pointers are not protected in the same way that free list pointers are. This PR provides refactorings to make it easier to add those changes.

@nwf I added a couple more commits on to this, that I have been working on as part of the larger refactoring.


Release Candidate

Changelog:

Break: all category factories now have new API, based on abstract factory
pattern.
Break: registry class now is an interface. This change is strictly required
to be able to add new categories without API breaking in the future.
Break: rename color class to termcolor_t.
Break: string view template class is now located in stdext namespace.
Break: JSON configuration factory has been completely hidden to prevent
sudden ABI breakage.
Feature: introducing new category - filters. Also as a proof of concept
a severity filter has been added, which can be used to route logging events
by their severity number.
Feature: file appender now has a nice builder class. Strictly speaking, all
category classes now have a nice builders, including experimental chained
builder.
Feature: flush threshold in file appender configuration now can be specified
as a binary unit value, i.e. 1000B, 2MB etc.
Feature: attribute value can now be constructed from nullptr.

Closes #107, #123, #124.


[Patcher] Update

Updated dependencies. successful_updates:
- file_path: /Users/marina/go/src/github.com/gruntwork-io/patcher-action/infrastructure-live/dev/eu-central-1/networking/vpc/terragrunt.hcl
updated_modules:
- repo: terraform-aws-service-catalog
module: networking/vpc
previous_version: v0.95.0
updated_version: v0.95.1
patches_applied:
count: 0

Testing locally using act. It also got my local changes from #3.


Added index() method for OccupiedEntry

This fix #31. Easy :-)

Thanks! I think it would be good to follow up with index methods on VacantEntry and then Entry itself too, I think. This rounds off and completes our API to make it more consistent.
One can spy the right value to use for VacantEntry from the line let index = self.map.entries.len(); that might be replaced by using the new method in fact.


Add error message for file permissions

part of: https://github.com/github/pages-engineering/issues/2428
Add an error message for when incorrect file permissions could be the cause for deployment failures.

I am hitting this error in my recent rustdoc deployments: https://github.com/DCNick3/shin/actions/runs/5300334374/jobs/9593874612
Is there anywhere I can find details about the "incorrect file permissions"?

I am encountering this problem as well.

Here's the log...
2023-07-19T05:37:47.0518637Z Requested labels: ubuntu-latest
2023-07-19T05:37:47.0518919Z Job defined at: rami3l/pacaptr/.github/workflows/docs.yml@refs/heads/master
2023-07-19T05:37:47.0519015Z Waiting for a runner to pick up this job...
2023-07-19T05:37:47.2445828Z Job is waiting for a hosted runner to come online.
2023-07-19T05:37:50.4001879Z Job is about to start running on the hosted runner: GitHub Actions 2 (hosted)
2023-07-19T05:37:52.9022422Z Current runner version: '2.306.0'
2023-07-19T05:37:52.9051210Z ##[group]Operating System
2023-07-19T05:37:52.9051857Z Ubuntu
2023-07-19T05:37:52.9052182Z 22.04.2
2023-07-19T05:37:52.9052467Z LTS
2023-07-19T05:37:52.9052891Z ##[endgroup]
2023-07-19T05:37:52.9053247Z ##[group]Runner Image
2023-07-19T05:37:52.9053636Z Image: ubuntu-22.04
2023-07-19T05:37:52.9053950Z Version: 20230710.1.0
2023-07-19T05:37:52.9054535Z Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20230710.1/images/linux/Ubuntu2204-Readme.md
2023-07-19T05:37:52.9055362Z Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230710.1
2023-07-19T05:37:52.9055819Z ##[endgroup]
2023-07-19T05:37:52.9056201Z ##[group]Runner Image Provisioner
2023-07-19T05:37:52.9056595Z <IP_ADDRESS>
2023-07-19T05:37:52.9056883Z ##[endgroup]
2023-07-19T05:37:52.9057669Z ##[group]GITHUB_TOKEN Permissions
2023-07-19T05:37:52.9058298Z Contents: read
2023-07-19T05:37:52.9058666Z Metadata: read
2023-07-19T05:37:52.9059250Z Pages: write
2023-07-19T05:37:52.9059680Z ##[endgroup]
2023-07-19T05:37:52.9064116Z Secret source: Actions
2023-07-19T05:37:52.9064722Z Prepare workflow directory
2023-07-19T05:37:52.9955145Z Prepare all required actions
2023-07-19T05:37:53.0149496Z Getting action download info
2023-07-19T05:37:53.2488149Z Download action repository 'actions/deploy-pages@v2' (SHA:12ab2b16cf43a7a061fe99da74b6f8f11fb77f5b)
2023-07-19T05:37:53.9807938Z Complete job name: deploy
2023-07-19T05:37:54.0741312Z ##[group]Run actions/deploy-pages@v2
2023-07-19T05:37:54.0741739Z with:
2023-07-19T05:37:54.0742301Z   token: ***
2023-07-19T05:37:54.0742588Z   timeout: 600000
2023-07-19T05:37:54.0742859Z   error_count: 10
2023-07-19T05:37:54.0743139Z   reporting_interval: 5000
2023-07-19T05:37:54.0743467Z   artifact_name: github-pages
2023-07-19T05:37:54.0743777Z   preview: false
2023-07-19T05:37:54.0744033Z env:
2023-07-19T05:37:54.0744289Z   CARGO_TERM_COLOR: always
2023-07-19T05:37:54.0744638Z   CARGO_REGISTRIES_CRATES_IO_PROTOCOL: sparse
2023-07-19T05:37:54.0745048Z ##[endgroup]
2023-07-19T05:37:54.4145201Z Artifact exchange URL: https://pipelines.actions.githubusercontent.com/UDgwt3Wna6q47f6ozDPk212ZhjhHJ01kbwJZwZhsF79t9FOAtF/_apis/pipelines/workflows/5595433732/artifacts?api-version=6.0-preview
2023-07-19T05:37:54.4715638Z Creating Pages deployment with payload:
2023-07-19T05:37:54.4716078Z {
2023-07-19T05:37:54.4717364Z 	"artifact_url": "https://pipelines.actions.githubusercontent.com/UDgwt3Wna6q47f6ozDPk212ZhjhHJ01kbwJZwZhsF79t9FOAtF/_apis/pipelines/1/runs/3881/artifacts?artifactName=github-pages&%24expand=SignedContent",
2023-07-19T05:37:54.4718198Z 	"pages_build_version": "d35a2b64234c000f1fcb4fd4a45b0dc903b524d8",
2023-07-19T05:37:54.4816714Z 	"oidc_token": "***"
2023-07-19T05:37:54.4817049Z }
2023-07-19T05:37:54.8511074Z Created deployment for d35a2b64234c000f1fcb4fd4a45b0dc903b524d8, ID: d35a2b64234c000f1fcb4fd4a45b0dc903b524d8
2023-07-19T05:37:59.8777142Z Getting Pages deployment status...
2023-07-19T05:38:00.1569657Z ##[error]Deployment failed, Please ensure that the file permissions are correct.
2023-07-19T05:38:00.1907372Z Evaluate and set environment url
2023-07-19T05:38:00.1912322Z Evaluated environment url: https://rami3l.github.io/pacaptr/
2023-07-19T05:38:00.1912852Z Cleaning up orphan processes


Compatibility Issues with PyPSA-EUR and Snakemake Version Updates

Checklist

[√] I am using the current master branch or the latest release. Please indicate.
[√] I am running on an up-to-date pypsa-eur environment. Update via conda env update -f envs/environment.yaml.

Describe the Bug
Please provide a description of what the bug is and add a minimal example/command for reproducing the bug.
Hi Lukas,
I just tried to run the code, but I encountered several bugs, likely due to updates in PyPSA-EUR. The first bug occurred when I attempted to run the Snakemake workflow as suggested by the documentation using the command:
snakemake -call results/octopus_fes/networks/elec_s_ec_lv1.5___LW_2030.nc --configfile config/config.default.yaml
The error indicated that the config file config.default.yaml was not a valid YAML or JSON file. After inspecting the file, I found an issue on line 133 where a space was missing before demand: and on line 134 before use_espeni: true. After fixing this, the config file worked fine.
However, due to version compatibility issues, it seems the Snakefile was written for Snakemake 7.7, and many changes have been introduced in newer versions. For example, HTTPRemoteProvider can no longer be imported from snakemake.remote.HTTP, among other changes caused by Snakemake and PyPSA-EUR version updates.
I suggest specifying a particular version of PyPSA-EUR and Snakemake to avoid these compatibility issues. This might help as a solution.

I believe I have resolved the compatibility issue by directly using the environment provided in this repository, rather than the PyPSA-EUR environment. A specific version -- PyPSA-eur-0.10.0 should work though. However, I noticed that the version setting for snakemake-minimal is not ideal. It currently says >=7.7.0, which could allow an upgrade to 8.0.0. To prevent this, I will make a commit to specify version 7.7.0 explicitly.


Error 0x80190194 When Updating Applications

Brief description of your issue
I am attempting to run winget update --all to update my applications. I have reset my sources and restarted my machine to the same issue.
Steps to reproduce
winget update --all, winget update --all --accept-package-agreements
Expected behavior
Applications would reinstall and update successfully.
Actual behavior
An unexpected error occurred while executing the command: Download request status is not success. 0x80190194 : Not found (404).
Environment
Windows Package Manager v1.7.11261
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.4355
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.22.11261.0

Duplicate of https://github.com/microsoft/winget-pkgs/issues/152915


question:  how to rename struct in bond schema

We're contemplating renaming a custom struct in our *.bond schema.   We use Bond's CompactBinary protocol.  I see the Schema Evolution rules documented here, but it doesn't address renaming a custom struct:  https://microsoft.github.io/bond/manual/bond_cs.html#schema-evolution
We looked into using an alias at the top of the file, but we're encountering compile time errors when aliasing a custom struct.  Do aliases only support builtin Bond types such as int64 ?   If so, what strategy would you recommend for renaming a custom struct?
Thanks!

The name of the struct is not encoded when using Compact Binary, Simple Binary, Fast Binary, or Simple JSON. Changing the struct name will not invalidate any serialized data, but you'll need to fix up any references to it in code.
One thing to watch out for is any use of bond.SchemaDef: the name and qualified name of the struct in SchemaDef instances will change. Depending on how you are using SchemaDef instances, this may or may not be a problem. (If you are using SchemaDef to deserialize Simple Binary, this will still work, as, again, the name isn't encoded in Simple Binary.)
Also, if you are using the deprecated bond_meta::name and bond_meta::full_name fields, this will cause these fields' values to change. The code on the reading side will likely need to be edited to handle both the old name and the new name.
The struct name is encoded in Simple XML, but the reader doesn't actually check the name for structs, so existing data deserializes fine. Other non-Bond consumers of this XML will likely be broken.
Hope this helps.

To answer the question about aliases and type converters, you can only convert scalars, blobs, and strings.

I'm going to close this issue. Please re-open it if something hasn't been addressed.


V4.0 bootstrap pagination

This commit provides minor improvements to the helper class Paginator in bl-kernel. The goal of this commit is to make the bootstrap pagination generated by bootstrapHTML() actually usable - since not even the two themes provided by default are using it.
Fixes:

variable name typo fixed in Paginator::bootstrapHTML() show_previous
removed page parameter from URL generated by Paginator::numberUrl() when pageNumber = 1. This is to prevent generating duplicate pages with different URL and negative impact to SEO since pageNumber = 1 is the default option in url.class.php.
Some elements of the bootstrap pagination were missing necessary CSS classes.

Improvements

added handling of the button text overrides to Paginator::bootstrapHTML() - the parameters were not used.
added handling of the parameter allowing to hide page numbers in pagination to Paginator::bootstrapHTML().
added extra parameters to hide the "first" and "last" pagination buttons + options to override their text in Paginator::bootstrapHTML().

This commit will be incompatible with themes using the original (albeit broken) function bootstrap_html due to renaming.

Thanks


go/packages: can't resolve golang_org vendored packages

What version of Go are you using (go version)?

$ go version
go version go1.11 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/dave/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/dave"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/qx/ypfpz5jd4jqc9r2r1z_mzpwm0000gn/T/go-build540247940=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
The golang.org/x/tools/go/packages package is not able to resolve the golang_org vendored packages in src/vendor - e.g. this import. Is this expected? Is there a work-around?
package main

import (
	"go/build"
	"path/filepath"
	"testing"

	"golang.org/x/tools/go/packages"
)

func TestVendor(t *testing.T) {

	importPath := "golang_org/x/crypto/chacha20poly1305"
	fromDir := filepath.Join(build.Default.GOROOT, "src", "crypto/tls")

	cfg := &packages.Config{
		Dir:   fromDir,
		Mode:  packages.LoadTypes,
		Tests: false,
	}

	pkgs, err := packages.Load(cfg, "pattern="+importPath)
	if err != nil {
		t.Fatal(err)
	}

	if len(pkgs) != 1 {
		t.Fatalf("expected 1 package, found %d", len(pkgs))
	}

	p := pkgs[0]

	if len(p.Errors) > 0 {
		for _, err := range p.Errors {
			t.Error(err)
		}
	}

	if p.Name != "chacha20poly1305" {
		t.Fatalf("expected chacha20poly1305, found %s", p.Name)
	}
}

Output:
=== RUN   TestVendor
--- FAIL: TestVendor (0.03s)
    vendor_test.go:35: -: cannot find package "golang_org/x/crypto/chacha20poly1305" in any of:
        	/usr/local/go/src/golang_org/x/crypto/chacha20poly1305 (from $GOROOT)
        	/Users/dave/src/golang_org/x/crypto/chacha20poly1305 (from $GOPATH)
    vendor_test.go:40: expected chacha20poly1305, found
FAIL

Dupe of #28375, which was duped to #26924. I just stopped looking at anything in $GOROOT/.../vendor, but I haven't thought about how good a general solution that is.


'KotlinProjectExtension not present' when writing Gradle plugins using 'kotlin-dsl'

Build scan link
https://scans.gradle.com/s/zjpogemu7wc4a
Plugin version
1.32.0
Gradle version
8.8
JDK version
17.0.11
(Optional) Kotlin and Kotlin Gradle Plugin (KGP) version
Kotlin 1.9.22 bundled inside Gradle 8.8
Describe the bug
Applying com.autonomousapps.dependency-analysis crashes Gradle during configuration phase.
Caused by: java.lang.TypeNotPresentException: Type org.jetbrains.kotlin.gradle.dsl.KotlinProjectExtension not present
	at org.gradle.api.reflect.TypeOf.captureTypeArgument(TypeOf.java:299)
	at org.gradle.api.reflect.TypeOf.<init>(TypeOf.java:96)
	at com.autonomousapps.subplugin.ProjectPlugin$KotlinSources$special$$inlined$the$3.<init>(TypeOfExtensions.kt:28)
	at com.autonomousapps.subplugin.ProjectPlugin$KotlinSources.<init>(ProjectPlugin.kt:1064)
	at com.autonomousapps.subplugin.ProjectPlugin.configureKotlinJvmProject(ProjectPlugin.kt:472)
	at com.autonomousapps.subplugin.ProjectPlugin.access$configureKotlinJvmProject(ProjectPlugin.kt:59)

To Reproduce
Project is very minimal, has root build.gradle.kts and one subproject.
Root:
plugins {
    id("com.autonomousapps.dependency-analysis") version "1.32.0"
}

my-plugin/build.gradle.kts
plugins {
    `kotlin-dsl`
}

gradlePlugin {
    plugins {
        create("myPlugin") {
            id = "pl.pkubowicz.myplugin"
            implementationClass = "pl.pkubowicz.MyPlugin"
            displayName = "My Plugin"
            version = project.version as String
        }
    }
}

repositories {
    mavenCentral()
}

The bug does not appear when the same configuration is used for a single-project build.
Expected behavior
I can execute gradle build as well as gradle buildHealth.
Additional context

This is basically the same problem as described at the link below, with the same basic solution: the kotlin-dsl plugin needs to be loaded in the same or a parent classloader to DAGP.
https://github.com/autonomousapps/dependency-analysis-gradle-plugin/wiki/FAQ#typenotpresentexception-type-orgjetbrainskotlingradledslkotlinprojectextension-in-kotlin-jvm-library.

Can the plugin be better at communicating the failure?
Good error messages give actionable advice. 'TypeNotPresentException' does not show user what action to take; it rather looks like a plugin author's mistake. It's a bit like showing NPE to a user – user won't expect that they should change the configuration.
For example, when Gradle crashes due to problems with Configuration Cache, they show you a link to Gradle User Guide covering how to fix common problems with Configuration Cache.
If the problem here is so common that it appeared in FAQ, maybe it should be handled in code? Even by catching the exception and displaying the link to FAQ you showed me.

That's a fair point.

Thank you!


fails to install on FreeBSD 9.3

linux version FreeBSD 9.3-RELEASE-p28
npm version 2.14.7
gcc version 4.2.1 20070831 patched [FreeBSD]
when using npm install node-gyp -g , I get:
npm-debug.log.txt
Any help would be appreciated.

@johenkel Please try using latest gcc.


deploying management cluster error

https://googlecloudplatform.github.io/kubeflow-gke-docs/docs/deploy/management-setup/
The same problem continues to occur in the process of proceeding with the tutorial.
The management cluster name "kubeflow-m" is valid.
gcloud services enable krmapihosting.googleapis.com \
	container.googleapis.com \
	cloudresourcemanager.googleapis.com
Operation "operations/acat.p2-174928385958-836eadbf-b770-4196-a5e4-fc4b82d2a278" finished successfully.
gcloud anthos config controller create kubeflow-m --location=asia-northeast1
Create request issued for: [kubeflow-m]
Waiting for operation [projects/kube-394803/locations/asia-northeast1/operations/operation-1691116470094-6020fbc794c2f-4fdc56cd-7a4d1b65] to complete...failed.
ERROR: (gcloud.anthos.config.controller.create) Operation https://krmapihosting.googleapis.com/v1/projects/kube-394803/locations/asia-northeast1/operations/operation-1691116470094-6020fbc794c2f-4fdc56cd-7a4d1b65 has not finished in 1800 seconds. The operations may still be underway remotely and may still succeed; use gcloud list and describe commands or https://console.developers.google.com/ to check resource state.
make: *** [Makefile:146: create-cluster] 오류 1

Looks like the command to create the management cluster timed out. Could you run the command gcloud containers cluster list to see whether the cluster you created exists? It feels more like a network/authentication issue as I could not replicate the error.


[Bug] Leaky boundary between GroupInfo and VerifiableGroupInfo

GroupInfo can be deserialized, thus skipping the verification step
GroupInfo unnecessarily implements Verifiable
GroupInfo content can be accessed prior to verification

To a certain degree, this is necessary so that the verifier can extract the signature key in new_from_welcome. However, the scope can be narrowed significantly here.

@kkohbrok Do you want to work on this? If not, I would like to tackle this :-)

Please do, but please wait until #1172 is merged. Everything else will just lead to more churn.

Alright :+1:

Removing the bug label since this doesn't lead to documented malfunction.


rm deprecated discovery-server.enabled

discovery-server.enabled is deprecated: https://github.com/trinodb/trino/commit/7e44c84098d391f3cffd2166c708556251a09695

oups, #48 already fixes that.


Fix doc comment

The of method for the Cfg struct mentions the command used to obtain  the configuration and parse. However, the command mentioned is incorrect, so recreating/testing the command at a terminal prompt results in an error. The correct command is added along with some clarifying information about the value that should be passed as a string.

bors r+