ScrollController jumpTo fails assertion 'activity.isScrolling': is not true

Steps to Reproduce
Code in words:
I start of with an CustomScrollView with an initial offset. If the user scrolls to the top from this initial offset, I will direct him to another screen. But if the user scrolls to the top from another offset, i want the ScrollController to jump back to the initial offset.
Code to reproduce:
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: TestScreen(),
    );
  }
}

class TestScreen extends StatefulWidget {
  static double initialScrollOffset = 50;

  _TestScreenState createState() => _TestScreenState();
}

class _TestScreenState extends State<TestScreen> {
  ScrollController _controller = ScrollController(
    initialScrollOffset: TestScreen.initialScrollOffset,
  );
  double scrollStartOffset;

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

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

  void _scrollListener() {
    if (_controller.offset == 0) {
      if (scrollStartOffset == TestScreen.initialScrollOffset) {
        // move to the next screen
      }
      _controller.jumpTo(TestScreen.initialScrollOffset);
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        centerTitle: true,
        title: Text(
          'TestScreen',
        ),
      ),
      body: NotificationListener<ScrollNotification>(
        onNotification: (ScrollNotification scrollNotification) {
          if (scrollNotification is ScrollStartNotification) {
            scrollStartOffset = scrollNotification.metrics.pixels;
          }
          return;
        },
        child: CustomScrollView(
          controller: _controller,
          slivers: <Widget>[
            SliverToBoxAdapter(
              child: Container(
                height: TestScreen.initialScrollOffset,
                color: Colors.yellow,
              ),
            ),
            SliverToBoxAdapter(
              child: Container(
                height: 1000,
                color: Colors.red,
              ),
            )
          ],
        ),
      ),
    );
  }
}

Expected results:
If the user scrolls to the top from a different position than the initialScreenOffset, the controller should jump back to this position.
Actual results:
════════ Exception caught by gesture ═══════════════════════════════════════════
The following assertion was thrown while handling a gesture:
'package:flutter/src/widgets/scroll_position.dart': Failed assertion: line 737 pos 12: 'activity.isScrolling': is not true.

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=BUG.md

When the exception was thrown, this was the stack
#2      ScrollPosition.didOverscrollBy
package:flutter/…/widgets/scroll_position.dart:737
#3      ScrollPosition.setPixels
package:flutter/…/widgets/scroll_position.dart:248
#4      ScrollPositionWithSingleContext.setPixels
package:flutter/…/widgets/scroll_position_with_single_context.dart:83
#5      ScrollPositionWithSingleContext.applyUserOffset
package:flutter/…/widgets/scroll_position_with_single_context.dart:126
#6      ScrollDragController.update
package:flutter/…/widgets/scroll_activity.dart:372
...
Handler: "onUpdate"
Recognizer: VerticalDragGestureRecognizer#7b432
    start behavior: start
════════════════════════════════════════════════════════════════════════════════

Logs

[flo@localhost bugtest]$ flutter analyze
Analyzing bugtest...
No issues found! (ran in 13.0s)

[flutter] flutter doctor -v
[✓] Flutter (Channel master, v1.17.1-pre.10, on Linux, locale en_US.UTF-8)
    • Flutter version 1.17.1-pre.10 at /home/flo/Documents/flutter
    • Framework revision 63f8b9a4d5 (29 minutes ago), 2020-04-03 10:34:14 -0700
    • Engine revision f5127cc07a
    • Dart version 2.8.0 (build 2.8.0-dev.19.0 fae35fca47)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /home/flo/Documents/android/sdk
    • Platform android-28, build-tools 28.0.3
    • ANDROID_SDK_ROOT = /home/flo/Documents/android/
    • Java binary at: /home/flo/Documents/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Android Studio (version 3.6)
    • Android Studio at /home/flo/Documents/android-studio
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.43.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.8.1

[✓] Connected device (1 available)
    • ONEPLUS A5010 • b6f09ad2 • android-arm64 • Android 9 (API 28)

• No issues found!

Hi @mawenzy
Thank you for filing the issue,
as workaround for now you can use
Timer(Duration(milliseconds: 1), () => _controller.jumpTo(TestScreen.initialScrollOffset));

instead of
_controller.jumpTo(TestScreen.initialScrollOffset);

Same issue is present with animateTo on the ScrollController any clue what might be causing it? I tried your work around, and while it prevents the assertion failure I cannot scroll anymore.

This issue is reproducible on the latest stable and master channels. The exception doesn't occur if applying the workaround above.

Sample code (be compatible with the new flutter version)
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: TestScreen(),
    );
  }
}

class TestScreen extends StatefulWidget {
  static double initialScrollOffset = 50;

  _TestScreenState createState() => _TestScreenState();
}

class _TestScreenState extends State<TestScreen> {
  ScrollController _controller = ScrollController(
    initialScrollOffset: TestScreen.initialScrollOffset,
  );
  double scrollStartOffset = 0.0;

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

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

  void _scrollListener() {
    if (_controller.offset == 0) {
      if (scrollStartOffset == TestScreen.initialScrollOffset) {
        // move to the next screen
      }
      _controller.jumpTo(TestScreen.initialScrollOffset);
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        centerTitle: true,
        title: Text(
          'TestScreen',
        ),
      ),
      body: NotificationListener<ScrollNotification>(
        onNotification: (ScrollNotification scrollNotification) {
          if (scrollNotification is ScrollStartNotification) {
            scrollStartOffset = scrollNotification.metrics.pixels;
          }
          return false;
        },
        child: CustomScrollView(
          controller: _controller,
          slivers: <Widget>[
            SliverToBoxAdapter(
              child: Container(
                height: TestScreen.initialScrollOffset,
                color: Colors.yellow,
              ),
            ),
            SliverToBoxAdapter(
              child: Container(
                height: 1000,
                color: Colors.red,
              ),
            )
          ],
        ),
      ),
    );
  }
}

Exception logs
======== Exception caught by gesture ===============================================================
The following assertion was thrown while handling a gesture:
'package:flutter/src/widgets/scroll_position.dart': Failed assertion: line 921 pos 12: 'activity!.isScrolling': is not true.

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=2_bug.md

When the exception was thrown, this was the stack:
#2      ScrollPosition.didOverscrollBy (package:flutter/src/widgets/scroll_position.dart:921:12)
#3      ScrollPosition.setPixels (package:flutter/src/widgets/scroll_position.dart:286:9)
#4      ScrollPositionWithSingleContext.setPixels (package:flutter/src/widgets/scroll_position_with_single_context.dart:78:18)
#5      ScrollPositionWithSingleContext.applyUserOffset (package:flutter/src/widgets/scroll_position_with_single_context.dart:122:5)
#6      ScrollDragController.update (package:flutter/src/widgets/scroll_activity.dart:385:14)
#7      ScrollableState._handleDragUpdate (package:flutter/src/widgets/scrollable.dart:716:12)
#8      DragGestureRecognizer._checkUpdate.<anonymous closure> (package:flutter/src/gestures/monodrag.dart:497:55)
#9      GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:275:24)
#10     DragGestureRecognizer._checkUpdate (package:flutter/src/gestures/monodrag.dart:497:7)
#11     DragGestureRecognizer.handleEvent (package:flutter/src/gestures/monodrag.dart:347:9)
#12     PointerRouter._dispatch (package:flutter/src/gestures/pointer_router.dart:98:12)
#13     PointerRouter._dispatchEventToRoutes.<anonymous closure> (package:flutter/src/gestures/pointer_router.dart:143:9)
#14     _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:625:13)
#15     PointerRouter._dispatchEventToRoutes (package:flutter/src/gestures/pointer_router.dart:141:18)
#16     PointerRouter.route (package:flutter/src/gestures/pointer_router.dart:127:7)
#17     GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:465:19)
#18     GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:445:22)
#19     RendererBinding.dispatchEvent (package:flutter/src/rendering/binding.dart:330:11)
#20     GestureBinding._handlePointerEventImmediately (package:flutter/src/gestures/binding.dart:400:7)
#21     GestureBinding.handlePointerEvent (package:flutter/src/gestures/binding.dart:363:5)
#22     GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:320:7)
#23     GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:293:9)
#24     _invoke1 (dart:ui/hooks.dart:158:13)
#25     PlatformDispatcher._dispatchPointerDataPacket (dart:ui/platform_dispatcher.dart:382:7)
#26     _dispatchPointerDataPacket (dart:ui/hooks.dart:91:31)
(elided 2 frames from class _AssertionError)
Handler: "onUpdate"
Recognizer: VerticalDragGestureRecognizer#323b1
  start behavior: start
====================================================================================================

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.7.7, on macOS 13.0.1 22A400 darwin-x64, locale en-EE)
    • Flutter version 3.7.7 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2ad6cd72c0 (10 hours ago), 2023-03-08 09:41:59 -0800
    • Engine revision 1837b5be5f
    • Dart version 2.19.4
    • DevTools version 2.20.1

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

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

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

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview.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
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

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

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • 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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 13.0.1 22A400 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 110.0.5481.177

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.9.0-1.0.pre.117, on macOS 13.0.1 22A400 darwin-x64, locale en-EE)
    • Flutter version 3.9.0-1.0.pre.117 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision cae768e3e0 (48 minutes ago), 2023-03-08 22:29:03 -0500
    • Engine revision a996e1d70a
    • Dart version 3.0.0 (build 3.0.0-312.0.dev)
    • DevTools version 2.22.2
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

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

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

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

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview.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
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

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

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • 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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 13.0.1 22A400 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 110.0.5481.177

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

I was getting the 'activity.isScrolling': is not true crash when using the SlidingUpPanel with a ListView inside where I shared the ScrollController. I was getting the crash when I quickly swipe up after closing the SlidingUpPanel. I fixed it by setting the physics: const BouncingScrollPhysics() to the ListView and now I'm not getting the crash anymore :)

@carles-mallafre  thanks for the solution it helped me a lot


how to receive sharing intent without app being open?

Is it possible that after I share to my app then the app won't be opened?
what causes the app to open?

