[IOTDB-2547] Rearrange Website Sidebar

Description
This change will rearrange the sidebar of the website.
Move UDF-Library docs to an independent column.

This PR has:

[x] been self-reviewed.

[ ] concurrent read
[ ] concurrent write
[ ] concurrent read and write

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods.
[ ] added or updated version, license, or notice information
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious
for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold
for code coverage.
[ ] added integration tests.
[ ] been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

Coverage increased (+0.01%) to 67.759% when pulling dd949fec449e544ed81fce3ed60acd0e22250134 on trin1t:sidebar-rearrange into ca5641a5ae79b1598104ebbd8a14ec714fc1dece on apache:master.


Scroll speed doubled when screen scaling is enabled

Does this issue occur when all extensions are disabled?: Yes/No

Version: 1.73.1
Release: 22314
Commit: 14f2d26367b7e8f03ff2352516ba27d6302dd7b1
Date: 2022-11-10T18:19:13.052Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.0.0-6-amd64
Sandboxed: No
Steps to Reproduce:

Run vscode in two different situations: With resolution 1920x1080 and scaling factor 1, and with 4k resolution and scaling factor 2.
Open some file, and turn the scroll wheel on the mouse by one notch.

Expected behavior:
It should scroll the same amount.
Actual behavior:
Scrolling is doubled when using screen scaling.

Potentially duplicates https://github.com/microsoft/vscode/issues/224714


Upgrade rails and gems

This will help the future maintenance by making all possible gems up to date and also removing git sources for govuk gems.

Looks good, merge on green


Problem with Umlauts since 0.1.6-beta

Hello!
First thank you for this cool piece of software, which I am using almost one year now.
After updating my Vikunja instance https://vikunja.klomp.de to 0.24.1 (from 0.23.??) and updating the Android app from 0.1.5-beta to 0.1.6-beta, all (german) umlauts are broken (in project names and also in task names).

The task should be "Bügel".
I don't know, if the api update or the app update caused the error. I think, it's the app, because in the web frontend and also in the desktop app the umlauts are still fine.

See: https://github.com/go-vikunja/app/issues/82
It's been solved and will be fixed in the new release

Sorry, didn't see that.
Thank you!


Update Peniko

This builds on https://github.com/linebender/peniko/pull/85

@DJMcNab This is to help make sure that all is well before we go and publish the new version of Peniko so this doesn't have to land by itself yet ...


Support for Template Display Name

It appears there's no way to add spacing or alternate casing to the name of a template. This suggests that the name is more of an identifier rather than a display name.
This especially becomes an obstacle when displaying multiple templates (cf. Accord Template Library) and trying to help people distinguish quickly between the contract clauses in a list.
Could one solution could be to add a display name to the metadata?

Yes, this is a great suggestion. We will be discussing this during the tech working group call this Wednesday.

Yes, this is a great suggestion. We will be discussing this during the tech working group call this Wednesday.

Correction: I'm expecting this to be on the agenda for the following week.

As per the discussion yesterday on the working group call, https://vimeo.com/350070645

The name of this metadata property is displayName.
The property should be a JavaScript string which allows it to be persisted to a template's package.json file. This gives us a lot of flexibility to include unicode characters, emojis etc. However it means that we should adopt JavaScript single character escape sequences e.g. \b,\n,\" etc.
The length of the property should be constrained to avoid it being used as a description field. I propose 214 characters to align with the package.json limit for package names.
Internationalization should follow the current convention with templates which is that templates have a single locale and are not dynamic to a user's locale. Templates that are the same except for localization are duplicated currently. Localization is discussed further in issue #174

Couple of comments:

Internally, this should probably be added to the cicero-core.Metadata module.
We may want to have an API call that lets a user set the displayName programmatically, and which makes sure exporting to an archive result in a package.json with the new name
Have we checked if there is a standard package.json documented field which would be appropriate?

I can't find a standard package.json field that performs the same function

Also, we don't currently provide a way to change the template name, instead this is immutable via the API once the template class is instantiated. The template constructor sets the template name via packageJson object. I suggest that we do the same for displayName.


virtual drop items

Drop virtual items around the player and they can't pick it up

Feature like Greedy in #1 so ...


AndroidManifest.xml not resolving android:icon and android:roundIcon

Steps to Reproduce
Created a new Flutter project using master and it doesn't resolve these icons.
<application android:name="io.flutter.app.FlutterApplication"
android:icon="@mipmap/ic_launcher"
android:roundIcon="@mipmap/ic_launcher_round"
With the Flutter alpha release this worked.  It is not finding the res/mipmap folders with these icons.
Logs
-flutter analyze found no errors.
Flutter Doctor
C:\Users\Tom\AndroidStudioProjects\MMusicRecorder>flutter doctor
[√] Flutter (on Microsoft Windows [Version 10.0.14393], channel master)
• Flutter at C:\Users\Tom\GitHub\flutter
• Framework revision 9cb12086bb (4 hours ago), 2017-05-09 14:03:16 -0700
• Engine revision cbbf956fcb
• Tools Dart version 1.23.0-dev.11.11
[√] Android toolchain - develop for Android devices (Android SDK 25.0.3)
• Android SDK at C:\Users\Tom\AppData\Local\Android\sdk
• Platform android-25, build-tools 25.0.3
• Java binary at: C:\Program Files\Android\Android Studio Preview\jre\bin\java
• Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-736)
[√] Android Studio (version 2.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Gradle version 3.2
• Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b06)
[√] Android Studio (version 2.4)
• Android Studio at C:\Program Files\Android\Android Studio Preview
• Gradle version 3.4.1
• Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-736)
[√] Connected devices
• None

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

This seems like a question of how the gradle project is configured.  flutter create generates a basic gradle project, but after that it's completely under the control of the developer.  We perhaps should change our template to have better defaults:
https://github.com/flutter/flutter/tree/master/packages/flutter_tools/templates/create/android.tmpl
That said, it's also possible I'm mis-understanding and there is a bug in flutter build itself.  But I suspect that all mipmap stuff is handled by the Android toolchain which we just call into?  @jakobr-google

This is all taken care of by the Android toolchain, and should work just like any other Android app.
We don't ship a ic_launcher_round icon (or reference it), so I assume that's something you've added? The mipmap folders are in android/app/src/main/res/, so that's where the icons should go. If that doesn't work, can you share your project (or a simplified version of it) with us?

Ok, will get to it this afternoon.
On Wed, May 10, 2017, 2:42 AM Jakob Andersen<EMAIL_ADDRESS>wrote:

This is all taken care of by the Android toolchain, and should work just
like any other Android app.
We don't ship a ic_launcher_round icon (or reference it), so I assume
that's something you've added? The mipmap folders are in
android/app/src/main/res/, so that's where the icons should go. If that
doesn't work, can you share your project (or a simplified version of it)
with us?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/9944#issuecomment-300431197,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUb3bQXRrVI_S8B4xlNTAt2blwVNcKks5r4YaSgaJpZM4NWD56
.

test_case_1:
1.
Created with "flutter create" with cmdline.
2.
Launch it with Android Studio.

Dialog pops up as follows:

Unknown Module Type
- Cannot determine module type  ("FLUTTER_MODULE_TYPE") for the following module "test_case_1".  The module will be treated as a Unknown module.
- (leave as is, not sure what to do here)
- Android frameworks is detected in the project
- I click on "Configure" and select "AndroidManifest.xml".  All seems ok.

