[webview_flutter: ^4.2.0] Error: Type 'WKPermissionDecision' not found when trying to build flutter app

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
When I try to build with command flutter run, I get following error message.

Launching lib/main.dart on iPhone 14 in debug mode...
Running pod install...                                             10.4s
Running Xcode build...
Xcode build done.                                           21.4s
Failed to build iOS app
Error (Xcode): ../../../../.pub-cache/hosted/pub.dev/webview_flutter_wkwebview-3.4.3/lib/src/webkit_proxy.dart:83:12: Error: Type 'WKPermissionDecision' not found.

Code sample
Code sample
import 'package:journalapp/utils/constant.dart';
import 'package:flutter/material.dart';
import 'package:journalapp/utils/customfirestore.dart';
import 'package:webview_flutter/webview_flutter.dart';

class WebViewPage extends StatefulWidget {
  const WebViewPage({super.key});

  @override
  WebViewPageState createState() => WebViewPageState();
}

class WebViewPageState extends State<WebViewPage> {
  String? privacypolicylink = '';
  String errorMsg = '';

  late final WebViewController _controller;

  @override
  void initState() {
    super.initState();
    loadPrivacyUrl();
    late final PlatformWebViewControllerCreationParams params;

    params = const PlatformWebViewControllerCreationParams();

    final WebViewController controller =
        WebViewController.fromPlatformCreationParams(params);

    controller
      ..setJavaScriptMode(JavaScriptMode.unrestricted)
      ..setBackgroundColor(const Color(0x00000000))
      ..addJavaScriptChannel(
        'Toaster',
        onMessageReceived: (JavaScriptMessage message) {
          ScaffoldMessenger.of(context).showSnackBar(
            SnackBar(content: Text(message.message)),
          );
        },
      );

    _controller = controller;
  }

  loadPrivacyUrl() async {
    try {
      CustomFirestore customFirestore = CustomFirestore();
      privacypolicylink = await customFirestore.loadPrivacyPolicyUrl();
      if (privacypolicylink != null) {
        _controller.loadRequest(Uri.parse(privacypolicylink!));
        setState(() {
          debugPrint('privacypolicylink: $privacypolicylink');
        });
      }
    } catch (error) {
      setState(() {
        debugPrint('Error here: $error');
        errorMsg = error.toString();
        debugPrint(error as String?);
      });
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Constant.primiumThemeSelected
            ? Constant.gradientStartColor
            : Constant.selectedColor,
        title: const Text('Privacy Policy'),
      ),
      body: Builder(builder: (BuildContext context) {
        return errorMsg != ''
            ? Text(
                errorMsg,
                style: const TextStyle(color: Colors.black),
              )
            : (privacypolicylink == ''
                ? const Center(child: CircularProgressIndicator())
                : WebViewWidget(controller: _controller));
      }),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[   +1 ms] executing: [/Users/abdullahkhan/Documents/Development/flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +11 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] 84a1e904f44f9b0e9c4510138010edcc653163f8
[        ] executing: [/Users/abdullahkhan/Documents/Development/flutter/] git tag --points-at 84a1e904f44f9b0e9c4510138010edcc653163f8
[  +82 ms] Exit code 0 from: git tag --points-at 84a1e904f44f9b0e9c4510138010edcc653163f8
[        ] 3.10.0
[   +1 ms] executing: [/Users/abdullahkhan/Documents/Development/flutter/] git rev-parse --abbrev-ref HEAD
[   +8 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[        ] executing: sw_vers -productName
[   +7 ms] Exit code 0 from: sw_vers -productName
[        ] macOS
[        ] executing: sw_vers -productVersion
[   +6 ms] Exit code 0 from: sw_vers -productVersion
[        ] 13.0
[        ] executing: sw_vers -buildVersion
[   +6 ms] Exit code 0 from: sw_vers -buildVersion
[        ] 22A8380
[        ] executing: uname -m
[   +2 ms] Exit code 0 from: uname -m
[        ] arm64
[   +2 ms] executing: sysctl hw.optional.arm64
[   +2 ms] Exit code 0 from: sysctl hw.optional.arm64
[        ] hw.optional.arm64: 1
[   +3 ms] executing: [/Users/abdullahkhan/Documents/Development/flutter/] git rev-parse --abbrev-ref --symbolic @{upstream}
[   +8 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{upstream}
[        ] origin/stable
[        ] executing: [/Users/abdullahkhan/Documents/Development/flutter/] git ls-remote --get-url origin
[   +7 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +34 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +1 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +17 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update.
[        ] Artifact Instance of 'GradleWrapper' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[        ] Artifact Instance of 'FlutterSdk' is not required, skipping update.
[        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'PubDependencies' is not required, skipping update.
[  +39 ms] executing: which security
[  +12 ms] executing: which openssl
[   +5 ms] executing: security find-identity -p codesigning -v
[+6645 ms]   1) 679CC82B4159178CD22C9EDF536E8CD43107E61A "Apple Development<EMAIL_ADDRESS>(93KR2ZGM79)" (CSSMERR_TP_CERT_REVOKED)
                      2) D39D8863060581CA5F633EEDC8ADE58D5C6C98AC "Apple Development<EMAIL_ADDRESS>(93KR2ZGM79)" (CSSMERR_TP_CERT_REVOKED)
                      3) BA733DBC0C578A1C10DA230E8606405CFFAF88B3 "Apple Development<EMAIL_ADDRESS>(93KR2ZGM79)" (CSSMERR_TP_CERT_REVOKED)
                      4) DB100857CA3DA899573D0EF5CEE90BA71A3FC36C "Apple Development<EMAIL_ADDRESS>(93KR2ZGM79)" (CSSMERR_TP_CERT_REVOKED)
                      5) 5B4DFD31447BA2EDC1DFBC79F71B3BAFA077A23C "Apple Development<EMAIL_ADDRESS>(93KR2ZGM79)" (CSSMERR_TP_CERT_REVOKED)
                      6) 4D62CE0F2D7043053780B4C6671F27A1F688A00E "Apple Development<EMAIL_ADDRESS>(93KR2ZGM79)"
                      7) 969CBD88A972D209EC6222BBD7F9998D18436686 "Apple Development<EMAIL_ADDRESS>(QXVLL9Y387)"
                      8) F479579FA988905B4CA5E86A791B90213D862C90 "Apple Development: Sam Klein (3345ST32W3)"
                      9) DFAB52A03184A77EF68BF16D1B46082100A8FF6C "Apple Development: Sam Klein (3345ST32W3)"
                         9 valid identities found

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.0, on macOS 13.0 22A8380 darwin-arm64, locale en-PK)
    • Flutter version 3.10.0 on channel stable at /Users/abdullahkhan/Documents/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 84a1e904f4 (10 days ago), 2023-05-09 07:41:44 -0700
    • Engine revision d44b5a94c9
    • Dart version 3.0.0
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/abdullahkhan/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.1

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

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

Hi @abdullah432
I have checked your code it is because of xcode version which version of xcode are you using ?
You have to use Xcode 13 and if you are using lower version make sure to upgrade version of xcode and try again.
if you are still getting same error even after updating xcode then make sure to let me know.

Hi @abdullah432
I have checked your code it is because of xcode version which version of xcode are you using ?
You have to use Xcode 13 and if you are using lower version make sure to upgrade version of xcode and try again.
if you are still getting same error even after updating xcode then make sure to let me know.

@ziyadgodil I'm using Xcode 14.3 as mentioned in flutter doctor output.

After upgrading to flutter version 3.10.1 from 3.10.0, now it's start working.


Invalid paths to JSON schema files

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
The ./.vscode/settings.json file refers to valid JSON schema files.
Observed Behavior
The ./.vscode/settings.json file specifies invalid paths to JSON schema files.
Steps to Reproduce
Using the SPFx generator v1.3.2:
Create a new onprem project (default values for all prompts):

open the ./.vscode/settings.json file
the file refers to<EMAIL_ADDRESS>which doesn't exist. Instead it should be pointing to<EMAIL_ADDRESS>
Create new online project (default values for all prompts):

open the ./.vscode/settings.json file
the file refers to:

<EMAIL_ADDRESS>instead of<EMAIL_ADDRESS>(different file name)
<EMAIL_ADDRESS>which doesn't exist

We closed this issue as it had not activity within last 180 days. This is a generic process we have decided to perform for issues, which have not been explicitly marked still to be "work in progress" based on tags. We are performing this cleaning to make sure that old issues that have already been solved (but not closed) or are no longer relevant are cleaned out and make the issues more manageable. If this issue still valid, we would ask you to open a new issue and follow the guidance in the issue template related on the recommended location. We do apologize any inconveniences this might cause. Please do remember that issues in the issue lists are also messages for others in the community, so you can also check if you can assist on any of them. “Sharing is caring!”


Group level settings

As per this article (https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/groups-settings-cmdlets) the following settings can be controlled at group level as well as tenant level:
AllowGuestsToAccessGroups (Let group members outside the organization access group content)

AllowToAddGuests (Let group owners add people to the organization)

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

ID: 6bb31720-7494-ab3f-6a3d-5c395649e75d
Version Independent ID: 81c36ae8-fbaa-c411-4bd2-a4db98e249e1
Content: Microsoft 365 guest sharing settings reference
Content Source: Enterprise/microsoft-365-guest-settings.md
Service: o365-solutions
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@officedocsbot assign @mypil

@b6202lam - Thank you for submitting feedback.
I will get this issue over to the Enterprise writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@b6202lam - According to the doc owner, this information is already covered in the Office 365 Groups section of the article as indicated on this comment.
We hope this will address your issue. Thank you.