Yes. After you've set up your app as written in the readme the host OS will show your app as an option on an sharesheet and open your app.
To access these files use ReceiveSharingIntent.getInitialText


Advice: improve custom method

Currently, creating a new template extending will prepare a new method @custommethod("some string") with ability to pass strings, but I advise you to improve ability to pass variables contains strings or arrays or objects or any kind of type, like this:
<div>
  @php
    $x = ['<input type="text" />', '<input type="password" />', '<input type="submit" />'];
  @endphp
  @formfields($x)
</div>

I think this way would introduce superior flexibility in use.

Okay I think that I could go through it in this way:
@formfields({!! $x !!})

Okay I think that I could go through it in this way:
@formfields({!! $x !!})

Thank you.


Add 18, remove 16

Signed-off-by: Shelley Lambert<EMAIL_ADDRESS>Checklist

[ ] npm test passes
[ ] documentation is changed or added (if applicable)

Thanks @sxa, updated!

@sxa - I've made the change you suggested, please confirm you approve and I'll merge this PR.

Wrong repo (sorry we need to archive this now). I did create a similar PR (https://github.com/adoptium/website-v2/pull/3940) which I merged today. I'd also be happy to remove JDK16 if we think it makes sense.


Make everything orderable.

There's a perferctly acceptable natural ordering for bool that is also
compatible with the ORDER BY clause.
It makes sense to define ordering for uuid and bytes as they are not
that different from strings in this perspective.
Objects can be ordered the same way as their id.
We can take the ordering that Postgres seems to define on its jsonb
type as the ordering for our json. This makes it possible to order json
in some stable fashion in the ORDER BY clause.
Add tests for validity of using different scalars in ORDER BY clause.

Sorting by Object or json in the ORDER BY clause is quite useful in practice for the same reason as sorting by uuid. It is often desirable to have a set ordered in a stable manner, but the specific ordering doesn't matter as much.
The most simple practical example is that the client-side can then check if the result set has changed by comparing it with the previously cached value element-by-element in order. Various things like whether the set has changed, which items have been added/removed can be nicely determined by relying on a stable ordering.
In general, it may be a good rule to have in EdgeDB, that everything should be orderable.

This is related to #356. Although this PR does not cover everything mentioned in that issue.

I've removed the inconsistent behavior between = and > operators that resulted in different casts applied to float/decimal operands and produced inconsistent results (such as = being False while <= and >= being both True). We now compare numeric types same way as Postgres (upcasting integers to decimals and downcasting decimals to floats).


Migrate ChannelActions.goToChannelById and goToChannel to redux

Summary
Affected areas:

Channel Switcher (CMD/Ctrl + K)
LHS switching to the next/prev channel and next/prev unread channel with the keyboard shortcuts

Ticket Link
https://mattermost.atlassian.net/browse/MM-12691
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)

@saturninoabril changes made

@hmhealey @saturninoabril changes made

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-06b4a4acac9aeae49.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-06b4a4acac9aeae49

Tested and passed on SpinMint no issues found. Documented testing steps in Jira task.

Spinmint test server destroyed

@enahum This change looks good!


Button for absolute or magnitude number of bins

@JoshuaSteer would this also apply to the analyse panel?

Yep


fmt: add new serde sub-module for integer timestamp integration

This adds a new jiff::fmt::serde sub-module that contains helper
sub-modules that work with Serde's with attribute. For example:
use jiff::Timestamp;

struct Record {
    #[serde(with = "jiff::fmt::serde::timestamp::second::required")]
    timestamp: Timestamp,
}

let json = r#"{"timestamp":1517644800}"#;
let got: Record = serde_json::from_str(&json)?;
assert_eq!(got.timestamp, Timestamp::from_second(1517644800)?);
assert_eq!(serde_json::to_string(&got)?, json);

This is inspired in part by how Chrono supports a similar use case.
It is expected that the behavior should be the same, although this
implementation does support the full gamut of integer types (including
a 128-bit integer number of nanoseconds). Moreover, the naming is
different. Chrono uses a flatter namespace, where as here, we bury
everything into sub-modules. The idea is to leave some room for future
expansion, although I'm not sure there is much else to add. I also feel
like spelling out timestamp instead of ts is a bit clearer.
The module paths are quite long, e.g.,
jiff::fmt::serde::timestamp::second::required and
jiff::fmt::serde::timestamp::second::optional. But they are
predictable. And to mitigate users needing to click around through a
deep module tree, we include the full tree in the jiff::fmt::serde
module documentation.
Ref #100, Closes #101

This PR is on crates.io in jiff 0.1.11.

Docs: https://docs.rs/jiff/0.1.11/jiff/fmt/serde/index.html


Update RecursiveArrayTools to the latest version

I need to upgrade the deps to fix a compatibility issue in package Bloqade.jl, can we release a patch version after merging this PR?

Sure, I'll take care of it.

@GiggleLiu See https://github.com/JuliaRegistries/General/pull/111735


replace guest element with persons

I see it as more flexible, and can work on both a channel and item level.

I did consider using the http://microformats.org/wiki/hcard standard as a child element, but I think having name, role, img and href is much simpler. The rest of the hCard standard is addresses, phone numbers etc., which are mostly irrelevant for podcasts.

One thing I didn't pick up on is the introduction of a type of collection tag with persons:
<podcast:persons>
   <podcast:person>
   <podcast:person>
   <podcast:person>
<podcast:persons>

We aren't doing that in other places where multiple tags can exist. @daveajones we may want to adopt a standard approach to whether or not we collect them under a parent.

One thing I didn't pick up on is the introduction of a type of collection tag with persons:
<podcast:persons>
   <podcast:person>
   <podcast:person>
   <podcast:person>
<podcast:persons>

We aren't doing that in other places where multiple tags can exist. @daveajones we may want to adopt a standard approach to whether or not we collect them under a parent.

I am actually leaning towards you having the right approach with transcripts. I feel like there's probably a reason why they didn't go with

in the RSS spec, but just a lot of 's.
So I feel by not having the parent element we are being "more in spirit" with the RSS 2.0 spec.

Yes, I very much don't like grouping things under a parent tag unless it's just obvious to everyone that it makes things easier.

I have no strong feeling about it.  If "person" tag with a "role" attribute makes implementation simpler than host/guest tags I'm all for it.  I'd just say, @MartinMouritzen get rid of that parent tag and just make it "person" everywhere.


add default recipes

Your main file is still named RecipeUtils.jl instead of RecipePipeline.jl.

This looks like we would also have to move convertToAnyVector, Formatted, etc.

What is the purpose of these formatters?

They handle element-wise type recipes like the DateTime recipe, that pass a formatter function.
https://github.com/JuliaPlots/Plots.jl/blob/6821148e8ee1d399defa173a1de126dc63d4093f/src/series.jl#L180-L190

That should be self-contained now. The question is: Is this too much, just move it or change something?

Is this too much

I think it's required. And probably we need some more ... _apply_type_recipe should probably also be moved here. That's an awesome start, though. Thanks!

just move it or change something?

Not sure, do you have suggestions for improvements?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@382d515). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master     #21   +/-   ##
========================================
  Coverage          ?   0.69%
========================================
  Files             ?       4
  Lines             ?     144
  Branches          ?       0
========================================
  Hits              ?       1
  Misses            ?     143
  Partials          ?       0

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 382d515...c48326a. Read the comment docs.

Can we merge these two PRs?

Looks great.
I lost track a little of the process here, but aren't we almost ready Plots side if we get some tests?

I think almost. I still think _apply_type_recipe and the recipes it is based on have to be moved to RecipePipeline. Still I'd like to merge this, so I can maybe work on tests based on this.


fix a bug where rename would fail for objectname . (dot)

o When using s3transfer and we have an object named "." (dot) and the platform is Linux, the s3transfer fails for object with an -EBUSY.
$ aws s3 sync s3://random-bucket-name/ /tmp/test
download failed: s3://random-bucket-name/. to ../../tmp/test/. [Errno 16] Device or resource busy

o The rename that is failing in above comes from s3transfer:
  File "/usr/lib/python2.7/site-packages/s3transfer/download.py", line 601, in _main
    osutil.rename_file(fileobj.name, final_filename)

o  Note that "." in Linux holds a special resemblance. Every directory contains the special name "." (dot), a shorthand name that is a map to the inode of the directory itself. That counts as a link to the directory. There can exist an object named "." but you can not overwrite this filename in Linux because of which python calls os.rename() which internally calls the rename(2) system call. The systemcall fails with -EBUSY causing this bug as this case is not handled yet.
o This is a corner case scenario if the customer is using s3transfer to copy objects named . to current working directory.
o As a fix to this issue, I propose that we change the object name to fileobj.name  and print the value in stderr.
Any feedback is appreciated.

Hi,
Instead of renaming "." file . Can we print warning & give option to enduser to decide .  What if enduser doesn't want it to be renamed. Printing a simple warning will be enough to hint enduser.


Is it possible to install it on Ubuntu?

Is it possible to install it on Ubuntu? if so how?

For help dm seve.services on instagram


Add viable comments to api-schema classes

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What

Add viable comments to classes in the api-schema classes to improve the readability of the package.
Some unnecessary classes are removed.
Some packages moved to core

Why
Improve the readability of the project.

Reference Server Preview is available here:https://anchor-ref-pr915.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr915.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr915.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr915.previews.kube001.services.stellar-ops.com/


(DO NOT MERGE) (GH-1325) Update modules to ship Ruby plugin helper

This updates the Puppetfile to include the new ruby plugin helper, which
provides helper methods for writing Bolt plugins in ruby. It also
updates the AWS inventory plugin to the newer version, which uses the
plugin helper
Closes #1325

CLA signed by all contributors.


Unit problem with MERRA2 data

The newly added MERRA2 reanalysis dataset has a problem with the unit for total cloud cover (clt). The CMORized file reports units="%" but the actual data are in the range of 0...1 (fraction).

