DBSubnetGroup manifest created from AdoptedResource missing subnetIDs

Describe the bug
One of our DBSubnetGroup manifests became detached from its AWS resource during a namespace migration. I created an AdoptedResource manifest to regenerate the manifest, but I received the following condition:
'DBSubnetGroup.rds.services.k8s.aws "default-subnet-group" is invalid: spec.subnetIDs: Required value'
This suggests to me that the DBSubnetGroup manifest that the controller is generating is not including all the required fields.
Here is the full AdoptedResource manifest and status I used (with ARN redacted):
apiVersion: services.k8s.aws/v1alpha1
kind: AdoptedResource
metadata:
  name: ar-default-subnet-group
  namespace: rds-operator-irsa
spec:
  aws:
    arn: REDACTED
    nameOrID: db-subnet-group
  kubernetes:
    group: rds.services.k8s.aws
    kind: DBSubnetGroup
    metadata:
      labels:
        app.kubernetes.io/instance: dev-rds-manifests
      name: default-subnet-group
      namespace: rds-operator-irsa
status:
  conditions:
  - message: 'DBSubnetGroup.rds.services.k8s.aws "default-subnet-group" is invalid:
      spec.subnetIDs: Required value'
    status: "False"
    type: ACK.Adopted

Steps to reproduce

Install RDS controller on a K8s cluster that can create RDS resources in an AWS account.
Create a DB Subnet Group in RDS.
Create an AdoptedResource manifest in the K8s cluster referencing the created DB Subnet Group.
Look at the conditions of the new AdoptedResource manifest.

Expected outcome
AdoptedResource reports a "True" Adopted status, and the DBSubnetGroup manifest appears in the cluster.
Environment
I started using v0.0.24 of the RDS controller before upgrading to v0.0.27 and then v0.0.28. All of them had this issue.

Kubernetes version 1.21.8 via Rancher
Using EKS (yes/no), if so version? No
AWS service targeted (S3, RDS, etc.) RDS

Looks like it's the same kind of problem as this issue.

i've tested with all CRD supported in RDS, dbClusterParamGroup & dbSubnetGroup are the only left ones that will have issues when using adopted resources.  dbClusterParamGroup is easy to fix. dbSubnetGroup needs this subnetIDs renamed, but subnetIDs I think should also be a reference to EC2 service subnet group which I'm not sure how to deal with it yet.

The fix is merged, please try with latest code and let us know if it's still broken

Sorry for the delay. I had to figure out how to build the Docker image. I can say that all the fixes work though!

Thanks for verifying it!


Microsoft.ContainerInstance/containerGroups (2018-10-01)

Describe the noise
Resource type
Microsoft.ContainerInstance/containerGroups
apiVersion
2018-10-01
Client (PowerShell, Azure CLI, or API)
PowerShell
Relevant ARM Template code
{
	"name": "containerGroup1",
	"type": "Microsoft.ContainerInstance/containerGroups",
	"apiVersion": "2018-10-01",
	"location": "[resourceGroup().location]",
	"properties": {
		"containers": [{
			"name": "nepeters",
			"properties": {
				"image": "neilpeterson/nepetersv1",
				"ports": [{
					"port": 80
				}],
				"resources": {
					"requests": {
						"cpu": 1,
						"memoryInGB": 2
					}
				}
			}
		}],
		"osType": "Linux",
		"ipAddress": {
			"type": "Public",
			"ports": [{
				"protocol": "TCP",
				"port": 80
			}]
		}
	}
}

Expected response
I expected no noise since the template has not been modified since the resources were deployed.
Current (noisy) response
Resource and property changes are indicated with these symbols:
  - Delete
  ~ Modify

The deployment will update the following scope:

Scope: /subscriptions/3762d87c-ddb8-425f-b2fc-29e5e859edaf/resourceGroups/aci-whatif

  ~ Microsoft.ContainerInstance/containerGroups/containerGroup1 [2018-10-01]
    - properties.ipAddress.ip: "<IP_ADDRESS>"
    ~ properties.containers: [
      ~ 0:

        - properties.instanceView:

            currentState.startTime: "2019-12-02T00:13:57Z"
            currentState.state:     "Running"

        ~ properties.resources.requests.cpu:        1 => 1
        ~ properties.resources.requests.memoryInGB: 2 => 2

      ]

Resource changes: 1 to modify.

Additional context
Seems like these two are not necessarily noise as each property is specified in the template. Regardless, the results are erroneous.
 ~ properties.resources.requests.cpu:        1 => 1
 ~ properties.resources.requests.memoryInGB: 2 => 2

@neilpeterson Thanks for the feedback. This is a very interesting one. We'll look into this.

The erroneous deltas were caused by type mismatch (double VS. integer).

In the template we used integers, but according to the RP's swagger they should be doubles:

@majastrz Looks like we have another edge case to handle...

The fix has been deployed to all public regions. @neilpeterson, can you confirm it's working for you?

Closing the issue. Please reopen if needed.


Some small fixes

Thank you :+1:
1st commit: :heavy_check_mark:

I added the same change in the implementation file
I reworded the commit message for the git 50-72 rule

2nd commit: :heavy_check_mark:
3rd commit: :x:

the SC_VECTOR is internal to the implementation, and does not need to be exposed
there is already sc_usb_devices_destroy_all() which works on any array of sc_usb_device (including "vectors")

Branch pr4371.

Merged into dev.

Hey @rom1v since we haven't tagged the repo with the "hacktoberfest" label, could you please tag this PR with "hacktoberfest-accepted" label? Thanks!

Scrcpy does not participate in hacktoberfest.
In their maintainer page, they say:

Be ready to review pull/merge requests, accepting those that are valid by merging them, leaving an overall approving review, or by adding the “hacktoberfest-accepted” label.

I definitely "left an overall approving review" (since the commits have been merged), it should be sufficient. Let me know if it isn't in the end.


Fix a bug that can crash deflate on some input when using Z_FIXED.

This bug was reported by Danilo Ramos of Eideticom, Inc. It has
lain in wait 13 years before being found! The bug was introduced
in zlib <IP_ADDRESS>, with the addition of the Z_FIXED option. That
option forces the use of fixed Huffman codes. For rare inputs with
a large number of distant matches, the pending buffer into which
the compressed data is written can overwrite the distance symbol
table which it overlays. That results in corrupted output due to
invalid distances, and can result in out-of-bound accesses,
crashing the application.
The fix here combines the distance buffer and literal/length
buffers into a single symbol buffer. Now three bytes of pending
buffer space are opened up for each literal or length/distance
pair consumed, instead of the previous two bytes. This assures
that the pending buffer cannot overwrite the symbol table, since
the maximum fixed code compressed length/distance is 31 bits, and
since there are four bytes of pending space for every three bytes
of symbol space.

Were there any conflicts that needed fixing? Is there anything we should make sure we review/test well?
I have never used Z_FIXED myself, and I am not sure whether we currently have good tests for that.

I ran some testing of this now, and found no corruption problems at any compression level.
It does seem to slow down compression by around 1.22% average on x86_64 across all compression levels except levels 0 and 1, but I do not see any way to avoid that.
Does deflate_quick need a similar fix I wonder?

I think it slows down because in the original code it writes whole contents of the variable in one instruction, but in the new code it splits it in half and shifts... This might need to be reverted/revised...


Can't run bloxstrap at all

While trying to run bloxstrap I got this error I tried installing a different version but got the same thing.
No idea how this happened or how to replicate it

Bloxstrap_20230714T182330Z.log

Do you use RoGold?

Yes I do

That error only happens if you launch a game through one of RoGold's features because it launches Roblox twice. You can just ignore the error as Roblox will launch and work just fine anyway. Though, 2.4.0 was supposed to make it less of a problem, unless you happen to be using an old build. I'll look at it later.

Alright thanks for all the help


added altclick option for select-features example

I added an alt+click option to the select-features example. This also shows how we can listen for 2 or more conditions in the Select Interaction for alt+click or shift+click.
This is my first contribution to OL3, and I hope to make more of these. Do review and let me know of any suggestions.

Looks like a resonable addition to the example, but you need to make Travis happy. :) You can check the code locally, see https://github.com/openlayers/ol3/blob/master/CONTRIBUTING.md.

travis has passed, I have also incorporated @elemoine suggestions on the older diff,

Could you please squash your commits into one commit?

I'll do that and make a fresh pull request, hope that'd be fine?

You could have used push --force to update your branch and this PR.


[integration_test] flutter drive --wasm does not report results even though tests completed

Steps to reproduce
Run web integration tests with flutter drive --wasm
Expected results
Results are reported
Actual results

Results are not reported
The command hangs forever
Results can be seen in the Chrome dev tools. The tests do actually complete.

Code sample
Use any of the official flutter packages that have been migrated to support WASM and have web integration tests
Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
[!] Flutter (Channel [user-branch], 3.24.0-1.0.pre.104, on macOS 14.5 23F79
    darwin-arm64, locale en-US)
    ! Flutter version 3.24.0-1.0.pre.104 on channel [user-branch] at
      /Users/rexios/repos/flutter
      Currently on an unknown channel. Run `flutter channel` to switch to an
      official channel.
      If that doesn't fix the issue, reinstall Flutter by following instructions
      at https://flutter.dev/setup.
    ! Upstream repository git@github.com:Rexios80/flutter.git is not a standard
      remote.
      Set environment variable "FLUTTER_GIT_URL" to
      git@github.com:Rexios80/flutter.git to dismiss this error.
    • Framework revision 7a2290c559 (60 minutes ago), 2024-07-10 15:16:33 -0400
    • Engine revision 1c23c8f640
    • Dart version 3.6.0 (build 3.6.0-22.0.dev)
    • DevTools version 2.37.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/rexios/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio
      Preview.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.11+0-17.0.11b1207.24-11852314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
    • Xcode at /Applications/Xcode-beta.app/Contents/Developer
    • Build 16A5171c
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • 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
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.11+0-17.0.11b1207.24-11852314)

[✓] VS Code (version 1.91.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.92.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile)     • emulator-5554             •
      android-arm64  • Android 14 (API 34) (emulator)
    • Aaron’s iPhone (mobile)         • 00008130-001C35CE3A20001C • ios
      • iOS 17.5.1 21F90
    • macOS (desktop)                 • macos                     • darwin-arm64
      • macOS 14.5 23F79 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin
      • macOS 14.5 23F79 darwin-arm64
    • Chrome (web)                    • chrome                    •
      web-javascript • Google Chrome 126.0.6478.127
    ! Error: Browsing on the local area network for Raj's Apple Vision Pro.
      Ensure the device is unlocked and attached with a cable or associated with
      the same local area network as this Mac. (code -27)
    ! Error: Browsing on the local area network for Joe’s iPhone. Ensure the
      device is unlocked and attached with a cable or associated with the same
      local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code
      -27)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Use any of the official flutter packages that have been migrated to support WASM and have web integration tests

