Document how NestedScrollView.physics/scrollDirection/reverse affect outer/inner scroll views

I read code ,find this code in scrollable.dart
void _updatePosition() {
   _configuration = ScrollConfiguration.of(context);
   _physics = _configuration.getScrollPhysics(context);
   if (widget.physics != null)
     _physics = widget.physics.applyTo(_physics);
   final ScrollController controller = widget.controller;
   final ScrollPosition oldPosition = position;
   if (oldPosition != null) {
     controller?.detach(oldPosition);
     // It's important that we not dispose the old position until after the
     // viewport has had a chance to unregister its listeners from the old
     // position. So, schedule a microtask to do it.
     scheduleMicrotask(oldPosition.dispose);
   }

   _position = controller?.createScrollPosition(_physics, this, oldPosition)
     ?? ScrollPositionWithSingleContext(physics: _physics, context: this, oldPosition: oldPosition);
   assert(position != null);
   controller?.attach(position);
 }

Hi @xinle,
Could you. please, provide flutter doctor -v
Thank you.

Hi @xinle,
Could you. please, provide flutter doctor -v
Thank you.

[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.12.6 16G29, locale zh-Hans-HK)
• Flutter version 1.9.1+hotfix.6 at /Users/xinle/flutter
• Framework revision 68587a0916 (3 个月前), 2019-09-13 19:46:58 -0700
• Engine revision b863200c37
• Dart version 2.5.0
[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/xinle/Library/Android/sdk
• Android NDK at /Users/xinle/Library/Android/sdk/ndk-bundle
• Platform android-29, build-tools 28.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[✓] Xcode - develop for iOS and macOS (Xcode 9.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 9.2, Build version 9C40b
• CocoaPods version 1.7.5
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 38.2.3
• Dart plugin version 191.8423
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.39.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.5.1
[!] Connected device
! No devices available
! Doctor found issues in 2 categories.

It doesn't work also for my project. I created a dartPad with the issue
https://dartpad.dartlang.org/84e6c7159a34e607cd5216c2fbf3d5b6

I forked the NestedScrollView and created a CustomNestedScrollView modified to work with NeverScrollableScrollPhysics.
https://gist.github.com/jamesblasco/5b609ecff66c3fd1b1f21cd36564d8db
The demo example : https://dartpad.dartlang.org/5b609ecff66c3fd1b1f21cd36564d8db

Reproducible on the latest master 1.22.0-10.0.pre.196.

flutter doctor -v
[√] Flutter (Channel master, 1.22.0-10.0.pre.196, on Microsoft Windows [Version 10.0.19041.508], locale en-US)
    • Flutter version 1.22.0-10.0.pre.196 at C:\Development\flutter_master
    • Framework revision ea039ed3f9 (9 hours ago), 2020-09-15 17:57:04 -0400
    • Engine revision 1cabedf8ac
    • Dart version 2.10.0 (build 2.10.0-124.0.dev)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Users\marku\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.6.5)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.6.30320.27
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 4.0)
    • Android Studio at C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] Connected device (6 available)
    • SM G950F (mobile)           • ce12171c51cc001c03 • android-arm64  • Android 9 (API 28)
    • sdk gphone x86 arm (mobile) • emulator-5554      • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)           • windows            • windows-x64    • Microsoft Windows [Version 10.0.19041.508]
    • Web Server (web)            • web-server         • web-javascript • Flutter Tools
    • Chrome (web)                • chrome             • web-javascript • Google Chrome 85.0.4183.102
    • Edge (web)                  • edge               • web-javascript • Microsoft Edge 85.0.564.44

• No issues found!

Code sample from DartPad
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: NestedScrollView(
          controller: PrimaryScrollController.of(context),
          physics: NeverScrollableScrollPhysics(),
          headerSliverBuilder: (context, _) => [
            SliverAppBar(
              title: Text('NestedScrollView issue'),
            ),
          ],
          body: SingleChildScrollView(
            physics: NeverScrollableScrollPhysics(),
            child: Column(
              mainAxisSize: MainAxisSize.min,
              children: <Widget>[
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                ),
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                ),
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                ),
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                ),
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                ),
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                ),
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                ),
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                ),
                Container(
                  margin: EdgeInsets.all(16),
                  height: 200,
                  width: 200,
                  color: Colors.blue,
                )
              ],
            ),
          ),
        ),
      ),
    );
  }
}

@jamesblasco Would you considering working on a PR for this?

I forked the NestedScrollView and created a CustomNestedScrollView modified to work with NeverScrollableScrollPhysics.
https://gist.github.com/jamesblasco/5b609ecff66c3fd1b1f21cd36564d8db
The demo example : https://dartpad.dartlang.org/5b609ecff66c3fd1b1f21cd36564d8db

Dear @jamesblasco , I have an open issue that might be related to this issue and your forked CustomNestedScrollView .
So can you have a look at this issue: #71289
Thanks a lot!

Sure, I will take a look. It has been a year since I don't work with this change so I might have forgotten a bit about it. But I still use it in my project with a PageView inside and works without problem for me

A quick workaround here. (Referred to #82391, which describes a root cause well.)

Fork NestedScrollView.
Replace updateCanDrag implementation
https://github.com/flutter/flutter/blob/87eddcb1a0df44af8f234108b56fba5a2cb5f86f/packages/flutter/lib/src/widgets/nested_scroll_view.dart#L1615-L1617
with

void updateCanDrag(double totalExtent) {
  if (physics.shouldAcceptUserOffset(this)) {
    context.setCanDrag(totalExtent > (viewportDimension - maxScrollExtent) || minScrollExtent != maxScrollExtent);
  } else {
    context.setCanDrag(false);
  }
}

Use this forked version of NestedScrollView.

Hi folks, I am not sure here what the expected vs. actual results here are. Currently if I set the inner (body) scroll view of a NestedScrollView physics to NeverScrollableScrollPhysics, the inner scroll view does not scroll - which I think is expected.
I checked against the linked Dartpad as well. If someone can update with the actual/expected behavior, we can take a look! It is also possible we may have already fixed it 🤞
Thanks!

Hey @lavinov-mercury thanks for the update! i see what you mean. I will look into this. :)

@Piinks I can also confirm, that fix proposed above worked as expected
@red282 would you mind to open a PR?

Hey @red282 thanks for sharing! Your workaround would not work in all cases though. I have checked it out and one such case it does not work for is mouse scrolling. Your proposed change to update setCanDrag does not affect mouse scrolling, only scrolling through dragging.
I also found that this may be working as intended. Here are my findings:
The NestedScrollView should absolutely pass down whatever ScrollPhysics are provided to NestedScrollView.physics (I have a patch coming shortly for this), but in the Dartpad example @lavinov-mercury provided, it still will not result in the desired behavior.
If you look at how ScrollView is implemented, the Dartpad example is running into this specific case:
https://github.com/flutter/flutter/blob/4f5cc21be239c3cdefcad89f2bbbe1c508c36026/packages/flutter/lib/src/widgets/scroll_view.dart#L112
In a vertical ScrollView (like the CustomScrollView in the Dartpad), if ScrollView.physics has not been set, it will apply AlwaysScrollableScrollPhysics. So setting CustomScrollView.physics to NeverScrollableScrollPhysics would be required in your use case. With the patch I have coming, Axis.horizontal would see the expected behavior. The NestedScrollView cannot do that because it does not know what the NestedScrollView.body is, let alone what parameters it may or may not have set.
Even with my patch to allow the inner scroll view to inherit NestedScrollView.physics properly, those AlwaysScrollableScrollPhysics in that vertical case will always be there taking precedence.
https://github.com/flutter/flutter/blob/4f5cc21be239c3cdefcad89f2bbbe1c508c36026/packages/flutter/lib/src/widgets/scrollable.dart#L408
I've noticed as well that this may be something folks expect, depending on their use case. I can see that physics is not the only property that would need to be set for both, the same goes for properties like scrollDirection and reverse. The NestedScrollView doesn't set these for the CustomScrollView either, so they can mismatch for different behaviors, or require both to be set.
I think we should definitely document all of this, so I will update this bug to reflect that.

Actually, my patch makes me think this is even more intentional. It breaks a lot of existing tests if I pass down the physics. I think the opening line in the docs should be emphasized more:

A scrolling view inside of which can be nested other scrolling views, with their scroll positions being intrinsically linked.

I think the option to have a different physics for the inner/outer scroll view is intentional, it is the linking of the position, having them move together in unison, that the NestedScrollView is primarily concerned with.


sync develop -> master

Ensuring master and develop branches are in alignment

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


DOC: sparse: explicit dtypes for nonzero()

$ python dev.py smoke-docs fails for me locally on dtype mismatch in sparse .nonzero() docstring. cc @dschult not sure if this is indeed a correct change?

This looks like a Numpy 2.2 issue where NumPy is now printing the shape on arrays that don't show all their values upon printing.  A way to hide those errors has been implemented for CI, but the doctest errors will still show up locally.  At least that is my understanding. Do you see these errors in a few other places across the library?