hi @axel-lauer great catch! A tricky one since the cmor checker has not picked it up - maybe we should point this to @sloosvel have an extra check in the cmor checker pick up if the data is 0-1 (which are perfectly valid percentages, but not actual ones). Would you be willing to have us a PR with a fix? Not a worry if not, but at the mo I am slammed, so it might have to wait until next week(ish) :beer:

@axel-lauer do you think you would have some time for this or should I stab at it myself? :beer:

@valeriupredoi This is my go at this:  #2850. Would be nice if you could take a quick look.


Bug: Stream#update_value always uses v1 timestamp field name.

https://github.com/attm2x/m2x-ruby/blob/master/lib/m2x/stream.rb#L80
Here, Stream#update_value uses :at for timestamps (from v1), but field name (for v2) should be :timestamp.

@jemc nice catch! Fixed on e662aba611a71af1d513c7a3afd19e1bcb25f8a2


Partially instantiate list when lower bound on length is known

Basically, something like this:
?- length_c(A,B), 3 #=< B.
   A = [_A, _B, _C|_D], clpz:(B in 3..sup), constrained:length_c(_D, _E),
   clpz(_E in 0..sup), constrained:list_c(_D), constrained:integer_c(_E).

I think this is impossible, because 3 #=< B doesn't trigger the verify_attributes/2 of constrained.pl, just the one of library(clpz).

Is this issue related to the following behavior? I had expected this constraint to instantiate the list to that known length.
[jhemann:constrained.pl] 15:27:16$ scryer-prolog -f
?- [constrained].
   true.
?- length_c(L,1).
   error(type_error(evaluable,constrained/0),(is)/2).
?-
[jhemann:constrained.pl] 15:31:13$ scryer-prolog --version
v0.9.4-210-gff034326-modified

It did when I last worked on this project, but something changed since then. This is the same bug I mentioned in https://github.com/bakaq/constrained.pl/pull/16#issuecomment-2470816292. I still haven't been able to reduce it to something simpler, but I noticed that if I change the definition of length/2 (yes, the one in library(lists)) in a trivial way (like literary adding , true. to the end of a clause) it fixes the issue, and that's why I think it's a miscompilation.

Aaaaaaaah.... Goootcha.


Datagrid - allow to show in iframe

Implement ?c=1 parameter resolving to show or hide menu and other gui parts.

@vojtamares

Maybe the parameter could me named openInIframe or iframe?


⚠ Redacted has degraded performance

In 19ee35d, Redacted ($RED) experienced degraded performance:

HTTP code: 200
Response time: 6233 ms

Resolved: Redacted performance has improved in 4c7cd14 after 12 minutes.


More handy options to buy

You can add also telegram bots
https://t.me/lnp2pBot - Telegram bot buy/sell P2P through LN (mostly LATAM but also in EU/US)
https://t.me/BitcoinVoucherBot - Azte.co Telegram Bot buy BTC on LN
Mobile broker apps (only buy BTC)
https://swanbitcoin.com - simple, KYC
https://amber.app - LN, DCA, only US/AUS
https://xsats.com/ - (only UK) simple, KYC, LN supported
https://pocketbitcoin.com/ - buy BTC with LN, non-custodial
https://relai.ch - simple no-KYC (Swiss bank)
https://bitkipi.com/ - easy to buy and 100% control your wallets, even connect with own node
https://pocketbitcoin.com/ - support LN, non-custodial
https://ramp.network - simple no-KYC
https://www.luno.com/ - KYC, simple mobile app
https://river.com/ - only US now
https://lastbit.io/ - simple, LN supported, also VISA card

Ok, I've updated the content to include an a Buy Online section.
A few things:

Amber app, does not have LN
xSats has suspended operations, working on a release with compliance to new regs
Lastbit (closed beta) is launching Striga (with an "early access" invite)


PowerShellResponseHandler is not public

executeCommandAndChain() not working because PowerShellResponseHandler is not public.

Same here :( Very disappointed that could not use execution chaining.

Same here

I created a pull request to fix the issue. Since this repository seems stale, you may need to use Jitpack to use the hotfixed version, instead of maven central, until the PR is accepted.


Github ot Feature

Github Bot Will COmment on Every PR Open and PR merged

@Sukriti-sood Can you please recheck?
There is already a bot for this feature I guess
If there is something different, please let me know and then I can assign and merge.

@yash982000 At the time when I created issue , workflow did not exists but someone created or after that and recently you merged , I was not aware that we can open or without issue.

@yash982000 At the time when I created issue , workflow did not exists but someone created pr after that and recently you merged , I was not aware that we can open pr without issue.

Oh  okay. Do open some small issue or feature you want or some good changes in the documentation and readme and I will be happy to assign and merge and give suitable credits :)

@yash982000 Thanks for the opportunity , I will surely Open a Issue.

@yash982000  I have Opened a Issue #157


Odds Sum Challenge in two ways.

Could you add a more verbose comment on the short method for calculating the sum of positive odd numbers? It would be helpful for others when they read your code.


Bump Rancher to v2.7.9

Problem:

We need to bump Rancher to v2.7.9.
Solution:
Bump Rancher to v2.7.9.

Related Issue:
https://github.com/harvester/harvester/issues/4782
Test plan:

New nodes can't join and Rancher keep complaing:
2023/11/17 08:55:44 [INFO] [planner] rkecluster fleet-local/local: non-ready bootstrap machine(s) custom-3eecdaaeac24 and join url to be available on bootstrap node

After few days investigation, the root cause may be that CAPI can't connect to rancher service. From Machine CR [1], its NodeHealth is false with WaitingForNodeRef reason. From CAPI controller source code, the Machine CR can pass ProviderID check, but it gets error in r.Tracker.GetClient, so the controller can't run remaining code to fix Machine CR (marking NodeHealth as true).
The error message in CAPI controller is:
"Reconciler error" err="failed to create cluster accessor: error creating client for self-hosted cluster: error creating dynamic rest mapper for remote cluster \"fleet-local/local\":
the server has asked for the client to provide credentials"
controller="machine" controllerGroup="cluster.x-k8s.io" controllerKind="Machine" Machine="fleet-local/custom-11a1422a6c10" namespace="fleet-local" name="custom-11a1422a6c10" reconcileID=77a0f08a-c816-4431-a2d9-e082c2d37064

Someone say the error is caused from different version between client-go and server [2]. In Rancher v2.7.9, it uses CAPI v1.4.4, CAPI uses controller-runtime v0.14.5, controller-runtime uses client-go v0.26.1. I tried to use different RKE2 versions like v1.26.1+rke2r1 or v1.26.10+rke2r2 with this Rancher update, but both can't fix the issue.
[1] Machine CR
apiVersion: cluster.x-k8s.io/v1beta1
kind: Machine
metadata:
  annotations:
    objectset.rio.cattle.io/applied: H4sIAAAAAAAA/5yTT3PTMBDFvwqzZzu15L/xDBcYTh04BIb7WlonIrLkkdYpTCffnbEpIWlpoD3aem/1232rexiIUSMjtPeAznlGNt7F+dN330hxJF4F41cKmS2tjL8xGlqY3IAOt6TTAdXOOILkWYO/cxTS7WEPLdwcRPLm1jj99jOpQPxPm8OBoAU1RfZDKgSKQkqslMj+yxpHVLPfeoUWjgmoQEuLX8xAkXEYoXWTtQlY7MgujSs7Raaw+p7umziXe/jxm+VXreQvOu84eJuOFt0s5DBdm8sO4w5ayIu6qmWmSYs+y8tcNiXKpirWFeZZjVRlWvX9WkACYU9n/mewHonOmdLg7RnYpZJY6auCh6TTZQHWpS67qsmwrJu+kE0lhawKWWdCrEXZ1LXO1lkptGwaRNkpKYuyyLtc9b3U+mnxOx/2FC7uPyZwNf3zfHtLxOkp5TiSmqPsvOfIAcclV+96s91Qvyz7aL5SiMY7aB+hHOZR742b29zcfnh3qvFanuNpVz5dRmVcHzBymBRPgV5G9n5h+Hh6fa9Gi4w8xYtpbQj1D2h7tJGeUv45Ox5/BgAA///dMbyVQgQAAA
    objectset.rio.cattle.io/id: unmanaged-machine
    objectset.rio.cattle.io/owner-gvk: /v1, Kind=Secret
    objectset.rio.cattle.io/owner-name: custom-11a1422a6c10
    objectset.rio.cattle.io/owner-namespace: local
    pre-terminate.delete.hook.machine.cluster.x-k8s.io/rke-bootstrap-cleanup: rke-bootstrap-controller
  creationTimestamp: "2023-12-12T15:27:40Z"
  finalizers:
  - machine.cluster.x-k8s.io
  generation: 3
  labels:
    cluster.x-k8s.io/cluster-name: local
    cluster.x-k8s.io/control-plane: "true"
    objectset.rio.cattle.io/hash: 3476720ded1f0353285a286496a307ae60dcff91
    rke.cattle.io/cluster-name: local
    rke.cattle.io/control-plane-role: "true"
    rke.cattle.io/etcd-role: "true"
    rke.cattle.io/machine-id: 95d5b680a578f42862126427011915877d09051d288aa2bc224543b3cff2dde
    rke.cattle.io/worker-role: "true"
  name: custom-11a1422a6c10
  namespace: fleet-local
  ownerReferences:
  - apiVersion: cluster.x-k8s.io/v1beta1
    kind: Cluster
    name: local
    uid: 0ae3d452-3dda-4aa2-855b-98dd57dd81e7
  resourceVersion: "3531"
  uid: a80a16c1-6313-45b5-a960-22118529f0da
spec:
  bootstrap:
    configRef:
      apiVersion: rke.cattle.io/v1
      kind: RKEBootstrap
      name: custom-11a1422a6c10
      namespace: fleet-local
    dataSecretName: custom-11a1422a6c10-machine-bootstrap
  clusterName: local
  infrastructureRef:
    apiVersion: rke.cattle.io/v1
    kind: CustomMachine
    name: custom-11a1422a6c10
    namespace: fleet-local
  nodeDeletionTimeout: 10s
  providerID: rke2://harvester-node-0
