Issue with $ref Schema Handling: Incorrect Required Property and Missing Description in Generated Dart Code

Steps to reproduce

Define swagger_parser: 1.18.1 or 1.18.2 in pubspec.yaml.
Define the yaml file as described in Your OpenApi snippet.
Run dart run swagger_parser.
Run flutter pub run build_runner build -d.

Expected results
I would like the code to be generated as follows.
import 'package:freezed_annotation/freezed_annotation.dart';

import 'gender.dart';

part 'user_request.freezed.dart';
part 'user_request.g.dart';

/// User request.
@Freezed()
class UserRequest with _$UserRequest {
  const factory UserRequest({
    /// User name.
    required String name,

    /// User gender.
   Gender? gender,
  }) = _UserRequest;

  factory UserRequest.fromJson(Map<String, Object?> json) => _$UserRequestFromJson(json);
}

Actual results
Specifically:

When a property is referenced using $ref, it becomes required even if it is not listed in the required section of the schema.
If nullable: true is specified in the referenced schema (e.g., Gender), it should be reflected in the generated Dart code. However, the required annotation is still added, making the property non-nullable.
Additionally, when using $ref, the description specified in the referenced schema is not carried over into the generated doc comments.

Generated Dart Code:
import 'package:freezed_annotation/freezed_annotation.dart';

import 'gender.dart';

part 'user_request.freezed.dart';
part 'user_request.g.dart';

/// User request.
@Freezed()
class UserRequest with _$UserRequest {
  const factory UserRequest({
    /// User name.
    required String name,
    required Gender gender,
  }) = _UserRequest;

  factory UserRequest.fromJson(Map<String, Object?> json) => _$UserRequestFromJson(json);
}

It would be helpful if the code generation could respect the nullable property and ensure that the description from the $ref schema is included in the generated code.
Your OpenApi snippet
openapi: 3.0.3
Example YAML:
paths:
  /sample:
    post:
      tags:
        - User
      summary: Sample API
      description: Sample API.
      operationId: PostSample
      requestBody:
        required: true
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/UserRequest'
      responses:
        '201':
          description: Created.
        default:
          $ref: '#/components/responses/default'
components:
  schemas:
    UserRequest:
      description: User request.
      type: object
      properties:
        name:
          type: string
          description: User name.
        gender:
          $ref: '#/components/schemas/Gender'
      required:
        # gender is not required, but required is added when generating code.
        - name
    Gender:
      type: string
      # here, even if you specify nullable, required is added when generating code.
      nullable: true
      enum:
        - MALE
        - FEMALE
        - OTHER
        - UNKNOWN
      # $ref is ignored description in code generation.
      description: User gender.

Code sample
The code sample is all as described above.
Logs
Logs
┃  ____ _ _ _ ____ ____ ____ ____ ____     ___  ____ ____ ____ ____ ____
┃  [__  | | | |__| | __ | __ |___ |__/     |__] |__| |__/ [__  |___ |__/
┃  ___] |_|_| |  | |__] |__] |___ |  \ ___ |    |  | |  \ ___] |___ |  \
┃

Generate...
> api_client v1.0.0:
    7 rest clients, 38 requests, 43 data classes.
    52 files with 2 068 lines of code.
    Success (0.03 seconds)

The generation was completed successfully. You can run the generation using build_runner.
cd medeta_app_packages/api_client && flutter pub run build_runner build -d
Deprecated. Use `dart run` instead.
Building package executable...
Built build_runner:build_runner.
[INFO] Generating build script completed, took 150ms
[INFO] Reading cached asset graph completed, took 59ms
[INFO] Checking for updates since last build completed, took 416ms
[INFO] Running build completed, took 1.4s
[INFO] Caching finalized dependency graph completed, took 32ms
[INFO] Succeeded after 1.5s with 3 outputs (6 actions)

Dart version and used packages versions
Dart version
❯ flutter --version
Flutter 3.24.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 5874a72aa4 (7 days ago) • 2024-08-20 16:46:00 -0500
Engine • revision c9b9d5780d
Tools • Dart 3.5.1 • DevTools 2.37.2

❯ dart --version
Dart SDK version: 3.5.1 (stable) (Tue Aug 13 21:02:17 2024 +0000) on "macos_arm64"

Packages version
1.18.1 or 1.18.2

Hello, thank you for this wonderful library!
I have the exact same problem.

Version 1.16.3 does not have this bug.
Version 1.16.4 does have this bug!!

As a workaround.
paths:
  /sample:
    post:
      tags:
        - User
      summary: Sample API
      description: Sample API.
      operationId: PostSample
      requestBody:
        required: true
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/UserRequest'
      responses:
        '201':
          description: Created.
        default:
          $ref: '#/components/responses/default'
components:
  schemas:
    UserRequest:
      description: User request.
      type: object
      properties:
        name:
          type: string
          description: User name.
        gender:
          nullable: true ← It's redundant, but it seems possible to do it this way
          $ref: '#/components/schemas/Gender'
      required:
        # gender is not required, but required is added when generating code.
        - name
    Gender:
      type: string
      # here, even if you specify nullable, required is added when generating code.
      nullable: true
      enum:
        - MALE
        - FEMALE
        - OTHER
        - UNKNOWN
      # $ref is ignored description in code generation.
      description: User gender.

Same problem too.
I hope this problem is resolved.


how to get ad to appear on only index page

Hi,
Got the plugin to work and it's great, but I need it to only appear on specific pages.  Is that possible?
Thanks.
Scott

I figured it out.  Found the AdMob.removeBanner() method and added it as an "onclick" to the links that take me to other pages.  It works fine, and banner reappears when I come back to the index page.


Add a fast Create Plan page - step 7

Depends on #898
Changes:

improve layout by wrapping title and info alert with a drawer. Now
the only component pushing the VM table down is the namespace bar.
disable all edits when create action is triggered
a) for description items - hide pencil icon
b) for form components - propagate isDisabled flag
remove 'Create and start' button
rename 'Create and edit' to 'Create'

Signed-off-by: Radoslaw Szwajkowski<EMAIL_ADDRESS>

Disabled form after an API error (new buttons visible)

Drawer before the change
Unnecessary scrollbar is displayed. The buttons are not visible.

Drawer with selected VMs after the change
The drawer wraps the items above the form which drags the whole table higher (only the namespace bar is left)

Validations
Incomplete mapping
Implemented based on backend check  as discussed here and here

Multiple NICs mapped to Pod Networking / NICs with empty NIC profile
Multiple NICs mapped to Pod Networking is based in backend check as discussed here
NICs with empty NIC profile is a special case of incomplete mapping (oVirt specific) - NIC without a profile cannot be mapped.

Multiple NICs on the same network
This is a special case of incorrect network setup - it may trigger Multiple NICs mapped to Pod Networking.


Add space in message about reporting full PREreview

Super tiny issue of adding a space to have full PREreview instead of fullreview. :D

Deferring to @jheretic @harumhelmy on this one, as I'm not positive this doesn't affect the API in this case?

It really doesn't matter if it can't be fixed. It's a tiny tiny thing. :D Thanks!