Hi, I actually don’t see any mentioning of O365 Group-level settings in the article (https://docs.microsoft.com/en-us/Office365/Enterprise/microsoft-365-guest-settings). I only see tenant-level settings. Could you follow up with the content owner?
From: Joyce Y<EMAIL_ADDRESS>Sent: Tuesday, August 20, 2019 12:38 PM
To: MicrosoftDocs/OfficeDocs-Enterprise<EMAIL_ADDRESS>Cc: Bernard Lam<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-Enterprise] Group level settings (#485)
@b6202lamhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fb6202lam&data=02|01|Bernard.Lam%40microsoft.com|5e6649db3460421ea75308d7258cd494|72f988bf86f141af91ab2d7cd011db47|1|0|637019159111420166&sdata=%2FJzLhp7OtHoLsDhJkdcMWWGlyPyVkOyJCVzdN3saZGA%3D&reserved=0 - According to the doc owner, this information is already covered in the Office 365 Groups section of the article as indicated on this commenthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-Enterprise%2Fpull%2F499%23issuecomment-523086586&data=02|01|Bernard.Lam%40microsoft.com|5e6649db3460421ea75308d7258cd494|72f988bf86f141af91ab2d7cd011db47|1|0|637019159111420166&sdata=xtzr4U3DP6MMpMfNy92FdDcpGmQMiyK%2FLgSkAbdA3E8%3D&reserved=0.
We hope this will address your issue. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-Enterprise%2Fissues%2F485%3Femail_source%3Dnotifications%26email_token%3DAJFRJYTRZBENEENV7ERBVCDQFQMYJA5CNFSM4IHVNEIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4W46HI%23issuecomment-523095837&data=02|01|Bernard.Lam%40microsoft.com|5e6649db3460421ea75308d7258cd494|72f988bf86f141af91ab2d7cd011db47|1|0|637019159111430163&sdata=O9MC%2B%2FGyYdWTWFHg0fmQO3HcCy567Bc3ZsHve7r%2FWvo%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJFRJYUH5MUCG43345N7MFDQFQMYJANCNFSM4IHVNEIA&data=02|01|Bernard.Lam%40microsoft.com|5e6649db3460421ea75308d7258cd494|72f988bf86f141af91ab2d7cd011db47|1|0|637019159111430163&sdata=HRyZvboQlcBZq8qcGev5sX7kcFEzTCpDMpC6BppU%2B5Q%3D&reserved=0.

Sorry, @b6202lam, I didn't follow your original comment. I've added a link to the group-level settings info in the section about the tenant level settings.

Ah, I see it now. Thx! (For some reasons I only got the Jul 18 version of the article just an hours ago. Now I’m seeing the Aug 21 version) maybe just timing issue. Sorry for the confusion.
From: Mike Plumley<EMAIL_ADDRESS>Sent: Wednesday, August 21, 2019 5:20 PM
To: MicrosoftDocs/OfficeDocs-Enterprise<EMAIL_ADDRESS>Cc: Bernard Lam<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-Enterprise] Group level settings (#485)
Sorry, @b6202lamhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fb6202lam&data=02|01|Bernard.Lam%40microsoft.com|791fab9ebada4c25674c08d7267d4861|72f988bf86f141af91ab2d7cd011db47|1|0|637020191833908339&sdata=kxv8yGuqvWOzojzr8hDw6PXbV7x53eaZBR00Z8sk6Gk%3D&reserved=0, I didn't follow your original comment. I've added a link to the group-level settings info in the section about the tenant level settings.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-Enterprise%2Fissues%2F485%3Femail_source%3Dnotifications%26email_token%3DAJFRJYTG7W4CYJNKLEP5APLQFWWO3A5CNFSM4IHVNEIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD43FFDY%23issuecomment-523653775&data=02|01|Bernard.Lam%40microsoft.com|791fab9ebada4c25674c08d7267d4861|72f988bf86f141af91ab2d7cd011db47|1|0|637020191833908339&sdata=KtjViYzAuogUKfjlwJ1S0u7YwbZ4s6uDpsUBfUA%2FdpE%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJFRJYROA47FHYSGBEIJC5LQFWWO3ANCNFSM4IHVNEIA&data=02|01|Bernard.Lam%40microsoft.com|791fab9ebada4c25674c08d7267d4861|72f988bf86f141af91ab2d7cd011db47|1|0|637020191833918336&sdata=THndzkyTstl2m1v5nPU7kz8nYI%2FsbA9zd6n3mc%2Fjtns%3D&reserved=0.

@b6202lam - From our understanding, the issue has been resolved. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Duplicate imports cause errors when using react-grid-layout in a pop-up window

In couple of places in your code you import from React more than once. In my application it causes an error when I use react-grid-layout in a pop-up window. The issue is similar to one discussed in the link below and the solution worked for me where I made duplicate imports. Unfortunately in this case the code is not in my hands.
https://stackoverflow.com/questions/62026031/useeffect-error-minified-react-error-321-gtm-implementation-instead-of-google
For example in ReactGridLayoutPropTypes.js you have two separate imports from react:
import React from "react"; import type {   Ref,   ChildrenArray as ReactChildrenArray,   Element as ReactElement } from "react";
In order to solve the issue it should be replaced by:
import React, {   type Ref,   type ChildrenArray as ReactChildrenArray,   type Element as ReactElement } from "react";
Also affected: GridItem.jsx, ReactGridLayout.jsx and utils.js

The lines reading 'import type' are removed when compiled so I doubt this is actually your issue.

The lines reading 'import type' are removed when compiled so I doubt this is actually your issue.

You probably right. I obsessively looked for duplicate React imports as it solved similar issue in another place. Will do more research.

Importing twice is not an issue. It's importing two different copies of React.

While in this specific case it is probably not an issue since you just importing types, duplicated imports actually cause problems.
If you'll look at the link below you'll see under reason number 3 that having more than one copy of React breaks hooks.
https://reactjs.org/docs/error-decoder.html/?invariant=321
In this specific case I probably blamed you by mistake and the problem lies in some component that I render inside the grid.

I think you're misunderstanding that message. It says "You might have more than one copy in the same app."
That's not the same as multiple imports. Importing the same thing more than once does not copy it, it just accesses the same reference more than once. "More than one copy" would be something like a global copy in node_modules/react, then some other copy at node_modules/some_dep/node_modules/react. This happens sometimes, e.g. when a dependency puts react in dependencies rather than peerDependencies, or when you have a version mismatch.

Maybe I mix different issues, but accepted answer for the problem in the link below as well as some comments suggest, that duplicate imports may cause issues and I actually experienced it by myself. In may application I use golden-layout library which provides functionality similar to yours plus allows to put components in tabs and to detach them in a popup. In my case the problems start to show only when I detach components to a pop-up. This is I think a rare use case so not many encounter it IRL.
https://stackoverflow.com/questions/62026031/useeffect-error-minified-react-error-321-gtm-implementation-instead-of-google

Again - imports don't work that way. You could import it a thousand times it won't make a thousand unique objects.

Looks like you are right. I believed in that Stack Overflow post but now I can't recreate my previous issue when I just split React imports in two rows. I'll write here when I'll find what causes my current issue.


Fix furnace, lit furnace to use top texture

Fixes the furnace texture (which was the front) and the lit furnace texture (which was redstone_torch_off)

Included in PR #95


fix: Fixed the bug that DashScopeAiUsage#getTotalTokens() was throwing NPE

Describe what this PR does / why we need it
Does this pull request fix one issue?
Fixes #141
Describe how you did it
Describe how to verify it
Special notes for reviews

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


chore: manual update of the playground

Summary
This PR manually updates the playground. GraphQL linting now works!

It works


ProtoWriterTools methods throw IOException for no reason

Many methods in ProtoWriterTools throw IO exceptions for no clear reason, e.g. writeLong() It results in depending classes / methods like Codec.write() to throw checked exceptions, too.
In my MerkleDb prototype, there are methods to serialize / deserialize objects to / from WritableSequentialData / ReadableSequentialData. These methods don't throw any checked exceptions. When I implemented these method for OnDiskKey and OnDiskValue classes, I found that the corresponding Codec method throw redundant exceptions. For now, I just try / catch / rethrow runtime exceptions, but in the future it would be great if codecs didn't throw anything.

Related to #118


refactor: caching is now a session property

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

https://fusion2.corp.google.com/invocations/5947050d-e187-4379-bd2d-463dd7f22fc3/targets/bigframes%2Fpresubmit%2Fe2e/log Are these failures related to the change?

I don't think so, will see if they fail again when rerunning the test suite.


Considerations for Chapter 5.3.1

First of all thank you for the great book :1st_place_medal:
I encountered some problems in Chapter 5.3.1...
The first problem:
The Chapter includes a the Dockerfile, with is all well and good until it comes to COPY . .
The issue with that command is that it copies everything. This includes unwanted files and folders like .env, .git, target etc.
This folders shouldn't be copied - especially target can get pretty big.
As a solution one can either use a .dockerignore file or use dedicated COPY statements.
Opt-in .dockerignore example:
/**
!./.sqlx
!./migrations
!./src
!Cargo.lock
!Cargo.toml
!configuration.yaml

Note: And Opt-out approach might be better for the book, because it's not easy for beginners to track errors when docker build... fails because of missing files.
The second problem:
In Chapter <IP_ADDRESS> we add our .env file to git.
Quote: Commit the .env file to version control - we will need it in CI soon enough!
In Chapter 5.3.1 is a list of the files in our git root. This list doesn't include the .env file.
Furthermore, if the .env file is copied to docker sqlx will ignore the sqlx-data.json.
Btw.:
Did you try shuttle.rs as an alternative to docker deployments?

Guess I should have read further first, but I was eager to try things out and stumbled across these points.
Maybe it would be a good idea to introduce the .dockerignore a bit earlier?
Especially the target folder seems problematic - mine had a size of 16GB and with the .env in place the build fails no matter what.
However, that's up to you - I considers this issues as discussed. So please feel free to close it whenever you see fit.

The book is built around the "fail first, fix afterwards" philosophy to cement the importance of each solution/pattern by clearly motivating why it's needed. The same applies here for .dockerignore!

I'm not arguing with that and I really like that approach. I'm just saying that while I found some hints that errors are expected or a solution to a problem was provided on previous chapters the same didn't apply here.
Take 4.5.13 where you introduce secrecy as an example.
There is an expected compiler error and some background to it.
The same goes for the previous chapters and for 5.3.4 where running the docker image for the first time comes with some errors.
This flow, for the lack of a better word, didn't apply to 5.3.1 where I couldn't go on because the docker build failed and I was left stuck.

I see what you mean—thanks for clarifying!


[FLINK-8703][tests] Migrate tests from LocalFlinkMiniCluster to MiniClusterResource

What is the purpose of the change
This PR contains the first batch of test ports from LocalFlinkMiniCluster to MiniClusterResource. No port required changes to MiniClusterResource.
Each test has it's own commit and is self-contained. If a test was not explicitly creating a configuration and the number of taskmanagers/slots matched the settings in AbstractTestBase the test was modified to extend that.
Otherwise, a separate MiniClusterResource @ClassRule was created.
Verifying this change
This change is covered by existing tests.

JobManagerHACheckpointRecoveryITCase still use LocalFlinkMiniCluster as it's default one. And we seem to be lost  refactor it. And I dont think it is a very easy work since in this ITCase, it use LocalFlinkMiniCluster.getLeaderGateway. And in the new pattern of AbstractTestBase. I dont know how to get that.

And we also seems lost refactor WebFrontendITCase which still use LocalFlinkMiniCluster.

yes, as mentioned in PR the description this does not port all tests. There are several tests that cannot be ported in isolation as they access specific methods on the LocalFlinkMiniCluster or communicate directly with components via akka.

Thanks @zentol. Yea. I agree with you.

I'm removing AccumulatorErrorITCase from the port due to FLINK-8721.

will merge this, as is, once travis gives a green light. The commits have the rather nice property that they don't require changes to other classes (like the MiniClusterResource); wouldn't want to destroy that by making parallelism protected. As for deriving the parallelism, we could do it but there are in general numerous things we could clean up in these tests.


Fix incorrect indentation in managing-a-custom-domain-for-your-github-pages-site.md

Why:
Closes #22180

What's being changed (if available, include any code snippets, screenshots, or gifs):
Fixes incorrect indentation of the note. Previously it was not indented and would cause the list to be split, with the next item starting at "1." again, see also description of #22180.
Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Now the list numbering seems to be correct, but Markdown rendering is broken for this note:

Maybe this is also only an intermittent issue like the broken rendering in #22180, though that is really weird then. I am seeing this in Firefox and Edge, both in private windows, so I don't think this is caused by the browser cache (?).

@Marcono1234 Thanks for opening a PR! I'll get this triaged for review ⚡
As for the markdown in the note, I followed your advice with Firefox and a private browser and have been unable to recreate it. Perhaps you are right and it was a temp bug or it may have been fixed in an internal PR. Either way, thanks for the heads up about it! ✨


Issue #2929: implement inspect for modules

Implement inspect for modules:

Name of module with file/line info
Scaladoc on module
Inherited modules
Module dependencies
Default task
Implemented tasks

@Shri333 let's move the discussion for this PR here rather than the original issue
I played around a bit myself and it does seem problematic to get the Scala declared methods via java reflection. Rather than using scala reflection at runtime, we can instead use scala reflection in the Discover.scala macro where we already do this kind of thing.
Discover.scala currently looks for every class that is part of our module hierarchy, and generates a tuple of two lists:

A list of mainargs entrypoints, used to support Task.Commands, which we need statically generated so we can perform typeclass resolution on the parameter names
A list of all method names, inherited or not, used to perform "did you mean foo?" error reporting

It should be possible to extend that tuple to include a third list for each class: a list of method names that are directly declared by this particular object. You would then be able to use that information in inspect to return the list of directly declared methods.
This approach is a bit roundabout compared to using scala-reflect at runtime, but it should work, and help us preserve the "no scala reflect at runtime" invariant we've maintained so far

@Shri333 let's move the discussion for this PR here rather than the original issue
I played around a bit myself and it does seem problematic to get the Scala declared methods via java reflection. Rather than using scala reflection at runtime, we can instead use scala reflection in the Discover.scala macro where we already do this kind of thing.
Discover.scala currently looks for every class that is part of our module hierarchy, and generates a tuple of two lists:
1. A list of mainargs entrypoints, used to support `Task.Command`s, which we need statically generated so we can perform typeclass resolution on the parameter names

2. A list of all method names, inherited or not, used to perform "did you mean foo?" error reporting

It should be possible to extend that tuple to include a third list for each class: a list of method names that are directly declared by this particular object. You would then be able to use that information in inspect to return the list of directly declared methods.
This approach is a bit roundabout compared to using scala-reflect at runtime, but it should work, and help us preserve the "no scala reflect at runtime" invariant we've maintained so far

@lihaoyi I was able to implement some basic changes to Discover, but for some reason, I am not able to get the value in inspect (it prints an empty Map when I print discover.value). It prints the mapping in the apply function though. https://github.com/com-lihaoyi/mill/pull/3532/commits/fb20fa98c5be0ba381033bf8e76fd921475458cd

@lihaoyi I pushed some changes (https://github.com/com-lihaoyi/mill/pull/3532/commits/fb20fa98c5be0ba381033bf8e76fd921475458cd) to get all declared methods in Discover.scala and add it to the tuple. When I print out the mapping in Discover.scala, it prints the correct values. However, when I access it in MainModule.scala with discover.value, the Map is empty. Do you know why this could be occurring?

@Shri333 when you say the Map is empty, are you referring to the entire Discover#value field? That needs to get populated by Mill for normal functionality to work, so it shouldn't be empty in the main branch.
You can try cherry-picking your minimal changes onto main to see whether it's still empty there. If it is empty after the cherry pick, it means your minimal changes are breaking something, so you'll need to bisect your minimal changes further to find out who is breaking it. If it's not empty after the cherry pick, then you'll need to bisect your other changes to figure out what is causing it to be empty

@Shri333 when you say the Map is empty, are you referring to the entire Discover#value field? That needs to get populated by Mill for normal functionality to work, so it shouldn't be empty in the main branch.
You can try cherry-picking your minimal changes onto main to see whether it's still empty there. If it is empty after the cherry pick, it means your minimal changes are breaking something, so you'll need to bisect your minimal changes further to find out who is breaking it. If it's not empty after the cherry pick, then you'll need to bisect your other changes to figure out what is causing it to be empty

@lihaoyi I was able to fix the issue. Turns out I was supposed to use evaluator.rootModule.millDiscover rather than Discover[t.module.type].

Also, I noticed that I can't print out information during tests that fail. Is there a way to view stdout/stderr for tests that fail? I am running ./mill main.resolve.test

@lihaoyi if I am not mistaken, resolve itself seems to be working incorrectly. If I have the following build.mill:
package build
import mill._, javalib._

trait TypeA extends Module {
  def foo = Task { "foo" }
}
trait TypeB extends Module {
  def bar = Task { "bar" }
}
trait TypeC extends Module {
  def baz = Task { "baz" }
}
trait TypeAB extends TypeA with TypeB

object typeA extends TypeA
object typeB extends TypeB
object typeC extends TypeC {
  object typeA extends TypeA
}
object typeAB extends TypeAB

Running mill resolve _:TypeA._ produces the following output:
typeA.foo
typeAB.bar
typeAB.foo

But, running mill resolve __:TypeA._ produces the following output:
clean
init
inspect
path
plan
resolve
show
showNamed
shutdown
typeA
typeA.foo
typeAB
typeAB.bar
typeAB.foo
typeB
typeC
typeC.typeA.foo
version
visualize
visualizePlan

Is this expected? This was also the weird behavior I was seeing in the resolve tests. I know that __ matches any # of segments while _ matches one segment, but I was confused about this output.

Thanks @Shri333 ! I'm traveling now but I will try to find time to review this PR as soon as possible

Went through the code and tried it out locally. This looks great!

Nice work @Shri333, and thanks for bearing with me and with the Mill codebase the last 2-3 weeks! Email me your bank transfer details and I will close out the bounty


Minimal web

Also change the navbar font family!

For the content within maintext div, "SAT PREP MADE EASY" is enough!

Ok niranjan
On Wed, 28 Feb, 2024, 10:56 pm Niranjan, @.***> wrote:

And the links don't have to point to anything. Let them be, empty.
—
Reply to this email directly, view it on GitHub
https://github.com/niranjank2022/Ezhuthu_Homepage/pull/2#issuecomment-1969487241,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5HG6Z4PYV7HBU6MGYMUEWTYV5SF7AVCNFSM6AAAAABD6LKOVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRZGQ4DOMRUGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Q：May I know how to change the CipheringAlg from default NEA2 to NEA0? (Because I want to decode nas message)

Dear gentleman：
Thanks for your great project first.
I found the nas message is encrypted after authentication response. May I know how to decode NAS meassage? Thanks very much.
Or maybe this is another question:  How to change the CipheringAlg from default NEA2 to NEA0?

Hi,
We're not implement other two encryption type for now, It is welcome to contribute with different types of NEA and NIA 😃

In config/amfcfg.conf change cipheringOrder to prefer NEA0:
- NEA0
- NEA2

The UE still needs to support NEA0, but I guess it does. You can check in the Registration Request:
UE security capability
    Element ID: 0x2e
    Length: 8
    1... .... = 5G-EA0: Supported
...

In config/amfcfg.conf change cipheringOrder to prefer NEA0:
cipheringOrder:
- NEA0
- NEA2

The UE still needs to support NEA0, but I guess it does. You can check in the Registration Request:
UE security capability
    Element ID: 0x2e
    Length: 8
    1... .... = 5G-EA0: Supported
...

Thanks mrlnc. But it seems doesn't work. Anyway , Thanks for your advice.

Hi,
We're not implement other two encryption type for now, It is welcome to contribute with different types of NEA and NIA 😃

Sorry, maybe I confused you. I just want to decode the NAS message(e.g Registartion accept) in anyway. But I'm a begeinner of Go, don't know which go file I should modify.
Do you have any method? I'm really appreciated.

Hi,
In our opinion, 3GPP required that all NAS message should be protected after completing security mode control procedure, so we reject any plain nas message after that procedure. At this moment, NEA0/NIA0 can not be enabled even if you change AMF config.
You may check amf/src/nas/nas_security/security.go and modify the code to enable plain NAS.

For debugging in the lab it certainly is very handy. Last time I checked, Wireshark didn't support NAS decryption even if you have the NAS session keys.
I think the standard requires NEA0 support but encourages you to activate it, whenever possible:

5.5 Requirements on the AMF
5.5.1 Signalling data confidentiality
The AMF shall support ciphering of NAS-signalling.
The AMF shall support the following ciphering algorithms:

NEA0, 128-NEA1, 128-NEA2 as defined in  Annex D of the present document.
The AMF may support the following ciphering algorithm:
128-NEA3 as defined in Annex D of the present document.
Confidentiality protection NAS-signalling is optional to use.
Confidentiality protection should be used whenever regulations permit.

3GPP TS 33.501 version 15.7.0 Release 15
So outside of productive use, it's fine.

@gprshome You have to enable the option in wireshark to even show NEA0 encoded messages:
Edit > Preferences > Protocols > NAD5-GS > [x] Try to detect and decode 5G-EA0 Ciphered messages.
Any real UE i've come acreoss does not work with NIA0.. So integrity is needed but NEA0 it totally fine.

Hi all,
We have release our new version v3.0.3 and add NEA0 as feature. We also set NEA0 as default on test script now.

This issue has been mentioned on free5GC. There might be relevant details there:
http://forum.free5gc.org/t/packet-is-corrupted-with-nia0/1404/2


fix(swarm): redirect to swarm overview page when the node availability is drain

Fixes https://github.com/portainer/portainer/issues/3330
Perhaps it would be better to always redirect to the swarm page so the behaviour is consistent? Happy to make that change if needed.

/azp run

@rstreefland sorry for the late update. I think the current behavior is fine.

@pull-dog down

@rstreefland sorry for the late update. I think the current behavior is fine.

Thanks for the review.

Thanks @rstreefland 👍


Removed unused vm_center toolbar

Purpose or Intent
Removed unused vm_center toolbar
Links
Related issue: #6259
Related issue: #6554

@miq-bot add_label refactoring


Formatter (case split): extra new lines are created between cases after formatting

Extra new lines are created between cases after formatting, such as:

Instead of:

And the latter seems more elegant and tidy (Idris / Agda and other languages that support case-split usually obey this format). Is there a way to avoid adding newlines after formatting? Thanks for answering!

I see you have raised an issue with purty so going to close this here.


remove unused import errors

I should have checked for unused imports before creating pull request #2. This removes the two unused import "errors" lines due to commit aaffb02 cleanups.

thanks!


deprecation warnings when compiling the app

Compiling the app version 2.1.12 with steps from comment: https://github.com/btcontract/wallet/issues/63#issuecomment-924162388
result in below warnings:
/home/appuser/app/simplebitcoinwallet/wallet/app/src/main/java/com/btcontract/wallet/BaseActivity.scala:57: method fromHtml in object Html is deprecated: see corresponding Javadoc for more information.
    def html: Spanned = android.text.Html.fromHtml(source)
                                          ^
/home/appuser/app/simplebitcoinwallet/wallet/app/src/main/java/com/btcontract/wallet/SetupActivity.scala:98: method startActivityForResult in class ComponentActivity is deprecated: see corresponding Javadoc for more information.
  def useBackupFile(view: View): Unit = startActivityForResult(new Intent(Intent.ACTION_OPEN_DOCUMENT).setType("*/*"), FILE_REQUEST_CODE)
                                        ^
/home/appuser/app/simplebitcoinwallet/wallet/app/src/main/java/com/btcontract/wallet/WalletApp.scala:321: method vibrate in class Vibrator is deprecated: see corresponding Javadoc for more information.
  def vibrate: Unit = if (null != vibrator && vibrator.hasVibrator) vibrator.vibrate(Array(0L, 85, 200), -1)
                                                                             ^
/home/appuser/app/simplebitcoinwallet/wallet/app/src/main/java/com/btcontract/wallet/utils/BiometricAuth.scala:20: method canAuthenticate in class BiometricManager is deprecated: see corresponding Javadoc for more information.
  def checkAuth: Unit = biometricManager.canAuthenticate match {
                                         ^
/home/appuser/app/simplebitcoinwallet/wallet/app/src/main/java/com/btcontract/wallet/utils/BiometricAuth.scala:32: method setDeviceCredentialAllowed in class Builder is deprecated: see corresponding Javadoc for more information.
        .setDeviceCredentialAllowed(true)
         ^
/home/appuser/app/simplebitcoinwallet/wallet/app/src/main/java/com/btcontract/wallet/utils/LocalBackup.scala:39: method getExternalStoragePublicDirectory in object Environment is deprecated: see corresponding Javadoc for more information.
  def downloadsDir: File = getExternalStoragePublicDirectory(DIRECTORY_DOWNLOADS)
                           ^
there were 18 feature warnings; re-run with -feature for details
7 warnings found

These are mostly present because wallet targets API levels down to 21 (Android 5.1) which do not have support for  modern analogues so app would crash without these lines on those phones.
OTOH in Scala specifically there is no easy way to suppress deprecation warnings so I'd rather let these be until support for older API levels is dropped.

maybe implement solution like this? (use newer API on newer android and older api on old android versions)
https://stackoverflow.com/questions/37899856/html-fromhtml-is-deprecated-what-is-the-alternative#37899983
another solution is different APK for older API like bluewallet does? (There 2 APK file to download)
https://github.com/BlueWallet/BlueWallet/releases/tag/v6.2.7
Also show message for devices with old Android, that it's unsafe and they should upgrade the version (device firmware upgrade or replace ROM to LineageOS, etc...)

maybe implement solution like this? (use newer API on newer android and older api on old android versions)
https://stackoverflow.com/questions/37899856/html-fromhtml-is-deprecated-what-is-the-alternative#37899983

This one won't eliminate an issue as legacy line will still stay and cause warnings.

another solution is different APK for older API like bluewallet does? (There 2 APK file to download)
https://github.com/BlueWallet/BlueWallet/releases/tag/v6.2.7

This seems too involved to maintain and I still want to have a single app in stores, guess this approach makes it impossible.

Also show message for devices with old Android, that it's unsafe and they should upgrade the version (device firmware upgrade or replace ROM to LineageOS, etc...)

This one is reasonable even though I assume old wallet users have reasons to do what they do, I've created an issue at https://github.com/btcontract/wallet/issues/77.

Added warning in https://github.com/btcontract/wallet/commit/88a2c8bb08c5b796795f078e57e682194a9b63aa for API < 24.


[UX] Homepage for Centurion

Acceptance criteria

[ ] Display "getting started" container using expandable container component with action stripe from cloudscape library
[ ] "Getting started" container should remain expanded (default state)
[ ] Use key value pair component within the "getting started" container to display information
[ ] Use accurate spacing/padding information as provided in the cloudscape
[ ] Use accurate font + font styling

Overview

Current state: No homepage. User needs to open the hamburger menu to access the list of dashboards or even to create a dashboard.
Proposed solution: Refer to the image below -

Overview section to provide a high-level information about Centurion with the ability to hide.
Dashboard table for creating, viewing and deleting the dashboard.
Users can further edit their dashboard once they view it.

NOTES

use https://refresh.cloudscape.aws.dev/
external link https://cloudscape.design/
use the visual refresh version as shown in the parent ticket screenshot / figma
use the design tokens where ever necessary for spacing and colors
https://cloudscape.design/components/app-layout/?tabId=api --> understand how app-layout works

### Tasks
- [ ] https://github.com/awslabs/iot-application/issues/949
- [ ] https://github.com/awslabs/iot-application/issues/994
- [ ] https://github.com/awslabs/iot-application/issues/1056
- [ ] https://github.com/awslabs/iot-application/issues/1057

Note:
Please make sure you use correct font styling. Refer to LINK
Reference links:
Cloudscape app layout - LINK
Cloudscape expandable container - LINK
Cloudscape table - LINK
Padding preview for the landing page:

SVGs for the illustrations used in the design:

++ Don't hesitate to reach out for any questions.

Updated illustrations

![visual 1](https://github.com/awslabs/iot-application/assets/141063047/3e6bad6c-6ac9-4d4d-afbb-5be3a417ce4e)
![visual 2](https://github.com/awslabs/iot-application/assets/141063047/ac3acd3b-e8ea-48a3-9e24-7f137fb64255)
![visual 3](https://github.com/awslabs/iot-application/assets/141063047/d3cdec3a-f4f3-48e3-9be3-fa99c7ad105a)


Make statistics fast again

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)

[ ] Did you read the contribution guide?
[ ] Did you update the docs? We use Numpy format for all the methods and classes.
[ ] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

Closing issues

Codecov Report

Merging #625 (552d30e) into master (85becfe) will decrease coverage by 0.31%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #625      +/-   ##
==========================================
- Coverage   84.60%   84.28%   -0.32%
==========================================
  Files         118      118
  Lines        6033     6244     +211
==========================================
+ Hits         5104     5263     +159
- Misses        929      981      +52

Impacted Files
Coverage Δ

etna/transforms/math/statistics.py
99.28% <100.00%> (+0.74%)
:arrow_up:

etna/analysis/plotters.py
20.95% <0.00%> (-1.20%)
:arrow_down:

etna/models/base.py
87.73% <0.00%> (-0.27%)
:arrow_down:

etna/models/__init__.py
100.00% <0.00%> (ø)

etna/models/catboost.py
100.00% <0.00%> (ø)

etna/analysis/__init__.py
100.00% <0.00%> (ø)

etna/models/holt_winters.py
98.82% <0.00%> (+0.04%)
:arrow_up:

etna/models/prophet.py
98.80% <0.00%> (+0.04%)
:arrow_up:

etna/models/sarimax.py
94.01% <0.00%> (+0.15%)
:arrow_up:

etna/models/seasonal_ma.py
97.36% <0.00%> (+0.22%)
:arrow_up:

... and 2 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[WIP] Fix #186 - New docs for request / response

Adding some new docs for $Request and $Response. Doing as a WIP for some rapid feedback on what would be most valuable to see. Starting off with a description of properties and methods and their descriptions.
Pull Request Creation Checklist

[x] Useful title (i.e. Fix #XXX: Description / Feature #XXX: Description / RFC #XXX: Description)
[x] Description of the changes you are making
[x] Added a link to the related Github issue in the description (i.e. type Fixes #XXX in the desciption of the PR)
[x] If you are still working on the code and would like some early feedback via the Pull Request process just add [WIP] to the beginning of your Pull Request title
[x] Checked the Polaris GitHub workflow guidance to make sure my code has the latest changes made to the master branch
[x] Throw small party 🎉

Fixes #186

That seems actually fine @Tiberriver256
To me, what was imoprtant is that  $Response / $Request are document somewhere, and that it is available in the read the docs.
it will defenitly be usefull, as my self found the info by chance looking at a blog post from Chen.


Implement descendent and sibling selectors

Background
According RFC i hope we can implement this feature to verify some things.
I don't want using string literal to define those selector. Declarative API is easier to maintain

Now i have time to implement this proposal. This may  be inconsistent with the proposal.

I think the opportunity is to create reference implementations to help make it easier to try specs.


Add storage cache for child trie and notification internals

This PR add child trie (CT) key value storage to the cache (see changes on into_committed method).

Notification part

From this point this PR also plug child trie values in the notification mechanism:

CT storage changes use the same subscription as top trie
wildcard on top does not mean wildcard on CT
RPC remain unchanged at this point (non breaking PR, another one will be needed to change storage subscription)

@tomusdrw @jacogr and all, those choice may not be really adequate (if we want a separate api for subscribing to ct value, part of this pr is wrong), should we consider the two first points ok?

storage cache part

quite straight forward but keeps pointing to the fact that CT api may be merge into standard api : lot of redundancy as in all CT code.
One bad point of the pr is the use of (Option<Vec<u8>>, Vec<u8>) as a key of lru cache (cannot use two cache as usual since a single lru seems better).
This leads to very awkward vec instantation on query (no Borrow implementation for tuple).
I think, but this is dependant on wether we want to unify ct api into parent api, that a StoragePath struct with encoded offset could do the job (depending on property of ord offset encoding position would change).
StoragePath would replace the couple child storage key and child key by an expanded path with position of the child switch in the path. This seems like an adequate representation (ct are quite similar to a standard trie branch but with a hop).

I've been trying a bit to use a single key for child (see HStorageKey of latest commit), but it seems to be counterproductive: this is quite orthogonal to #2209, and trying to generalize its usage is pushing thing way to far.
I think I will drop this approach and create a pr as @tomusdrw suggested.
Maybe if #2209 change to a full key path approach HStorageKey could be resurected (or for higher level operation where cycle need to be forbid: but the Ord property might not be interesting and a standard encoding may be more straight forward).

In latest commit:

fix the lru caches to use memory estimation instead of number of key value.
change the lru caches to relate to configuration memory limit
change rpc calls to use the hashes cache

There is still a problem with those lru cache: that is 4 lru list sharing a size limit, so the removal strategy being local (last use of the lru list where we insert content) a
lru can deplete another one and it is basically first using takes memory.
I am thinking on using 4 counter with a ratio of given memory (child ratio and hash ratio need to be defined and combined (hash first applied, child on result)).
This change would also allow a cleaner lru struct than the current tools method.
Any opinion on default value for them: I would go hash 10% and child 2% (assuming there is no child use and if there is you use it you really want to change that value to 90% or such) ?

@cheme who can review this effectively? @arkpar ?

The PR just copy what is done on general storage for child storage, but there is a bit of a gray area around the actual use case of the shared cache (difference with local cache).
So it would be best if the original code writer can take a peek, cc @arkpar (I would also welcome better design idea than the ratio I put in place).

Hash queries are only used for CODE and maybe for a couple of other keys at the moment. So the cache for hashes does not grow really. I'd put a fixed limit for hashes for now. 64 kilobytes maybe. As for the main/child ratio let's make it 50/50 but add an additional configuration option that enables/disables child tree cache.

Also, I would not bother taking any overhead into size calculations. Such as internal overhead for linked hash map items or vector capacity. These highly depend on the internals of external libraries  and the allocator, that might change with a new version. We'd just say that that the configured size is the data size and not the consumed memory size, which might be 20% or so higher on average.

So FWIU, hashes lru use is pretty limited, so I switch to proposed fix length value of 64ko and remove the hashes lru ratio configuration.
Similarily lru_child_hashes probably does not make much sense, so I remove it too.
Also removed the fix per element overhead, I wanted it to account for the use case of spammed small key value, but this usecase is partially limited by the key length that need to grow to grow in value.
If I want to put it back, it probably make more sense to apply it directly when calling accessing the lru size and simply add number of element * fix cost.


Quick mode

In most cases I have not found the statistical regression very useful. The result from a single sample with sufficient number of iterations is almost always good enough, I am yet to see an example where this is really useful. This fancy stuff takes more CPU and generates its own load impacting the benchmarks and taking more time. And in fact it is also doing wrong computations which has gone unnoticed for a long time perhaps because of the complexity. We can use this mode when we really  want that kind of rigor or have reason to believe that it will be useful.
For day to day or minute to minute runs we can use a fast mode which keeps it simple and stupid, would do a bit of warmup, measure one sample quickly and give the results. We can use a --quick option to enable this. This will save time and we can rely on the simplicity.

Implemented via PR #13


Total Completed Entries for Collection Log is Missing

#174 implemented the total completed entries for the collection log but they do not appear in the rich embed messages.

Same explanation as https://github.com/pajlads/DinkPlugin/issues/190#issuecomment-1475084013 - #174 is part of https://github.com/pajlads/DinkPlugin/releases/tag/v1.3.2, which has not been approved into the plugin hub by RuneLite yet: https://togithub.com/runelite/plugin-hub/pull/4124

I see! Thanks for the rapid response!


Fix typo in comment for ui-variables.less

New pull request resulting from #119.

New pull request for you @simurai. Thanks!

Would you mind rebasing this so the only commit is the one for fixing the typo? You can do this with git rebase -i. After you do that, you can update this PR with git push -f.

@lee-dohm Do you mean to remove the 2 (Change tree-view... + Revert "Change tree-view...) commits so the history is cleaner?
Not sure if rebase can do that. Doesn't it need to be a new branch from master and then apply the change manually again or cherry-pick Fix typo in comment from this branch?

Do you mean to remove the 2 (Change tree-view... + Revert "Change tree-view...) commits so the history is cleaner?

Yes, that's what I meant. And not so much from a "cleanliness" perspective but a "confusion" perspective :grinning: I was a little confused when "fixing a typo" had commits talking about other things.
git rebase -i (for "interactive") can do this kind of thing. You just have to force push the results. (See the Git documentation. If you just delete the lines for the two commits from the list and select "pick" for the one you want to keep, they will be removed from history.)

@tsangkenneth or @simurai we can merge as is too :grinning:

:+1: for less "confusion".
@tsangkenneth If you wanna give git rebase -i give a try. Or if that doesn't work out, just make a new PR.

Cool, did git rebase -i and now it's just the one clean commit. Thanks @lee-dohm!

Perfect! :+1:  Thanks.

You're welcome @tsangkenneth :grinning: Thanks for helping out!


Add fsoc solution push --wait

Description
oNew feature - n solution push, wait for solution to be installed. If solution is not successfully installed in the given time (300s default), command fails.
# No wait, same as before
fsoc solution push

# wait for 300s
fsoc solution push -w

# wait for 60s
fsoc solution push -w=60

# wait forever
fsoc solution push -w=0

Type of Change

[x] New Feature

Codecov Report

Merging #36 (55f26b8) into main (5eb1174) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   30.26%   30.26%
=======================================
  Files          23       23
  Lines        3000     3000
=======================================
  Hits          908      908
  Misses       2023     2023
  Partials       69       69

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


call process of Session()?

Why Session() by default calls public convenience init(configuration: URLSessionConfiguration = URLSessionConfiguration.af.default,
delegate: SessionDelegate = SessionDelegate(),
rootQueue: DispatchQueue = DispatchQueue(label: "org.alamofire.session.rootQueue"),
startRequestsImmediately: Bool = true,
requestQueue: DispatchQueue? = nil,
serializationQueue: DispatchQueue? = nil,
interceptor: RequestInterceptor? = nil,
serverTrustManager: ServerTrustManager? = nil,
redirectHandler: RedirectHandler? = nil,
cachedResponseHandler: CachedResponseHandler? = nil,
eventMonitors: [EventMonitor] = [])
Can you give me some clues？
Thank you！

This convenience initializer is the best way to create a new Session instance, as it handles the setup of the underlying URLSession automatically and ensures that the proper DispatchQueues are used.


Clean up Blazor.Build package

[x] Review the contents of the package and remove and refactor content
[x] Replace exes launched by the process with MSBuild tasks
[x] Replace using custom build of ILLink with the one in the SDK
[ ] Any further work based on the SDK team's feedback

Moving this to Preview 5 as there is some follow-up work left here with the SDK team.

We can use https://github.com/aspnet/AspNetCore/issues/17644 to track figuring out what to do with the task that's in our build. There's an offline discussion about the PR:  https://github.com/aspnet/AspNetCore/pull/17874 that has an overlap with that issue.


Add mock adapter implementation

Of course, this mock-adapter doesn’t really adapt anything so isn’t really an adapter. Is that a problem? Does this need to be renamed to mock-client instead?

I think we originally planned it to be mock-client, not sure why it ended up as an adapter.

You can rename the package and the repository, I will remove the old package from packagist and readd it as soon as this gets merged.


(GH-1327) Reword add_facts return documentation

This rewords the add_facts documentation for better readability
Closes #1327

CLA signed by all contributors.

LGTM, but can't review or merge.


[PDI-14913]- Corrected and fixed logic of exposing Kettle data over a…

… web service using "passing output to servlet" checkbox. Affected steps : Json Output, Text File Output and XML Output.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-json-plugin/build.xml clean-all resolve jacoco && ant -f plugins/kettle-json-plugin/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-xml-plugin/build.xml clean-all resolve jacoco && ant -f plugins/kettle-xml-plugin/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/step/StepMetaInterface.java
  engine/src/org/pentaho/di/trans/steps/textfileoutput/TextFileOutputMeta.java
  engine/src/org/pentaho/di/www/RunTransServlet.java
  engine/test-src/org/pentaho/di/www/RunTransServletTest.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsonoutput/JsonOutputMeta.java
  plugins/kettle-xml-plugin/src/org/pentaho/di/trans/steps/xmloutput/XMLOutputMeta.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change:  + .39%
Complexity Change:  + .51%
Instruction Change:  + .08%

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change:  + 1.01%
Complexity Change:  + 1.06%
Instruction Change:  + .17%

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -.66%:small_red_triangle_down:
Complexity Change: -.93%:small_red_triangle_down:
Instruction Change: -.13%:small_red_triangle_down:

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -2.08%:small_red_triangle_down:
Complexity Change: -2.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.53%:small_red_triangle_down:
Complexity Change: -4.05%:small_red_triangle_down:
Instruction Change: -.35%:small_red_triangle_down:
Line Change: -.97%:small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -.14%:small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change:  + 3.33%
Complexity Change:  + 1.85%
Instruction Change:  + .28%

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change:  + 2.04%
Complexity Change:  + 2.04%
Instruction Change:  + .32%
Line Change:  + .81%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

org.pentaho.di.trans.steps.textfileoutput.TextFileOutputMeta

Complexity Change: -.36%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:
Line Change: -.13%:small_red_triangle_down:
Method Change: -.92%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.www.RunTransServlet

Branch Change:  + 9.09%
Complexity Change:  + 23.81%
Instruction Change:  + 8.92%
Line Change:  + 10.23%
Method Change:  + 40.00%

org.pentaho.di.www.TransformationMap

Branch Change:  + 1.14%
Complexity Change:  + 1.54%
Instruction Change:  + .16%
Line Change:  + .77%

org.pentaho.di.trans.steps.jsonoutput.JsonOutputMeta

Complexity Change: -.30%:small_red_triangle_down:
Instruction Change: -.02%:small_red_triangle_down:
Line Change: -.06%:small_red_triangle_down:
Method Change: -.91%:small_red_triangle_down:

org.pentaho.di.trans.steps.xmloutput.XMLOutputMeta

Complexity Change: -.74%:small_red_triangle_down:
Instruction Change: -.15%:small_red_triangle_down:
Line Change: -.27%:small_red_triangle_down:
Method Change: -1.72%:small_red_triangle_down:


Issues with cpu percentage in host stats collections blocks host registration

Two issues were seen:
First
Traceback (most recent call last):
  File "/var/lib/cattle/pyagent/cattle/agent/event.py", line 91, in _worker_main
    resp = agent.execute(req)
  File "/var/lib/cattle/pyagent/cattle/agent/__init__.py", line 15, in execute
    return self._router.route(req)
  File "/var/lib/cattle/pyagent/cattle/plugins/core/event_router.py", line 13, in route
    resp = handler.execute(req)
  File "/var/lib/cattle/pyagent/cattle/plugins/core/event_handlers.py", line 32, in execute
    type.on_ping(event, resp)
  File "/var/lib/cattle/pyagent/cattle/plugins/docker/compute.py", line 54, in on_ping
    self._add_resources(ping, pong)
  File "/var/lib/cattle/pyagent/cattle/plugins/docker/compute.py", line 92, in _add_resources
    'physicalHostStats': self.host_info.collect_data()
  File "/var/lib/cattle/pyagent/cattle/plugins/host_info/main.py", line 17, in collect_data
    data[collector.key_name()] = collector.get_data()
  File "/var/lib/cattle/pyagent/cattle/plugins/host_info/cpu.py", line 81, in get_data
    data.update(self._get_cpu_percentages())
  File "/var/lib/cattle/pyagent/cattle/plugins/host_info/cpu.py", line 52, in _get_cpu_percentages
    stat_prev['timestamp'])
  File "/var/lib/cattle/pyagent/cattle/utils.py", line 96, in timestamp_diff
    time_current_conv = self._timestamp_convert(time_current)
  File "/var/lib/cattle/pyagent/cattle/utils.py", line 105, in _timestamp_convert
    return datetime.strptime(t_strip, "%Y-%m-%dT%H:%M:%S.%f")
  File "/usr/lib/python2.7/_strptime.py", line 328, in _strptime
    data_string[found.end():])
ValueError: unconverted data remains: 87-0

Second
Traceback (most recent call last):
  File "/var/lib/cattle/pyagent/cattle/agent/event.py", line 91, in _worker_main
    resp = agent.execute(req)
  File "/var/lib/cattle/pyagent/cattle/agent/__init__.py", line 15, in execute
    return self._router.route(req)
  File "/var/lib/cattle/pyagent/cattle/plugins/core/event_router.py", line 13, in route
    resp = handler.execute(req)
  File "/var/lib/cattle/pyagent/cattle/plugins/core/event_handlers.py", line 32, in execute
    type.on_ping(event, resp)
  File "/var/lib/cattle/pyagent/cattle/plugins/docker/compute.py", line 54, in on_ping
    self._add_resources(ping, pong)
  File "/var/lib/cattle/pyagent/cattle/plugins/docker/compute.py", line 92, in _add_resources
    'physicalHostStats': self.host_info.collect_data()
  File "/var/lib/cattle/pyagent/cattle/plugins/host_info/main.py", line 17, in collect_data
    data[collector.key_name()] = collector.get_data()
  File "/var/lib/cattle/pyagent/cattle/plugins/host_info/cpu.py", line 81, in get_data
    data.update(self._get_cpu_percentages())
  File "/var/lib/cattle/pyagent/cattle/plugins/host_info/cpu.py", line 47, in _get_cpu_percentages
    stat_prev = stats[-2]

It seems the date format is not what is expected.  The dates coming back look like 2015-02-04T23:21:38.251266323-07:00
Because of these issues the agent never successfully registers.

What OS?

Ubuntu 14.10, docker 1.5.0-rc3 if that matters

This was addressed in pull #75


Scrollview's viewport broken by rootwidget being an EffectWidget

This is the scrollable inventory with option buttons above, all children of a nested tab layout for inventory. Project source code at: https://github.com/misterdidactic/PnPPipBoy/
This is with no effects even applied to the EffectWidget (the root widget):

Same thing happening with effects applied. The entire viewport just isn't hiding the children of the scrollview. The touch area is still working, i.e. I can't press a button that has been drawn outside of the scrollview. Any presses are going through to the widgets drawn underneath.

+1 Can confirm that this occurs if the EffectWidget is any ancestor of the ScrollView. In my case the scrollview appears as all black. Any hints about what could cause this, and how one might fix this in kivy or circumvent this?

I narrowed this down a bit further. It is related to the StencilView. In my case, any widgets that have an EffectWidget as and ancestor and that are placed between StencilPush/StencilUse and StencilPop instructions, are not being drawn. However, I can touch them at the positions where they would otherwise appear.

EffectWidget uses Fbos internally, which can easily lead to stencil problems. I think stencils should work with Fbos now, so maybe they just need the right argument to be passed.

fbo has a with_stencilbuffer parameter. But that has its own problems: https://github.com/kivy-garden/garden.graph/issues/7. These fbo issues are quite pervasive and hasn't been solved yet.

The behavior is a bit odd. Adding a with_stencilbuffer=True to the fbo used by the EffectWidget causes the effect widget to be blank. Adding with_depthbuffer=True causes the ScrollView to render in a manner consistent with the original poster. That is, the contents of the scrollview are rendered with the desired effect, but the clipping behavior does not work. This makes sense since the effect widget fbo does not have a stencil buffer.
My suspicion is that if I can get the EffectWidget to render properly with a stencilbuffer, and then enable both a stencilbuffer and a depthbuffer enabled, that it would render properly. Unfortunately, I'm not sure how to go about investigating this further and would really appreciate any hints or guidance.

I found a solution to this issue by modifying the ScrollView instead. I changed it to inherit directly from Widget, and use a scissor instruction instead of a stencil to get it to clip. This seems like a good idea. It's easy to calculate the scissor rect using widget.to_window. Unfortunately, I can't figure out how to trigger a recalculation of the rect when the position of widget changes due to the position of a parent changing. For instance if a ScatterView is an ancestor of the widget and it is translated. How do I bind something to when the screen position of a widget changes?


Document NMP calculations

Describe the task
As a Student Developer I want document the calculations from NMP So That I can begin designing it's implementation
Acceptance Criteria

e.g. Given (Context), When (Action carried out), Then, (expected outcomes)

[ ] Calculations necessary for the berries workflow is documented in the projects Wiki

Additional context

Add any other context about the task here.

Is there a spreadsheet for the calculation values?


Cannot install openfst-1.4.1

Hello,
after cloning eesen, I went to tools directory and typed make. However, it was encountering an error when installing openfst-1.4.1
The error was shown as below:
./../include/fst/replace.h: In constructor 'fst::ArcIterator<fst::ReplaceFst<A, T, C> >::ArcIterator(const fst::ReplaceFst<A, T, C>&, fst::ArcIterator<fst::ReplaceFst<A, T, C> >::StateId)':
./../include/fst/replace.h:1253:57: error: expected ';' before '::' token
->template CacheBaseImpl<typename C::State, C>::InitArcIterator(
^~
Makefile:328: recipe for target 'fst.lo' failed
I've attached a full output from terminal here
out_make.txt

This is a known issue with older versions of openfst and recent version of
g++.  I encountered this last week with the same version of OpenFst,
although not in the context of eesen.  Following this patch fixed the issue
for me (ubuntu 18 + g++ 7.4):
https://github.com/hfst/hfst/commit/134d5a5aac3fd015c4955b6f5fdfede42159e402
2019年9月15日(日) 20:02 JeromeNi<EMAIL_ADDRESS>
Hello,
after cloning eesen, I went to tools directory and typed make. However, it
was encountering an error when installing openfst-1.4.1
The error was shown as below:
./../include/fst/replace.h: In constructor
'fst::ArcIterator<fst::ReplaceFst<A, T, C> >::ArcIterator(const
fst::ReplaceFst<A, T, C>&, fst::ArcIterator<fst::ReplaceFst<A, T, C>

::StateId)':
./../include/fst/replace.h:1253:57: error: expected ';' before '::' token
->template CacheBaseImpl<typename C::State, C>::InitArcIterator(
^~
Makefile:328: recipe for target 'fst.lo' failed

I've attached a full output from terminal here
out_make.txt https://github.com/srvk/eesen/files/3614099/out_make.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/srvk/eesen/issues/215?email_source=notifications&email_token=AABVUA5SODPJDC5DMIZUEYTQJZ2ETA5CNFSM4IW3OQD2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HLOJQIQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABVUA43XEOW7VAFPIYQDIDQJZ2ETANCNFSM4IW3OQDQ
.

Thank you, the patch worked for me!


chore(main): release 1.1.2

:robot: I have created a release beep boop
1.1.2 (2023-12-19)
Bug Fixes

Dont stringify (d75a2ee)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cedricziel/node-red-contrib-mattermost-app/releases/tag/v1.1.2 :sunflower:


fix: navigation path naming for apisix-java-plugin-runner

Fixes: #[Add issue number here]
Changes:
fixed the navigation path name for apisix-java-plugin-runner

cc @juzhiyuan @Yiyiyimu
till then that uniform id proposal did not pass this will work

LGTM, thanks!

It seems I can not see apisix-java-runner in navbar, is that expected?

It seems I can not see apisix-java-runner in navbar, is that expected?

No, there should be apisix-java-runner

It seems I can not see apisix-java-runner in navbar, is that expected?

No, there should be apisix-java-runner, I think you have unmerged changes

@iamayushdas you can check the preview page

@iamayushdas you can check the preview page

oops, we got a problem here

@Yiyiyimu have a look again please


Overall raster overhaul

Summary by Sourcery
Enhance the raster plotting capabilities with new optimization methods and support for special instructions. Improve the element tree operations by adding conversion for EGV binary objects. Refactor the raster plotting code to include debugging features and consider laser spot size in high DPI settings. Update the GUI to reflect these changes with new options and tooltips.
New Features:

Introduce new optimization methods for raster plotting, including greedy path optimization and crossover techniques.
Add support for special treatment instructions in raster plotting algorithms.
Implement conversion of EGV binary objects to elements in the element tree operations.

Enhancements:

Refactor raster plotting to include debug levels and test data generation for improved diagnostics.
Enhance raster plotting with the ability to consider laser spot size for high DPI settings.
Improve the handling of raster image creation and conversion to paths with new submenu options.

Documentation:

Update tooltips and documentation for raster engraving settings to reflect new optimization options and considerations for laser dot size.

@sourcery-ai review


[SUPPORT] Question on Bootstrapped hudi table

Tips before filing an issue

Have you gone through our FAQs?

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
We have an existing parquet table and used DeltaStreamer to bootstrap the table with METADATA_ONLY and COPY_ON_WRITE option. I was able to successfully bootstrap it. I then performed few inserts and upserts to the hudi table. I am able to see the updates when I query the hudi table.
The issue I'm seeing is that the original/source parquet file that I bootstrapped looks unchanged. What I mean here is that, for example, initially the original/source parquet file has 20 rows. After bootstrapping and converting it to a hudi table and performing few inserts and upserts added 5 more rows. Total rows now that I can query from hudi table snapshot is 25 rows. So far looks good.
But if I look at the original/source parquet file, it contains the same 20 rows it had before. I was expecting that it would replace the existing source parquet with all the updates that took place and generate a new parquet file reflecting all updates.
If we have any downstream consumers of the parquet file(who are not using hudi clients for querying) at source location, then what is the right way to propagate these hudi updates back to parquet file for downstream consumers so that they can see most up to date data.
To Reproduce
Steps to reproduce the behavior:

Use DeltaStreamer to bootstrap an existing parquet table with METADATA_ONLY and COPY_ON_WRITE options
Do few inserts to add few rows to the hudi table
We will notice that the original/source parquet file is not updated/replaced.

Expected behavior
Hudi updates/replaces the original/source parquet file with all updates.
Environment Description

Hudi version : 0.12.0

Spark version : 3.3

Hive version :

Hadoop version :

Storage (HDFS/S3/GCS..) : Local Macbook Pro

Running on Docker? (yes/no) : No

Additional context
What is the right way to periodically export hudi tables as parquet format files for other external consumers who are not using hudi clients for querying ?
Can we do incremental export of hudi tables to parquet format files ?
Stacktrace
Add the stacktrace of the error.

looks like the behavior you see is expected.
@yihua : can you take this up please.

@yihua Could you please provide any update ?

@rtdt99 The behavior is expected.  The bootstrap operation is going to reference the parquet files from the original parquet table path.  Any new data or updates are written to the base path of the new Hudi table, and the original parquet table is not touched, because now any new data is written using Hudi's data format.  The consumer has to query the table using the new Hudi table base path to get the latest snapshot.

@yihua Thank you! Also, is it possible to run exporter job for a specific date/time range instead of exporting the whole hudi table snapshot to parquet format ?

@rtdt99 Currently HoodieSnapshotExporter doesn't provide that functionality.


android studio 3.0.1 not show anko layout preview

kotlin_version:1.2.10
anko_version: 0.10.4

gradle version: 3.0.1

Running on the phone can be displayed, but the preview can not be displayed, this is a bug！

@evansherry would you please provide a link to your working project repo. I am stack for 3 days with the same problem, and tried almost everything

You need build your project everytime when you need preview


ITI-44_request_raw.xml is not raw

In samples, ITI-44_request_raw.xml has only the PRPA_IN201301UV02 element, no SOAP envelope.

Fixed


[BUG] English translation needed for a tooltip (see image)

OS INFO :

Unfortunately, the table component is not internationalized for this, I think this does not affect the use.

Unfortunately, the table component is not internationalized for this, I think this does not affect the use.


Fixing arrow_annotation_spec tests

@tejal-shah This test is also fixed in the synchronous highlighting PR: https://github.com/concord-consortium/collaborative-learning/pull/2019

ok, I'll remove this fix. Thanks for the headsup @tealefristoe


Improve YAKS report naming

Given YAKS test
Test integration finished with status: Passed
AutoRemove namespace yaks-99e23130-85a7-4a73-a3c3-67094e7697c1
Test results: Total: 2, Passed: 2, Failed: 0, Errors: 0, Skipped: 0
	 (integration.feature:3): Passed
	 (integration.feature:7): Passed

generates the following report
➜  camel-k-example-basic git:(master) ✗ yaks report -o junit
<?xml version="1.0" encoding="UTF-8"?><testsuites>
  <testsuite name="org.citrusframework.yaks.JUnitReport" errors="0" failures="0" skipped="0" tests="2" time="0">
    <testcase name="Integration Basic is running" classname="integration.feature:3" time="0"></testcase>
    <testcase name="Integration routing prints items" classname="integration.feature:7" time="0"></testcase>
  </testsuite>
</testsuites>

Systems that work with test results (ReportPortal in my case) sorts results according to testsuite name (on a given level). All yaks testsuite name is equal to org.citrusframework.yaks.JUnitReport. This behavior results in very confusing reports.
I would suggest replace testsuite name with the feature name e.g:
cat test/integration.feature | grep Feature
Feature: all integrations print the correct messages

@christophd is that possible?

good point! I think it should be possible to have a better testsuite name. we would have multiple testsuites in the report then (one for each feature)


[demo] helm install error: function "mustFromJson" not defined

Documentation: https://opentelemetry.io/docs/demo/kubernetes-deployment/
Step: To install the chart with the release name my-otel-demo, run the following command:

helm install my-otel-demo open-telemetry/opentelemetry-demo

Try with

helm version
version.BuildInfo{Version:"v3.4.0-rc.1"

helm install my-otel-demo open-telemetry/opentelemetry-demo
Error: parse error at (opentelemetry-demo/templates/_pod.tpl:11): function "mustFromJson" not defined

src:
https://github.com/open-telemetry/opentelemetry-helm-charts/blob/36d6b236a92745b82858fd2e703cd5669ad23cfa/charts/opentelemetry-demo/templates/_pod.tpl

@gerbil the demo prerequisites require using helm 3.9+.  Can you try with a 3.9+ version of helm?


How to include easyloggingpp in a library

So, I have a library that is being built independently of a separate project and both the library and the project use easyloggingpp. My question is, can I have more than one INITIALIZE_EASYLOGGING macro without consequence? And if not, is there a way to figure out if INITIALIZE_EASYLOGGING has already been used during compilation (a #ifndef or something like that)?

I have a same question.

I have a same question.  there is a sample use esaylog in dll in history version, but it remove later, why?

you can reference :
https://github.com/zc-yoke/easylogging

@sujx0123 thank you!


Test with Postgres 14

Test with Postgres 14
record if any other changes will be needed

related to #46


Added mining inflation chart to stats page

See https://github.com/marcdeb1/lbry-supply-chart

yey, excited to get this up on explorer :)


Cannot read property 'reason' of undefined

Issue Id: cc5915e6-9f36-552a-b980-dcec91d217beVersions - 1.10.2Stack TypeError: Cannot read property 'reason' of undefined/vs/workbench/parts/debug/electron-browser/debugEditorContribution.ts#L324:45 (reason)/vs/workbench/parts/debug/electron-browser/debugEditorContribution.ts#L185:8 (toggleExceptionWidget)/vs/base/common/event.ts#L217:12 (apply)/vs/base/common/eventEmitter.ts#L315:9 (func)/vs/base/common/eventEmitter.ts#L182:4 (safeInvoke1Arg)/vs/base/common/eventEmitter.ts#L201:8 (_emitToSpecificTypeListeners)/vs/base/common/eventEmitter.ts#L216:8 (_emitEvents)/vs/editor/common/commonCodeEditor.ts#L229:7 (emit)/vs/workbench/browser/parts/editor/textResourceEditor.ts#L103:14 (setModel)vs/workbench/electron-browser/vs/base/common/winjs.base.raw.js#L1170:0 (onComplete)

Dup of #22929


exercises(collatz-conjecture): clarify error

See:

https://github.com/exercism/zig/pull/360#issuecomment-1815159705
https://github.com/exercism/zig/pull/363#issuecomment-1873940161

I'll fix this soon.

Hi, just wanna say this would be a great update for the exercise. I myself just trying to learn zig from scratch but this exercise is unclear on the declaring error part. I have to spend more time to find out that i have to add my own error handling (from the tests) than to finish the exercise. Adding clearer instructions might really help on solidifying users skills on the language than spending time to find out a hidden requirement.


Roll Flutter from 39a38b7882fe to f3717d70d7d2 (4 revisions)

https://github.com/flutter/flutter/compare/39a38b7882fe...f3717d70d7d2
2022-07-06<EMAIL_ADDRESS>Enable Windows platform_channel test (flutter/flutter#106973)
2022-07-06<EMAIL_ADDRESS>Roll Flutter Engine from d7b6d9bbc0f2 to d87f730fae7b (2 revisions) (flutter/flutter#107144)
2022-07-06<EMAIL_ADDRESS>Roll Flutter Engine from ed6adf55e029 to d7b6d9bbc0f2 (2 revisions) (flutter/flutter#107120)
2022-07-05<EMAIL_ADDRESS>Roll Flutter Engine from 8197dec7e27b to ed6adf55e029 (1 revision) (flutter/flutter#107116)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Updated QAT instructions

This fixes #516
Signed-off-by: Eric Adams<EMAIL_ADDRESS>

I added another rev of this. I might have accidentally pushed this to master as well so just ignore that if I did.

@eadamsintel Please add a subsystem to your commit message, travis is failing due to that.
@klynnrif Can you take a look?

I updated the commit message with a subsystem.

/test


PINGDOM-CHECK - ID number formatting Problem

Hi, i'm playing around with Monitoror and wanted to display some Pingdom Checks.
I'm running Monitoror v3.1.0 on MacOS High Sierra 10.13.6 with Chrome 80.0.3987.132
Ping & HTTP Checks are working fine but when im trying the following example config:
{ "type": "PINGDOM-CHECK", "label": "Pingdom Test", "params": { "id": 1000001 } },
the Monitotor Webinterface requests the following URL: http://<IP_ADDRESS>:8080/api/v1/pingdom/default/check?id=1.000001e%2B06
So basically the ID being requested is formatted as 1.000001e+06 which would translate to 1000001.
The request of course fails with HTTP Error 500 {"status":500,"message":"invalid configuration, unable to parse/check queryParams"}.
If i manually request the id as intended, everything works and my Pingdom result is displayed.
So there seems to be a number formatting problem somewhere, would be cool if you could take a look :)

Thank you for the quick fix :)


Question on extension grants

Hi @leastprivilege / @brockallen
My application connects to different databases based on the choice of the user at login. I asked a similar question before but this one is about extension grants.
At the time of login user is asked for the division and I found the way to include that division id in the token.
Now if the user changes the division I want to replace the division id in token to the new division without asking the user their credentials again. Its just a change of one claim in the token.
Is this possible using extension grants? If not then what is the recommended solution?
Thanks in advance.
Noufal

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use StackOverflow for that. This has the advantage that questions and answers can be easily found by search engines, and that there are more people answering questions than just us.
For IdentityServer3
https://stackoverflow.com/questions/tagged/?tagnames=identityserver3&sort=newest
For IdentityServer4
https://stackoverflow.com/questions/tagged/?tagnames=identityserver4&sort=newest
For commercial support
https://identityserver.io/


Force and torque non newtonian

Description of the problem
The current forces and torque calculated as a postprocess tool use a constant viscosity. For non Newtonian flows, viscosity varies in space, therefore the viscosity should be calculated on each quadrature point when calculating the total force and torque.
Description of the solution
The methods calculate_forces() and calculate_torque() now calculated the value of viscosity for each quadrature point when the non_newtonian_flow flag is activated, and this viscosity is being used.
How Has This Been Tested?
A new test has been added to the Grad-Div solver where the force field is calculated using a Carreau rheolgical model.

[x] application_tests/gs_navier_stokes_2d/cylinder_carreau_gd

Very good implementation. I wonder if the same could be directly implemented in the sharp_interface solver in this PR or should we do another PR?

Another PR.
Small PRs are good PRs.


[BUG]You don't have permission to configure pipelines V2.6.3

Rancher Server Setup

Rancher version:2.6.3
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:k3s

Information about the Cluster

Kubernetes version:v1.21.7

User Information

What is the role of the user logged in? (Admin/Cluster Owner)

Describe the bug
You don't have permission to configure pipelines V2.6.3  when i login with Cluster Owner
Screenshots

Check the release notes:
https://github.com/rancher/rancher/releases/tag/v2.6.0
Legacy features are features hidden behind the legacy feature flag, which are various features/functionality of Rancher that was available in previous releases. These are features that Rancher doesn't intend for new users to consume, but if you have been using past versions of Rancher, you'll still want to use this functionality.

When you first start 2.6, there is a card in the Home page that outlines the location of where these features are now located.

The deprecated features from v2.5 are now behind the legacy feature flag. Please review our deprecation policy for questions.

The following legacy features are no longer supported on Kubernetes 1.21+ clusters:

Logging (Legacy)
CIS Scans (Legacy)
Istio 1.5
Pipelines


Re-introduce array container models

This reverts the original changes in PR: https://github.com/DataDog/datadog-api-client-python/pull/1431 and removes the models added in: https://github.com/DataDog/datadog-api-client-python/pull/1499
closes: https://github.com/DataDog/datadog-api-client-python/issues/1501

I think we're missing the NotifyEnd* models?

I omitted these purposefully along with FindingTags and ListFindingsData, since these models were introduced post 2.12.0 release.


Replace hardcoded<PHONE_NUMBER> with time.Second.

While here declare HttpRequestTimeout as a time.Duration so there is no
need for a type conversion.

Looking around the code some more I notice that NewTransportConfig() takes a "httpRequestTimeout int" argument but it is never used to fill in the TransportConfig struct. Is that a mistake?
Ideally I would like to also modify NewTransportConfig() to take a time.Duration instead of an int to minimize the number of type conversions, but this would require modification of code using the library which might be too invasive.
One such user (hinted at by #33) is https://github.com/infobloxopen/docker-infoblox/blob/master/common/config.go. It appears that the GridConfig struct there uses neither an int or time.Duration for its HttpRequestTimeout member but rather an uint. This is probably why a type conversion to int is performed at line 40 in https://github.com/infobloxopen/docker-infoblox/blob/master/ea-defs/create_ea_defs.go:
transportConfig := ibclient.NewTransportConfig(
		config.SslVerify,
		int(config.HttpRequestTimeout),
		int(config.HttpPoolConnections),
)

I guess all of this raises the question if an attempt should be made to normalize on the usage of time.Duration for such struct members to avoid the need for type conversions as well as a kind of implicit documentation that the struct member indeed holds a time duration, or if such a backwards incompatible change is not OK.
I guess otherwise the "easy" way out is to deal with type conversion from int to time.Duration in NewTransportConfig().

I notice that NewTransportConfig() takes a "httpRequestTimeout int" argument but it is never used to fill in the TransportConfig struct. Is that a mistake?

I didnt quite get this question. You mean in the docker-infoblox lib?
It makes sense to normalize it as you said, but it'll impact the dependent libraries.
I think good way to handle this is versioning/tag ing it before going further with this PR.
@jkraj  what say ?

@chinmayb Yes I agree with you to create a new version from existing code base. I am planning to add a Readme and example code before creating a new version/tag.

I notice that NewTransportConfig() takes a "httpRequestTimeout int" argument but it is never used to fill in the TransportConfig struct. Is that a mistake?

I didnt quite get this question. You mean in the docker-infoblox lib?

No, see NewTransportConfig() in https://github.com/infobloxopen/infoblox-go-client/blob/master/connector.go#L33:
func NewTransportConfig(sslVerify string, httpRequestTimeout int, httpPoolConnections int) (cfg TransportConfig) {
	switch {
	case "false" == strings.ToLower(sslVerify):
		cfg.SslVerify = false
	case "true" == strings.ToLower(sslVerify):
		cfg.SslVerify = true
	default:
		caPool := x509.NewCertPool()
		cert, err := ioutil.ReadFile(sslVerify)
		if err != nil {
			log.Printf("Cannot load certificate file '%s'", sslVerify)
			return
		}
		if !caPool.AppendCertsFromPEM(cert) {
			err = errors.New(fmt.Sprintf("Cannot append certificate from file '%s'", sslVerify))
			return
		}
		cfg.certPool = caPool
		cfg.SslVerify = true
	}

	return
}

It takes an "httpRequestTimeout int" but it is never used to fill in the returned "cfg TransportConfig".

@eest Understood. I missed that. I'll create a PR.

@chinmayb: cool. Maby you should decide if this PR should be merged or not before opening that one since it modifies the struct member type that is not currently filled in? If you just set it with the current type of int then this PR is no longer valid.

Currently i have set the httpRequestTimeout and the PR is merged.  Can we close this PR ?

@chinmayb: so you do not like the changes i have proposed then? Since you have modified NewTransportConfig() I would need to modify this PR somewhat though, since I changed the struct field from int to time.Duration which is no longer valid after you started setting it with your recent PR.

@eest As you mentioned in your earlier comments changing datatype of HttpRequestTimeout in Transportconfig struct will force to change who are all using this client. Can you please update this PR to address the hardcoded value alone then we can merge this.

@jkraj: Since there seemed to be interest in normalizing on time.Duration in the discussion above I added a commit that adds a type conversion for the HttpRequestTimeout field. This way the library can internally depend on the time.Duration type even if external users currently expect to feed it an int. Does that make sense?

@eest: Now things make sense