status:
  addresses:
  - address: <IP_ADDRESS>
    type: InternalIP
  - address: harvester-node-0
    type: Hostname
  bootstrapReady: true
  conditions:
  - lastTransitionTime: "2023-12-12T15:27:40Z"
    status: "True"
    type: Ready
  - lastTransitionTime: "2023-12-12T15:27:40Z"
    status: "True"
    type: BootstrapReady
  - lastTransitionTime: "2023-12-12T15:27:40Z"
    status: "True"
    type: InfrastructureReady
  - lastTransitionTime: "2023-12-12T15:27:40Z"
    reason: WaitingForNodeRef
    severity: Info
    status: "False"
    type: NodeHealthy
  - lastTransitionTime: "2023-12-12T15:27:40Z"
    status: "True"
    type: PlanApplied
  - lastTransitionTime: "2023-12-12T15:27:54Z"
    status: "True"
    type: Reconciled
  infrastructureReady: true
  lastUpdated: "2023-12-12T15:27:54Z"
  observedGeneration: 2
  phase: Provisioned

[2] https://stackoverflow.com/questions/76177011/error-validating-data-the-server-has-asked-for-the-client-to-provide-credential

@mergify backport v1.2


add missing defer close for RunWithTimeout

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 24.77%. Comparing base (e279609) to head (cbc8dc6).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   24.68%   24.77%   +0.09%
==========================================
  Files          26       26
  Lines         786      787       +1
==========================================
+ Hits          194      195       +1
  Misses        575      575
  Partials       17       17

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

@emar-kar Any suggestions? Feel free to correct me if I am wrong, thanks a lot!

@WangYihang as far as I know, when you use make it always allocates memory on the heap. As soon as channel escapes the function, I would say it will be collected. But it leads to unpredictable time of collection, especially if there is data in the channel. I would prefer to have more clear understanding when resources are being released. If the tasks have huge consumption, it will require for more work for GC. Maybe I just nitpicking, but I think all those things can be avoided, by small defer with closing the channel.


Dialog buttons are not focusable

Dialog buttons (positive, negative, neutral) should be focusable because they don't respond to keyboard navigation (you can try it in emulator or by attaching bluetooth or usb keyboard to your device)

This is now in the latest code, it will be included in the next release

<IP_ADDRESS> is now available

Had to skip up to <IP_ADDRESS> due to issues with <IP_ADDRESS>

@afollestad awesome, thanks!

Hi,
On <IP_ADDRESS> i can't focus Dialog buttons (positive, negative, neutral).
So my app need remote controler or gamepad...
Have you solution?
Thanks for your help.


Require keyword file argument in add_argument

This is cleaner and gives a better output

Awesome! Thank you.


WIP: Refactored CCPACSWingRibsDefinition to use Cache

looks like the unit tests deadlocked^^

i found the issue, i just accessed the cache itself during the build routine via the member variable and not the passed reference.
i debugged the issue using a mutex i found on the internet, which checks for such errors.

can we merge?


Update Gradle to use variables from root project

Motivation
Android Target API Level 26 will be required in August 2018.
https://android-developers.googleblog.com/2017/12/improving-app-security-and-performance.html

@r3nya thank you for the PR

🙇


Race conditions initializing flashMode

Fixes #433
I found race conditions initializing flashMode on both iOS and Android. The issue can occur when a the type parameter is also specified, as the camera reinitialized. The parameters get set on the old instance of the camera and never get reinitialized on the new camera instance.

Thank you.


NodeJS async call to TSP Segmentation Fault/Invalid Pointer

Calling this code yields some various strange errors for me. Including a segmentation fault and an invalid pointer error. Is there anything in my code going wrong? Since the TSP tutorial is written in Python this is my take on it.
std::vector<int> Tsp::SolveTsp() {
  if (n > 500)
    cout<< "Specify an instance size smaller than 500."<<endl;
  else if (n > 0)  {
    RoutingModel routing(n, 1, RoutingModel::NodeIndex(0));
    RoutingSearchParameters parameters = RoutingModel::DefaultSearchParameters();
    parameters.set_first_solution_strategy(FirstSolutionStrategy::LOCAL_CHEAPEST_INSERTION);
    routing.SetArcCostEvaluatorOfAllVehicles(NewPermanentCallback(this, &Tsp::Distance));
    const Assignment* solution = routing.Solve();

    if (solution != nullptr) {
      cout << "Solving a TSP of size " << n << " x " << n << endl;
      cout << "Total length is " << solution->ObjectiveValue() << endl;

      int loopcounter = 0; //is necessary because the for loop does not run on incremental order, but on route order
      for (int64 node = routing.Start(0); !routing.IsEnd(node); node = solution->Value(routing.NextVar(node))) {
        order.at(loopcounter) = routing.IndexToNode(node).value();
	//order.at(loopcounter) = (int)node;
        loopcounter++;
      }
    } else
      cout << "No solution found."<<endl;
  } else
    cout << "Specify an instance size greater than 0."<<endl;
  return order;
}

Maybe the way I do the parameters is wrong? Or should I read parameters from flags?
I gladly hear any mistakes I made or provide more information if necessary.

Hi,
Do you have some log or backtrace from gdb ?
What is order ? what's returned by order.size() ?