Enable notes from our synced obsidian vault to come up as suggestions on the fleeting notes app when using the [[ and typing the first few letters on android

Is your feature request related to a problem? Please describe.
My android phone syncs my vault to my android. I use fleeting notes to capture so can’t see the relevance of linking notes together in the app itself
Describe the solution you'd like
Is there any way of getting the notes from obsidian vault to come up as suggestions on the app when using the [[ and typing the first few letters? Like using the search query to find any other reference to a note before creating a new note so we can much more effectively build our zettelkasten
Describe alternatives you've considered
Create a new note in Fleeting Notes and add [[links]] that are within your vault to that note. - too cumbersome when adding links / new notes all the time in obsidian

This is a fantastic feature but I'll need to think of a way to seamlessly integrate this. I'm very much leaning towards creating a fleeting note from within the Obsidian plugin with all the links within it. If you do have one-way + delete enabled it should not ignore this note with a title specified by you.

Thanks matt
On Sat, 21 Jan 2023, 15:09 Matthew Wong, @.***> wrote:

This is a fantastic feature but I'll need to think of a way to seamlessly
integrate this. I'm very much leaning towards creating a fleeting note from
within the Obsidian plugin with all the links within it. If you do have
one-way + delete enabled it should not ignore this note with a title
specified by you.
—
Reply to this email directly, view it on GitHub
https://github.com/fleetingnotes/fleeting-notes-obsidian/issues/62#issuecomment-1399268453,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMA4G4YDR5GIIGQPT25BMU3WTP3Z7ANCNFSM6AAAAAAUAX6GBU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ahh wow - did you find a way to implement this Matt?
On 2 Feb 2023 at 22:39:02, Matthew Wong @.***> wrote:

Closed #62
https://github.com/fleetingnotes/fleeting-notes-obsidian/issues/62 as
completed via #64
https://github.com/fleetingnotes/fleeting-notes-obsidian/pull/64.
—
Reply to this email directly, view it on GitHub
https://github.com/fleetingnotes/fleeting-notes-obsidian/issues/62#event-8426192997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMA4G46PIKNMP2BH4L3ANR3WVQZQNANCNFSM6AAAAAAUAX6GBU
.
You are receiving this because you authored the thread.Message ID:
<fleetingnotes/fleeting-notes-obsidian/issue/62/issue_event/8426192997@
github.com>

Yep @sharetocreate ! you can try it out now! it's released on version v0.5.2. SImply go to the settings and enable "Sync Obsidian [[links]] to Fleeting Notes"

Amazing - well done mate - will try in a bit
On 2 Feb 2023 at 23:25:36, Matthew Wong @.***> wrote:

Yep @sharetocreate https://github.com/sharetocreate ! you can try it
out now! it's released on version v0.5.2. SImply go to the settings and
enable "Sync Obsidian [[links]] to Fleeting Notes"
[image: image]
https://user-images.githubusercontent.com/20890995/216473720-039316a5-8980-488e-984e-0ae776e53064.png
—
Reply to this email directly, view it on GitHub
https://github.com/fleetingnotes/fleeting-notes-obsidian/issues/62#issuecomment-1414506423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMA4G44C62LJ5UGCQMSTPG3WVQ67BANCNFSM6AAAAAAUAX6GBU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry but how to update? Not seeing the feature & play store comes up with
cant install fleeing notes, when clicking on update
I normally click on the updates via email, but would be great if app could
update automatically with the rest if poss. Looking forward to try out the
links feature - best. Marc
On 2 Feb 2023 at 23:35:44, ShareTo Create @.***> wrote:

Amazing - well done mate - will try in a bit
On 2 Feb 2023 at 23:25:36, Matthew Wong @.***> wrote:

Yep @sharetocreate https://github.com/sharetocreate ! you can try it
out now! it's released on version v0.5.2. SImply go to the settings and
enable "Sync Obsidian [[links]] to Fleeting Notes"
[image: image]
https://user-images.githubusercontent.com/20890995/216473720-039316a5-8980-488e-984e-0ae776e53064.png
—
Reply to this email directly, view it on GitHub
https://github.com/fleetingnotes/fleeting-notes-obsidian/issues/62#issuecomment-1414506423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMA4G44C62LJ5UGCQMSTPG3WVQ67BANCNFSM6AAAAAAUAX6GBU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@sharetocreate you'll need to install the Fleeting Notes Sync plugin from within Obsidian. The community plugin performs the syncs the links from within Obsidian. So it's not something within the Fleeting Notes app. Hopefully this helps.

Ah i see. Thanks Matt. Updated. Is there any other steps? I have one way
sync but dont see a note created with any links in either obsidian or
fleeting notes anddroid app so think im missing something.
On Fri, 3 Feb 2023, 14:42 Matthew Wong, @.***> wrote:

@sharetocreate https://github.com/sharetocreate you'll need to install
the Fleeting Notes Sync plugin from within Obsidian. The community plugin
performs the syncs the links from within Obsidian. So it's not something
within the Fleeting Notes app. Hopefully this helps.
[image: image]
https://user-images.githubusercontent.com/20890995/216631173-cfa84907-71b1-49cf-a3f0-79d7fa03d0c4.png
—
Reply to this email directly, view it on GitHub
https://github.com/fleetingnotes/fleeting-notes-obsidian/issues/62#issuecomment-1415965958,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMA4G43DJZX6ZCRKTTZXDFTWVUKM7ANCNFSM6AAAAAAUAX6GBU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Emailed also via my account email so can disregard this thread to save
duplicating :)
On Fri, 3 Feb 2023, 15:48 ShareTo Create, @.***> wrote:

Ah i see. Thanks Matt. Updated. Is there any other steps? I have one way
sync but dont see a note created with any links in either obsidian or
fleeting notes anddroid app so think im missing something.
On Fri, 3 Feb 2023, 14:42 Matthew Wong, @.***> wrote:

@sharetocreate https://github.com/sharetocreate you'll need to install
the Fleeting Notes Sync plugin from within Obsidian. The community plugin
performs the syncs the links from within Obsidian. So it's not something
within the Fleeting Notes app. Hopefully this helps.
[image: image]
https://user-images.githubusercontent.com/20890995/216631173-cfa84907-71b1-49cf-a3f0-79d7fa03d0c4.png
—
Reply to this email directly, view it on GitHub
https://github.com/fleetingnotes/fleeting-notes-obsidian/issues/62#issuecomment-1415965958,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMA4G43DJZX6ZCRKTTZXDFTWVUKM7ANCNFSM6AAAAAAUAX6GBU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Just to update people reading this thread. The problem was resolved by logging in and out within the Obsidian plugin.


Response Documentation

How can i add a response comment on this documentation.
Thank you

This is supported now.


Add graceful handling of cache write failure due to already existent cache file caused by race condition.

These changes address the concurrent cache write issue revealed by https://github.com/thephpleague/glide/issues/38.

It looks like something is wrong with Scrutinizer. Perhaps Travis didn't finish in time to report code coverage?:

The code coverage data was not received within the specified time. Please make sure your third-party service is configured to send code coverage data.

Yeah, I just reran the tests and everything looks cool. Thanks for putting this together Kevin!


[Monitoring/Rancher v2.5.6] After enabling monitoring , Pod pushprox-kube-etcd-client failed to scrape metrics

After deploy rancher-monitoring version 16.6.0, the prometheus target "serviceMonitor/cattle-monitoring-system/rancher-monitoring-kube-etcd" is down.
it is a hosted/imported cluster
Logs from pushprox-kube-etcd-client pod :
level=info ts=2021-11-02T14:28:57.947Z caller=main.go:269 msg="URL and FQDN info" proxy_url=http://pushprox-kube-etcd-proxy.cattle-monitoring-system.svc:8080/ fqdn=<IP_ADDRESS>
level=info ts=2021-11-02T14:29:09.124Z caller=main.go:232 msg="Got scrape request" scrape_id=05a4ab71-6e88-4311-a757-16e3fa6738ef url="http://<IP_ADDRESS>:2379/metrics?_scheme=https"
level=error ts=2021-11-02T14:29:09.130Z caller=main.go:101 err="failed to scrape https://<IP_ADDRESS>:2379/metrics: Get "https://<IP_ADDRESS>:2379/metrics\": x509: certificate is valid for <IP_ADDRESS>, <IP_ADDRESS>, not <IP_ADDRESS>"
level=info ts=2021-11-02T14:29:09.138Z caller=main.go:113 msg="Pushed failed scrape response"

I'm being hitted by this same issue, I think this bug report should be reopened

Although i only use rke, not rancher itself, but our issue is the same

Getting the same error on a rke1 provisioned cluster. The Service monitor is listed as active but the prod is getting the same log errors as listed above.
System info:
rke: v1.3.12
Kubernetes: v1.22.10+
Monitoring app: 100.1.3+up19.0.3

This issue should not have been closed. It is still present on Rancher 2.6.9, and because of what johnisell mentioned above, it looks to be disabled on all RKE v1 clusters.
Rancher Monitoring v2 was supposed to fix this according to https://github.com/rancher/rancher/issues/18619#issuecomment-673168369 , but it's still broken.

Looks like the Rancher docs team will document the solution, as mentioned in https://github.com/rancher/rancher/issues/31650#issuecomment-1362080340 .


How to set Name and Where values?

The Ice Cubes macOS app doesn't set Name and Where values for keychain items so I want to submit a PR that sets those values. Can that be done with the set method?
https://github.com/Dimillian/IceCubesApp/issues/1792
https://github.com/Dimillian/IceCubesApp/blob/main/Packages/AppAccount/Sources/AppAccount/AppAccount.swift#L19

Library setup method: Swift Package Manager
Version of the library:
Xcode version: Xcode 14.0
OS version: macOS Sonoma 14.2.1

https://github.com/evgenyneu/keychain-swift#setup-with-swift-package-manager-in-swift-package

If you're using KeychainSwift in a Swift package, make sure to specify a name.

@evgenyneu If we change the package name do existing keychain items still work?
https://github.com/Dimillian/IceCubesApp/issues/1792#issuecomment-1897839118

No idea @diogoguimaraes, I would run an experiment and try it to find out.


Game crashes with Agricraft's Seed Analyzer

According to your testing, does the issue occur with ONLY Immersive Portals mod installed?
Yes
Forge Version
40.2.0
ImmersivePortals Version
1.4.17-mc1.8.2
Latest Log
https://gist.github.com/Nox13last/8c88793cc19893c2c78f6d6785200c18
Crash Report (if applicable)
https://gist.github.com/Nox13last/ec2c216aba2ee9faf1d8b366912f222d
Steps to Reproduce
I have earmarked each step (as significant) with chat messages. These will be parenthetical. I figured you could use something to CTRL+F at each step.

Generate a new Creative World.
Pull the following items out of the Creative Inventory drawer: Obsidian, Fire Charge, Seed Analyzer.
(Optional) Build a Nether Portal. Standard configuration.
(Optional) Ignite the Nether Portal with a Fire Charge. (<Noxedwin> SPAWNING IMMERSIVE NETHER PORTAL)
Place a Seed Analyzer. (<Noxedwin> PLACING SEED ANALYZER)
Use the Seed Analyzer (<Noxedwin> USING SEED ANALYZER)
Watch.

What You Expected
For the Seed Analyzer UI to open. Also, for the Immersive Portal to continue to function.
What Happened Instead
The game explodes, performing an emergency world-save and then crashing to desktop.
Additional Details
I cannot confirm that this issue has not already been reported (#59), but as the OP had vanished last year, I am creating a fresh issue. I will crosspost them. Considering how well-written that issue ticket is written, I would honestly defer to it over this one.
The other element of this report is the presence of Agricraft. The version used here is 1.18.2-3.1.4. This is only two releases behind the final 1.18.2 release.
Please Read and Confirm The Following

[X] I have confirmed this bug can be replicated without the use of Optifine.
[X] I have confirmed the details provided in this report are concise as possible and does not contained vague information (ie. Versions are properly recorded, answers to questions are clear).
[X] I have confirmed this does not happen on the fabric version (If it does please report to here instead).
[X] I have confirmed I'm not using a hybrid server (Magma, Cauldron) or a different (non official) Forge Server release.
[X] I have confirmed this issue is unique and has not been reported already.

In cooperation with the referenced issue ticket, I would like to confirm that building a nether portal is unnecessary.
That issue ticket is really well-written - I can't do a better job myself. Shame the OP didn't come back to it.

In cooperation with the referenced issue ticket, I notice that the OP believes that the problem may lie in rendering a graphical object that IP is trying to get hold of.
With that said, there is an option in Agricraft's config to use GUIs instead of in-world animations. It does not help.

Crashy offers this, which is probably more of the same information.

Here is a comparison between the two modes.
GUI Off:

GUI On:

In 'GUI Off' mode, the mouse-wheel rotates the helix to highlight each rung (base-pair).


feat(GroupTheory/Exponent): lemmas around Monoid.exponent and Subgroup/Submonoid

Proves some helpful properties of Monoid.exponent on Submonoid G and Subgroup G:

the exponent of a group is equal to 1 iff the monoid is trivial (previously only the reverse implication was proven)
the exponent of G divides the exponent of H if there exists a multiplication-preserving injection from G to H
Monoid.exponent G = Monoid.exponent H if G ≃* H
Monoid.exponent ⊤ = Monoid.exponent G
g ^ Monoid.exponent H = 1 if g ∈ H
generalizes one_lt_exponent to LeftCancelMonoid

This PR belongs to my series of PRs around the formalization of Rubin's theorem, although these lemmas are only really needed for one step of the proof.
A small tweak will be necessary once #9975 merges.

Thanks!
maintainer merge


Make an URL link for sub document

Make an URL link for sub document by repository_url
to repository image & title

45712e8 thanks for quickly update @jian7061 💯


Not able to upgrade Wordpress app from 2.1.10 -> 2.1.11 version.

Rancher server version - v2.2.4-rc15
Steps to reproduce the problem:
Create a custom cluster.
Deploy "Wordpress" app version 2.1.10.
Once app is deployed successfully , upgrade to "2.1.11" version.
App is stuck in "Installing" state :

It’s because of the label for chart in spec.template.metadata.labels https://github.com/helm/helm/issues/2494#issuecomment-311691146

Observed the same issue on an upgrade from v2.1.9 to v2.1.10-rc5.

Create AWS and DO RKE clusters in rancher v2.1.9 .
Deploy mysql catalog of version v0.3.7, docker-registry catalog of version v1.4.3 and wordpress catalog of version v2.1.10 in the cluster.
Upgrade rancher server to v2.1.10-rc5
After rancher upgrade, upgrade mysql catalog to version v0.3.8. Upgrade docker-registry catalog to version to v1.6.1 . Both catalogs get upgraded successfully
Upgrade wordpress catalog to v2.1.11. It is stuck in installing state with the below error

Failed to install app wordpress. Error: UPGRADE FAILED: Deployment.apps "wordpress-wordpress" is invalid: spec.template.metadata.labels: Invalid value: map[string]string{"app":"wordpress-wordpress", "chart":"wordpress-2.1.11", "release":"wordpress"}: `selector` does not match template `labels`

Will be addressed in https://github.com/rancher/charts/pull/173, but for now, we may only test upgrade from v1.0.5 to  v2.1.10.
https://github.com/helm/helm/issues/2494#issuecomment-340939417

deployment spec selectors are no longer mutable as of apps/v1beta2 and this is noted upstream in kubernetes/kubernetes#50719. Short of helm delete --purge myrelease and running helm install again on your chart, there is no way to mutate the selector labels.

because the spec.template.metadata.labels fields are immutable, so we need to select delete and recreate option to force upgrade and it will works as expected.

Verification Steps:

Deploy the WordPress app with version 2.1.10
select the deployed app and view the Secret of wordpress-mariadb, copy the auto-generated mariadb-password and mariadb-root-password to somewhere you can revisit it later.
upgrade the wordpress app to version 2.1.11 and copy paste the mariadb-password and mariadb-root-password to the answers respectively.
select the Delete and recreate resources if needed during the upgrade checkbox and click upgrade.
the app should get upgraded with success.

addressed in https://github.com/rancher/charts/pull/174, upgrade from template version 2.1.10 or 2.1.11 to the current version 2.1.12 requires to select Delete and recreate resources if needed during the upgrade option,  additionally, the immutable field label chart is removed from v2.1.12.

We cannot change the released version. So 2.1.10 and 2.1.11 still have problem.

Upgrade "Wordpress" from "2.1.10" to  "2.1.11", you need to select force upgrade
Upgrade "Wordpress" from "2.1.10"/"2.1.11" to "2.1.12", you need to select force upgrade
Upgrade "Wordpress" from "1.0.5" to  "2.1.12", it works without selecting force upgrade
v1.0.5
Upgrade "Wordpress" from "2.1.12" to  other ongoing version, it works without selecting force upgrade

@loganhz  This is still failing:
Upgrade "Wordpress" from "1.0.5" to "2.1.12", it works without selecting force upgrade
v1.0.5

you will need to use force-upgrade for "Wordpress" from "1.0.5" to "2.1.12", the description above should be updated, thanks.

@guangbochen  The only to upgrade to  new version - "2.1.12"  that is introduced from any exisying version is to use "force-upgrade" . Is this acceptable ?

Sorry. 1.0.5 also have the issue. So yes, you have to always select force upgrade.
In the future, if we release 2.1.13, and then you can upgrade to 2.1.13 from 2.1.12 without using force-upgrade

@alena1108 @guangbochen If we are introducing a new version - "2.1.12" , is it not possible for us to make it upgradable from all existing versions or at least few of them ?  Right now we seem to be in a worse situation from before when we were atleast able to upgrade from 1.0.5 to latest version ( 2.1.11) without any problems.

we need to support upgrade from v2.1.10, v2.1.11 to v2.1.12 with those immutable fields spec.selector.matchLabels and spec.template.metadata.labels changes, (https://github.com/helm/helm/issues/2494#issuecomment-340939417), this is noted restriction in the upstream https://github.com/kubernetes/kubernetes/pull/50719 with our current default deployment version apps/v1beta2, and this change will also require v1.0.5 to do a force-upgrade to be compatible with the above upgrade.  @sangeethah @alena1108 .

for example, this also noted in the helm/stable charts, https://github.com/helm/charts/pull/7686/files#diff-b290a06961a11374d72f584e5925762cR53 , and definitely, it is better for us to add the notes in our app's README file.

met the same error on Rancher: master 6/13 with the default global library https://git.rancher.io/charts branch master

With rancher min and rancher max version introduced into 2.3, we should hide the older templates in 2.3 that don't work for upgrading to the latest wordpress.

hide older templates will require us to modify the questions.yml file of the existing chart, the CI lint in rancher/charts repo will fail due to making changes without bump chart version, but we can still process the changes.
also, hide older templates will only prevent new users from using the older versions, but they can still do the upgrade if they have installed previously, a better solution is if we can get UI to filter out min and max app version support https://github.com/rancher/rancher/issues/22266.

@loganhz @guangbochen With master that has the min/max support for the catalog, we should update wordpress to hide these older templates.

The root cause is the Wordpress version. Our min and max is for Rancher version.


[Fix #11996] Fix an error for Style/IfWithSemicolon

Fixes #11996.
This PR fixes an error for Style/IfWithSemicolon when without branch bodies.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Looks good. Thanks!


chore: bump snaplen to 65535

Issue #, if available:
fixes #16
Description of changes:
While trying to adopt request mirroring, we are seeing something similar in #16, I think bumping to 65535 bytes would help real use cases. Thanks!
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

close per this comment, https://github.com/aws-samples/http-requests-mirroring/issues/16#issuecomment-2214326228


[db] add ability to enable SSL for database connection

Adds the ability to enable SSL for the database connection.
Maintains existing default behavior of not being enabled.
Adds tests for setting options and default options on the database connection.

--
Ran tests locally and they pass as expected:
$ POSTGRES_TEST_DATABASE=test POSTGRES_TEST_PASSWORD=test POSTGRES_TEST_USERNAME=test POSTGRES_TEST_HOST=localhost yarn test

Once this is merged, I intend to open PRs for graphprotocol/indexer to update if accepted.
Thank you.

I've put up a version bump PR https://github.com/graphprotocol/common-ts/pull/122 which will include this.

I've put up a version bump PR #122 which will include this.

Thank you very much.  I have the indexer MR just about ready.


dubbo3.0在nacos注册中心上看不到消费者信息，订阅者列表这个功能是不能用吗，根本查不到信息

This is the ability provided by Nacos, you can see if it is a problem with Nacos

No news is good news. Please feel free to create a new issue if you have any question.


Translate quint variable initialization as equality, not assignment

Fixes #2863

[ ] Tests added for any new code
[ ] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entries added to ./unreleased/ for any new functionality

Codecov Report
Attention: Patch coverage is 90.64748% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 78.91%. Comparing base (456a572) to head (22b8270).

Files
Patch %
Lines

...ain/scala/at/forsyte/apalache/io/quint/Quint.scala
90.15%
13 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2864      +/-   ##
==========================================
+ Coverage   78.88%   78.91%   +0.02%
==========================================
  Files         467      467
  Lines       15959    15972      +13
  Branches     2592     2556      -36
==========================================
+ Hits        12589    12604      +15
+ Misses       3370     3368       -2

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

This isn't a full fix for #2863 yet: it only ensures primes are not added to assignments that occur within the body of the init predicate. For the general case, we need to ensure primes do not appear in any operator that is used by the init predicate.
We can solve that this way:

A function referencedDeclarationNames : QuintOpDef => List[String] that gathers just the non-builtin operator names that are referenced within the body of the give operator definition.
call referencedDeclarationNames on the init pred, then find all definitions with those names, and recurse on the results, until all results are exhausted (only empty lists returned)
This should produce a list of all and only the operators that are used in the init pred, and it should be in topopological order.
Set the isInit int the context (instead of doing it at https://github.com/informalsystems/apalache/blob/22b8270302da7acec0d3e4dbd5a7af619922d222/tla-io/src/main/scala/at/forsyte/apalache/io/quint/Quint.scala#L790-L792)
Fold over those gathered operators with this context, using the same logic as https://github.com/informalsystems/apalache/blob/22b8270302da7acec0d3e4dbd5a7af619922d222/tla-io/src/main/scala/at/forsyte/apalache/io/quint/Quint.scala#L815-L817
Finally, set isInit to false, and the continue to process the rest of the declarations

Some additional care will need to be take about where the init-relevant operators are placed back into the sequence of converted operators, in case there other operators that refer to them, which must occur later. Placing the init-relevant operators after all variable declarations but before any others should be safe, since they are guaranteed not to refer to other operators.
Alternatively, a different strategy could be employed on the quint side to remove the need for this complexity, as suggested on #2863.
Unfortunately, I won't have time to complete this work, as I am losing my company access to github etc. now.
<3


File Not Found Error

face_recognition version:
Python version:
Operating System:

Description
Amazing code btw, I am a beginner and I copied the code for the live webcam into my windows 10 command prompt, I changed the first image and face encoding to put my own in there, but it says it can't be found.
What I Did
This is what i changed
me_image = face_recognition.load_image_file("me.jpg")
me_face_encoding = face_recognition.face_encodings(me_image)[0]

This is what I get
Traceback (most recent call last):
  File "live_face_camera_add_people", line 14, in <module>
    me_image = face_recognition.load_image_file("me.jpg")
  File "C:\Users\jnort\AppData\Local\Programs\Python\Python37-32\lib\site-packages\face_recognition\api.py", line 83, in load_image_file
    im = PIL.Image.open(file)
  File "C:\Users\jnort\AppData\Local\Programs\Python\Python37-32\lib\site-packages\PIL\Image.py", line 2652, in open
    fp = builtins.open(filename, "rb")
FileNotFoundError: [Errno 2] No such file or directory: 'me.jpg'
[ WARN:0] terminating async callback

Please Help!!!

Is "me.jpg" in the same directory the script is running from? If not, either put it there, or try setting an absolute path.
me_image = face_recognition.load_image_file("C:\Users\Jacob\Pictures\me.jpg")
or whatever

If you haven't yet solved the issue, here are a few things that fixed my error:

Make sure your image and script are in the same directory.
Next, Just include file on its own, without the folder.
eg: Instead of "Folder/file.jpg", change it to, "file.jpg"


Link to other part of documentation is broken

Search terms you've used
Checked the open Issues
Bug description
in the documentation storybook:
https://solid-ui-react.docs.inrupt.com/?path=/story/hooks-usesession--page
the Link to the Session class documentation is broken and gives a File not found
To Reproduce

browse to https://solid-ui-react.docs.inrupt.com/?path=/story/hooks-usesession--page
in the useSession > Return Values > Click on Session

Expected result
Link navigates to https://docs.inrupt.com/developer-tools/api/javascript/solid-client-authn-browser/classes/Session.html#
Actual result
link navigates to https://docs.inrupt.com/developer-tools/api/javascript/solid-client-authn-browser/classes/session.html and shows:
File not found
The URL you requested does not exist or has been removed.

found another one: https://docs.inrupt.com/developer-tools/api/javascript/solid-client/modules/resource_soliddataset.html#getsoliddataset

Hi @michaud , I'm sorry this issue was never replied to. We are deprecating this library, so we will not address the open issues.


[DOC] Inaccurate example(s) for register snapshots

What do you want to do?

[ ] Request a change to existing documentation
[ ] Add new documentation
[X] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
Seems to affect all versions (only bothering to look at 1.3 and newer):
e.g.:

https://opensearch.org/docs/1.3/api-reference/snapshots/create-repository/
https://opensearch.org/docs/2.5/api-reference/snapshots/create-repository/

I don't think we're defining the base_path parameter correctly, or if we are then we aren't explaining it completely.
In the description of base_path the example (my/snapshot/directory) uses a relative path:
The path within the bucket where you want to store snapshots (e.g. my/snapshot/directory). Optional. If not specified, snapshots are stored in the bucket root.

Same with the example:
PUT _snapshot/my-opensearch-repo
{
  "type": "s3",
  "settings": {
    "bucket": "my-open-search-bucket",
    "base_path": "my/snapshot/directory"
  }
}

The documentation needs to explain how the path defined here translates to the mounted volume being used. Is it relative to the WKDIR defined in the Dockerfile? But that would only apply to Docker images. Does it use an environment variable like OPENSEARCH_HOME?
I think it probably must be an absolute path, but since this needs corrected anyway let's find out what happens when you specify a relative path and include that on the page as well.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
I don't have any additional info - if help is needed we'll have to rope in one of the devs.

Removed "1.3" from the title because I realized that it's the same across all the maintained versions.

After a little further thought I think what might need to happen is that S3 and fs repositories should probably be explained separately unless the differences and distinctions are few enough to just call out differences as needed.
But we should have a clear understanding of two behaviors:

When you're using an S3 bucket as a snapshot repository, how does the path specified translate to s3://<web-bucket>/? In the example used would it look like s3://my-open-search-bucket/my/snapshot/directory/? I'm curious about how it concatenates the strings to form the URI.
When you're using a mounted fs volume for the repository, how is the path resolved? For example, if I mount a volume to /mnt/snapshots and then want to register a snapshot that would be stored in a directory called some_directory then what would the command look like?

I also noticed that we don't define location very well for the fs repo:
The shared file system for snapshots. Required.

That doesn't describe what the parameter does at all. Can we do a pass on required fields and make sure they're all defined clearly?

It looks like this is much more comprehensively documented at https://opensearch.org/docs/latest/tuning-your-cluster/availability-and-recovery/snapshots/snapshot-restore/#register-repository. Thinking of the best way to integrate these together.

I can confirm that the steps and config parameters are correct as described in the walkthrough above. base_path only exists for the s3 config so the type of path is irrelevant. I also confirmed that when setting up an s3 repository that the s3:// prefix is not required and I was able to successfully take snapshots to both s3 and a local path. What I'm going to do however since the overlap between the user guide and API reference is confusing, I'm going to restructure the two pages.


Adding styling to Contact Us Page to make it look more lively

Feature request-
Contact us page of website looks very plain. I wanted to add some styling and effects to the Contact us page so it will look more lively.
Is your feature request related to a problem? Please describe. (optional)
A clear and concise description of what the problem is.
Describe the solution you'd like
Adding styling to the paragraph written over there like other websites will look more appealing to visitors. Also instead of directly giving links I can add a favicon and will provide the mail link after hovering over them by using some effects.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Describe how this feature will be useful to our readers.
A clear and concise description of how this feature will be useful for our  other readers.
Additional context
Add any other context or screenshots about the feature request here.

Note:

If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.

/assign

Should I start working o this? @AnanyaNegi @Pranav016 @RushanKhan1 @shudhanshu03

Should I start working o this? @AnanyaNegi @Pranav016 @RushanKhan1 @shudhanshu03

Go on , I have checked no one is working on similar issue . You are already assigned .

/gssoc/

/level2/

Sure thanks. I will make a PR in two days.

Would you mind telling me what I have done wrong? @shudhanshu03 @AnanyaNegi @Pranav016 @RushanKhan1

Hey @ankitaugale23  , I'm participant on GSSOC'21 I want to contribute on this issue will you please assign it to me?

@ankitaugale23 Did you pull the new changes and are you working on the updated code?

Contact us page already updated, reopen issue if you have more improvement suggestions.


Test what happens if someone interjects in your private conversation

If i click a button on @beeme1mr's conversation with davis, will davis respond to me or ignore me? should decide how we want to handle this.

After the slack rewrite for #178 this is no longer an issue. Behavior is as follows.
User clicks another user's button
They are served an ephemeral message that only they can see advising them to wait or speak in a private message
User tries to start a conversation while another user is talking
They are sent a private message informing them that davis is talking to someone else and that they should wait or talk in a private message


Problem with WebBrowser.NavigateToString()

@generalloki
Version : 0f81a342224beaa98e73088392fab971b763c9f6
private void loadWebBrowser()
  {
      string html = "
Sample HTML File

First Sample paragraph.

Second Sample paragraph.
";
      webBrowser1.NavigateToString(html);
  }

Error :

@generalloki
Version : 3298e9d8f0776af1ce72f5de32e104fa6c589640
This issue seems not to be fixed for me :

@neoxeo, Could you please attach the full project with the problem? Also copy/paste error text so I could translate it to English? I have checked the problem in ControlsTest project (test action "Navigate To String") and it works fine. Also is it IE or Edge?

@generalloki
Sorry, yes.
Here is error in english  :

System.AccessViolationException : 'Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
Here is a testproject to reproduce error :
testWebBrowser.zip

I run project from testWebBrowser.zip and it's fine.
Used cmd: dotnet run --framework net8.0

So something could be with your Edge installation. Try to use IE backend. This call sets desired backend version WebBrowser.SetBackend(WebBrowserBackend value)

@generalloki
I will test this quickly. Strange thing is that navigate url works fine

@generalloki
Same error with this code :
private void button1_Click(object? sender, EventArgs e)
{
    // webBrowser1 load local html file
    string html = "
Sample HTML File

First Sample paragraph.

Second Sample paragraph.
";
    //webBrowser1.Navigate("https://google.com");
    webBrowser1.NavigateToString(html);
}

@generalloki
After clean .nuget cache and generate new nuget packages with latest version, all works fine with Edge and IE.
I don't know what is the reason of error but now it is solve. Sorry.


Update made to starter-notebook and a submission csv file was created

Excellent initial model, let's keep improving it.


ci: switch to Github Actions

Run lint once, on LTS Node.js
Run tests on all supported platforms
Drop Node.js 11 which is EOL for a long time already

What's the current status on this one?


Question : advice on how to integrate with storybook

Hi,
Thank you for this great tool! It makes it super easy to use and configure handlebars and nunjucks templates in my vite projects.
I was wondering if you had any advice regarding integration with storybook's vite builder. I have use webpack and handlebars-loader webpack plugin https://github.com/pcardune/handlebars-loader to load .hbs templates inside component.stories.js storybook files in the past... I don't have a lot of experience with Vite so I am not sure how to go about it, since when I try to import an .hbs template inside a js file I get the following error:
Internal server error: Failed to parse source for import analysis because the content contains invalid JS syntax. You may need to install appropriate plugins to handle the .hbs file format, or if it's an asset, add "**/*.hbs" to assetsInclude in your configuration.
any advice on how to get it working would be greatly appreciated.
thank you

In case it helps, I got some initial setup working with the new storybook 7 beta (which uses vite as a builder) and a plugin I put together in order to be able to import the handlebars templates as modules within stories.js files. I'll keep working on it to try to get partials working as well. You can check my progress so far in the following repo:
https://github.com/offbeat-dev/offbeat_storybook-v7-vite-hbs


On connection resume the extension list is duplicated

If the bot resumes connection (I.E. PC was asleep and woken up, with the bot running) every extension in the loaded list is duplicated.

This also affects the server list.

Should be fixed now.


Autoupdate

chipmunk should support auto-updating. Scopes:

chipmunk as it is
plugins and addons

close. With version 1.20.0 implemented update mechanism for portable versions only. Doesn't require any certificates across all platform.


[UPDATE ALGORITHM]  Kadane's Algorithm

Description:
The current implementation of Kadane's Algorithm can be improved for better functionality and usability. The following updates are proposed:
1.Edge Case Handling: Ensure the algorithm can handle an empty array input, where no subarray exists.
2.Dynamic Memory Allocation: Utilize dynamic memory allocation (malloc) for handling large input sizes efficiently.
3.Input Validation: Add validation to handle incorrect input for both array size and elements.
4.Modular Code Structure: Separate input handling, result computation, and result display into individual functions for improved code organization and readability.
These changes are necessary to enhance the robustness of the algorithm and make it scalable for larger inputs, while improving the user experience by handling potential input errors.
Checklist:

[YES ] Contributor in GSSoC-ext
[YES ] Want to work on it

Affected Algorithm:
Algorithm: Kadane's Algorithm
Directory: /algorithms/dynamic_programming/kadane_algorithm.c
Additional Information:
By implementing dynamic memory allocation, the algorithm will be capable of processing large datasets. Input validation ensures that invalid data will not cause unexpected behavior, thus improving the program's robustness. Additionally, the modular code structure will facilitate future updates or maintenance.

@13Sharad  Hey, I hope you doing well. I like to contribute to this issue. Can you assign this to me ? Thank you !

Do not create unnecessary PR until you complete the previous one.


Tabs Invalid color format for '#'

Description of the Bug
The code examples in:
https://kivymd.readthedocs.io/en/0.104.0/components/tabs/index.html
https://kivymd.readthedocs.io/en/0.104.0/themes/color-definitions/index.html
Generate multiple error messages:
[WARNING] [ColorParser ] Invalid color format for '#'
I think generated by the Tabs Component.
On Windows when using pythonw (so app is a new process) these example crash
On Windows when using python these examples run with the above error message repeated.
Introduced with 0.104.0
Code and Logs
Screenshots
Versions

OS:  win10
Python:  3.6.1
Kivy: 1.11.1
KivyMD: 0.104.0

Thank you for the prompt response.


Firefox extensions

I would like to install firefox with additional extensions / add-ons from my configuration file.
What is the default procedure to add plugins for applications available in the nixpkgs expressions? Do they have to be maintained manually? Keep in mind there are probably thousands of add-ons with constantly changing versions and checksums.
I wrote a little script that can be added to a build phase (e.g. postInstall). It automatically downloads & installs the extensions specified via an additional extensions ? [ ] function parameter.
The url attribute:
extension_urls = map (name: "https://addons.mozilla.org/firefox/downloads/latest/${name}/addon-${name}-latest.xpi") extensions;

The script:
# download extensions and extract them to installation folder
for xpiurl in $extension_urls; do
  wget $xpiurl -O tmp.xpi
  id=$(unzip -qc tmp.xpi install.rdf | xmlstarlet sel \
      -N rdf=http://www.w3.org/1999/02/22-rdf-syntax-ns# \
      -N em=http://www.mozilla.org/2004/em-rdf# \
      -t -v \
      "//rdf:Description[@about='urn:mozilla:install-manifest']/em:id")
  mkdir -p $out/lib/browser/extensions/$id
  unzip tmp.xpi -d $out/lib/browser/extensions/$id
  rm tmp.xpi
done

See installing extensions for more details.

The problem of this approach is that you have no idea which version of the extension you got and unless there is a Firefox update, you will get no update of the extensions.  Basically, you should be doing no network access at all, except under fetchURL.
Thus the extensions would have to be listed with their checksum, in order to verify that you are not downloading a compromised version.  As you don't want a modified version, you probably don't want to have a URL which is possibly changing over time.  Do addon.m.o have a static address for addons?  Can we get a listing with all addons versions?
Some addons are also shipped with some binaries, it would be nice if we could wrap these binaries in a way which is working with Nix packages, and avoid references to /lib.

Thus the extensions would have to be listed with their checksum, in order to verify that you are not downloading a compromised version.

The extensions are served over https. Doesn't that implicitly guarantee that we can't retrieve compromised versions? (The identity of the addons.mozilla.org host is validated using a chain of certificates starting at the root certificate installed on our system. The data transferred between AMO and our system is encrypted).

Do addon.m.o have a static address for addons?

https://addons.mozilla.org/firefox/downloads/latest/<id>/addon-<id>-latest.xpi  for latest version
https://addons.mozilla.org/firefox/downloads/file/<id>/<name>-<version>-<applications>-<platform>.xpi for specific version (<applications> seems to be either fx for FF only or tb+fx for FF & Thunderbird, <platform> seems to be linux).

Can we get a listing with all addons versions?

Haven't found a web service or something more accessible unfortunately. I can only think of web page parsing to extract all that information.

I found another tutorial about how to integrate add-ons into firefox. The corresponding tool may allow installing a list of extensions after firefox has been installed. I will investigate further.

As a side note, there is the possibility to specify & lock-down a lot of the configuration options for firefox at install time, using an enterprise version of firefox with a slower release cycle - have a look at the documentation. Could this be a potential new firefox-esr package?

Thus the extensions would have to be listed with their checksum, in order to verify that you are not downloading a compromised version.

The extensions are served over https. Doesn't that implicitly guarantee that we can't retrieve compromised versions? (The identity of the addons.mozilla.org host is validated using a chain of certificates starting at the root certificate installed on our system. The data transferred between AMO and our system is encrypted).

No, the security of the transport does not ensure that the content is not changed.  Plus we use the checksum to store the compressed version in the nix store.

Can we get a listing with all addons versions?

Haven't found a web service or something more accessible unfortunately. I can only think of web page parsing to extract all that information.

Hum … I don't think scrapping any website is a good idea.  Maybe @clouserw will have a better idea on how to get a list of all addons with their checksums.

As a side note, there is the possibility to specify & lock-down a lot of the configuration options for firefox at install time, using an enterprise version of firefox with a slower release cycle - have a look at the documentation. Could this be a potential new firefox-esr package?

We don't need an ESR version to change the preferences of Firefox, but I will not recommend abusing this, as this will provide a different user experience.

I don't have good news for you. :(
Add-ons used to be hosted on ftp.mozilla.org.   See https://bugzilla.mozilla.org/show_bug.cgi?id=1074960 for that discussion.  What you want, a feed of add-on versions, is https://bugzilla.mozilla.org/show_bug.cgi?id=1140538.  With a single developer working on AMO though, it's not at the top of any priority lists.
It's cliche, but if you're interested in writing a patch for it, we could help you get the site up and running.  Find us in #amo on irc.mozilla.org if so.  Cheers.

Can this be closed?

Would it be correct to say that the only declarative plugins for Firefox are those specified inside the firefox wrapper expression, anything else would have to be installed imperatively, or somebody has to write an expression for it and put it into nixpkgs?


Under macOS 15, your program has a bug. After restarting the computer, there will be a blank window on the desktop.

Under macOS 15, your program has a bug. After restarting the computer, there will be a blank window on the desktop.~
v1.0.3~

Can confirm !

Can confirm ! 可以确认!

I referenced the author's related Swift code and developed one using Python 3. I will wait to see if the author updates it recently. If not, I will create a repository and release it.

pretty sure this should still be open. it's not resolved here or in the MAS version

still seeing it on sequoia


installation failure on Windows 7

using Rust 1.37:
> cargo install cargo-generate

…

error: failed to run custom build command for `openssl-sys v0.9.49`

Caused by:
  process didn't exit successfully: `C:\Users\GEOBER~1.QUA\AppData\Local\Temp\cargo-installtZN3dn\release\build\openssl-
sys-947476c6f3d76372\build-script-main` (exit code: 101)
--- stdout
cargo:rustc-cfg=const_fn
running "perl" "./Configure" "--prefix=C:\\Users\\GEOBER~1.QUA\\AppData\\Local\\Temp\\cargo-installtZN3dn\\release\\buil
d\\openssl-sys-f6d269dfd9654d51\\out\\openssl-build\\install" "no-dso" "no-ssl3" "no-unit-test" "no-comp" "no-zlib" "no-
zlib-dynamic" "no-engine" "no-async" "no-asm" "VC-WIN64A"
Configuring OpenSSL version 1.1.1d (0x1010104fL) for VC-WIN64A
Using os-specific seed configuration

--- stderr

******************************************************************************
This perl implementation doesn't produce Windows like paths (with backward
slash directory separators).  Please use an implementation that matches your
building platform.

This Perl version: 5.26.2 for x86_64-msys-thread-multi
******************************************************************************
thread 'main' panicked at '

Error configuring OpenSSL build:
    Command: "perl" "./Configure" "--prefix=C:\\Users\\GEOBER~1.QUA\\AppData\\Local\\Temp\\cargo-installtZN3dn\\release\
\build\\openssl-sys-f6d269dfd9654d51\\out\\openssl-build\\install" "no-dso" "no-ssl3" "no-unit-test" "no-comp" "no-zlib"
 "no-zlib-dynamic" "no-engine" "no-async" "no-asm" "VC-WIN64A"
    Exit status: exit code: 127

    ', C:\Users\Geobert.Quach\.cargo\registry\src\github.com-1ecc6299db9ec823\openssl-src-111.6.0+1.1.1d\src\lib.rs:363:
13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

warning: build failed, waiting for other jobs to finish...
error: failed to compile `cargo-generate v0.4.0`, intermediate artifacts can be found at `C:\Users\GEOBER~1.QUA\AppData\
Local\Temp\cargo-installtZN3dn`

Caused by:
  build failed

@Geobert could you pls check if that issue is still happening with the latest version?

Hi, unfortunately I don't have access to the machine where I encounter this error anymore :-/ Let's close this.


[release-0.7] Prep docs and references for v0.7.0 release

Signed-off-by: hasheddan<EMAIL_ADDRESS>
Description of your changes

Prep docs and references for v0.7.0 release
Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[x] Updated any relevant documentation, examples, or release notes.
[ ] Updated the dependencies in app.yaml to include any new role permissions.

can you make the target branch the 0.7 release branch?


Update Istio objects for deploying the adapter

Updating the istio objects for deploying the latest adapter.

@PhilipGough just copied the ones on testdata and added an extra rule to avoid applying 3scale adapter to the whole service mesh and denying all traffic by default.


DynamoDB - Support list_append operation

Fixes #847
Adds a test to verify the original use case is now working
Adds functionality to support  list_append operations when updating DynamoDB items

Coverage decreased (-0.07%) to 93.485% when pulling 730c4be1a3f91ec7dbacdf61dd5f70118dc6670d on bblommers:feature/847 into b6fe526addcfec6fe981a75d81d4025a9fbabc65 on spulec:master.


[Feature Request] Add configurable Height and Width for both level of the ship's container upgrade

Right now you can only modify Width and then Height, but I already have changed the base size of the ship container with another mod and I would like the upgrades to just add more height.

If you have other mod altering container size you can just increase it further. Providing compatibility with such mods is a tricky thing.


VNET selective peering

Azure documentation issue guidance
https://learn.microsoft.com/en-us/azure/templates/microsoft.network/virtualnetworks/virtualnetworkpeerings?pivots=deployment-language-bicep
and
https://learn.microsoft.com/en-us/azure/templates/microsoft.network/virtualhubs/hubvirtualnetworkconnections?pivots=deployment-language-arm-template#vnetroute-1
Azure VNet peering has options to do "selective peering" and route only selected address spaces instead of the whole VNet ip space.
You can drastically reduce lateral movement possibilities by being granular, instead of having the peering route all your CIDRs.
This is not especially advertised in the official documentation, apart from a laconic parameter description (no example nor explanation). I have discovered this using the terraform documentation https://registry.terraform.io/providers/hashicorp/azurerm/3.116.0/docs/resources/virtual_network_peering
I have tested in my lab and is working.
First create your resources and you can copy/paste the script and to run it from Azure Cloud Shell.
#Enable the feature
az feature register -- namespace Microsoft.Network -- name AllowMultiplePeeringLinksBetweenVnets
#Create your peering
az network vnet peering create --name test-peer --local-subnet-names subnet2 --remote-subnet-names default2 --peer-complete-vnets false --vnet-name vnet1 --remote-vnet vnet2 -g test

Best regards,
Andrei Pintica
Azure IaaS VM Sr. Support Escalation Engineer from CSS ARTS VM EMEA Team

@andreipintica
Thanks for your feedback! We will investigate and update as appropriate.

Looking forward to hearing from you.

@tfitzmac Kindly, any updates around this ?

The docs are generated from Azure REST API specifications, so updates to the property descriptions would need to happen there. https://github.com/Azure/azure-rest-api-specs/tree/main/specification/network/resource-manager/Microsoft.Network

The docs are generated from Azure REST API specifications, so updates to the property descriptions would need to happen there. https://github.com/Azure/azure-rest-api-specs/tree/main/specification/network/resource-manager/Microsoft.Network

@tfitzmac OK, I understand when you are going to publish this update ?
Best regards,
Andrei Pintica
Azure IaaS VM Sr. Support Escalation Engineer from CSS ARTS VM EMEA Team

@andreipintica - those doc descriptions are handled by the Network team. Please work with them to update the REST API specification.


cmd/nm: TestGoExec failing on Plan 9

CL 115975 changed TestGoExec. This test is now failing on Plan 9.
--- FAIL: TestGoExec (1.49s)
    nm_test.go:161: want R type for runtime.epclntab symbol, but have D
    nm_test.go:161: want R type for runtime.erodata symbol, but have D
    nm_test.go:161: want R type for runtime.rodata symbol, but have D
FAIL
FAIL	cmd/nm	4.320s

See https://build.golang.org/log/b4bb61cd5942848398474e07807984e34817bceb

CC @alexbrainman

Sorry about braking plan9 builder. I did not even noticed.

--- FAIL: TestGoExec (1.49s)
nm_test.go:161: want R type for runtime.epclntab symbol, but have D
nm_test.go:161: want R type for runtime.erodata symbol, but have D
nm_test.go:161: want R type for runtime.rodata symbol, but have D
FAIL

It probably is not important if runtime.epclntab (and others) is "R" or "D". I do not know.
Maybe adjust the test to expect "D" for runtime.epclntab on plan9? Maybe just skip that part of the test?
I will let you decide.
Alex

I think it should be fine if we expect "D" runtime.epclntab on Plan 9.


exception

1:when I use : client.Connect(device.DeviceAddress, BluetoothService.Handsfree); exception is following:
System.AggregateException:“One or more errors occurred. (Specified argument was out of the range of valid values. Arg_ParamName_Name)”
2:when I use : client.Connect(device.DeviceAddress, BluetoothService.SerialPort); above exception is not invaked ,but have following exception: DataAvailable = “((InTheHand.Net.Sockets.WinRTNetworkStream)stream).DataAvailable”  “System.NotSupportedException” exception
how can I do it?

When I use bluetooth in Maui, it's error :Client = “client.Client” invoke “System.PlatformNotSupportedException” exception.


question about sample details？

Hello, thank you for your great work and generous contribution.  There‘re two details in code: (1)  in the script(train_dtu.sh), # make
sure num_depth* interval = 203.45. (2) in t&t datasets, the depths are reversed. What's the role of these two operations？

It's confusing and complicated to explain indeed. DTU gives a fixed depth range for each view, so the most ideal way to deliver the dataset is to give the min depth as well as the max depth and let users to apply their own ways of depth division. However, the dataset (or to say, the preprocessed version) assumes a default D, so it gives the depth interval instead (2.5mm in the text). As a result, if we want to change the value of D, we need to leverage a new parameter to scale the interval to make sure the range of depth is still the same.
I suppose you are referring to the flag of inverse_depth. It is an advantage with recurrent regularization, and you can check the paper of R-MVSNet for details.

Thank you a lot


[SUggestion] the ability to load files into the vpcs.

On Linux you can use 9pfs: https://github.com/copy/v86/blob/master/docs/filesystem.md
For other OSes we already have an issue: https://github.com/copy/v86/issues/824

what about windows 2000?

Please read:

For other OSes we already have an issue: https://github.com/copy/v86/issues/824


Programs and tools for creating documentation

Looking for any tools we can recommend for helping people create documentation or test usability or really anything that we discuss in the course.

https://readme.com is one to try out.

https://readthedocs.org/

Mkdocs has been super easy to use in conjunction with github pages: https://www.mkdocs.org/#installation Been writing the course to use this in #10.

A draft of this is set up over on https://github.com/jhudsl/itcr-template-documentation


isso: <IP_ADDRESS> -> 0.13.0

Description of changes
This uses buildNpmPackage now and Node.js 20.
Also, packages the HTML documentation in a new output.
https://isso-comments.de/news/#isso-version-0-13-0-released
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Contributes towards #229910.

Result of nixpkgs-review pr 233399 run on x86_64-linux 1

3 packages built:

isso
isso.dist
isso.doc

Result of nixpkgs-review pr 233399 run on x86_64-linux 1

3 packages built:

isso
isso.dist
isso.doc


docs: update acbuild link

@lucab done

@lucab done

hmm, not sure this went through, still looks like the old commit title..

@jonboulle Doesn't seem like it to me..

@cdrage I think we were talking about the commit message, e.g. "Change acbuild links from appc/build to containers/build " --> "docs: update acbuild link" or similar. Sorry for the confusion..

@jonboulle Ohhhh, I didn't quite catch that in the contributing link... I'll change that.

Thanks :-)
Here's the specific link: https://github.com/coreos/rkt/blob/master/CONTRIBUTING.md#format-of-the-commit-message

@jonboulle done :)

woohoo! Thanks for your patience!

Skipping CI for simple docs change.


Convert other errors to diagnostics when possible

I think this converts the rest of the error events that are caused by file locations to use diagnostics.
The commits are pretty targeted and should be fairly easy to review one at a time.

Looks like some Windows failures relating to path separators. Probably need to substitute ${pathlib.sep} into some test cases.

(this is ready for review, the errors are github cache errors, will rerun shortly)


Suggesting to use an outside cron job to update nodes?

Are we really suggesting it's best practice to setup a third party cron job to apply security updates to nodes?
Why would we do this versus using a built tool like Kured? Seems like a dangerous suggestion.

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

ID: 9659eab1-b7f7-d163-ff0a-577b7496b5c8
Version Independent ID: 0892de76-78f8-4c1c-c09c-60af3f275320
Content: Handle AKS node upgrades with GitHub Actions - Azure Kubernetes Service
Content Source: articles/aks/node-upgrade-github-actions.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

I see you have documentation for Kured: https://docs.microsoft.com/en-us/azure/aks/node-updates-kured

@Aaron-ML
Thanks for the feedback! We are currently investigating and will get back on this.

@Aaron-ML Apologies in delayed response.
I checked with internal team on this, GitHub actions are first party. Kured is an opensource project with bugs and is not a supported tool. Kured with unattended upgrades just upgrades the running VMs, new VMs will come with the old image.
Based on your requirements, you use the one that is better for your environment.
Hope this helps.

@vikaskumawat80
Interesting that a workflow automation tool is first party to update AKS nodes. Haven't had any issues with Kured on bugs.
Not a big fan of uploading creds on to a GHA job and having an outside system upgrade my nodes. Maybe one day we could have this option that is actually integrated? https://cloud.google.com/kubernetes-engine/docs/how-to/node-auto-upgrades


[Fud] Rename SourceType.Terminal to something less confusing

This name has caused unnecessary confusion several times now and should probably be named something else.
SourceType.Interactive would probably be a choice that better captures the intuition.
This should be relatively easy and I'm happy to tackle it later but feel free to scoop me if you wanna kill a handful of minutes.

Since we're moving to fud2 (#1878) this will no longer be relevant


[bug]: Dialog opening without trigger?

Describe the bug
I'm using the example from https://ui.shadcn.com/docs/components/dialog and when I use the component, the Dialog automatically opens without any trigger? Also I can't close it. Is this a bug or am I doing something wrong?
import { Button } from "@/components/ui/button"
import {
  Dialog,
  DialogContent,
  DialogDescription,
  DialogFooter,
  DialogHeader,
  DialogTitle,
  DialogTrigger,
} from "@/components/ui/dialog"
import { Input } from "@/components/ui/input"
import { Label } from "@/components/ui/label"

export function DialogDemo() {
  return (
    <Dialog>
      <DialogTrigger asChild>
        <Button variant="outline">Edit Profile</Button>
      </DialogTrigger>
      <DialogContent className="sm:max-w-[425px]">
        <DialogHeader>
          <DialogTitle>Edit profile</DialogTitle>
          <DialogDescription>
            Make changes to your profile here. Click save when you're done.
          </DialogDescription>
        </DialogHeader>

              Name

            <Input
              id="name"
              defaultValue="Pedro Duarte"
              className="col-span-3"
            />

              Username

            <Input
              id="username"
              defaultValue="@peduarte"
              className="col-span-3"
            />

        <DialogFooter>
          <Button type="submit">Save changes</Button>
        </DialogFooter>
      </DialogContent>
    </Dialog>
  )
}

Affected component/components
Dialog
How to reproduce
Copied the Dialog example and when I use the component, the Dialog automatically opens without clicking on any trigger plus I can't seem to close the Dialog when it's opened.
Codesandbox/StackBlitz link
No response
Logs
No response
System Info
macOs, Chrome

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

Hi @OhYello ,
I was going through this issue and trying to resolve it, but I need more details to understand and replicate the problem.
Could you share the relevant code snippet where you’re using the Dialog component?  OR you could provide the actual project repository or a minified version of the project in a new repository. This will help in reproducing the issue.
Any additional information or context you can provide would be extremely helpful. Thanks

@25zeeshan looks like I found the problem, forceMount was being used in the <DialogPortal> component


Incremental view not implemented yet, for merge-on-read datasets

I am having this exception when i am trying to read a dataset which is stored in MOR format.
When i have gone through the documentation, it says MOR supports incremental pull. Though there is nothing explicit about MOR not supporting incremental views from spark data source.
Below is the spark code.
val df =sc.read. format("org.apache.hudi")
.option(DataSourceReadOptions.VIEW_TYPE_OPT_KEY, DataSourceReadOptions.VIEW_TYPE_INCREMENTAL_OPT_VAL)
.option(DataSourceReadOptions.BEGIN_INSTANTTIME_OPT_KEY, startTime)
.load(basepath)
df.show()
Exception is pasted below.
Pleas let me know if there is something i am missing here
org.apache.hudi.exception.HoodieException: Incremental view not implemented yet, for merge-on-read datasets
at org.apache.hudi.IncrementalRelation.(IncrementalRelation.scala:53)
at org.apache.hudi.DefaultSource.createRelation(DefaultSource.scala:65)
at org.apache.hudi.DefaultSource.createRelation(DefaultSource.scala:46)
at org.apache.spark.sql.execution.datasources.DataSource.resolveRelation(DataSource.scala:318)
at org.apache.spark.sql.DataFrameReader.loadV1Source(DataFrameReader.scala:223)
at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:211)
at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:178)
at com.playngodataengg.dao.DataAccessS3.readDataHudiRefinedS3(DataAccessS3.scala:145)
at com.playngodataengg.controller.RoundDataTransform.processData(RoundDataTransform.scala:30)
at com.playngodataengg.action.RoundData$.main(RoundData.scala:15)
at com.playngodataengg.action.RoundData.main(RoundData.scala)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at com.intellij.rt.execution.CommandLineWrapper.main(CommandLineWrapper.java:66)

@HariprasadAllaka1612  I need to dig into this issue more. But I am quite sure you can incrementally query MOR datasets. Here is an example from the Docker demo page which does this for reference - https://hudi.apache.org/docker_demo.html#incremental-query-with-spark-sql

@HariprasadAllaka1612 Looks like spark sql for incremental queries work on a Hive table. The Docker demo that I pasted also refers to a Hive table. For spark datasource this not supported natively. I ll create a tracking Jira issue for this and update docs.

@HariprasadAllaka1612 This is still not implemented for Spark datasource but we plan to implement it soon. @bhasudha  please find the ticket here : https://issues.apache.org/jira/browse/HUDI-58

closing this in favor of https://issues.apache.org/jira/browse/HUDI-58

@bhasudha - all the examples on https://hudi.apache.org/docs/quick-start-guide.html#setup-spark-shell for PIT queries are for reading from disk. Does documentation exist for Incremental queries from hive tables as you mentioned?

@bhasudha - all the examples on https://hudi.apache.org/docs/quick-start-guide.html#setup-spark-shell for PIT queries are for reading from disk. Does documentation exist for Incremental queries from hive tables as you mentioned?

We have a docker demo setup that brings up all services and runs queries from Hive, spark and Presto on actual Hive tables. You can refer to that page for sample commands - https://hudi.apache.org/docs/docker_demo.html#step-7-a-incremental-query-for-copy-on-write-table.

@n3nash - From your comment on 11-08-2019 this would be implemented soon, but I can't tell from https://issues.apache.org/jira/browse/HUDI-58 if it is now implemented. Any ETA on when this will be supported?

@WilliamWhispell Work for this is tracked in HUDI-651, HUDI-920, all tracked for 0.6.0, end of the month.
@bhasudha has the changes, to make this work via SparkSQL/Hive. HUDI-920 is about natively pulling the changs usi ng spark datasource


Android Basics: Shared ViewModel

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-shared-viewmodel?authuser=1&continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-3-pathway-4%3Fauthuser%3D1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-shared-viewmodel#2
In which task and step of the codelab can this issue be found?
Task: 3
Step: "Update title in app bar"
Sub-step: 4
Describe the problem
The text "Open navigataion/nav_graph.xml" should read "Open navigation/nav_graph.xml"
Additional information
Include screenshots if they would be useful in clarifying the problem.

@hassan0599
Thank you so much for reporting this issue. This is fixed and will be published soon.


Add Kafka

This issue is a:
[ ] Bug Report
[ ] Enhancement suggestion
[*] Feature request
Enhancement suggestions / feature requests
If you are requesting an enhancement to a feature already present, or a new feature, please outline below the feature you would like to see, and how it would benefit you and the wider Civo Kubernetes userbase:
Example chart
https://bitnami.com/stack/kafka/helm

Sure, We will give it a look to add.

@saiyam1814 Kafka added with #178  issue can be closed


docs: remove fntlnz

What type of PR is this?

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

/kind bug
/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

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

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Checklist

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] CHANGELOG.md updated

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

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

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

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

b864529 docs: remove fntlnz

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Closing in favour of https://github.com/falcosecurity/charts/pull/368
/close

@leogr: Closed this PR.

In response to this:

Closing in favour of https://github.com/falcosecurity/charts/pull/368
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Latest gcloud sdk 349.0.0 breaks kpt installed manifests

While running through the installation guide I encountered a blocker that required me to specifically downgrade the Google Cloud SDK installed version of kpt because the current manifests do not have the correct API version (v1alpha1 instead of v1) for the latest version of kpt - 1.0.0-beta1.
Specifically I was running through the guide in cloud shell as I have a Mac as my dev machine and the guide said bash would be an issue there.
I hit the following error (after the config connector stuff which i've omitted for brevity):
....
Build directory: ./build
Component path: common/asm
Apply component resources: common/asm
Found Makefile, call 'make apply' of this component Makefile.
make[1]: Entering directory '/home/aknight/gcp-blueprints/kubeflow/common/asm'
curl https://storage.googleapis.com/csm-artifacts/asm/install_asm_1.9.3-asm.2-config2 > install_asm;
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 79280  100 79280    0     0  7038k      0 --:--:-- --:--:-- --:--:-- 7742k
rm -rf ./asm
kpt pkg get<EMAIL_ADDRESS>./
Package "asm":
Fetching<EMAIL_ADDRESS>From https://github.com/GoogleCloudPlatform/anthos-service-mesh-packages
 * tag               1.9.3-asm.2+config2 -> FETCH_HEAD
Error: Kptfile at<EMAIL_ADDRESS>has an old version ("v1alpha1") of the Kptfile schema.
Please update the package to the latest format by following https://kpt.dev/installation/migration.
make[1]: *** [Makefile:43: download-asm-package] Error 1
make[1]: Leaving directory '/home/aknight/gcp-blueprints/kubeflow/common/asm'
make: *** [Makefile:67: apply] Error 1

When I run kpt version I see: 1.0.0-beta.1 which in the cloud shell case is installed by the apt package:
Package: google-cloud-sdk-kpt
Version: 349.0.0-0

I looked at the kpt release notes and found this entry: https://github.com/GoogleContainerTools/kpt/releases/tag/v1.0.0-beta.1 which would mean the manifests need to be upgraded to v1
I was able to work around this by forcing a downgrade on the gcloud sdk installed version of kpt in cloud shell via this command: sudo apt-get install google-cloud-sdk-kpt=346.0.0-0 which is prior to this version in which gcloud sdk upgraded the bundled version of kpt: https://cloud.google.com/sdk/docs/release-notes#34900_2021-07-20
With that workaround in place it installed just fine but as it stands the quickstart does not work with the latest kpt in general and also does not work with the latest kpt installed by the Google Cloud SDK.

Thanks for reporting the issue @ilkelma!
/cc @zijianjoy
we will be supporting kpt v1 starting from Kubeflow 1.4.
Downgrading gcloud or installing a separate kpt binary https://github.com/GoogleContainerTools/kpt/releases/tag/v0.39.2 is current workaround.
Welcome contributions on doc updates or integrating kpt v1 with Kubeflow 1.3 if you are interested

This issue should have been resolved with the latest gcloud version and kpt version (kpt v1.0.0-beta.6 or above). Would you like to give it a try? Thank you!

Marking this issue as resolved as we have released Kubeflow v1.4 for a while now. Feel free to reopen this issue if problem still persists.


Remove highline gem

Describe the Enhancement:
Highline dependency is too restrictive.
Describe the Need:
It prevents other gems from being updated.
Current Alternative
#8381 already supplied the alternative
Can We Help You Implement This?:
From searching the code it doesn't seem like highline is being used at all. It could be as simple as removing the gem and unused methods.

@reedlaw There's a few other places we use highline that need to get updated to use the tty gems. #8381 was the first part of this work, but it was also the easy part.

./lib/chef/knife/core/ui.rb and ./lib/chef/formatters/indentable_output_stream.rb still use it.

highline.ask should be pretty easily replaceable by tty-prompt.
I can't figure out a way to get tty-table to look like highline.list
2.7.0-preview1 :008 > HighLine.new.list(['one', 'two', '12' ], :columns_down)
 => "one  two  12 \n"
2.7.0-preview1 :009 > HighLine.new.list(['one', 'two', '123' ], :columns_down)
 => "one  two  123\n"
2.7.0-preview1 :010 > HighLine.new.list(['one', 'two', '1234' ], :columns_down)
 => "one   two   1234\n"
2.7.0-preview1 :011 > HighLine.new.list(['one', 'two', '12345' ], :columns_down)
 => "one    two    12345\n"

There was a braking change introduced by switching to pastel:
irb(main):015:0> highline.color("test", [:bold, :white])
=> "\e[1m\e[37mtest\e[0m"

irb(main):016:0> pastel.decorate("test", [:bold, :white])
Traceback (most recent call last):
       16: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/bin/irb:23:in `<main>'
       15: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/bin/irb:23:in `load'
       14: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/irb-1.2.3/exe/irb:11:in `<top (required)>'
       13: from (irb):16
       12: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/2.7.0/forwardable.rb:235:in `decorate'
       11: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:60:in `decorate'
       10: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:140:in `lookup'
        9: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:140:in `fetch'
        8: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:141:in `block in lookup'
        7: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:153:in `code'
        6: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:153:in `each'
        5: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:158:in `block in code'
        4: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:236:in `validate'
        3: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:195:in `valid?'
        2: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:195:in `all?'
        1: from /Users/rmoriz/.asdf/installs/ruby/2.7.1/lib/ruby/gems/2.7.0/gems/pastel-0.7.4/lib/pastel/color.rb:195:in `block in valid?'
NoMethodError (undefined method `to_sym' for [:bold, :white]:Array)
Did you mean?  to_s
               to_set

This breaks the knife plugin https://github.com/eligible/knife-bastion/blob/master/lib/chef/knife/bastion_base.rb#L42

Considering this done enough now that it's out of chef itself


BitmapText getBounds is not refreshed if the text is changed

var bitmapText = new PIXI.extras.BitmapText("text using a fancy font!", {font: "35px Desyrel", align: "right"});
bitmapText.text = "New text";

bitmapText.getLocalBounds after this will return correct value, but bitmapText.getBounds will not.
Culprit is that getBounds base implementation, it seems, is not overwritten, and does not call this.validate(), while getLocalBounds does:
https://github.com/pixijs/pixi.js/blob/34cd01eb0f204848972de325b1b85c0b6f7b8c8e/src/extras/BitmapText.js#L351-L361
Base getBounds implementation is not relying on getLocalBounds, thus it still returns incorrect value:
https://github.com/pixijs/pixi.js/blob/34cd01eb0f204848972de325b1b85c0b6f7b8c8e/src/extras/BitmapText.js#L351-L361
Sorry, creating full-blown JSFiddle requires setting up bitmap font somewhere and bit hard :). Let me know if above is not sufficient to reproduce, and I'll create working example.

I thought I'd chime in to confirm that the bug is still there in PIXI v3.0.8. Note that the width of getBounds() is different from getLocalBounds() of the unscaled text.

Fixed in v4 👍


Test-DbaDbCompression - Arithmetic overflow error converting bigint to data type numeric.

Bug Report
Run Test-DbaDbCompression against database with BLOB tables (755'105,875 MB).
Got this error - Arithmetic overflow error converting bigint to data type numeric.
General Troubleshooting steps

[x] Verified running latest release of dbatools?
[x] Verified errors are not related to permissions?
[x] Can duplicate in new/fresh PowerShell.exe session?

Version Information

Operating System (Name|Version): WIndows 10 1703
PowerShell Version: 5.1.15063.413
SQL Server (Edition|Version): 11.0.6607

poor report I know...

I think that's enough info 👍

Problem with implicit conversation in tmp_cte


Submission to https://beautifulopen.com/

Ho,
I'd like to suggest https://www.rgraph.net for inclusion into https://beautifulopen.com/
Cheers!

Thanks! I think I'm going to pass on this one for now.

Hi,
Why's that? It makes pretty charts, it's Open Source, it's easy to use and
integrate into your website. Can't go wrong!
Richard
On Mon, 24 Feb 2020 at 15:43, Trek Glowacki<EMAIL_ADDRESS>wrote:

Thanks! I think I'm going to pass on this one for now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/trek/beautiful-open/issues/511?email_source=notifications&email_token=ABF34LEVAH6NVPBRDH275F3REPTK7A5CNFSM4KGKIGBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMYKYUQ#issuecomment-590392402,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABF34LECC73RPSRMTS6E5SLREPTK7ANCNFSM4KGKIGBA
.


graphiql@2

This branch collects all the breaking changes for the next major version of graphiql and will be merged once we're ready to release it.
For context, we decided on this approach because using changeset pre-releases would also block other packages from this repo (vscode-graphql etc.) from publishing non-prerelease versions (see here)
All the changes here have already been reviewed and approved individually:

#2513
#2515
#2540
#2524
#2539
#2543
#2571
#2581
#2582
#2586
#2588
#2592
#2595
#2596

@thomasheyenbrock i had an idea for automating @next releases! This I think this will solve the problem completely.
in the canary script, we can just add conditional behavior for graphiql-v2 and non-forked branches with next/ branch, which skips the canary logic.
instead, it attempts a pre-release if a .changeset/pre.json is present. However in most cases when pushing commits, if the versions haven't been incremented since the last next release, nothing will happen.
so, when you're ready to publish another series of @next tags, just increment the versions where you need them and you're good to go!  You may also want to increment in dependencies, though this can feel redundant with ^ resolution, it's more important when it's ready for a stable release to set a working baseline ofc.
for example, you won't need to make another codemirror-graphql pre-release again unless it changes, so you will be able to just push a commit that increments the next tag versions for graphiql and @graphiql/react and others as needed.
it's actually much simpler than the canary logic itself so it should be easier to implement.

@acao sounds like it should work. I'm just wondering how many next releases we actually need before having v2 ready, so not sure if it's worth the effort now. (At least for me who hasn't touched any of this canary logic it would take me some time to dig into it.)
For future major versions this would definitely be very nice to have!

@thomasheyenbrock or here rather :)


Alternative to setToString

setToString with actions is not very pragmatic as it is now. A bit of code duplication and also risk for forgetting to add/remove actions from the seconds location.
Unfortunately, I don't have any better idea as of yet.

Still very useful. Alternative can be webpack plugin.

It's good enough and dispatcher reports actions without overridden toString.

Using (later) decorators (https://github.com/wycats/javascript-decorators) could be a more pragmatic solution, e.g.
@action
export function logged(authData) {
  dispatch(logged, authData);
}

Yeah, i like it.


[WIP] Cache node_modules in travis and create build-dev to build faster

@michellethomas for the record, testing against a dev build is a mistake: you want to be sure your production build is correct.

I agree with @victornoel on this one.
I think I tried the node_modules in cache before and it actually slowed things down. Maybe because the crazy number of files. At the time I thought we could tar.gz the folder into cache to speed things up. Probably worth a try.

Sure, sorry I was just experimenting to see how much different things would affect timings, not really proposing that we should do it, should have made that more clear in the description.


Fix url for redis service documentation

This PR replaces the currently dead link, with the correct url for the Redis Service documentation.

This looks good, however there is a small change required to https://github.com/pivotal-cf/cf-redis-broker/blob/master/brokerintegration/catalog_test.go which has the documentation URL hardcoded. We will accept this merge, and make the change to the test as required. Thanks.


[Oversight] Mistake in the new docs

I couldn't figure out why my script was saying my IItemStack import was null, so I started digging through your source and figured it out. In the docs it says minetweaker.item but it should say crafttweaker.item
image for reference
For the record, I love the new docs. They're so much nicer than the old ones.

doc issues go here:
https://github.com/jaredlll08/CraftTweaker-Documentation/issues


core: create hardfork for strict script check

Revert commit 762a8da76a7f387adb639d9b4d125e126e322634. Port the https://github.com/neo-project/neo/pull/2881. Close #3064.
TODO:

[x] Create hardfork-enabling code for strict script check after https://github.com/neo-project/neo/issues/2873 resolution.

Ready for review.


not really an issue

I haven't even looked at the code but I would assume saving the filter view to session that exists be easily modified to save to multiple views in the database.
Does that make sense?
Simple answer or quick recommendation to save me time is enough so I can roll my own solution for this. I can imagine it may be as simple as saving some json to the db with a named identifier and then pulling the json back out and applying.

you can even pull all the data to the frontend

if you have read the code in examples/src before, you may realize that cache all the data in the frontend and render them with some conditions is ok

but you might be aware of that, the data ain't static as the database keeps CURD all the time

close the issue if you have no more questions.

I just want to save the filter options. I assumed that's what your save did
but it sounds like it saves the data too.
Thanks,
Eric Green
Pitbull Audio/CTO
Cell: (619) 985 - 1270
Email: admin<EMAIL_ADDRESS><EMAIL_ADDRESS>On Thu, Feb 15, 2018 at 7:22 PM, Ken Berkeley<EMAIL_ADDRESS>wrote:

but you might be aware of that, the data ain't static as the database
keeps CURD all the time
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OneWayTech/vue2-datatable/issues/71#issuecomment-366134626,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIpKf4TCMh4BUSo1MU5xpiu37ayW00bqks5tVPR_gaJpZM4SH1F7
.

locolStorage.setItem('cond1', this.query)

Should be too hard I hope I've already been tweaking it to remove most of
the jquery stuff that can be done in vue easily and to accommodate for b4.
Thanks,
Eric Green
Pitbull Audio/CTO
Cell: (619) 985 - 1270
Email: admin<EMAIL_ADDRESS><EMAIL_ADDRESS>On Thu, Feb 15, 2018 at 7:40 PM, Eric Green<EMAIL_ADDRESS>wrote:

I just want to save the filter options. I assumed that's what your save
did but it sounds like it saves the data too.
Thanks,
Eric Green
Pitbull Audio/CTO
Cell: (619) 985 - 1270 <(619)%20985-1270>
Email: admin<EMAIL_ADDRESS><EMAIL_ADDRESS>On Thu, Feb 15, 2018 at 7:22 PM, Ken Berkeley<EMAIL_ADDRESS>wrote:

but you might be aware of that, the data ain't static as the database
keeps CURD all the time
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OneWayTech/vue2-datatable/issues/71#issuecomment-366134626,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIpKf4TCMh4BUSo1MU5xpiu37ayW00bqks5tVPR_gaJpZM4SH1F7
.

so that is the query only or both the query and the results of the query.
if it's just the query before the submitting to the backend it shouldn't be
to hard to save that to the database to be used as a preset view.
Thanks,
Eric Green
Pitbull Audio/CTO
Cell: (619) 985 - 1270
Email: admin<EMAIL_ADDRESS><EMAIL_ADDRESS>On Thu, Feb 15, 2018 at 7:42 PM, Ken Berkeley<EMAIL_ADDRESS>wrote:

locolStorage.setItem('cond1', this.query)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OneWayTech/vue2-datatable/issues/71#issuecomment-366136693,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIpKf3y5QvbuMHwORAOVtrMYnU2wlJIsks5tVPk4gaJpZM4SH1F7
.