@Rexios80 I tried with video_player and camera and tried running with wasm enabled but it is giving me below error:
Could not find an option named "wasm".
Can you provide an example with which it works ?

I forgot to mention you have to be on master channel

Thanks for the tip. Switched to latest master and then trying to run integration_test from video_player or camera gives me unable to find main_test.dart` file. Please share the plugin you are using to run the scenario.

Welp it looks like maybe none of the integration test projects in the packages repo actually support compiling to WASM on the main branch. I'll create issues for that. If you want to see this issue in action right now checkout this PR and run the following commands:
cd packages/google_maps_flutter/google_maps_flutter_web/example
flutter drive -d chrome --web-port 7357 --wasm --driver test_driver/integration_test.dart --target integration_test/projection_test.dart

To clarify the integration tests actually do compile to WASM, but the index.html files are out of date

I created https://github.com/flutter/flutter/issues/151663 to track this

test driver sometimes saying all tests were successful when they were clearly not by looking at the Chrome dev console

This could be https://github.com/flutter/flutter/issues/129041

checkout this PR and run the following commands:
cd packages/google_maps_flutter/google_maps_flutter_web/example
flutter drive -d chrome --web-port 7357 --wasm --driver test_driver/integration_test.dart --target integration_test/projection_test.dart

Following this, I observed the similar behavior as reported.

This appears to have been caused by issues driving examples that use flutter_bootstrap.js rather than an issue with running tests compiled to WASM. See https://github.com/flutter/packages/pull/7115#issuecomment-2229569641 for details.

Note that there are still some issues with test result reporting, but I believe those are different enough to get their own tickets


“BusinessEventsBundleBatchProcessor “ is not existing from the batch job UI

Hi Experts.
In Platform Upgrade 27, I see that you guys revised the rules on how the business event is being triggered. Currently i created a customization in which after running the business events batch job (this batch job gets data from the BUSINESSEVENTSCOMMITLOG table), an MS Flow Business event will be triggered and sends an email. However, I'm trying to digest on what do you mean by this statement: "Dedicated threads are allocated to process business events by the system, which ensures faster processing of business events." -> So do you mean, automatically? Since batch jobs in PU 27 are now ineffective? So the current situation is not working for us as of the moment, thus prompted to do the workaround from your article (https://docs.microsoft.com/en-us/dynamics365/unified-operations/dev-itpro/business-events/home-page#business-events-processing) However, BusinessEventsBundleBatchProcessor class does not even exist from the batch UI. I would greatly appreciate if you guys can help me trigger the business events. That would be so much helpful

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

ID: 655c157f-5478-4c71-7cd4-1b04383264ee
Version Independent ID: 6555a353-87cd-c919-0c3e-dcfc4ede9909
Content: Business events - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/business-events/home-page.md
Service: dynamics-ax-applications
GitHub Login: @Sunil-Garg
Microsoft Alias: sunilg

@johnhenryboco, thanks for the feedback.
@Sunil-Garg, please review and consider making updates to the topic.
Bug 342562 has been logged to track this issue.

i am on PU28 and i cannot seem to get the event to trigger either. i am setting up Flow integration in our Tier-2 Sandbox environment. am i missing a step?

Enable Business Event
Create an MS Flow.
Trigger the event. (i have tried When Invoice is Created on Sales Order and When an alert rule is triggered)

is there any Batch Job that i need to have running in the background?

Pls check the troubleshooting guide. The last issue described is most likely the reason. Let us know if that does not help.
Get Outlook for iOShttps://aka.ms/o0ukef

From: jonathansulte<EMAIL_ADDRESS>Sent: Sunday, July 21, 2019 8:56:46 PM
To: MicrosoftDocs/dynamics-365-unified-operations-public<EMAIL_ADDRESS>Cc: Sunil Garg<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics-365-unified-operations-public] “BusinessEventsBundleBatchProcessor “ is not existing from the batch job UI (#1456)
i am on PU28 and i cannot seem to get the event to trigger either. i am setting up Flow integration in our Tier-2 Sandbox environment. am i missing a step?

Enable Business Event
Create an MS Flow.
Trigger the event. (i have tried When Invoice is Created on Sales Order and When an alert rule is triggered)

is there any Batch Job that i need to have running in the background?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-unified-operations-public%2Fissues%2F1456%3Femail_source%3Dnotifications%26email_token%3DAHNKDWYYURGS3DM4GCHIDYDQAUVX5A5CNFSM4H6G3XRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2OWG3Q%23issuecomment-513631086&data=02|01|sunilg%40microsoft.com|4838da0f09e648c1e12008d70e589dfa|72f988bf86f141af91ab2d7cd011db47|1|0|636993646082441367&sdata=ZCGniPGo%2FK9zp92lQ5Jcmw9ZCAlao8KjsC4hAHRxonE%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHNKDW5CUZ54S7BRZ5FYOBLQAUVX5ANCNFSM4H6G3XRA&data=02|01|sunilg%40microsoft.com|4838da0f09e648c1e12008d70e589dfa|72f988bf86f141af91ab2d7cd011db47|1|0|636993646082451356&sdata=VE6gFRvOFx2r9rfViiKOR7KahWgwagQpYsnr9lfllmM%3D&reserved=0.

thanks @Sunil-Garg . are you able to post the link to the specific one you are refering to? thanks in advance

Here is the link https://docs.microsoft.com/en-us/dynamics365/unified-operations/dev-itpro/business-events/troubleshooting
Specifically, pls use the following fix to make this work. If this does not help, pls email me at<EMAIL_ADDRESS>and we can take a look.
One of the reasons why this issue can occur is if certain parameters are not set as expected in the BusinessEventsParameters table. This is due to an update in which some of the business events parameters not being set correctly.
In a non-production environment, you must update the BusinessEventsParameters table so that Enabled =1; ProcessorThreads = 1; EndPointRetryCount = 3; BundleSize = 50; EndPointsPerEvent = 10. After this update, restart the batch service and run IISReset to pick up the latest values.
For production environments, log a support request with Microsoft to get these updated.

@Sunil-Garg
The sentence "This behavior is controlled using a parameter in the business events parameter table, which is turned off by default to indicate that manual batch job will not be effective by default" can be updated clarifying that the name of the parameter is "BATCHENABLED" and it must be set to 0 for the dedicated job to work. Cross reference to the troubleshooting page can be added as well. This seems to be a common issue when Business events are not getting triggered at all because the BATCHENABLED is set to 1

Thanks. The wiki has been updated per your suggestion.


Toggle field not remembering current state

In my form.yaml file, I have the following field:
.validate.required:
    type: toggle
    highlight: true
    default: true
    options:
        true: YES
        false: NO
    validate:
        type: bool
    label: Required

and this correctly shows up in the admin panel and works with my form.md header. However, the selected option in the admin panel is overridden by the default value every time I refresh. I've tried removing default: true with no success.
How do I get the value in the admin panel to show the current state?

This might be due to your field name .validate.required... Can you try another name such as validate_required ?


Fix: TOC navigation in pages theme

Fixes enketo/enketo-express#568. This was a regression introduced by an overzealous find/replace in #969.

have also begun to fail after apparent changes in Chromium's rasterization

Ugh!


python3Packages.pipx: 1.0.0 -> 1.1.0

Description of changes
https://github.com/pypa/pipx/releases/tag/1.1.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This PR seems to conflict with master:
$ git merge --no-commit --no-ff 3935bbd1fc37894c6743598efada98417f818e19
Auto-merging pkgs/development/python-modules/pipx/default.nix
CONFLICT (content): Merge conflict in pkgs/development/python-modules/pipx/default.nix
Automatic merge failed; fix conflicts and then commit the result.

I think #184816 has updated it, so I am going to close this PR.


MODCON-93. Improve transaction behavior when migrating existing users to consortia

Purpose
https://issues.folio.org/browse/MODCON-93
Overview:
We faced an issue on Rancher when some modules created sample users without last name populated and these couple broken users rolled back the main outer transaction to affiliate all existing users, so as a result primary affiliation was not created for all users even correct.
This behaviour started to apprear after we made some changes in transactional model to levarage PostgreSql transactional advisory locks.
And this advisory lock was added to eliminate the possibility to add users from multiple  tenants simultaneously and it requires transaction, so outer transaction when we started migration all existing users were used.
Implementation approach:
Use transaction propagation REQUIRES_NEW to create a new transaction to process each individual user, in this case if some corrupted user will not be created - only a particular transaction for that user will be reverted, but not outer transaction.
Bonus

Code is refactored to extract separate service that allows to avoid duplication of creating primary affiliations in 2 places
After adding logic to pocess user domain events by system user - creating primary affiliation for that event processing also should use system user context

Karate tests passed:


Single endpoint with union of devices and virtual-machines?

How hard would it be to make a single endpoint which exposes both devices and virtual-machines?
Reason: AFAICS, currently every Prometheus scrape job needs to be duplicated, since prometheus http-sd can only scrape a single URL.  If you've got a lot of rewriting rules, that can be painful.
A union endpoint would solve this, and __meta_netbox_model can be used to distinguish between the objects if necessary.
However, I can see there is much Django ORM queryset and serializer magic going on, so it may not be easy. Maybe this is relevant.

At the moment this plugin is build as a simple new view to the existing netbox api, but i understand the issue.
My solution is the described duplicate scrape job.
If you have time for this feel free to start an PR. I would love to have the feature my self.

I realised you should be able to put multiple http_sd_configs under the same job:
- job_name: netbox
  http_sd_configs:
    - url: https://netbox.example.com/api/plugins/prometheus-sd/devices/
      refresh_interval: 10m
      authorization:
        type: Token
        credentials:<PHONE_NUMBER>abcdef0123456789abcdef01234567
    - url: https://netbox.example.com/api/plugins/prometheus-sd/virtual-machines/
      refresh_interval: 10m
      authorization:
        type: Token
        credentials:<PHONE_NUMBER>abcdef0123456789abcdef01234567

So whilst there is duplication of the credentials, there's little other reason for a combined endpoint.


ipfs-migrator: 6 -> 1.4.0

Motivation for this change
New version of the ipfs repo migration toos. This is required to migrate existing ipfs repos to repo v7.
Also switch to upstream versioning scheme (our v 6 was officially 1.3.0).
Needs backport to 18.09 since existing ipfs repos from 18.03 will have to be migrated to v7 in 18.09.
Things done

[x] built in a sandbox on NixOS

@GrahamcOfBorg build ipfs-migrator

Failure on x86_64-darwin (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/anacrolix/jitter
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/jbenet/go-multiaddr
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/anacrolix/utp
github.com/ipfs/fs-repo-migrations/ipfs-2-to-3/Godeps/_workspace/src/github.com/whyrusleeping/go-logging
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/jbenet/go-multiaddr-net/utp
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/jbenet/go-multiaddr-net
github.com/ipfs/fs-repo-migrations/sharness/dependencies/pollEndpoint
github.com/ipfs/fs-repo-migrations/stump
builder for '/nix/store/3v433gb5pqhslnn7pp9w1w8bdkkkncg1-ipfs-migrator-1.4.0.drv' failed with exit code 3
error: build of '/nix/store/3v433gb5pqhslnn7pp9w1w8bdkkkncg1-ipfs-migrator-1.4.0.drv' failed

Success on aarch64-linux (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/k1qjrp714ahnq1zs2p8byngp2dgm2xbm-ipfs-migrator-1.4.0-bin/bin
patching script interpreter paths in /nix/store/k1qjrp714ahnq1zs2p8byngp2dgm2xbm-ipfs-migrator-1.4.0-bin
checking for references to /build in /nix/store/k1qjrp714ahnq1zs2p8byngp2dgm2xbm-ipfs-migrator-1.4.0-bin...
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
/nix/store/k1qjrp714ahnq1zs2p8byngp2dgm2xbm-ipfs-migrator-1.4.0-bin

Success on x86_64-linux (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/fr1wks0qh0811pvi3as5a7zg4yvcn0jw-ipfs-migrator-1.4.0-bin/bin
patching script interpreter paths in /nix/store/fr1wks0qh0811pvi3as5a7zg4yvcn0jw-ipfs-migrator-1.4.0-bin
checking for references to /build in /nix/store/fr1wks0qh0811pvi3as5a7zg4yvcn0jw-ipfs-migrator-1.4.0-bin...
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
/nix/store/fr1wks0qh0811pvi3as5a7zg4yvcn0jw-ipfs-migrator-1.4.0-bin

Success on aarch64-linux (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
/nix/store/k1qjrp714ahnq1zs2p8byngp2dgm2xbm-ipfs-migrator-1.4.0-bin

Success on x86_64-linux (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
/nix/store/fr1wks0qh0811pvi3as5a7zg4yvcn0jw-ipfs-migrator-1.4.0-bin

Failure on x86_64-darwin (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
github.com/ipfs/fs-repo-migrations/ipfs-2-to-3/Godeps/_workspace/src/github.com/whyrusleeping/go-logging
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/anacrolix/utp
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/jbenet/go-multihash
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/jbenet/go-multiaddr
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/jbenet/go-multiaddr-net/utp
github.com/ipfs/fs-repo-migrations/Godeps/_workspace/src/github.com/jbenet/go-multiaddr-net
github.com/ipfs/fs-repo-migrations/sharness/dependencies/pollEndpoint
github.com/ipfs/fs-repo-migrations/stump
builder for '/nix/store/3v433gb5pqhslnn7pp9w1w8bdkkkncg1-ipfs-migrator-1.4.0.drv' failed with exit code 3
error: build of '/nix/store/3v433gb5pqhslnn7pp9w1w8bdkkkncg1-ipfs-migrator-1.4.0.drv' failed

Success on x86_64-linux (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/j8pkyd66wabxcppnr47ha55vrb78mhz1-ipfs-migrator-7-bin/bin
patching script interpreter paths in /nix/store/j8pkyd66wabxcppnr47ha55vrb78mhz1-ipfs-migrator-7-bin
checking for references to /build in /nix/store/j8pkyd66wabxcppnr47ha55vrb78mhz1-ipfs-migrator-7-bin...
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
/nix/store/j8pkyd66wabxcppnr47ha55vrb78mhz1-ipfs-migrator-7-bin

Success on aarch64-linux (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/blpgpms02rifmjmyjwv4vqk5hi4rb8cx-ipfs-migrator-7-bin/bin
patching script interpreter paths in /nix/store/blpgpms02rifmjmyjwv4vqk5hi4rb8cx-ipfs-migrator-7-bin
checking for references to /build in /nix/store/blpgpms02rifmjmyjwv4vqk5hi4rb8cx-ipfs-migrator-7-bin...
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
/nix/store/blpgpms02rifmjmyjwv4vqk5hi4rb8cx-ipfs-migrator-7-bin

Success on x86_64-darwin (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
find: '/nix/store/4idqmkxfmfwgq0jc6mw17sjmvyyqmzx0-ipfs-migrator-7': No such file or directory
/nix/store/9bnnz5csj1gdajrmlvsb0c51fmigwjpc-ipfs-migrator-7-bin

Success on x86_64-linux (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/yrn922crbnvk0kgkm478wfcm2vpsx8my-ipfs-migrator-7-bin/bin
patching script interpreter paths in /nix/store/yrn922crbnvk0kgkm478wfcm2vpsx8my-ipfs-migrator-7-bin
checking for references to /build in /nix/store/yrn922crbnvk0kgkm478wfcm2vpsx8my-ipfs-migrator-7-bin...
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
/nix/store/yrn922crbnvk0kgkm478wfcm2vpsx8my-ipfs-migrator-7-bin

Success on aarch64-linux (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/40rmhfplg71jhrjw1frh1s0gq571agzn-ipfs-migrator-7-bin/bin
patching script interpreter paths in /nix/store/40rmhfplg71jhrjw1frh1s0gq571agzn-ipfs-migrator-7-bin
checking for references to /build in /nix/store/40rmhfplg71jhrjw1frh1s0gq571agzn-ipfs-migrator-7-bin...
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
/nix/store/40rmhfplg71jhrjw1frh1s0gq571agzn-ipfs-migrator-7-bin

Success on x86_64-darwin (full log)
Attempted: ipfs-migrator
Partial log (click to expand)
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
find: '/nix/store/wdz9z3f9h22z90dxszdwmkkwcjq7bzxk-ipfs-migrator-7': No such file or directory
/nix/store/8b7an4izkr6bzhf03bm30cn9cij14gjp-ipfs-migrator-7-bin

cc @elitak @fpletz

backported: 95b11f3cb9d..347723980a1


Crash on Launch

Describe the bug
Crash on launch
To Reproduce
Install Dynamic Trees
Install other mods
launch
Dynamic Trees Beta 2
NeoForge 47.1.78
crash-2024-01-30_10.38.14-client.txt

crash-2024-03-27_18.47.30-client.txt
and me

watut-forge-1.19.2-1.0.14.jar is issue for @FF101UALviv
mixininheaven-mc1.17.1-1.20-v0.0.1-hotfix.jar is issue for @lanwoodall423

crash unrelated to dynamic trees


Issue/6994:Have an option for the server to wait for the database to be up (#7217)

Pull request opened by the merge tool on behalf of #7217

Merged into branches iso7 in fa3fd595d5c179d0d8a7ae0a616a271d53916b04


[BugFix] Avoid reporting negative values for server latency.

While working on testing changes in #8808, noticed that we report negative
values for ResponseDelayMs in high QPS/low latency usecases. This can happen if the
response from the server is received before the query is marked as submitted. This
typically happens if request is sent to server but there's a delay in calling the
callback in _channel.writeAndFlush(cb).

Codecov Report

Merging #8892 (b9188de) into master (c50b3c4) will decrease coverage by 6.60%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #8892      +/-   ##
============================================
- Coverage     69.64%   63.04%   -6.61%
- Complexity     4611     4837     +226
============================================
  Files          1730     1761      +31
  Lines         90314    92176    +1862
  Branches      13421    13821     +400
============================================
- Hits          62900    58108    -4792
- Misses        23054    29897    +6843
+ Partials       4360     4171     -189

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
66.40% <0.00%> (+0.22%)
:arrow_up:

unittests2
14.87% <0.00%> (+0.61%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Fail to pick up file updates in boot2docker

Hi guys,
http://stackoverflow.com/questions/32821555/not-getting-real-file-update-in-vm/32826034#comment53548751_32821555
Would so nice if I can get it to work! If you care to help:
I'm a Windows user. And I've been using docker for a while to get away from having to install another OS.
Boot2docker works really well on Windows, and I've recently switched to docker machines, which seems to simply wrap boot2docker if you do local dev with virtualbox, also working pretty nicely.
Some issue I discovered about boot2docker, or the Linux it uses: it often fails to pick up file updates from the shared folder. It wasn't a problem before because the base images I used to build my own images don't have problems although they run in a problematic docker host (boot2docker vm). Official python / ruby / rails images all seem to able to pick up file updates fine. Recently I wanted to build an image that suits my particular use case out of a minimal base image, and I discovered this project. It's great to have such a tiny base image! I got it to work after some trial and error fun. Then, I discovered the problem I described above... didn't even realize there was such issue in boot2docker before I used alpine as my base image.
Anyway, thank you very much, if you read through all this. Any help will be greatly appreciated! Thanks!

@princemaple Take a look at gliderlabs/pagebuilder#2. It has some information about what causes this problem and a temporary fix.

@mattaitchison yep the command works. Looks like it's the same issue. Thank you very much!

@mattaitchison this makes me wonder why other base images are not affected.. I'm a newbie in this area. Do you happen to know why? I'm keen to get an easy / generic fix for my own dev env.

Most likely because alpine uses musl.

Might be a problem with the shared folder in underlying boot2docker image, please check the above referenced issue.

thanks @rdsubhas !

The problem is gone with the latest release of docker toolbox (docker machine, boot2docker).
It's boot2docker issue, not really alpine's fault.

:+1: @princemaple thanks!


Allow modification of source code from inside of Enso

There are two important abilities of language server which require access deep into the engine internals that we'd like to expose to (privileged) Enso programs:

control execution & deliver visualizations (addressed by #7833)
text editing operations

This issue proposes to create a builtin to expose access to module Source and allow its modifications from inside of Enso (enabled by a CLI option). By having such builtin, the IDE could submit execution requests that modify the source code. Or modify the sourcecode directly from the y-websocket server.

On hold until the yjs server is running withing GraalJS


nacos does not support springboot-2.2.0-release?

我们鼓励使用英文，如果不能直接使用，可以使用翻译软件，您仍旧可以保留中文原文。
We recommend using English. If you are non-native English speaker, you can use the translation software.
Which Component
Nacos Config
Describe what problem you have encountered
see here https://github.com/alibaba/nacos/issues/1982
Describe what information you have read
eg. I have read the reference doc of Sentinel

Now spring cloud alibaba is 2.1.0.release support  spring boot 2.1.x.
Now spring cloud alibaba is 2.2.0.release support spring boot 2.2.x
but 2.2.0.release Has not been released

when will spring cloud alibaba 2.2.0.release be released?

when will spring cloud alibaba 2.2.0.release be released?

duplicate with https://github.com/alibaba/spring-cloud-alibaba/issues/1153


error throw inside promise.then or promise.catch function, can not get the right context

Namespace remains active if error is thrown in catch of Promise
any idea why?

Any update on this?

Use Promise.reject(error) instead of throw, or clear expired context by using setInterval  as a workaround solution.
setInterval(()=>{
    let names = Object.keys(process.namespaces || {});
    for(let n of names) {
        let namespace = process.namespaces[n];
        if (expired(namespace )){
            delete process.namespaces[n];
        }
    }
}, 60 * 1000);
function expired(namespace){
    //check if context is expired, usually by adding time stamp
}

I think this is by design.  Any call back that runs as a result of a rejected promise is conceptually a continuation from the point the error was thrown.
If you want to stop the propagation of contexts into an error handler, you'll need to handle that explicitly

@hayes @carlisliu thanks for the response I really appreciate it. I think I did a bad job describing the issue correctly. Let's have a look at the following code:
const cls = require("continuation-local-storage");

const ns = cls.createNamespace("namespace1");

ns.run(() => {
    ns.set("name", "Amazing Name");

    setTimeout(() => {
        Promise.resolve()
            .then(() => {
                for (var i = 0; i < 10; i++) {
                    if (i === 8) {
                        console.log(ns.get("name"));
                        throw new Error("this is an error");
                    }
                }
            })
            .catch(err => {
                console.log("catching the errors");
            });
    });
}, 1000);

setTimeout(() => {
    console.log(ns.get("name"));
}, 2000)

I would be expecting the following output in the console:
Amazing Name
catching the errors
undefined

However what I get back is following:
Amazing Name
catching the errors
Amazing Name

@carlisliu going with your suggestion, if I do use Promise.reject instead of throw it does seem to be working fine, however my concern is what about the unhandled exceptions? I mean in cases where piece of code inside then block throws an exception which I am not wrapping in try/catch? Doesn't that defeat the whole point of having just one catch block at the end of the promise chain?
Regarding the setTimeout to clear the context does not really help much in my case since I am using cls for instrumentation in a server application, in case of multiple parallel requests, it becomes unpredictable.
Please let me know your thoughts, I do not want to follow anti patters and refactor a huge code base with try/catch floating around everywhere

I can try this after work, but I have a theory.  I can test it out when I get off work tonight, but the 2 cases I am currious about are removing the catch and/or replacing it with a then (and not throwing) to confirm it's related to the throwing of an error. The other is putting the ns.run into a timeout of 0.

Hi all !
Any update on this ?
On my project I get back to v0.6.4 but I guess I'm missing some important fixes doing this.

If I understand the commit correctly, I think it can be done in other ways external to this library. Therefore, I would probably simply revert the change.
For example:
const cls = require('continuation-local-storage')
const ns = cls.createNamespace('test')

ns.run(() => {
  let promise

  ns.set('foo', 'bar')

  ns.run(() => {
    ns.set('foo', 'baz')

    promise = new Promise((resolve, reject) => {
      resolve()
    })
  })

  promise
    .then(() => {
      console.log('Value in resolution context:', ns.get('foo'))
    })

  bind(promise)
    .then(() => {
      console.log('Value in then context:', ns.get('foo'))
    })
})

function bind (promise) {
  const deferred = defer()

  return promise
    .then(function () {
      deferred.resolve.apply(deferred, arguments)
      return deferred.promise
    })
    .catch(function () {
      deferred.reject.apply(deferred, arguments)
      return deferred.promise
    })
}

function defer () {
  const deferred = {}

  deferred.promise = new Promise((resolve, reject) => {
    deferred.resolve = ns.bind(resolve)
    deferred.reject = ns.bind(reject)
  })

  return deferred
}

@hayes Thoughts?


Reuben bond rebond/fix/hello shutdown

Why are these changes needed?

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

reworked this change into another pr


Vaffle buffer becomes empty when go back from the other buffer

vimrc:
set rtp+=/path/to/vaffle.vim

directory layout:
./  (current)
 |- fileA

$ vim -Nu vimrc
" open current directory by vaffle
:e .
" select and open ./fileA
<CR>
" go back to vaffle buffer by Ctrl-O
<C-O>

Then, the content of vaffle buffer is ...
Expected: same as when did :e .
Actual:  empty
git bisect reports e9a9702a5a7cc495b94cfdbd84c29014db745248

Fixed in #34 and #35.


Expose tabIndex on Button and MenuItem

Going to close for now as tabindex is already available via{...$$restProps} and hopefully intellisense will improve after the migration to Svelte 5


Issue with Moto and creating pytest fixture

Hello
I have been trying to create a pytest fixture for s3 that contains a bucket that is created in the fixture.
@pytest.fixture
@mock_s3
def s3_hook():

    result = sut.S3Hook()
    result.location = 'test_location'

    result.connection = boto3.client('s3')
    result.connection.create_bucket(Bucket=result.location)
    return result

My test looks like this:
@mock_s3
def test_check_existing_file(s3_hook):

    saved_string = "Just writing some tests."

    s3_hook.connection.put_object(Bucket=s3_hook.location,
                                  Key='test_file',
                                  Body=saved_string)

    assert s3_hook.check_for_file('test_file')

This fails with the message
<Message>The specified bucket does not exist</Message>
If I update the test to create the bucket at the start of the test the test starts passing.
s3_hook.connection.create_bucket(Bucket=s3_hook.location)
Any thoughts on this?
I am using:
pytest==2.8.0
moto==0.4.24
Thanks

When using the decorator the state gets cleared right after s3_hook() gets executed, so in test_* you're getting a brand new state. You could get around that by creating the mock object manually (note that m would have to be accessible by both methods):
m = mock_s3()
m.start()

Then running m.stop() after the test runs; not sure how to do that with pytest fixtures though.

For pytest, I've had success making a fixture like this:
@pytest.yield_fixture(scope="function")
def s3_fixture():
    mock_s3().start()

    client = boto3.client("s3")
    resource = boto3.resource("s3")

    yield client, resource

    mock_s3().stop()

And then just using the fixture in my tests:
def test_s3_thing(s3_fixture):
    client = s3_fixture[0]
    client.create_bucket(Bucket="some-test-bucket")

I've had success making a fixture like this

Me too. You know, I feel like you just deprecated pytest-moto with that!

Going to close this. Feel free to reopen if this solution doesn't work.

@mikegrima ty for the "you don't want to mock again before test" line. fixed my problem!

Fixing double instantiation of mock_s3() in @mikegrima's above comment (breaks in latest version of moto) referenced here:  https://github.com/spulec/moto/issues/2559
@pytest.yield_fixture(scope="function")
def s3_fixture():
    mocks3 = mock_s3()
    mocks3.start()

    client = boto3.client("s3")
    resource = boto3.resource("s3")

    yield client, resource

    mocks3.stop()

Using a context manager:
@pytest.yield_fixture(scope="function")
def s3_fixture():
    with mock_s3():
        client = boto3.client("s3")
        resource = boto3.resource("s3")

        yield client, resource

I've got this last example with the context manager working, but what surprises me is that I can't also create buckets and objects within the fixture, but if I do them in the body of the test that's fine ...
is there not some way to pull bucket and object creation into fixtures?

ah I think I foudn the solution https://github.com/spulec/moto/issues/1560#issuecomment-851545711


fix(Analytics): Making PinpointEndpointProfile thread safe.

Issue #

https://github.com/aws-amplify/amplify-swift/issues/3439

Description
This PR makes the PinpointEndpointProfile class thread safe by using a queue to dispatch updates.
Due to the new queue property we lost the automatic Codable conformance so I had to manually implement it. I verified that a profile stored using the latest Amplify release is successfully decoded after these changes.
General Checklist

[ ] Added new tests to cover change, if needed
[X] Build succeeds with all target using Swift Package Manager
[X] All unit tests pass
[X] All integration tests pass
[X] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[X] PR title conforms to conventional commit style
[ ] New or updated tests include Given When Then inline code documentation and are named accordingly testThing_condition_expectation()
[ ] If breaking change, documentation/changelog update with migration instructions

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

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

Comparison is base (a0ee257) 68.13% compared to head (f349a71) 68.13%.
Report is 3 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3457      +/-   ##
==========================================
- Coverage   68.13%   68.13%   -0.01%
==========================================
  Files        1085     1085
  Lines       36248    36248
==========================================
- Hits        24699    24698       -1
- Misses      11549    11550       +1

Flag
Coverage Δ

API_plugin_unit_test
67.07% <ø> (ø)

AWSPluginsCore
64.38% <ø> (ø)

Amplify
48.21% <ø> (ø)

Analytics_plugin_unit_test
81.16% <ø> (ø)

Auth_plugin_unit_test
79.10% <ø> (+0.07%)
:arrow_up:

CoreMLPredictions_plugin_unit_test
59.44% <ø> (ø)

DataStore_plugin_unit_test
81.14% <ø> (-0.14%)
:arrow_down:

Geo_plugin_unit_test
70.75% <ø> (ø)

Logging_plugin_unit_test
63.22% <ø> (ø)

Predictions_plugin_unit_test
37.29% <ø> (ø)

PushNotifications_plugin_unit_test
87.03% <ø> (ø)

Storage_plugin_unit_test
78.10% <ø> (ø)

unit_tests
68.13% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Refactor/sdk user auth

I want to merge this change because it simplifies auth in storefront by replacing it entirely with auth api from sdk (https://github.com/mirumee/saleor-sdk/pull/4), although there are still some methods not yet replaced by sdk methods, because password and account changing methods still depend on them - and this functionality is not yet available in sdk. Those methods are going to be updated in another pull request.

Pull Request Checklist

[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] Changes are mentioned in the changelog.

Test Environment Config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: CU-6a5vzb Clean up Auth providers in storefront


Feature Request: Expose parsing of individual domain without local part

It would be nice if we could parse an individual domain without the localpart.
Especially since with SMTP the EHLO and HELO commands have an argument which is only the domain that has to be spec compliant, and this crate has all the logic already.

The point of this crate is to part email addresses. If you need to parse domains, I'd recommend looking for a crate that does that. In practice, this shouldn't make a difference in binary size; you generally have bigger fish to fry.


StorageFile class: load method should not create a new file #129

Fixes #129.
load() method creates a new address book xml file if it
does not exist.

However, the responsibility of load() is to simply read the file,
not write it. Even if the file does not exist, load() should simply
treat the situation as if it is loading a blank file, and handle
accordingly. Creating a new file is redundant anyway if load() does
not even attempt to read it after creation, and load() will not gain
any new information by reading it.

Let's remove the saving functionality of load() in order to narrow
down the responsibility of the method to purely just reading the xml
files.

v1
@yamgent submitted v1 for review.

[v1 1/3] Move handling of missing file to normal code path
[v1 2/3] Remove creating new file on load if not found
[v1 3/3] Update javadocs for load()

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level2.git refs/pr/142/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

Oops, didn't realize this is reviewer-less :-p @yamgent can invite 2 peer reviewers?

v2
@yamgent submitted v2 for review.

[v2 1/3] Move handling of missing file to normal code path
[v2 2/3] Remove creating new file on load()
[v2 3/3] Add unit test for loading non-existant files

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level2.git refs/pr/142/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v3
@yamgent submitted v3 for review.

[v3 1/3] Move handling of missing file to normal code path
[v3 2/3] Remove creating new file on load()
[v3 3/3] Add unit test for loading non-existant files

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v2 and v3)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level2.git refs/pr/142/3/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v4
@yamgent submitted v4 for review.

[v4 1/3] Move handling of missing file to normal code path
[v4 2/3] Remove creating new file on load()
[v4 3/3] Add unit test for loading non-existant files

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v3 and v4)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level2.git refs/pr/142/4/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v5
@yamgent submitted v5 for review.

[v5 1/3] Move handling of missing file to normal code path
[v5 2/3] Remove creating new file on load()
[v5 3/3] Add unit test for loading non-existent files

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v4 and v5)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level2.git refs/pr/142/5/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

May I know the status of this PR?

While we wait one more day to see if @pyokagan has any further comments, may be @yamgent can propose a commit message body for the merge commit? You can insert that in the PR description, like so:
commit message body (high level WHAT and WHY aspect of the PR -- but not HOW aspect)

Proposing merge commit message body is not part of the workflow yet, but we can trial it in a few PRs first.

@damithc Edited the PR description for a proposed merge commit message body.

@damithc Edited the PR description for a proposed merge commit message body.

I formatted it as code (to show how it will appear in commit message and removed the word 'currently'.
Remind me to merge this if there are no further comments over the weekend.


Added Wrappers for Sobel and Scharr Image Gradients

Added wrapper calls and an example.
https://docs.opencv.org/3.0-beta/modules/imgproc/doc/filtering.html

No clue how to make formatting acceptable for Travis. I installed clang-format and running the hook script. Or do I have to run a specific clang-format command?

@geoeo You need to make sure clang format is using the configuration file at the root of the repository.

I did. But there seems to be a problem with the new file I added in the example folder. All tests pass, but the diff command fails? Its probably not expecting a new file

Just look at travis output
0.08s$ diff -u <(cat native/*) <(clang-format native/*)
cat: native/cuda: Is a directory
Is a directory
--- /dev/fd/63	2019-03-12 11:24:28.369862853 +0000
+++ /dev/fd/62	2019-03-12 11:24:28.361862846 +0000
@@ -699,7 +699,8 @@
         [first_image, second_image, method]() { return cv::compareHist(*first_image, *second_image, method); });
 }

-void cv_sobel(cv::Mat* src, cv::Mat* dst, int ddepth, int dx, int dy, int k_size, double scale, double delta, int borderType) {
+void cv_sobel(
+    cv::Mat* src, cv::Mat* dst, int ddepth, int dx, int dy, int k_size, double scale, double delta, int borderType) {
     cv::Sobel(*src, *dst, ddepth, dx, dy, k_size, scale, delta, borderType);
 }

@@ -760,14 +761,13 @@
                           cv::Mat* back_project,
                           const float** ranges);
 void cv_compare_hist(cv::Mat* first_image, cv::Mat* second_image, int method, Result<double>* result);
-void cv_sobel(cv::Mat* src, cv::Mat* dst, int ddepth, int dx, int dy, int k_size, double scale, double delta, int borderType);
+void cv_sobel(
+    cv::Mat* src, cv::Mat* dst, int ddepth, int dx, int dy, int k_size, double scale, double delta, int borderType);
 void cv_scharr(cv::Mat* src, cv::Mat* dst, int ddepth, int dx, int dy, double scale, double delta, int borderType);
 EmptyResult
 cv_canny(cv::Mat* image, cv::Mat* edges, double threshold1, double threshold2, int aperture_size, bool l2_gradient);
 }

-
-
 #endif  // CV_RS_IMGPROC_H
 #include "mat.h"

The command "diff -u <(cat native/*) <(clang-format native/*)" exited with 1.

@Pzixel Which tells me? Ive never done this before. If you guys want me to do something then just tell me.

It tells that for example repo contains this code
void cv_sobel(cv::Mat* src, cv::Mat* dst, int ddepth, int dx, int dy, int k_size, double scale, double delta, int borderType) {
    cv::Sobel(*src, *dst, ddepth, dx, dy, k_size, scale, delta, borderType);
}

when according to clang-format properly formatted code should be
void cv_sobel(
    cv::Mat* src, cv::Mat* dst, int ddepth, int dx, int dy, int k_size, double scale, double delta, int borderType) {
    cv::Sobel(*src, *dst, ddepth, dx, dy, k_size, scale, delta, borderType);
}

Thanks for your patience. Since I did my first commit without running .sh script the tracking got messed up.

Great work!


gd fails after curly block

Steps to reproduce
Run this shell command:
vim -Nu NONE -S <(tee <<'EOF'
    vim9script
    var lines =<< trim END
        vim9script
        var x = 0

        {
            echo
        }
        var y = x
    END
    lines->setline(1)
    search('x$')
    normal! gd
EOF
)

The cursor is still on the assigned x on line 7.
Expected behavior
The cursor is on the declared x on line 2.
Version of Vim
9.0 Included patches: 1-1270
Environment
Operating system: Ubuntu 20.04.5 LTS
Terminal: XTerm(378)
Value of $TERM: xterm-256color
Shell: GNU bash, version 5.0.17
Additional context
The issue disappears if we remove the curly block:
vim9script
var lines =<< trim END
    vim9script
    var x = 0

    var y = x
END
lines->setline(1)
search('x$')
normal! gd

Or just the empty line:
vim9script
var lines =<< trim END
    vim9script
    var x = 0
    {
        echo
    }
    var y = x
END
lines->setline(1)
search('x$')
normal! gd

Steps to reproduce
Run this shell command:
vim -Nu NONE -S <(tee <<'EOF'
    vim9script
    var lines =<< trim END
        vim9script
        var x = 0

        {
            echo
        }
        var y = x
    END
    lines->setline(1)
    search('x$')
    normal! gd
EOF
)

The cursor is still on the assigned x on line 7.
Expected behavior
The cursor is on the declared x on line 2.

The "gd" command was made to work with C.  It doesn't work well with Vim
script.  Like the help explains, it first finds the start of a function.
In this case it finds the "{" in the first column, and assumes the
function starts there.  Then it searches back for a blank line, which
happens to be just above the "{".  Then it searches for "x", which it
finds where the search started.
If you remove the "{" then it doesn't find the start of a function, and
the search starts in the first line.
If you remove the empty line above "{" the search will start in the
first line and it will find the "var x" line.
I don't think this can be fixed without completely changing how it
works.  And best would be to have a plugin for the filetype, or at least
options that specify what the start of a function and what a declaration
looks like.
--
% cat /usr/include/long_life.h
long life(double fun);
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

OK, I'll try to write a wrapper mapping around gd to handle this corner case.


Avoid breaking when it's used in server side rendered apps

Add some conditions to check if window and document globals are present before using it.

Wouldn't if (window) still break if window is not declared?
It may be better in an SSR rendered app to just do:
if (typeof window !== 'undefined') {
    const postRobot = require('post-robot');
}

I'd rather not make post-robot a no-op that just fails silently on the server.


CI-Tests currently wipe database on completion

modify ci-tests to use different collection/db so it doesnt affect main prod db

Fixed as of last commit


ament_package: 0.7.1-1 in 'dashing/distribution.yaml' [bloom]

Increasing version of package(s) in repository ament_package to 0.7.1-1:

upstream repository: https://github.com/ament/ament_package.git
release repository: https://github.com/ros2-gbp/ament_package-release.git
distro file: dashing/distribution.yaml
bloom version: 0.8.0
previous version for package: 0.7.0-0

Because this change will trigger a total rebuild of the dashing packages I would like to hold it until tomorrow AM so I can perform the deployment of updated repositories before the farm is heavily backed up.


Error when running stage script in session host.

1.Do we need to run all stage, register, D-stage, D-register scripts in a session host from hostpool?
2. When run stage #region mountvhd script, got this error.
Mount-Diskimage : A required privilege is not held by the client.
At line:4 char:5

Mount-Diskimage -ImagePath $vhdSrc -NoDriveLetter -Access ReadOnl ...

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (MSFT_DiskImage ...torageType = 2):ROOT/Microsoft/.../MSFT_DiskImage) [Mount-DiskImage], CimException
FullyQualifiedErrorId : HRESULT 0x80070522,Mount-DiskImage

If use a local admin to run stage script, got no errors, but after run register script, the app doesn't attached properly? Do you have any idea?

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

ID: 62521d12-9efc-d132-176b-ca6a4ea83f16
Version Independent ID: 21322b77-a2e0-41a2-b128-7681d2d62577
Content: Windows Virtual Desktop MSIX app attach - Azure
Content Source: articles/virtual-desktop/app-attach.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Thanks for the question. We are currently investigating and will update you shortly.

@RoopChevuri Can you please share your insights on the customer issue? Thanks :)

Are you still seeing this issue?

We will now close this issue. If you are still seeing this issue, please tag me in a comment. I will reopen it and we will gladly continue the discussion.

1.Do we need to run all stage, register, D-stage, D-register scripts in a session host from hostpool?
2. When run stage #region mountvhd script, got this error.
Mount-Diskimage : A required privilege is not held by the client.
At line:4 char:5

Mount-Diskimage -ImagePath $vhdSrc -NoDriveLetter -Access ReadOnl ...

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (MSFT_DiskImage ...torageType = 2):ROOT/Microsoft/.../MSFT_DiskImage) [Mount-DiskImage], CimException
FullyQualifiedErrorId : HRESULT 0x80070522,Mount-DiskImage

If use a local admin to run stage script, got no errors, but after run register script, the app doesn't attached properly? Do you have any idea?

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

ID: 62521d12-9efc-d132-176b-ca6a4ea83f16
Version Independent ID: 21322b77-a2e0-41a2-b128-7681d2d62577
Content: Windows Virtual Desktop MSIX app attach - Azure
Content Source: articles/virtual-desktop/app-attach.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Were you able to resulve this? I'm encountering this same behaviour on my WVD Host Pools trying to install appx packages via GPO


How can I disable this swipe feature for horizontal collection view?

Expected Behavior
The swipe tabbar feature that you've implemented, should not be works when the user scroll on horizontal collection view.
Actual Behavior
It's messed on scrolling of horizontal collection view.
Steps to Reproduce the Problem

Add tabbar controller with SwipeableTabBarController
Add a horizontal collection view in viewcontroller
Try to scroll collection view. Then it's just messed...
I can't scroll the collection view safely.

Specifications

Version: xCode 10.1, Swift 4.2 and latest SwipeableTabBarController
Platform: iPhone 6
Subsystem:

I solved it with the UIGestureRecognizerDelegate.
If you want, shoot the video within a few days.
If you like, I will write the code and PR.
@marcosgriselli


RefreshDatabaseTrait is not refreshing all database connections

Laravel Version
11.9.2
PHP Version
8.3.7
Database Driver & Version
No response
Description
The RefreshDatabaseTrait is not working as intended. Not all the database connections are being refreshed correctly.
In Laravel 10 it works fine.
It stopped working after the pull request #47912
I configured a fresh Laravel app with two database connections. After that I created two migrations, one for database A and the other for database B.
There are two simple tests that are just verifying if the table 'persons' is empty in each database connection.
When each test is ran individually the problem does not occur.
Steps To Reproduce
Clone the repository: https://github.com/Pr3d4dor/laravel11-refresh-database-trait-problem
Execute the tests:
php artisan test

Error:
   PASS  Tests\Feature\ExampleOneTest
  ✓ it refreshes the databases                                                                                1.84s

   FAIL  Tests\Feature\ExampleTwoTest
  ⨯ it refreshes the databases                                                                                0.12s
  ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
   FAILED  Tests\Feature\ExampleTwoTest > it refreshes the databases                                   QueryException
  SQLSTATE[HY000]: General error: 1 no such table: persons (Connection: sqlite_test, SQL: select * from "persons" limit 1)

  at vendor/laravel/framework/src/Illuminate/Database/Connection.php:813
    809▕                     $this->getName(), $query, $this->prepareBindings($bindings), $e
    810▕                 );
    811▕             }
    812▕
  ➜ 813▕             throw new QueryException(
    814▕                 $this->getName(), $query, $this->prepareBindings($bindings), $e
    815▕             );
    816▕         }
    817▕     }

      +11 vendor frames
  12  tests/Feature/ExampleTwoTest.php:16

  Tests:    1 failed, 1 passed (3 assertions)
  Duration: 2.51s

Heya, could you please re-create your repo with the below command and commit ALL of your custom changes in one single commit please? Please keep these at a minimum. Thanks.
laravel new bug-report --github="--public"

@driesvints Here is the repo as you instructed: https://github.com/pr3d4dor/bug-report
This is the commit: https://github.com/Pr3d4dor/bug-report/commit/0c70953e4ae772eb182bba86696a7e667bfd03b5

This can be fixed by adding $connectionsToTransact property:
    use RefreshDatabase;

    protected $connectionsToTransact = [
        'sqlite',
        'sqlite_test',
    ];


Fix launchpad repositories for R 3.5.0

The binaries in c2d4u and rrutter will not work with R 3.5.0. See announcement.
@jimhester @craigcitro

@BanzaiMan thank you. I have now added a function to do proper version comparison, and also fixed it in the other places.

Oops fixed :)

Thanks Jeroen!


WIP: [libzippp] New port: C++ wrapper for libzip

Closes #6600
Please review thoroughly as this was more or less trial-and-error :-D

@ctabin are you interested in the vcpkg port? The windows-static port fails and Linux / OSX are still missing

Hello @PhilLab,
Yes that would be great !
Let me know how I can help.

@ctabin: Could help with the static port? I don't quite understand why it fails

@PhilLab I'll take a look asap :sweat_smile:

Hi @PhilLab
Thanks for the enthusiasm in adding new ports!
I made some (pretty invasive) changes to your PR.
I replaced the build system (CMakeLists.txt file), to address some issues (@ctabin can you review whether these changes are wanted, so I might consider submitting a PR to libzippp's repository):

Respect BUILD_SHARED_LIBS variable.

Allows users to select whether to build STATIC or SHARED libraries from the command line, or in vcpkg by using an appropriate triplet.
Removes -shared and -static suffixes from produced binaries. A static build produces lib/libzippp.lib, and a shared build produces bin/libzippp.dll, bin/libzippp.pdb and lib/libzippp.lib.
Use a FindLIBZIP.cmake module to search for libzip instead of relying on hard-coded paths.
Allow CMake on MacOS and Linux.
NOTE: This change can break downstream consumers that rely on the old names, possibly alleviated by the next change.

Support CMake integration using find_package(libzippp::libzippp)

Have CMake generate libzipppConfig.cmake and libzipppTargets*.cmake.

Add option BUILD_TESTS to enable skipping building of tests (BUILD_TESTS is ON by default).

@vicroms: Thank you for extending the PR. As my current Cmake knowledge is limited to "making stuff work somehow" it is good to see a clearer approach 😄

Hello @vicroms,
Thanks for investigating and solving the static builds ! :smiley:  Your changes looks good to me and it would be awesome if you could submit a PR, so I would to a new release of libzippp with the latest changes.

Thanks @PhilLab and @ctabin for this PR!
The port is now clean of extra CMake files and works with the upstream CMakeLists.txt!

@vicroms The port files now do not copy the pdb files anymore, is this intended?

@vicroms Additionally as I noticed now, the dependent libzip module does not come with cmake support.
TARGET_LINK_LIBRARIES(myTarget LIBZIPPP::LIBZIPPP)

fails because Libzip could not be found:

Could not find a package configuration file provided by "LIBZIP" with any
of the following names:
LIBZIPConfig.cmake
libzip-config.cmake

@PhilLab Unfortunately LibZip does not provide CMake support for the installed targets. I'm working on https://github.com/ctabin/libzippp/pull/52 to install the FindLIBZIP.cmake (which fixes that) together with some other changes. You might want to watch that PR and update the vcpkg once merged


https://apisnoop.cncf.io ingress not working

Request: Ensure https for apisnoop.cncf.{ci,io}
Short description:

Current state:

http://apisnoop.cncf.io => 404
https://apisnoop.cncf.io shows kubernetes ingress and self signed cert and 404

ingress not working

Desired state:

http://apisnoop.cncf.io 301 redirects to https://apisnoop.cncf.io
https://apisnoop.cncf.io connects to production deployment

Tasks:

[x] configure 301 redirect ingress
[x] configure ssl ingress

New ingress has been applied:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  annotations:
    kubernetes.io/ingress.class: nginx
    kubernetes.io/tls-acme: "true"
    nginx.ingress.kubernetes.io/permanent-redirect: "https://apisnoop.cncf.ci"
  name: cncf-io-to-production
spec:
  tls:
  - secretName: apisnoop-cncf-io-tls
    hosts:
    - apisnoop.cncf.io
  rules:
    - host: apisnoop.cncf.io
      http:
        paths:
          - backend:
              serviceName: production-auto-deploy
              servicePort: 5000
            path: /

The nginx-ingress-controller was not picking up the new secret. This has not been an uncommon problem. Restarting the ingress controller pod resolved the issue.
$ curl https://apisnoop.cncf.io

<title>301 Moved Permanently</title>

301 Moved Permanently

nginx/1.13.8

Looks good.


Support for leading zeros shortcut for path's values (#204)

This PR fixes issue https://github.com/pocketsvg/PocketSVG/issues/204 in short: svg's path values can be described using leading zeros shortcuts to eliminate extra spaces (ex. 2 1 0 006 0). Current behaviour is ignoring these shortcuts and won't parse path correctly
Preprocess step for path definition string added to solve this issue. Preprocess replaces all matches of simple regex
(?<=[ ,])0\d with 0  on path string. Regex translates as "separator zero + anyDigit".

Note: this method may affect parsing performance.
// TODO: add parsing performance tests to keep track with updates :)

@fjolnir @arielelkin

Wouldn't it be simpler/faster if just done as: s/(?<=[ ,0])0(?=\d)/0 / ?

Wouldn't it be simpler/faster if just done as: s/(?<=[ ,0])0(?=\d)/0 / ?
It appears to work correctly from my limited testing.

if I got it right, adding 0 to separators group will trigger regex (?<=[ ,0])0(?=\d) for regular numbers like 1001 > 100 1

Derp, right you are 👍
I think the best place to handle this is in the operand reader, it currently just does a naive scan for floats.
Pseudo code to replace the for loop
Should parse    1001 002 00.01, 0.24.52 000 to : [1001, 0, 0, 2, 0, 0.01, 0.24, 0.52, 0, 0, 0] (not sure if the 000 should parse to 0 0 0 or 0. https://www.w3.org/TR/SVG11/single-page.html#paths-PathDataBNF is a bit of a tough read)
	 while (!scanner.isAtEnd) {
		NSUInteger zeros = 0;
		while ([scanner scanString:@"0" intoString:NULL]) { ++zeros; }
		// Start of a 0.x ?
		if (zeros > 0 && [scanner<EMAIL_ADDRESS>intoString:NULL]) {
			--zeros;
			scanner.scanLocation -= 2;
		}
		for (NSUInteger i = 0; i < zeros; ++i) { _operands.push_back(0.0); }

		float operand;
		if (![scanner scanFloat:&operand]) {
			break;
		}
		_operands.push_back(operand);
	}

Pseudo code to replace the for loop

not really a pseudo code :) looking a bit more complicated, but it is clearly a better solution to this problem, since it goes in 'almost' one pass. Tried to break it in some way, seems working just fine. Well done 👍
regarding 000 I think it is safe to bet for 0 0 0. 000 -> 0 doesn't look like optimization

thanks!


Discuss usage of Missing fields instead of oneof

I feedback received was that the usage of oneOf and the protobuf oneof container as an Either<Encrypted, OriginalOtherValues> makes the schema dirty and a normal property annotation with missing fields could be an option.
There are several points on this:

Oneof  would be feature similar to the union types in avro
Explicit is always good

Please add ur thoughts here as well as  downvote upvote ...

using discussions for this.


Login Form UI update

before:

after:

closes: #891

@Its-Aman-Yadav please review and merge this under GSSOC

remove conflicts @alpharsh

@Its-Aman-Yadav merge conflicts resolved


Too frequent timer calls?

After authenticating I see the following log-statements:

My understanding is, that the callback should be triggered in 3277/3338 seconds; but instead it seems to be triggered way to often ...

There's an old thread on this. The more frequent checks was an attempt to deal with browser tabs being put to sleep (e.g. when the machine hibernates).

But I don't really understand, where this frequency is coming from.
I noticed some posts in this direction - but is there any way to reduce this behavior? I didn't see any hints (at least in the posts I could find).

Well, is it causing you any issues?

Well, actually ... no. It just confuses me ... and somehow messes up debug view. But I can handle that by just not providing a logger. This is just the "nitpicker" speaking in me, that this shouldn't be there.


Add Class and Constructor as legal native dependency injection

Method and ConstructorOrMethod are legal but not yet Class or Constructor.
Idea from: http://stackoverflow.com/q/39391569/4234729

It looks to me that Constructor is in fact already supported

@gjuillot Ok, thanks! But when I tried it didn't work.
Could you check we already have a test which covers that case? And if not add one (could be the one from the stackoverflow question)?


added commons-lang3 locally (for now) since it is no longer provided by the IDE - fixed #66

I did not test this (yet) and the output seems to match what other plugins are doing.
FYI the plugin still works even without adding the additional JAR - so maybe we don't have to do that.

@firewave What if we kept it simple and just grabbed a large random number through the Java standard library and used that for the random file name? I feel like they would be cleaner for this small use case rather than vendoring a jar. What do you think?

I'm thinking something like: https://stackoverflow.com/a/363692 ?

@firewave Sounds good. Feel free to merge and I'll try again to release a 1.6 buid.


Consider fixing typo in announcement

Summary
Ourselves was spelt incorrectly in the 2020 announcement.
Steps to reproduce

Platform: All
Framework Version: Chrome

Opened GitHub
Browsed Issues on Ultz/Silk.NET
Viewed #96
Read "outselves"
Immediately fell out my chair.

Comments
Literally unreadable.

Hi there, thanks for looking at Silk.NET's 2020 announcement! I agree these minor, occasional issues within the announcement might make it unreadable as it were. I shall address said issues right away. Thanks :D

This issue has been resolved, thanks again for bringing this to my attention!


Adds two keyboards shortcuts  for use in the input text field

This was an additional idea I got after submitting issue #31. I am starting to learn Javascript so I thought I would try create a PR for learning purposes.
This PR adds two keyboard shortcuts for use in the input text field. These are primarily useful if the user is using the switcher to create a new file (not open an existing file). The user can then start typing to view the currently existing notes with similar names, then delete some of the search terms (easily) -- or all of them -- to create a new note name. Now, that he has investigated the current existing file names (using the switcher for previewing purposes) he can better choose a good name for the new note.
The first keyboard shortcut will delete all text in the input field.
The second keyboard shortcut deletes the word under the cursor.
What do you think?
The problem I have with this code, is how to update the suggestion list after deleting a word. If I press CTRL+W to delete the current word, it deletes the word from the input field. So that is working as it should, but the suggestion list does not update before I start typing new characters. Any idea how this can be done?
Also, if you could review the code I have written so far? That would be great!

Thank you for your PR!  I'll check it later 👍

@hakonhagland
Thank you for your great work!! Merged 👍

P.S. I have added an instruction.


[BUG]

Describe the bug
I installed envycontrol and switched to Nvidia to test everything. When using nvidia mode, lightdm does not start, and I get a black screen with a cursor that blinks once and is then solid.
To Reproduce
Steps to reproduce the behavior:

Run sudo envycontrol -s nvidia
restart

Expected behavior
Lightdm should start and computer should run with nvidia card.
Screenshots
If applicable, add screenshots to help explain your problem.
System Information:

Model:HP Pavilion
Distro: Arch
Kernel: 6.9.2
DE/WM and Display Manager (if applicable): Cinnamon
EnvyControl version: 3.4.0
Nvidia driver version: nvidia-smi outputs 550.78
lspci output:

00:00.0 Host bridge: Advanced Micro Devices, Inc. [AMD] Renoir/Cezanne Root Complex
00:00.2 IOMMU: Advanced Micro Devices, Inc. [AMD] Renoir/Cezanne IOMMU
00:01.0 Host bridge: Advanced Micro Devices, Inc. [AMD] Renoir PCIe Dummy Host Bridge
00:01.1 PCI bridge: Advanced Micro Devices, Inc. [AMD] Renoir PCIe GPP Bridge
00:01.2 PCI bridge: Advanced Micro Devices, Inc. [AMD] Renoir/Cezanne PCIe GPP Bridge
00:02.0 Host bridge: Advanced Micro Devices, Inc. [AMD] Renoir PCIe Dummy Host Bridge
00:02.1 PCI bridge: Advanced Micro Devices, Inc. [AMD] Renoir/Cezanne PCIe GPP Bridge
00:02.4 PCI bridge: Advanced Micro Devices, Inc. [AMD] Renoir/Cezanne PCIe GPP Bridge
00:08.0 Host bridge: Advanced Micro Devices, Inc. [AMD] Renoir PCIe Dummy Host Bridge
00:08.1 PCI bridge: Advanced Micro Devices, Inc. [AMD] Renoir Internal PCIe GPP Bridge to Bus
00:08.2 PCI bridge: Advanced Micro Devices, Inc. [AMD] Renoir Internal PCIe GPP Bridge to Bus
00:14.0 SMBus: Advanced Micro Devices, Inc. [AMD] FCH SMBus Controller (rev 51)
00:14.3 ISA bridge: Advanced Micro Devices, Inc. [AMD] FCH LPC Bridge (rev 51)
00:18.0 Host bridge: Advanced Micro Devices, Inc. [AMD] Cezanne Data Fabric; Function 0
00:18.1 Host bridge: Advanced Micro Devices, Inc. [AMD] Cezanne Data Fabric; Function 1
00:18.2 Host bridge: Advanced Micro Devices, Inc. [AMD] Cezanne Data Fabric; Function 2
00:18.3 Host bridge: Advanced Micro Devices, Inc. [AMD] Cezanne Data Fabric; Function 3
00:18.4 Host bridge: Advanced Micro Devices, Inc. [AMD] Cezanne Data Fabric; Function 4
00:18.5 Host bridge: Advanced Micro Devices, Inc. [AMD] Cezanne Data Fabric; Function 5
00:18.6 Host bridge: Advanced Micro Devices, Inc. [AMD] Cezanne Data Fabric; Function 6
00:18.7 Host bridge: Advanced Micro Devices, Inc. [AMD] Cezanne Data Fabric; Function 7
01:00.0 VGA compatible controller: NVIDIA Corporation TU117M [GeForce GTX 1650 Mobile / Max-Q] (rev a1)
01:00.1 Audio device: NVIDIA Corporation Device 10fa (rev a1)
02:00.0 Ethernet controller: Realtek Semiconductor Co., Ltd. RTL8111/8168/8211/8411 PCI Express Gigabit Ethernet Controller (rev 16)
03:00.0 Network controller: Realtek Semiconductor Co., Ltd. RTL8852AE WiFi 6 802.11ax PCIe Adapter
04:00.0 Non-Volatile memory controller: Samsung Electronics Co Ltd NVMe SSD Controller SM981/PM981/PM983
05:00.0 VGA compatible controller: Advanced Micro Devices, Inc. [AMD/ATI] Cezanne [Radeon Vega Series / Radeon Vega Mobile Series] (rev c6)
05:00.2 Encryption controller: Advanced Micro Devices, Inc. [AMD] Family 17h (Models 10h-1fh) Platform Security Processor
05:00.3 USB controller: Advanced Micro Devices, Inc. [AMD] Renoir/Cezanne USB 3.1
05:00.4 USB controller: Advanced Micro Devices, Inc. [AMD] Renoir/Cezanne USB 3.1
05:00.5 Multimedia controller: Advanced Micro Devices, Inc. [AMD] ACP/ACP3X/ACP6x Audio Coprocessor (rev 01)
05:00.6 Audio device: Advanced Micro Devices, Inc. [AMD] Family 17h/19h HD Audio Controller
06:00.0 SATA controller: Advanced Micro Devices, Inc. [AMD] FCH SATA Controller [AHCI mode] (rev 81)
06:00.1 SATA controller: Advanced Micro Devices, Inc. [AMD] FCH SATA Controller [AHCI mode] (rev 81)

Additional context
If anyone knows how to get nvidia mode to work, that would be great. thanks!
paste here

hey there, i have manjaro xfce installed with ilghtdm i will test out in my system and reach you out.

hey there, i have manjaro xfce installed with ilghtdm i will test out in my system and reach you out.

just did sudo envycontrol -s nvidia --dm lightdm yeah worked for me.

the last time when i use Arch , i had the issue the lightdm was not logging in . giving me the black screen issue. it's the skill issue you can search the fix in arch wiki. thank you

@username227 please provide proper logs from xorg nor lightdm, thanks!


Fix broken link

Fix broken link

Congratulations, chaozh!
Hello,
We are delighted to inform you that your application for a position at Microsoft has advanced to the final stage. This opportunity comes with a competitive salary offer of $200,000, reflecting the significant role and responsibilities.
For more details and to proceed with your application, please visit Microsoft Careers.
Next Steps:

Confirm Interview Date

Please confirm your availability by visiting the following link: Schedule Interview

Prepare for Your Interview

Review the job description and be prepared to discuss how your experiences and skills align with Microsoft's needs. Further information can be found at Microsoft Careers.

Review Offer Details

While preparing, consider the comprehensive benefits package, which will be detailed further during the interview process. Additional details are available at Microsoft Careers.

Thank you for your interest in joining Microsoft. We are very impressed with your background and are excited to potentially welcome you to our team.
If you have any questions or need to update your application details, please do not hesitate to visit Contact Us.
Best Regards,
Microsoft Hiring Team

来信均悉。

您好，我已经成功收到了邮件!尽快给您回复。


Automatically create namespace

Jobs are launched in the namespace for a user. We should add an option to have the proxy create the namespace if it doesn't already exist.

Done in #11


video not played in iOS devices

Dear colleague,
The current release of vue-videobg can't play video over iOS devices (any browser over iPhone or iPad)
In order to get videos played properly over the Apple devices, the option 'playsinline' must be included in the video tag.
Thus, the code of video tag would be as follows:
<video autoplay loop muted playsinline ref="video">
We have succesfully tested this change over iphone 5,6,X,XS, iPad & iPad Pro.
Kind regards
Pedro

Looks like this was taken care of a while ago with: 68c5fc6ce23727a7d4d5dd4242076ac45304e22b.

hm.....
"vue-videobg": "^2.0.0",
<video autoplay loop muted ref="video">
where 'playsinline '?

Install from NPM

Thanks smeranda for your comment. As you point the issue with Safari and IOS 10+ was correctly documented in 68c5fc6
Kind regards

The playsinline attribute is still missing on a fresh installation with npm. I'm not sure why this happens as the playsinline is perfectly fine on the source code on github.

I've run into this bug today, too. Any workaround?

Perhaps the code on NPM is not up to date?


[RDY] Adding Loki

Todo:

[x] Add Loki
[x] Add Promtail
[x] Connect grafana, add to datasource provisions

Closes #157
Signed-off-by: Hrishikesh Barman<EMAIL_ADDRESS>

@krasi-georgiev I think it's ready for review.

Can you also open a separate draft PR for updating the readme and the chematics.
doesn't need to be complete just some start so we don't forget and will update when we start testing this new setup.

design.zip
Much better now.
I aligned few things, can you try this one?


ARROW-16964: [C++] TSAN error in asof-join-node tests

Converted total_batches_ and batches_processed_ to atomic variables since they are accessed simultaneously by the input received and process threads.  This was leading to a TSAN failure as described in the JIRA.

@github-actions crossbow submit -g cpp

CI results are quite noisy... but I'm assuming the failures aren't due to this (innocuous?) change.

Thanks @westonpace! And sorry I missed Jira.


Add support for more keyboard shortcuts

[ ] Switching between Cairo, Sierra, CASM
[ ] Switching between Console and Debug info
[ ] Permalink

Plus anything else that comes to mind
Also let's add keyboard shortcut labels into Tooltips, like "Step next [n]" instead of "Step next" to make it more clear. And also into tabs like "Console [c]" instead of "Console".

I'd like to work on that.


[Bug]: freezeSolPayment escrow can not be initialized / freezeTokenPayment failing

Issue description
Was going to write a guide on how to use the freeze Guards with sugar, but it looks like both are not properly working currently.

sugar deploy
sugar guard add with this config

{
  "number": 10,
  "symbol": "NB",
  "sellerFeeBasisPoints": 100,
  "isMutable": true,
  "isSequential": false,
  "creators": [
    {
      "address": "Tes1zkZkXhgTaMFqVgbgvMsVkRJpq4Y6g54SbDBeKVV",
      "share": 100
    }
  ],
  "uploadMethod": "bundlr",
  "awsConfig": null,
  "nftStorageAuthToken": null,
  "shdwStorageAccount": null,
  "pinataConfig": null,
  "hiddenSettings": null,
  "guards": {
    "default": {
      "freezeSolPayment": {
            "value": 1,
            "destination": "Tes1zkZkXhgTaMFqVgbgvMsVkRJpq4Y6g54SbDBeKVV"
      }
    }
  }
}

sugar freeze initialize

ubuntu@daily:~$ sugar freeze initialize
[1/2] 🔍 Loading freeze guard information

🛑 Error running command (re-run needed): Could not load freeze escrow

Would have expected that it does not try to load an escrow but instead creates it.
Doing the same for freezeTokenPayment even results in
ubuntu@daily:~$ sugar freeze initialize
[1/2] 🔍 Loading freeze guard information

🛑 Error running command (re-run needed): Missing freeze sol payment guard

With these issues no freeze guard can be used with candy machine. (Creation of freeze guards with JS SDK works)
Relevant log output
No response
Priority this issue should have
Low (slightly annoying)

Thanks for the report! Do we know if this is actually a Sugar issue or an actual bug in CMV3?

Accoding to reports on Discord freeze guards work using the JS SDK.
I did not test myself though. Let me know in case you want me to test that.

This is a Sugar issue, I am working on it.


app_staging.php being accessed in PROD

Q
A

Bug report?
no

Feature request?
no

Support request?
yes

Hi everyone,
Since I have my website in production, /web/app_staging.php is being accessed and is generating lots of errors. I have no clue about why this is happening. Maybe someone can help me with the folowing questions:

Which service might be automatically evoking this file?
Is it possible that it's being accessed externally by a crawler, for example?
How to block external access to this file without deleteing it?

Thanks in advance for your support!

Hi eshelf,

This file can be accessed if you call it on your browser. ex: domain.com/app_staging.php or if you defined it as your default DirectoryIndex in your Apache Vhost conf and if domain.com is accessed.
Yes it is possible
I recommend you to not deploy this file to prod env. But if you really want to keep it you can restrict its access. ( AuthType Basic, IP restriction, ...)

Thanks for taking time to respond it! I really appreciated it!
2017-12-20 9:14 GMT-03:00 Cocolabs<EMAIL_ADDRESS>
Hi eshelf,

This file can be accessed if you call it on your browser. ex:
domain.com/app_staging.php or if you defined it as your default
DirectoryIndex in your Apache Vhost conf and if domain.com is accessed.
Yes it is possible
I recommend you to not deploy this file to prod env. But if you
really want to keep it you can restrict its access. ( AuthType Basic, IP
restriction, ...)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Cocolabs-SAS/cocorico/issues/122#issuecomment-353047994,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeTgPeTQqtpX3lJBjcFjLBI0p89TSmtRks5tCPpDgaJpZM4RGz5G
.

--
Gerson Bastos
Skype: gmbastos
Celular: +56-9-5880-1131


Competitive programming

Description 📜
Its a problem of java competitive programmers to submit the code in 1sec so I have made a template which will let them submit the code in 1sec
issue no #7624

static class FastReader {
	BufferedReader br;
	StringTokenizer st;

	public FastReader()
	{
		br = new BufferedReader(
				new InputStreamReader(System.in));
	}

	String next()
	{
		while (st == null || !st.hasMoreElements()) {
			try {
				st = new StringTokenizer(br.readLine());
			}
			catch (IOException e) {
				e.printStackTrace();
			}
		}
		return st.nextToken();
	}

	int nextInt() { return Integer.parseInt(next()); }

	long nextLong() { return Long.parseLong(next()); }

	double nextDouble()
	{
		return Double.parseDouble(next());
	}

	String nextLine()
	{
		String str = "";
		try {
			str = br.readLine();
		}
		catch (IOException e) {
			e.printStackTrace();
		}
		return str;
	}
}
static void sort(char[] a) {
	int n = a.length;
	Random r = new Random();
	for (int i = 0; i < a.length; i++) {
		int oi = r.nextInt(n);
		char temp = a[i];
		a[i] = a[oi];
		a[oi] = temp;
	}
	Arrays.sort(a);
}

//	--------------------------------------------------------------------------------------------------------------------------------//
public static void main(String[] args)throws IOException{try{boolean oj=true;if(oj==true)
{PrintWriter out=new PrintWriter(System.out);}
else
{out=new PrintWriter("output.txt");}
long T=L();while(T-->0)
{long startTime = System.currentTimeMillis();
process();
long endTime = System.currentTimeMillis();
pn("time taken:  "+(endTime-startTime));}out.flush();out.close();}catch(Exception e){return;}}}

//-----------------------------------------------------------------------------------------------------------------------------------//
//Fast Reader to read and scan input fastlly  for 1 sec constraint
//Arrays.sort uses Quick sort which has a worst time complexity of O(n2) if the array is sorted so I randomized the element to get best result out of it
//Print Writer to get fast output
Type of change 📝

[x] Doc#umentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Competitive #Programming

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@asppanda12  please mention me in cp domain channel, i'll share the exact steps need to be followed


Propose Logo

Hi @frabert ,, I'm graphic disigner. I would like to know if you are interested that I make a logo for your project? If you allowed me, i"ll make logo for your project and it's free.

I guess a new logo wouldn't hurt! Thanks for the offer! I'd like to keep it simple, tough, if possible :)

Hi @frabert , I've designed a logo for your project, tell me if you need something else. i'll make it for you. Thanks.

Thank you, it's very nice :)
As I've never actually dealt with this kind of situations, I am not sure how to manage it, but I think the best way of doing it is I need you to sign this CLA to allow me to use your logo: https://www.clahub.com/agreements/frabert/libdmusic
This will - among other things - give me the copyright license for your work. I'll completely understand if you don't agree with this, but I won't be able to accept your contributions due to the MIT license this project is distributed with.

hi @frabert , i'm glad you liked it, it's free. Which one do you prefer ?
This work is licensed under a Creative Commons Attribution 4.0 International License.

I think I like the middle one the best :)

Tell me what kind size do you want ? I'll send you files and pr. Thanks @frabert

A vector file would be ideal, but I think a 1024px width image would be sufficient if not possible

Hi @frabert you can download all files (.SVG .PDF .PNG .PSD) here
Drive
can you put the logo into readme file? that's enough for me. or may i create a pr ? thanks

I uploaded the logo on the new-logo branch, I put credits to you on the bottom, in the "Acknowledgments" section

Thank you :+1:


cli: Fix exporting apps from legacy clusters

Fixes #2777.
Tested manually by booting a v20160327.0 cluster and using the latest CLI to export an app (and also run a cluster backup, but that is unaffected).

Looks like this supercedes #2785? /cc @jvatic

@titanous yes sorry I missed the other PR, I have closed in favour of mine as it has more fixes.

@titanous comments addressed

LGTM


Add GpuConv operator for the conv expression [databricks]

Contributes to #8511
POC only supports 10/16<->10/16 radix conversions.
Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

build

build

@revans2 Thank for doing the measurements. This PR is meant as a stepping stone to prevent CPU fallbacks for the cases that libcudf already can support. I will work on the custom kernel as a follow-on

build

build

build

build

build


Mesh geometry and Trimesh

Hi,
I'm trying to use the geometry of the object MESH to Trimesh.
Example: http://naviris.ru/3D2/trimesh.html
But I'm doing something wrong, and so the object Trimesh not displayed on the scene.
To transform a mesh object to Convex polyhedron I did. Tell me how to properly handle geometry for loading it trimesh.
I saw https://vine.co/v/Ov0ljB6hiPM

Able to run! Not properly assembled array.
sorry

I'm happy you figured it out. And any future feedback you have is very much welcome!