Hi @Mizux,
Order is a vector containing the order of the route. As you can see it is returned by the function. It is passed to Javascript.
The error results from running the program asynchronously through Node.js, as an add-on module. I have found a workaround to prevent the error from occurring, since the error does occur with Node,js v8 but not with Node.js v10.
Still, I attach the various errors I came across here, because it might be of use for you and contain useful information about the behaviour of OR-tools in this setting.
This is the output after running it from Node:
*** Error in `node': munmap_chunk(): invalid pointer: 0x00007f9b5240f720 ***
node(_ZNSt10_HashtableIiiSaIiENSt8__detail9_IdentityESt8equal_toIiESt4hashIiENS1_18_Mod_range_hashingENS1_20_Default_ranged_hashENS1_20_Prime_rehash_policyENS1_17_Hashtable_traitsILb0ELb1ELb1EEEE21_M_insert_unique_nodeEmmPNS1_10_Hash_nodeIiLb0EEE+0x128)[0xa7b6c8]
  5.53323
***C++ output***
aftersolvetsp
afterresize
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research12PathOperator20InitializePathStartsEv+0x746)[0x7f9b53472f46]
after put in array
before delete
afterdeleter
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research12PathOperator19InitializeBaseNodesEv+0x1e)[0x7f9b5347319e]
The order in the first calculation is: 0,2,1,3
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research12PathOperator7OnStartEv+0x9)[0x7f9b534733b9]
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research22VarLocalSearchOperatorINS_6IntVarExNS_24IntVarLocalSearchHandlerEE5StartEPKNS_10AssignmentE+0x203)[0x7f9b5347c833]
/~/or-tools/ubuntu-16.04/lib/libortools.so(+0x83e266)[0x7f9b53460266]
/~/or-tools/ubuntu-16.04/lib/libortools.so(+0x83e222)[0x7f9b53460222]
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research15FindOneNeighbor4NextEPNS_6SolverE+0x235)[0x7f9b53471a95]
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research6Solver12NextSolutionEv+0x235)[0x7f9b533912f5]
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research6Solver14SolveAndCommitEPNS_15DecisionBuilderERKSt6vectorIPNS_13SearchMonitorESaIS5_EE+0x2f)[0x7f9b5339259f]
/~/or-tools/ubuntu-16.04/lib/libortools.so(+0x83db0b)[0x7f9b5345fb0b]
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research6Solver12NextSolutionEv+0x3ae)[0x7f9b5339146e]
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research6Solver5SolveEPNS_15DecisionBuilderERKSt6vectorIPNS_13SearchMonitorESaIS5_EE+0x28)[0x7f9b533919a8]
/~/or-tools/ubuntu-16.04/lib/libortools.so(_ZN19operations_research12RoutingModel33SolveFromAssignmentWithParametersEPKNS_10AssignmentERKNS_23RoutingSearchParametersE+0x206)[0x7f9b534d7d66]
/~/build/Debug/module.node(_ZN19operations_research3Tsp8SolveTspEv+0x17f)[0x7f9b604ea843]
/~/build/Debug/module.node(_Z7ExecuteP10napi_env__Pv+0xbb)[0x7f9b604e84ab]
node[0x1408461]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x76ba)[0x7f9b62b076ba]
/lib/x86_64-linux-gnu/libc.so.6(clone+0x6d)[0x7f9b6283741d]

This is output from Valgrind:
==5065== Thread 7:
==5065== Invalid free() / delete / delete[] / realloc()
==5065==    at 0x4C2F24B: operator delete(void*) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==5065==    by 0xA7B6C7: std::_Hashtable<int, int, std::allocator<int>, std::__detail::_Identity, std::equal_to<int>, std::hash<int>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<false, true, true> >::_M_insert_unique_node(unsigned long, unsigned long, std::__detail::_Hash_node<int, false>*) (in /usr/bin/node)
==5065==    by 0xA333F45: operations_research::PathOperator::InitializePathStarts() (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA33419D: operations_research::PathOperator::InitializeBaseNodes() (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA3343B8: operations_research::PathOperator::OnStart() (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA33D832: operations_research::VarLocalSearchOperator<operations_research::IntVar, long long, operations_research::IntVarLocalSearchHandler>::Start(operations_research::Assignment const*) (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA321265: operations_research::(anonymous namespace)::CompoundOperator::MakeNextNeighbor(operations_research::Assignment*, operations_research::Assignment*) (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA321221: operations_research::(anonymous namespace)::CompoundOperator::MakeNextNeighbor(operations_research::Assignment*, operations_research::Assignment*) (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA332A94: operations_research::FindOneNeighbor::Next(operations_research::Solver*) (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA2522F4: operations_research::Solver::NextSolution() (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA25359E: operations_research::Solver::SolveAndCommit(operations_research::DecisionBuilder*, std::vector<operations_research::SearchMonitor*, std::allocator<operations_research::SearchMonitor*> > const&) (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA320B0A: operations_research::(anonymous namespace)::NestedSolveDecision::Apply(operations_research::Solver*) (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA25246D: operations_research::Solver::NextSolution() (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA2529A7: operations_research::Solver::Solve(operations_research::DecisionBuilder*, std::vector<operations_research::SearchMonitor*, std::allocator<operations_research::SearchMonitor*> > const&) (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0xA398D65: operations_research::RoutingModel::SolveFromAssignmentWithParameters(operations_research::Assignment const*, operations_research::RoutingSearchParameters const&) (in /home/jim/docs/platform/modules/tsp/or-tools/ubuntu-16.04/lib/libortools.so)
==5065==    by 0x98DD842: operations_research::Tsp::SolveTsp() (module_tsp.cc:102)
==5065==    by 0x98DB4AA: Execute(napi_env__*, void*) (module.cc:43)
==5065==    by 0x1408460: ??? (in /usr/bin/node)
==5065==    by 0x5AEE6B9: start_thread (pthread_create.c:333)
==5065==    by 0x5E0B41C: clone (clone.S:109)
==5065==  Address 0xb8da720 is on thread 7's stack
==5065==  in frame #2, created by operations_research::PathOperator::InitializePathStarts() (???:)
==5065==

So the callstack towards the error is:
std::_Hashtable<int, int, std::allocator<int>
operations_research::PathOperator::InitializePathStarts()
operations_research::PathOperator::InitializeBaseNodes()
operations_research::PathOperator::OnStart()
operations_research::VarLocalSearchOperator<operations_research::IntVar, long long, operations_research::IntVarLocalSearchHandler>::Start(operations_research::Assignment const*)
 operations_research::(anonymous namespace)::CompoundOperator::MakeNextNeighbor(operations_research::Assignment*, operations_research::Assignment*)
operations_research::(anonymous namespace)::CompoundOperator::MakeNextNeighbor(operations_research::Assignment*, operations_research::Assignment*)
operations_research::FindOneNeighbor::Next(operations_research::Solver*)
operations_research::Solver::NextSolution()
 operations_research::Solver::SolveAndCommit(operations_research::DecisionBuilder*,
operations_research::(anonymous namespace)::NestedSolveDecision::Apply(operations_research::Solver*)
 operations_research::Solver::NextSolution()
 operations_research::Solver::Solve(operations_research::DecisionBui
 operations_research::RoutingModel::SolveFromAssignmentWithParameters(operations_research::Assignment const*
 operations_research::Tsp::SolveTsp()

And it occurs when calling:
const Assignment* solution = routing.Solve();

Another error I encountered was:
!!!0[15:52:08] ./ortools/constraint_solver/constraint_solver.h:4948: Check failed: element != nullptrUnknown variable Segmentation fault (core dumped)

A lot of information, I hope you can find an explanation of what is going wrong here.
Regards

Could you replace   const Assignment* solution = routing.Solve(); with
  const Assignment* solution = routing.SolveWithParameters(parameters);
src: https://github.com/google/or-tools/blob/master/ortools/constraint_solver/routing.h#L693

Yes, I tried that but it did not make any difference.

Note:
order.at(loopcounter) = routing.IndexToNode(node).value();

should be
order.at(loopcounter) = routing.IndexToNode(node);

IndexToNode() return a NodeIndex i.e. NodeIndex IndexToNode(int64 index) const;
cf https://github.com/google/or-tools/blob/master/ortools/constraint_solver/routing.h#L945
typedef RoutingNodeIndex NodeIndex;
https://github.com/google/or-tools/blob/master/ortools/constraint_solver/routing.h#L211
And RoutingNodeIndex is an int.
DEFINE_INT_TYPE(RoutingNodeIndex, int);
https://github.com/google/or-tools/blob/master/ortools/constraint_solver/routing_types.h#L34

I will mark this bug as obsolete as we do not use the Callback anymore.


chore: apply capitalization guidelines - iteration 8 (#12343)

SUMMARY
Apply capitalization guidelines defined in #12343.
@junlincc
TEST PLAN
Check if modified texts are following capitalization guidelines.
ADDITIONAL INFORMATION

[x] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

please specify all your changes in this PR, include screenshot and test plan, also for all your future PRs. @michael-s-molina 🙏

please specify all your changes in this PR, include screenshot and test plan, also for all your future PRs. @michael-s-molina 🙏

Codecov Report

Merging #12454 (c531c66) into master (078a8a1) will decrease coverage by 6.05%.
The diff coverage is 64.70%.

@@            Coverage Diff             @@
##           master   #12454      +/-   ##
==========================================
- Coverage   66.81%   60.75%   -6.06%
==========================================
  Files        1015      529     -486
  Lines       49544    19616   -29928
  Branches     5080     5080
==========================================
- Hits        33102    11918   -21184
+ Misses      16312     7489    -8823
- Partials      130      209      +79

Flag
Coverage Δ

cypress
?

javascript
60.75% <64.70%> (ø)

python
?

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

Impacted Files
Coverage Δ

...end/src/SqlLab/components/TemplateParamsEditor.jsx
13.88% <ø> (ø)

...et-frontend/src/SqlLab/reducers/getInitialState.js
33.33% <ø> (-16.67%)
:arrow_down:

...rc/explore/components/controls/AnnotationLayer.jsx
0.00% <ø> (-51.18%)
:arrow_down:

...perset-frontend/src/views/CRUD/alert/AlertList.tsx
76.56% <ø> (ø)

...set-frontend/src/views/CRUD/alert/ExecutionLog.tsx
100.00% <ø> (ø)

...CRUD/alert/components/AlertReportCronScheduler.tsx
88.46% <ø> (ø)

...ntend/src/views/CRUD/annotation/AnnotationList.tsx
76.92% <ø> (+0.83%)
:arrow_up:

...ews/CRUD/annotationlayers/AnnotationLayersList.tsx
77.88% <ø> (ø)

...perset-frontend/src/views/CRUD/chart/ChartCard.tsx
72.22% <ø> (-8.34%)
:arrow_down:

...perset-frontend/src/views/CRUD/chart/ChartList.tsx
72.44% <ø> (-9.45%)
:arrow_down:

... and 690 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #12454 (c531c66) into master (078a8a1) will decrease coverage by 6.05%.
The diff coverage is 64.70%.

@@            Coverage Diff             @@
##           master   #12454      +/-   ##
==========================================
- Coverage   66.81%   60.75%   -6.06%
==========================================
  Files        1015      529     -486
  Lines       49544    19616   -29928
  Branches     5080     5080
==========================================
- Hits        33102    11918   -21184
+ Misses      16312     7489    -8823
- Partials      130      209      +79

Flag
Coverage Δ

cypress
?

javascript
60.75% <64.70%> (ø)

python
?

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

Impacted Files
Coverage Δ

...end/src/SqlLab/components/TemplateParamsEditor.jsx
13.88% <ø> (ø)

...et-frontend/src/SqlLab/reducers/getInitialState.js
33.33% <ø> (-16.67%)
:arrow_down:

...rc/explore/components/controls/AnnotationLayer.jsx
0.00% <ø> (-51.18%)
:arrow_down:

...perset-frontend/src/views/CRUD/alert/AlertList.tsx
76.56% <ø> (ø)

...set-frontend/src/views/CRUD/alert/ExecutionLog.tsx
100.00% <ø> (ø)

...CRUD/alert/components/AlertReportCronScheduler.tsx
88.46% <ø> (ø)

...ntend/src/views/CRUD/annotation/AnnotationList.tsx
76.92% <ø> (+0.83%)
:arrow_up:

...ews/CRUD/annotationlayers/AnnotationLayersList.tsx
77.88% <ø> (ø)

...perset-frontend/src/views/CRUD/chart/ChartCard.tsx
72.22% <ø> (-8.34%)
:arrow_down:

...perset-frontend/src/views/CRUD/chart/ChartList.tsx
72.44% <ø> (-9.45%)
:arrow_down:

... and 690 more

Continue to review full report at Codecov.

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

I see error in one test which can be triggered by this change.
Could you take care of this failing test?
AssertionError: Timed out retrying: Expected to find content: 'Add annotation layer' but never did.

I see error in one test which can be triggered by this change.
Could you take care of this failing test?
AssertionError: Timed out retrying: Expected to find content: 'Add annotation layer' but never did.


Continue to have master branch

Since the branch was renamed to main even relatively recent codebases tha depend on hexl (from ~ a year ago) are now broken. Example:
git clone -b 3.6.6 https://github.com/microsoft/SEAL
cd SEAL
cmake -S . -B build -DCMAKE_INSTALL_PREFIX=$LIBDIR -DSEAL_USE_INTEL_HEXL=ON

It would have been a much smoother transition and better experience for everyone if master branch was kept in the repo even if further development is done in main or, as some projects do, if master was mirroring main, which can be done with a very simple script.

Yes.
Steps to reproduce:
docker run -it --rm ubuntu:focal

# when in docker, run:
apt update && apt install -y build-essential cmake git && \
cd $HOME && mkdir $HOME/libdir && LIBDIR=$HOME/libdir && \
git clone --branch 1.2.3 https://github.com/intel/hexl && cd hexl && \
cmake -S . -B build -DCMAKE_INSTALL_PREFIX=$LIBDIR && \
cmake --build build && cmake --install build && \
cd $HOME && \
git clone -b 3.6.6 https://github.com/microsoft/SEAL && cd SEAL && \
cmake -S . -B build -DCMAKE_INSTALL_PREFIX=$LIBDIR -DSEAL_USE_INTEL_HEXL=ON && \
cmake --build build && cmake --install build && \
cd $HOME

Error:
...
-- Configuring done
-- Generating done
-- Build files have been written to: /root/SEAL/build/thirdparty/hexl-src/cmake/cpu-features/cpu-features-download
Scanning dependencies of target cpu_features
[ 11%] Creating directories for 'cpu_features'
[ 22%] Performing download step (git clone) for 'cpu_features'
Cloning into 'cpu-features-src'...
fatal: invalid reference: master
CMake Error at cpu-features-download/cpu_features-prefix/tmp/cpu_features-gitclone.cmake:40 (message):
  Failed to checkout tag: 'master'

make[2]: *** [CMakeFiles/cpu_features.dir/build.make:92: cpu_features-prefix/src/cpu_features-stamp/cpu_features-download] Error 1
make[1]: *** [CMakeFiles/Makefile2:76: CMakeFiles/cpu_features.dir/all] Error 2
make: *** [Makefile:84: all] Error 2
CMake Error at thirdparty/hexl-src/cmake/cpu-features/CMakeLists.txt:16 (message):
  Build step for cpu-features failed: 2

-- Configuring incomplete, errors occurred!
See also "/root/SEAL/build/CMakeFiles/CMakeOutput.log".

Yes.
It is not complaining about HEXL not having a master branch but about cpu-features no longer having a master branch (Which is a HEXL dependency). Since HEXL v1.2.2 (~a year ago) we stopped pointing to master and instead we are pointing to a specific commit hash of the cpu-features. You shouldn't see this error from SEAL v3.7.1 on as they switched to HEXL v1.2.2 there.


LUN-301: Add loading and error handling

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword LUN.

Will test once the decision on conditional rendering/hook logic is made.

Things I noticed:

If you go to Bereiche verwalten, the screen jumps a little bit because the items don't occupy space until loaded

It should show a correct loading box with correct height for loading information form the cms, but we will have jumping after loading id's from async storage, as we cannot know before loading ids from storage how many boxes we need. Is that what you mean or is the loading box for loading data from cms not working?

Things I noticed:

If you go to Bereiche verwalten, the screen jumps a little bit because the items don't occupy space until loaded

It should show a correct loading box with correct height for loading information form the cms, but we will have jumping after loading id's from async storage, as we cannot know before loading ids from storage how many boxes we need. Is that what you mean or is the loading box for loading data from cms not working?

Things I noticed:

If you go to Bereiche verwalten, the screen jumps a little bit because the items don't occupy space until loaded

It should show a correct loading box with correct height for loading information form the cms, but we will have jumping after loading id's from async storage, as we cannot know before loading ids from storage how many boxes we need. Is that what you mean or is the loading box for loading data from cms not working?

We could just not show anything until everything is loaded to avoid jumping. I think it is the Individuelle Bereiche that is pushed down by the professions once loaded.

Things I noticed:

If you go to Bereiche verwalten, the screen jumps a little bit because the items don't occupy space until loaded

It should show a correct loading box with correct height for loading information form the cms, but we will have jumping after loading id's from async storage, as we cannot know before loading ids from storage how many boxes we need. Is that what you mean or is the loading box for loading data from cms not working?

We could just not show anything until everything is loaded to avoid jumping. I think it is the Individuelle Bereiche that is pushed down by the professions once loaded.

But before getting the information from Async storage if we need to load zero, one or five professions, we cannot know how much space we need to reserve for the profession. At this point we can not avoid jumping unless we block rendering at all, until we have this data, which is in general not recommended.


Find-JsonContent gets confused on roleAssignments

Try on the following - the return object of Find-JSONContent returns both properties in the same object instead of separate objects.
{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {},
    "variables": {},
    "resources": [
        {
            "type": "Microsoft.Authorization/roleAssignments",
            "apiVersion": "2020-04-01-preview",
            "name": "some guid",
            "properties": {
                "roleDefinitionId": "[concat('Microsoft.Authorization/roleDefinitions/', 'some roledef')]", //this should be flagged by ttk
                "principalId": "[reference('some UAI', '2018-11-30').principalId]"
            }
        }
    ]
}

make sure we have a unit test that covers this case (and comment why it's there since we don't have unit tests for the framework)

make sure we have a unit test that covers this case (and comment why it's there since we don't have unit tests for the framework)

Need to see if Find-JsonContent finds one or many objects (targetKey or the parent object)

Need to see if Find-JsonContent finds one or many objects (targetKey or the parent object)

yes we can use a switch - will need to scrub for the tests that need it
IDs-should-be-derived...
uris-should-be...

@bmoore-msft:  I decided to check the impact of simply returning multiple matches by default, and it does not appear to impact CI/CD.
With this in mind I retract my suggestion about a [switch] (and have retracted the stub of the [switch])


OSC-Platform: AWS key for S3 bucket redhat-osc-physical-landing-647521352890 has been exposed in github

AWS key for account ocp-odh-landing-s3-user that has access to the redhat-osc-physical-landing-647521352890 S3 bucket has been exposed in github repo https://github.com/joemoorhouse/hazard/blob/53bc89ed1fe9325c3ff12aa8e584f2d42322cfd2/credentials.env on Mon Jan 02 2023 18:08:49 GMT-0500 (EST) . AWS automatically applied policy AWSCompromisedKeyQuarantineV2 to affected account.

Actions taken :

Owner of the repository has been contacted.  Instructions based on https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/removing-sensitive-data-from-a-repository#fully-removing-the-data-from-github
AWS key has been deactivated on Jan 03 at ~ 11:28 am (EST)
Analysis of the audit logs shows that on January 02 starting from 18:08 up to 18:11 (EST) there were several attempts from IP address <IP_ADDRESS> to execute basic exploratory calls such as ListHostedZones , ListUsers and so on . All of those were unsuccessful due to lack of privileges. Subsequent ListBuckets calls originated from other IP were also unsuccessful . Based on this analysis there are no indications that exposed key has been used in malicious activity

exposed AWS key has been rotated . Respective services in OS-C clusters have been updated to use new key
Quarantine policy have been removed


neon: 0.31.2 -> 0.32.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/neon/versions.
meta.description for neon is: "An HTTP and WebDAV client library"
meta.homepage for neon is: "https://notroj.github.io/neon/"
meta.changelog for neon is: "https://github.com/notroj/neon/blob/0.32.2/NEWS"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/afwii80ykxa9anbx6c0v2p5zs6ws25i9-neon-0.32.2/bin/neon-config had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.32.2 with grep in /nix/store/afwii80ykxa9anbx6c0v2p5zs6ws25i9-neon-0.32.2
found 0.32.2 in filename of file in /nix/store/afwii80ykxa9anbx6c0v2p5zs6ws25i9-neon-0.32.2

Rebuild report (if merged into master) (click to expand)

593 total rebuild path(s)

494 package rebuild(s)

494 x86_64-linux rebuild(s)
99 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
ChowKick
adapta-gtk-theme
alfaview
alfis
almanah
apostrophe
aravis
ashpd-demo
astroid
audacious
audaciousQt5
audio-recorder
authenticator
aws-workspaces
balsa
baresip
bespokesynth
bespokesynth-with-vst2
birdfont
blanket
bookworm
bottles
brasero
brasero-original
bubblemail
byzanz
cadaver
calls
cawbird
celluloid
chatty
chrome-gnome-shell
cinnamon.cinnamon-common
cinnamon.cinnamon-control-center
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.cjs
cinnamon.pix
cinnamon.xreader
citrix_workspace
citrix_workspace_20_04_0
citrix_workspace_20_06_0
citrix_workspace_20_09_0
citrix_workspace_20_10_0
citrix_workspace_20_12_0
citrix_workspace_21_01_0
citrix_workspace_21_03_0
citrix_workspace_21_06_0
citrix_workspace_21_08_0
citrix_workspace_21_09_0

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/afwii80ykxa9anbx6c0v2p5zs6ws25i9-neon-0.32.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A neon https://github.com/r-ryantm/nixpkgs/archive/2cbb397acf58a32d8b75f6caf1878d4fae409742.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/afwii80ykxa9anbx6c0v2p5zs6ws25i9-neon-0.32.2
ls -la /nix/store/afwii80ykxa9anbx6c0v2p5zs6ws25i9-neon-0.32.2/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

Bisect says that on staging and staging-next after 2cbb397acf58a32d8b75f6caf1878d4fae409742 neon: 0.31.2 -> 0.32.2 `cataver fails to build as:
$ nix build -f. -L cadaver
...
       > checking for openssl pkg-config data... no
       > checking for library containing RSA_new... not found
       > configure: error: could not find library containing RSA_new


