ScrollController not triggered on content change (scroll to top)

When the child content is changed (from large content to small) and the content is not scrollable anymore (content too small), the view scrolls to the top. This event is not triggered at all. Neither with controller.addListener(...) function nor with a NotificationListener<ScrollNotification> widget.
We're not notified if the view scrolls up due to content change.
CustomScrollView(
    controller: controller,
    slivers: <Widget>[
        ...
    ],
);

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.6, on macOS 11.2.1 20D74 darwin-x64, locale de-CH)
    • Flutter version 1.22.6 at /Users/xx/Development/Tools/flutter
    • Framework revision 9b2d32b605 (7 weeks ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/xx/Library/Android/sdk
    • Platform android-29, build-tools 28.0.3
    • ANDROID_HOME = /Users/xx/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/xx/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (1 available)
    • Philippe (mobile) • 00008030-0015295011F0802E • ios • iOS 14.4

• No issues found!

Hi @YAPPDEF
Can you please upgrade to the latest  stable and try again
flutter channel stable
flutter upgrade --force
flutter doctor -v
If the problem persists, Can you please provide your flutter doctor -v with the upgrade and  logs, and a minimal complete reproducible code sample?
Thank you

Hey @TahaTesser
Thanks for your fast response. I cannot switch as I'm preparing a release. Tried it on a collegues mac with newest flutter version - same same.

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.1, on macOS 11.0.1 20B29 darwin-arm, locale de-CH)
    • Flutter version 2.0.1 at /Users/xx/Development/flutter
    • Framework revision c5a4b4029c (7 days ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/xx/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/xx/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/xx/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (2 available)
    • bunteblume00 (mobile) • 00008101-000E75180801401E • ios            • iOS 14.4
    • Chrome (web)          • chrome                    • web-javascript • Google Chrome 88.0.4324.192

• No issues found!

minimal complete reproducible code sample
import 'package:flutter/material.dart';

class TestWidget extends StatefulWidget {
  @override
  _TestWidgetState createState() => _TestWidgetState();
}

class _TestWidgetState extends State<TestWidget> {
  final _controller = ScrollController();

  @override
  void initState() {
    super.initState();
    _controller.addListener(onScroll);
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: CustomScrollView(
        controller: _controller,
        slivers: <Widget>[
          SliverAppBar(expandedHeight: 200),
          SliverList(
            delegate: SliverChildListDelegate(
              [_TestChild()],
            ),
          ),
        ],
      ),
    );
  }

  void onScroll() => print('onScroll');

  @override
  void dispose() {
    _controller?.removeListener(onScroll);
    super.dispose();
  }
}

class _TestChild extends StatefulWidget {
  @override
  _TestChildState createState() => _TestChildState();
}

class _TestChildState extends State<_TestChild> {
  bool _extended = true;

  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      onTap: () => setState(() => _extended = !_extended),
      child: _extended
          ? Container(
              color: Colors.red,
              height: 1000,
            )
          : Container(
              color: Colors.yellow,
              height: 100,
            ),
    );
  }
}

To reproduce, scroll down on the expanded view (red): onScroll is logged as the view is scrolling.
When you reach the bottom, switch the view (click in red area). View scrolls then to the top, but no scroll log..

Hi @YAPPDEF
Thanks for a reproducible code sample, I can see the on the master channel as well.

code sample
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: TestWidget(),
    );
  }
}

class TestWidget extends StatefulWidget {
  @override
  _TestWidgetState createState() => _TestWidgetState();
}

class _TestWidgetState extends State<TestWidget> {
  final _controller = ScrollController();

  @override
  void initState() {
    super.initState();
    _controller.addListener(onScroll);
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: CustomScrollView(
        controller: _controller,
        slivers: <Widget>[
          SliverAppBar(expandedHeight: 200),
          SliverList(
            delegate: SliverChildListDelegate(
              [_TestChild()],
            ),
          ),
        ],
      ),
    );
  }

  void onScroll() => print('onScroll');

  @override
  void dispose() {
    _controller?.removeListener(onScroll);
    super.dispose();
  }
}

class _TestChild extends StatefulWidget {
  @override
  _TestChildState createState() => _TestChildState();
}

class _TestChildState extends State<_TestChild> {
  bool _expanded = true;

  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      onTap: () => setState(() => _expanded = !_expanded),
      child: _expanded
          ? Container(
              color: Colors.red,
              height: 1000,
            )
          : Container(
              color: Colors.yellow,
              height: 100,
            ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.1, on macOS 11.2.3 20D91 darwin-x64, locale
    en-GB)
    • Flutter version 2.0.1 at /Users/tahatesser/Code/flutter_stable
    • Framework revision c5a4b4029c (8 days ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 10 (API 29)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 11.2.3 20D91
      darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome
      89.0.4389.82

• No issues found!

[✓] Flutter (Channel master, 2.1.0-11.0.pre.229, on macOS 11.2.3 20D91
    darwin-x64, locale en-GB)
    • Flutter version 2.1.0-11.0.pre.229 at
      /Users/tahatesser/Code/flutter_master
    • Framework revision d20ec4c7d8 (2 hours ago), 2021-03-11 22:18:51 -0800
    • Engine revision 1b68503bc8
    • Dart version 2.13.0 (build 2.13.0-125.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 10 (API 29)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 11.2.3 20D91
      darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome
      89.0.4389.82

• No issues found!

Thank you

/cc @Piinks

Hi @pdefuns I think this is working as intended. When the size of the scroll content changes, it does not actually scroll, it just resizes.
You can listen for this change by using a ScrollMetricsNotification. See below:
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: TestWidget(),
    );
  }
}

class TestWidget extends StatefulWidget {
  @override
  _TestWidgetState createState() => _TestWidgetState();
}

class _TestWidgetState extends State<TestWidget> {
  final _controller = ScrollController();

  @override
  void initState() {
    super.initState();
    _controller.addListener(onScroll);
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: NotificationListener<ScrollNotification>(
        onNotification: scrollNotification,
        child: NotificationListener<ScrollMetricsNotification>(
          onNotification: metricsNotification,
          child: CustomScrollView(
          controller: _controller,
          slivers: <Widget>[
            SliverAppBar(expandedHeight: 200),
            SliverList(
              delegate: SliverChildListDelegate(
                [_TestChild()],
              ),
            ),
          ],
              ),
        ),),
    );
  }

  void onScroll() => print('onScroll');

  bool scrollNotification(_) {
    print('scroll notification');
    return false;
  }

  bool metricsNotification(_) {
    print('metrics notification');
    return false;
  }

  @override
  void dispose() {
    _controller.removeListener(onScroll);
    super.dispose();
  }
}

class _TestChild extends StatefulWidget {
  @override
  _TestChildState createState() => _TestChildState();
}

class _TestChildState extends State<_TestChild> {
  bool _expanded = true;

  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      onTap: () => setState(() => _expanded = !_expanded),
      child: _expanded
          ? Container(
              color: Colors.red,
              height: 1000,
            )
          : Container(
              color: Colors.yellow,
              height: 100,
            ),
    );
  }
}


Suppress transpile-on-save success messages

Love the Atom plugin. But I get success on Save so often when I'm really only interested in transpile failures. I'd love to see a preference for suppressing notification for successes.

Yes I agree. Will put in next release.

Just pushed out 0.7.0. Adds a new flag to optionally suppress success messages.

Sweet! Thanks.


[WIP] Add E2E testing for Rollover ES feature

Which problem is this PR solving?
Add E2E testing to the Rollover ES feature implemented in #267
Short description of the changes

Added smoke E2E Test for the Rollover ES feature
Added E2E Test to check everything works properly when the feature is enabled after the deployment
Added E2E Test to check the correct indices are created when the feature is enabled before the deployment
Extract some common logic from the ES index test suite to reuse some logic in the Rollover ES test

Todo:

Add testing for the readTTL parameter

@iblancasa you have to sign off your commit. See the details of DCO

@jpkrohling, ok I'll rewrite the tests using kuttl


libsodium-dev libpcap-dev

I would like to request libsodium-dev libpcap-dev to be included.

PRs welcome.

I really dont know enough about yocto to help. I tried building it a year ago and I remember it not working. Would be really nice if the docker file worked.


Typo for "set stop healthservice"

It should be "net stop healthservice".

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

ID: 44e919d1-bf8c-38af-29a5-63a2e679850d
Version Independent ID: 03a4bc5a-6666-bc24-f315-4324278e50ae
Content: Troubleshooting issues with Azure Change Tracking
Content Source: articles/automation/troubleshoot/change-tracking.md
Service: automation
Sub-service: change-inventory-management
GitHub Login: @bobbytreed
Microsoft Alias: robreed

@henrywang08 Thanks for the comment. We are actively investigating and  working on it. We will get back to you shortly with an update.

@henrywang08 This is just to confirm you that the required document changes are done.


test: enable hubble tls in ci

Enable TLS for Hubble and verify that the expected certificate is served.

test-me-please

test-me-please


Not listening

When I run app.py on my laptop. it does all of the startup but after listeining it doesnt listen at all and doesnt come up with any error.

When I run app.py on my laptop. it does all of the startup but after listeining it doesnt listen at all and doesnt come up with any error.

Yesterday I released a new version of this repo with a fix for this on Windows. Might be worth downloading it again!
Let me know how you get on 😎

Ok I will redownload it and see what happens. Thanks for thr suggestion

How far did you get with this?

Same problem. It wont listen but i think i know why because i havent set up an voice for it yet so ill do that when i next open the folder tommorow

Hi there, just checking in on this? If it's resolved could you be awesome and mark this issue as resolved?

Hi there, just checking in on this? If it's resolved could you be awesome and mark this issue as resolved?

I havent had chance to look yet but ill mark this as closed an if i have the problem ill reopen it again


Output of skymap should be a numpy array

Refactor skymap so that the output is a numpy array that can be used a result in its own right, as well as sent for plotting. For an example of desired functionality, please consult https://github.com/mjuric/incubator/blob/master/lsdb/lsdb-dataframes.ipynb (e.g., cell 13).