I have investigated further and I do not get any smoke-docs failures when running:
python dev.py smoke-docs -t scipy/sparse/_base.py.
When I run ipython and try the provided code, it matches what you put as the output. (It prints the dtype)
When I add the dtypes to the doctest, it passes smoke-docs. When I remove the dtypes, it also passes smoke-docs.
When I add a syntax error both methods catch the syntax error. So I'm quite sure the smoke-docs are munging the standard numpy array printing output to a form that matches any recent version of NumPy.
But I don't know how smoke-docs works here. I've seen it before where the doc-tests are passing even though the numpy array output doesn't match some versions of numpy.
I can say that scipy.sparse has not changed whether the numpy arrays created by nonzero print an explicit dtype.  I can also say that the method count_nonzero has a doctest that should fail if nonzeros doctest fails.
Can you explain more how this is appearing? Can you check the doctest for the method count_nonzero in _base.py?

$ python dev.py smoke-docs -t scipy/sparse/_base.py
... snip ...
================================= test session starts ==================================
platform linux -- Python 3.12.4, pytest-8.2.2, pluggy-1.5.0
rootdir: /home/br/repos/scipy/scipy
configfile: pytest.ini
plugins: cov-5.0.0, hypothesis-6.108.0, xdoctest-1.2.0, xdist-3.6.1, anyio-4.4.0, scipy_doctest-1.6.dev0, timeout-2.3.1
collected 6 items

scipy/sparse/_base.py ...F..                                                     [100%]

======================================= FAILURES =======================================
_____________________ [doctest] scipy.sparse._base._spbase.nonzero

Dropping a breakpoint into an editable install of scipy-doctests,
$ git diff
diff --git a/scipy_doctest/impl.py b/scipy_doctest/impl.py
index 8f87bed..1e4e5c5 100644
--- a/scipy_doctest/impl.py
+++ b/scipy_doctest/impl.py
@@ -320,6 +320,8 @@ class DTChecker(doctest.OutputChecker):
         if want.lstrip().startswith("#"):
             return True

+        breakpoint()
+
         # try the standard doctest
         try:
             if self.vanilla.check_output(want, got, optionflags):

I see what's happening is that indeed this line https://github.com/scipy/scipy/blob/main/scipy/sparse/_base.py#L899
is strictly speaking not quite right:
        >>> from scipy.sparse import csr_array
        >>> A = csr_array([[1,2,0],[0,0,3],[4,0,5]])
        >>> A.nonzero()
        (array([0, 0, 1, 2, 2]), array([0, 1, 2, 0, 2]))

array([0, 0, 1, 2, 2]) is by default (at least on my linux system), int64, but A.nonzero() creates arrays of int32.
Why does not not fail on your system?

one thing that did change somewhat recently, is https://github.com/scipy/scipy/blob/main/scipy/conftest.py#L551 : previously scipy-doctest relied on numpy definition of equality (and then int32_array == int64_array as long as the values match); from version 1.4 it has grown an option to fail on dtype mismatch---and this is what this strict_check flag turns on.

So, are you locally on Windows? Or do you have a previous versin of scipy-doctest , which is lax on dtype checking?
Then of course there's a question of what's going on CI.
Whether this all is worth spending time on, I honestly don't know. That it does not fail on CI we can consider as good enough and move on, of course :-).
So maybe the right thing to do indeed depends on what's the intent of A.nonzero() is: does it mean to make any guarantees on its output dtypes?
If not, then ain't broken don't fix.

Thank you Dan for the explanation! So, while the status quo is not exactly consistent, there are both reasons for inconsistencies, and reasons beyond just wanting consistency. This PR is not a place to decide on those, even if doctesting helped to highlight some small issues.So it's as much as can be expected from doctesting anyway.
Given a domain expert conclusion that this patch is correct and wanted, I'm going to merge it as approved. Thank you for the review @dschult !


The Big Code Rework™

Draft PR that we can merge once we've made all the rework changes

Current progress? What's left to do now?

Main things, I think, are improving config stuff and reducing code duplication between third and fourth

ok I'm going to do a code review on this and then merge it. might not be perfect, but i want this merged so i can start working on Actual Features again

First note is that before merging I should definitely write some docs on how to add a new feature

This might be about ready now! Perhaps someone should read over the docs, see if it's comprehensible enough lol


Endpoint locality prioritization

Defaults to off and has to be enabled via an env var in Pilot.
Signed-off-by: Liam White<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: liamawhite
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS
pkg/OWNERS

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

@liamawhite: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test e2e-simpleTests-cni

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

@liamawhite: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio-integ-k8s-tests

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

@liamawhite: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio_auth_sds_e2e

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

@liamawhite: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio-pilot-multicluster-e2e

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

@liamawhite: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test e2e-simpleTests-cni

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

@liamawhite: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test istio-integ-k8s-tests

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

@liamawhite: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
395fef7d01d2aa46887c15c1ef192e9766ccfb37
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test istio_auth_sds_e2e

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

@liamawhite: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
74b3f8f4853af25bcb1136a631b44f3ab8805872
link
/test istio-pilot-multicluster-e2e

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

This is not right.   For different proxies, pushEds concurrently, the ClusterLoadAssignment modification may influence each other. That is why I implement a shallow copy in https://github.com/istio/istio/pull/11256/files#diff-6713acfe7b9e389e7dcd429d08bee859R260


Minor tweaks to health content standard and service standard

Received feedback after launch. Approved by clinicians. Ready to publish next Monday.

PR: https://github.com/nhsuk/nhsuk-service-manual/pull/1769


Comment here to be added as a collaborator

Comment below - you can say anything
We will all be using this repository today. Even though it's public right now, you will not be able to make changes until you're given the correct permissions. We've automated this process via the GitHub API and some fancy chatops. Once you comment, we will add you as a collaborator.
You'll also start to receive a lot of emails. (:exclamation:) You can slow these down immediately by clicking the 'Unwatch' button and selecting either the 'Not watching' or 'Ignoring' option.

👋🏾 👋🏾 Please add me as a collaborator

Hello GitHub Teacher.


Fix language issues

Resolves #323.
It also adds several more languages that Tor Browser can be downloaded in.
@intrigeri this appeared to work for me, but do you mind testing this with your system language set to French and confirm it fixes the issue?

I'm pretty sure this works so I'm going to go ahead and merge. I need to make a release because there's now a signature verification error that only happens in the python2/twisted codebase. But please re-open #323 if this doesn't resolve it for you.


Allow save operation for cytoscape directly

Signed-off-by: Ashish Tiwari<EMAIL_ADDRESS>Notes for Reviewers
This PR fixes

K8s_manifests field is deprecated since that is no longer used after the source_content feature.
Allows Cytoscape to directly save their resultant patternfiles given the ID

Signed commits

[ ] Yes, I signed my commits.

Codecov Report
Base: 16.75% // Head: 12.56% // Decreases project coverage by -4.18% :warning:

Coverage data is based on head (75859b2) compared to base (890073f).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6679      +/-   ##
==========================================
- Coverage   16.75%   12.56%   -4.19%
==========================================
  Files         115      115
  Lines       11239    11194      -45
==========================================
- Hits         1883     1407     -476
- Misses       8881     9473     +592
+ Partials      475      314     -161

Flag
Coverage Δ

gointegrationtests
?

unittests
12.56% <0.00%> (+0.05%)
:arrow_up:

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

Impacted Files
Coverage Δ

