empty message

every time i go to use the bot this happenes

Does the bot have the Embed Links permission for the channel/role you want it to use?

The embed links option needs to be turned on in your own user settings in the category "text"


Daterange working as a modal

I need to make it so that when I call the action to open the calendar, it can be like a modal, open to the center. How could it be possible?

I think this go outside the scope of this component, and this can be easily achieved by render the calendar inline inside a modal (a div with the proper z-index value) and capture the changes yourself by using the events.
Because a modal depends highly on your design.
Besides, the team is trying to allocate time to move to Vue 3.
Do you agree?


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 179 ms

Resolved: ClamAV performance has improved in ea16397.


SAK-45758 Column was renamed and there was an unchaged reference

https://jira.sakaiproject.org/browse/SAK-45758

@davidpbauer this look okay? see SAK-44153

LGTM


Fixed step panel disable issue when failing

After https://github.com/intuit/karate/pull/677, when steps fail in between running all steps it disables all following steps, this restricted user to continue debugging and forcing to reset, so changed to enable all steps after a run.
Before:

After:

Relevant Issues : (compulsory)
Relevant PRs : (optional)
Type of change :

[ ] New feature
[X] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

wow thanks for the detailed visual ! merged


test: Run E2E and CI on staging branches

Fixes #N/A
Description
This change updates our workflow files to run E2E testing and CI on staging branches
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.028%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5395

Relevant Lines:
6577

💛  - Coveralls


Configure automated releases

The workflow is like this:
All development work should be done via PRs against the alpha branch.
When a PR is merged to alpha, the tests are run, then semantic-release runs to determine if a new release is required. If it is, then an alpha package is published to NPM.
Production releases are accomplished by opening a PR from alpha to main.
When a PR is merged into main, semantic-release runs and if a new package release is required, a package is published to NPM.

The challenge here is that branch protection rules that require a PR thwart semantic release, which needs to push tags, updated package.json and CHANGELOG.md directly to the branch.
A potential solution to this is detailed here: https://gonzalohirsch.com/blog/semantic-release-and-branch-protection-rules/

This solution requires installing a GitHub App in the repository, which is a permissions issue.
GitHub Copilot suggests an alternative: turn off branch protection, and:
Developers create feature branches off the alpha branch and open pull requests to merge their changes back into alpha.
When the changes in alpha are ready to be released, the CI runs semantic-release on the alpha branch. semantic-release determines the next version number based on the commits, generates the release notes, updates the version number in package.json, creates a git tag, pushes the changes back to alpha, and publishes the package to the npm registry.
When alpha is ready to be merged into main, a pull request is opened to merge alpha into main.
The CI runs semantic-release on the main branch. semantic-release determines the next version number based on the commits, generates the release notes, updates the version number in package.json, creates a git tag, pushes the changes back to main, and publishes the package to the npm registry.
The CI then merges the main branch back into alpha to ensure alpha has the latest version number and git tag.

This workflow is implemented now. This allows releases to take place, but does not enforce pull requests to the release branches (main and alpha). This means we need to have explicit knowledge and manual discipline to make sure that releases are not done inadvertently.
I've put in a request to have a GitHub App added to the repo to allow this one to be implemented later. That workflow will enforce PRs to be merged to create a release at the repo level.

Automated releases from main are working.


documentation https link doesn't work

Expected behavior

https://docs.swiftnio.io works

Actual behavior
just hangs
Steps to reproduce

open https://docs.swiftnio.io

further info:
$ curl -v https://docs.swiftnio.io
* Rebuilt URL to: https://docs.swiftnio.io/
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
[hangs]

ping @tomerd

fixed


Tabula 1.0 discombobulated by Furman Center PDFs

The PDF doesn't render properly:

And autodetect doesn't really work coz of the gridless design.
When manually selected, the parsing kinda works...
Tons of data-packed PDFs at http://furmancenter.org/research/sonychan/

Thanks for the report @jqnatividad.

The rendering bug is due to JPedal, the PDF rendering library we rely on. It'll be fixed when we get around to switching to PDF.js (#362)
Indeed, autodetect won't work if the tables aren't perfectly delimited by a grid, so this behaviour is expected.


Dynamically pick  K8S default version picking for AKS/GKE/EKS to be most recent

Right now, we just hard code the version we preselect for EKS, AKS, and GKE.
We recently updated AKS from 1.8.x to 1.11.x, but not the other providers.
Ideally, we would make this version picking dynamic, but for v2.1 we should at least update the default version for the other providers.

@westlywright as a small, quick fix, can you make the same update to GKE and EKS that you made to AKS? Change the default in the UI to the most recent version that the provider has

amazonElasticContainerServiceConfig does not have a k8s version parameter
googleKubernetesEngineConfig uses the data we fetch from /meta/gkeVersions

Version - v2.2.2-rc2
Verified fixed


migrate the com.kazurayam.inspectus.materialize.* packages out of this project into the inspectus project

derived from

https://github.com/kazurayam/inspectus4selenium/issues/24

I will migrate

com.kazurayam.inspectus.materialize.discovery
com.kazurayam.inspectus.materialize.selenium
com.kazurayam.inspectus.materialize.url

from the inspectus4selenium project into the inspectus project.

done at v0.7.0


debian build: add python3-setuptools to dependencies

Description
This PR fixes the marvin build error for ubuntu
it looks there is a regression in the marvin build , related to #6889
[INFO] --- exec-maven-plugin:1.2.1:exec (package) @ cloud-marvin ---
Traceback (most recent call last):
  File "setup.py", line 20, in <module>
    from setuptools import setup, find_packages
ImportError: No module named 'setuptools'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "setup.py", line 23, in <module>
    from distribute_setup import use_setuptools
ImportError: No module named 'distribute_setup'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "setup.py", line 27, in <module>
    raise RuntimeError("python setuptools is required to build Marvin")
RuntimeError: python setuptools is required to build Marvin

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with
SystemVM template(s). I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_multiplication_x: debian :heavy_check_mark: suse15. SL-JID 4700

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_multiplication_x: debian :heavy_check_mark: suse15. SL-JID 4701

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4702

@blueorangutan test

@weizhouapache a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan test ubuntu22 kvm-ubuntu22

Trillian Build Failed (tid-5310)

Trillian Build Failed (tid-5311)

@weizhouapache is this ready for review/merge?

@weizhouapache is this ready for review/merge?

Yes @DaanHoogland , moved to ready for review.

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_multiplication_x: suse15. SL-JID 4730

@blueorangutan test ubuntu22 kvm-ubuntu22

@DaanHoogland a Trillian-Jenkins test job (ubuntu22 mgmt + kvm-ubuntu22) has been kicked to run smoke tests

Trillian Build Failed (tid-5370)

Trillian test result (tid-5376)
Environment: kvm-ubuntu22 (x2), Advanced Networking with Mgmt server u22
Total time taken: 39554 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6930-t5376-kvm-ubuntu22.zip
Smoke tests completed. 100 look OK, 4 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_DeployVmAntiAffinityGroup
Error
35.65
test_affinity_groups.py

test_DeployVmAntiAffinityGroup_in_project
Error
88.37
test_affinity_groups_projects.py

test_03_deploy_and_scale_kubernetes_cluster
Failure
22.84
test_kubernetes_clusters.py

test_07_deploy_kubernetes_ha_cluster
Failure
0.06
test_kubernetes_clusters.py

test_08_upgrade_kubernetes_ha_cluster
Failure
0.06
test_kubernetes_clusters.py

test_09_delete_kubernetes_ha_cluster
Failure
0.06
test_kubernetes_clusters.py

test_hostha_enable_ha_when_host_in_maintenance
Error
304.24
test_hostha_kvm.py

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 4779

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4782

Trillian test result (tid-5405)
Environment: kvm-ubuntu22 (x2), Advanced Networking with Mgmt server u22
Total time taken: 43956 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6930-t5405-kvm-ubuntu22.zip
Smoke tests completed. 90 look OK, 6 have errors, 9 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_add_primary_storage_disabled_host
Error
0.69
test_primary_storage.py

test_01_primary_storage_nfs
Error
0.14
test_primary_storage.py

ContextSuite context=TestStorageTags>:setup
Error
0.26
test_primary_storage.py

ContextSuite context=TestPrivateGwACL>:setup
Error
0.00
test_privategw_acl.py

test_01_migrate_VM_and_root_volume
Error
83.33
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
49.70
test_vm_life_cycle.py

test_01_secure_vm_migration
Error
163.43
test_vm_life_cycle.py

test_02_unsecure_vm_migration
Error
273.95
test_vm_life_cycle.py

test_03_secured_to_nonsecured_vm_migration
Error
144.57
test_vm_life_cycle.py

test_08_migrate_vm
Error
43.95
test_vm_life_cycle.py

test_02_upgrade_kubernetes_cluster
Failure
518.66
test_kubernetes_clusters.py

test_03_deploy_and_scale_kubernetes_cluster
Failure
255.43
test_kubernetes_clusters.py

test_07_deploy_kubernetes_ha_cluster
Failure
88.52
test_kubernetes_clusters.py

test_08_upgrade_kubernetes_ha_cluster
Failure
47.40
test_kubernetes_clusters.py

test_09_delete_kubernetes_ha_cluster
Failure
44.31
test_kubernetes_clusters.py

ContextSuite context=TestKubernetesCluster>:teardown
Error
129.71
test_kubernetes_clusters.py

test_02_list_snapshots_with_removed_data_store
Error
9.61
test_snapshots.py

test_02_list_snapshots_with_removed_data_store
Error
9.61
test_snapshots.py

test_hostha_enable_ha_when_host_in_maintenance
Error
303.26
test_hostha_kvm.py

all_test_create_network
Skipped
---
test_create_network.py

all_test_network
Skipped
---
test_network.py

all_test_deploy_vgpu_enabled_vm
Skipped
---
test_deploy_vgpu_enabled_vm.py

all_test_network_acl
Skipped
---
test_network_acl.py

all_test_deploy_virtio_scsi_vm
Skipped
---
test_deploy_virtio_scsi_vm.py

all_test_deploy_vm_iso
Skipped
---
test_deploy_vm_iso.py

all_test_privategw_acl_ovs_gre
Skipped
---
test_privategw_acl_ovs_gre.py

all_test_network_permissions
Skipped
---
test_network_permissions.py

all_test_deploy_vm_root_resize
Skipped
---
test_deploy_vm_root_resize.py

The changes are only in build dependency, as packages are Debian build are passing that's enough to accept this PR. Merging based on packaging, and LGTMs.


Replace relations with DTOs.

Program Version.

Pre-alpha
Problem Description.
Currently there is no specific way to relate database objects whenever sending them back to the consumer. It would be best to create a folder that includes a bunch of DTOs.
Problem Solutions.
Create a folder that contains all DTOs. DTOs are objects that are not queried, but are derived from queried data. Usually to do with relations/associations.
Other Details.
The goal is to be able to serialize straight into these objects, rather than parsing them in memory.