release: Add script to automatically bump projects

Today every Kata release someone has to clone each repo increase the release number and do a PR.
To reduce the overhead we should have an script that does this (and potentially be triggered by the CI).
./send-new-kata-version-pr <VERSION>

@egernst


Concat doesn't keep order that it was passed

I'm having a big issue with the concat function not keeping the order that I pass. This is very big issue because I have to load the scripts in the order that I passed to them in order for them to work.
Here's the code that I'm using to try and create a combined.js file.
export async function combinedjs() {
  await this
    .source([
      'node_modules/jquery/dist/jquery.min.js',
      'node_modules/bootstrap/dist/js/bootstrap.min.js',
      path.join(app.js, 'plugins', '_*.js')
    ])
    .concat('combined.js')
    .target(dest.js);
}

The problem is that I'm trying to load jquery then bootstrap then any custom plugins that are authored. However the output of the concat file is inconsistent because it's based on when the async function is finished. So if one file in the source loads faster than another it will be added to the combined.js file first.
Sometimes jQuery loads first, sometimes Bootstrap, and sometimes one of the plugins does.

👍 I ran into this a few weeks ago, been on my list but haven't had time to conjure/implement a quick fix.
@bucaran I'm thinking of a source() options object, which would be optional, just like how it is for target()
yield this.source(src, { obj })

This specific problem could be handled as obj.order: boolean or obj.parallel, but that could be misinterpreted for start()'s parallel config parameter.