Open AndroidManifest.xml and it shows @mipmap/ic_launcher in red (can't resolve).

I check the mipmap folders, the flutter ic_launcher png's exist.

Try to Build project now (Make Modules 'android', 'test_case_1' and get Error:
- Cannot start compilation: the output path is not specified for module "android".  Specify the output path in the Configure Project.
- Project Structure dialog comes up (not sure what to do here, so leave as is)
4.

flutter run
(all is ok, runs app in Emulator)

flutter build apk
(all is ok, builds apk with success)

Now the Project says "Migrate Project to Gradle?".  So it is now not building with Gradle but the cmd line works fine for build and run in Emulator.

Also; I updated the mipmap folders with my own icons and when I build it for the Emulator and Play Store, it has my icons.  So all seems to be ok except it still shows red and now not building with Gradle.

See attached project...
test_case_1.zip

These all look like issues with Android Studio. It doesn't have the Flutter plugin, so it doesn't know what FLUTTER_MODULE_TYPE is, and when you "configure" the Android framework by pointing at the manifest, it apparently completely ignores that we already have a complete Android project (with Gradle build files) in the android/ folder.
I would recommend using Intellij IDEA as the IDE for developing Flutter apps (https://flutter.io/intellij-setup/). You can definitely use Android Studio, but I would open the project by importing android/build.gradle instead of pointing it at the AndroidManifest.xml file (should happen automatically if you open the android/ folder instead of the main project folder). If you import it as a Gradle project, the icons resolve fine. You won't get all the Fluttery goodness (like hot reload) without the Flutter plugin, though.
flutter run and flutter build apk build the app using Gradle.
cc @mit-mit to see what we can do to improve the Studio experience.

Looking at the IntelliJ IDEA Install setup, it only has macOS and Linux.  I
assume it also works with Windows?
On Thu, May 11, 2017, 3:48 AM Jakob Andersen<EMAIL_ADDRESS>wrote:

These all look like issues with Android Studio. It doesn't have the
Flutter plugin, so it doesn't know what FLUTTER_MODULE_TYPE is, and when
you "configure" the Android framework by pointing at the manifest, it
apparently completely ignores that we already have a complete Android
project (with Gradle build files) in the android/ folder.
I would recommend using Intellij IDEA as the IDE for developing Flutter
apps (https://flutter.io/intellij-setup/). You can definitely use Android
Studio, but I would open the project by importing android/build.gradle
instead of pointing it at the AndroidManifest.xml file (should happen
automatically if you open the android/ folder instead of the main project
folder). If you import it as a Gradle project, the icons resolve fine. You
won't get all the Fluttery goodness (like hot reload) without the Flutter
plugin, though.
flutter run and flutter build apk build the app using Gradle.
cc @mit-mit https://github.com/mit-mit to see what we can do to improve
the Studio experience.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/9944#issuecomment-300753467,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUb74y_tORidpXwE0hdrbyXlDq5KiFks5r4udigaJpZM4NWD56
.

Yes, you can definitely use IntelliJ IDEA on Windows also. I'll get that page updated!

Another place where it only refers to Mac and Linux, not Windows:
"Getting Started with Flutter"
"Before you begin"
On Thu, May 11, 2017, 6:35 AM Michael Thomsen<EMAIL_ADDRESS>wrote:

Yes, you can definitely use IntelliJ IDEA on Windows also. I'll get that
page updated!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/9944#issuecomment-300790875,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUb5OyUCbpZqaegZIdKni0mUySSluoks5r4w6WgaJpZM4NWD56
.

Good catch! Addressing in https://github.com/flutter/website/pull/556
On Fri, May 12, 2017 at 6:14 PM, tombirmingham<EMAIL_ADDRESS>wrote:

Another place where it only refers to Mac and Linux, not Windows:
"Getting Started with Flutter"
"Before you begin"
On Thu, May 11, 2017, 6:35 AM Michael Thomsen<EMAIL_ADDRESS>wrote:

Yes, you can definitely use IntelliJ IDEA on Windows also. I'll get that
page updated!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/9944#issuecomment-300790875,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPUb5OyUCbpZqaegZIdKni0mUySSluoks5r4w6WgaJpZM4NWD56>

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/9944#issuecomment-301120220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANAxilDqp-nc-RDIrBf_8gPNO4PcXdThks5r5IVqgaJpZM4NWD56
.

Going to close this bug as it sounds like the original issue was resolved.
The request to support more Flutter development in Android Studio is tracked in https://github.com/flutter/flutter-intellij/issues/384, feel free to "up vote" that bug.

for me this  happen when i placed generated icon from appicon.com to res directory.  generated file  name mismatch  with androidManifest.xml. renaming file  name solved the  issue.


CSS Shapes shipped in Safari and Opera

Status on CSS Shapes should be updated

Platatus does not manage status for other browsers but depends on their status tracking. It does show in-development in webkit status, probably because it is partial? Also Opera didn't add status to chrome status.

Not sure why WebKit split it out into two - their implementation covers level 1 (it's only missing level 2 features). Shouldn't Platatus at least report "Developing" for WebKit instead of "No Signals"?

See https://webkit.org/status/#specification-css-shapes-level-1, there's no info.

It links to https://webkit.org/status/#feature-shape-outside-for-floats, which is "In Development" with a note saying it's shipped but with a prefix. That's what I was referring to by the "split it out into two"

Well, for example https://webkit.org/status/#specification-css-inline-layout-module-level-3 includes https://webkit.org/status/#feature-initial-letter, but https://webkit.org/status/#specification-css-inline-layout-module-level-3 is set as In Development on their website.
They should update the info on https://webkit.org/status/#specification-css-shapes-level-1 as well. Unfortunately it's hard to maintain data for all the browsers, this is why we rely on their data.

#314 might be able to address this as caniuse has correct data for this scenario and probably other edge cases as well where Opera or Webkit aren't represented.

I've filed https://github.com/GoogleChrome/chromium-dashboard/issues/265 for Opera.


Fix reading metrics will always get stuck in some cases

Motivation
When javaInstanceRunnable has not been initialized yet, and client calls FunctionsStatsGenerator#generate, the NPE will be triggered.
Since only IOException is caught here, the client's request will be blocked all the time.
The following unit tests are affected and often fail to execute:
org.apache.pulsar.io.PulsarFunctionE2ETest#testReadCompactedSink
Modifications
Catch all exceptions

This fix is okay for me
Be can we also fix the NPE ?
do you have a stacktrace ?
if we do not fix the NPE now we will forget about it
a NullPointerException is always a bug and must be fixed.

This situation will occur when Function startup is not completely finished.But at this time there is a client request.
Even if NPE is not thrown, other exceptions must be thrown.
The status received by the client is all 500, there is no big difference.


Add asserts checking non-set commands: addresses #411

The changes that are included in the PR.

Addresses #411
Added assertType for commands that are not applicable for SET Data structures.

Design document, if any.

Commands related to STACKINT, Queue, BITs, GET were modified to return consistent messages.

Information on any implementation choices that were made.

Evidence of sufficient testing. You MUST indicate the tests done, either manually or automated.

existing test cases pass + added new test cases to check this behaviour.

Hi @JyotinderSingh , @lucifercr07
I have updated most of the commands to handle the error messages that did not return the wrong error type. I have updated the set data encoding (strings and ints, like Redis).
Please review and share your views. If everything looks good, please merge.
Thanks,
Apoorv


Update equinix/cloud-provider-equinix-metal Docker tag to v3.8.1

This PR contains the following updates:

Package
Update
Change

equinix/cloud-provider-equinix-metal
minor
v3.6.2 -> v3.8.1

[!WARNING]
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

Release Notes

kubernetes-sigs/cloud-provider-equinix-metal (equinix/cloud-provider-equinix-metal)
v3.8.1
Compare Source
What's Changed

chore(deps): bump softprops/action-gh-release from 1 to 2 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/520
chore(deps): bump golangci/golangci-lint-action from 3.7.0 to 4.0.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/513
chore(deps): bump google.golang.org/protobuf from 1.31.0 to 1.33.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/521
chore(deps): bump golang.org/x/oauth2 from 0.13.0 to 0.19.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/526
chore(deps): bump sigs.k8s.io/yaml from 1.3.0 to 1.4.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/527
chore(deps): bump golang.org/x/net from 0.17.0 to 0.23.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/536
chore(deps): bump golangci/golangci-lint-action from 4.0.0 to 5.1.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/539
chore(deps): bump github.com/equinix/equinix-sdk-go from 0.32.0 to 0.38.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/541
chore(deps): bump k8s.io/klog/v2 from 2.100.1 to 2.120.1 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/504
chore(deps): bump golangci/golangci-lint-action from 5.1.0 to 6.0.1 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/547
fix: enable equinix-sdk-go debugging when PACKNGO_DEBUG env var is set by @ctreatma in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/551
chore(deps): bump k8s.io/apimachinery from 0.26.10 to 0.30.2 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/535
chore(deps): bump docker/build-push-action from 5 to 6 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/555
chore(deps): bump github.com/equinix/equinix-sdk-go from 0.38.0 to 0.41.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/556
chore(deps): bump k8s.io/klog/v2 from 2.130.0 to 2.130.1 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/558
fix: remove accidentally committed test files by @cprivitere in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/559
fix: handle svc objects with empty annotations by @cprivitere in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/560

Full Changelog: https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/compare/v3.8.0...v3.8.1
v3.8.0
Compare Source
What's Changed

:seedling: fix multiple CVEs by @cprivitere in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/480
Bump google.golang.org/grpc from 1.49.0 to 1.56.3 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/473
chore(deps): bump actions/setup-go from 4 to 5 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/485
chore(deps): bump golang.org/x/crypto from 0.14.0 to 0.17.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/487
fix: hook the EMLB client up to our debug env var by @ctreatma in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/479
chore(deps): bump github.com/hashicorp/go-retryablehttp from 0.7.4 to 0.7.5 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/492
chore(deps): bump github.com/packethost/packngo from 0.30.0 to 0.31.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/488
chore(deps): bump github.com/google/uuid from 1.3.1 to 1.5.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/489
chore: removed packngo and used metal-go apis by @aayushrangwala in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/486
chore: update go module path by @cprivitere in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/438
chore(deps): bump actions/cache from 3 to 4 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/503
chore(deps): bump github.com/google/uuid from 1.3.1 to 1.6.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/507
chore(deps): bump github.com/equinix/equinix-sdk-go from 0.30.0 to 0.32.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/506
:bug: avoid panic by using GetMd5() instead of Md5.Get by @cprivitere in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/508

Full Changelog: https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/compare/v3.7.0...v3.8.0
v3.7.0
Compare Source
This release adds private beta support for Equinix Metal's upcoming Load Balancer service.
What's Changed

Bump golangci/golangci-lint-action from 3.4.0 to 3.5.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/418
doc: Update docs to be inline with templates by @cprivitere in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/417
Bump golangci/golangci-lint-action from 3.5.0 to 3.6.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/421
Bump github.com/hashicorp/go-retryablehttp from 0.7.2 to 0.7.4 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/435
Bump github.com/packethost/packngo from 0.29.0 to 0.30.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/434
Bump k8s.io/klog/v2 from 2.90.1 to 2.100.1 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/432
Update Readme to suggest v3.6.2 for the CCM deployment by @aayushrangwala in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/437
Bump golangci/golangci-lint-action from 3.6.0 to 3.7.0 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/441
Bump actions/checkout from 3 to 4 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/452
Bump docker/setup-buildx-action from 2 to 3 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/454
Bump docker/login-action from 2 to 3 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/455
Bump docker/build-push-action from 4 to 5 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/456
Bump docker/setup-qemu-action from 2 to 3 by @dependabot in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/453
refactor: cleanup of cpem by @cprivitere in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/463
feat: add beta SDK for Equinix Metal Load Balancer service by @ctreatma in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/471
feat: add support for Equinix Metal Load Balancer by @ctreatma in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/470
fix: return a valid LoadBalancerStatus for BGP-based load balancers by @ctreatma in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/474
fix: use the correct external port for control plane LBs by @ctreatma in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/475
fix: use quay.io registry in deployment.yaml by @ctreatma in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/478

New Contributors

@aayushrangwala made their first contribution in https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/pull/437

Full Changelog: https://github.com/kubernetes-sigs/cloud-provider-equinix-metal/compare/v3.6.2...v3.7.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (v3.8.1). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Spellcheck doesn't work with node 4.x.x

see spellcheck package listed here:
https://github.com/nodejs/node/issues/2798
npm install of spellcheck on node 4.1.2 gives me node-gyp errors as described in that nodejs issue

get the same issue as well.

Nodehun seems to be a good alternative that works with newer node versions.


Incorrect FEN gets printed after clicking a new computer game

Describe the bug
Well I am kind of a python expert(1 year experience) so I know how things work. When I start my second bot match, things don't work right. The bot starts moving random pieces to incorrect locations(the piece cannot technically move there) and as a result, the bot fails to continue. When I check what the problem was, I was surprised! It was not the bot, but the page html that caused the issue. Basically in the 2nd bot match, instead of (class=piece wp square-22), some pieces are (class=piece square-22 wp). This is just an example of what happens to some pieces. So when you take (in get_fen.py) piece_name = piece.get_attribute("class").split()[1], the first word is not wp anymore, it is square! Further, piece_name[1].upper() the second letter here is q (s>q<uare), that's right, the piece is read as a queen instead of a pawn. I have attached a screenshot of this. This change of the position of the word doesn't appear to affect the chess pieces in chess.com, it appears to be correct. Its just that get_fen is reading it wrong. I actually also found out a working fix for this.
Replace This:
piece_name = piece.get_attribute("class").split()[1]     (LINE 13)
With This:
if piece.get_attribute("class").split()[1][0] != "s":
piece_name = piece.get_attribute("class").split()[1]
else:
piece_name = piece.get_attribute("class").split()[2]
fen += piece_name[1].upper() if piece_name[0] == "w" else piece_name[1].lower()
This also doesn't cause any new bugs
To Reproduce
Steps to reproduce the behavior:

Go to Vs Computer in Chess.com
Complete first bot game
Play new game (without refreshing browser)
See error

Expected behavior
Wrong FEN gets printed and bot fails
Screenshots

Desktop:

OS: Windows 10 Home Single Language
Browser Microsoft Edge
Version 91.0.864.48

Thanks for submitting this issue. Are you able to make a PR directly? If not I'll do one tomorrow when I've got time. Cheers!


Make image_pyramids "Support"

This should not be merged before https://github.com/hubmapconsortium/portal-ui/pull/1690 is in place. The umbrella issue is https://github.com/hubmapconsortium/portal-ui/issues/1576. Filing as draft for now: @ilan-gold , if you have thoughts?

@john-conroy -- I'm going to start looping you in on indexing changes. This PR is tiny, but please let me know if you have questions about the bigger picture.

@yuanzhou -- Ping me when the results of a reindex are on TEST, or let me know if you have other concerns? No rush. Thanks!

@mccalluc I was gonna merge and redeploy on TEST with reindex, but we are still waiting for @ilan-gold 's approval.

@yuanzhou Our PR custom is that a "comment" is as good as an approval.

@mccalluc the changes have been deployed on TEST and reindex went well

@yuanzhou -- Great! We need to do a little bit of UI work to catch up to this. I'll post an update here and on slack.


Production build can't find module lightning

I suspect this is caused by #1521 given lib/lnd.js didn't exist before then.
This is going to be tricky to reproduce in dev, but it'll require getting an npm run build going then attempting to run it.
Nov  6 01:57:31 ip-172-31-81-59 web: ⨯ Error: Cannot find module 'lightning'
Nov  6 01:57:31 ip-172-31-81-59 web: Require stack:
Nov  6 01:57:31 ip-172-31-81-59 web: - /var/app/staging/lib/lnd.js
Nov  6 01:57:31 ip-172-31-81-59 web: at Module._resolveFilename (node:internal/modules/cjs/loader:1140:15)
Nov  6 01:57:31 ip-172-31-81-59 web: at /var/app/current/node_modules/next/dist/server/require-hook.js:55:36
Nov  6 01:57:31 ip-172-31-81-59 web: at Function.resolve (node:internal/modules/helpers:188:19)
Nov  6 01:57:31 ip-172-31-81-59 web: at c (/var/app/current/.next/server/chunks/4926.js:3935:447)
Nov  6 01:57:31 ip-172-31-81-59 web: at /var/app/current/.next/server/chunks/4926.js:1936:2990 {
Nov  6 01:57:31 ip-172-31-81-59 web: code: 'MODULE_NOT_FOUND',
Nov  6 01:57:31 ip-172-31-81-59 web: requireStack: [ '/var/app/staging/lib/lnd.js' ]
Nov  6 01:57:31 ip-172-31-81-59 web: }

Wild guess, but it's related to relative paths which are different in the production build?

That would make sense. I just don't know where the relative path is coming from.

This is due to the way the code gets the absolute path of the protobuf files
  const lightningModulePath = req.resolve('lightning')
  const pathForProto = proto => join(lightningModulePath, protosDir, proto)

likely webpacks does some module mangling in production to remove unused code and somehow we end up without this module that is needed at runtime.
Imo we need to revert back to the privoxy workaround or just for the ln-service and lightning repos

I've come up with a workaround https://github.com/stackernews/stacker.news/pull/1548 , but i can't test it fully in the dev environment.


How to read binary file ?

Thanks for the great package. For now, on README, I only see excelize can read file from xlsx file, but in my application, I'm storing excel file as binary (blob) in MySQL. Is that possible to read excel file from the blob of MySQL with excelize ?

You can use the OpenReader function to extract an excel file from any reader. You can use a buffer to pass your blob through as a reader.
That said, is it strictly necessary for you to store excel files as blobs in a database like that? You might have an easier time storing the location of the files in your database and using some other persistent storage solution (even just a location on local disk if your application is simple enough) to store the actual files.

Thanks for your quick response. So basically, Instead of OpenFile, I can use OpenReader to solve my problem.
We thought about storing on local disk or AWS S3 solution but with our application, storing excel files in MySQL is the best way.
Again, thanks for your support.

I had a similar use case:
I have a very large excel file coming in through a web-socket connection as a blob ( []byte ) .
I needed to decode the blob into an excel file.
        messageType, FileDataInBytes, err := conn.ReadMessage()
	excelFile, err := excelize.OpenReader(bytes.NewReader(FileDataInBytes))

OpenReader read data stream from io.Reader and return a populated spreadsheet file.

This implementation gives me an easy way to decode blob to an excel.

Great Package, Thank you :)


Using OCMArg setTo generates an error Implicit conversion

Xcode 12.3
OCMock 3.8.1
platform :ios, '14.3'
inhibit_all_warnings!

target 'iOSUnitTestExample' do
  target 'iOSUnitTestExampleTests' do
    inherit! :search_paths
    pod 'OCMock'
  end
end

    id mockURLSession = OCMClassMock([NSURLSession class]);
    OCMStub([mockURLSession sharedSession]).andReturn(mockURLSession);
    NSURLRequest *req = [NSURLRequest new];
    OCMStub([mockURLSession dataTaskWithRequest:[OCMArg setTo:req] completionHandler:([OCMArg checkWithBlock:^BOOL(id block) {
        return YES;
    }])]);

Implicit conversion of an indirect pointer to an Objective-C pointer to 'NSURLRequest * _Nonnull' is disallowed with ARC
Incompatible pointer types sending '__autoreleasing id *' to parameter of type 'NSURLRequest * _Nonnull'

I think you're misunderstanding what setTo: does. It's used in cases where a value is returned via a pointer passed as an argument, e.g. some methods in Apple's frameworks take an NSError ** argument to return an error. This does not seem to be the case for dataTaskWithRequest:completionHandler:, which actually takes a request as its first argument.


Automated cherry pick of #6737: ctyun sync host wire fix

Cherry pick of #6737 on release/3.1.
#6737: ctyun sync host wire fix

Codecov Report

Merging #6738 into release/3.1 will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           release/3.1   #6738   +/-   ##
===========================================
  Coverage         7.48%   7.49%
===========================================
  Files              585     585
  Lines            98953   98954    +1
===========================================
+ Hits              7411    7420    +9
+ Misses           90970   90961    -9
- Partials           572     573    +1

Flag
Coverage Δ

#aFlag
7.49% <0.00%> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/cloudcommon/policy/resources.go
0.00% <ø> (ø)

pkg/cloudprovider/objectstore.go
2.51% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/guestdrivers/managedvirtual.go
0.00% <0.00%> (ø)

pkg/compute/models/buckets.go
1.24% <0.00%> (ø)

pkg/compute/models/guestdisks.go
0.58% <ø> (+0.01%)
:arrow_up:

pkg/compute/models/secgroups.go
1.90% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/snapshots.go
1.56% <0.00%> (+<0.01%)
:arrow_up:

pkg/hostman/hostinfo/hostinfo.go
0.00% <0.00%> (ø)

pkg/hostman/isolated_device/isolated_device.go
3.22% <0.00%> (-0.06%)
:arrow_down:

pkg/util/rbacutils/policyset.go
75.00% <0.00%> (+9.78%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Use PathComponents as parameters

Parameter and PathComponents are very clever, so it's a shame we need to resort to strings to use them. With this PR you can define things like:
// In routes.swift for example
extension PathComponent {
    static let api: Self = "api"
    static let publicApi: Self = "public-api"
}

// In TodoController.swift
extension PathComponent {
    static let todo: Self = "todo"
}

fileprivate extension PathComponent {
    static let todoId: Self = .parameter("todoId")
}

and then
func boot(routes: RoutesBuilder) throws {
    let tokenProtected = routes.grouped(.api, .todo).grouped(UserToken.authenticator().middleware(), User.guardMiddleware())
    tokenProtected.get(.todoId, use: getHandler)
}

func getHandler(req: Request) throws -> EventLoopFuture<Todo> {
    Todo.find(req.parameters.get(.todoId), on: req.db)
        .unwrap(or: Abort(.notFound))
}

But because of StringLiteralConvertible you can also still do this if you really want to:
func getHandler(req: Request) throws -> EventLoopFuture<Todo> {
    Todo.find(req.parameters.get("todoId"), on: req.db)
        .unwrap(or: Abort(.notFound))
}

Checklist

[x] Circle CI is passing (code compiles and passes tests).
[x] There are no breaking changes to public API.
[ ] New test cases have been added where appropriate.
[x] All new code has been commented with doc blocks ///.

Sorry for the confusion, I actually added back the stringly version because because although a string would be converted into a PathComponent on account of it being StringLiteralConvertible, that conversion would turn "userId" into .constant("userId") and not .parameter("userId"), which is what I'm testing for. So I added back the string version. If you then call get with a string it prefers the version that takes an actual string, rather than a string literal convertible.
If I remove the requirement that it must be a .parameter it would fail in a different way (apart from being philosophically dubious), since it would not just work with "userId", it would also work with `constant("userId").
For the record I think it is unfortunate that we use slightly different syntax for the route definition and for getting the parameters, but I suppose it's only really an issue when you start thinking in terms of PathComponent.


Auto generate content for VNext Preview feature

Fix #5619
As we ship website builds out of main we need to clearly indicate to users on Schema Explorer and the Designer which features are vNext preview feature and "not final"

if you want to mark some version as preview in schema explorer, you just need to change style like

.ac-schema-version-1-6::after
  content: "Preview";
  color: White;
  background-color: blue;
  padding: 0px 4px 4px 4px;
  margin: 1px;
  font-size: 120%;
  border-radius: 4px;

maxSupportedVersion is introduced. latest <= maxSupportedVersion. if any class or property is over latest version number, then it's assumed the preview version in designer.

Because /explorer heavily depends on markdown, image instead of css is used to mark the 'preview'.
Below is screenshot which assume 1.5 is the preview feature.

Designer warning when preview property is used but target version is less than preview version

Designer target version

Below is screenshot which assume 1.1 is the preview feature.

document on /explorer site

Microsoft Reviewers: Open in CodeFlow

Looks good!


Get ApiContractViolation error when authenticating with my corp account using the new broker

Does this issue occur when all extensions are disabled?: Yes/No

Version: 1.96.0-insider (user setup)
Commit: 0c9dd26823dc38dd59d6240e1b6698add43d942c
Date: 2024-11-18T05:05:45.838Z
Electron: 32.2.3
ElectronBuildId: 10561341
Chromium: 128.0.6613.186
Node.js: 20.18.0
V8: 12.8.374.38-electron.0
OS: Windows_NT x64 10.0.22631
Steps to Reproduce:

In extension code, we use the following lines to acquire a token for our app

const scopes: string[] = [
        `VSCODE_CLIENT_ID:${ourFirstPartyClientAppId}`,
        `VSCODE_TENANT:common`,
        `${ourFirstPartyServerAppId}/.default`
    ];

    try {
        const session = await vscode.authentication.getSession("microsoft", scopes, { createIfNone: true });
        if (!session) {
            throw new NotSignedInError();
        }
        return session.accessToken;
    } catch (error) {
        ext.outputChannel.error('Failed to get a Microsoft access token', error);
        throw error;
    }

When getSession is called, VS Code prompts for the user to sign in and then shows the auth broker's account picker. I selected my MSFT corp account and for some reason I had to select it twice. After selecting it twice, an error is thrown and is caught by our catch block. The error is NativeAuthError: ApiContractViolation: (pii)

This is likely because you have not added the Broker redirect uri to your app's configuration. You should add another public client redirect uri which is in the form:
ms-appx-web://Microsoft.AAD.BrokerPlugin/<your-client-id>

Let me know if this fixes it for you.

We added the redirect uri and it's working now. Closing this issue.


Fix "CheckedExceptionMapperTest.testThrowException"

Many ways to wrap a null application. So make sure we don't try to access it's classes...

https://github.com/apache/tomee/commit/abd5fae5bac2d79b14c281cd4bf9f0ef65bde04a


Refactor 'destination' configuration for consistency across Fleet and Application

What would you like to be added:
Refactor 'destination' configuration for consistency across Fleet and Application
Why is this needed:
In our current implementation, we have a 'destination' configuration that behaves differently in Fleet and Application. In Fleet, it allows specifying a cluster, while in scenario Application, it allows targeting a group of clusters with a specific label.
This discrepancy can lead to user confusion as the goal is the same—to designate a target destination for a task—but the method differs.
To address this, I propose we refactor the 'destination' configuration to support both settings methods—specifying a single cluster or a group of clusters with a tag—in both  Fleet and Application.

/assign

What do you think and how we can better design the target clusters

What do you think and how we can better design the target clusters

I think we can support both settings methods in any crd


Bugs/auth

Fixes admin vulnerability.

Codecov Report

Merging #416 into feature/email will increase coverage by 0.05%.
The diff coverage is 100%.

@@                Coverage Diff                @@
##           feature/email     #416      +/-   ##
=================================================
+ Coverage          75.98%   76.04%   +0.05%
=================================================
  Files                 73       73
  Lines               1649     1653       +4
=================================================
+ Hits                1253     1257       +4
  Misses               396      396

Impacted Files
Coverage Δ

api/src/controllers/user.js
82.14% <100%> (+0.66%)
:arrow_up:

api/src/controllers/auth.js
98.24% <100%> (+0.06%)
:arrow_up:

Continue to review full report at Codecov.

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


fromElement()

Have you considered a fromElement() function that would take an existing SVG shape element, and get the attributes needed from there?

That's an interesting idea. Can you give me a use case? And do you think you'd pass a DOM Node or something else?


[Bug] Error in login page after install

Describe the bug
The admin login page doesn't work after a fresh install.

Make sure to follow the instruction in the readme.md or download the preview version

The build with gulp is also showing errors:
[19:53:41] Requiring external module babel-register
TypeError: Cannot read properties of undefined (reading 'replace')

I copied the preview template for now. But gulp is very outdated.


JS

JS
Use Case
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptable Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

JS Added


Update tapir-core, tapir-json-circe, ... to 0.20.0

Updates

com.softwaremill.sttp.tapir:tapir-core
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-openapi-circe-yaml
com.softwaremill.sttp.tapir:tapir-openapi-docs
com.softwaremill.sttp.tapir:tapir-zio
com.softwaremill.sttp.tapir:tapir-zio-http4s-server

from 0.16.16 to 0.20.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

labels: library-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #190.


Merged API into Main

Refresh of device control repo

Reorganized and cleaned up
Generated documentation for samples
Added Getting Started for Windows


Fix inline-notification script import

What I did

Fix inline-notification script import

Thanks
cc @daKmoR maybe consider adding eleventy-plugin-add-web-component-definitions


Improve responsiveness and style

issue - #15
improved responsiveness on galaxy fold
removed margin-left to align items to the center
instead using justify-content: center;
removed package lock.json (it is not necessary)

Hey, can you add the label 'hacktoberfest-accepted', it will let their algorithm detect this PR.
Thanks


Proposal: Switch to pre-commit.ci service instead of custom pre-commit jobs

I think it is actually better to switch to the established pre-commit.ci service.
It will run our lint job, and do periodic update PRs for the .pre-commit-config.yaml config.
Ref: https://pre-commit.ci/

There is two thumbs up, which I guess means "yes, let's do it". This has to be done by someone with higher privileges.  I can add the service only to my own repositories.
After you added it, we can configure it in .pre-commit-config.yaml.
It can generate an update PR, 'weekly', 'monthly' or 'quarterly'. I think monthly is a solid choice, between being up-to-date and having too much noise.
Also the service can "autofix" PRs, if you like. I am not a fan of that feature, I prefer to do that locally and have CI tell me if there is something wrong. But this is of course up for discussion.
Ref: https://pre-commit.ci/#configuration

I've added the pre-commit.ci in here https://github.com/rdmorganiser/rdmo/pull/1166 and included the config as you mentioned. You think it looks good?

Once #1166 is merged. You can delete the custom lint job.
https://github.com/rdmorganiser/rdmo/blob/1cdcf9ea6c28c6e4398691ce3093726e79ae2dfb/.github/workflows/ci.yml#L37-L39


Getting a tree of all items under an item

What is the best practice of getting the full tree of items below an item?
The documentation says  "By using these request builders you can retrieve all of the items under a collection no matter the size." But how exactly should this be done? The example is incomplete in the call to .get() within success().
Is there an example of a sequence of requests? What should be used to wait until the result of a request is available before issuing the next request?

The best way to get the full hierarchy of items within a OneDrive is to use the Sync Changes API, this can be done with an IDeltaRequest.  To see a full example of how to retrieve a delta collection and make requests to get all pages see the OneDrive Explorer for Android's DeltaFragment
Here is truncated example, which would work with any result of a CollectionPage.  We create the request against the resource (the logic within refresh()), then for the ICallback<*Page> handler we use a function to generate the next handler, then within the handler if we have another page we make a recursive call to get the next page
    private void refresh() {
        oneDriveClient
            .getDrive()
            .getRoot()
            .getDelta("deltaToken")
            .buildRequest()
            .get(pageHandler());
    }

    private ICallback<IDeltaCollectionPage> pageHandler() {
        return new ICallback<IDeltaCollectionPage>() {
            @Override
            public void success(final IDeltaCollectionPage page) {
                // Be productive with page.getCurrentPage()

                // Handle scenario with multiple pages
                if (page.getNextPage() != null) {
                    page.getNextPage()
                        .buildRequest()
                        .get(pageHandler());
                }
            }
        };
    }

Thanks! Unfortunately the sample app crashes on my Windows emulator with a hierarchy of 50-60 items.
Tracking the response down in the sample code leads to mPageContents in BaseCollectionPage. The interface IBaseCollectionPage, which BaseCollectionPage implements, involves - indirectly - a JsonObject, which in turn maintains a LinkedTreeMap of its children. Is it correct that mPageContents is a flat list representation of the tree in the JsonObject?

@sw-dev1 Could you create a new issue a gist of the app crashing along with the devices logcat on the sample application?
Yes, mPageContents list representation of the item tree element in the json response, which is fully described here.

Thanks again for the clarification.
The crash happens when testing against a live account, which would contain personal data in the logcat. I tried to reproduce the behavior with a test account having fewer items. The app does not crash but the emulator screen freezes and scrolling is not possible. Back button works though. Would you like me to create an issue with this?

@sw-dev1 Without a log of what is happening on the device during the crash I won't be able to address this issue.  If you can a log and you can share it to me privately or if there is an exception stack trace then I can work this issue.

I could recreate the crash with the first account and then extracted the necessary (in my view) part of the logcat that does not contain personal data. As requested, I have then created an issue for [onedrive-explorer-android] (https://github.com/OneDrive/onedrive-explorer-android):  [App crash on windows emulator at view.delta with large hierarchy] (https://github.com/OneDrive/onedrive-explorer-android/issues/63). Hoping it helps.

I'm going to close out this issue, as it is a bug in the sample application not this SDK, thanks for filing!

@peternied Thanks for your help so far.
Next challenge is to use the collected list of items from the delta pages. It seems that the SDK design imposes that the complete logic for manipulating the result of all requests needs to be in the success method of the callback. In this case however, due to the recursive calls of the request from within success, this would make the success method very complex and difficult to test.
While looking for a solution I thought AsyncMonitor would be a way  to go. However get() in IBaseDeltaRequest accepts only  and not <AsyncMonitor<>> as the template parameter.
Would there be a similar facility in the SDK for a delta request?
If not, would wait() and notify() or something around synchronize be recommendable?
Cheers.

@peternied Thanks for your help.
Next challenge is to use the collected list of items from the delta pages. It seems that the SDK design imposes that the complete logic for manipulating the result of all requests needs to be in the success method of the callback. In this case however, due to the recursive calls of the request from within success, this would make the success method very complex and difficult to test.
While looking for a solution I thought AsyncMonitor would be a way to go. However get() in IBaseDeltaRequest accepts only <"IDeltaCollectionPage> and not <"AsyncMonitor<"IDeltaCollectionPage>> as the template parameter.
Would there be a similar facility in the SDK for a delta request?
If not, would wait() and notify() or something around "synchronize" be recommendable?
Cheers.

Well you could use a SimpleWaiter to wait for a response without any pages and then trigger another flow that uses that information, or you could use a second layer of callbacks that aggregated list of page contents.  I'm thinking that second method would work a little better for this purposes.
You could also look to add an extension within DeltaRequest that would let you get the kind of response you want with a pattern like the AsyncMonitor.
    // Example using an nested callback
    private void refresh() {
        final MyCallback callback= new MyCallback() {
            public final List<Item> mItems = new List<Item>();
            public void totalSuccess() { }
            public void anyFailure(...) {}
        };
        oneDriveClient
            .getDrive()
            .getRoot()
            .getDelta("deltaToken")
            .buildRequest()
            .get(pageHandler(callback));
        waiter.waitForSignal();
    }

    private ICallback<IDeltaCollectionPage> pageHandler(final MyCallback myCallback) {
        return new ICallback<IDeltaCollectionPage>() {
            @Override
            public void success(final IDeltaCollectionPage page) {
                // Be productive with page.getCurrentPage()
                myCallback.mItems.Add(page.getCurrentPage())

                // Handle scenario with multiple pages
                if (page.getNextPage() != null) {
                    page.getNextPage()
                        .buildRequest()
                        .get(pageHandler(callback));
                } else {
                    myCallback.totalSuccess();
                }
            }
        };
    }

Thank you. I'll try this out.


Artist Invites: Card Height / Description Tweaks

https://www.pivotaltracker.com/story/show/150095755

          1 Message

:book:
Nice, more deletions than insertions :red_circle:

  Generated by :no_entry_sign: Danger

@alanpeabody Sorry, this is still wip but @ellodanger added the please review label – how can avoid that? Should I just wait until I’m ready to start a PR?

@gblakeman if you add WIP to the PR title it won't add the needs review.

@alanpeabody Ah, gotcha. 👌 I’d been adding the label, but not in the title.

:+1:

@alanpeabody I changed my approach to this a bit, so you might want to take a peak again. Good news is that’s all CSS-based changes with a tiny bit of changed markup.


Add support for m4b format

Currently, polaris does not recognize audiobook files in m4b extension. The format could be added easily though, as M4B is essentially an MP4 audio file.

Thank you!


Charts[Gateway] Add Helm Chart Tests For Gateway API Routes

Need to create helm chart tests for all gateway api routes as below:

haproxy-grpc-route
network-node-grpc-route
envoy-proxy-grpc-web-route

:tada: This issue has been resolved in version 0.12.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Publicize a minimum supported Zig version?

Howdy! zigmod seems useful (and fairly simple, woohoo!), but as one who doesn't follow the nightlies of Zig, it looks like I'm in for a world of hurt trying to use it (or something's actually broken and I'm just attributing it to Zig versions?)
./deps.zig:55:48: error: no member named 'path' in '[]const u8'
        .pkg = Pkg{ .name = "args", .path = .{ .path = dirs._xzxo5rnug8wj ++ "/args.zig" }, .dependencies = null },
                                               ^
./deps.zig:53:9: note: referenced here
    pub const _xzxo5rnug8wj = Package{
        ^
./deps.zig:59:26: note: referenced here
        .directory = dirs._6tp2yx5wnq30,
                         ^
./deps.zig:58:9: note: referenced here
    pub const _6tp2yx5wnq30 = Package{
        ^
/usr/lib/zig/std/meta.zig:367:68: note: referenced here
pub fn declarations(comptime T: type) []const TypeInfo.Declaration {
                                                                   ^
./deps.zig:12:38: note: referenced here
    inline for (std.meta.declarations(package_data)) |decl| {

Since I see a ton of references to zig-0.9 in the changelog, I went on a whim and assumed I needed to step back through the release history to eventually find something 0.8 compatible. I gave up somewhere around v60 and decided to just not manage dependency versions with a tool for now.
It'd be super helpful to have a line in the README to the effect of "here's the last version that supported the latest stable Zig compiler series" or so - similar to Gyro and a couple other tools I've seen.
Cheers!

the version in the readme is the minimum version as I tend to only update that line when i need to update zigmod code after something upstream broke it

Is there a plan to one day start tracking upstream's stable release channel in any awy? As it stands it looks like this pushes a "use Zig nightlies" requirement downstream which is a bit unfortunate (even if I understand why it's currently that way).

i could potentially add a flag that prints release-compatible output if there's want for it. zigmod won't be around by the time zig reaches 1.0 so if its useful for people now im not super against it
generally speaking, zigmod also pushes for using the nightly version of libraries (which currently manifests itself in keeping up with the language changes) but its a pattern i'm personally going to continue going forward. in conjunction with Stability Bit Versioning of aforementioned libraries.
this is shown in the package index ive been working on https://aquila.red/ which implements this and does not support using semver

I wouldn't feel right asking you to do this for just me when I've found a workaround in the meantime - if other folks comment here, maybe it'll be worth reopening this discussion, but otherwise, I respect your stance (and appreciate the link to that talk - it's an intriguing way to look at versioning, so thank you!)
cheers!

Zigmod currently works with the 0.9.0 release

I am trying to get zigmod into homebrew-core, but I found it not working with the stable zig 0.10.1, should I hack the min zig version requirement to see how the build works?

zigmod HEAD is meant to work with zig master, https://github.com/nektro/zigmod/releases/tag/r84 will work with 0.10.1 though

I did a quick build but got the following failures
==> zig build --prefix /opt/homebrew/Cellar/zigmod/84 -Drelease-fast=true
thread 3266281 panic: Your Zig version v0.10.1 does not meet the minimum build requirement of v0.11.0-dev.193+eed82ca28
/private/tmp/zigmod-20230223-12812-j2zsn0/zigmod-r84/deps.zig:147:64: 0x10295340b in checkMinZig (build)
    if (current.order(min).compare(.lt)) @panic(exe.builder.fmt("Your Zig version v{} does not meet the minimum build requirement of v{}", .{current, min}));
                                                               ^
/private/tmp/zigmod-20230223-12812-j2zsn0/zigmod-r84/deps.zig:85:24: 0x10292b893 in addAllTo (build)
    checkMinZig(builtin.zig_version, exe);
                       ^
/private/tmp/zigmod-20230223-12812-j2zsn0/zigmod-r84/build.zig:24:18: 0x1028ee9b3 in build (build)
    deps.addAllTo(exe);
                 ^
/opt/homebrew/Cellar/zig/0.10.1/lib/zig/build_runner.zig:232:28: 0x1028dfb37 in runBuild (build)
        .Void => root.build(builder),
                           ^
/opt/homebrew/Cellar/zig/0.10.1/lib/zig/build_runner.zig:214:17: 0x1028dc53b in main (build)
    try runBuild(builder);
                ^
/opt/homebrew/Cellar/zig/0.10.1/lib/zig/std/start.zig:614:37: 0x1028dff0b in main (build)
            const result = root.main() catch |err| {
                                    ^
???:?:?: 0x199c83e4f in ??? (???)
???:?:?: 0x3208ffffffffffff in ??? (???)

https://github.com/nektro/zigmod/commit/8f01102e1f1cc122ee478528bcf8c11125aa018c will work

let me give that a try.

had some issues of accessing this resource
git https://gist.github.com/nektro/d468fea84f8217e4c26ee8fbeeea38cc commit-24c01e844396d18a90bbbfb91f8a5a12023e9379

that's #90

just pushed https://github.com/nektro/d468fea84f8217e4c26ee8fbeeea38cc as backup


Using gems in _config.yml is now deprecated

This is probably just a documentation issue, but with Jekyll 3.5.0, using gems in _config.yml is deprecated in favor of plugins. A warning is thrown when Jekyll runs:
Deprecation: The 'gems' configuration option has been renamed to 'plugins'. Please update your config file accordingly.
See https://github.com/jekyll/jekyll/pull/5130 for background.

Thanks for the heads up!


Dataset title appears as "GitHub Supplement" in cellxgene explorer

See wrangling thread for details and link to private CXG.
This does not reproduce on staging and may be another variation of Dataset title appears as withlabels.local in explorer.

Closing as a duplicate.


Send HNW text changes to production

👷 Deploy Preview for bongosian-press processing.

Name
Link

🔨 Latest commit
81be4ca3e38452a3079ddcade93924938d8120b5

🔍 Latest deploy log
https://app.netlify.com/sites/bongosian-press/deploys/65ff628b250ce60008e07255


Add option to only run diagnostics on file save.

On large files, Merlin can take 30+ seconds to check for errors, which leads to situations where the user pauses while typing, triggering a long diagnostics check, and then finishes their line and hits save shortly after, which leads to a 29 second pause before the actual error check on the correct file contents can run.

Thanks for the PR.
I agree that in cases where Merlin is taking too long this is a problem. However, unless I'm missing something, it doesn't seem like we really need special handling for this; can't the user just set a high value for reason.debounce.linter in their preferences? In fact, I think this is the reason I never got around to making a separate preference for diagnostics only on save.
It doesn't seem like you can use Infinity there, which would be ideal, but you can still set the delay to something like 1 hour (3600000) which should case the diagnostics to never trigger in practice while the user is editing, except on save.
Perhaps this should be made clearer in the documentation.

Actually it looks like "Infinity" (with quotes) does work as expected although the preferences panel gives a warning about it being an incorrect type.

I've updated the vscode-reasonml settings spec so that it no longer complains if "Infinity" is used instead of an integer.

The string "Infinity" isn't a great value to indicate that in Atom. The reason is that Atom provides the ability to annotate config variables with types so that they're rendered more appropriately to the user for editing. It would be a shame if we had to change this config value to a string from an int and lose some sanity checks in the Atom configuration for this package. Maybe a value like 0 or -1 would be better to indicate infinity?

The way it works now is that there are two types of values that are allowed for that setting, an integer or specifically the string "Infinity". This is enforced by the JSON schema spec for the settings, which you can see here.
I was under the impression that Atom is also using JSON schema for the settings. Is that not the case?

Just to be clear, the main reason I was suggesting for using "Infinity" is because it doesn't require any changes to the logic of the server. "Infinity" will be parsed as the JS number value Infinity. If we used something else, we wouldn't get that for free.

Indeed, it looks like Atom is also using JSON schema based on this.

Okay, that would make sense too.  One issue we observed is that Nuclide wouldn't send the save event unless we used the rich value for textDocumentSync instead of just the enum, but I don't know enough about LSP internals to know if that's a problem with Nuclide or if that's working as expected.  Sounds like the VS Code plugin will always send save events?

It doesn't seem to work in Nuclide to use oneOf, for some reason. This setting works:
      "debounceTime": {
        "title": "Testing debounce time config (ms)",
        "type": "integer",
        "default": 200
      },

and it renders an item in the config, but not this one:
      "debounceTime": {
        "title": "Testing debounce time config (ms)",
        "oneOf": [
          {"type": "integer"},
          {"enum": ["Infinity"]}
        ],
        "default": 200
      },

This could be an Atom- or Nuclide-specific issue, or it could be that Atom doesn't support a lot of the JSON schema, but I'm not sure which.

One issue we observed is that Nuclide wouldn't send the save event unless we used the rich value for textDocumentSync instead of just the enum, but I don't know enough about LSP internals to know if that's a problem with Nuclide or if that's working as expected. Sounds like the VS Code plugin will always send save events?

Hmm, I'm not sure what this is about. Certainly the vscode plugin currently seems to be working as expected with regard save events, etc., as far as I can tell.
However, if using the extended capability value for textDocumentSync really makes a difference for nuclide for some reason then I think it's okay to change it. Can we split that part out into a separate PR?

After looking over the language protocol and talking internally, we think that Nuclide not sending didSave notifications unless this is set is actually a bug in Nuclide, so we'll just fix it there and use a long debounce :)

This could be an Atom- or Nuclide-specific issue, or it could be that Atom doesn't support a lot of the JSON schema, but I'm not sure which.

@arxanas I looked through the Atom config code and found this. It would appear that they rolled their own validation system for some reason but it seems to only support basic types.
I'm guessing all of the other things like requiring unique items in arrays or min/max length are not checked either. One way you could test this one would be to try using the following setting and see if it complains if you using a value of ["merlin", "merlin"].
"reason.diagnostics.tools": {
  "type": "array",
  "items": {
    "enum": ["merlin", "bsb"]
  },
  "default": ["merlin"],
  "maxItems": 2,
  "uniqueItems": true,
  "description":
    "Specifies which tool or tools will be used to get diagnostics. If you choose both \"merlin\" and \"bsb\", merlin will be used while editing and bsb when saving."
}

Anyway, it's been awhile since I've used nuclide, but even if JSON schema support is incomplete in Atom, is this something you can address in nuclide? Do you have a separate UI for nuclide config settings where you could support more of this stuff?

I'm 90% certain that the Nuclide configuration is implemented just in terms of the Atom configuration. To configure it you go to the package settings for Nuclide and find the subsection for the Nuclide package you want to configure.


[List][material-next] Add all List* components

Duplicates

[x]  I have searched the existing issues

Latest version

[x]  I have tested the latest version

Summary 💡
This issue is to track the work for adding the List* component to @mui/material-next, moving forward with the Material you effort (#29345)

[ ]  Copy component files from material (v5) to material-next: PR  (WIP)
[ ]  Remove deprecated components and componentsProps props (not applicable, the component already uses the slots & slotProps props (WIP)
[ ]  Migrate component to Typescript   (WIP)
[ ]  Ensure slots & slotProps have keys for each slot  (WIP)
[ ]  Drop support for ThemeProvider in favor of CssVarsProvider  (WIP)
[ ]  Implement Material You design specs  (WIP)
[ ]  Add component playground to v5 docs  (WIP)
[ ]  Refactor styles to use component CSS Variables  (WIP)

Note: If a task is marked as [WIP] it means someone is working on it

Thanks for taking this one @sai6855!
After we implement the Material 3 styles, it would be interesting to think about how we should share styles between the Menu and Select components; for example, see the MenuItem and Option components.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixing styles on gallery item page

Thank you for your contribution to the KodaDot NFT gallery.
👇 Do a quick check before the merge.
PR type

[x ] Bugfix
[ ] Feature
[x ] Refactoring

What's new?

[x ] #958 related

Before submitting Pull Request, please make sure:

[ x] My contribution builds clean without any errors or warnings
[ x] I've merged recent default branch -- main and I've no conflicts
[ x] I've tried respect high code quality standards
[ x] I've didn't break any original functionality
[ x] I've posted screenshot of demonstrated change in this PR

What I've done

Restyling the structure of page.
Make price chart and history chart as foldable.
Merge Price box with Actions if it's for Collecting and not the owner of NFT
When creator, showing rest buttons would be shown.

Screenshot

[x ] My fix has changed something on UI, a screenshot for others, is best to understand changes.

Added test case https://deploy-preview-1134--nftkodadot.netlify.app/rmrk/detail/7479313-7a147cba01afccbf17-GA-GENERATIVE_ASTERISM_23-0000000000000023

Hey price chart doesn't work for me

Hello @MiracleCorunus have you made any progress?
Till it's not fixed I'll put this PR as a draft.

Hey, @roiLeo @vikiival let's do a quick review, seems most of the code looks working to me!
Price chart works, seems that's why added timeout there?

Added test case https://deploy-preview-1134--nftkodadot.netlify.app/rmrk/detail/7479313-7a147cba01afccbf17-GA-GENERATIVE_ASTERISM_23-0000000000000023

hey, @MiracleCorunus how did you resolve RoiLeo's suggestions? Because I see resolved but not new commit 👀

We should test this one before merging 👀
As lot of things might changed

@MiracleCorunus can you check why charts not working in your PR?
This is test case https://deploy-preview-1134--nftkodadot.netlify.app/rmrk/detail/9624153-7c95ac83655cdc8b26-ACV04-DINOS_PLANET_57-0000000000000057

Please remove double texts, like price chart when unfolded

Remove second "history" when unfolded

Hey @MiracleCorunus we want to close on this.
I'll send you a lower bounty as you lack responsive be around and quality code.
Please update your KSM address in PR template in this issue and I can process payout.
Let's pick probably easier issue next time, seems this one was quite complex and hard for you.

This is my Kusama Wallet Address
HcEvTkvAPYEEGEndFV9sMmS3P83jdtwhwZq37GfdzA3mLrL

This is my Kusama Wallet Address
HcEvTkvAPYEEGEndFV9sMmS3P83jdtwhwZq37GfdzA3mLrL

This is my Kusama Wallet Address
HcEvTkvAPYEEGEndFV9sMmS3P83jdtwhwZq37GfdzA3mLrL

Payment sent https://kusama.subscan.io/extrinsic/10176854-4
Thank you


Separate security scan options for PR and scheduled

For trivy scan:

the scan should fail if problematic code is being added in a PR
the scan should succeed and upload sarif artifact in case of scheduled scans

For this, two different ways of invoking trivy scan should be available and configurable.
As of https://github.com/coopnorge/github-workflow-supply-chain-security-validation/pull/87, only the latter is supported even though it is used in PRs everywhere.

potentially related: https://github.com/coopnorge/github-workflow-supply-chain-security-validation/issues/90


[BUG] Error deleting CRC cluster on Windows

General information

OS: Windows
Hypervisor: Hyper-V

CRC version
PS C:\Users\crcqe> crc version
CodeReady Containers version: 2.1.0+81a5f334
OpenShift version: 4.10.6
Podman version: 3.4.4

Steps to reproduce

Start CRC from powershell
Issue crc delete -f or crc delete on a running cluster.

Expected
Cluster deleted without any errors being reported.
Actual
PS C:\Users\crcqe> crc delete
Do you want to delete the instance? [y/N]: y
failed to remove podman system connection exit status 1: time="2022-04-21T11:42:34+02:00" level=error msg="The handle is invalid."

can you put your crc config view1 here too? preset looks to be podman` ?

@gbraad preset was openshift. Will log into the machine & get the config now.


[PE-D][Tester B] Needless case sensitivity used

commands like p/ , e/, etc are needlessly case sensitive. Restricts the usability of the application.

Labels: type.FeatureFlaw severity.Low
original: Saezenn/ped#6

Can add a footnote in the UG or something, this doesn't seem very valid to me.


idea/feature - real time collaborative coding

Well, it's not really an issue. It would be cool if github had idea/cool feature ideas bucket. Anyway, I was screensharing some code with someone with whom I am working. I remembered the R kernel and thought it would be neat to do some real-time collaborative coding. There are some web editors where you share a link and code together. Is this already available? I am not familiar with the architecture of the service but it would be great for educational purpose and collaboration. Just throwing it out there.
Also, what is the difference between tmpnb and try.jupyter. Is tmpnb deprecated?

The realtime collaborative features of the notebook are ongoing, mostly motivated in https://github.com/jupyter/jupyter-drive.
tmpnb.org and try.jupyter.org are actually both running on top of tmpnb.org. https://github.com/jupyter/try.jupyter.org is really just an single static HTML page with an iframe containing tmpnb.org. As far as the tech is concerned, tmpnb (this repo), is our current instant, anonymous, ephemeral setup.

Yes, I can read that info in your docs. Please add it to the wishlist either here or in the relevant repo. Thanks. @takluyver @minrk

Which piece for the wishlist? Realtime collaboration?

This type of feature has to be enabled in the underlying container and is certainly on the radar of @Carreau both in issues and in the current roadmap.
Closing since not part of tmpnb.


[Fern's Documentation] Update "Publish a Java SDK to Maven Central" Registration Process

What can be improved
Hello, I'm opening this on behalf of the Maven Central team. We just wanted to let you know that our Account Registration and Namespace / groupId Registration processes have changed, so the fern/docs/pages/fern-sdks/guides/publish-to-maven-central.mdx documentation is no longer accurate. We got a support request by a user that mentioned that documentation, so we thought it would be a good idea to reach out.
(if you're curious about the details, we deprecated issues.sonatype.org due JIRA on-prem reaching its end-of-life)
Relevant Screenshots
The content to be reworked would be steps 2-5, but we can provide screenshots if that is desired.
Link a PR (optional)

Hey @amy-keibler thanks for letting us know! This is really helpful.
Would you be up to make a contribution? Here's the Markdown behind that page: https://github.com/fern-api/fern/blob/main/fern/docs/pages/fern-sdks/guides/publish-to-maven-central.mdx

We're updated the documentation about Publishing your Java SDK with Maven


Order of start/Model creation shouldn't matter.

This PR makes it so that Models can be created before or after can.Feathers.connect().  I also put in a check at line 56 to only create a new Model.ready internal deferred if the current one isn't still pending.  I've also update the tests with a Task model created after can.Feathers is already connected.

I managed to find a test where this doesn't work.  Back to the drawing board.


Cleanup 02

So I deleted unnecessary files in the hf-oscillator directory, merged defines.h from this directory into defines.h in the main directory, added the compiled versions for 115 and 135 MHz to /build and changed README.md a bit.
Please, before you accept this pull request, try compiling and running the code on RP2040 in your environment to make sure it works in general (in my environment it works without problems, but sure is sure...).
My next planned task is to ensure that the oscillator is turned off after the packets have been transmitted. In the current state, this should not be used on a real balloon because the oscillator is constantly running and would interfere with other users transmitting in other timeslots on the same frequency.

my intention with keeping the oscillator on between packets was to keep the temperature (and frequency) more stable. But I agree with you, if you change that in a future pull I'm fine with that.
Also, the way I have it now, if doing both ZAchtek and U4b simultaneously, it does the zachtek packets right before the "more important" u4b packets, so the temp should be stable for the U4B packets anyway.


Review the links from docs to Obsidian help

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
All links to the Obsidian help site should work
Current behaviour
Link to 'Obsidian's built-in callouts' in this page:
https://obsidian-tasks-group.github.io/obsidian-tasks/getting-started/
Goes to here:
https://help.obsidian.md/How+to/Use+callouts
Which is a 404.
Steps to reproduce
See above
Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[ ] Windows

Obsidian Version
N/A
Tasks Plugin Version
N/A
Checks

[ ] I have tried it with all other plugins disabled and the error still occurs

Possible solution
Search the docs source and check all the links to https://help.obsidian.md/ still work.

This has been fixed upstream.
https://github.com/obsidianmd/obsidian-docs/commit/a76da734f1659ec951555e7d66ed6028a581b4db
The fix has not yet been published.

It turns out that redirects have now been set up in the Obsidian docs, so no action is required.


memmem not available on macos

It is available in macos like most FreeBSD functions however it does not seem to be avAilable on ios, see https://developer.apple.com/library/archive/documentation/System/Conceptual/ManPages_iPhoneOS/

You might as well also spam in all the missing neon intrinsics you want :)

#[cfg(all(unix, not(target_os = "ios")))]  enough? How many places does it need spammed?
https://github.com/rust-lang/libc/search?q=memmove

@chadbrewbaker I don't understand what you mean. Architecture-specific intrinsics are accessible in stdarch and memmove is available on ios.

@JohnTitor Thanks!


p keyword in SQL

I recently upgraded highlight.js version from 8.4 to latest version and noticed one small regression between our highlighting: the stand-alone letter p is now recognised as a key word. For example in the following query:
SELECT p FROM pets p
I went through the changelog and saw that it was added in version 8.7, probably under one of the following changes:

Oracle keywords added to SQL language definition by Vadimtro
Postgres keywords added to SQL language definition by Benjamin Auder

Not sure if " p " is actually a keyword or if it's a bug...

Hm… That sure looks strange. Along with things like acc acce accep accept and some other one-letter keywords. @Vadimtro could I bother you to shed some light on it? It was added in c915a19a.

Thanks my fault in review :-( I think we can simply remove all one-char keywords from that list and moderate all other short keywords.

@Sannis can you take this on then?

@isagalaev sure.


Changes to tests can block updates in Firefox

Hi @dissolve
Looks like you are motoring with your parser and finding all sorts of issues with the tests. Great to see someone putting a lot effort into a parser.
Would just like to point out that the tests are used for automated updating of the microformats-shiv code in the Firefox browser core library.  I built Mozilla a script that will not allow them to update the microformats-shiv code without it passing these tests. On a new pull of my code it also converts the tests and adds them to Mozilla's internal test suite. The script pulls the tests directly off this repo. Mozilla only use the parser internally. I am not sure they do new pulls of updated versions of the  microformats-shiv anymore.
If there is an issue with the tests they should be changed, I am not saying otherwise, just want to let you known
I would love to help work through the issues you are having with the tests at the moment and check my own parser to understand each problem but I just do not have the time at the moment.
Cheers
Glenn

thanks @glennjones, thats good to know.
I have finally made it through all the issues and now I have at least documented them all, so there shouldn't be any more bugs getting raised from me for a while.

@glennjones do you have time to at least review/accept pull requests for any test-related changes to microformats-shiv parser?

@tantek would love to, but my work circumstances have changed.
I am back at full-time, plus some. Last week I pulled a 7 day week and its going to be the same this week.  Its not that I am dropping my interest in microformats, just a lack of time due to current commercial pressures.  This level workload is going with me for the rest of 2017.  My open source work and conference going has been very heavily hit across the board. I am already 3 month behind on PRs for my open source Node.js project which has 40k download a month.
If I can find time, I will try, but its unlikely to any time soon. Sorry!
Glenn

Thanks @glennjones, appreciate the update. @mixedpuppy is this something you can help with? In particular can we at least answer (here in the issue) the questions that @glennjones raises in the initial issue description at the top?

What questions are being raised?  Seems more a description of how the tests are put together for us.

Following up on @Zegnat's comment on #119…
Is this issue still valid? I did some quick looking and found this Bugzilla bug but I'm not familiar enough with the particulars to know the state of that bug.


Add total_artifact_bytes benchmark metric.

This is needed to properly evaluate the impact of eager constant evaluation.

Looks like the recent Github token change might break the comment posting


MGMT-17259: Agent should use specified matchPatterns

MGMT-17259: Agent should use specified matchPatterns
MGMT-17294: Device conditions for containers and systemd units

/hold wait for #183


How to use the checkboxes

Hello,
how to use the checkboxes (Marketing/ Personalization/ Analysis) on the Osano page in the CookieConsents plugin?
Unfortunately you can't find anything about this in the settings and in the API description.
Greetings

If you use the current dev version you can use the type 'compliance'
The docs seem to be a few versions behind and are not that comprehensive to begin with.
Have a look at https://github.com/osano/cookieconsent/blob/dev/src/options/popup.js for all the options.

Thank you very much for the information, it helped me a lot.

How did you get it to work in the end? I can only see that we can use the categories type, but where do we define the categories? It doesn't work for me.

Using the category type is apparently also the right solution.
I have defined the categories myself look into categories.map( ( category, index ) => ...

@Nesfiran thanks. How do you define the categories? Obviously the variable is not defined and what is the structure even supposed to look like? Help is much appreciated.

Please see the link posted above starting at line 62 for all possible options.
You initialize the script as follows, from here on you can define all variables yourself.
const cc = new CC({
//...options,
type   : "categories"
})
For example:
const categories = [Category 1, Category 2, Category 3];


Can't import  tokenizer_from_json

When I  use 'keras.preprocessing.text.tokenizer_from_json', can't find.
I check keras/preprocessing/text.py, find there is no tokenizer_from_json;
Then add "tokenizer_from_json = text.tokenizer_from_json", is ok;

and add "from tensorflow.python.keras.preprocessing.text import tokenizer_from_json"  in
"site-packages/tensorflow/_api/v1/keras/preprocessing/text/init.py"

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

cool!


Dragging hotspot

Hello @mpetroff
Is there an example of how to make hotspot draggable? I have read doc/json-config-parameters.md but it does not work for me.
Thank you in advance.

Here's an example:
<!DOCTYPE HTML>

    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Hot spots</title>
    <link rel="stylesheet" href="../src/css/pannellum.css"/>

Note that this was added after v2.5.6 was released, so you need to use the Git master branch.


[CHORE]: Install initial packages, setup tests and linting for V2

Initialize repo packages along with tests and linting

Closed by #526


Change logo colors to match site.

Client will need to supply original, high quality logo if they would like this change to be implemented.

Can we close this or does it need to be reviewed?

We should probably just close it. It's been sitting here for a while.

closing...

Reopened because the client asked for this in her most recent email

Gonna do this in a different PR


fix(bpf): Compile eBPF probe with -Wno-unknown-attributes

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug
/area libscap-engine-bpf
/area build

Does this PR require a change in the driver versions?
No

What this PR does / why we need it:
In some cases we use clang 7 to build the eBPF probe. That version of clang is too old to support some of the compiler attributes which appear in the source code for more modern kernel versions.  So disable warnings associated with unknown attributes.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
This PR is a porting of a patch made by @jcpittman144
Does this PR introduce a user-facing change?:

NONE

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

Needs approval from an approver in each of these files:

driver/OWNERS [LucaGuerra]

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

LGTM label has been added.  Git tree hash: 2ac42c10e9b467d3b94e94fd84038f18f13af39c


Cleaning dataset

Continue data preparation by cleaning and transforming the dataset. Ensure that you document your process in RMarkdown.

The raw data has been prepared and cleaned, using a variety of common data operations in R, involving dplyr, tidyverse, or data.table. Common operations are merging, aggregating, de-deduplication, reshaping, converting dates, or using regular expressions. Basic programming concepts are made use of appropriately to increase speed and minimize errors (e.g., looping, vectorization, writing functions, handling errors/debugging). Additional variables are created from the raw data (feature engineering).


improving findability of control root info

The control root is a key part of our version control story, but at the moment it's not super easy to find an appropriate source of truth for what this actually is.
This PR makes it easier to navigate from the terminology entry to the relevant section in the codebase.
On a follow-up PR, would be great to align the naming here: it seems that what we call control root in the terminology page is called ALLOWED_IDS_ROOT in the recursion repo.

I haven't really thought about the naming question, but I think ALLOWED_IDS_ROOT is perhaps a better choice than control root, since it's less likely to be confused with control ID.

Addressed by #1588


chat commands: Expand boss abbreviations in !lvl

Closes #14794

Why doesn't this just use the getName()?


Add noFormat to install schema

Needed for https://github.com/kairos-io/kairos-agent/pull/32

Codecov Report

Merging #27 (3629f47) into main (c90740d) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage   33.07%   33.07%
=======================================
  Files          10       10
  Lines         520      520
=======================================
  Hits          172      172
  Misses        322      322
  Partials       26       26

Impacted Files
Coverage Δ

schema/install_schema.go
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Llvm ir

Although complicated, this is very possible

Maybe compile to C instead
Regardless, need garbage collection, so wil need to finish analyzer and usage tracker
Type inferencing necessary
Then will need a C AST build and THEN turn to string..

Source map in memory might be necessary to track Generated C code

Replaced with JVM.


dokku-pages using deprecated plugin trigger 'pre-build-dockerfile'

I get the error message:
Deprecated: please upgrade plugin to use 'pre-build' plugin trigger instead of pre-build-dockerfile
in my deployment logs. It's okay for now but I'm guessing in the future this will be a problem as it's deprecated in dokku now 😅
This is using the --minimal flag for dokku pages deployment; I'm on the latest version of Dokku.
Let me know if you need any other deets!

Thanks for reporting!
It's been a while since I dove into this package, I can't seem to see any references to pre-build-dockerfile at all... 🤔

Deprecated: please upgrade plugin to use 'pre-build' plugin trigger instead of pre-build-dockerfile

This message is probably coming from dokku on the server, and means you need to upgrade something.
This dokku-pages package simply pushes "static files" + "Dockerfile" to "git" and doesn't even know about any plugins on your dokku server


Spotify plugin fails to run

Microsoft PowerToys version
0.81.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
PowerToys Run Spotify fails to load on start-up, disabling/enabling in settings doesn't help .

✔ Expected Behavior
PowerToys Run Spotify plugin should successfully load?
❌ Actual Behavior
PowerToys Run Spotify plugin won't load
Other Software
No response

It could be just as simple as needing an update

https://github.com/waaverecords/PowerToys-Run-Spotify/releases/tag/v0.81.1

I had forgotten I installed this as a 3rd party plugin, thank you for being so polite about it! Updating the plugin did indeed work :)


%5Btool_crash%5D RPCError

%23%23 Command%0A%60%60%60sh%0Aflutter run%0A%60%60%60%0A%0A%23%23 Steps to Reproduce%0A1. ...%0A2. ...%0A3. ...%0A%0A%23%23 Logs%0ARPCError%0A%60%60%60console%0A%0A%60%60%60%0A%60%60%60console%0A%1B%5B32m%5B%E2%9C%93%5D%1B%5B39m Flutter %28Channel stable%2C 3.24.0%2C on macOS 15.1 24B5009l darwin-arm64%2C locale en-US%29%0A    %1B%5B32m%E2%80%A2%1B%5B39m Flutter version 3.24.0 on channel stable at %2FUsers%2Fm2%2Fdevelopment%2Fflutter%0A    %1B%5B32m%E2%80%A2%1B%5B39m Upstream repository https%3A%2F%2Fgithub.com%2Fflutter%2Fflutter.git%0A    %1B%5B32m%E2%80%A2%1B%5B39m Framework revision 80c2e84975 %2813 days ago%29%2C 2024-07-30 23%3A06%3A49 %2B0700%0A    %1B%5B32m%E2%80%A2%1B%5B39m Engine revision b8800d88be%0A    %1B%5B32m%E2%80%A2%1B%5B39m Dart version 3.5.0%0A    %1B%5B32m%E2%80%A2%1B%5B39m DevTools version 2.37.2%0A%0A%1B%5B32m%5B%E2%9C%93%5D%1B%5B39m Android toolchain - develop for Android devices %28Android SDK version 35.0.0%29%0A    %1B%5B32m%E2%80%A2%1B%5B39m Android SDK at %2FUsers%2Fm2%2FLibrary%2FAndroid%2Fsdk%0A    %1B%5B32m%E2%80%A2%1B%5B39m Platform android-35%2C build-tools 35.0.0%0A    %1B%5B32m%E2%80%A2%1B%5B39m ANDROID_HOME %3D %2FUsers%2Fm2%2FLibrary%2FAndroid%2Fsdk%0A    %1B%5B32m%E2%80%A2%1B%5B39m ANDROID_SDK_ROOT %3D %2FUsers%2Fm2%2FLibrary%2FAndroid%2Fsdk%0A    %1B%5B32m%E2%80%A2%1B%5B39m Java binary at%3A %2FApplications%2FAndroid Studio.app%2FContents%2Fjbr%2FContents%2FHome%2Fbin%2Fjava%0A    %1B%5B32m%E2%80%A2%1B%5B39m Java version OpenJDK Runtime Environment %28build 17.0.11%2B0-17.0.11b1207.24-11852314%29%0A    %1B%5B32m%E2%80%A2%1B%5B39m All Android licenses accepted.%0A%0A%1B%5B32m%5B%E2%9C%93%5D%1B%5B39m Xcode - develop for iOS and macOS %28Xcode 15.4%29%0A    %1B%5B32m%E2%80%A2%1B%5B39m Xcode at %2FApplications%2FXcode.app%2FContents%2FDeveloper%0A    %1B%5B32m%E2%80%A2%1B%5B39m Build 15F31d%0

Decoded content above:
Command
flutter run

Steps to Reproduce

...
...
...

Logs
RPCError

[✓] Flutter (Channel stable, 3.24.0, on macOS 15.1 24B5009l darwin-arm64, locale en-US)
 • Flutter version 3.24.0 on channel stable at /Users/m2/development/flutter
 • Upstream repository https://github.com/flutter/flutter.git
 • Framework revision 80c2e84 (13 days ago), 2024-07-30 23:06:49 +0700
 • Engine revision b8800d88be
 • Dart version 3.5.0
 • DevTools version 2.37.2

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
 • Android SDK at /Users/m2/Library/Android/sdk
 • Platform android-35, build-tools 35.0.0
 • ANDROID_HOME = /Users/m2/Library/Android/sdk
 • ANDROID_SDK_ROOT = /Users/m2/Library/Android/sdk
 • Java binary at: /Applications/Android Studio.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 15.4)
 • Xcode at /Applications/Xcode.app/Contents/Developer
 • Build 15F31d%0

Closing this as a duplicate of https://github.com/flutter/flutter/issues/117988


How to constraint units?

Suppose we want represent radius of a circle. In Documentation it was done using kind<>. How can enforce that radius of negative length cannot be created:
quantity_kind<radius, si::kilometre, double> d(-123 * km); // shoud not work

Do library provide any facilities for to create such constraints?

Yes. By using not  double, but a representation type that does such a check. https://www.boost.org/doc/libs/1_78_0/libs/safe_numerics/doc/html/safe_range.html seems to fit the bill.

Thank you for your answer. Unfortunately safe_range doesn't work with floating point types, but i was able to write simple double wrapper that does this check.

Can this issue be closed now?

Thank you for your answer. Unfortunately safe_range doesn't work with floating point types, but i was able to write simple double wrapper that does this check.

@WBurzynski sounds interesting. Would you be willing/able to share some code snippets?

@RalphSteinhagen This is wrapper for my use case, that is just holding geographic data and ensuring their value is valid:
template<intmax_t min, intmax_t max> class ConstrainedDouble
{
public:
    ConstrainedDouble() = default;
    constexpr ConstrainedDouble(double new_value)
    {
        if (new_value <= min || new_value >= max)
        {
            throw std::invalid_argument(
                fmt::format("value outside of range <{},{}>", min, max));
        }
        value = new_value;
    }
    [[nodiscard]] bool operator==(const ConstrainedDouble &) const = default;

    [[nodiscard]] friend constexpr ConstrainedDouble operator*(ConstrainedDouble lhs,
                                                               ConstrainedDouble rhs)
    {
        return ConstrainedDouble(lhs.value * rhs.value);
    }
    [[nodiscard]] friend constexpr ConstrainedDouble operator/(ConstrainedDouble lhs,
                                                               ConstrainedDouble rhs)
    {
        return ConstrainedDouble(lhs.value / rhs.value);
    }
    [[nodiscard]] constexpr double getValue() { return value; }
private:
    double value;
};
class Degree : public units::named_unit<Degree, "deg", units::no_prefix>
{
};
class DimAngle : public units::base_dimension<"A", Degree>
{
};
using Longitude = units::quantity<DimAngle, Degree, ConstrainedDouble<-180, 180>>;
using Latitude = units::quantity<DimAngle, Degree, ConstrainedDouble<-90, 90>>;

Later i changed approach, because conversions were a little clunky, but I'm sure it's something that can be improved.

I just pushed some related updates to the glide_computer:
https://github.com/mpusz/units/blob/bded48987d075b94a58b50c126843a9e6cc853e1/example/glide_computer/include/geographic.h#L39-L44