See for instance here on line 33. This could be expressed much better with the commonplace term "DTO." With such, the first task would be making a folder in src with this, then creating a file containing a struct with a similar structure as PageModuleRelation


Still in development?

Is this still in development? I note that it's been a while since the last commit.

We have just released first beta version. 99.9% of compatibility tests are passing. Failing tests are using model imports that are currently developed and will be available in version 0.2.0. 90% of models from compatibility tests execute in less than 5 microseconds (Rust benchmarks), of course the latency depends on the machine. When used as a service you have to add time spent on TCP communication, but this may be optimized in many ways. So, as you have written, you need to execute models million times, so I guess with single DMNTK server it could take ~3..100 seconds. Of course running multiple servers will make it better. DMNTK can be optimized with binary size by compiling using Rust on the target operating system. DMNTK is coud-ready, may be initialized in any container using environment variables. Documentation is now rudimentary, but I am ready to help 24/7/365 if needed.


Not able to process file with [ in the file name, ie Remove-Item or Rename-Item

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
(a) Create a test folder, ie. c:\Test;
(b) Create files and one or two with a left square bracket in the name [, ie: C:\Test\Test [File
(c) Create a file list variable, ie $fileList = Get-ChildItem -path 'C:\Test' -File
(d) Run test code
ForEach ($file in $fileList) {   Remove-Item $file.fullname }
(e) Files without the left square bracket are removed. This is tested with version 5.5 and 7.3.3. Same problem with Rename-Item.
Tested with: (a) PowerShell (5.5 and 7.3.3) in Administrative mode, (b) Ran with -STA, (c) Tested in ISE mode. Further testing, PowerShell couldn't "find" the file, even though PS found the [files with Get-ChildItem to create the $fileList.
Expected behavior
The expected behavior is cmdlet Remove-Item or Rename-Item to behave as expected for all file in the folder. Remove-Item or Rename-Item to be able to find the file in the $fileList created by Get-ChildItem and to complete the command for "ALL" files in the $fileList.

All files in the folder had the "same" attribute - normal.

Actual behavior
Remove-Item/Rename-Item performed the expected action on files without the left square bracket in the filename, but could not find the file with the square bracket in the filename to perform the task.

Error details
Message in PowerShell 7.3.3 and PowerShell 5.1
Remove-Item: Cannot retrieve the dynamic parameters for the cmdlet. The specified wildcard character pattern is not valid: Instructions on Accessing CyberDefense Clinic[35.pdf

Important to Note: When the cmdlet is run as a standalone, Remove-Item $file.Fullname, where $file.fullname has the full path and filename, NO error was returned and the file was NOT removed (or rename as in the case of rename

Environment data
Name                           Value
----                           -----
PSVersion                      7.3.3
PSEdition                      Core
GitCommitId                    7.3.3
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
Four image files are uploaded - Left Window [PowerShell] - Right Window [File Explorer Folder]:
(a - Before) [ PS-Empty ] [ Explorer: Folder/Files]
(b - After) [PS ran script ] [ Explorer: Files with left square bracket were not removed (others were removed) ]
(c - Standalone) [ PS ran cmdlet ] [ Explorer: no affect on files ]

You need to use -LiteralPath‘ for pwsh not to treat the path as one with wildcards. You can also pipe the file object into the cmdlet and it will bind to PSPath which is an alias for -LiteralPath`.

Related to https://github.com/PowerShell/PowerShell/issues/12168

Thanks, figured it out… from the web articles.
From: Jordan Borean @.>
Sent: Tuesday, March 14, 2023 2:18 AM
To: PowerShell/PowerShell @.>
Cc: Gnart @.>; Author @.>
Subject: Re: [PowerShell/PowerShell] Not able to process file with [ in the file name, ie Remove-Item or Rename-Item (Issue #19323)
You need to use -LiteralPath‘ for pwsh not to treat the path as one with wildcards. You can also pipe the file object into the cmdlet and it will bind to PSPath which is an alias for -LiteralPath`.
—
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/issues/19323#issuecomment-1467430284, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMDZVA52SZDV34TYXWLXE23W4AEQ3ANCNFSM6AAAAAAVZ67HFE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Is there anyway to pull the issue from GitHub, as not to receive anymore reply?
From: Jordan Borean @.>
Sent: Tuesday, March 14, 2023 2:18 AM
To: PowerShell/PowerShell @.>
Cc: Gnart @.>; Author @.>
Subject: Re: [PowerShell/PowerShell] Not able to process file with [ in the file name, ie Remove-Item or Rename-Item (Issue #19323)
You need to use -LiteralPath‘ for pwsh not to treat the path as one with wildcards. You can also pipe the file object into the cmdlet and it will bind to PSPath which is an alias for -LiteralPath`.
—
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/issues/19323#issuecomment-1467430284, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMDZVA52SZDV34TYXWLXE23W4AEQ3ANCNFSM6AAAAAAVZ67HFE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Route value is being logged as HTTP verb only and not the path

Hi,
I know this has been mentioned before but I cannot find a way to fix this.  The route information in the metrics just contains to the HTTP verb and not path.  This means all of the metrics are being grouped together.  We are using .NET 2.1 with version 2 of app.metrics.  We have imported the following nuget packages: -
App.Metrics.AspNetCore v2.0.0
App.Metrics.AspNetCore.Health v2.0.0
App.Metrics.AspNetCore.Mvc v2.0.0
App.Metrics.AspNetCore.Reporting v2.0.0
App.Metrics.AspNetCore.Tracking v2.0.0
App.Metrics.Reporting.InfluxDB v2.0.0
In startup we have: -
ConfigureServices: -
var metricsBuilder = AppMetrics.CreateDefaultBuilder()
.Configuration.Configure(options =>
{
options.AddEnvTag(settings.EnvironmentTag);
});
We then setup influx DB
services.AddMetricsTrackingMiddleware();
services.AddMetricsReportScheduler();
var metrics = metricsBuilder.Build();
services.AddMetrics(metrics);
services.AddMvc().AddMetrics();
Configure:-
app.UseMetricsAllMiddleware();
This has been working in the past and I can't figure out what has changed to produce this error.  Can somebody please help?
Cheers,
Des

We have found the issue.  The website had been upgraded to .NET 2.2 and as such we had to upgrade app.metrics to the latest version: -
App.Metrics.AspNetCore v3.2.0
App.Metrics.AspNetCore.Health v3.2.0
App.Metrics.AspNetCore.Mvc v3.2.0
App.Metrics.AspNetCore.Reporting v3.2.0
App.Metrics.AspNetCore.Tracking v3.2.0
App.Metrics.Reporting.InfluxDB v3.2.0
This resolved the issue.


Test HDC for broken movement model fix

related to this
https://github.com/Hivemapper/odc-api/issues/93

already in 3.6, needs validation

fixed


Affine Cipher

A type of monoalphabetic substitution cipher in which each letter is mapped to its numeric equivalent, encrypted using a linear transformation, and then converted back to a letter.
Add input and output field also so that people can experiment

I would like to contribute to this issue. @shrinidhihegde3


Add scrolling text feature

Hi! I'm sory but I can not add new code to existing file because memory size is too small.

Thanks, but there's too much going on atm. It may take several weeks until I can review it.

@marcelstoer any update on this? :)

no update


build: switch package manager to pnpm

:tada: This PR is included in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Documentation Improvement: Checking versions of prerequsites

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description
This PR improves documentation. For people who already have pre-existing versions of node and mongo, provide instructions for making sure they have the right versions. This can help prevent contributors from experiencing incompatibility errors.

@systimotic Thanks for the feedback.  We factored it in and added a 2nd commit to this PR.  Question: I know the best practice is to have only a single commit per PR.   Should we squash the commits somehow?  This is our first PR, so not sure how to do this.

Yesss please! You can find instructions on how to squash your commits here

Thanks @systimotic, I squashed the commits .

@atjonathan, I removed the extra newline as you requested.


[opengl] glCurrentPaletteMatrixARB range check error message off by 1

https://github.com/DragonMinded/libdragon/blob/4409fe77e208aaef77800b684602d4ea5a74da0b/src/GL/matrix.c#L158-L159
but the allowed values are 0 to MATRIX_PALETTE_SIZE-1 (inclusive)
I think it should be either
[0, %d))", index, MATRIX_PALETTE_SIZE);
or
[0, %d])", index, MATRIX_PALETTE_SIZE-1);

Fixed by #494


Change records default processor

What is the problem / what does the code in this PR do
The (raw-)records processor is set at the daq. However, when re-making records, let's make sure to make it in a compressed format where ever possible.
We do need to keep in mind though that this might (potentially) lead to problems if the buffers are too big to fit into zstd.
This is especially relevant for (re)processing records.

@ershockley , we were/are on the DAQ, but not as a default elsewhere


when running rooter start --debug, rabbitmq crashes

This is my first time using rooter and having difficulty in getting a magento2 setup up and running
After running rooter start --debug ,

rabbitmq comes up and then crashes
created new project and on the above command , it straight away crashed with below error

When running rooter env:status I see the below
rabbitmq        │ 3097 │ -      │ -      │ 0         │ Completed
Plz help
Content from the commandline
[rabbitmq	] 2024-02-25 21:00:26.670094+00:00 [notice] <0.44.0> Application rabbitmq_prelaunch exited with reason: {{shutdown,{failed_to_start_child,prelaunch,{badmatch,{error,{{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}},{child,undefined,net_sup_dynamic,{erl_distribution,start_link,[#{clean_halt => false,name => rabbit_prelaunch_3115@localhost,name_domain => shortnames,net_tickintensity => 4,net_ticktime => 60,supervisor => net_sup_dynamic}]},permanent,false,1000,supervisor,[erl_distribution]}}}}}},{rabbit_prelaunch_app,start,[normal,[]]}} [rabbitmq	] {"Kernel pid terminated",application_controller,"{application_start_failure,rabbitmq_prelaunch,{{shutdown,{failed_to_start_child,prelaunch,{badmatch,{error,{{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}},{child,undefined,net_sup_dynamic,{erl_distribution,start_link,[#{clean_halt => false,name => rabbit_prelaunch_3115@localhost,name_domain => shortnames,net_tickintensity => 4,net_ticktime => 60,supervisor => net_sup_dynamic}]},permanent,false,1000,supervisor,[erl_distribution]}}}}}},{rabbit_prelaunch_app,start,[normal,[]]}}}"} [rabbitmq	] Kernel pid terminated (application_controller) ({application_start_failure,rabbitmq_prelaunch,{{shutdown,{failed_to_start_child,prelaunch,{badmatch,{error,{{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}},{child,undefined,net_sup_dynamic,{erl_distribution,start_link,[#{clean_halt => false,name => rabbit_prelaunch_3115@localhost,name_domain => shortnames,net_tickintensity => 4,net_ticktime => 60,supervisor => net_sup_dynamic}]},permanent,false,1000,supervisor,[erl_distribution]}}}}}},{rabbit_prelaunch_app,start,[normal,[]]}}}) [rabbitmq	]  [rabbitmq	] Crash dump is being written to: erl_crash.dump...done
Partial content from the log file
=erl_crash_dump:0.5 Sun Feb 25 21:00:28 2024 Slogan: Kernel pid terminated (application_controller) ({application_start_failure,rabbitmq_prelaunch,{{shutdown,{failed_to_start_child,prelaunch,{badmatch,{error,{{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}},{child,undefined,net_sup_dynamic,{erl_distribution,start_link,[#{clean_halt => false,name => rabbit_prelaunch_3115@localhost,name_domain => shortnames,net_tickintensity => 4,net_ticktime => 60,supervisor => net_sup_dynamic}]},permanent,false,1000,supervisor,[erl_distribution]}}}}}},{rabbit_prelaunch_app,start,[normal,[]]}}}) System version: Erlang/OTP 25 [erts-<IP_ADDRESS>] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] Taints: crypto Atoms: 13698 Calling Thread: scheduler:1 =scheduler:1 Scheduler Sleep Info Flags:  Scheduler Sleep Info Aux Work: THR_PRGR_LATER_OP Current Port:  Run Queue Max Length: 0 Run Queue High Length: 0 Run Queue Normal Length: 0 Run Queue Low Length: 0 Run Queue Port Length: 0 Run Queue Flags: OUT_OF_WORK | HALFTIME_OUT_OF_WORK | NONEMPTY | EXEC Current Process: <0.0.0> Current Process State: Running Current Process Internal State: ACT_PRIO_NORMAL | USR_PRIO_NORMAL | PRQ_PRIO_NORMAL | ACTIVE | RUNNING Current Process Program counter: 0x0000000110b8c710 (init:sleep/1 + 64)

Hello @atlenquiryuk, I hope you're doing well.
I would suggest stopping your environment and deleting the RabbitMQ folder from the .devenv/state/rabbitmq path.
Try to start the command again.
Ok?
Thanks,
Cristiano.

Hi @cristiano-pacheco
Thanks for getting back.
Will try and revert back.

Hi @cristiano-pacheco
I just followed your instructions and there is no rabbitmq foleder (.devenv/state/rabbitmq)
see screenshot
Any suggestions ?
Thanks

Could you please share the content of your devenv.nix file?
On Mon, 26 Feb 2024 at 18:36 atlenquiryuk @.***> wrote:

Hi @cristiano-pacheco https://github.com/cristiano-pacheco
I just followed your instructions and there is no rabbitmq foleder
(.devenv/state/rabbitmq)
see screenshot
Any suggestions ?
Thanks
Screenshot.2024-02-26.at.21.34.08.png (view on web)
https://github.com/run-as-root/rooter/assets/139079656/10803bfd-f98e-4765-a2fc-47266f2ba98d
—
Reply to this email directly, view it on GitHub
https://github.com/run-as-root/rooter/issues/9#issuecomment-1965334605,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGZOBPCIKOGASN4ZGEIURDYVT55JAVCNFSM6AAAAABDZFHM5CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRVGMZTINRQGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @cristiano-pacheco
Yes, see below.
When I start the environment, I get the following

`{ pkgs, inputs, lib, config, ... }:
let
rooterBin = if builtins.getEnv "ROOTER_BIN" != "" then builtins.getEnv "ROOTER_BIN" else "rooter";
composerPhar = builtins.fetchurl{
url = "https://github.com/composer/composer/releases/download/2.2.22/composer.phar";
sha256 = "1lmibmdlk2rsrf4zr7xk4yi5rhlmmi8f2g8h2izb8x4sik600dbx";
};
in {
dotenv.enable = true;
env = {
PROJECT_NAME = "rooter-mage";
PROJECT_HOST = "rooter-mage.rooter.test";
    NGINX_PKG_ROOT = pkgs.nginx;
    DEVENV_STATE_NGINX = "${config.env.DEVENV_STATE}/nginx";

    DEVENV_PHPFPM_SOCKET = "${config.env.DEVENV_STATE}/php-fpm.sock";

    DEVENV_DB_NAME = "app";
    DEVENV_DB_USER = "app";
    DEVENV_DB_PASS = "app";

    DEVENV_AMQP_USER = "guest";
    DEVENV_AMQP_PASS = "guest";
};

# PACKAGES
packages = [
    pkgs.git
    pkgs.gnupatch
    pkgs.curl
    pkgs.yarn
    pkgs.gettext
    pkgs.n98-magerun2
];

# Composer 2.2.x required by Magento2 <=2.4.6
scripts.composer.exec = ''
    php ${composerPhar} $@
'';

# process-compose
process.implementation="process-compose";
process.process-compose={
    "port" = config.env.DEVENV_PROCESS_COMPOSE_PORT;
    "tui" = "false";
    "version" = "0.5";
};

# PHP
languages.php = {
    enable = true;
    package = inputs.phps.packages.${builtins.currentSystem}.php81.buildEnv {
        extensions = { all, enabled }: with all; enabled ++ [ redis xdebug xsl ];
        extraConfig = ''
            memory_limit = -1
            error_reporting=E_ALL
            xdebug.mode = coverage,debug
            sendmail_path = ${pkgs.mailpit}/bin/mailpit sendmail -S <IP_ADDRESS>:${config.env.DEVENV_MAIL_SMTP_PORT}
            display_errors = On
            display_startup_errors = On
        '';
    };
    fpm.phpOptions =''
        memory_limit = -1
        error_reporting=E_ALL
        xdebug.mode = debug
        sendmail_path = ${pkgs.mailpit}/bin/mailpit sendmail -S <IP_ADDRESS>:${config.env.DEVENV_MAIL_SMTP_PORT}
        display_errors = On
        display_startup_errors = On
    '';
    fpm.pools.web = {
        listen = "${config.env.DEVENV_PHPFPM_SOCKET}";
        settings = {
            "clear_env" = "no";
            "pm" = "dynamic";
            "pm.max_children" = 20;
            "pm.start_servers" = 6;
            "pm.min_spare_servers" = 1;
            "pm.max_spare_servers" = 10;
        };
    };
};

# JS
languages.javascript = {
    enable = true;
    package = pkgs.nodejs-18_x;
};

# nginx
services.nginx = {
    enable = true;
    configFile = "${config.env.DEVENV_STATE_NGINX}/nginx.conf";
};

# DATABASE
services.mysql = {
    enable = true;
    package = pkgs.mariadb_106;
    settings = {
        mysqld = {
            port = config.env.DEVENV_DB_PORT;
            innodb_buffer_pool_size = "2G";
            table_open_cache = "2048";
            sort_buffer_size = "8M";
            join_buffer_size = "8M";
            query_cache_size = "256M";
            query_cache_limit = "2M";
        };
    };
    initialDatabases = [{ name = "${config.env.DEVENV_DB_NAME}"; }];
    ensureUsers = [
        {
            name = "${config.env.DEVENV_DB_USER}";
            password = "${config.env.DEVENV_DB_PASS}";
            ensurePermissions = { "${config.env.DEVENV_DB_NAME}.*" = "ALL PRIVILEGES"; };
        }
    ];
};

# mailpit
services.mailpit = {
    enable = true;
    uiListenAddress   = "<IP_ADDRESS>:${config.env.DEVENV_MAIL_UI_PORT}";
    smtpListenAddress = "<IP_ADDRESS>:${config.env.DEVENV_MAIL_SMTP_PORT}";
};

# Redis
services.redis = {
    enable = true;
    port = lib.strings.toInt ( config.env.DEVENV_REDIS_PORT );
};

# ElasticSearch
services.elasticsearch = {
    enable = true;
    port = lib.strings.toInt ( config.env.DEVENV_ELASTICSEARCH_PORT );
    tcp_port = lib.strings.toInt ( config.env.DEVENV_ELASTICSEARCH_TCP_PORT );
};

# RabbitMQ
services.rabbitmq = {
    enable = true;
    port = lib.strings.toInt ( config.env.DEVENV_AMQP_PORT );
    managementPlugin = {
        enable = true;
        port = lib.strings.toInt ( config.env.DEVENV_AMQP_MANAGEMENT_PORT );
    };
};

}
`

Hello @atlenquiryuk, thank you for sharing it.
Let's do a quick test.
Please change your devenv.nix file to have this content:
{ pkgs, inputs, lib, config, ... }:

let
    rooterBin = if builtins.getEnv "ROOTER_BIN" != "" then builtins.getEnv "ROOTER_BIN" else "rooter";
    pkgs-stable = import inputs.nixpkgs-stable { system = pkgs.stdenv.system; };
    composerPhar = builtins.fetchurl{
        url = "https://github.com/composer/composer/releases/download/2.2.22/composer.phar";
        sha256 = "1lmibmdlk2rsrf4zr7xk4yi5rhlmmi8f2g8h2izb8x4sik600dbx";
    };
    magerun2Phar = builtins.fetchurl{
        url = "https://github.com/netz98/n98-magerun2/releases/download/7.2.0/n98-magerun2.phar";
        sha256 = "0z1dkxz69r9r9gf8xm458zysa51f1592iymcp478wjx87i6prvn3";
    };
in {
    dotenv.enable = true;
    env = {
        PROJECT_NAME = "rooter-mage";
        PROJECT_HOST = "rooter-mage.rooter.test";

        NGINX_PKG_ROOT = pkgs.nginx;
        DEVENV_STATE_NGINX = "${config.env.DEVENV_STATE}/nginx";

        DEVENV_PHPFPM_SOCKET = "${config.env.DEVENV_STATE}/php-fpm.sock";

        DEVENV_DB_NAME = "app";
        DEVENV_DB_USER = "app";
        DEVENV_DB_PASS = "app";

        DEVENV_AMQP_USER = "guest";
        DEVENV_AMQP_PASS = "guest";
    };

    # PACKAGES
    packages = [
        pkgs.git
        pkgs.gnupatch
        pkgs.curl
        pkgs.yarn
        pkgs.gettext
    ];

    scripts.composer.exec = ''php ${composerPhar} $@''; # Composer 2.2.x required by Magento2 <=2.4.6
    scripts.magerun2.exec = ''php ${magerun2Phar} $@''; # magerun2 without hardlock to php8.2

    # process-compose
    process.implementation="process-compose";
    process.process-compose={
        "port" = config.env.DEVENV_PROCESS_COMPOSE_PORT;
        "tui" = "false";
        "version" = "0.5";
    };

    # PHP
    languages.php = {
        enable = true;
        package = inputs.phps.packages.${builtins.currentSystem}.php81.buildEnv {
            extensions = { all, enabled }: with all; enabled ++ [ redis xdebug xsl ];
            extraConfig = ''
                memory_limit = -1
                error_reporting=E_ALL
                xdebug.mode = coverage,debug
                sendmail_path = ${pkgs.mailpit}/bin/mailpit sendmail -S <IP_ADDRESS>:${config.env.DEVENV_MAIL_SMTP_PORT}
                display_errors = On
                display_startup_errors = On
            '';
        };
        fpm.phpOptions =''
            memory_limit = -1
            error_reporting=E_ALL
            xdebug.mode = debug
            sendmail_path = ${pkgs.mailpit}/bin/mailpit sendmail -S <IP_ADDRESS>:${config.env.DEVENV_MAIL_SMTP_PORT}
            display_errors = On
            display_startup_errors = On
        '';
        fpm.pools.web = {
            listen = "${config.env.DEVENV_PHPFPM_SOCKET}";
            settings = {
                "clear_env" = "no";
                "pm" = "dynamic";
                "pm.max_children" = 20;
                "pm.start_servers" = 6;
                "pm.min_spare_servers" = 1;
                "pm.max_spare_servers" = 10;
            };
        };
    };

    # JS
    languages.javascript = {
        enable = true;
        package = pkgs.nodejs_20;
    };

    # nginx
    services.nginx = {
        enable = true;
        configFile = "${config.env.DEVENV_STATE_NGINX}/nginx.conf";
    };

    # DATABASE
    services.mysql = {
        enable = true;
        package = pkgs.mariadb_106;
        settings = {
            mysqld = {
                port = config.env.DEVENV_DB_PORT;
                innodb_buffer_pool_size = "2G";
                table_open_cache = "2048";
                sort_buffer_size = "8M";
                join_buffer_size = "8M";
                query_cache_size = "256M";
                query_cache_limit = "2M";
            };
        };
        initialDatabases = [{ name = "${config.env.DEVENV_DB_NAME}"; }];
        ensureUsers = [
            {
                name = "${config.env.DEVENV_DB_USER}";
                password = "${config.env.DEVENV_DB_PASS}";
                ensurePermissions = { "${config.env.DEVENV_DB_NAME}.*" = "ALL PRIVILEGES"; };
            }
        ];
    };

    # mailpit
    services.mailpit = {
        enable = true;
        uiListenAddress   = "<IP_ADDRESS>:${config.env.DEVENV_MAIL_UI_PORT}";
        smtpListenAddress = "<IP_ADDRESS>:${config.env.DEVENV_MAIL_SMTP_PORT}";
    };

    # Redis
    services.redis = {
        enable = true;
        port = lib.strings.toInt ( config.env.DEVENV_REDIS_PORT );
    };

    # ElasticSearch
    services.elasticsearch = {
        enable = true;
        port = lib.strings.toInt ( config.env.DEVENV_ELASTICSEARCH_PORT );
        tcp_port = lib.strings.toInt ( config.env.DEVENV_ELASTICSEARCH_TCP_PORT );
    };

    # RabbitMQ
    services.rabbitmq = {
        package = pkgs-stable.rabbitmq-server;
        enable = true;
        port = lib.strings.toInt ( config.env.DEVENV_AMQP_PORT );
        managementPlugin = {
            enable = true;
            port = lib.strings.toInt ( config.env.DEVENV_AMQP_MANAGEMENT_PORT );
        };
    };
}

Change the content of your devenv.yaml file to:
allowUnfree: true
inputs:
  nixpkgs:
    url: github:NixOS/nixpkgs/nixpkgs-unstable
  nixpkgs-stable:
    url: github:NixOS/nixpkgs/nixos-23.05
  phps:
    url: github:fossar/nix-phps
    inputs:
      nixpkgs:
        follows: nixpkgs

Change your .envrc file to:
source_url "https://raw.githubusercontent.com/cachix/devenv/d1f7b48e35e6dee421cfd0f51481d17f77586997/direnvrc" "sha256-YBzqskFZxmNb3kYVoKD9ZixoPXJh1C9ZvTLGFRkauZ0="

export NIXPKGS_ALLOW_INSECURE=1
dotenv_if_exists .env
watch_file .env

use devenv

Delete the files and folders with the command below:

devenv.lock
.devenv.flake.nix
.devenv
.direnv

After it, please, restart your terminal and start the project again with the debug mode.
What have we changed?
We added a stable input thought the devenv.yaml file and we changed the declaration of the RabbitMQ service to use the RabbitMQ server from the stable channel. With it it will use RabbitMQ's 3.11 version instead of 3.12
devenv.yaml file

nixpkgs-stable:
    url: github:NixOS/nixpkgs/nixos-23.05

devenv.nix file

services.rabbitmq = {
        package = pkgs-stable.rabbitmq-server;

Please, let me know what are the results.
Thanks,
Cristiano.

Hi @cristiano-pacheco
I followed your steps, now after running start in debug mode and when I rus status in anther cmd tab I get the below.
I checked the devenv.nix and I can see the project name is already declared here PROJECT_NAME = "rooter-mage";
So not sure why its throwing the below error

Hello @atlenquiryuk, the issues you are facing are very weird.
How can this happen if the variable is already set? I don't know what to say 🙈
Maybe define this variable in your .env file and make a new test. Please.

@atlenquiryuk The error you shared in your last screenshot looks like the environment variables from .env are not set in your current shell.
Can you check

.env file and its content contains the variables
if direnv allow . was executed

Running env in the shell of the project should contain the variables from .env file.

Hi @mwr @cristiano-pacheco
I finally got pass that issue.
@mwr you are right, I did source ~/.bashrc and then all worked.
Now I'm facing a different issue when running rooter magento2:db-install

Thanks

@atlenquiryuk Great that it worked now.
It seems the error you have now is a timeout of the php process triggered.
Let me create an PR for that.
With the latest release https://github.com/run-as-root/rooter/releases/tag/1.0.0-beta6 we already introduced a timeout for mysql:dump, we should probably have the timeout for the processes configurable on a higher level.

Hi @mwr
Thank you.
While I wait for PR and merge, is there a temp fix that I can put to fix this ?.
Thanks

Hello @atlenquiryuk, you can run the bin/magento setup:install command manually.

Hi @cristiano-pacheco @mwr
Just another thing to consider for the release, in the install command, I'm getting error The "--elasticsearch" option does not exist.
This might need fixing

Hi @cristiano-pacheco @mwr
Just another thing to consider for the release, in the install command, I'm getting error The "--elasticsearch" option does not exist.
This might need fixing .
In latest version it uses opensearch as the default

Hello @atlenquiryuk,
I'm not able to reproduce the timeout issue. According to my debug session, it will never occur because the timeout is disabled for that command.
opensearch is not the default. It will be used only if you set the DEVENV_OPENSEARCH_PORT env variable is set AND DEVENV_ELASTICSEARCH_PORT is not set.

@cristiano-pacheco thanks for double checking.
@atlenquiryuk what is the result of the magento install command when you run it manually.
Do you also get a timeout?

Hi @mwr
I'm getting error The "--elasticsearch" option does not exist. This one is really strange, I pinged elasticsearch to check it status and all looks good. So not sure why Im getting this error.
When using bin/magento setup:install, i dint receive a timeout error.
Any suggestions on the "--elasticsearch" option does not exist issue ?
Thanks

@atlenquiryuk What magento version are you using?
you can check the options by running bin/magento setup:install --help

Just to add to the previous comment.
Im able to access traffic dashboard

Not able to access mailpit at https://rooter-mage-mail.rooter.test/

Update
Im able to access mailpit by visiting http://<IP_ADDRESS>:18223/ as per the screenshot from traffic but not the magento website

Hi @mwr @cristiano-pacheco
Thank you guys so much for your help, got it finally working.
@cristiano-pacheco dint realise rooter doesn't add a host entry into the hosts file. Might be helpful to the other folks following the guide if you could add this to the it.
Thanks again

Hi @cristiano-pacheco @mwr
Do you plan to add varnish support for magento 2 environment ?
Thanks

@cristiano-pacheco dint realise rooter doesn't add a host entry into the hosts file. Might be helpful to the other folks following the guide if you could add this to the it.

rooter uses dnsmasq and does register a rooter.test route to be routed to traefik.
Maybe did you get an error during rooter installation while running: rooter install?
This command should create the setup for dnsmasq, traefik and also ssl certs.

Do you plan to add varnish support for magento 2 environment ?

Yes we do, I have created a public issue for this: https://github.com/run-as-root/rooter/issues/10

Hi @mwr
Thanks for getting back.
Yes, I did run the install command and as far as I remember everything had gone well.
After adding the domains to host file, https works (see screenshot). May be there was an issue and I dint notice.
.
If I run rooter install now would that cause any issue to existing environments ?
Thanks

Hi @mwr
Just wanted to give an update incase it helps anyone else.
In /etc/resolver/rooter.test I had the port entry missing, so added it like below and all good now. No need of adding entry to /etc/hosts anymore.

Thanks again for all your help

@techguyuk Great that you got it working.

If I run rooter install now would that cause any issue to existing environments ?

You can re-run rooter install. It also has an option --force to ensure every step is re-run again. With out the force option it would detect existing files and not overwrite them.

In /etc/resolver/rooter.test I had the port entry missing, so added it like below and all good now. No need of adding entry to /etc/hosts anymore.

Not sure how it can happen that there is no port written to that file.
Was the whole line missing or just the port-number?

Hi @mwr
There were 2 files in /etc/resolver/

test   : this file had 2 entries including the port
rooter.test  : this file only had one entry without the port

Hi @mwr @cristiano-pacheco
Sorry to bother you again.
Im trying to setup xdebug, I followed the steps mentioned in the doc by

running export XDEBUG_CONFIG="idekey=PHPSTORM-XDEBUG" in the command line
switching xdebug on in phpstorm
add a breakpoint in pub/index.php
reload magento webpage

Page loads with no break point trigger.
Any help ?

Hi @mwr
I got this working by doing some phpstorm settings as below

installed browser xdebug plug and configured use phpstorm, it sends a xdebug session cookie with value PHPSTORM when enabled

configured phpstorm as below

now enable debugging in phpstorm

load webpage

stops at the added break point


Line wrap for &&

I've seen f13b93e and probably because of that my code is wrong. My snippet looks like this
            .assertValueAt(2) {
                it is ImageChangeEvent.ImageUpdated
                        && it.position == 0
                        && it.changedFields.contains(CollectImage.IMAGE_MODE)
                        && it.image.imageMode == newImageMode
            }

Kotlin Lint complaints with Line must not begin with "&&". I couldn't find anything regarding that in the Kotlin style guide and to me this rule seems wrong. Can you point me to the rule in the style guide? Is there a way to disable this specific check?
On a side note, this is working fine and Kotlin Lint doesn't complain
            .assertValueAt(2) {
                it is ImageChangeEvent.ImageUpdated
                        && it.position == 0 &&
                        it.changedFields.contains(CollectImage.IMAGE_MODE)
                        && it.image.imageMode == newImageMode
            }

See https://github.com/shyiko/ktlint/issues/163#issuecomment-369418775.
Also, thank you for the snippet.

Fixed in 0.20.0.

@shyiko No, it's still not working as expected.
if (!isLocal
    && workspace.userId != account.userId
    && workspace.contactId != account.userId) {
}

Class.kt:378:25: Line must not begin with "&&"
Class.kt:379:25: Line must not begin with "&&"

Should I create a new issue or do you want to reopen this one?

@vRallev It works in accordance with https://github.com/shyiko/ktlint/issues/163#issuecomment-369418775.

I can't follow the comments in the link. They're mostly talking about +/- and not && or ||. I couldn't find anything about that. Note that the auto-formatter inside of IntelliJ formats my sample from above like this:
if (!isLocal
    && workspace.userId != account.userId
    && workspace.contactId != account.userId
) {
    // do something after 4 space indent
}

But for the formatter the following is also fine
if (!isLocal &&
    workspace.userId != account.userId &&
    workspace.contactId != account.userId
) {
    // do something after 4 space indent
}

So there's no evidence which one of them is correct.

How can we get the intellij auto-formatter to always move those operators to the end of the line? I can't find any options for that setting 🤔

How can we get the intellij auto-formatter to always move those operators to the end of the line? I can't find any options for that setting 🤔

Did you find a solution for that?

How can we get the intellij auto-formatter to always move those operators to the end of the line? I can't find any options for that setting thinking

Did you find a solution for that?

Have you found the settings for that?


count and page params for search broken

Summary
The count and page params for the search() method is broken.
Reproduction Steps
Invoking these won't work
api.search('...', count=1, page=3)
api.search('...', count=1)
api.search('...', page=3)

Expected Result
The final proper SearchResult object
Actual Result
Bad request error with 400 status code
System Information
N/A
Checklist

[X] I have searched all open and closed issues for any duplicates.

Additional Information
I'm opening this issue so that I can fix it later when I get the time.

Looks like this got fixed itself.


What would be the equivalent pinout if this software is used on a regular Pi Pico?

What is the Pinout for ADDRESS and DATA as laid out on a standard Pi Pico? I.e Does DATA 0-7 correspond to GPIO0,1,2,3,4,5,6,7 etc?

Happy to answer, but I don't understand the question.

Ah, I see it uses an I/O Expander, thanks anyway.

Happy to answer, but I don't understand the question.

I expanded and rephrased my question, in short, i see you built a custom board, but could this be used with a regular pi pico?

The address pins are GPIO 0-17 and the data pins are GPIO are 22-29. The Pico Pi is 40 pins while this is intended to be used in place of 32-pin ROMS and the layout of the pins with the pico pi would not be compatible with any standard ROM pin layouts because of how the GND and power pins are placed.

...Not to mention that there are several GPIOs which aren't exposed on the Pico board (GPIO 23, 24, 25, 29), so even if one created a "converter" board, and even if that converter board didn't have problems with physical dimensions, it still wouldn't be sufficient electrically.


Wrap send_pay in awaitable channel futures

It would be great if send_pay could be wrapped in awaitable channel futures, just like pay and trampoline_pay. The Breez SDK uses send_pay to pay for reverse swaps. They sometimes fail with failed: WIRE_TEMPORARY_CHANNEL_FAILURE (First peer not ready), meaning the channels between greenlight and the LSP are not yet reestablished at the moment of calling send_pay.

We have this patch pending deployment, but are fighting with the CI to get them deployed. I'll update here once the patches are deployed.

This patch has been deployed last week :+1:


Houdini 17.5 and USD Hydra plugin?

Hi,
I've tested out building the Hydra plugin using Houdini 17.5.327 and I got pretty close but seems like there is a few things that I can't seem to figure out and just throwing it out there to see if you have solved this. Please feel free to reject this issue.
First off, houdini 17.5 now uses a USD that ships with Houdini, and building anything using anything other than it's inhouse USD will make it pretty sad. So for instance, building the OP_gusd that ships with Houdini is now done like this:
export HOME=/usr/local/USD.19
cmake3 /opt/hfs17.5.327/toolkit/usd_houdini_plugins -DCMAKE_C_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/gcc -DCMAKE_CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/g++ -DCMAKE_PREFIX_PATH=/opt/hfs17.5.327/toolkit
make -j12 install

I've looked at how that is setup and used that as a base to generate this CMakeLists.txt for the Hydra plugin. It compiles fine but segfaults om some missing functions that are not dynamically linked. First test it segfaults on startup with this:
/opt/hfs17.5.327/bin/houdinicore-bin: symbol lookup error: /usr/local/USD.19/houdini17.5/dso/Hydra.so: undefined symbol: _ZN32pxrInternal_v0_19__pxrReserved__16GusdGU_PackedUSD7installER19GA_PrimitiveFactory

That points to:
pxrInternal_v0_19__pxrReserved__::GusdGU_PackedUSD::install(GA_PrimitiveFactory&)
which by inspecting the .so file is undefined in the Hydra.so file. It does exists in the libHoudiniUSD.so though file, but it seems to fall off during dynamically linking it.
It fails here:
https://github.com/dreamworksanimation/dwa_usd_plugins/blob/master/third_party/houdini/plugin/Hydra/plugin.cpp#L34
Removing that line made it go further since that is done on startup in 17.5 and above when talking to sidefx, so no need to do that anymore. But there are still some functions that are missing and it now fails here:
/opt/hfs17.5.327/bin/houdinicore-bin: symbol lookup error: /usr/local/USD.19/houdini17.5/dso/Hydra.so: undefined symbol: _ZNK32pxrInternal_v0_19__pxrReserved__16GusdGU_PackedUSD10getUsdPrimE16UT_ErrorSeverity

looking at the libgusd.so file that is the lib it used before the libHoudiniUSD.so file, it does seem to have more dynamic functions exposed.
nm -D /usr/local/USD.19/houdini17.5/lib/libgusd.so | grep getUsdPrim
000000000014ab40 W _ZNK32pxrInternal_v0_19__pxrReserved__15GusdMeshWrapper10getUsdPrimEv
00000000002882b0 T _ZNK32pxrInternal_v0_19__pxrReserved__16GusdGU_PackedUSD10getUsdPrimE16UT_ErrorSeverity

vs libHoudiniUSD.so
nm -D /opt/hfs17.5.327/dsolib/libHoudiniUSD.so | grep getUsdPrim
0000000000460cf0 T _ZN32pxrInternal_v0_19__pxrReserved__23HUSDgetUsdPrimPredicateE25HUSD_PrimTraversalDemands

I'm not too familiar with what would make the ligbusd.so file have that function show up vs the libHoudiniUSD.so file not having it. I take it that if I were to add the USD_API to that function that it would make it to the dynamically linked functions, but that would require rebuilding libHoudiniUSD.so I think.
If you have any pointers or ideas of solving this they would be more than welcome.
Many thanks,
Dan

Hi Dan.
I'm afraid I don't have any helpful advice for you here. We have skipped Houdini 17.5 in favor of Houdini 18, so that we can use HoudiniUsdBridge to use Houdini with our internal USD build. We haven't yet tried building this extension with Houdini 18, but if we attempt it, I expect we may run into the same problems you are with API changes to gusd. I'm hoping we won't need to do that though, since Houdini 18 has some pretty great Hydra integration already, which I think removes the need for this extension.

Hi Alan,
Yeah the HoudiniUsdBridge will be the thing to use in 18, and most likely pave the way to build this plugin as well. Although as you mentioned, maybe there will not be much need for this extension in 18 with Lops etc. However, this plugin will still be needed if you were to take it outside of the lops workflow since hydra is not working with SOPs, so we would need to compile this too. But with the HoudiniUsdBridge it might be easier to get going.
To fix this for 18, if needed, I think the main thing to do is to make certain functions having the USD_API added to them to be exposed in the dynamic linking.
I think we will do the same as you guys and target 18 instead of trying to fix this for 17.5.
Closing this issue.
Thanks for your help Alan.
Cheers,
Dan


Windows 11 Impeller Toolchain Errors: PathNotFoundException: Cannot open file

Steps to reproduce
Using Windows 11 machine. Followed along with this article: https://medium.com/flutter/getting-started-with-flutter-gpu-f33d497b7c11.
Ran a few commands:
flutter config --enable-native-assets
flutter clean
flutter run -d<PHONE_NUMBER>825 --enable-impeller --verbose
Encountered command line build errors.
Expected results
The test application to be running.
Actual results
[  +11 ms] Building native assets for package:gpu_tests failed.
[   +1 ms] PathNotFoundException: Cannot open file, path = 'E:\Dev\Repos.r2d2Proton\gpu_tests\shaders.shaderbundle.json' (OS Error: The system cannot find the file specified.
[        ] , errno = 2)
https://gist.github.com/r2d2Proton/ef0b569ef16a32dc3e8ffced94cb0b9e
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

I was following with this (https://docs.flutter.dev/ui/animations/tutorial) and nearly the same effect as in the image is not completely redrawn. I will upload a picture in a moment. Almost like the backbuffer is not cleared. Note - I noticed a very similar problem with a completely different rendering.

Hi @r2d2Proton

"E:\Dev\Repos.r2d2Proton\gpu_tests.dart_tool"

Can you try moving/copying the project to a location without . in the path to see if the error persists? If the error does not disappear, can you share the sample code? And please provide the output of flutter doctor -v. Thanks!

Thank you @huycozy . I have removed the period in the path and still encountered the error. I updated the gist (https://gist.github.com/r2d2Proton/ef0b569ef16a32dc3e8ffced94cb0b9e).
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter clean
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter run -d<PHONE_NUMBER>825 --enable-impeller --verbose
[   +3 ms] Could not interpret results of "git describe": 3.27.0-1.0.pre.74
:
:
[ +220 ms] Unhandled exception:
[   +2 ms] PathNotFoundException: Cannot open file, path = 'E:\Dev\Repos\r2d2Proton\gpu_tests\shaders.shaderbundle.json' (OS Error: The system cannot find the file specified.
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter doctor -v
[✓] Flutter (Channel main, 3.27.0-1.0.pre.74, on Microsoft Windows [Version 10.0.22631.4317], locale en-US)
• Flutter version 3.27.0-1.0.pre.74 on channel main at E:\Dev\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision e08ad36dd9 (2 days ago), 2024-10-16 19:15:06 +0200
• Engine revision 46c28aaa46
• Dart version 3.7.0 (build 3.7.0-27.0.dev)
• DevTools version 2.40.1
[✓] Windows Version (Installed version of Windows is version 10 or higher)
[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
• Android SDK at E:\Dev\Android\Sdk
• Platform android-35, build-tools 35.0.0
• ANDROID_HOME = E:\Dev\Android\Sdk
• Java binary at: E:\Dev\Android\Android Studio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 21.0.3+-12282718-b509.11)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.11.4)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.11.35312.102
• Windows 10 SDK version 10.0.26100.0
[✓] Android Studio (version 2024.2)
• Android Studio at E:\Dev\Android\Android Studio
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 21.0.3+-12282718-b509.11)
[✓] VS Code (version 1.94.2)
• VS Code at C:\Users\Jerry\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.98.0
[✓] Connected device (4 available)
• SHIELD Android TV (mobile) •<PHONE_NUMBER>825 • android-arm64  • Android 11 (API 30)
• Windows (desktop)          • windows       • windows-x64    • Microsoft Windows [Version 10.0.22631.4317]
• Chrome (web)               • chrome        • web-javascript • Google Chrome 130.0.6723.58
• Edge (web)                 • edge          • web-javascript • Microsoft Edge 129.0.2792.89
[✓] Network resources
• All expected network resources are available.
• No issues found!

And here is my project after removing the period in the path: https://github.com/r2d2Proton/gpu_tests
Again, thank you @huycozy for your help :)

@huycozy Also, the toolchain recreated the original directory for the assets. Please see this issue: https://github.com/flutter/flutter/issues/157164

Looks like this is caused by your shaders.shaderbundler.json file name is not correct, let's try renaming it like this (after renaming, I see the app runs. there is only a black background color, though):

@huycozy After renaming the shaderbundle to the same name in the article, my_renderer.shaderbundle.json, still got the same error:
Unhandled exception:
PathNotFoundException: Cannot open file, path = 'E:\Dev\Repos\r2d2Proton\gpu_tests\my_renderer.shaderbundle.json' (OS Error: The system cannot find the file specified.
And I issued numerous commands:
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter clean
Deleting build...                                                    6ms
Deleting .dart_tool...                                              14ms
Deleting Generated.xcconfig...                                       0ms
Deleting flutter_export_environment.sh...                            1ms
Deleting ephemeral...                                                2ms
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter channel main
Switching to flutter channel 'main'...
Upgrading engine...
Successfully switched to flutter channel 'main'.
To ensure that you're on the latest build from this channel, run 'flutter upgrade'
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter upgrade
Upgrading Flutter to 3.27.0-1.0.pre.179 from 3.27.0-1.0.pre.74 in E:\Dev\flutter...
:
Running flutter doctor...
:
• No issues found!
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter pub add flutter_gpu --sdk=flutter
"flutter_gpu" is already in "dependencies". Will try to update the constraint.
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter pub add flutter_gpu_shaders
"flutter_gpu_shaders" is already in "dependencies". Will try to update the constraint.
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter config --enable-native-assets
Setting "enable-native-assets" value to "true".
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter pub add native_assets_cli
"native_assets_cli" is already in "dependencies". Will try to update the constraint.
E:\Dev\Repos\r2d2Proton\gpu_tests>flutter run --enable-impeller
Connected devices:
[1]: SHIELD Android TV<PHONE_NUMBER>825)
[2]: Pixel 7 Pro (28311FDH3008TW)
[3]: Windows (windows)
[4]: Chrome (chrome)
[5]: Edge (edge)
Please choose one (or "q" to quit): 2
Unhandled exception:
PathNotFoundException: Cannot open file, path = 'E:\Dev\Repos\r2d2Proton\gpu_tests\my_renderer.shaderbundle.json' (OS Error: The system cannot find the file specified.
Here is my updated repo: https://github.com/r2d2Proton/gpu_tests
Thank you for the help.

I meant the file name has a redundant r character :)
- my_renderer.shaderbundler.json
+ my_renderer.shaderbundle.json

I see it's not correct on the repo from your latest change.

@huycozy  I see. r happy. Testing now with the correct filename. I do suspect that to be the case. A file the build tools cannot find. Following the tutorial again, using the same project but from flutter clean, etc. Will update my findings shortly.

@huycozy I apologize for not noticing the misspelling. Sorry for wasting your time on a user-error. The triangle renders properly on my Pixel 7 phone :) I will continue to test the flutter_scene too. I am anxious to give this a try in my own app as it progresses, the flutter_gpu, flutter_scene, and my own app. Thank you again. I should know better to over check the spelling. I did check multiple times but the 'r' escaped me :D :D

@huycozy And got flutter_scene working too :D :D

Glad I can help. Closing the issue as solved.


Create a msg.Deduplicate function/method

Copy/improve SkyDNS' dedup https://github.com/skynetservices/skydns/blob/0078e9f5139e334e75b6f16f1faa4c938ed79954/server/server.go#L875 function over in this lib.

Another being tested/implemented in SkyDNS: short CNAMEs before A/AAAA in a message. This is not to break stub-resolvers like glibc.

Dedup should work on a []dns.RR. Should it also sort CNAMEs before A records? If so Cleanup might be a better name (or at least something different than Dedup). It also ties in with #223 because dedupping makes the msg smaller.
In general I think we want:

Dedup([]RR)
SomethingSomething(dns.Msg) that does dedup as one of the steps.

Adding some commments from @asergeyev in here

ideally work on wireformat and get the TTL out from there, dropping the whole Sprintf (for now at least)
check the TTLs and keep the longest value when dedupping
Look at CNAMEs as well, maybe DNAME and friends and filter those too
Normalize the domain name before comparing, because of DNS case preservation crap
A Go map might not be the best thing for this.

It looks like this post has been changed since it arrived in email.  Re TTL longest vs shortest values...
http://tools.ietf.org/html/rfc2181#section-5.2
Should an authoritative source send such a malformed RRSet, the
client should treat the RRs for all purposes as if all TTLs in the
RRSet had been set to the value of the lowest TTL in the RRSet.  In
no case may a server send an RRSet with TTLs not all equal.

[ Quoting<EMAIL_ADDRESS>in "Re: [dns] Create a msg.Deduplicate ..." ]

It looks like this post has been changed since it arrived in email.  Re TTL longest vs shortest values...

Yes, I edited after I saved. Github sends immediately.

http://tools.ietf.org/html/rfc2181#section-5.2
Should an authoritative source send such a malformed RRSet, the
client should treat the RRs for all purposes as if all TTLs in the
RRSet had been set to the value of the lowest TTL in the RRSet.  In
no case may a server send an RRSet with TTLs not all equal.

Yep, indeed we want lowest.
/Miek
--
Miek Gieben

As for normalizing, in dnssec.go we have: https://github.com/miekg/dns/blob/master/dnssec.go#L289 (strings.ToLower). There is a bytes.ToLower that might work as intended on the packed byte slice.

strings.ToLower and bytes.ToLower are identical ... Create new string/byteslice by passing runes thru map func. Since that func is probably not allowed to mess with "source material";  you might want to check if there are chars in A-Z range before you call it (lowercasing of non-ASCII characters in dns labels is not allowed by RFC I think...)

(and if there are places where you can mess with source racing thru bytes and inplace adding 0x20 to A-Z is usually faster)

[ Quoting<EMAIL_ADDRESS>in "Re: [dns] Create a msg.Deduplicate ..." ]

(and if there are places where you can mess with source racing thru bytes and inplace adding 0x20 to A-Z is usually faster)

Ack, we can do that later. Right now I can mess with the material because I
have a freshly allocated buffer.

Multiple iterations done. Settled on the one I just merged, API might still change though.


feat: add grid-template-columns/rows

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Simplification of bounds could be a lot better

We often get things like this:
ab = allocate(heap, 4, {
  {[buffer_min(abc, 0), buffer_max(abc, 0)], (buffer_extent(c, 0) * 4), 3},
  {[buffer_min(c, 0), buffer_max(c, 0)], 4, <>}
}) {
 serial loop(i in [buffer_min(abc, 0), buffer_max(abc, 0)], 3) {
  crop_dim(abc, 0, [i, min(buffer_max(abc, 0), (i + 2))]) {
   crop_dim(ab, 0, [i, min(buffer_max(abc, 0), (i + 2))]) {
    call(<fn>, {a, b}, {ab})
   }
   call(<fn>, {ab, c}, {abc})
  }
 }
}

In this case, the min(buffer_max(abc, 0), .) expressions are unnecessary. crop_dim (and crop_buffer) clamp implicitly. And even if they didn't (or we didn't know that), the inner crop_dim should be able to simplify its clamp knowing that the outer crop_dim has already been clamped.

After #128, there are a lot more examples of needing to clean up bounds of these unnecessary clamps.


Mattermost and Thunderbird

This not really a bug, I would like to know if it is possible to use mattermost on Thunderbird as client?
I don't find information about this feature, but I saw thunderbird in the list of clients.

Thunderbird is listed as an email client, as in you can use it to receive email sent by the system. As far as I know, there is no way to use Thunderbird as a UI for Mattermost.

Ok thanks!

I’m fairly certain you can open additional tabs (chrome or chromium) and I bet you could pin that with mattermost web client.

I was able to do it using the Open In New Tab plugin, but I haven't figured out how to pin it.


It has been used well, but it suddenly appears gamepad disconnected

All text of the prompt is "A gamepad has disconnected and is not available anymore".Please tell me what's going on and how to solve it
.thank you very much.

I vaguely remember that I installed irungentoo/Xiaomi by mistake_ After the scp in the gamepad is used, your program will not work properly.（I can only use irungentoo/Xiaomi_ Mi program of gamepad.Then there is a bug in his program. In Elden Ring, the lt key of the handle cannot be used, but it is normal in other games.）So I want to know how to continue using your program.

So I want to know how to continue using your program. I'm sorry to trouble you

It seems that you are having problems after installing SCPToolkit, right? If that's the case, SCP is not compatible with ViGem (that's meant to be its successor and that mi-360 uses), so you may try with a full uninstallation of SCPToolkit

看来您在安装 SCPToolkit 后遇到了问题，对吧？如果是这种情况，SCP与ViGem不兼容（这是它的继任者，而mi-360使用），所以你可以尝试完全卸载SCPToolkit。

thanks bro
solved the problem with your method


Reaaaaaaaact: Add barebones new frontend in React

See RFC at #199
This adds everything we need to get started recreating the frontend in
react / typescript / bootstrap / styled-components:

New webpack config
Easy offline dev with webpack-dev-server
React-bootstrap
React-typescript
Component to connect to socket.io and get the pulls
Available on the main app at /frontend
TS Types for the Pull object and its components
styled-components all ready and set up
Keeps the old UI around and functional

👀 👀 👀 👀 👀 👀 👀 👀 👀 👀 👀 👀

What do you think about using Preact here to kick the tires with it? It should be a drop-in replacement, and save >100KB on load.

Not really sure how to setup Pulldasher locally or how to run the frontend:start script on Cominor so at the moment any suggestions on how to properly setup the /frontend side of Pulldasher will be awesome.
To note, it also seems that process.env.BACKEND_URL returns undefined in the file frontend/src/page-context.ts. Since in the console it logs a 404 error when fetching for the page context using the BACKEND_URL.

Steps Taken:

Git cloned Pulldasher to my Cominor Home directory.
Ran update-pulldasher
Went to https://pulldasher.cominor.com/frontend/
Only saw the titles for all the Divs:

CR 🎈

CR 🎈

@danielbeardsley, should this be dev_blocked (more commits definintely incoming) or if it's ready for more QA/CR?

@danielbeardsley, should this be dev_blocked (more commits definintely incoming) or if it's ready for more QA/CR?

And any input on @ardelato's comment would be appreciated https://github.com/iFixit/pulldasher/pull/200#issuecomment-752793567

And any input on @ardelato's comment would be appreciated https://github.com/iFixit/pulldasher/pull/200#issuecomment-752793567

Looks nice!
CR :+1: through 733c455

Looks nice!
CR :+1: through 733c455

Sorry @ardelato ! forgot I didn't reply yet here.
Pulldasher can now be run locally (well this UI anyway).
Clone the repo locally then:

npm install
npm run frontend:start (should open a browser with an in-progress version of pulldasher like below)

Sorry @ardelato ! forgot I didn't reply yet here.
Pulldasher can now be run locally (well this UI anyway).
Clone the repo locally then:

npm install
npm run frontend:start (should open a browser with an in-progress version of pulldasher like below)

CR :+1: through b06c00b

CR :+1: through b06c00b

deploy_block❓ Confirmation on current working behavior.  Should pulls-whether fake or from GitHub-appear on the React Based Pulldasher?
@danielbeardsley I was able to get Pulldasher working locally, but it seems that the pulls from the dummy-pulls.json are not being populated on the frontend.
In addition, there still seems to be a socket WebSocket issue on Cominor.
Steps Taken:
Local Pulldasher:

Ran git clone git@github.com:iFixit/pulldasher.git on my local computer
cd into the directory and ran git checkout reaaaaaaaaaaact
Ran npm install and it successfully completed the postinstall script:  "webpack && npm run frontend:build"
Ran npm run frontend:start and was able to load Pulldasher on localhost
Did not see any pulls from the dummy-pulls.json file

Cominor Pulldasher:

Ran git clone git@github.com:iFixit/pulldasher.git in my cominor home directory
cd into the directory and ran git checkout reaaaaaaaaaaact
Ran update-pulldasher --run
Went to https://pulldasher.cominor.com/frontend/
Noticed that pulls were not being populated either

deploy_block❓ Confirmation on current working behavior.  Should pulls-whether fake or from GitHub-appear on the React Based Pulldasher?
@danielbeardsley I was able to get Pulldasher working locally, but it seems that the pulls from the dummy-pulls.json are not being populated on the frontend.
In addition, there still seems to be a socket WebSocket issue on Cominor.
Steps Taken:
Local Pulldasher:

Ran git clone git@github.com:iFixit/pulldasher.git on my local computer
cd into the directory and ran git checkout reaaaaaaaaaaact
Ran npm install and it successfully completed the postinstall script:  "webpack && npm run frontend:build"
Ran npm run frontend:start and was able to load Pulldasher on localhost
Did not see any pulls from the dummy-pulls.json file

Cominor Pulldasher:

Ran git clone git@github.com:iFixit/pulldasher.git in my cominor home directory
cd into the directory and ran git checkout reaaaaaaaaaaact
Ran update-pulldasher --run
Went to https://pulldasher.cominor.com/frontend/
Noticed that pulls were not being populated either

dev_block 🐛
Actually there seems to be a bug that @BaseInfinity stumbled upon.
It seems that the Hovering feature no longer works on the main page.
Pulldasher on master
https://user-images.githubusercontent.com/22064420/103838218-5a1edb00-5041-11eb-93d9-c03d8e3af5c1.mp4
Pulldasher updated with this branch
https://user-images.githubusercontent.com/22064420/103838226-6014bc00-5041-11eb-9e8a-69061f184427.mp4

dev_block 🐛
Actually there seems to be a bug that @BaseInfinity stumbled upon.
It seems that the Hovering feature no longer works on the main page.
Pulldasher on master
https://user-images.githubusercontent.com/22064420/103838218-5a1edb00-5041-11eb-93d9-c03d8e3af5c1.mp4
Pulldasher updated with this branch
https://user-images.githubusercontent.com/22064420/103838226-6014bc00-5041-11eb-9e8a-69061f184427.mp4

Confirmation on current working behavior. Should pulls-whether fake or from GitHub-appear on the React Based Pulldasher?

The behavior you witness is the current state of the project (pulls do not appear year). It's a WIP for sure.
dev_block 👍  though on the tooltip bug. We've experienced this when trying to upgrade jQuery. See https://github.com/iFixit/pulldasher/issues/174

Confirmation on current working behavior. Should pulls-whether fake or from GitHub-appear on the React Based Pulldasher?

The behavior you witness is the current state of the project (pulls do not appear year). It's a WIP for sure.
dev_block 👍  though on the tooltip bug. We've experienced this when trying to upgrade jQuery. See https://github.com/iFixit/pulldasher/issues/174

I fixed the problem with the tooltips!
See recent commits!

un_dev_block :+1:

QA :+1:
Tooltips on the old UI still work.
The new UI is a WIP and this is a fair chunk of the way there. (At least the hard bootstrapping part is done).

I'm gonna re-mark this as QA :+1:


feat(pie-modal): Add close button

[Added] - close button to modal

more of a question: when we close the modal, is it expected that the dialog element stays in the DOM?

more of a question: when we close the modal, is it expected that the dialog element stays in the DOM?

Yeah the default behaviour of the dialog element toggles between display none and display block.


Increments progress bar after every 1000 records received from Tandem pump

Does what it says on the tin.

 [tandem] progress bar

This seemed to increment within a quite small range - like it jumped to 47% then increment slowly to 48%, then 49%, then bigger jumps when done fetching. So we might still want to fiddle later, but better than it was! Merging into basal-simulator now to try to reduce branches.


Fix work with transit dependencies in setup.py

[x] fix install_requires

    install_requires=[
        'logger<EMAIL_ADDRESS>    ]

[x] remove dependency_links

[x] fix requirements.txt

logger<EMAIL_ADDRESS>

[x] update topics in classifiers

    classifiers=[
        "License :: OSI Approved :: MIT License",
        "Programming Language :: Python :: 3.10",
        "Operating System :: OS Independent",
        "Intended Audience :: Developers",
        "Topic :: Software Development"
    ]

done
https://github.com/obervinov/vault-package/pull/27/commits/f4cd2392b88d650471d3ad80af57d6942e3a38b9


Feature Request: Sound meter / db meter

I was about to build a sound sensor, measuring the sound level in my kitchen.
The reason I want this is to adjust the volume of my google home speaker depending of the surrouding level of noise.
Would this be possible to do with Wall Panel?

I suppose it is possible, but would it be developed, that is kind of another question.  I do not think this would be a high priority feature request.

I think this could be useful to detect smoke alarms.  Ive been looking for a way to make home assistant listen to smoke alarm, the way alexa does, and having the wallpanel detect the sound and sending an MQTT message would solve this perfectly.


Remove "field" as suffix from class names ?

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
Many classes in code base are named with the suffix "Field" (LaxObjectField, MboxSha1SumActorField(BaseActorField), etc.). I would tend to not include this suffix for two reasons:

as it does not describe the object itself
I find it a bit confusing, and makes the code harder to read

Describe the solution you'd like
I suggest removing these suffixes if possible.

Thanks @Leobouloc for your feedback.
This issue will be fixed with the PR #294


Same bank label for two ram strips

Describe the bug
The si.memLayout method returns two RAM strips, both with identical bankLabels. The raw request yielded the same information.
Current Output

Expected behavior
This ram strips have 2 and 4 slots.

Environment (please complete the following information):

systeminformation package version: 5.22.0
OS: Windows 11

I modified command and added field tag. It's match with physical slots.
 Get-CimInstance Win32_PhysicalMemory | select DataWidth,TotalWidth,Capacity,BankLabel,MemoryType,SMBIOSMemoryType,ConfiguredClockSpeed,FormFactor,Manufacturer,PartNumber,SerialNumber,ConfiguredVoltage,MinVoltage,MaxVoltage,Tag | fl

@Aspedm for now I added tag number to the bank to avoid adding an extra property (for windows only). Version 5.22.3 just released.


Cherry-pick changes for v0.11.0-rc2 into release-v0.11.x branch

Changes
This PR includes all the changes in the needs-cherry-pick list that are relevant to v0.11.0-rc2.
The list of merged commits that I'm working from are here: https://github.com/tektoncd/pipeline/pulls?q=is%3Apr+is%3Aclosed+label%3Aneeds-cherry-pick
I've also brought over a few other commits which modified docs that we were in turn dependent upon.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

/hold

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

The commits in this branch were cherry-picked from master and have already passed through the CLA process once during Pull Request review before they were merged.

/approve
/lgtm

/hold cancel


Clicking the back button in Contribution Tab after uploading an image, return again to Contribution Tab

Summary:
Clicking the back button in Contribution Fragment after uploading an image, return again to Contribution Fragment
Steps to reproduce:
How can we reproduce the issue?

Open commons
click on the + button and upload an image
Then you will return to Contribution Tab. Now
press the back button. You will again return to Contribution Tab

What did you expect the app to do, and what did you see instead?
Clicking the back button in Contribution Tab after uploading an image, should exit the app.
Device and Android version:
Pixel 3 API 30
Commons app version:
lastest prodDebug master
Would you like to work on the issue?
YES

Thanks a lot Ayan for reporting the issue and being willing to work on it!


add cargo.toml file as editable

Kindly add cargo.toml in the src folder, So packages can be get easily, Which would make things easier for getting stuff thats custom and not supported here. e.g: access_control.

Unfortunately this is currently not possible due to compile times. We need to use pre-compiled crates for speed and convenience. We are aiming to cover the most useful crates for Solana programming so you can request a crate if you couldn't find what you are looking for. access_control will be supported in the next update.


SpanBuilderR.asChildOf() will throw exception if Span/SpanContext is null

It is perfectly valid to, for instance, attempt to extract span context from an incoming request and have that extract() return null, and then expect to create a "child" span of this context (resulting in a root span in the case of no extracted context).
Further, it is valid to pass a null span/context to asChildOf() in at least the Jaeger Java implementation of OpenTracing and probably others, but you wouldn't be able to fulfill that use case without an exception.
Point is, checking for null is cool. :)
An example is available in another contrib project, web-servlet-filter (lines 174-180):
https://github.com/opentracing-contrib/java-web-servlet-filter/blob/master/opentracing-web-servlet-filter/src/main/java/io/opentracing/contrib/web/servlet/filter/TracingFilter.java

Fixed in #16

Changes merged to master.


Code example problem

Does the code in the second block check the opposite condition that it should?
IE
NameLength > InputBufferLength - FIELD_OFFSET
instead of
InputBufferLength - FIELD_OFFSET > NameLength

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

ID: 3c9648c0-e775-0338-4150-3c1e7329abe8
Version Independent ID: d955e6cd-d2c3-cabf-4ec3-b17a693ce473
Content: Failure to Validate Variable-Length Buffers - Windows drivers
Content Source: windows-driver-docs-pr/kernel/failure-to-validate-variable-length-buffers.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Absolutely right @alextechcc! Thanks for the sharp eye, and apologies for the delay making the fix. I corrected the direction of the greater than operator, and added some parenthetical remarks around the example to help make it less of a brain twister, hopefully. The update should appear on the live site within a few minutes.


this method from which paper

this method from which paper,thanks

The paper will be released later.


Fix checking workspaceFolders in rHelpProviderOptions

What problem did you solve?
Fix checking workspaceFolders in the initialization of the help panel.
(If you do not have screenshot) How can I check this pull request?

No open workspace folders.
Open or create an R file.
vscode-R should be activated properly.

@Ikuyadeu We might need to quick release for this as the extension activation for no open workspace folder is broken.

Would it make sense to also check the length of the array?
         cwd: ((workspace.workspaceFolders !== undefined && workspace.workspaceFolders.length > 0) ? workspace.workspaceFolders[0].uri.fsPath : undefined)

The documentation seems to imply that an array of length 0 won't occurr (List of workspace folders or undefined when no folder is open.), but better be safe than sorry

@ManuelHentschel Agreed.
Note that both vscode-R and languageserver support multi-root workspace. We might revisit the code where workspaceFolders[0] is used and see how we could properly handle multi-root workspace in these cases.


Enable Bgra8 on OpenGL backend

Replacing the TODO parts related to B8_G8_R8_A8, since using Bgra8 format type with OpenGL backend causes runtime panics, because of these unhandled cases.

This change is

@homu r+

:pushpin: Commit 6e199b3 has been approved by kvark

:hourglass: Testing commit 6e199b3 with merge 38a6aa3...

:sunny: Test successful - status


[zh]Translate contribute/participating.md

Translate content/en/docs/contribute/participating.md to Chinese.
Refer : https://github.com/k8smeetup/website-tasks/issues/2103

/assign @markthink


One-line, Multi-line label flag

For your discussion after Bart is back home: would you be so kind to consider a flag to allow multiple-lines labels. A flag could be avoided using text wrap as it was functioning in the previous release.

Link to the discussion in forum
https://discourse.nodered.org/t/text-wrap-in-node-red-contrib-ui-multistate-switch/42242


Bug:  Instagram won't appear in the share dialog for my app, but will appear for other apps with the same file

Capacitor version:

Run npx cap doctor:
Latest Dependencies:
@capacitor/cli: 4.7.1
@capacitor/core: 4.7.1
@capacitor/android: 4.7.1
@capacitor/ios: 4.7.1
Installed Dependencies:
@capacitor/cli: 4.7.0
@capacitor/core: 4.7.0
@capacitor/android: 4.7.0
@capacitor/ios: 4.7.0
Library version:

4.0.0

Your Plugin Configuration

Affected Platform(s):

Android

Current Behavior

Share dialog won't show  Instagram , but shows WhatsApp , when I try to share the same image from withing email or WhatsApp instagram appears normally
Expected Behavior

Instagram should be displayed on the share dialog
Sample Code or Sample Application Repo

Everything is default
Reproduction Steps

call the share dialog in a phone with instagram installed
  const handleShare = (m) => {
    async function share() {
      if (!url.match(/^[a-zA-Z]+:\/\//)) {
        url = "http://" + url;
      }
      let nurl = url.replace(/^(https?:\/\/)?(www\.)?/i, "$1www.") ;

      await fetch(nurl)
        .then(res => {
          return res.blob()
        })
        .then(blob => {
          return blob.arrayBuffer();
        })
        .then(ab => {
          const uint8Array = new Uint8Array(ab);

          var chunkSize = 1024;
          var str = '';
          for (var i = 0; i < uint8Array.length; i += chunkSize) {
            var chunk = uint8Array.subarray(i, i + chunkSize);
            str += String.fromCharCode.apply(null, chunk);
          }
          var base64String = btoa(str);

          FileSharer.share({
            filename: name,
            contentType: "application/png",
            base64Data: base64String,
          }).catch(error => {
            //alert("File sharing failed" + error.message);
            console.error("File sharing failed", error.message);
          });
        })
        .catch((error) => {
          alert(error);
        });

    }
    share();
  }

I was using the wrong content type
             contentType: "application/png",
the correct mimetype would be image\png


VSCode remote cpp debug not working

System Version
Version: 1.35.0-insider (user setup)
Commit: a5536b8f5a16a10d859f3dec1e59701671bf069e
Date: 2019-05-20T05:17:36.269Z
Electron: 3.1.8
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 6.1.7601
Steps to Reproduce:

Download lastest vscode insider with remote extension package and C/C++ extension.
Connect to a Arch server with remote ssh.
git clone https://github.com/microsoft/vscode-remote-try-cpp.git
Set a breakpoint at line 7.
Press F5
The debug hangs, binary is compiled, but the gdb is not started. A little blue box is running on left top conner.

Other details(Logs) are listed here. I reported to both repo because I am not sure which one is the correct one.
All help needed, Thx.

I repro the same issue, in fact it is not just over remote, C++ debugging does not seem to work in Insiders at all.  Code builds via tasks.json but the debugger hangs with the little blue progress work crawling relentlessly across the top of the debug panel. The exact same settings work in regular VS Code.
Debugging outside VS Code (i..e gdb in the terminal) is fine.
Debugging in regular VS Code also works fine (tested with v1.34.0).
I enable extension debug output but there doesn't seem to be much there:
  Processing folder (recursive): /usr/lib/llvm-6.0/lib/clang/6.0.1/include/
  Processing folder (recursive): /usr/local/include/
  Processing folder (recursive): /usr/include/
  Processing folder (recursive): /home/steve/projects/hello_linux/
  Discovering files: 5078 file(s) processed
  0 file(s) removed from database
Done discovering files.
Parsing remaining files...
  Parsing: 0 files(s) processed
Done parsing remaining files.
Database safe to open

VS Code version details:
Version: 1.35.0-insider
Commit: a5536b8f5a16a10d859f3dec1e59701671bf069e
Date: 2019-05-20T05:15:35.934Z
Electron: 3.1.8
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Linux x64 4.15.0-48-generic snap

tasks.json:
{
    "tasks": [
        {
            "type": "shell",
            "label": "build",
            "command": "/usr/bin/g++",
            "args": [
                "-g",
                "${workspaceFolder}/main.cpp",
                "-o",
                "${workspaceFolder}/a.out"
            ],
            "options": {
                "cwd": "/usr/bin"
            }
        }
    ],
    "version": "2.0.0"
}

launch.json
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "(gdb) Launch",
            "type": "cppdbg",
            "request": "launch",
            "program": "${workspaceFolder}/a.out",
            "args": [],
            "stopAtEntry": false,
            "cwd": "${workspaceFolder}",
            "environment": [],
            "externalConsole": false,
            "MIMode": "gdb",
            "setupCommands": [
                {
                    "description": "Enable pretty-printing for gdb",
                    "text": "-enable-pretty-printing",
                    "ignoreFailures": true
                }
            ],
            "preLaunchTask": "build"
        }
    ]
}

same here,
tested against a ubuntu 1804 remote and a Centos7.6 reomte
mabybe it works for WSL, but that is not a thing here.

Most likely this is a duplicate of #73964.

Fixed in latest Insiders.

Still experiencing the problem at Version:
Version: 1.35.0-insider (user setup)
Commit: 4ca38ce5584d7cd67b435b3c32ef1240c6a29628
Date: 2019-05-21T05:18:31.618Z
Electron: 3.1.8
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 6.1.7601

Still experiencing the problem at Version:
Version: 1.35.0-insider (user setup)
Commit: 4ca38ce
Date: 2019-05-21T05:18:31.618Z
Electron: 3.1.8
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 6.1.7601

@weinand

@liushiqi9 please create a new issue for your problem with reproducible steps.
The issue hat has been fixed and the original author has confirmed that it works for him now.


feat(vendors): remove vendors, build multi-arch image, move circleci to github actions

Signed-off-by: shubhamchaudhary<EMAIL_ADDRESS>
Removed vendors
Build multi-arch image
move circleci to GitHub actions

build:
Pipiline:
https://github.com/ispeakc0de/scheduler-gitaction-test/actions/runs/1003910957
push:
pipeline:
https://github.com/ispeakc0de/scheduler-gitaction-test/actions/runs/1003959421
image:
release:
pipeline:
https://github.com/ispeakc0de/scheduler-gitaction-test/actions/runs/1003960916
image:


doc/metrics: use secp384r1 curve with SHA384 signature

This aligns with the type of key/signature used by LXD itself.
Signed-off-by: Simon Deziel<EMAIL_ADDRESS>

Documentation preview available at: https://linuxcontainers.org/lxd/docs/pr.9943/


Fix "Invalid buffer id" with highlights after bwipe

after this commit neomake doesn't clean old signs after no errors are reported when running Neomake! clippy. I'm using Vim 8.0 patches 1-197.

@datanoise
Please create a new issue, and test if https://github.com/neomake/neomake/pull/935/commits/3ffccf2ebafeae87c656631f07f836616f467182 causes it.
If you can provide a minimal test case, that would be nice, too.

@datanoise
Also, some echom a:bufnr in https://github.com/neomake/neomake/pull/935/commits/3ffccf2ebafeae87c656631f07f836616f467182#diff-aa5e4e3df2470a8b9dcc0c080cabd219R114 might help to debug it (if it's that commit).


make it rails independent

Hope it will fix #53
It still works within my Rails 4 app, but I haven't tested it yet with Sinatra or Padrino.
So I think it is just an initial version.

I found very simple Sinatra application, remove some stuff and push it.
The application uses the gem from my fork.
It seems like it works fine.
I created db/seeds directory with dummy files.
The gem defines the tasks correctly so we can see them by running rake -T and execute them via rake.

Cool. Id been thinking how to run the tests in different environments so this could be helpful.
a, pre, code, a:link, body { word-wrap: break-word !important; }

Hi James, yes, I will try to complete this PR.
#60 introduces a new littile Rails dependency so rebase would be required.

This is manually merged onto master.

Hi James, thank you!
I am sorry because I didn't have enough time to complete this 😞
But I definitely will test it once again with simple Sinatra app (which I would share).
What do you mean by "manually merged"? I don't see any changes on master branch related to this.
Am I missing something?

Hey @ivanovaleksey thanks for the work, you pushed things forward when I wasn't doing so!
You should see your changes now on master with a few tweaks from the merge. Unfortunately it's late here and Travis is complaining so I'll have to do a little more to fix that, but it's otherwise looking good.

Now I see the commit. Is there something else I can help you with (besides from testing it with Sinatra app)?

If you fancy having a crack at #43 it's easy to re-create but I've not had time to add duplicate checking. I'd like it to bomb out with a polite error when a 2nd seed of the same name is defined. In the meantime I'm going to look at multi environment tests as I should cert against rails 5.

Back on topic, I'll leave this open until we have some tests in place for non rails apps.