In the mean time if someone needs to do this before this is fixed they can use the following script I wrote. There's no external dependancies needed, it's just using what's already is install with flyjs.
import ConcatWithSourcemaps from 'concat-with-sourcemaps';
import { read, write } from 'fly/lib/utils';
import globby from 'globby';
import path from 'path'
async function concat(files, output = 'dist/js/combined.js', sep = '\n\n') {
  const parsed = path.parse(output);
  files = await globby(files);
  let content = files.map(read);
  content = await Promise.all(content);

  const cat = new ConcatWithSourcemaps(true, parsed.base, sep);

  for (var i = 0; i < files.length; i++) {
    cat.add(files[i], content[i]);
  }

  const sourceMapOut = path.join(parsed.dir, `${parsed.name}.map.json`);
  content = cat.content + sep + '// @ sourceMappingURL=' + path.basename(sourceMapOut);

  await Promise.all([
    write(output, content),
    write(sourceMapOut, cat.sourceMap)
  ]);
}

// example
export async function combinedjs() {
  var files = [
    'node_modules/jquery/dist/jquery.min.js',
    'node_modules/bootstrap/dist/js/bootstrap.min.js',
    path.join(app.js, 'plugins', '_*.js')
  ];
  await concat(files, 'dist/lib/js/combined.js'));
}

@lukeed @bucaran I also noticed that in the current concat function it's not actually outputting the source map file or embed the sourcemap into the file

@lukeed I thought we were going to drop support for source maps in 1.0. Good thing we didn't.
@tjbenton How hard would it be to wrap that into a PR our way :)

@bucaran We technically did by forcibly turning them off everywhere.
@tjbenton With the exception of concat, every other task that could use a sourcemap is almost always handled by the external library, and they do a better job that what ours did.
I suppose we could reenable them as an option on concat only, defaulting to false.


Retry 5xx errors

Just saw this error while waiting for job to finish:
Thu, 21 Apr 2022 12:56:52 GMT 2022-04-21 12:56:52 +0000 - download-buildkite-artifact -  INFO - Waiting for build 7596 to finish.
Thu, 21 Apr 2022 12:56:52 GMT 2022-04-21 12:56:52 +0000 - download-buildkite-artifact -  INFO - Build is in scheduled state.
Thu, 21 Apr 2022 12:57:23 GMT 2022-04-21 12:57:23 +0000 - download-buildkite-artifact -  INFO - Build is in running state.
Thu, 21 Apr 2022 18:40:00 GMT Traceback (most recent call last):
Thu, 21 Apr 2022 18:40:00 GMT   File "/action/download_artifacts.py", line 416, in <module>
Thu, 21 Apr 2022 18:40:00 GMT     commit, output_path):
Thu, 21 Apr 2022 18:40:00 GMT   File "/action/download_artifacts.py", line 277, in main
Thu, 21 Apr 2022 18:40:00 GMT     build = get_build(buildkite, org, pipeline, build_number)
Thu, 21 Apr 2022 18:40:00 GMT   File "/action/download_artifacts.py", line 64, in get_build
Thu, 21 Apr 2022 18:40:00 GMT     return buildkite.builds().get_build_by_number(org, pipeline, build_number, include_retried_jobs=True)
Thu, 21 Apr 2022 18:40:00 GMT   File "/usr/local/lib/python3.6/site-packages/pybuildkite/builds.py", line 255, in get_build_by_number
Thu, 21 Apr 2022 18:40:00 GMT     query_params=query_params,
Thu, 21 Apr 2022 18:40:00 GMT   File "/usr/local/lib/python3.6/site-packages/pybuildkite/client.py", line 131, in get
Thu, 21 Apr 2022 18:40:00 GMT     as_stream=as_stream,
Thu, 21 Apr 2022 18:40:00 GMT   File "/usr/local/lib/python3.6/site-packages/pybuildkite/client.py", line 82, in request
Thu, 21 Apr 2022 18:40:00 GMT     response.raise_for_status()
Thu, 21 Apr 2022 18:40:00 GMT   File "/usr/local/lib/python3.6/site-packages/requests/models.py", line 953, in raise_for_status
Thu, 21 Apr 2022 18:40:00 GMT     raise HTTPError(http_error_msg, response=self)
Thu, 21 Apr 2022 18:40:00 GMT requests.exceptions.HTTPError: 502 Server Error: Bad Gateway for url: https://api.buildkite.com/v2/organizations/horovod/pipelines/horovod/builds/7596?include_retried_jobs=True&per_page=100

Fixed in bff61b0.

Fix works:
2022-04-22 17:50:17 +0000 - download-buildkite-artifact -  INFO - Waiting for build 7617 to finish.
2022-04-22 17:50:17 +0000 - download-buildkite-artifact -  INFO - Build is in scheduled state.
2022-04-22 17:51:49 +0000 - download-buildkite-artifact -  INFO - Build is in running state.
2022-04-22 19:05:27 +0000 - download-buildkite-artifact -  INFO - Getting build 7617 failed, retrying in 30s.
Traceback (most recent call last):
  File "/action/download_artifacts.py", line 279, in main
    build = get_build(buildkite, org, pipeline, build_number)
  File "/action/download_artifacts.py", line 64, in get_build
    return buildkite.builds().get_build_by_number(org, pipeline, build_number, include_retried_jobs=True)
  File "/usr/local/lib/python3.6/site-packages/pybuildkite/builds.py", line 255, in get_build_by_number
    query_params=query_params,
  File "/usr/local/lib/python3.6/site-packages/pybuildkite/client.py", line 131, in get
    as_stream=as_stream,
  File "/usr/local/lib/python3.6/site-packages/pybuildkite/client.py", line 82, in request
    response.raise_for_status()
  File "/usr/local/lib/python3.6/site-packages/requests/models.py", line 960, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 502 Server Error: Bad Gateway for url: https://api.buildkite.com/v2/organizations/horovod/pipelines/horovod/builds/7617?include_retried_jobs=True&per_page=100
2022-04-22 19:34:35 +0000 - download-buildkite-artifact -  INFO - Getting build 7617 failed, retrying in 30s.
Traceback (most recent call last):
  File "/action/download_artifacts.py", line 279, in main
    build = get_build(buildkite, org, pipeline, build_number)
  File "/action/download_artifacts.py", line 64, in get_build
    return buildkite.builds().get_build_by_number(org, pipeline, build_number, include_retried_jobs=True)
  File "/usr/local/lib/python3.6/site-packages/pybuildkite/builds.py", line 255, in get_build_by_number
    query_params=query_params,
  File "/usr/local/lib/python3.6/site-packages/pybuildkite/client.py", line 131, in get
    as_stream=as_stream,
  File "/usr/local/lib/python3.6/site-packages/pybuildkite/client.py", line 82, in request
    response.raise_for_status()
  File "/usr/local/lib/python3.6/site-packages/requests/models.py", line 960, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 502 Server Error: Bad Gateway for url: https://api.buildkite.com/v2/organizations/horovod/pipelines/horovod/builds/7617?include_retried_jobs=True&per_page=100
2022-04-22 19:43:53 +0000 - download-buildkite-artifact -  INFO - Build is in passed state.


CANCELED exporting to docker image format: Error: exit status 1