server/handlers/meshery_pattern_handler.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/start.go
1.70% <0.00%> (-57.03%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
1.12% <0.00%> (-53.94%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.17% <0.00%> (-47.06%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.88% <0.00%> (-45.29%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.22% <0.00%> (-41.49%)
:arrow_down:

mesheryctl/internal/cli/root/root.go
24.24% <0.00%> (-40.91%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.26% <0.00%> (-36.96%)
:arrow_down:

mesheryctl/internal/cli/root/version.go
0.00% <0.00%> (-32.95%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.03% <0.00%> (-23.03%)
:arrow_down:

... and 3 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


How can we import this on Quartus?

Hi Paulo,
I am a retro-computing enthusiast and I have been searching for a solution to use UART over JTAG for small Z80 emulations I write inside FPGA boards.
By searching for something that could be used without NiosII I have found this project of yours, but I am too much of a noob to get it to even import onto one of my Quartus projects.
I have tried using the Altera's JTAG-Uart module, the one with the Avalon bus, but I couldn't adapt it to work without NIOS.
I converted it onto a schematic element and I do get a few characters (64 to be exact) to print onto the nios2-terminal but that's it, it doesn't go any further, that's when I started searching the web for an alternative.
I was looking for something that I could use to emulate a regular UART, like a 16550 or 6850, just using the USB JTAG connection without having to plug a USB-UART dongle into th board.
Muito obrigado por qualquer ajuda.
Marcelo.

Hi Marcelo,
This repository contains a wrapper for using the alt_jtag_atlantic in Bluespec. If you use Bluespec, the wrapper exposes two FIFOs (rxfifo and txfifo) where you can get/put bytes. It is really easy to use, just see the examples.
If you use Verilog or VHDL, you can instantiate alt_jtag_atlantic directly. Its signals are:

r_ena (output): tells if you can write a byte to the UART during the next cycle (important: it's not during the current cycle! it's during the next one).
r_dat and r_val (inputs): provide a byte to r_dat and assert r_val high to write the byte to the UART.
t_dav (input): assert high if you want to read a byte from the UART during the next cycle.
t_dat and t_ena (outputs): if t_ena is high, then a byte is ready to be read from t_dat.

Please note alt_jtag_atlantic is not a part of this project. It is a "builtin" module of Altera's synthesizer. If you are using Verilog or VHDL you don't need anything from this project (for the hardware part), only the documentation above.
As for the software part, you can refer to the software directory.
Please feel free to reopen this issue if you need any further information.

FYI this may be a helpful example: https://github.com/tommythorn/yarvi/blob/master/shared/axi_jtaguart.v

Awesome!
I will give it a try.
Thank you so much.
On Wed, Apr 28, 2021, 7:52 PM Tommy Thorn @.***> wrote:

FYI this may be a helpful example:
https://github.com/tommythorn/yarvi/blob/master/shared/axi_jtaguart.v
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thotypous/alterajtaguart/issues/1#issuecomment-828852297,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD74NCZT2ARHD2UBD2AUVO3TLCNS3ANCNFSM43TXYU4Q
.


fix: iOS focus

What
iOS focus method did not do anything because the device stayed in continuousAutoFocus mode
Changes

use autoFocus and autoExpose instead when focus is called
enable isSubjectAreaChangeMonitoringEnabled
when subject area is changed then switch back to continuousAutoFocus and continuousAutoExposure

Tested on

iPhone 13 Pro iOS 16.3
iPhone 6s Plus iOS 13.1.2

Related issues
https://github.com/mrousavy/react-native-vision-camera/issues/1376#issuecomment-1483633319

Having the same issue with tap to focus, I tested Levipro's version and it works perfectly for me on iOS (iPhone 13 pro)

So after focus is complete, it returns back to continuous auto focus? makes sense


instantaneous_rate() function should take a list of spike trains as input

The function to calculate the instantaneous rate profile should be able to deal with lists of spike trains (like the time_histogram() function), e.g., to calculate the smoothed PSTH.

@rgutzen can you take care of this feature request? Thanks!

see pull request #166


Add workspace dependencies

This ensures that if a leaf dependency has been modified that the upstream consumers are rebuilt.

Info on the workspaces protocol: https://pnpm.io/workspaces


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/235 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit fa77430:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Added missing space in the transaction submission message

The logged message didn't contain a space between method name and params (resulted in e.g. submitPublicKeyparams:).
In this PR we added a space.
Sample message that used to be logged:
2021-06-01T16:03:57.284+0200	DEBUG	keep-contract-BondedECDSAKeep	submitting transaction submitPublicKeyparams: [49 219 244 160 164 177 151 147 225 55 231 231 13 50 114 78 174 57 148 142 172 39 121 149 129 195 214 68 39 231 163 221 134 243 60 24 160 47 117 146 221 39 240 12 61 223 158 199 217 56 123 186 74 106 19 207 204 16 203 241 234 62 64 254]

Seems like we're getting unrelated celo failures @_@


Improve action on resource table

Description
closes #2879
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Failed to merge changes into iso4 due to merge conflict.Please open a pull request for these branches separately.

Merged into branches iso5, master in 9c233c60a70efa473dd7d2f3912a9adbed417e06

Not closing this pull request due to previously commented issues for some of the destination branches. Please open a separate pull request for those branches. You can safely close this pull request and delete the source branch.


Make eksdReleaseRef a pointer to support previous releases

Issue #, if available:
Description of changes:
Make this field a pointer support backwards compatibility with previous releases
Testing (if applicable):
Ran e2e tests with latest minor release for docker & vsphere
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


Can't delete or edit documents

When I click the 'Delete' button next to a document and then click 'Yes' to confirm, the server dies with this error:
/home/iast/repos/adminMongo/node_modules/mongodb/lib/mongo_client.js:465
      throw err
            ^
Error: Argument passed in must be a single String of 12 bytes or a string of 24 hex characters
at new ObjectID (/home/iast/repos/adminMongo/node_modules/mongodb/node_modules/mongodb-core/node_modules/bson/lib/bson/objectid.js:38:11)
at ObjectID (/home/iast/repos/adminMongo/node_modules/mongodb/node_modules/mongodb-core/node_modules/bson/lib/bson/objectid.js:29:42)
at /home/iast/repos/adminMongo/routes/index.js:740:62
at /home/iast/repos/adminMongo/node_modules/mongodb/lib/mongo_client.js:462:11
at process._tickCallback (node.js:355:11)

Also, when I click the 'Edit' button next to a document, the server doesn't log any errors, but the UI shows a big red error message that says 'Error - Invalid document ID length'
Let me know what kind of info would be helpful in debugging.  Thanks, this is a great project!

Both issues seem related. How long is the '_id' value? The '_id' is being cast to an ObjectID for updates/edits and queries. If this value is not 12 bytes or 24 hex characters there will be issues with the current code.
It's not common to use custom length "_id" values. Are you setting the "_id" value yourself or letting MongoDB do it?

The application uses _id values other than 12 byte values (some strings, some longer byte values).  I'm not in a position to change this implementation.  Would you consider this out of the scope for this project?

I don't think it is out of scope. I'm happy to remove the validation on 12 bytes of data as it is not needed. I'm finding it hard to replicate/fix the query to ensure adminMongo is not trying to find the document based on an ObjectID when your ID's are not MongoDB ObjectID's. Do you want to have a look at it for your setup? Here is where it will fail when not using ObjectID's for example: https://github.com/mrvautin/adminMongo/blob/master/routes/index.js#L339

This should be fixed in fdd4dd216f0fc709f547a86883f0dca9518eb834, sorry about the commit message not mentioning it at all.. ;)
If further issues or a better way of doing it, please comment on issue.

Thanks!


Sudden error: You don't have 'phantomjs' installed

I start to get this error when i run my task. After this message comes nothing happens, and i have to ctrl-c to get control again.
I have tried to uninstall gulp-svg-sprites and then install it again, but the same error comes back.

should be working with the current version


planner: governance voting

Describe your changes
Fold in delegate / validator voting functionality to planner.
Issue ticket number and link
References https://github.com/penumbra-zone/penumbra/issues/4307
Checklist before requesting a review

[x] If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

with the governance passing, do we want to change the base branch in this PR?

Exciting, good job, can you edit the title/description and we'll send it

cc @cratelyn


Limit tags shown in the tags.index that have associated public profiles

and also on the homepage tags list

Good call - maybe also the ones with a count greater than N in general?


Bug/register page allignment

Fixes #2 Improving the allignment of Register page.

@harshbhar0629 fix this text it is misaligned and share your full screenshot I want to know where you've placed the side text

Rest looks good 💯

Ok

@vaibhavyadav-dev Please review it again:)
Current UI:

Is this repo paticipate in gssoc??


Use extension functions to access Realm

Replace the existing RealmManager class by Kotlin's extension functions.

[x] Write an extension function for each method in RealmManager
[x] Make RealmManager use those extension functions
[x] Properly integrate Realm in place of RealmManager where necessary
[x] Check for unnecessary null checks
[x] Remove RealmManager

[ ] Compact the test.realm file

@fernandog: I added a bunch of tests regarding data management in this PR, and in particular some for getting the (filtered) logs. It seems to be working as expected. So I will enable it in the next beta release of ShowsRage. I'll ping you when it's ready so you can confirm that it's working fine.


Static props are permanent

Static props are permanent. They do not delete with the /staticremove command OR if you delete the prop.
There is no way to delete/remove a static prop at all.

FYI:
Latest Stock Github CW with Stock HL2RP Schema.
No extra addons/files at all.

I'll be marking this Issue as Solved. A user (murray-elijah) pointed out a lot of Issues that are solved or expired, so I'm working on that to tidy up Clockwork's Issues.


added khu.ac.ir

add Kharazmi University

@mhajiloo unfortunately we are not allowed to distribute licenses to domains in .ir domain zone - please check the license terms - https://www.jetbrains.com/store/terms/ i'm very sorry but have to close the request. If you have any questions, please reply.


Bayes LOSVD bugfixes: crash when checking for pops columns in BayesLOSVD data, kinematic maps not created

This PR fixes the following problems connected to Bayes LOSVD:

When using BayesLOSVD, pops columns were incorrectly identified causing a crash when calling BayesLOSVD.convert_losvd_columns_to_one_multidimensional_column(). This happened regardless of pops columns existing or not.
Bayes LOSVD kinematic maps were not created for iterations 2+. The reason is that matplotlib raises an error or warning when re-registering a colormap. Fixed this by avoiding re-registering the same colormap in each iteration.

Tested successfully with tutorial 4_BayesLOSVD.ipynb and a changed configuration file to allow for more iterations (n_max_mods: 10, n_max_iter: 3).
Please have a look at the code, test with any BayesLOSVD model and merge if successful ;-)

Will merge per today's core team meeting...


Added a  wait for a max of 30 seconds for LB connectivity to succeed

@alena1108 @cloudnautique , can you please review ?

LGTM


Modsecurity prevents mediawiki login on ISS10

I have removed all rules, but I am still not able to login  to my mediawiki installation. No logs are generated.

Setting SecStreamInBodyInspection to "On" should solve your issue.
Thanks.


Child cell lat/lng outside of parent lat/lng ?

Hi
I'm looking at a resolution 6 H3 cell 86139ec57ffffff and at one of it resolution 9 child 89139ec5493ffff.
The library correctly states via cellToChildren or cellToParent that both are related, but when using cellToBoundary I get lat/lng for the child cell outside of the boundaries of the parent cell.
The resulting two geometries are
GEOMETRYCOLLECTION(POLYGON((-134.7726597 58.428654900000026,-134.7926704 58.40057869999998,-134.7571973 58.374116000000015,-134.7017216 58.37569730000001,-134.6816275 58.403761799999955,-134.7170924 58.43025670000003,-134.7726597 58.428654900000026)),POLYGON((-134.7619536 58.431210300000004,-134.7593701 58.43016260000002,-134.7566931 58.430801299999985,-134.7565995 58.43248779999996,-134.7591832 58.433535500000005,-134.7618603 58.432896700000015,-134.7619536 58.431210300000004)))
and you can see (for ex on https://clydedacruz.github.io/openstreetmap-wkt-playground/) that the child is well outside of the parent. I was expecting every child to at least intersect with the parent geometry.
Is this expected due to distortion at these latitudes?

Yes, this is expected from H3. There are some more details of this on https://h3geo.org/docs/highlights/indexing. These child cells are logically but not necessarily geographically contained.
This is not an effect of latitude but rather a property of the hierarchical hexagonal grid. Hexagons do not cleanly subdivide into 7 children, so there is a fractal pattern around the edges as you subdivide.

Okay, thanks a lot for clarifying - I guess if I had computed the chain of children from resolution 6 to 9 I would have seen this.   I guess my thinking was mislead by the usual example of the doc where only the direct children are shown.


Missing funds

Hello a couple of weeks ago i sent 1776 tethers to my copay wallet and only received 10 usd please help me

Please send me a message. I can guide you with some instructions.
My email<EMAIL_ADDRESS>

Hello,
We are closing this thread, please if you still need support, open a new one.
Thanks.

my question is, is there a way you can re mine the block to unlock the funds I have lost to do so I will provide you with all available information like the hash number the wallet address the date etc

De : BGalliotti<EMAIL_ADDRESS>Envoyé : 20 avril 2018 12:03:01
À : bitpay/copay
Cc : Doublef1000; Author
Objet : Re: [bitpay/copay] Missing funds (#7606)
Closed #7606https://github.com/bitpay/copay/issues/7606.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/bitpay/copay/issues/7606#event-1586508684, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ag872a_PlMT2l5EmU1pE3bGImoauSuEvks5tqga1gaJpZM4RKoyg.

Is there a way to retrieve the missing funds because i can prove i sent 1800 usd in tethers and only retrieved 0.10 usd

Hello, unfortunately there is no way to recover those funds other than recovering the wallet with the backup (12-word phrase, file,..)
What you raise in this comment

my question is, is there a way you can re mine the block to unlock the funds I have lost to do so I will provide you with all available information like the hash number the wallet address the date etc

(re mining a block) would need to at least own 51% of the nodes of the bitcoin network to begin.. Practically impossible, this is not affordable neither it is worth it.
Thanks for your contribution.

@Doublef1000
I can recover your wallet if you want need help send me email<EMAIL_ADDRESS>


pub_sub example with FastAPI

Describe the issue
The code sample with FastAPI (/quickstarts/pub_sub/python/sdk/order-processor-fastapi) doesn't work quite right (maybe it's just me). Order-processor-sdk and checkout-sdk start, but order-processor-sdk doesn't accept messages. The Flask example works correctly
URL of the docs
quickstarts/pubsub-quickstart
Expected content
dapr.yaml
version: 1
common:
  resourcesPath: ../../components/
apps:
  - appID: order-processor-sdk
    appDirPath: ./order-processor-fastapi/
    appPort: 6001
    command: ["uvicorn", "app:app"]
  - appID: checkout-sdk
    appDirPath: ./checkout/
    command: ["python3", "app.py"]

Screenshots

The solution is to change the order-processor-sdk port from 6001 to 8000 and update the commands for both apps
version: 1
common:
  resourcesPath: ../../components/
apps:
  - appID: order-processor-sdk
    appDirPath: ./order-processor-fastapi/
    appPort: 8000
    command: ["/bin/bash", "-c", "source .venv/bin/activate && uvicorn app:app"]
  - appID: checkout-sdk
    appDirPath: ./checkout/
    command: ["/bin/bash", "-c", "source .venv/bin/activate && python3 app.py"]


KVO category

KVOController - Simple, modern, thread-safe key-value observing for iOS and OS X.

I am ok with a KVO category
Can we move this one there as well?

Hanson - Lightweight observations and bindings in Swift, with support for KVO and NotificationCenter.

@arasu01 @serhii-londar @Mylittleswift


R6-Bot

Hello hmes!! I love the work you did, I have a question im new into all of this of discord bots, Im a mechatronic engineer. I was wondering if there's any way to give a user on discord their R6 Rank as a Role on the server after they submit their username, i would really appreciate if you could help me out with this

Hey Brunituns, you can use the rank function of this API to obtain the user’s rank, and then use the role function of discord.js to assign roles. However, I am not familiar with role assignment in discord.js, so I may not be able to help much.

It will be a challenge first i have to understand how to create disc bots and then look at the discord.js to figure out the rest. Thanks for the reply!!


Add mixins for fedora36 and fedora37 os types

This PR adds support for static and virtual machines running Fedora 36 or Fedora 37 by skipping the EPEL install on setup.

Any updates on this?

Sorry, something got stuck with our pipeline. This should be in now. Thanks!

Not a problem, thanks for approving/merging!


podman-compose: 0.1.8 -> 1.0.3

Motivation for this change
podman-compose is outdated.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

podman-compose

+1 it's not even possible to run a command in a running container via "podman-compose exec" due to https://github.com/containers/podman-compose/issues/308


⚠ Bofrost.de has degraded performance

In 735131b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1280 ms

Resolved: Bofrost.de performance has improved in cbba7a7 after 9 minutes.


Enhanced Script with AI Features and Issue Resolutions

This Pull Request introduces several key enhancements and resolves specific issues in the original script, focusing on AI-based optimizations, anomaly detection, and code quality improvements as flagged by SonarLint.
Key Features Added:
AI-Based Anomaly Detection:
Integrated an IsolationForest model to detect and remove anomalies from the dataset before processing. This enhances the reliability of the data used in training models, ensuring that outliers do not negatively impact the results. Implemented a fixed seed (random_state=42) in the IsolationForest to ensure reproducibility, addressing the SonarLint warning regarding the absence of a seed (python:S6709).
AI-Optimized Batch Size:
Added a dynamic batch size optimization feature that intelligently calculates the optimal batch size based on the dataset. This ensures efficient data processing and reduces the need for manual tuning.
AI-Based Intelligent Chunking Strategy:
Introduced an AI-driven approach to determine the best chunk size for HDF5 storage. This optimizes I/O performance by considering the memory footprint of the dataset and aligning it with the batch size.
Issues Resolved:
SonarLint Warning (python:S6709) - Random State for Reproducibility:
Resolved by adding a random_state parameter with a fixed seed in the IsolationForest model. This ensures that the results are reproducible across different runs. SonarLint Warning (python:S1226) - Avoid Reassigning batch_size:
Instead of reassigning the batch_size variable, a new variable optimized_batch_size was introduced to store the AI-optimized batch size. This maintains the integrity of the original batch_size value while applying the optimized setting.
Impact:
These updates significantly enhance the script’s functionality, making it more robust, efficient, and reliable. The AI-driven features automate key aspects of data handling, while the code quality improvements ensure maintainability and clarity.

This pull request introduces several enhancements and resolves specific issues in the script, primarily focusing on AI-driven optimizations and addressing SonarLint warnings.
Key Changes:
AI-Enhanced Anomaly Detection:
Integrated an IsolationForest model to detect and remove data anomalies, ensuring a cleaner and more reliable dataset for model training.
Added a fixed seed (random_state=42) to ensure reproducibility, which resolves the SonarLint warning python:S6709.
Dynamic Batch Size Optimization:
Implemented an AI-based method to dynamically calculate the optimal batch size based on the dataset size, improving processing efficiency.
Intelligent Chunking for HDF5 Storage:
Introduced a strategy to determine the optimal chunk size for HDF5 storage, enhancing I/O performance.
SonarLint Compliance:
Addressed the python:S1226 warning by introducing a new variable to handle the optimized batch size, avoiding reassignment of the original batch_size.
Impact:
These changes enhance the script’s efficiency, reliability, and maintainability. The AI-driven features automate key processes, while the SonarLint fixes ensure the code is clean and compliant.


Cluster: Update the images and solution to tWAS <IP_ADDRESS> (eGA: Aug 30, 2022)

Runbook: https://github.com/WASdev/azure.websphere-traditional.cluster/blob/main/docs/howto-update-for-was-fixpack.md

tWAS ND CI/CD: https://github.com/WASdev/azure.websphere-traditional.image/actions/runs/2978841935

9.0.20220902

IHS CI/CD: https://github.com/WASdev/azure.websphere-traditional.image/actions/runs/2979244564
osDiskSasUrl: https://storage297924456430.blob.core.windows.net/vhds/ihs297924456430.vhd?st=2022-09-01T13%3A37Z&se=2022-10-02T13%3A37Z&sp=rl&sv=2021-06-08&sr=c&sig=KK4WaVfldS1c4o5%2BNnSjxaXkvmrTLUovOAQlR51LDeM%3D
dataDiskSasUrl: https://storage297924456430.blob.core.windows.net/vhds/ihs297924456430datadisk1.vhd?st=2022-09-01T13%3A37Z&se=2022-10-02T13%3A37Z&sp=rl&sv=2021-06-08&sr=c&sig=KK4WaVfldS1c4o5%2BNnSjxaXkvmrTLUovOAQlR51LDeM%3D

9.0.20220902

Submitted the image for publishing

ND and IHS images are submitted to go live

Images are live. Updated the solution files.
Packaging: https://github.com/WASdev/azure.websphere-traditional.cluster/actions/runs/2980963644

Solution submitted for review:

Preview test failed. Jianguo fixed it in this PR and also added Integration Test Workflow.
https://github.com/WASdev/azure.websphere-traditional.cluster/pull/175
Integration Test and packaging successful: https://github.com/WASdev/azure.websphere-traditional.cluster/actions/runs/3001957454

Submitted for publishing again:

Jianguo verified the cluster solution update:

Submitted for final publishing:

Cluster solution is live:


ConfigMap/Secret volumes should be mounted as read-only

As I'm reading through the volume module, it appears as though we do not prepare the ConfigMap and Secret host mounts as read-only. We just write the data to a filepath in the volume and provide that directory to the Provider.
https://github.com/deislabs/krustlet/blob/0b99368e91934d489ade88be99188ec632509475/crates/kubelet/src/volume/mod.rs#L41
https://github.com/deislabs/krustlet/blob/0b99368e91934d489ade88be99188ec632509475/crates/kubelet/src/volume/mod.rs#L163
In Kubernetes, those volumes are mounted as read-only.

@bacongobbler After looking at this some more, I think this is a per provider implementation detail (as we might not be actually "mounting" something depending on what it is). I don't think this is currently possible from the WASI side of things. However, we could set the file permission modes to readonly if you think that might help as well. But at least from the WASI side, if they have full access to the directory, they should be able to just modify permissions back

However, we could set the file permission modes to readonly if you think that might help as well.

I believe this is how Kubernetes handles read-only permissions, though I believe they would use cgroups and vfs mounts to set up read/write permissions.
https://docs.oracle.com/en/operating-systems/oracle-linux/6/adminsg/ol_cgconfig_cgroups.html
Here's where the kubelet sets up standard volume mounts:
https://github.com/kubernetes/kubernetes/blob/ea0764452222146c47ec826977f49d7001b0ea8c/pkg/kubelet/kubelet_pods.go#L257-L266
And here's where the kubelet sets up raw block volumes:
https://github.com/kubernetes/kubernetes/blob/ea0764452222146c47ec826977f49d7001b0ea8c/pkg/kubelet/kubelet_pods.go#L129-L135

Generally the providers don't mount the volume themselves, the CSI plugin does when it is called. But in the configmap/secrets case, we aren't mounting anything. We are just preopening the file from WASI. A container provider will be loading the local file system path as a volume into the container, but there is no such thing for WASI, so this is a little unclear. Perhaps we should sync up on this at the next meeting


Add Empty cache support

The MQTT library sets a clean session by default, but we want to change this behavior by disabling it.
mqtt-espidf - disable_clean_session
Closes #115.

I see a couple of problems here:

First, send_emptycache is never called, while it should be called in the on_connected callback, after send_introspection.

Yes right, my error I had left it out after testing the release1-0.

Second, I'm not 100% sure that disabling clean session in the current state of the SDK is the right thing to do, but this is more a "are we sure? let's reason about the corner cases" rather than a "don't do this"

If you don't disable it, the field session_present passed in on_connected function will  always be 0, and the empty_cache ( also introspection and subscription) will be resent on every connection.


chore: Finsupp.coe_finset_sum simp

From LeanCamCombi

Please follow the PR title conventions. You can find them here: https://leanprover-community.github.io/contribute/commit.html
bors d+

bors merge


docs(scully): add collage club to showcase

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/scullyio/scully/blob/main/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Other... Please describe:

Add Collage Club to Scully showcase
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

 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.


Could you please provide the file of preprocessing.zip

The URL in the file download.sh cannot download the file properly. I can't access anything in there. Could you please provide the file of preprocessing.zip ?

Hi! Thanks for pointing out this file issue. Please, find the file in following link: https://drive.google.com/file/d/1exfeojXRnK9eeJjtcBYrBvL5Px3U29zU/view?usp=sharing

hello！Thank you very much for your documentation. I just remembered that because the website where you need to download files is down, all the files in it are not available. I'm so sorry that could you please send a copy of the complete file that needs to be downloaded in download.sh? Thank you very much.
七米工资
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "ericashimomoto/parameter-efficient-tvg"                                                                                @.>;
发送时间: 2024年5月4日(星期六) 晚上10:25
@.>;
<EMAIL_ADDRESS>主题: Re: [ericashimomoto/parameter-efficient-tvg] Could you please provide the file of preprocessing.zip (Issue #1)
Hi! Thanks for pointing out this file issue. Please, find the file in following link: https://drive.google.com/file/d/1exfeojXRnK9eeJjtcBYrBvL5Px3U29zU/view?usp=sharing
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

could you update all the other download links in the ./download.sh?  many thanks.


Dynamically render component from ComponentSpec

Motivation
It's tedious to manually create wrappers for native Angular components even with a component generator script, particularly because some components may only exist internally in our downstream codebase.
We will still provide a way to manually specify components the current way, for advanced use cases, but for >80% of use cases, we should be able to code-gen the component.
Challenges
One of the difficulties is that it can produce difficult to understand errors
Design
Outputs
What it needs to generate?

Angular:

Template
Component renderer

Python:

Component API
Export symbol in mesop/__init__.py

Tests (although, this is less important if everything is code-generated)

Inputs

Python API
Angular component source

How
Use TS compiler (or just parse AST) of the specified Angular component.
Generate a ComponentNgSpec based on the input and output. This will turn into a proto. We can use this proto to: 1) generate the Python file (may or may not be checked-in) and 2) use it to dynamically resolve the Angular component.
import { Injectable, ViewContainerRef, ComponentRef } from '@angular/core';
import { YourComponent } from './path-to-your-component';

@Injectable({
  providedIn: 'root'
})
export class DynamicComponentService {
  constructor(private componentFactoryResolver: ComponentFactoryResolver) {}

  createComponent(viewContainerRef: ViewContainerRef): ComponentRef<YourComponent> {
    const componentRef = viewContainerRef.createComponent(SomeComponent);

    // Optional: Interact with the component
    // componentRef.instance.someInput = 'some value';
    // componentRef.instance.someOutput.subscribe(() => { ... });

    return componentRef;
  }
}

Steps:

[x] Do DynamicComponentService, but manually write the code.
[x] Specify ComponentSpec protocol to interact with DynamicComponentService
[x] Write protocol data by hand and have DynamicComponentService programmatically use ComponentSpec data
[x] Make sure compiler optimizations don't break the above steps
[x] Generate Python class based on ComponentSpec
[x] Generate ComponentSpec using TS compiler (AST traversal)
[x] Create CLI to generate ComponentSpec using Angular Component TS source file as input

This is pretty much done, it's worked for a few components (checkbox, button and input). I suspect there'll be some kinks with the more complicated ones, but that's to be expected.


Countless dependency cycles in core package

Summary
I am trying to use cardano-sdk in SES environment. I am unable to do so getting
Class extends value undefined is not a constructor or null

Most likely, this is because of dependency cycles. I tried using madge which found ~165 dependency cycles in the core package. Most of them I suspect, can be fixed with import type.
Steps to reproduce the bug
Run
yarn madge --circular src/index.ts --debug --extensions ts in packages/core
Actual Result
`✖ Found 165 circular dependencies!
Expected Result
Found 0 circular dependencies!
SDK version
0.28.4
Environment type

[ ] Node.js
[ ] Browser
[ ] Browser Extension
[X] Other

Environment details
No response

We actaully have a circular dependency check... it seems the one you run is stronger than the one currently implemented; we'll take a look.
Once said that, it seems you called "steps to reproduce the bug" the check you performed to discover circular dependencies: could you please tell which is the command which gives you the first error reported?
Thank you

Thanks for the response, we are building a Metamask snap for Cardano, and I get the error when building the snap, just running yarn build. I don't see much in the insides of the building or the SES environment. It would IMO make sense to look deeper into it if fixing the circ. dependencies did not fix the issue.
Btw some of the circ. dependencies are quite direct

Cardano/Address/Address.ts > Cardano/Address/BaseAddress.ts
Cardano/Address/Address.ts > Cardano/Address/ByronAddress.ts

Thank you for a report! It seems like core package is missing a circular-deps:check script that is present in other packages, so we didn't catch those issues :facepalm:
We will fix it, but can't make any promises on the timeline. In case you need this resolved asap, code contributions are welcome

Thanks, I will consider fixing it myself :+1: Since it should be "just moving stuff around", would you be up for making a quick npm release for that or is there some time-consuming process for making releases?

Publishing packages is one-click, no problem :)

Hmm, I don't see it working without breaking changes, you can create Address in BaseAddress and BaseAddress in Address. Would you open to release also with breaking changes? I am working on it, but I can already see that at least this needs to be changed.

Perhaps you could create a list of remaining issues that you couldn't solve without breaking changes, I'll take care of those

That would be great, thanks. The thing is tho, I need to have it, just to try the package and move forward with my other work. If its ok, I ll try fixing everything, also with breaking changes, possibly open PR. You can then decide to reuse what makes sense to you and change what does not. This way I can move forward and you can have control of how the breaking changes look, without me pinging you every day waiting for it to be ready :smile:

Thanks @PeterBenc, yes this is a good approach. We're a little behind on contributor documentation but please be aware we're implementing the Conventional Commits standard with atomic commits.
It's great to see you considering this SDK for your project. Please feel free to open any discussions during your evaluation

Update
It turned out that the dependency cycles were not the issue, SES env I am working with does not include built-in websocket which caused the error, isomoric-ws extends the websocket, thus the Class extends value undefined is not a constructor or null.
I fixed the issue by mocking the isomorfic-ws package in webpack.
Regarding the dependency cycles. I fixed them all. But honestly, nothing that I would be proud to share. More time and changes are needed for the proper solution.

Thank you for sharing the solution


Revert #20 fix: update DeleteHandler behaviour to succeed if resource does not exist

Description of changes:
According to the resource type contract, the Delete handler "MUST return FAILED with a NotFound error code if the resource didn't exist prior to the delete request". This means that the Delete handler cannot have the same behaviour as the DeleteStateMachine API, which would return successfully if the resource did not exist.
This change reverts PR #20 - fix: update DeleteHandler behaviour to succeed if resource does not exist #20
.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

No changes, as the revert was done in https://github.com/aws-cloudformation/aws-cloudformation-resource-providers-stepfunctions/pull/21/files


Add Definition: Automatic .308 Rifle

Add a Model for automatic .308 Rifles

FN SCAR 20S
FN SCAR H
and similiar. Change mechanism rom .308 Rifles which are non automatic but BoltAction-operated.

Planned Definition Name: Rifle_308AR
Planned Parts

RifleBarrel_308AR
RifleBoltAssembly_308AR
RifleTriggerGroup_308AR
RifleReceiver_308AR
RifleSafety_308AR
RifleStock_308AR
RifleSights_308AR
RifleActionGroup_308AR

Has been merged into develop with #89


How to check user has authorized to (GSuite Domain-Wide Delegation of Authority) to my application using nodejs

Hello,
Is there any example that I validate the authority of the service account using nodejs?
I have Client ID, client secret Json file and Email address and Scope are https://www.googleapis.com/auth/admin.reports.audit.readonly.
Its domain wide not a user wide.
I need a nodejs example, which will check if the customer delegate / authorize domain-wide authority to my application. I don't want the user to go to consent UI and grant access to my application. In my case the user (domain administrator) has done all the steps (grant third-party applications with domain-wide access) manually, and I need to see if he correctly grants the access to my application.
Thank you in advance.

I got it working.
Thank you.

I have the same problem. Can you send me an example?

and the answer is? @khatekar


small optimization in multiplayer_synchronizer.cpp

i changed it to instead of dividing by 1000 then by 1000 again, it will divide by the result of 1000 times 1000

why not just divide by 1000 000.0?

Just like your previous 2 attempts at such optimization PRs, this change does just about nothing.
Even if this PR does get merged, please do not take that as encouragement to post more such "optimization" PRs and instead focus on bugfixes and actual improvements if you want to contribute to Godot.

I agree with RedworkDE, the only thing this optimizes is when the engine is compiled without optimization (and even then it's a micro-optimization that has no impact on anything).
With -O1 or higher, both variants compile to the exact same assembly:
https://godbolt.org/z/MYabhjYGj
So I'll reiterate my comment from https://github.com/godotengine/godot/pull/78424#issuecomment-1596653212
If you want to actually help improve the engine, please coordinate with us to work on something that will indeed make things better.


Emit metrics for compaction priority queues

In #3460 a new bounded priority queue was introduced for compaction jobs.  Metrics need to be emitted for these priority queues so that the current amount queued can be known.  Knowing this can useful for users tuning settings and for K8s autoscaling of compactors.

I spent some time looking over #3640 for these changes and started working on the logic to add the following metrics.

Queue Size
Jobs Queued
Job Time Spent in Queue.

Are there any other metrics that should be added?

Are there any other metrics that should be added?

It may be useful to have a sense of the number of jobs being rejected or ejected because the queue is at max size.  But I am not sure how to capture this as a metric.

Added the following metrics:

Number of Queues
Size of Queue
Number of Jobs in Queue
Number of Jobs Rejected
Number of Jobs Dequeued.

I'm not sure how to best track job queue time as that starts making a LOT of time series metrics that are short lived.
We could work on adding telemetry exposing the job queue time there, but that's outside of metrics.
We could also expose a MaxWait metric per queue, but I'd love to actively track the wait time vs calculate it based on when the job leaves the queue.

@ddanielr - Did #3551 resolve this, or is there more to do here?


Custom alerts actions in dropdown menu?

Hello,
thanks for your work on karma!
In the karma deployment we have it is somewhat common to create tasks/issues from an alert for further followup. A "click here to create a new task for this alert" is possible today via annotations (i.e. creating a task is an HTTP GET) but unwieldy (i.e. having to attach the same annotation when defining alerts).
I was wondering if there's interest in having this feature in karma itself? Namely sth like being able to define (templated) links that will show up in the alert's dropdown, alongside "alert source links" and "silence this alert" actions. What do you think ?
thanks again!

Being able to move some annotation links to the actions menu might be a nice way of decluttering  the UI when there's plenty of them.

Being able to move some annotation links to the actions menu might be a nice way of decluttering  the UI when there's plenty of them.

Yes I think moving some annotations to the drop down is going to be nice. What do you think re: adding also custom actions from karma configuration itself ?

Yes I think moving some annotations to the drop down is going to be nice. What do you think re: adding also custom actions from karma configuration itself ?

Can you give me an example of such action?

Can you give me an example of such action?

Any action that's likely common to all alerts would fit the bill I think, e.g. "open a new task". The advantage of having the configuration in Karma itself (in addition to be able to turn annotations into drop-down actions) would be that common actions don't need to be added as annotations to all alerts. Hope that makes sense!

Any action that's likely common to all alerts would fit the bill I think, e.g. "open a new task". The advantage of having the configuration in Karma itself (in addition to be able to turn annotations into drop-down actions) would be that common actions don't need to be added as annotations to all alerts. Hope that makes sense!

How would you define "open a new task" action? Is that a link to something?

How would you define "open a new task" action? Is that a link to something?

That's correct, for example in phabricator you can build URLs to create a new task, with fields pre-filled via query string https://www.mediawiki.org/wiki/Phabricator/Help/Task_Creation_URL. I'm assuming similar systems have the same capability.
A different but related example might be "escalate this alert" to e.g. create a new incident from the alert in tools like pagerduty.
Hope that helps!

That's correct, for example in phabricator you can build URLs to create a new task, with fields pre-filled via query string https://www.mediawiki.org/wiki/Phabricator/Help/Task_Creation_URL. I'm assuming similar systems have the same capability.
A different but related example might be "escalate this alert" to e.g. create a new incident from the alert in tools like pagerduty.
Hope that helps!

On one hand you can add annotation links. On the other, as you point out, doing so for all alerts might be a bit of toil.
My only concert is that a global action link might not be useful enough and soon someone will request a way to template it, and then we're back to generating links from labels, which one can already do via annotations.

On one hand you can add annotation links. On the other, as you point out, doing so for all alerts might be a bit of toil.
My only concert is that a global action link might not be useful enough and soon someone will request a way to template it, and then we're back to generating links from labels, which one can already do via annotations.

I see what you are saying re: templating within Karma (and in fact that's what I had in mind originally). I'd be happy too with what you suggested, namely being able to move certain annotations into the drop-down menu. I think that'd be already a very good step in the right direction!

I see what you are saying re: templating within Karma (and in fact that's what I had in mind originally). I'd be happy too with what you suggested, namely being able to move certain annotations into the drop-down menu. I think that'd be already a very good step in the right direction!

Added new option for moving annotations into dropdown menu, example:
annotations:
  actions:
    - docs
    - jira


Image does not render correctly when included in ternary operator

tl;dr
A React Image component did not render correctly when included inside a ternary operator. To get around this, I had to separate the Image as a standalone component. After creating this separate component and requiring it in the original component,  the Image rendered correctly according to the logic of the ternary.
Is this the correct behavior for the Image component?
Considering that the separate component and the original Image are functionally the same, I don't see how creating the Image as a separate component would cause this change in behavior.
Background
I wanted to create a 'like' button--its default 'unliked' state being a grey heart and its 'liked' state being a color-filled heart.
To implement this I used a ternary operator in render function's return block.

render: function() {
  return(
    <View style={{flex:1}}>
      <TouchableOpacity onPress={this.props.updateHearts}>
        { this.props.hasPressedHeart ?
         <Image
            source={FullHeart}
            style={{width:30, height:30}}
          />
          :
          <Image
            source={EmptyHeart}
            style={{width:30, height:30}}
          />
        }
      </TouchableOpacity>
    </View>
  );
}

In effect, if the user had already pressed the heart, the image would be a "FullHeart," and if not, then the image would be an "EmptyHeart."
The Issue
The correct image would not render after new props were passed to the component.
I verified that the correct props were correct my rendering the logic of the ternary as text, like so:
<Text>
  {this.props.hasPressedHeart ? 1 : 0}
</Text>

Even when the ternary evaluated to false and displayed "0", the corresponding ternary for the image did not evaluate to "FullHeart".
The Workaround
To get around this problem, I created the heart images as separate components and required them like so:
<View style={{flex:1}}>
  <TouchableOpacity onPress={this.props.updateHearts}>
    <View>
      { this.props.hasPressedHeart ?<FullHeart/> : <EmptyHeart/> }
    </View>
  </TouchableOpacity>
</View>

var FullHeart = React.createClass({
  render(){
    return(
      <Image
        source={{uri: 'http://i.imgur.com/SXHb8nG.png?1'}}
        style={{width:30, height:30}}
      />
    );
  }
})

After doing this, the correct image was rendered after receiving new props.

It definitely works for me. Not sure what you do in the updateHearts function.

@mrblueblue - if you're still having this issue, ping me and we can look into it further!


Rewritten code and added documentation

Rewritten code and added documentation.

Thanks but these changes are already done... By some other user...

Okay, my bad. (Btw: the other user's version doesn't contain a validation on textareas yet, to check if they don't already have a placeholder.)

Thy checked fr the placeholder In textarea too.. N thanks for your efforts..!


Success generate but got another blank page

js:
generatePDF(invoice_no) {
var element = document.getElementById("export");
var opt = {
margin: 0,
filename: ${invoice_no}.pdf,
image: { type: "jpg", quality: 0.98 },
html2canvas: {scale:2},
jsPDF: { unit: "cm",format:[21,29.7], orientation: "portrait" }
};
html2pdf().set(opt).from(element).toCanvas().save()
},

Is the entire pdf made of blank pages?  How long is the document?

filename should look like this i guess: ${invoice_no} + '.pdf'

@Moziezez The OP was using a template string with backtick ` characters in the code, which is fine and will work well, but github uses those to let you format text as a "code" section, so it took those characters out and applied the formatting to that section, which is why it looks strange in the post.  For future reference, @xinyun-onesoftlab, you can use triple backticks ``` to make a code block like this:
generatePDF(invoice_no) {
    var element = document.getElementById("export");
    var opt = {
        margin: 0,
        filename: `${invoice_no}.pdf`,
        image: { type: "jpg", quality: 0.98 },
        html2canvas: {scale:2},
        jsPDF: { unit: "cm", format: [21, 29.7], orientation: "portrait" }
    };
    html2pdf().set(opt).from(element).toCanvas().save();
}

By the way I was asking about document length because there is a known issue where long documents are rendered entirely blank.  The html3pdf solution mentioned there may resolve this issue.


Update how-to-stop-start-server-cli.md

error appears
argument --resource-group/-g: expected one argument
if no resource-group is passed

@sunilagarwal
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

please change the owner to myself

@AlicjaKucharczyk - I've reassigned the PR to you, but the article's author metadata can't be changed here in the public repo. Could you please open a PR in the private repo (azure-docs-pr) to update the article's author? After you change that information, whenever someone modifies this article, you will receive a notification.
IMPORTANT: When the changes in this PR are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@AlicjaKucharczyk - I've reassigned the PR to you, but the article's author metadata can't be changed here in the public repo. Could you please open a PR in the private repo (azure-docs-pr) to update the article's author? After you change that information, whenever someone modifies this article, you will receive a notification.
IMPORTANT: When the changes in this PR are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

changed the author in another PR


Previously available function Ship#rotated_by() not longer available

Description
Previously available function Ship#rotated_by() is not longer available
Environment

Python version: 3.11.2
Operating System: Debian 12
Version of the Python SoCha Client: 2.1.1

Expected Behavior
Calling the function Ship#direction.rotated_by(n) returns a new Direction.
Actual Behavior
Calling the function Ship#direction.rotated_by(-1) leads to an error:
AttributeError: 'builtin_function_or_method' object has no attribute 'rotated_by'

This error occurres due to a duplicate caused by adding a method named direction to the Ship class, which already had an attribute with the same name. A new version will be released on Sunday evening to fix this bug.


make some tutorial graphic

show at the beginning: give context what are we doing? don't be overly technical. also how does it work what does it mean? some limitations?

move some of it to #51


5.6 [Concurrency]: Synchronize adding a child task to a TaskGroup with parent task

Description:
When a task is adding adding new children to a task group, we need to
synchronize with the task status record lock of the parent task that has the
task group, to prevent races with concurrent cancellation and escalation.
Risk:
Low, this is adds extra synchronization which prevents a race between child task creation and cancellation
Reviewed by: @ktoso , @rjmccall
Testing:
Not easy to test since this is about fixing a hard-to-reproduce race between concurrent task cancellation and creation.
Original PR:
https://github.com/apple/swift/pull/40520
Radar-Id: rdar://problem/86311782

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - c3e6d71c99c81e4d1cd73773ad4ef7fcb25b7531

failures seem unrelated:
Test Result (2 failures / +2)
Swift(iphonesimulator-x86_64).SILOptimizer.global_init_with_empty.swift
Swift-validation(iphonesimulator-x86_64).stdlib.HashingRandomization.swift

@swift-ci please test macOS


Unit turns off due to call to set_temperature shortly after turning on

I'm trying to automate my units with:

Home Assistant
Schedy
This awesome integration

When Schedy activates a schedule, it turns on heating and sets the temperature in short succession.
Unfortunately the result is that devices switches on and immediately off again.
This is what is shown in the log:

(Supervisor because Schedy runs in AppDaemon add-on)
What's even worse is that Schedy sees this as a manual change by the user and will back off for a configurable amount of time.
The built-in resend mechanism in Schedy doesn't trigger.
I checked the service calls by schedy, via HA debug tools, and for the call to set_temperature there is no hvac_mode argument.
So it should leave it at 'Heating' as set by the previous service call.
Could it be this method is called with incorrect arguments?
https://github.com/jeatheak/Mitsubishi-WF-RAC-Integration/blob/07c0f9f0eb40c5453ced12e620b121bcd17b84f3/custom_components/mitsubishi_wf_rac/climate.py#L105

The history belows shows how Schedy 'fixes' the issue after one hour, because the back off timer expires.
Then it only set's the mode, without the temperature and it stays on:

Hi @RudolfRendier
I'm having a similar issue, but I'm unsure if it is related; what is the "PR 61" you found?

PR #61 is merged in v0.5.0


Copy Cargo.toml keywords

from https://github.com/japaric/heapless/blob/master/Cargo.toml: no-std, no-heap

Fixed in https://github.com/llogiq/compact_arena/pull/21


Update dd-trace-api, dd-trace-ot to 0.106.0

Updates

com.datadoghq:dd-trace-api
com.datadoghq:dd-trace-ot

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

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

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.datadoghq" }
}]

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

Superseded by #604.


Loading a model from a file for embeddings not found

Add an embeded text file to a modelfile with a model from a file may error:
creating parameter layer  Error: unexpected error opening model to generate embeddings: stat /Users/username/.ollama/models/manifests/registry.ollama.ai/library/latest: no such file or directory

I have the same problem running the ollama serve, and by default is not open to <IP_ADDRESS>, apparently only to localhost is accessible.

Add an embeded text file to a modelfile with a model from a file may error:
creating parameter layer  Error: unexpected error opening model to generate embeddings: stat /Users/username/.ollama/models/manifests/registry.ollama.ai/library/latest: no such file or directory

Can I ask you what you're trying to do ? You want to pass a text file to the model so that you can ask questions about that text file ?
If yes, can I ask you how you're doing it ?

I just relaunched the server and I'm able to send all request without issue at all


Job throws buffer.js TypeError

Whenever my job in particular is called, I receive this error.
SyncedCron: Exception "Example Job" TypeError: must start with number, buffer, array or string
    at fromObject (buffer.js:208:11)
    at Function.Buffer.from (buffer.js:85:10)
    at myJob(server/myJob.js:575:34)
    at Object.job (server/myJob.js:614:12)
    at packages\percolate_synced-cron.js:242:26
    at scheduleTimeout (packages\percolate_synced-cron.js:290:7)
    at packages\percolate_synced-cron.js:338:42
    at [object Object].EVp.withValue (packages\meteor.js:1134:15)
    at packages\meteor.js:452:25
    at runWithEnvironment (packages\meteor.js:1188:24)

I can't share my code specifically, but it's a pretty large function with a bunch of synchronous API requests, using Meteor's http package, that query an API for some stuff, and compare it to contents of my collections. Testing the function normally in a Meteor.method doesn't throw any errors, but it does take about half a minute to finish running. Am I tripping some sort of timeout, are the HTTP calls to blame, what's going on here?

@leosco not this package's fault. You, or a package you're using are doing something in 'myJob.js' at line 575 which throws an error. Use a try/catch block for that line, log the stuff you're passing to whatever's going on at line 575 whenever an exception is found. According to the Buffer.from docs the type of the argument must be one of the ones your error says.
My guess is you're doing something like
const dbDoc = dbCollection.findOne(`query`);
functionWhichUsesBuffer(dbDoc);

And the function which uses buffers is not checking that dbDoc is defined and an accepted type.
You should start using check. It's neat and spares you lots of frustration.

No, you're totally right. I had missed a catch in case the buffer was empty. Sorry for wasting your time 😢

@leosco , no worries! Happy coding!


[FLINK-31700][Connectors/MongoDB] Fix MongoDB nightly CI failure

Fix MongoDB nightly CI failure
https://issues.apache.org/jira/browse/FLINK-31700

Error:  /home/runner/work/flink-connector-mongodb/flink-connector-mongodb/flink-connector-mongodb/src/main/java/org/apache/flink/connector/mongodb/sink/writer/context/DefaultMongoSinkContext.java:[33,8] org.apache.flink.connector.mongodb.sink.writer.context.DefaultMongoSinkContext is not abstract and does not override abstract method getAttemptNumber() in org.apache.flink.api.connector.sink2.Sink.InitContext

This PR made some changes to make flink-mongodb-connector compatible with both 1.16 and 1.17.
We can build it of flink 1.17 via maven command as follows:
mvn clean verify -U -B --no-transfer-progress -Dflink.version=1.17.0 \
    -DaltDeploymentRepository=validation_repository::default::file:/tmp/flink-validation-deployment \
    -Dscala-2.12 -Darchunit.version=1.0.0

The version of archunit.version seems not compatible from flink 1.16(0.22.0) to flink 1.17(1.0.0).
https://newreleases.io/project/github/TNG/ArchUnit/release/v0.23.0
We need to manually specify the version and introduce additional configuration archRule.failOnEmptyShould=false to keep compatibility.

Hi @dannycranmer.
Could you help review it when you have time?


Plugin not working

just running any sample file, it shows the error message in the prints
i tried to solve this bugs for myself, but nothing i did made it worked.
i also didn't understood how to build the meshes, but congratulations for the plugin idea, it's exactly what i always wanted in godot!

Same issue here, samples open ok but moving the points throws
E 0:00:09:0145   parse_message: Condition "p_args.size() < 10" is true. Returning: ERR_INVALID_DATA
<C++ Source>   scene/debugger/scene_debugger.cpp:174 @ parse_message()
and
E 0:00:09:0145   debug: Condition "_try_capture(command, data, captured) != OK" is true. Continuing.
<C++ Source>   core/debugger/remote_debugger.cpp:475 @ debug()

Thanks for bringing this to my attention, I'm due to take a look soon. Could either of you let me know which version of Godot you experienced the issue in?

Mine was 4.0.2, the latest stable in that time

Hi, I've merged in some fixes and updated the Godot asset library link, I'm awaiting approval. In the mean time, if you need to get through the problem now, I'd recommend cloning the project from Github manually and copying the asset in.


Dynamic Variables

Hi,
I have a problem with dynamic variables like:
{{$randomFirtsName}}, {{$randomLatitude}}, {{$randomLongitude}}, {{$randomColor}}, {{$randomCity}}, {{$randomCountry}}, {{$randomAdjective}}, {{$randomFileName}}
After conversion, they are not recognized and cause an error when performing the load test.

For those experiencing the same issue, I have provided a forked NPM package version that contains this fix:
Replace in your packages.json:
"dependencies": {
    "postman-to-k6": "^1.5.0"
  }

to
"dependencies": {
    "@apideck/postman-to-k6": "^1.6.0"
  }

This is a forked NPM package, to provide this new feature. The current maintainers are quite overloaded by the acquisition of K6 by Grafana. By forking the repo & package, users can keep using new PR's & (security) fixes. At a later stage the changes can potentially be merged back in the original postman-to-k6 repo.
The changelog contains the differences between the original package & the forked version.

Apologies for the inactivity on this project.
Due to the inability to properly support this project, the k6 team has decided not to continue its development. The primary reason is that k6 scope has grown significantly with the launch of k6 extensions. The team is prioritizing extending k6 capabilities over the converters.
We suggest you post your issue at api-deck/postman-to-k6. The project is active and maintained on this fork.


Can't expand reference in Find All References

Sorry for the large steps...

cd vscode
git remote add joao https://github.com/joaomoreno/vscode.git
git fetch joao
git checkout ae0f60e9469942097769b25e21ad597ba2db85aa
Open that folder in Code
Open src/vs/platform/telemetry/common/telemetry.ts
Find All References of isOptedIn in line 36.

Can't reproduce after updating to last night's build...

This can happen like so: (1) provider reports n references (potentially in non existent files), (2) we render all references (unchecked due to cost issues), and (3) load the file when you expand the tree note. If then loading of the file fails we show that behaviour. I have favoured that over big in your face error messages.
Pretty sure you had a stale tsserver leading to this


Get Authentificate User Data

Hi
I'm using your package for accessing an api. It works perfectly but I've got the problem that once the user is logged or accessed correctly, I cannot get User information such as id or user name.
neither Auth::user() (returns null), $request->user() or whatever works correctly. At the controller I.e. the Auth::check returns false.
How can I get information about user logged??
thanks

@luisferparra , can you confirm that your issue is with this library specifically?


:sparkles: Add the AppliedManifestWork update reconciliation

Summary
This PR will add reconciliation on the update events for the AppliedManifestWork objects.
The reconcile loop will find and process changes to the list of resources in the AppliedManifestWork object.
The WorkStatus object on the hub will be deleted when the workload object on the WEC is deleted.
Related issue(s)
Fixes # https://github.com/kubestellar/kubestellar/issues/1724

/cc @ezrasilvera
/cc @pdettori
/cc @nirrozenbaum
/cc @vMaroon

Overall looks fine, one question I have is if this is a breaking change - does this change still allow to run scenario 4 without other changes ?

This is not a breaking change and should work the same way as it is today for a single wrapped object. Scenario 4 was also tested locally with the changes in this PR.

@andreyod Did you build a new image and test KS singleton test with it?

@andreyod couple of Q:

I don't see an informer on the status objects, how do you cope with changes/deletions of those objects ?  (or this is next PR )?
How do you handle restart of the status Addon - do you go over all objects and then delete all WorkStatus that don't have corresponding objects on the WEC ?

@andreyod Did you build a new image and test KS singleton test with it?

I build a local image, change the helm chart to use this image, and test KS singleton.

@andreyod couple of Q:

I don't see an informer on the status objects, how do you cope with changes/deletions of those objects ?  (or this is next PR )?

Those are valid points that need to be discussed and addressed. I think it's out of scope for this PR.

Correct, there is no informer on WorkStatuses in the addon agent on the WEC. The WorkStatus objects reside in the transport space and we need to find the way to handle manual changes/deletions of those objects. This can be done in the addon agent or in the KubeStellar status controller.
I guess you refer to a case in which workload objects where deleted while the status addon agent was down. In this case WorkStatus objects will be deleted by garbage collector only when the Manifast is deleted.

Is there a documented or automated procedure for testing a new version of this repo (not using container images or helm charts produced from earlier versions)?

@MikeSpreitzer you can build a local image and install the local helm chart that uses that image:
https://github.com/kubestellar/ocm-status-addon/blob/main/docs/contributors.md#making-local-build
https://github.com/kubestellar/ocm-status-addon/blob/main/docs/contributors.md#install-chart-from-local-repo-using-local-build