It might be better to have skymap.histogram or skymap_histogram being a numpy array containing the ``histogram;; and keep the skymap (being a map) a figure.

closed in #245


multiple plugins are not correctly handled

https://github.com/entelecheia/hyfi/blob/f3774ab27e10419221f0f23fe804d542c37c1ae4/src/hyfi/composer/composer.py#L147

fixed.


Set size of hits to 0 for tag cloud query

Note: Targeted for develop for now, but we could aim for a hotfix (upcoming one or the next) if preferred.
Description
Set a size parameter to 0 in the request to get ES tagcloud aggregations. It's confusing, but the number of aggregations returned is based on a different size parameter. And the UI only looks at the aggregations portion of the response.
Recommendation from here: https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations.html#agg-caches
Locally, with a prod snapshot, it makes a huge difference -- the ES response has a took property that indicates how long it takes for ES to process the request, and it went from about 1 second to 1 millisecond (yes, 1,000ms vs. 1ms).
The response is with the value set to 20:
"hits": {
    "total": {
      "value": 1832,
      "relation": "eq"
    },
    "max_score": 1,
    "hits": [
      {
        "_index": "workflows",
        "_type": "_doc",
        "_id": "99",
        "_score": 1
      },
... (and 19 more)

With it set to 0:
...
 "hits": {
    "total": {
      "value": 1832,
      "relation": "eq"
    },
    "max_score": null,
    "hits": [] // Empty!
  },
...

I verified that the aggregations part of the response, what the UI uses, is identical for both requests.
Review Instructions
Follow the instructions in dockstore/dockstore#5220. You should not see a turtle next to the request getting the aggregations.
Issue
dockstore/dockstore#5220
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that your code compiles by running npm run build
[x] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[x] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[x] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[x] Do not use cookies, although this may change in the future
[x] Run npm audit and ensure you are not introducing new vulnerabilities
[x] Do due diligence on new 3rd party libraries, checking for CVEs
[x] Don't allow user-uploaded images to be served from the Dockstore domain
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

Found the refernced ES documentation via this article (see "Set size parameter to 0")

This will help a lot, yay!


Responsively hiding prefix icons in forms is buggy

Applying any of the .hide-* classes to .prefix in an input field are resulting in buggy UI. Properties margin-left and width:calc(100% - 3rem) of the inputs and/or labels leave empty space in the place of icon.

Can you post a simple codepen to demonstrate the issue? it would help us identify the problems more quickly.

@acburst This should help explain the problem: http://codepen.io/sushruth/pen/NqMVXE . Try resizing the window.

@media only screen and (max-width: 992px){
  .input-field .prefix.hide-on-med-and-down ~ label,
  .input-field .prefix.hide-on-med-and-down ~ input {
    margin-left:0;
  }
  .input-field .prefix.hide-on-med-and-down ~ input{
    width: 100%;
  }
}

Closed due inactivity. Feel free to reopen, if it's still necessary.


saves only the names of the files. Adds command to create migration

[x] Add command to create migration of the correct format
[x] Add option to --showstats
[x] Saves migration runs in the database and will only run what's not present. Will error out on wrong order / error in migration

@matzeso true, i had a longer discussion with nikita yesterday about migrations and I think what you laid down here is what confused me: Whats the purpose of the rewrite if they still have to run in order. So either we allow migrations being run out of order or we go with the proposal below:
Proposal:

add a workflow that fails, if the master branch has newer migrations then the one in the branch
add a fireway command that will update a migration to a newer date (fireway update <file>) so it becomes easy to fix

Or both

What are we going to do about this now? Do we want to enforce orderly migrations? If yes, do we already have a decision regarding Ulimas Proposals? Is it going to be part of this ticket / PR?
BTW. I can't tag anyone apart from you since they are not reviewer/part of the repo.


git-credentials secret is not properly loading

What happened?
Following these steps https://github.com/crossplane-contrib/provider-terraform#private-git-repository-support
I was unable to get crosspane-terraform to pull from a private repo.
I was able to get it working by creating a .git-credentials file inside the container with kubectl ssh -u root, so it seems to stem from the file not getting mounted into the pod properly.
The error message I get is
could not read Username for 'https://github.com': No such device or address\n"}

How can we reproduce it?
My configurations are as follows
apiVersion: pkg.crossplane.io/v1alpha1
kind: ControllerConfig
metadata:
  name: terraform-config
  labels:
    app: crossplane-provider-terraform
  annotations:
    eks.amazonaws.com/role-arn: arn:aws:iam::OMITTED:role/crossplane
spec:
  image: crossplane/provider-terraform-controller:v0.4.0-rc.0.2.gf3bd7e0
  args:
    - "--debug"
    - "--poll=5m"
  podSecurityContext:
    fsGroup: 2000
---
apiVersion: pkg.crossplane.io/v1
kind: Provider
metadata:
  name: crossplane-provider-terraform
spec:
  package: crossplane/provider-terraform:v0.4.0
  controllerConfigRef:
    name: terraform-config
---
apiVersion: tf.crossplane.io/v1alpha1
kind: ProviderConfig
metadata:
  annotations: {}
  name: aws-irsa
spec:
  credentials:
    # Terraform natively supports IRSA auth so we can use None here
    - filename: aws.json
      source: None
 ---
apiVersion: tf.crossplane.io/v1alpha1
kind: ProviderConfig
metadata:
  name: default
spec:
  credentials:
  - filename: .git-credentials
    source: Secret
    secretRef:
      namespace: crossplane-system
      name: git-credentials
      key: .git-credentials

Secret
k get secret git-credentials -o yaml
apiVersion: v1
data:
  .git-credentials: OMITTED
kind: Secret
metadata:
  name: git-credentials
  namespace: crossplane-system
type: Opaque

What environment did it happen in?
Crossplane version:
package: crossplane/provider-terraform:v0.4.0
image: crossplane/provider-terraform-controller:v0.4.0-rc.0.2.gf3bd7e0
Same issues with version 0.3.0

Hi @brett-au , thanks a lot for report.
I've double-checked on my side, the .git-credentials are getting propagated properly from a Secret
/tmp/tf/cd081adf-d90f-4dac-b4c4-2ed54643ba26/.git-credentials
/ $ cat /tmp/tf/cd081adf-d90f-4dac-b4c4-2ed54643ba26/.git-credentials
<EMAIL_ADDRESS>
Please note that we are mounting them under /tmp/tf/$uuid per each Workspace MR and not globally within the container.
I've noticed you are using two ProviderConfigs in your configuration, are you sure you are using default one for the propagation test?
apiVersion: tf.crossplane.io/v1alpha1
kind: ProviderConfig
metadata:
  annotations: {}
  name: aws-irsa
spec:
  credentials:
    # Terraform natively supports IRSA auth so we can use None here
    - filename: aws.json
      source: None
 ---
apiVersion: tf.crossplane.io/v1alpha1
kind: ProviderConfig
metadata:
  name: default
spec:
  credentials:
  - filename: .git-credentials
    source: Secret
    secretRef:
      namespace: crossplane-system
      name: git-credentials
      key: .git-credentials

Hi @ytsarev , Is there is a way to integrate the git credentials via a ssh key

@balu-ce not currently, but there is PR https://github.com/crossplane-contrib/provider-terraform/pull/77 which addresses this functionality. Unfortunately, PR requires some substantial rework/update

So I finally got around to testing the suggested fix, and I coupled my 2 ProviderConfigs into 1, this worked perfectly... but a new issue appeared.  Now it gives me a 404 under kubectl describe workspace  vs an Auth issue.

I have confirmed this URL is valid
The kicker is, if I exec into the terraform provider pod, I can terraform init and plan successfully.
/ $ cd tf/f370dafb-fd78-4282-b24f-468cb479a35f/
/tf/f370dafb-fd78-4282-b24f-468cb479a35f $ terraform init

Initializing the backend...

Initializing provider plugins...
- Reusing previous version of hashicorp/aws from the dependency lock file
- Using previously-installed hashicorp/aws v4.51.0

Terraform has been successfully initialized!

You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.

If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.```

Hi,
I have the same problem. It seems that the terraform provider doesn't pick the credential file.
apiVersion:  tf.upbound.io/v1beta1
kind: ProviderConfig
metadata:
  name: tf-provider-config
spec:
  credentials:
    - filename: credentials
      source: Secret
      secretRef:
        namespace: crossplane-system
        name: aws-secret
        key: creds
    - filename: .git-credentials
      source: Secret
      secretRef:
        namespace: crossplane-system
        name: git-credentials
        key: .git-credentials
  configuration: |
      provider "aws" {
        shared_credentials_files = ["credentials"]
        profile = "default"
      }

      terraform {
        backend "kubernetes" {
          secret_suffix     = "state"
          namespace         = "crossplane-system"
          in_cluster_config = true
        }

$ kubectl exec -ti -n crossplane-system provider-terraform-c07358410444-6854b8df6f-nq4b6 -- cat /tmp/tf/cb7865e9-a58e-4036-a8e5-ccad1df5f228/.git-credentials
<EMAIL_ADDRESS>
apiVersion: tf.upbound.io/v1beta1
kind: Workspace
metadata:
  name: tf-remote-test
spec:
  forProvider:
    source: Remote
    module: https://gitlab.com/<redacted>/tf-basic-module.git
  # All Terraform outputs are written to the connection secret.
  writeConnectionSecretToRef:
    namespace: default
    name: terraform-workspace-remote-test
  providerConfigRef:
    name: tf-provider-config

But I got
$ kubectl describe workspaces
Warning  CannotConnectToProvider  9s (x5 over 24s)  managed/workspace.tf.upbound.io  cannot get remote Terraform module: error downloading 'https://gitlab.com/<redacted>/tf-basic-module.git': bad response code: 403

The credential is good. Both the username and the token are valid and could be used to pull the repo manually.
But the provider couldn't use the credential file it seems.

Ok I solved the issue by changing the source to: git::?ref=master
In the second example in the doc there is this prefix. :D
In the first one there no "git::" prefix, I should have read through al the docs :D

@milkfinch could you point to the problematic place in the doc? The PR with the fix would be the most amazing :)

On the page https://github.com/upbound/provider-terraform in the example after 'modul:' the URI doesn't have the 'git::' prefix. But without that my setting didn't work.
In the git examples folder the workspace-remote.ysml has the right URI.


Update all versions to 23.10

Looks like when branch-23.10/v23.10.00a was created, update-version.sh was not in the repo, so no versions were updated.
This PR just runs update-version.sh for 23.10.00.

/merge


Add warn macro to the Jinja function reference docs

Logging here since we're in the middle of a migration 🤐

This is part of the exceptions docs


Fix: Resolved startup issue for Example site on Windows environment

Updated the build and watch npm scripts in the package.json of the react-resizable-panels-website package to make it work in a Windows environment. These changes were based on the package.json from the react-resizable-panels package.
It would be great if it also works on other environments like Mac.

It would be great if it also works on other environments like Mac.

Since I'm a Mac user, and the main developer here, that's definitely a requirement 😄 but this change is fine.


node down scenario: pod stuck at ContainerCreating state

related to #635
Steps to reproduce:

Create a Deployment with one pod attached to longhorn volume mount to /data

apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: longhorn-volv-pvc
spec:
  accessModes:
    - ReadWriteOnce
  storageClassName: longhorn
  resources:
    requests:
      storage: 2Gi
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: volume-test
  labels:
    name: volume-test
spec:
  replicas: 1
  selector:
    matchLabels:
      name: volume-test
  template:
    metadata:
      labels:
        name: volume-test
    spec:
      containers:
      - name: volume-test
        image: nginx:stable-alpine
        volumeMounts:
        - name: volv
          mountPath: /data
      tolerations:
      - key: "node.kubernetes.io/unreachable"
        operator: "Exists"
        effect: "NoExecute"
        tolerationSeconds: 20
      volumes:
      - name: volv
        persistentVolumeClaim:
          claimName: longhorn-volv-pvc

Shutdown the node which the pod scheduled to.

Monitor the pods, as soon node is detected to be down, the old pod will be in terminating state.

Force delete the terminating pod.
kubectl delete pod TERMINATING_POD_NAME --force --grace-period=0

New pod will be stuck in ContainerCreating state

Inspect stuck pod using kubectl describe pod STUCK_POD_NAME
you should see the following error:

  Warning  FailedAttachVolume  39s   attachdetach-controller  Multi-Attach error for volume "pvc-9b68216f-fcd9-11e9-8054-26db997fc043" Volume is already used by pod(s) TERMINATED_POD_NAME

Due to #820 , now we need to wait longer for the volume to be detached by Kubernetes. Did the volume ultimately detach?

No, the volume still in Attached state
Also, In volume details page, Pod Name field value is the name of the stuck pod.

@meldafrawi Just merged a fix.

@yasker will check it again.

Fixed.
Volume now get detached after pod deleted.


Add "Clear Recycle Bin" to the context menu when the Recycle Bin is opened

@R3voA3 It's already there as 'Empty recycle bin' when the recycle bin has deleted items

Strange. It didn't show up for me, otherwise I would not have reported it.

@R3voA3 Which version of Files are you using?

I restarted the app and now it's showing fine. I think this can be closed for now.


Contrast of comments in the current line is low

The code comment and git blame annotation from GitLens is barely visible if the carret is at that line.
I think it should be possible to define a seperate background-color or color for the line where the carret curently is.

Yh, I had that issue, I might make it the very dark colour used in the rest of the UI, I will change it to the colour is #14171a. I forgot the exact setting in the file, thanks for these, gonna work on these the next day :P

more dark => more nice

#AllTheDark

Much better right?

Looks good to me, closing

Yep, agree


Unable to enabled uld2raw

Hello,
Good anyone help on the openwrt configuration please..
But I am trying to running the Udp2 raw but I am feeling at. Kindly help...

since you didn't specify any details, just read this article
https://iopq.wordpress.com/2024/11/27/troubleshooting-udp2raw-wireguard/


Add support for Android SafetyNet attestation

See https://www.w3.org/TR/webauthn/#android-safetynet-attestation

Fixed, support is available.


Don't create IInterface classes

In the Dart Language Tour there is a section on Implicit Interfaces that talks about how every class in Dart has an implicit interface. This is why Dart and Flutter do not have IUnknown and friends. Please delete IUrlOpen as it is strongly non-idiomatic.

Issue fixed in remove IUrlOpen, Thanks for the reporting,


Document how to see validation issues while in development

Is there already a solution to this?

Sorry, I mean the server validation of the Json models the endpoint is receiving.
If I send a payload which does not correspond to the Swagger model, I would like to know what's wrong.
May I lack some insight on how to retrieve that from the server implementation (using akka-http) but wondering if something implementation agnostic exists.

Ah, this currently doesn't exist; I'm thinking it may be how we're doing the content validation exception, failing over to other available unmarshallers.
What kind of message would you expect to see? I'm assuming a Circe path would not be the best user experience; possibly a JSON path?

@blast-hardcheese anything that actually provide an hint and supports custom validation messages.
I'm not sure about what looks like a circe-path or JSON path yet.. (I've left Scala since 2012 and am just back to it, landing in the ecosystem - looking at it ATM)

@blast-hardcheese ok, I understand the problem now with circe-path.
I'm not sure but I guess the JSON Path would be extracted from the circe-path so this means we can have some way to handle directly a circe-path and possibly use a transformer to get a JSON path.
Exposing the circe DecodingFailure would be a good and the minimal, necessary first step.


Update ASU header with CLS -> College of Integrative Sciences and Arts

College of Letters and Sciences is now the College of Integrative Sciences and Arts.
Old: https://cls.asu.edu
New: https://cisa.asu.edu

College of Letters and Sciences is not in the header and so no change is necessary.


RUMM-2087 upgrade okhttp

What does this PR do?
Upgrade the OkHttp dependency to use the last available API

Codecov Report

Merging #75 (9ab4715) into develop (ff6d99b) will not change coverage.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             develop      #75   +/-   ##
==========================================
  Coverage      90.18%   90.18%
  Complexity       134      134
==========================================
  Files             20       20
  Lines            397      397
  Branches          45       45
==========================================
  Hits             358      358
  Misses            24       24
  Partials          15       15

Impacted Files
Coverage Δ

...m/datadog/gradle/plugin/internal/OkHttpUploader.kt
87.72% <80.00%> (ø)


bug: div toolbar-title created by shadow root overwrites header z-index, causing graphical error

Bug Report
Ionic version:
[x] 5.22.0
Current behavior:
When using an ion-toolbar within an ion-header, using an ion-title sets the ion-toolbar's title. This causes the shadow root to apply a  element. This element inherits the base z-index of 10, and is not overwritten by changing the z-index of the header, toolbar, or title. This leads to a strange graphical glitch where part of the header looks like it is at a different z-index from the rest.
Expected behavior:
The z-index should match the z-index of the parent elements.
Steps to reproduce and related code:
Using the following html:
<ion-header>
  <ion-toolbar>
    <ion-title>
      Test
    </ion-title>
  </ion-toolbar>
</ion-header>

and the following css class:
.z-change {
  z-index: 0;
}

Applying the "z-change" class to the ion-header, ion-toolbar, ion-title, or any combination of the above results in the aforementioned graphical issue, where part of the header is z-index 0 and part is z-index 10.
Other information:
Ionic info:
Ionic:

   Ionic CLI                     : 5.4.16 (C:\Users\Austin\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework               : @ionic/angular 4.11.10
   @angular-devkit/build-angular : 0.803.22
   @angular-devkit/schematics    : 8.1.3
   @angular/cli                  : 8.1.3
   @ionic/angular-toolkit        : 2.2.0

Cordova:

   Cordova CLI       : 9.0.0<EMAIL_ADDRESS>   Cordova Platforms : none
   Cordova Plugins   : no whitelisted plugins (0 plugins total)

Utility:

   cordova-res : not installed
   native-run  : not installed

System:

   NodeJS : v10.16.0 (C:\Program Files\nodejs\node.exe)
   npm    : 6.9.0
   OS     : Windows 10

Thanks for the issue. Can you provide a repo with the code required to reproduce this issue?

From my understanding the framework generates Z indexes on its own. Are you using Ion header/footer in your app component sandwitching an ion-router-outlet ?

@liamdebeasi I've uploaded a version of the repository I was using, slimmed down as much as is reasonable to only have the necessary parts to demonstrate the issue.
https://github.com/atlowell-smpl/Ionic-Angular-Z-Index-Header-Toolbar-Demonstration
Interestingly, when removing some of the styling, it seemed to change the graphical glitch I was seeing. After some more testing, I realized that it had to do with how the margins of the ion-split-pane I was using below the header were being set.  Essentially, if the split pane contents were set to overlap with the header, the z-index difference disappeared, while as if the margins were set to push them down below the header, the z-index difference existed.
This is what I was seeing in my repository:

Using the following code:
<ion-header class="z-page">
  <ion-toolbar>
    <ion-title>
      bevuu.app
    </ion-title>
  </ion-toolbar>
</ion-header>

.z-page {
    z-index: 0 !important;
}

ion-toolbar {
    margin-top: 20px;
}

ion-title {
    margin-left: 3.5vw;
    font-family: 'Source Sans Pro';
    font-weight: bold;
    font-size: 46px;
    letter-spacing: 1.1px;
    line-height: 58px;
}

span.title-secondary {
    //color: #ffa737;
    color: var(--ion-color-secondary);
}

And this is what I see in the test repository I linked:

As you can see, the main difference is how the margins are set. In my repository, an additional margin is placed at the top, causing the contents of the split pane to overlap with the contents of the header on the menu, causing that strange graphical glitch. Interestingly enough, the right side of the split pane doesn't have that issue. I find it likely that somewhere in my code an additional margin is set to cause this behavior.
Either way, I would still consider this a bug. As you can see, the shadow dom is still applying divs to the header with z-indexes of 10. This happens even if you extend the ".z-page" class to the ion-toolbar and ion-title.

@NikolaPeevski not quite. I'm using a layout that looks similar to the following:
<ion-header>
  <ion-toolbar>
    <ion-title>Hello</ion-title>
  </ion-toolbar>
</ion-header>
<ion-split-pane contentId="main-content">
  <ion-menu contentId="main-content" type="overlay">
    <ion-content>
      Menu
    </ion-content>
  </ion-menu>
  <ion-router-outlet id="main-content"></ion-router-outlet>
</ion-split-pane>

The full details can be seen in the repository I linked.  I am using an ion-router, but only as part of an ion-split-pane. And I do not use an ion-footer.

Thanks for the follow up. It looks like there is margin: 8px being applied to the body element via a user agent style sheet.
Add the following to global.scss:
body {
  margin: 0px;
}

Does that adding that CSS resolve the issue?

Adding that seems to have no effect.

Are you adding this to global.scss in an Ionic application? I noticed that the repo you provided does not have that file added.

Yes, I added it to the global.scss file in the project that I first noticed the issue on (screenshotted above).

Can you update the provided repo with the global.scss file? When I add it myself the issue is resolved.

Added to the repository. It does not fix the issue on my end.  My thinking is that any margin-related solution would be similar to the bandaid "fix" I mentioned above, in that it will hide the problem in some cases but will not actually address the root issue, which is that the generated shadow root components have a different z-index.

Thanks for the follow up. Could you clarify what the issue you are running into is? Perhaps I misunderstood the original post. The application seems to be displaying fine for me.

The ion-header in the example repository has a "z-page" class applied to it, which sets the "z-index" of the header to 0. Since 0 is also the z-index of the page, both components should be at the same level. If you inspect the ion-header in a developer console, you can see that the header itself has a z-index of 0, but that the generated shadow root components have z-indexes of 10 (what the header defaults to). As a result, you end up with the "height difference" separation bar below the header that you can see in that screenshot, despite the fact that the header is set to be at the same z-index of the page.
Maybe I'm misunderstanding how the ion-header is supposed to work?  I assumed that the ion-header is supposed to use the z-index to determine how/whether to show the separator. So for instance, a lower z-index than the page would cause an inverse height difference bar, and the same z-index would cause no bar (smooth transition). But I can't seem to get it to exhibit any of this behavior, the bar is always present unless it it written over by other components (such as in the margin example I mentioned earlier).

As an example of the expected appearance, here is what I get when I remove the  component but keep the toolbar and title:

Thanks for the follow up. When running in MD mode, we do apply a background-image to the ::after pseudo element for ion-header to give it a divider/drop shadow effect. Rendering an actual box-shadow is a bit less performant than rendering an image, so we just render a background image instead.
In order to get rid of the divider, you can just do:
ion-header::after {
  content: initial !important;
}

Does this resolve your issue?

Adding that does seem to cause the desired behavior, thank you!
I was under the impression that that background image in the ::after was conditional on the z-score, but I guess that was a faulty assumption on my part. Thanks for your help, I'll close the issue.


Encode the pound/hash/number symbol (#)

The URL code for # is %23. When a WebDAV server receives a URL containing a literal #, it ignores everything after the symbol (e.g. http://host.com/file#1234.txt becomes http://host.com/file), presumably because # is reserved for HTML anchors.
Now, webdav_client uses Uri.encodeFull to encode URLs. From the docs:

All characters except uppercase and lowercase letters, digits and the characters !#$&'()*+,-./:;=?@_~ are percent-encoded.

This means that if any method from webdav_client receives a file path containing the symbol #, it will not encode it and the server will receive a truncated request URL, which results in unexpected behaviour.

Only just now realized that this is a duplicate of #47 (after running it through Google Translate).
I'd suggest keeping it open so others don't make the same mistake as me.

Some defects will be fixed or refactored in the later stage. At present, I am busy. Thank you very much


Error: Building package zlib:x64-windows failed with: BUILD_FAILED

CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:44 (message):
Command failed: D:/Lib/vcpkg/downloads/cmake-3.10.0-win32-x86/bin/cmake.exe;--build;.;--config;Release;--target;install;--;-v
Working Directory: D:/Lib/vcpkg/buildtrees/zlib/x64-windows-rel
See logs for more information:
D:\Lib\vcpkg\buildtrees\zlib\install-x64-windows-rel-out.log
D:\Lib\vcpkg\buildtrees\zlib\install-x64-windows-rel-err.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_cmake.cmake:63 (vcpkg_execute_required_process)
scripts/cmake/vcpkg_install_cmake.cmake:24 (vcpkg_build_cmake)
ports/zlib/portfile.cmake:26 (vcpkg_install_cmake)
scripts/ports.cmake:72 (include)
Error: Building package zlib:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: zlib:x64-windows
Vcpkg version: 0.0.99-2017-11-28-6c5f52daf3cf1c3d7c0f8430f62436bf1d298b1b

install-x64-windows-rel-out.log

A similar error occurred in my environment as well. I attach a link to the log.
https://gist.github.com/bahaki386/f1322b6967d9ca2b4e323f0ce599b0bd

Maybe,In the standard library,missing type definition of std::off64_t occurs this error.

In my case,I tried
1.update to recent version of VS
2.exectue git pull
3.execute bootstrap-vcpkg.bat
4.execute vcpkg integrate install
and this errror is gone.

Please open a new issue if this is still a problem for you.


How did you generated datas for KUKA?

Did you manually created datasets of block stacking task?

Oh, I think there are some links in appendix...

hi,I just face the same problem,the error like this:
"FileNotFoundError: [Errno 2] No such file or directory: 'logs/multiple_cube_kuka_convnew_real2_128'"
could you please tell me how can i get this data,hope for your reply


Decentralization

I see you are the best Layer2 solutions of Ethereum till now but there's more like making it more decentralized like Ethereum is strongly decentralized.

Strongly agree on this one


setting fileBrowser to width 0 in the ini makes it 100% instead

I wanted to permanently disable the file browser.
The only option I found was to set the width of the pane in the ini to 0. However, the size becomes 100% then.
I suspect even with size 0 the filebrowser pane would be still active (e.g. by using a shortcut to toggle into it) so it might not a good idea to allow size 0 at all.

I have to agree with nilsgey. This is partially blessed's fault: it was never meant to accomodate 0 widths and heights. If you want an element hidden, you should call .hide(). I'll work on a fix to allow 0 widths though.

@nilsgey this will be fixed with the next release of slap, which will include the (as of yet nonexistent) next release of blessed.
@chjj just tested that commit, works great, thanks!

@nilsgey this fix has been released as part of 0.1.25. Enjoy!


Roadmap to v1

Publication

[ ] Create a build task for distribution #109
[ ] Select and add license

Documentation

[ ] Finish documentation website (sumup/circuit-docs)
[ ] Add a concrete example of an app using Circuit-UI #126
[ ] Add personality to the docs and examples with lots of circuit puns

Collaboration

[ ] Add contributing guideline
[ ] Add GitHub issue and PR template

I think there are some components missing, which need input from design:

[] Fully-fledged CC input (@Inkyov is working on one with validations but we need design/interaction too)
[] Modals
[] Selected cards
[] Table component
[] Link component

Plus:

[] Usage guidelines from design in the circuit-docs

These are the main ones that come to mind. Not sure if it matters really for v1 though, as for that I guess we just want a stable API for components we do have?

https://choosealicense.com/no-permission/


feat - add execution details api

Issue: https://github.com/novuhq/go-novu/issues/48
Adds the methods for execution details per: https://api.novu.co/api#/Execution%20Details/ExecutionDetailsController_getExecutionDetailsForNotification
Uses the QueryBuilder interface i added in the previous PR for Messages

@Cliftonz I can't find any execution api endpoints that aren't included already.So I think this issue should be closed as it is completed.


[prototype] Should we make prototype.transforms._utilis.py public?

🚀 The feature
A large number of utility methods such as query_bounding_box, has_any, has_all etc are being used regularly by our new Transform Classes but are not exposed publicly to the users. We should discuss which of them we want to expose publicly, to assist Transform authoring on the new API.
Motivation, pitch
While users start developing transforms using the new API, they would be forced to write quite some extra code to achieve basic tasks like checking the provided input types, ensuring the spatial size is consistent across all items etc. Currently all these methods are private because they live in the _utils.py private module and they are not exposed via __init__.py. This is likely to push users of the new API to use private methods in their code or force them to write a lot of extra duplicate code to do basic operations.
Alternatives
We can leave the methods private (or even prefix them with _, though this seems an overkill because they are not exposed via any public module) and let users write their own utilities.
Additional context
No response
cc @vfdev-5 @bjuncek @pmeier

Maybe I haven't expressed it before, but I was always in favor of having these public just for the reason you described: without them, we are making it harder for users to write custom transforms. That will hinder adoption and that is the last thing I want.

@pmeier I agree with your points. There are numerous methods we use for our standard transform authoring that start with _ while others dont. I think some of them are marked with _ incorrectly.
 We won't make them public now (they are in a private module and not exposed) but it will become clearer what's the proposal.

Just to understand you correctly: an prefixed _ should mean "this should not be exposed" while everything else will be either exposed or is a candidate for which we need to decide later?

Correct. One example I had in mind is the _isinstance utility method that is probably extremely useful to expose. I'm sure the _ here was put to avoid the conflict with the builtin method (so we need a better name), but looking at the _utilis.py I don't see a clear proposal to what could be public and what is definitely private. This is probably low-priority now as we first want to get feedback from users concerning the existing transforms and then let them do authoring (once the API is stable) but it's good to mark it on our backlog.

Closed in #6989.


Treams installation failure due to NumPy 2.0.0 update

treams fails to initialize when all the requirements are installed via pip automatically.
Here is how to recreate the bug. I'm using clean virtual environment on a Windows 11 machine, Python 3.10.2.
(venv) > pip install treams

Collecting treams
  Using cached treams-0.4.1-cp310-cp310-win_amd64.whl (1.6 MB)
Collecting numpy
  Using cached numpy-2.0.0-cp310-cp310-win_amd64.whl (16.5 MB)
Collecting scipy>=1.6
  Using cached scipy-1.14.0-cp310-cp310-win_amd64.whl (44.8 MB)
Installing collected packages: numpy, scipy, treams
Successfully installed numpy-2.0.0 scipy-1.14.0 treams-0.4.1

As the installation requirements don't specify a NumPy version, the newly released 2.0.0 is installed. This, however, leads to the following error in a Python script:
>>> import treams

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "...\venv\lib\site-packages\treams\__init__.py", line 63, in <module>
    from treams._core import (  # noqa: F401
  File "...\venv\lib\site-packages\treams\_core.py", line 8, in <module>
    import treams._operators as op
  File "...\venv\lib\site-packages\treams\_operators.py", line 8, in <module>
    import treams.special as sc
  File "...\venv\lib\site-packages\treams\special\__init__.py", line 204, in <module>
    from treams.special import _gufuncs, _ufuncs  # noqa: F401
  File "src\\treams\\special\\_gufuncs.pyx", line 1, in init treams.special._gufuncs
ValueError: numpy.dtype size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject

This seems to be similar to this issue in pandas also caused by the latest NumPy major update. It's not clear for me why this issue occurs. A temporary fix might be adding numpy<2.0.0 to the requirements.

https://stackoverflow.com/questions/78650222/valueerror-numpy-dtype-size-changed-may-indicate-binary-incompatibility-expec

Hey @levdik, thanks for posting this here. We are working to fix the issue soon. It seems like numpy 2.0 has introduced some incompatibilities going beyond having to recompile the binaries against the new version. We'll likely publish a version to PyPi that pins an older numpy version this week. This will allow us to thoroughly investigate possible compatibility problems with numpy 2.0.
Best Regards
JD

@jan-david-fischbach thank you very much for the quick response and fix. Much appreciated.


Zhg general refactory

General refactoring in all Cluster Admin module, corresponding #50 PR of Engine Executor

Codecov Report

Merging #10 into master will increase coverage by 1.75%.
The diff coverage is 68.42%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
+ Coverage   71.57%   73.33%   +1.75%
==========================================
  Files           9       10       +1
  Lines          95      120      +25
==========================================
+ Hits           68       88      +20
- Misses         27       32       +5

Impacted Files
Coverage Δ

...exception/EngineExceptionAndRejectionHandler.scala
23.07% <ø> (ø)

...ain/scala/org/marvin/cluster/ClusterAdminApp.scala
65.51% <65.51%> (ø)

...in/scala/org/marvin/cluster/api/AdminHttpAPI.scala
73.68% <77.77%> (+17.16%)
:arrow_up:

Continue to review full report at Codecov.

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


Find windows vulnerability

Hello,
is it planned to support vulnerabilities from https://msrc.microsoft.com/update-guide
(windows vulnerabilities and KB fixes)

I believe this is an issue with osv.dev so I will transfer it over there


feat: Add Azure Blob configuration to the object storage setting

Released issue: https://github.com/GreptimeTeam/greptimedb-operator/issues/215

Summary by CodeRabbit

New Features

Added support for Azure Blob storage, allowing users to configure blob storage options.
Introduced new fields and methods for managing blob storage credentials and settings.

Bug Fixes

Enhanced validation checks for the new BlobStorage provider to ensure proper configuration.

Documentation

Updated documentation to reflect the new Azure Blob storage capabilities and configuration options.

@daviderli614 It's mostly LGTM. I think we should use' blobandAZBlobas part of the naming instead of usingblob` directly.

@daviderli614 It's mostly LGTM. I think we should use azblob and AZBlob as part of the naming instead of using blob directly.

I changed it to use azblob to define this structure:
  objectStorage:
    azblob:
      container: "greptimedb"
      secretName: "azblob-credentials"
      endpoint: "https://<storage-account>.blob.core.windows.net"
      root: "cluster-with-blob-data"


Ensure minimum node version is specified

The code ensures node version is correct, but the user is not warned of requirements in connection with planning the installation of pxt.

Hi @OwenBrotherwood, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Need to change activity Eval() signature in templates

In github.com/TIBCOSoftware/flogo-cli/tools/activity/files.go template the Eval() method has the wrong type of error.
This creates an error when building a custom Activity.

Fixed in commit: 70d61fd


Add an --active flag to src add to immediately activate the added data source

Currently, if there's a non-zero number of data sources, you need to do
> sq add https://feed.acme.com/rss.xml @rss_acme
> sq src @rss_acme         # make @rss_acme the active data source
> sq '.item'

It should be possible to do:
> sq add https://feed.acme.com/rss.xml @rss_acme --active
> sq '.item'

Implemented.


append typing of JsonSchema to allow format

format goes alongside type. It's not a nested object which you supply to type.
e.g.
  {
    "type": "string",
    "format": "email",
  }

So instead of changing types typing, format needs to be added as a new item in JsonSchema itself.

Not sure this needed to be closed... the types are missing the format option after all. Would be good to get them in there - the main way to update would just be to make a new commit in your branch, and the changes would be reflected in this PR.

No worries. I'll reopen.

reopened as #1387


Backfill @geeknoid with @costinm as top level owner

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

Test name
Commit
Details
Rerun command

e2e-simpleTestsMinProfile_istio
f7316fec42a52c91beeee2aa900e332edaccb025
link
/test e2e-simpleTestsMinProfile_istio

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.

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

Test name
Commit
Details
Rerun command

e2e-simpleTestsMinProfile_istio
f7316fec42a52c91beeee2aa900e332edaccb025
link
/test e2e-simpleTestsMinProfile_istio

release-test_istio
f7316fec42a52c91beeee2aa900e332edaccb025
link
/test release-test_istio

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.

/retest


Does not work with python 3.6

Unfortunately installation fails under python 3.6. I hacked the matlab engine to allow 3.6 to be "believed supported", but then imatlab had a fit (below).
I'm at a loss.

Traceback (most recent call last):
File "/Users/jslater/anaconda/lib/python3.6/site-packages/matlab/engine/init.py", line 42, in
pythonengine = importlib.import_module("matlabengineforpython"+_PYTHONVERSION)
File "/Users/jslater/anaconda/lib/python3.6/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 978, in _gcd_import
File "", line 961, in _find_and_load
File "", line 948, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'matlabengineforpython3_6'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/jslater/anaconda/lib/python3.6/site-packages/matlab/engine/init.py", line 58, in
pythonengine = importlib.import_module("matlabengineforpython"+_PYTHONVERSION)
File "/Users/jslater/anaconda/lib/python3.6/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 978, in _gcd_import
File "", line 961, in _find_and_load
File "", line 948, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'matlabengineforpython3_6'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/jslater/anaconda/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/Users/jslater/anaconda/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/Users/jslater/anaconda/lib/python3.6/site-packages/imatlab/main.py", line 3, in
from ._kernel import MatlabKernel
File "/Users/jslater/anaconda/lib/python3.6/site-packages/imatlab/_kernel.py", line 17, in
import matlab.engine
File "/Users/jslater/anaconda/lib/python3.6/site-packages/matlab/engine/init.py", line 61, in
'MathWorks Technical Support for assistance: %s' % e)
OSError: Please reinstall MATLAB Engine for Python or contact MathWorks Technical Support for assistance: No module named 'matlabengineforpython3_6'

If the engine does not support Py3.6 then the kernel will not support it either.  I've clarified this in the README.
Some tricks are possible to work around this limitation (see e.g. https://pastebin.com/Ti6FyDTa) but I will not provide support for them.


Adding functionality to sync on foreign_person_id

@ianlofs
The confusing argument order of email, attributes, foreign_key is so this is backwards compatible, so it might be accepted upstream

👍


Test Issue created at Thu, 25 Nov 2021 15:09:16 GMT

Test issue body  Thu, 25 Nov 2021 15:09:16 GMT

Comment on issue at Thu, 25 Nov 2021 15:09:18 GMT


Update upgrade-data-controller-direct-cli.md

Command to check version was missing namespace parameter.

Microsoft authors of articles need to make updates through the private repository, so the updates can be staged and validated by current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
 Public repo URL: https://github.com/MicrosoftDocs/{docs-repo}
 Private repo URL: https://github.com/MicrosoftDocs/{docs-repo-pr}
We have closed this pull request. Please resubmit the updates through the private repo. For more information, see the contributor guide.


Result level caching is not working

Result level caching is currently broken in master for all kind of queries.
Affected Version
Master
Description
During the query execution on the broker side, ClientQuerySegmentWalker wraps the CachingClusteredClient.run method in a chain of query runners. One of the participating runner is ResultLevelCachingQueryRunner which is responsible for populating result level cache. It assumes that when this chain is invoked, the responseContext is updated with the E-TAG. This E-TAG is used in ResultLevelCachingQueryRunner and if this E-TAG is null, result level cache is not populated. The flow is much better explained in #8713
The problem is that chain of query runners now include a RetryQueryRunner on top of CachingClusteredClient.run which similar to UnionQueryRunner in #8713, turns an eager load into a lazy load. Because of this, ResultLevelCachingQueryRunner never actually sees the E-TAG when it tries to access it. It's populated in the responseContext later on during the query execution. Because of this, result level cache is never populated.

@abhishekagarwal87 thanks for reporting. Since this bug was introduced in https://github.com/apache/druid/pull/10082, 0.19 will have the same issue.


using Stephen's fix to not minify javascript templates

Additional fix to not insert CDATA

Sorry. The test data sets are very intertwined at this point.

I'm not sure what you mean by that, sorry. Do you need me to do anything else so we can merge?

All tests should pass. You can either adjust them so they do or it'll have to wait until someone else can do it.

awww


[RFE] Eventrouter deployed via Rancher

What kind of request is this (question/bug/enhancement/feature request):
Feature Request
Request
Events do not persist, and the default is that the event-ttl be set to 1 hour.  Increasing this TTL can affect cluster performance, but teams facing failures can easily, and often do, take much longer than an hour to root cause and resolve an issue (especially for lower severity issues. Information important to users will be rotated out early. As basically another source of log information, it would be very helpful if Rancher had a solution to this.
The ability to deploy eventrouter (https://github.com/heptiolabs/eventrouter) as we do Logging and Monitoring would meet this need.
gz#13864

This would be a great feature. On large clusters with many pods and many events, it would be no good solution to increase event-ttl.
A rancher eventrouter app like logging and monitoring maintained by rancher would be a fine solution for this.


Update for nise cdr cetb

This PR implements for the main branch several improvements made in code forks for CDR, NISE, and CETB->Bootstrap early uses of this codebase.

TODO for tomorrow: review various BT paramter sets with @sc0tts . I would like to ensure that the docstring/comments around each parameter set make it clear what sensor(s)/data products each set is valid for, and any other useful/important notes regarding each set of parameters.

I think this looks like a good point to merge things. The way we handle params/fields in the bootstrap code is still in flux, but functional as-is. Our upcoming work to support a broader assortment of sensors/hemisphere combinations will result in further changes & refactoring.


[BUG] Input field for RAM in the custom compute offering needs polishing

Custom compute offering:

Click inside the RAM field, hit backspace on keyboard (to delete the number 1 (which is btw a very bad default value, perhaps 256?) - the field will get value of "MB MB" then if you type a number (100), you get the stuff from the screenshot
Can the "MB" be placed outside of the actual field, so it makes more sense visually - same way as the "MHz" for the CPU speed field?

Fixed in https://github.com/apache/cloudstack-primate/pull/600

@andrijapanicsb Could you verify that this has been fixed on latest master and close it ?

LGTM


Styling

A number of styling changes as requested.

I think we should remove the hover. Otherwise this pull is good to go!


Multiple molecules

Multiple molecules are no longer a problem.

It works fine on my structures. Try printing offset at the end and comparing it with the number of atoms you have.

This PR is outdated. Look at #12 for an updated version that should fix the bugs in this one.


ListCommand - sort jobs alphabetically

When running -l in apps with lots of jobs, it is hard to visually locate the job you need. Let's sort the job names alphabetically.

@ebondareva : let's make sure we update the BOM to bootique-job:0.15-SNAPSHOT

@andrus bootique-job version updating is requested in https://github.com/bootique/bootique-bom/pull/1


For Windows, How do I load an image resource and draw it on a GraphicsView?

I simply want to draw an image resource onto a GraphicsView (on Microsoft's Windows platform).  This seems like a fairly basic task.  After all, it's a "Multi-plAtform UI" right? :)
So, I fired up Google, found this documentation, and followed the sample code.  After failing to succeed, I went back to the documentation and noticed I missed this warning:
"Warning: The PlatformImage type isn't supported on Windows."
Any chance the page can be updated to provide something more useful? Perhaps, something like "On the Windows platform..." do this instead?  I'm really trying to like MAUI, but it simply doesn't seem ready for prime time yet.
I hope these early experiences don't scare other people away from it.  For my part, I'm going to wait 6 months and try again.  I was super-excited about its promise, but less so now,

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

ID: 0ca86947-204f-9c45-223d-724fb59b4938
Version Independent ID: 0ca86947-204f-9c45-223d-724fb59b4938
Content: Draw graphical objects - .NET MAUI
Content Source: docs/user-interface/graphics/draw.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

@techfan101
I hear you. I really do. I'm afraid there isn't a great alternative for the lack of PlatformImage on Windows. It used to work, during the preview phase. But then a refactoring broke it on Windows and it's stuck that way in the .NET 6 release. While it's an easy fix to make, the product team can't change the public API until .NET 7.
A possible workaround gets mentioned here but it's ugly.
Sorry I don't have better news to tell you.

@techfan101  https://github.com/dotnet/Microsoft.Maui.Graphics/pull/464


Significado indeterminado para el campo .B de los articulos en el dataset Cranfield

https://github.com/Gologle/gologle/blob/502f74a3466828a98c1820b0569e2783b02cb2c1/src/parsers/cranfield.py#L24
Ejemplo:
.I 3
.T
the boundary layer in simple shear flow past a flat plate .
.A
m. b. glauert
.B
department of mathematics, university of manchester, manchester,
england
.W
the boundary layer in simple shear flow past a flat plate .
the boundary-layer equations are presented for steady
incompressible flow with no pressure gradient .

.I 5
.T
one-dimensional transient heat conduction into a double-layer
slab subjected to a linear heat input for a small time
internal .
.A
wasserman,b.
.B
j. ae. scs. 24, 1957, 924.
.W
one-dimensional transient heat conduction into a double-layer
slab subjected to a linear heat input for a small time
internal .
  analytic solutions are presented for the transient heat
conduction in composite slabs exposed at one surface to a
triangular heat rate .  this type of heating rate may occur, for
example, during aerodynamic heating .

Podria ser la entidad q lo publica, revista, universidad?

Nos quedamos con bibliography
self.bib = match["B"].strip()

@svex99 mala mia
self.bib = match["bib"].strip()


Add support for per_core_M < M for BMMs

Passing pipelines: https://github.com/tenstorrent/tt-metal/actions/runs/8972191114


I cant signup in https://chat.openai.com/auth/login can i use this Tg Bot?

No, you have to wait until OpenAI allows more signups.  This bot is basically a bridge between OpenAI/chatGPT and Telegram


Terasology should have layers, like an ogre.

This is a request for a new component type: a LayerComponent. Or a TagComponent, or a LabelComponent. The actual name is one of those things up for debate.
In the context of the rendering engine a LayerComponent would be useful. This component would only hold one (string) information: the layer name. By having a layer component a rendering node can find all entities in a given layer and ignore all others.
This would allow, for example, for entities with a mesh components to be rendered at different stages of the rendering pipeline: meshes in the "world" layer would be rendered while rendering the world while meshes with the "sky" layer would be rendered separately.
Notice that an entity should be able to have more than one LayerComponent attached to it, so that the same entity can be rendered (with different shaders) by multiple nodes.
There might be other ways of doing this, perhaps with existing components or inheriting from them. For example an entity with a Mesh component could instead have a SkyMesh component that is exactly the same apart from the name. When it's time to render the sky only entities with SkyMesh components might get rendered. But it feels like a misuse of the component classes.
In any case, I am open to suggestions.

 Cakes have layers! Nobody needs smelly onions.
My initial thought is that we'd want any sort of renderable component to have layer data in it. Which is where again OOP tempts us to give in and make an abstract RenderableComponent that specific renderable things could extend so the base layering stuff is available to all. Temptation leads to the Dark Side.
Rather than a single LayerComponent - how many actual layers could we envision? Is it just World or Skysphere? Maybe a UI layer for meshy things in the foreground?
If you had a very small number of named layer components you could outright qualify in a Node which entities you care about purely through component names, which feels more entity system-y. So you could ask for all entities with CelestialComponent + SkysphereLayerComponent to have a Celestial Node draw planets and moons, while AuroraBorealisComponent + SkySphereLayerComponent would know via DAG ordering that its effects should be drawn in the Skysphere layer later on since it is further in the foreground?
Within layers, say regular objects in the world, it seems more like the responsibility of their LocationComponent and occlusion math to figure out what should render when? Other than when the DAG knows for sure something comes before something else.
As far as multiple Components of the same type being attached to the same entity that's not supported at present and I'm not sure if it would be a good thing - rather substantial architecture switch. But I get the impression that making more Components is relatively cheap. For instance it may be better attaching three unique but empty Components instead of a single Component with three Strings in it (or three copies of the same Component with a single String that differs). I think in the context of the ES it would be faster and more natural to simply look for the presence of a Component than going digging in a Component for actual variables. Also easier to eventually improve with an entity indexing system.
I'm probably not spot on here, but I hope that helps! Happy to dig deeper if this gets you thinking and coming up with more details and explanations :-)

Thank you for the feedback @Cervator. If I think about the sky, it might have the following layers:

background layer - i.e. the Milky way and extra-galactic objects
deep sky layer - stars mostly, but also other galactic objects such as nebulae
planetary layer - planets, moons, comets
atmospheric layer - sky color, meteors, auroras

Effectively the layers we are discussing here, because of the distances involved,  can be considered entirely separate and hence rendered separately. Once a layer is rendered the depth buffer is reset and the next layer is rendered completely behind or completely in front of the previous layer. So, given what you are saying I could just aim to tag entities with components such as "BackgroundTag", "DeepSkyTag", "PlanetaryTag" and "AtmosphereTag".
And another idea that has been sloshing in my head is to convert RenderSystems into layers. I.e. shadowcasting entities would simply need a mesh component and a ShadowCastingTag component. Same goes for semi-transparent objects: attach a SemiTransparentTag component to the entity and the mesh gets rendered by the node handling semi-transparent objects - no need to implement a RenderSystem for it.
I thought of using layers also to connect specific entities to cameras. An example would be cloaked character: it might be invisible from the player camera but it might show up in the feed from an infrared security camera instead. I guess we could implement this again by having renderable entities having a MainCameraTag and an InfraredCameraTag.
Ok, fine, you convinced me.

I plan to take care of this soon. The changes in the codebase will be minimal.


Missing compatibility with Android Studio Hedgehog | 2023.1.1

When updating to Android Studio Hedgehog, the IDE prompts downgrading Phrase from version 5.1.1 to 4.0.1. The latest version 5.1.1 only has support for up-to Android Studio Giraffe.
Will there be a new plugin version with support for the newer IDE versions?
Thanks

Hi @luispinho, thanks for reporting! I just opened an internal ticket for fixing this.

@luispinho we are also grateful for any contributions, so if you have some time, feel free to open a PR (the most time consuming chore being installing and trying out the latest AS version). that way you wouldn't have to wait for us :smile: thanks!

I have the same issue as well

@luispinho unfortunately, currently we don't have resources to keep this plugin maintained. As the plugin is open-sourced, we're happy to accept pull requests though.
In the meantime, please use phrase-cli command line tool, which is used by the plugin under the hood anyway.


turbolinks:before-cache doesn't do what I expect

Hey, not sure if this is a bug or just that I have broken expectations?
I've put a sample project together here and description of the issue & how to replicate:
https://github.com/EdwardIII/turbolinks-events

Hi Edward,
I think you’re misunderstanding how jQuery’s remove method works.
$(document).on 'turbolinks:before-cache', ->
    $('body').remove('.spinner-wrap')

First you need to select the elements you want to remove, and then you call remove().
$(document).on 'turbolinks:before-cache', ->
    $('body .spinner-wrap').remove()

Closing because this doesn’t seem to be an issue with Turbolinks.

Rookie error, sorry!


Upgrade husky to 1.0.0

Seems like the install size went from a few hundred kb to 1,22MB
Since we haven't ran into any issues with husky before there's no need to upgrade.


Unrecognized cop or department RSpecRails/HttpStatus

Hello! Today I have faced with interesting issue:
Error: unrecognized cop or department RSpecRails/HttpStatus found in /home/circleci/vendor/bundle/ruby/3.3.0/gems/rubocop-rspec_rails-2.28.1/config/default.yml
Did you mean `RSpecRails/HaveHttpStatus`?

Current stack:
ruby 3.3.0 (2023-12-25 revision 5124f9ac75) [arm64-darwin22]
'rubocop', '~> 1.62', '>= 1.62.1'
'rubocop-performance', '~> 1.20', '>= 1.20.2'
'rubocop-rspec', '~> 2.28'
Here is failed step on CI:
https://app.circleci.com/pipelines/github/on-strum/ruby-on-strum-logs/79/workflows/112f2214-e921-4cc7-8fde-1ee8952a157e/jobs/627
Here is my current Rubocop configuration:
https://github.com/on-strum/ruby-on-strum-logs/blob/master/.circleci/linter_configs/.rubocop.yml
What am I doing wrong? Thanks!

Similar issue in other repo: https://github.com/rubocop/rubocop-rspec/issues/1846#issuecomment-2027510761
I'm curious if the release notes are correct and the names should be aliased ... or if that's wrong and we need to rename these with this version bump.

I am getting the same error

I realized it's a strange phenomenon.
When rubocop-rails and rubocop-rspec are present, the setting RSpec/Rails/HttpStatus causes an Ambiguous cop name error.
❯ rubocop -V
1.62.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.31.2, running on ruby 3.3.0) [arm64-darwin23]
  - rubocop-rails 2.24.1
  - rubocop-rspec 2.28.0

The following do not cause an error:
# .rubocop.yml
require:
  - rubocop-rails

RSpec/Rails/HttpStatus:
  EnforcedStyle: numeric

❯ bundle exec rubocop
.rubocop.yml: RSpec/Rails/HttpStatus has the wrong namespace - replace it with Rails/HttpStatus
Inspecting 4 files
....

4 files inspected, no offenses detected

However, the following do cause an error:
require:
  - rubocop-rails
  - rubocop-rspec

RSpec/Rails/HttpStatus:
  EnforcedStyle: numeric

❯ bundle exec rubocop
Error: Ambiguous cop name `RSpec/Rails/HttpStatus` used in /ydah/sandbox_capybara_with_rspec/.rubocop.yml needs department qualifier. Did you mean Rails/HttpStatus or RSpecRails/HttpStatus?

What if you change the configuration key from RSpec/Rails/HttpStatus: to RSpecRails/HttpStatus:?

No errors or warnings are raised.
❯ bundle exec rubocop
Inspecting 4 files
..C.

What if you change the configuration key from RSpec/Rails/HttpStatus: to RSpecRails/HttpStatus:?

require:
  - rubocop-rspec
  - rubocop-performance

AllCops:
  DisplayCopNames: true
  DisplayStyleGuide: true
  TargetRubyVersion: 2.5
  SuggestExtensions: false
  NewCops: enable

I don't understand why this message is produces, I didn't use rubocop-rails at all. For me it's not an expected behavior.

Is rack included in the Gemfile? If no, this is probably resolved by the latest master rubocop-rsec_rails.
https://github.com/rubocop/rubocop-rspec_rails/pull/13

@ydah No. Here is an example what actually used: https://github.com/on-strum/ruby-on-strum-logs/blob/010d14f09e20e0aa3dc9c65ceaa4dad2962f8992/.circleci/gemspecs/latest

I see.The following PR will probably solve the problem.

https://github.com/rubocop/rubocop-rspec_rails/pull/13

If you add rubocop-rspec_rails to the Gemfile with the master branch specified, the error should go away. If so, wait for the next release.

FYI, A fixed version has been released:
https://github.com/rubocop/rubocop-rspec_rails/releases/tag/v2.28.2
Thank you for your report.

@ydah Thanks! With this latest release all works as expected. Cheers!


feat: Add support for executing custom user data

This commit introduces the ability to run a custom script prior to installing and starting the runner service. Users can now pass custom script (runs as sudo) to configure their runners with specific requirements before the runner is fully initialized.
A new input parameter custom_script has been added. It accepts a string value representing the custom script. If this parameter is provided, the script will be executed before the runner service starts, allowing for custom configurations or setup tasks to be performed seamlessly.
Address: #13

@vprashar2929 Thanks for the quick turn-around.
Could you also update the README.md Inputs and example with the new parameter


Allow dash in ident as it is legal in source names

@arnej27959 please reviev

I considered that, and I don't think so (it must be preceded by a letter character).


User info on translation

Again, something small and just putting my thoughts out there: We had two cases where someone was over motivated and started to translate a few thing without checking the glossary or the forum or anything. We had to undo most of his changes:

Search by translator
See history to get previous translation

We could have downloaded an older PO file, but for those who aren't familiar with git, transifex had a nice solution.
I guess this is quite an edge case, not saying it's a must to have that, just pointing out that this feature was used before.

The translations history is already in place, and opening a translator instance (I mean the online translation page) showing all the translations made by an user is very simple (and is already in my to-do list :wink: )

Community Translation supports two kinds of translators:

normal translators: they can translate everything, but if a translation is approved they can just submit a suggestion
reviewers: these users can approve/reject suggestions.

@Remo would it be enough?

About translators, yes, but I'm not sure who manages the team. We currently have a few translators who are allowed to add new translators to their language. Either this or the admin has to manage all the teams.

Either this or the admin has to manage all the teams.

I vote for the first option. Furthermore since CT users will start from scratch, it's a good opportunity to fix the translation teams

I'm fine either way, we just have to be aware of the consequences. If we have language managers we have to maek them aware of that fact, an email would be nice to introduce them to the new system, if the admin manages it we need someone that has time to manage them. I know that not all existing language managers would be happy with that though. If someone did most of the work and then the admin approves someone who didn't even read the glossary and destroys the existing translations people get upset.

I don't think that a global administrator for all the languages is a suitable approach: I don't think anybody knows all the 50+ languages we currently have.
Every single translated string of Community Translation has the following properties:

current: every English (source) string may have up to 1 current translation for every language.
approved: a nullable boolean value that holds the approval state of every translation. It can be true if the translation is approved, false if it's not approved, or null if it's waiting for review.

In order to describe this approach, let's talk for instance about the Klingon translation team.
Alice is the Klingon coordinator, and she marks all the current translations as approved.
Let's assume that Bob wants to help translating Klingon: he ask to enter the Klingon translation group.
Alice approves his join request, and Bob becomes a Klingon translator.
Let's talk about the the following cases (but there are more):

Bob changes the translation of an existing approved string
Since the previous translation is marked as approved, the current translation is not changed, we keep the approved one, but a notification is sent to Alice telling her that Bob is suggesting a new translation.
Alice can then approve it (and it'll become the current translation), or reject it.
Bob translates a string that previously wasn't translated
Since there's no current translation, the Bob's one is marked as the current one, but Alice is notified that Bob added a new translation that should be approved.
Alice is then able to approve the Bob's translation (or again reject it)

What are the problems with this approach?

No problem at all, just make sure we aggregate the change notifications, otherwise one might receive a few hundred e-mails ;-) One mail per day would be sufficient

That's what I have in my mind: with the old version of CT the notifications where sent immediately  (so we could have had hundreds of notifications).
With the new version of CT this is no more the case: we store the notifications in a database table (already implemented), and we'll have a CLI commands that aggregates the notifications and delivery them (still to be implemented).


[Feature request] output stereo3d image into separate left and right file

Hi, would it be possible to add (another) output option for outputting the stereo 3d image as two left right files? eg image_left.png and image_right.png instead of combined side by side?

@T0biasCZe There have been some other things in the shortlog, but now this option is available in the next branch (soon they will become available in the main branch).


Warning: This development build of composer is over 60 days old.

This is with pulling the php7 alpine image.

I noticed the latest tags are not being build automatically and a lot of them are lagging behind. I'll investigate and see if we can add some post hooks to the composer repository to automatically trigger builds of the latest tags when changes are committed/merged in.

I have triggered a build for most tags. However the latest for alpine seems to have failed, due to some pear issues. Not sure yet how to resolve this as I do not understand why there are even pear dependencies inside this container.


Clarification 10k network rules

I have a /16 Internal network which is 65534 Ip address, this is placed within an IP Group.
Does this count as a one rule ?
Does this calculation only apply which IP address /ranges have been used over IP Groups.
Does this calculation work with both the source and destination, within the rule ? For example internal range /16 to internal range. /16
Reference section
"Do not exceed 10k network rules, and make sure you use IP Groups. When creating network rules, remember that for each rule, Azure actually multiples Ports x IP Addresses, so if you have one rule with four IP address ranges and five ports, you will be actually consuming 20 network rules. Always try to summarize IP ranges."

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

ID: 215d0a3c-ef5b-7581-dbdf-e418b4891276
Version Independent ID: 3647b4bf-6a85-8257-5792-39c5eae6785f
Content: Azure Well-Architected Framework review of Azure Firewall - Azure Architecture Center
Content Source: docs/networking/guide/well-architected-framework-azure-firewall.md
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @rohilla-shweta
Microsoft Alias: rosanto

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


tmt import does not authenticate

Trying to re-import data (make tmt to recreate main.fmf) that is related to the PR https://src.osci.redhat.com/tests/mariadb55/pull-request/9. I get:
$ tmt -r ../.. test import
Checking the '/home/hhorak/rhel/mariadb/mariadb-tests/Sanity/daemon-selinux-context' directory.
Makefile found in '/home/hhorak/rhel/mariadb/mariadb-tests/Sanity/daemon-selinux-context/Makefile'.
task: /CoreOS/mariadb/Sanity/daemon-selinux-context
summary: Checks if daemon is running in correct selinux context
test: ./runtest.sh
framework: beakerlib
contact: Branislav Blaskovic<EMAIL_ADDRESS>component: mariadb
duration: 5m
require: library(mariadb55/basic)
recommend: mariadb mariadb-server
Purpose found in '/home/hhorak/rhel/mariadb/mariadb-tests/Sanity/daemon-selinux-context/PURPOSE'.
description:

Nitrate Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/nitrate/base.py", line 200, in _server
    Config().nitrate.username,
AttributeError: 'Section' object has no attribute 'username'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/nitrate/xmlrpc_driver.py", line 393, in do_command
    return getattr(self.server, verb)(*params)
  File "/usr/lib64/python3.10/xmlrpc/client.py", line 1122, in __call__
    return self.__send(self.__name, args)
  File "/usr/lib64/python3.10/xmlrpc/client.py", line 1464, in __request
    response = self.__transport.request(
  File "/usr/lib64/python3.10/xmlrpc/client.py", line 1166, in request
    return self.single_request(host, handler, request_body, verbose)
  File "/usr/lib/python3.10/site-packages/nitrate/xmlrpc_driver.py", line 129, in single_request_with_cookies
    raise xmlrpclib.ProtocolError(
xmlrpc.client.ProtocolError: <ProtocolError for tcms.engineering.redhat.com/xmlrpc/: 401 Authorization Required>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/tmt", line 8, in <module>
    tmt.cli.main()
  File "/usr/lib/python3.10/site-packages/click/core.py", line 1137, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.10/site-packages/click/core.py", line 1062, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.10/site-packages/click/core.py", line 1668, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3.10/site-packages/click/core.py", line 1668, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3.10/site-packages/click/core.py", line 763, in invoke
    return __callback(*args, **kwargs)
  File "/usr/lib/python3.10/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/lib/python3.10/site-packages/tmt/cli.py", line 508, in import_
    common, individual = tmt.convert.read(
  File "/usr/lib/python3.10/site-packages/tmt/convert.py", line 388, in read
    common_data, individual_data = read_nitrate(
  File "/usr/lib/python3.10/site-packages/tmt/convert.py", line 431, in read_nitrate
    testcases = list(nitrate.TestCase.search(
  File "/usr/lib/python3.10/site-packages/nitrate/mutable.py", line 912, in search
    for inject in Nitrate()._server.TestCase.filter(dict(query))]
  File "/usr/lib/python3.10/site-packages/nitrate/base.py", line 205, in _server
    Nitrate._connection = xmlrpc_driver.NitrateKerbXmlrpc(
  File "/usr/lib/python3.10/site-packages/nitrate/xmlrpc_driver.py", line 443, in __init__
    login_dict = self.do_command("Auth.login_krbv", [])
  File "/usr/lib/python3.10/site-packages/nitrate/xmlrpc_driver.py", line 395, in do_command
    raise NitrateXmlrpcError(verb, str(params), e)
nitrate.xmlrpc_driver.NitrateXmlrpcError: Error while executing cmd 'Auth.login_krbv([])' --> <ProtocolError for tcms.engineering.redhat.com/xmlrpc/: 401 Authorization Required>
[hhorak@hhorak-t490s daemon-selinux-context] (fmf-daemon-selinux-context %)$ klist
Ticket cache: KCM:16119:93577
Default principal<EMAIL_ADDRESS>
Valid starting       Expires              Service principal
03/07/2022 17:20:02  03/08/2022 03:20:02<EMAIL_ADDRESS>03/07/2022 17:20:04  03/08/2022 03:20:02  HTTP/tcms.engineering.redhat.com@
	Ticket server<EMAIL_ADDRESS>

Using this version:
tmt-redhat-provision-minute-0.8-3.noarch
python3-tmt-1.11.0-1.20220304224842340357.main.2.gd0d7461.fc35.noarch
tmt-1.11.0-1.20220304224842340357.main.2.gd0d7461.fc35.noarch
tmt-provision-container-1.11.0-1.20220304224842340357.main.2.gd0d7461.fc35.noarch

FTR https://issues.redhat.com/browse/EXDSP-587

yay, it made the trick, thanks!


Update OnDiskBitmap to match core

Ref: https://github.com/adafruit/circuitpython/pull/4823
The class signature of OnDiskBitmap has been changed in core as of 7.0.0-alpha-3.
The Blinka implementation should be updated to match.

Fixed by #75.


Fix bug that did not allow setting of times in a timetable

fixes #231

Current coverage is 54.55% (diff: 0.00%)

Merging #241 into dev will increase coverage by 0.02%

@@                dev       #241   diff @@
==========================================
  Files           180        180
  Lines          3802       3800     -2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2073       2073
+ Misses         1729       1727     -2
  Partials          0          0

Powered by Codecov. Last update bed301d...236a497

Oof, how did this one get by?


Scan Rate

You may have slowed down the scan rate to find pokemon. But the refresh rate on 1.2 was ideal, this new one is only useful for a scan radius of like 4 or 5, anything more and it takes way too long to scan back through the area as it will take 10 minutes which is the spawn timer for pokemon.

agreed... cycling through a 7 radius, and you may only find pokemon that have recently been scanned only have a time of 7 mins left before they disappear, scan radius in 1.2 was perfect, if OP could find a way to get back to that, that would be great!

Where do you specify radius, guys?

On the login screen

I'll just ask here instead of posting new issue:
So is this why my scanner doesn't always show every pokemon inside the radius? Mine is set to 10.

yea but since niantic is adding security to their system.....scanning to fast will be a problem really soon :/

definitely 1.2 scan is a lot faster. but not as accurate. hopefully the scan doesnt cost me a dragonite haha.

yea scanning is so much slower in 1.6
had to change radius down to 7

even a radius of 7 will not work under Niantics new security measures. Your speed is measured by the distance traveled between each update. You can only trigger discovery at that speed so you have to be within a distance from your last node that calculates to around 25mph, otherwise no events are triggered. it also seems like you have to be going that speed for a couple updates before it will start triggering again once it stops. A radius of 3 even triggers this speed issue upon resetting a search cycle. What is needed is either a helix shaped scan pattern that oscillates or click to scan small 2 block radius that will only perform a scan one time and forces a delay before accepting a new scan. Right now the game is paying close atttention to your speed, they've basically nerfed driving. Going to fast gets you nothing at all

using 3 radius just now i have many mons that appear with 3 mins or under.  2 long battles elsewhere and by then you've got 30 secs to pick between 2 mons disappearing in 30 secs.  its def. frustrating for sure - but better than walking around out in the heat :)

still using 1.2 because i cant run the full exe files in the new versions because my antivirus found virus after I run the program and I don't want running it if isn't safe...I don't know if is safe or not and its false alarm by AVG
hopefully the 1.2 exe is safe so i still using it and it still work well :)
it can find all the Pokemon nearby!

@PhilipposVassil There are no insecurities with this program, the virus detection from AVG is a False-Positive.

@AVellucci Are you sure? Because in IT lessons told us to trust it because sometimes tell us the truth.
In a lab show as a file that is created by them and test it in 2 different antivirus one has found it and the other not...
That why i am so careful about it

I won't discuss Anti-Viruses and how they work here but I can confirm as an experienced IT Professional that the Malware that AVG is detecting is a False-Positive.
The .nuget has the same files in it as the .exe except that the .exe will use an installation script instead of extracting files.


Real-time Code Analysis (pyflakes?) incorrectly flags exp() function as undefined

From<EMAIL_ADDRESS>on 2012-05-24T07:48:01Z
Spyder Version:  2.1.9
Python Version:  2.7.2
Qt Version:      4.7.4, PyQt4 (API v1) 4.8.6 on Windows
What steps will reproduce the problem?

Open new file
type: exp(1)
flag shows with text "undefined name 'exp'" What is the expected output? What do you see instead? Expect to not see message

Please provide any additional information below
. If I understand the structure of Spyder correctly, this is probably and issue with pyflakes?
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1045

From pierre.raybaut on 2012-05-24T09:19:42Z
Strictly speaking, if there is no 'from numpy import exp' or 'from math import exp' in your script before the expression 'exp(1)', the code analysis (powered by pyflakes, indeed) is right to state that the 'exp' function is undefined.
Am I misunderstanding your report?

From<EMAIL_ADDRESS>on 2012-05-24T10:22:56Z
My bad -- I apparently misunderstood (slightly) how the functionality is supposed to work. Please close this as a Nobug.
Thanks for getting back to me so quickly -- I'm enjoying the environment!

From pierre.raybaut on 2012-05-24T10:25:16Z
I'm glad that you like it.
Please do no hesitate to create other "Issues" (bug reports or feature requests).
thx
Status: Invalid