Version of copa
V 0.5.1
Expected Behavior
Patched container image should created successfully.
Actual Behavior
I am following this YT video https://www.youtube.com/watch?v=DUt4wnKV4a8&t=5s  by @sozercan and running below commands:
$ pulling nginx:1.18.0 container image from DockerHub
sudo docker pull nginx:1.18.0
$ use trivy scan the nginx:1.18.0 container image saving the output to nginx.1.18.0.json
sudo trivy image --vuln-type os --ignore-unfixed -f json -o nginx.1.18.0.json nginx:1.18.0
$ use trivy to output the number of vulnerabilities in the nginx:1.18.0 container image
sudo trivy image --vuln-type os --ignore-unfixed nginx:1.18.0 | grep Total
$ use trivy to output the dpkg vulnerabilities in the nginx:1.18.0 container image
sudo trivy image --vuln-type os --ignore-unfixed nginx:1.18.0 | grep dpkg
$ verify the dpkg version in the nginx:1.18.0 container image
sudo docker run  nginx:1.18.0 dpkg --version
$ run buildkit in a container locally, we'll need it to run Copa
sudo docker run --detach --rm --privileged -p <IP_ADDRESS>:8888:8888/tcp --name buildkitd --entrypoint buildkitd moby/buildkit:v0.11.6 --addr tcp://<IP_ADDRESS>:8888
$ confirm the buildkitd container is running
sudo docker ps
$ use copa to patch the nginx:1.18.0 container image outputting the patched container image to nginx:1.18.0-patched
copa patch -i docker.io/library/nginx:1.18.0 -r nginx.1.18.0.json -t 1.18.0-patched -a tcp://<IP_ADDRESS>:8888
The last steps giving me below error:
[+] Building 29.7s (6/6) FINISHED                                                                                => CACHED docker-image://docker.io/library/nginx:1.18.0                                                                                0.1s
=> => resolve docker.io/library/nginx:1.18.0                                                                                0.1s
=> apt update                                                                                3.0s
=> sh -c apt install --no-install-recommends --allow-change-held-packages -y libc6 libgcrypt20 debian-archive-keyring dpkg libicu63 libssh2-1 libxpm4  22.7s
=> diff (apt update) -> (sh -c apt install --no-install-recommends --allow-change-held-packages -y libc6 libgcrypt20 debian-archive-keyring dpkg libic  0.1s
=> merge (docker-image://docker.io/library/nginx:1.18.0, diff (apt update) -> (sh -c apt install --no-install-recommends --allow-change-held-packages   0.0s
=> CANCELED exporting to docker image format                                                                                3.7s
=> => exporting layers                                                                                3.6s
=> => exporting manifest sha256:d5c3b92f1c697cb3c834e013bb2c148c2ab8ac2730d033327b7bb399c0f3fc6d                                                        0.0s
=> => exporting config sha256:fc8ffb3e67699b44db8e5801f52b249e6c1a573d6bab86bf8f8f52fbac5ea6fc                                                          0.0s
=> => sending tarball                                                                                0.0s
Error: exit status 1
Steps To Reproduce
On Ubuntu 22.04 OS:
$ pulling nginx:1.18.0 container image from DockerHub
sudo docker pull nginx:1.18.0
$ use trivy scan the nginx:1.18.0 container image saving the output to nginx.1.18.0.json
sudo trivy image --vuln-type os --ignore-unfixed -f json -o nginx.1.18.0.json nginx:1.18.0
$ use trivy to output the number of vulnerabilities in the nginx:1.18.0 container image
sudo trivy image --vuln-type os --ignore-unfixed nginx:1.18.0 | grep Total
$ use trivy to output the dpkg vulnerabilities in the nginx:1.18.0 container image
sudo trivy image --vuln-type os --ignore-unfixed nginx:1.18.0 | grep dpkg
$ verify the dpkg version in the nginx:1.18.0 container image
sudo docker run  nginx:1.18.0 dpkg --version
$ run buildkit in a container locally, we'll need it to run Copa
sudo docker run --detach --rm --privileged -p <IP_ADDRESS>:8888:8888/tcp --name buildkitd --entrypoint buildkitd moby/buildkit:v0.11.6 --addr tcp://<IP_ADDRESS>:8888
$ confirm the buildkitd container is running
sudo docker ps
$ use copa to patch the nginx:1.18.0 container image outputting the patched container image to nginx:1.18.0-patched
copa patch -i docker.io/library/nginx:1.18.0 -r nginx.1.18.0.json -t 1.18.0-patched -a tcp://<IP_ADDRESS>:8888
Error for above step:
INFO[0037] Validated package openssl version 1.1.1n-0+deb10u6 meets requested version 1.1.1n-0+deb10u6
[+] Building 0.0s (0/5)
=> docker-image://docker.io/library/nginx:1.18.0                                                                                0.0s
[+] Building 29.7s (6/6) FINISHED                                                                                => CACHED docker-image://docker.io/library/nginx:1.18.0                                                                                0.1s
=> => resolve docker.io/library/nginx:1.18.0                                                                                0.1s
=> apt update                                                                                3.0s
=> sh -c apt install --no-install-recommends --allow-change-held-packages -y libc6 libgcrypt20 debian-archive-keyring dpkg libicu63 libssh2-1 libxpm4  22.7s
=> diff (apt update) -> (sh -c apt install --no-install-recommends --allow-change-held-packages -y libc6 libgcrypt20 debian-archive-keyring dpkg libic  0.1s
=> merge (docker-image://docker.io/library/nginx:1.18.0, diff (apt update) -> (sh -c apt install --no-install-recommends --allow-change-held-packages   0.0s
=> CANCELED exporting to docker image format                                                                                3.7s
=> => exporting layers                                                                                3.6s
=> => exporting manifest sha256:d5c3b92f1c697cb3c834e013bb2c148c2ab8ac2730d033327b7bb399c0f3fc6d                                                        0.0s
=> => exporting config sha256:fc8ffb3e67699b44db8e5801f52b249e6c1a573d6bab86bf8f8f52fbac5ea6fc                                                          0.0s
=> => sending tarball                                                                                0.0s
Error: exit status 1
Are you willing to submit PRs to contribute to this bug fix?

[ ] Yes, I am willing to implement it.

@sozercan , please suggest?

I think I had a similar error some time back.
You can try running buildkit in other modes, rather than "Buildkit over TCP". Maybe try the "buildkit in a container" example (refer the docs). That seemed to have fixed the issue for my particular case.

I think I had a similar error some time back. You can try running buildkit in other modes, rather than "Buildkit over TCP". Maybe try the "buildkit in a container" example (refer the docs). That seemed to have fixed the issue for my particular case.

I tried to run below command to run Buildkit  without TCP:
sudo docker run --detach --rm --privileged --name buildkitd --entrypoint buildkitd "moby/buildkit:v0.12.0"
But when I tried to run below command, it's starts giving permission issue:
copa patch -i docker.io/library/nginx:1.21.6 -r nginx.1.21.6.json -t 1.21.6-patched --addr docker-container://buildkitd
Error: failed to get status: rpc error: code = Unavailable desc = rpc error: code = Unavailable desc = connection error: desc = "error reading server preface: command [docker exec -i buildkitd buildctl dial-stdio] has exited with exit status 1, please make sure the URL is valid, and Docker 18.09 or later is installed on the remote host: stderr=permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get "http://%2Fvar%2Frun%2Fdocker.sock/v1.24/containers/buildkitd/json": dial unix /var/run/docker.sock: connect: permission denied\n"

@ajmal-yazdani why run buildkit with sudo privileges when you can run without it? Probably that's the cause of the permission issue. In docs as well, it is shown to run without sudo privileges.
Just do:
docker run --detach --rm --privileged --name buildkitd --entrypoint buildkitd "moby/buildkit:v0.12.0"

and:
copa patch -i docker.io/library/nginx:1.21.6 -r nginx.1.21.6.json -t 1.21.6-patched --addr docker-container://buildkitd

@anubhav06, I am using Azure Ubuntu VM and problem is I can't install homebrew with root user and at the same time I can't use docker run without sudo.
I am seeing only documentation to install copa using `brew' only.
What's your thoughts here?

oh 🙂
then maybe try running copa also with sudo, since you are running docker with sudo

No Copa not running with sudo. Saying unknown command
On Sat, Nov 4, 2023, 7:31 PM Anubhav Gupta @.***> wrote:

oh 🙂
then maybe try running copa also with sudo, since you are running docker
with sudo
—
Reply to this email directly, view it on GitHub
https://github.com/project-copacetic/copacetic/issues/404#issuecomment-1793451890,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOHVAVBQ6NPFC53IKLOUPDYCZDEJAVCNFSM6AAAAAA63A5BF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGQ2TCOBZGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

huh :/
I'm no expert with Linux but you can try this:
sudo -E env "PATH=$PATH" copa patch ....
more details here

looks like this is duplicate of #452


[BACKLOG-29336] Updated sample to account for Sample DB user change.

@pentaho/bb-8 / @kcruzada

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.

Backports:
8.3 - #6634
<IP_ADDRESS> - #6635


Add OLED display saver feature

Adds a configuration option that allows you to define a time in minutes referring to the control inactivity timeout, and then the oled display will be turned off.
This is useful for those who always keep the controller connected to the computer, and some computers keep the USB powered when they are turned off.

Why is fsdata.c changed in this pr?

@bjszd Because there were changes in the "src/Pages/DisplayConfig.js" file of the web-config submodule. These changes are in this PR: https://github.com/OpenStickCommunity/WebConfigurator/pull/6

@arntsonl Oh right, I fixed that. Thanks


Shepherd: "sequence" not indexed

In https://github.com/heycam/webidl/commit/d99fdf9e6ac6e0682703d8a790569c901701a41f I export sequence and record. Record succeeded, but sequence did not. Is that because of the angle brackets in lt="sequence|sequence<T>"?
This blocks https://github.com/whatwg/fetch/pull/418.

Hmm, that shouldn't cause any problem. Plenty of CSS non-terminals use angle brackets, after all. Looking into it.

@plinss Yo, I can't figure this out, as things seem to work correctly locally.  https://heycam.github.io/webidl/#idl-sequence defines sequence and sequence<T> correctly, but it's not showing up in the db.

The anchor is in the DB, but just not as a definition, because there's no <dfn> tag there, just a heading (though it has a data-dfn-type attribute...)

sequence<T> is defined by https://heycam.github.io/webidl/#sequence-type and that seems to be in the DB ok (though maybe not with the linking text you're expecting because that got placed on the heading before...)

@plinss Headings have been able to define definitions for quite a while. We discussed this and you updated Shepherd accordingly months ago.

Right you are, forgot about that, sorry.
Looking deeper I found the problem: the heading has the linking text of "sequence|sequence<T>", but then there's the other <dfn> within that section that also has the linking text of "sequence<T>" (form its text content). There's code that detects the duplicated linking text and when it conflicts with a heading dfn, the conflicting linking text gets deleted from the heading and the heading gets demoted from dfn to a regular heading.
Possible solutions are:

resolve the conflict by removing the sequence<T> linking text in the heading (or stop using the heading as a dfn and move the "sequence" linking text to the dfn)
resolve the conflict by removing the second dfn
I modify the duplicated linking text code to only demote the heading when all of its linking texts are deleted (currently any deletion triggers the demotion)

I'm not entirely sure if option 3 is safe, but it smells reasonable to me on the surface (then again, maybe not). The issue is that you do have two dfns in the same section trying to define the same thing, so do you really want to different dfns with different linking texts?
I suppose another option would be for the duplicated linking text code to continue to demote the heading, but move any leftover linking texts to the sub-dfn. This would move all the links to the non-heading dfn.

I do think you are correct that IDL is somewhat confusing how it handles these things. I can look into a solution there, though it's somewhat surprising to me how Shepherd works given that Bikeshed does not complain. Does Shepherd not consider lt attributes authoritative?

@plinss It's a bug in Shepherd, then, that it's paying any attention whatsoever to the text content of the DFN, as the element has an lt defining its linking text instead.
The DFN should only be defining the term "sequence type", while the heading defines the terms "sequence" and "sequence", per the Bikeshed definition contract.

Digging through it more I agree, there are a few cases where I do append the text content of the dfn into the linking text but those are all in legacy spec support (mostly for older SVG specs). Will have a fix momentarily.

Danke!

Fixed and the DB is updated.

Thanks!


Change nevermind plugin regex

Currently corobo says sorry, whenever there is nm or nevermind in a sentence(when corobo is first word), so make it a full match. It should only respond when we say corobo nm  or corobo nevermind.
label: type/regex difficulty/newcomer

I would like to work on this issue.
Can anyone assign me?

I'd like to take this up since there is no activity since 2 months.
Can I be assigned the same?
CC: @jayvdb

I want to work on this issue. Can I be assigned to this?

@Vamshi99
In the nevermind.py plugin, when we are setting the regex pattern, does it ignore that corobo is the first word of the command, and start to match just after it?