Add setMaxRows to IOperation

Need to use method operation.setMaxRows( number )
Issue: #41

The test "HiveClient.connect" > should use tcp connection by default:  Fails in the current master branch:
     TypeError [ERR_MISSING_ARGS]: The "options" or "port" or "path" argument must be specified

I've fixed the test setting port and path to validate that client.connectionProvider is a instance of TcpConnection

Codecov Report

Merging #42 (0cc7702) into master (27acece) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
- Coverage   93.32%   93.30%   -0.03%
==========================================
  Files          58       58
  Lines        1049     1045       -4
  Branches      107      107
==========================================
- Hits          979      975       -4
- Misses         38       39       +1
+ Partials       32       31       -1

Impacted Files
Coverage Δ

lib/connection/auth/helpers/KerberosStep.ts
71.73% <0.00%> (-0.61%)
:arrow_down:

lib/result/JsonResult.ts
80.64% <0.00%> (-0.31%)
:arrow_down:

lib/utils/WaitUntilReady.ts
100.00% <0.00%> (ø)

lib/connection/connections/TcpConnection.ts
76.92% <0.00%> (ø)

...onnection/auth/helpers/MongoKerberosAuthProcess.ts
100.00% <0.00%> (ø)

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 27acece...0cc7702. Read the comment docs.

Hi @drzippie ! Looks fine, the wierd thing that test failed, but seems to be correct. I'll merge and release changes soon. Cheers =)

@lenchv  ... sorry, seems to be needed execute a npm run build, and commit dist/contracts/IOperation.d.ts
I had erroneously assumed that dist was generated automatically

whoops, I also forget about this. I'll build and release it, no worries

whoops, I also forget about this. I'll build and release it, no worries


[doc] Improve metadata parsing doc

Follow up to https://github.com/huggingface/safetensors/pull/243
Changes:

Changed some wording. Added references/links when necessary. For example, https://developer.mozilla.org/en-US/docs/Web/HTTP/Range_requests because not many people know about range requests.
Added image for both lightmode and darkmode
Added small python example
Added example outputs to js and python scripts

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

The description is updated https://github.com/huggingface/safetensors/pull/245#issue-1713802353

followed upon in #248


tests do not run when starting with flutter run -t

If you execute any test including a vrouter with flutter run -t <path>, the navigation does not work. It works when running without showing the UI (flutter test). I tested this by just running this code from vwidget_test.dart
import 'package:flutter/material.dart';
import 'package:flutter/widgets.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:vrouter/vrouter.dart';

main() {
  testWidgets('Stacked VWidget', (WidgetTester tester) async {
    await tester.pumpWidget(
      VRouter(
        routes: [
          VPage(
            path: '/',
            pageBuilder: (LocalKey key, Widget child) =>
                MaterialPage(key: key, child: child),
            widget: Builder(
              builder: (BuildContext context) => TextButton(
                child: Text('VWidget1'),
                onPressed: () => VRouter.of(context).push('/settings'),
              ),
            ),
            stackedRoutes: [
              VPage(
                path: '/settings',
                pageBuilder: (LocalKey key, Widget child) =>
                    MaterialPage(key: key, child: child),
                widget: Text('VWidget2'),
              ),
            ],
          ),
        ],
      ),
    );

    await tester.pumpAndSettle();

    // We should start at '/'

    final vWidget1Finder1 = find.text('VWidget1');
    final vWidget2Finder1 = find.text('VWidget2');

    expect(vWidget1Finder1, findsOneWidget);
    expect(vWidget2Finder1, findsNothing);

    // Try navigating to '/settings'
    // Tap the add button.
    await tester.tap(find.byType(TextButton));
    await tester.pumpAndSettle();

    // The navigation should have been redirected to / because we popped instead
    // So only VWidget should be visible
    final vWidget1Finder2 = find.text('VWidget1');
    final vWidget2Finder2 = find.text('VWidget2');

    expect(vWidget1Finder2, findsNothing);
    expect(vWidget2Finder2, findsOneWidget);
  });
}

This is the error:
flutter: ══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
flutter: The following TestFailure object was thrown running a test:
flutter:   Expected: no matching nodes in the widget tree
flutter:   Actual: _TextFinder:<exactly one widget with text "VWidget1" (ignoring offstage widgets):
flutter: Text("VWidget1", dependencies: [DefaultTextStyle, MediaQuery])>
flutter:    Which: means one was found but none were expected

flutter run -t test/widget_test.dart does not work for you?
I just tried and it is working for me:

VRouter 1.1.0+20
Flutter: Channel master, 2.2.0-11.0.pre.51
Tested on chrome

What do you have ?

Just tested in chrome, that works.
I was testing on an IOS Simulator before, that still does not work.

This is the output of flutter doctor
[✓] Flutter (Channel stable, 2.0.4, on macOS 11.2.3 20D91 darwin-x64, locale
    en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2020.3.3)
[✓] VS Code (version 1.55.2)
[✓] Connected device (1 available)

• No issues found!

Oh well I'll try in IOS then. However you'll have to wait a week because I don't have my Mac with me right now, sorry about that

No problem, since the tests are working fine without a device, this is not a pressing issue. I wanted to report it anyway.

Hi, I finally looked into this issue !
From what I understand, we were just not doing the right thing.
I am not an expert of integration testing since I've only used unit and widget testing in flutter so far but here is what I understand (correct me if I'm wrong).
Flutter has 3 types of testing:

unit
widget
integration

The one we are interesting in is the 3rd one since we want to run the test on device. If I make an integration test and run it on device, it seems to work without issue.

See this example:

Your files should be:
test_app/
  lib/
  test_driver/
    app.dart
    app_test.dart

app.dart:

import 'package:flutter/material.dart';
import 'package:flutter_driver/driver_extension.dart';
import 'package:vrouter/vrouter.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return VRouter(
      routes: [
        VPage(
          path: '/',
          pageBuilder: (LocalKey key, Widget child, name) =>
              MaterialPage(key: key, child: child, name: name),
          widget: Material(
            child: Builder(
              builder: (BuildContext context) => TextButton(
                child: Text('VWidget1'),
                onPressed: () => VRouter.of(context).push('/settings'),
              ),
            ),
          ),
          stackedRoutes: [
            VPage(
              path: '/settings',
              pageBuilder: (LocalKey key, Widget child, name) =>
                  MaterialPage(key: key, child: child, name: name),
              widget: Text('VWidget2'),
            ),
          ],
        ),
      ],
    );
  }
}

app_test.dart:

import 'package:flutter_driver/flutter_driver.dart';
import 'package:test/test.dart';

void main() {
  group('Stacked routes test', () {
    final vWidget1Finder = find.text('VWidget1');
    final vWidget2Finder = find.text('VWidget2');

    late FlutterDriver driver;

    // Connect to the Flutter driver before running any tests.
    setUpAll(() async {
      driver = await FlutterDriver.connect();
    });

    // Close the connection to the driver after the tests have completed.
    tearDownAll(() async {
      driver.close();
    });

    test("starts at '/'", () async {
      // Wait and find the text of the '/' page
      await driver.waitFor(vWidget1Finder);
    });

    test("Go to '/settings'", () async {
      // First, tap the button to navigate
      await driver.tap(vWidget1Finder);

      // Wait and find the text of the '/settings' page
      await driver.waitFor(vWidget2Finder);
    });
  });
}

You can run flutter drive --target=test_driver/app.dartwith you emulator open, and you will see the test execute and succeed.

If we consider the first two. I don't think we are supposed to unit/widget test on device.

A "proof" of that is the following test, made **without** vrouter, which does not work with `flutter run`

import 'package:flutter/material.dart';
import 'package:flutter/widgets.dart';
import 'package:flutter_test/flutter_test.dart';

main() {
  testWidgets('Stacked VWidget', (WidgetTester tester) async {
    var count = 0;
    await tester.pumpWidget(
      MaterialApp(
        home: Material(
          child: Builder(
            builder: (BuildContext context) => TextButton(
              child: Text('VWidget1'),
              onPressed: () => count++,
            ),
          ),
        ),
      ),
    );

    await tester.pumpAndSettle();

    expect(count, 0);

    // Find the button
    final button = find.byType(InkWell);
    expect(button, findsOneWidget);

    // Tap the add button.
    await tester.tap(button);
    await tester.pumpAndSettle();

    expect(count, 1);
  });
}

If you run this test with flutter run -t <path> you will see that it fails. Even "better", if you look at the error message, it fails for the same reason your test (with vrouter) was failing.
I think this proves that this is not the intended usage.

Now the true mystery is way your test (with vrouter) works on the web ?? I don't know but I'm not sure this is intended.
Please feel free to correct me if I've made a rookie mistake. As I said, I'm not an expert on integration testing.

So, I looked back into this and you are right. I am trying to run a "widget test" on a device which is not the intended way for testing.
As it says here it is supported at least for debugging

During a flutter run session on a widget test, you can also interactively tap parts of the screen for the Flutter tool to print the suggested [Finder].

I just tried again with a brand new flutter project and the default test. This also crashes on a device so the issue is not with vrouter. Maybe it is a flutter issue, it does not really matter because it is just for debugging tests and finders which it can. Thanks for looking into this, closing it now.


RealSense error calling rs2_load_json

dear support,
i've just upgraded librealsense on my pc from v 5.10.1 to 5.12
i'm getting the following error ( software used to work in previous version)
RealSense error calling rs2_load_json(dev:0xe957a0, json_content:0xe957e0, content_size:3513):
get_xu(id=2) failed! Last Error: Input/output error
RealSense error type Invalid Value.
i'm using d415,with the latest firmware (5.9.11).
i haven't changed my code or the input file.
here is the code:
rs400::advanced_mode MyDevice=MyPipelineProfile.get_device();
MyDevice.load_json(...file name....);
i've tried creating a new json config file using realsense-viewer , default settings -> download ,
but this didn't fix the problem.
the viewer is also unable to use a config file it generates.
( it can load the file, but then opening the rgb stream failes, with an error message about an invalid profile).
please advise.
Omer.

i've changed the logging level to info ,
here is the data printed
07/06 13:32:18,870 INFO [140471706744640] (environment.cpp:85) Found 4 unreachable streams, 1740 extrinsics deleted
07/06 13:32:18,958 INFO [140471706744640] (backend-v4l2.cpp:624) Trying to configure fourcc Z16
07/06 13:32:18,963 INFO [140471706744640] (backend-v4l2.cpp:624) Trying to configure fourcc YUYV
07/06 13:32:18,968 WARNING [140471519139584] (backend-v4l2.cpp:866) Empty frame has arrived.
07/06 13:32:18,968 WARNING [140471519139584] (backend-v4l2.cpp:866) Empty frame has arrived.
07/06 13:32:18,968 WARNING [140471519139584] (backend-v4l2.cpp:866) Empty frame has arrived.
07/06 13:32:18,968 WARNING [140471519139584] (backend-v4l2.cpp:866) Empty frame has arrived.
07/06 13:32:19,408 INFO [140471519139584] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:19,408 INFO [140471519139584] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:19,408 INFO [140471519139584] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:19,408 INFO [140471519139584] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:19,503 INFO [140471527532288] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:19,503 INFO [140471527532288] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:19,503 INFO [140471527532288] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:19,503 INFO [140471527532288] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:19,575 INFO [140471519139584] (metadata-parser.h:236) frame_number - last_frame_number 0
07/06 13:32:26,594 WARNING [140471706744640] (types.cpp:57) get_xu(id=2) failed! Last Error: Input/output error
hope this helps

[Realsense Customer Engineering Team Comment]
hi @omerbrandis,
would like to confirm that you are totally fine with 2.10.1 and not working with v2.12.0?
do you mean Realsense viewer save a new JSON file and can't load it back? or Realsense viewer can't read old JSON file?
i would like to reproduce this, so need to know which OS you are using?

Dear realsense customer engineering team.

I had a few problems with v2.10.1, and was instructed to upgrade.

I am now working with 2.12,
It did not solve the original problem i faced before
And it introduced the new problem that i described in this thread.

I am usung ubuntu 16.04

Realsense viewer cant load back a file that it itself has gendrated in version 2.12.
( file loads but later cant open the rgb stream).

+V5.12 Viewer "cant load" a file generated with 5.10.1 viewer.

my cpp code fails in r2_load_json for files generated with both the v5.10.1 and v
5.12 viewer.

Omer.

[Realsense Customer Engineering Team Comment]
hi @omerbrandis,
I think we should start from your #4. my realsense viewer from v2.12.0, can save JSON file and read it back.
and then RGB can't open is an known bug listed on v2.12.0, (DSO-9543).
you can enable Realsense viewer-->RGB camera-->Availale Streams-->Color, then you should able to start RGB streaming with your setting.
please let me know how it works, thanks.

Dear realsense customer engineering team.

I was not able to find details about the known bug DSO-9543.
can you please provide a link to detail information?

I did not understand your suggestion -
"you can enable Realsense viewer-->RGB camera-->Availale Streams-->Color, then you should able to start RGB streaming with your setting."

my real problem is not being able to use function  rs2_load_jsonj ,
I added the description for the error in the "VIEWER" because i guessed that they are caused by the same problem, and it's an easy way to prove that the error is in librealsense and not in my code.
( i don't require a workaround for the viewer's inability to load a json file )
thanks,
Omer.

[Realsense Customer Engineering Team Comment]
hi @omerbrandis,
please refer to https://github.com/IntelRealSense/librealsense/pull/1888.
please let me know how it works, thanks.

dear  Customer Engineering Team

i've reviewed #1188, but i'm afraid i did not understand it .
on the one hand  : " Root cause = Loading/Saving settings currently only supports stereo stream (Depth,IR)"
on the other " The fix: Disable only Depth/IR streams upon load settings and enable only requested streams afterwards"

this seems to contradict itself.

is the suggested solution a workaround ?
or will the api change permanently ?
( i would rather avoid performing a software change to workaround the problem.
i already have a simpler workaround - downgrading libreal sense to version that works )

thanks,
Omer.

[Realsense Customer Engineering Team Comment]
hi @omerbrandis,
the idea is to verify if Realsense viewer can read jason file at your side to verify if there's environment issue, where Realsense viewer from libRealsense v2.12.0 have known issue when reading jason file which fixed by pull request #1888.
so if Realsense viewer is ok to load jason file it generated, we can make sure the environment and .json file is no problem, so would suggest you to have a try with libRealsense v2.13.0 (have pull request #1888 integrated), to see if realsenxe viewer can load json file and if your code works too.
for your question 2), if you are referring pull request #1888, it's permanent fix.
and good to know you have a workaround so this issue not blocking you.
please let me know how it works, thanks.

hi
I intended to close this issue after i have tested v 2.13.0 and made sure it solves the problem.
but i will not be able to do this in the near future .
Thanks,
Omer.

I am having this exact same issue using librealsense v2.16.0 and Firmware version 5.10 on a D415 and a TX2 tegra device.
What is the fix?


Remove shadowing and add POM to generated artifact

This is a PR on top of https://github.com/sicpa-dlab/didcomm-jvm/pull/64

I migrated all 9.16-preview multi classes/interfaces to Kotlin and refactored the code such that stuff related to multiple recipients is now part of this project. Intentionally, I left all those step-by-step micro commits in place for you to verify what actually changed. Each of those commits should be valid (i.e. all tests pass). Feel free to squash if you like.


MLPerf not working for Resnet50 model from Model Zoo

MlPerf works fine for the following resnet50 model -
Zenodo Resnet50 : https://zenodo.org/record/2541184#.XUDRbeTsZaQ
But when I try the following model, it does not work -
Model Zoo Resnet50 : https://github.com/onnx/models/tree/master/vision/classification/resnet/resnet50
The reason is caused by the following line in main.py()
processed_results = self.post_process(results, qitem.content_id, qitem.label, self.result_dict)
This is returning a blank array.
The command I am using -
python python\main.py --profile resnet50-onnxruntime --model resnet50.onnx --dataset-path ImageNetImages\First_10 --output results.json --data-format NCHW --max-latency 0.2 --count 10 --accuracy
I had to change the profile because of different output -
"resnet50-onnxruntime": {         "dataset": "imagenet",         "outputs": "gpu_0/softmax_1",         "backend": "onnxruntime",         "max-latency": LATENCY_RESNET50,     },

those are not the same models. For mlperf you MUST use or (derive from) https://zenodo.org/record/2541184#.XUDRbeTsZaQ


Multiple Executors on Single Core Chips

I am currently working with an STM32H743 and would like to integrate multiple different priority levels in my program.
The approach I was hoping to use is multiple executors, as stated in the embassy documentation "You can also create the Executor manually, and you can in fact create multiple Executors."
Nowhere does it state that this is only possible for multi-core chips, but in playing around with it and from my understanding on how the executor should work in managing the underlying tasks, it seems like it is only possible to have one executor per core.
I was also not able to find anything in an example, that uses more than one executor either.
Could anyone elaborate on whether I'm correct or, if not, how to correctly use multiple executors on a single-core chip?

this is the example you're looking for https://github.com/embassy-rs/embassy/blob/main/examples/stm32f4/src/bin/multiprio.rs

this is the example you're looking for https://github.com/embassy-rs/embassy/blob/main/examples/stm32f4/src/bin/multiprio.rs

That's exactly it, thanks for the help. I was only looking in the H7 examples...


Normalize Markdown in .pot files

When mdbook-xgettext extracts translatable text, it would be great if it could normalize the strings. This would make it possible for us to reformat the entire course without fearing that the translations get destroyed while doing so.
The normalization would take Markdown like this
# This is a heading

This is another heading
=======================

A _little_
paragraph.

```rust,editable
fn main() {
    println!("Hello world!");
}
```

* First
* Second

and turn it into these messages in the .pot file:

"This is a heading" (atx heading is stripped)
"This is another heading" (setext heading is stripped)
"A _little_ paragraph." (soft-wrapped lines are unfolded)
"fn main() {\n    println!("Hello world!");\n}" (info string is stripped, we should instead use a #, flag)
"First" (bullet point extracted individually)
"Second"

Like in google/comprehensive-rust#318, we should do this in a step-by-step fashion and make sure to apply the transformations to the existing translations. It would also be good if we have a way to let translators update their not-yet-submitted translations.

I see the appeal, but I'm also wary of making it too easy to add too much code to a slide.  I think a wide-open .rs file in a text editor invites adding extra lines.

I don't think I'm going to get a chance to work on this issue soon.

That's alright, I'll look at it in the background and see what I can come up with.

@mgeisler I'm even hesitant, do you think I should wait a bit with rust-unofficial/patterns#359 and reinitialize the de.po-file when this feature is implemented?

I would not wait for this feature. My plan is to make it easy to move to a new version and keep the existing translations will continue to work. People (like yourself!) have put a lot of work into translations and we need to preserve this.
Concretely, I'm planning on providing a small program which people can run on the existing translations to normalize the Markdown found in them. So if the messages.pot file or a xx.po file contains
#: src/welcome.md:10
msgid ""
"* Give you a comprehensive understanding of the Rust syntax and language.\n"
"* Enable you to modify existing programs and write new programs in Rust.\n"
"* Show you common Rust idioms."
msgstr ""

Then the tool will turn the msgid field into three, one for each bullet point:
#: src/welcome.md:10
msgid "Give you a comprehensive understanding of the Rust syntax and language."
msgstr ""

#: src/welcome.md:11
msgid "Enable you to modify existing programs and write new programs in Rust."
msgstr ""

#: src/welcome.md:12
msgid "Show you common Rust idioms."
msgstr ""

The msgstr field will be split the same way, but in this case it was empty. You should upgrade to a new version of mdbook-i18n-helpers and run this program on the existing translations in a single step. Translators can run the same program on their in-progress work.

it feels a bit inefficient and super fiddly to just move two words in a sentence in the right place (due to \r \n), because the rest is somewhat fine.

I'm not sure why you have to fiddle with individual words like this? Markdown doesn't care about single newlines, so you can break your paragraphs any way you like in the msgstr fields.
msgstr ""
"Hallo,\n"
"ich bin\n"
"ein kleiner\n"
"Text."

The this will result in intermetiary Markdown looking like
Hallo,
ich bin
ein kleiner
Text.

and this in turn renders into HTML exactly the same way as if it the Markdown had been
Hallo, ich bin ein kleiner Text.

A different way to put this is that the translators are building up a full Markdown document, but doing it paragraph by paragraph. This implies that the translators must obey the Markdown formatting rules. If you mis-translate the above to
msgid "Hallo, I am a little text."
msgstr ""
"Hallo,\n\n"
"ich bin\n"
"ein kleiner\n"
"Text."

Then mdbook sees this Markdown
Hallo,

ich bin
ein kleiner
Text.

Now you have two paragraphs in the final book because of how Markdown uses empty lines to separate paragraphs:

Hallo,

ich bin
ein kleiner
Text.

Does that help to explain things?

It wouldn't factor in the second line of the translation as it only translates line by line, it seems?

Aha... thanks for explaining, now I understand what you mean!
I don't actually know how or if the line break influences the translations done with cloud-translate... in any case, I think removing the newlines would be something best left for that project.

So my assumption is, that if I would restart and regenerate the translation process, and send everything again via cloud-translate after formatting it would translate it better than now, ending up in less work overall.

From speaking to translators of the Rust course, I think people have had very mixed experiences with using cloud-translate: it gets a lot of things wrong because of the specialized context. Perhaps it works better for larger books, I'm not sure.
Feel free to open an issue about this over in https://github.com/mgeisler/cloud-translate — we could perhaps have the tool strip out things like \n and other formatting characters.

I wanted to add a fuzz test to ensure that #25 doesn't "invent" new Markdown events. However, this is proving more difficult than I thought since the underlying pulldown-cmark-to-cmark library isn't completely round-tripping the Markdown input. See https://github.com/Byron/pulldown-cmark-to-cmark/pull/55 for the discussion.

This was fixed with the 0.2.0 release! :rocket:  Please remember to run mdbook-i18n-normalize on our PO files to get the benefits of this.

@simonsan, thanks, you're very welcome! If you have a project which uses it, please add it to the README!


Merge remote-tracking branch 'refs/remotes/apache/master'

@leongu-tc this is an empty commit/pull request.  Could you please close this pull request.


UI API called on a background thread

Am I doing sth wrong?
=================================================================
Main Thread Checker: UI API called on a background thread: -[UIView layer]
PID: 17661, TID: 301320, Thread name: (none), Queue name: com.hackemist.SDWebImageDownloader (QOS: UNSPECIFIED), QoS: 0
Backtrace:
4   APPNAME                             0x0000000108e44e8b -[DACircularProgressView circularProgressLayer] + 43
5   APPNAME                             0x0000000108e4511b -[DACircularProgressView progress] + 43
6   APPNAME                             0x0000000108e5924c -[IDMZoomingScrollView setProgress:forPhoto:] + 316
7   APPNAME                             0x0000000108e52cad __42-[IDMPhotoBrowser configurePage:forIndex:]_block_invoke + 109
8   APPNAME                             0x0000000108e48a33 __40-[IDMPhoto loadUnderlyingImageAndNotify]_block_invoke + 195
9   APPNAME                             0x0000000108e67fcd -[SDWebImageDownloaderOperation start] + 2317
10  Foundation                          0x000000010d97dcff __NSOQSchedule_f + 369
11  libdispatch.dylib                   0x000000010f6cf43c _dispatch_client_callout + 8
12  libdispatch.dylib                   0x000000010f6d5856 _dispatch_continuation_pop + 967
13  libdispatch.dylib                   0x000000010f6d3c86 _dispatch_async_redirect_invoke + 780
14  libdispatch.dylib                   0x000000010f6db1f9 _dispatch_root_queue_drain + 772
15  libdispatch.dylib                   0x000000010f6dae97 _dispatch_worker_thread3 + 132
16  libsystem_pthread.dylib             0x000000010fb925a2 _pthread_wqthread + 1299
17  libsystem_pthread.dylib             0x000000010fb9207d start_wqthread + 13

Happening the same to me :(

I've checked some pull requests, found https://github.com/thiagoperes/IDMPhotoBrowser/pull/294 has fixed the issue.
So all you have to do is changing the podfile from
pod 'IDMPhotoBrowser'

to
pod 'IDMPhotoBrowser', :git => "https://github.com/rannger/IDMPhotoBrowser.git"

I tested the solution, it may fix the issue temporarily before the author fixing it officially.


[hotfix][dirty]print dirty data when dirty type is log for doris & ftp plugin

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
print dirty data when dirty type is log for doris & ftp plugin

PR merged.


Update footer to match Marketing website

[description]:

update 'Pricing' copy to 'Subscriptions'

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a42aea532339adcc549a7ffc4b9719b22e9b8bf6/


[all]  consistent testing across all jsstyles repositories

[ ] create a separate repository for tests, build, lint, which is essentially the same we have in jss repo
[ ] lint tests of plugins, convert them to es6
[ ] rewrite plugins tests to mocha

Maybe it's worth to follow monorepo approach before starting with this ?

@kof

lint tests of plugins, convert them to es6
So you want to test everything in node js env only?

not sure about monorepo, in our plugins based architecture it would make a wrong impression.

So you want to test everything in node js env only?

Why? We are testing now here in all browsers.

Why? We are testing now here in all browsers.

I mean, if you want to use es6 syntax, files should be transpiled to es5 somehow before passing them to browser

@kof

not sure about monorepo, in our plugins based architecture it would make a wrong impression.

Why? Babel is also plugin based tool and still has third party plugins.
So you have jss and core jss-* plugins, lets say there are 8 of them
9 npm packages
Now you want to add repo for crosstestings
You wan't be able test third party plugins there, until they get merged to jsstyles namespace.
In monorepo case:
You have one repo
All testings in it (also crosstestings)
Still 9 npm packages
Still can not test third party plugins
So in both cases it's problem of third party plugin developers to test and ship working plugins

If we provide one repo which has everything common for all jss repositories, it could be also used by third-party projects too and would lead to a consistent code base.
I could think of a way to run tests from all third-party plugins. However this is a separate topic. This would be useful to see if a release is going to break something and additionally notify plugin developer.

Of course we could do exactly the same in monorepo style too. I am just not sure if there is much of a benefit.

The biggest benefit I can see is being able to make pull requests with all changes for all plugins.

done


Can't install on Iris for Health

Hi,
I try to install zpm on iris for health with those commands :

cd /tmp

wget -q https://pm.community.intersystems.com/packages/zpm/latest/installer -O zpm.xml

iris session irishealth

USER>Do $system.OBJ.Load("/tmp/zpm.xml","ck")

Error :

Compiling routine %ZPM.PackageManager.Developer.ResourceReferenceAttributes.1
ERROR #6281: XMLTYPE of class %ZPM.PackageManager.Developer.Processor.CSPApplication must be able to differentiate child classes of %Installer.Activity.
ERROR #5090: An error has occurred while creating projection %ZPM.PackageManager.Developer.Processor.CSPApplication:XMLEnabled.
Detected 1 errors during compilation in 1.812s.
ERROR #6281: XMLTYPE of class %ZPM.PackageManager.Developer.Processor.CSPApplication must be able to differentiate child classes of %Installer.Activity.
ERROR #5090: An error has occurred while creating projection %ZPM.PackageManager.Developer.Processor.CSPApplication:XMLEnabled.
Detected 1 errors during load.
2019-09-30 11:35:43 0 %ZPM.Installer: ERROR #6281: XMLTYPE of class %ZPM.PackageManager.Developer.Processor.CSPApplication must be able to differentiate child classes of %Installer.Activity.
ERROR #5090: An error has occurred while creating projection %ZPM.PackageManager.Developer.Processor.CSPApplication:XMLEnabled.
2019-09-30 11:35:43 0 %ZPM.Installer: Installation failed at 2019-09-30 11:35:43
2019-09-30 11:35:43 0 %Installer: Elapsed time 4.49325s
Time spent 4.519202
Load finished successfully.

Tested on :

IRIS for UNIX (Ubuntu Server LTS for x86-64 Containers) 2019.2.0S (Build 260U) Thu Jun 13 2019 01:59:29 EDT

IRIS for UNIX (Apple Mac OS X for x86-64) 2019.1.0S (Build 111U) Wed Jan 16 2019 00:31:57 EST

well, I managed to reproduce the error on docker version intersystems/irishealth-community:2<IP_ADDRESS>9.0, so, later I'll try to fix it.

Hello @daimor,
I've the same problem.
There is two %Installer.Activity subclasses with the same XMLTYPE and XMLNAME parameters :

%ZHSLIB.PackageManager.Developer.Processor.CSPApplication.

I think, it would be easily fixed by a simple change

Version 0.1.0 of ZPM should now work on IRIS for Health as well


Add automated testing and packaging

Test every push.
If the tests pass, package and verify.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: RedTurtleDonkey:x: useruser seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Old.


Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[ ] iOS
[x ] Android
[ ] UWP
[ ] watchOS
[ ] tvOS
[ ] Tizen
[ ] .NET Standard

Feature Library

[ ] Core
[ ] BluetoothLE
[ ] Beacons
[ x] Locations
[ ] HTTP Transfers
[ ] Notifications
[ ] Sensors

Version of Library
1.1
Version of OS(s) listed above with issue
Steps to Reproduce
1.Create a project NOT using Shiny.ShinyAndroidApplication
2. Add other components as per
https://allancritchie.net/posts/shiny11

In the OnCreate method try and implement Shiny.AndroidShinyHost as per docs
https://shinydocs.azurewebsites.net/docs/gettingstarted

Expected Behavior
Allow implementation of Shiny.AndroidShinyHost or
ShinyHost.Init as per error
Actual Behavior
Does not allow Shiny.AndroidShinyHost
An error occurs on startup
Container has not been setup - have you initialized the Platform provider using ShinyHost.Init?
When i try and use ShinyHost.Init there is no Init method on ShinyHost
It seems that the option of not using the Shiny.ShinyAndroidApplication in the case that you already have your own has been disabled.

The docs are not done.  Follow the samples

AndroidShinyHost does exist for Android 9+ targets of you are going with your own application

I am using 10 but will have another look
I did try and find a sample that doesn't use Shiny.ShinyAndroidApplication and couldn't find one.  Maybe you could tell me which one please

I did a search for AndroidShinyHost in all samples provided and could not find one so I assume there are none.  Maybe it would be worth adding one for those not using the Shiny.ShinyAndroidApplication

@burrowj I'm pretty busy with things.  This isn't intended to be a support channel.  You are welcome to scan the source code for what you need.  It all exists.  AndroidShinyHost.Init is there.  You can also look at the ShinyAndroidApplication source code and see how it calls init under the hood.


Master solo

Hi, in Faderport V2, when master track is selected, if you solo it, you can't unsolo it whith the button.
Thanks for your work!

Will put this on the list


Parsing xml tag element

Hello,
we've an XML that is like this
<News item="23BA"  type="Glasses" date="01-02-2019">   <OPT> </OPT> </News>
we are able to parse all the xml (I just didn't paste all the xml because we are interested on getting the first elements).
We would like to obtain item, type and date.
we've wrote the following DDL :
CREATE EXTERNAL TABLE test(   item STRING,   type STRING,   date STRING )ROW FORMAT SERDE 'com.ibm.spss.hive.serde2.xml.XmlSerDe' WITH SERDEPROPERTIES ( "column.xpath.item"="/News/item/text()", "column.xpath.type"="/News/type/text()", "column.xpath.date"="/News/date/text()", ) STORED AS INPUTFORMAT 'com.ibm.spss.hive.serde2.xml.XmlInputFormat' OUTPUTFORMAT 'org.apache.hadoop.hive.ql.io.IgnoreKeyTextOutputFormat' LOCATION 'path to xml' TBLPROPERTIES ( "xmlinput.start"="<News>", "xmlinput.end"="</News>");
Then we  perform these procedures :

create DDL table with no problems
-put the XML file into the folder
-Select * from test

and it doesn't output nothing. We tried using the @ tag for each one of item,type and date like this /News/@item
Can you please tell us if this library can parse multiple elements tags ?
and if you could help us to understand why we cannot get these 3 elements?

This project is no longer maintained.

Is there an alternative ?


feat: update action input type

What are these changes for?

What are these changes for?

somehow my local always changes to this when I do git commit.

What are these changes for?

somehow my local always changes to this when I do git commit for this repo.

I was asking about the PR as a whole.


Problems compiling sketch

Not sure what's happening here.  I followed all the directions and I'm just trying to get the main sketch to compile but I'm getting errors with the ESP8266WiFi library.
(I tried putting this in a code block but there's too many single quotes in there for it to get formatted correctly.)
I tried updating to latest Arduino IDE and updating all boards and libraries but still getting the error.
Any idea what could be causing these errors?
C:\Users\Chris\Documents\Arduino\libraries\ESP8266WiFi\src\ESP8266WiFi.cpp: In member function 'void ESP8266WiFiClass::beginSmartConfig()':
C:\Users\Chris\Documents\Arduino\libraries\ESP8266WiFi\src\ESP8266WiFi.cpp:393:76: error: cannot convert 'sc_type' to 'sc_callback_t {aka void ()(sc_status, void)}' for argument '1' to 'bool smartconfig_start(sc_callback_t, ...)'
 smartconfig_start(SC_TYPE_ESPTOUCH, &ESP8266WiFiClass::_smartConfigDone);

                                                                        ^

C:\Users\Chris\Documents\Arduino\libraries\ESP8266WiFi\src\ESP8266WiFi.cpp: In member function 'bool ESP8266WiFiClass::smartConfigDone()':
C:\Users\Chris\Documents\Arduino\libraries\ESP8266WiFi\src\ESP8266WiFi.cpp:409:35: error: 'smartconfig_get_status' was not declared in this scope
 return smartconfig_get_status() == SC_STATUS_LINK_OVER;

                               ^

Multiple libraries were found for "ESP8266mDNS.h"
Used: C:\Users\Chris\Documents\Arduino\libraries\ESP8266mDNS
Not used: C:\Users\Chris\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266mDNS
Multiple libraries were found for "ESP8266WiFi.h"
Used: C:\Users\Chris\Documents\Arduino\libraries\ESP8266WiFi
Not used: C:\Users\Chris\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi
exit status 1
Error compiling for board NodeMCU 1.0 (ESP-12E Module).

Hi,
I just compiled the current verision successfully.
I had a similar problem with multiple libraries and had parallel installation of the arduino IDE (normal installation and portable version) - After removing all an a fresh clean installation I got it work. Sorry that I can't give more details.


rdma_rxe ibv_post_send soft lockup

Fixed by https://github.com/Nugine/rdma/compare/d2b3190aecc84d64d4616ae6a9f9f1b20ee2f052...cff469d8d3f0cf51490389666dbf7eae6188eca8.
The real cause is in the kernel module rdma_rxe. I don't know why it happens.
To reproduce:
git checkout d2b3190aecc84d64d4616ae6a9f9f1b20ee2f052
for i in `seq 1 100`
do
    just bench-pingpong-rc
done

$ uname -srv
Linux 5.4.0-113-generic #127-Ubuntu SMP Wed May 18 14:30:56 UTC 2022
$ pkg-config --modversion libibverbs
<IP_ADDRESS>
$ pkg-config --modversion librdmacm
<IP_ADDRESS>

https://www.rdmamojo.com/2013/01/26/ibv_post_send/

If this is an RC QP, verify that the rnr_retry value that was configured in ibv_modify_qp() isn't 7 since this may lead to retry infinite time in case of RNR flow


Terminal Variables

I happen to ssh into a VM for my development and I use SSH to ssh into that environment.  Is there a way to pass in the current directory, etc?  Normally I'd ask on something like stack overflow, but I expect something like this isn't implemented, since I've tried a number of different things.  For example, I'd like to have bash run a command to cd to a the directory that VS Code is in.  The Bash part isn't the issue, it's the Visual Studio Code part.  Variables mentioned elsewhere in the documentation do not appear to work for the terminal arguments, unless I am doing something wrong.

VS Code variables will resolve in the env setting, and you could then reference them in your shellArgs:
  "terminal.integrated.env.linux": {
    "MYCWD":"${workspaceRoot}"
  }

Also this is new in the latest v1.20 https://code.visualstudio.com/updates/v1_20#_variables-resolved-in-env-setting

Thanks!  Sorry I failed to follow up with this one.


Uncaught TypeError

Uncaught TypeError: Cannot read property 'contains' of null
at t._handleZoneBlur

I'm sorry but this basically says nothing?

Closing this issue as this error does not seem to be related to this project. If you can provide more details we can reopen the issue.

Whenever I select a tab, i get this error in the developer console (Chrome). The error is -
Uncaught TypeError: Cannot read property 'contains' of null
at t._handleZoneBlur
However, when clicked, the tabs change and the css gets applied correctly.

Can you provide a minimal working example that reproduces the error?


Add new search commands

Implement the latest search commands:

[ ] metadata search
[ ] accounts search
[ ] receipts search
[ ] namespaces search

@rg911 Is this solved? I've installed the latest version, but can't search metadata, accounts, receipts, nor namespaces yet.
The goal of this issue was to have a search command for the above-mentioned collections, in a similar way to:

https://github.com/nemtech/symbol-cli/blob/main/src/commands/block/search.ts
https://github.com/nemtech/symbol-cli/blob/main/src/commands/mosaic/search.ts

Reopened 👍


Create a list of TaleWorld Mission Events

For New Contributors
How to contribute
Description

Create a google doc with all registered events in MissionNetworkComponent.AddRemoveMessageHandlers()
Separate events by client and server.
Give each event a brief description. Feel free to group events and give them a shared description.
Later we will determine which events will be needed on our end.
Intended Design

N/A
Location

Your Google Drive
Related Issues

N/A
Requirements

N/A
Additional information

N/A
Definition of Done

[ ] List of events created
[ ] Each event has a short description or is grouped with a group description

No longer needed


Add rpath to jlcall (Mac OS X)

On Mac OS X (10.11.2) I could sucessfully compile jlcall but not run it due to the following error:
Invalid MEX-file '/.../jlcall/m/jlcall.mexmaci64':
dlopen(/.../jlcall/m/jlcall.mexmaci64, 1): Library not loaded: @rpath/libjulia.dylib
  Referenced from: /.../jlcall/m/jlcall.mexmaci64
  Reason: image not found

Error in Jl/get_mex_handle (line 58)
      jlcall('', Jl.julia_home, Jl.julia_image);

The solution of compiling with -rpath in #7 doesn't work under OS X and even setting the environment variables DYLD_LIBRARY_PATH and/or LD_RUN_PATH also doesn't appear to work. The solution I have provided is what is used by homebrew-julia so it should work on other OS X versions.

Thanks for the contributions. Can we say that jlcall is basically functional on OS X now?

I would say so.


Add bypass URI directive to Caddyfile

The bypass uri directive would allow to bypass authorization for specific path.
Dev note: reuse matcher from ACL.
For example,
https://subdomain.dulanic.com {
	import main
	route {
		jwt {
			bypass url prefix match /api
			trusted_tokens {
				static_secret {
					token_secret {env.token_shared_secret}
				}
			}
			allow roles verified
			auth_url https://auth.dulanic.com/auth/
		}
		reverse_proxy test:80 {
			import proxyheaders
		}
	}
}

See https://github.com/greenpau/caddy-auth-portal/issues/122#issuecomment-824866555

FYI, starting working on this feature.


Run 'make upgrade-klone' and 'make generate'

Manual Makefile modules upgrade, since the auto-upgrade failed: https://github.com/cert-manager/cert-manager/actions/runs/10417930573/job/28853077737
The following error can be seen there:
Error: pkg/client/clientset/versioned/typed/acme/v1/challenge.go:29:2: no required module provides package k8s.io/client-go/gentype; to add it:
	go get k8s.io/client-go/gentype

This error is caused by the difference between the client-go version and the generator version.
The new generators and client-go have to be upgraded at exactly the same time.
Bumping client-go resulted in also having to bump controller-runtime.
Lastly, these new k8s and c/r versions introduced some breaking changes that had to be fixed.
Kind
/kind cleanup
Release Note
NONE

/approve

Note: The upgrade to client-go here fixes the issue in #6753


Add ANSIBLE_RUNNER_DEBUG_CMD

Setting this env will allow the administrator to override the default ansible command to use with arbitrary ansible job commands. Potentially useful for mocked out testing for scale...

/test ansibleee-operator-build-deploy-kuttl
recheck

Please re-open if this is still needed.


GM Menu Button Inaccurate

Foundry 0.8.7, TA 4.1.2
The button is labeled "Force migration of current Scene", but it acts on the active scene, not the current scene.
As shown below, the first scene is the active scene, but the second is the current/visible scene.

I didn't even know that the active scene isn't the current viewed scene. I'll reword that in the next update

Changed in next update.


Comment directive conflicts with postgREST

@see https://postgrest.org/en/stable/api.html#openapi-support

Not sure whether there is anything we can do at all other than updating the postgREST. However want to bring attention to it in case there is a workaround.

I think this may need to move over to postgrest
there's no work-around we could introduce in pg_graphql that would impact how postgrest displays comments
cc @steve-chavez
everything pg_graphql related is scoped to the @graphql(<json>) block

I would expect some top level fields that all tools can choose to use. If we want to document our data model for example it's gonna be quite annoying to have to repeat ourselves once for graphql and once for rest.
Maybe we can borrow from markdown and yaml frontmatter, where it is possible to both have structured and unstructured (markdown) data?
COMMENT ON TABLE tbl $$

graphql:
  description: my graphql description
rest:
  settings: my rest settings
---
# My awesome table

- [ ] Will rock your socks off

$$;

The top level description would be the default, and it would be the value of the markdown body.

Btw I’m currently thinking of we proxy the openAPI response from pgRest and modify the document to parse the @graphql directive ourselves and expose that to clients. Though this really feels like something that ought to work at a supabase level given that rest & graphql are both natively supported on the platform, feels weird that there isn’t a way to document API across both of them at the same time easily.

Maybe we can borrow from markdown and yaml frontmatter, where it is possible to both have structured and unstructured (markdown) data?

That looks good, we had a similar proposal on postgREST before: https://github.com/PostgREST/postgrest/issues/2028#issuecomment-973832177.
Reference: https://assemble.io/docs/YAML-front-matter.html

Though this really feels like something that ought to work at a supabase level given that rest & graphql are both natively supported on the platform

True. Both tools need to agree on a format for this.

@steve-chavez currently the unexpected output is only visible in PostgREST 's OpenAPI doc so I think it make sense to continue this conversation over there (on https://github.com/PostgREST/postgrest/issues/2028#issuecomment-973832177 or a separate issue).
I'm sold on adopting a common format like yaml (or toml). It'd only take a couple days to implement backwards compatibly in pg_graphql so let me know when you're ready and we can knock out a spec and get it implemented in both


Enforce Package name approval for preview release

APIView returns 202 is package is not approved for preview release. Package must be approved (basic approval like package name) in APIView to release a package as preview if it was never approved or released as GA before.

@scbedd FYI. Python release for a preview version will start failing once this PR is merged if that package was never GAed before and require approval from architects. I will be sending out an email but it's a heads up in case if anyone reaches out about this error directly.


Pvoigt 880

Evaluate a point on a pseudo-Voigt based on the value of a motor.
Description
Adds pseudo-Voigt peak intensity model to bluesky.examples
Motivation and Context
This is a common peak shape function for X-ray diffraction.  The function provides a realistic signal for development, testing, and simulation.
How Has This Been Tested?

unit test test_pseudovoigt() added to bluesky/tests/test_examples.py
all tests pass

@danielballan, @tacaswell : code & unit test added (no obvious place to add documentation shown in #880), ready for review & merge

Codecov Report

Merging #885 into master will increase coverage by 0.07%.
The diff coverage is 97.56%.

@@            Coverage Diff             @@
##           master     #885      +/-   ##
==========================================
+ Coverage   85.34%   85.42%   +0.07%
==========================================
  Files          18       18
  Lines        4981     5022      +41
==========================================
+ Hits         4251     4290      +39
- Misses        730      732       +2

Impacted Files
Coverage Δ

bluesky/examples.py
80.13% <97.56%> (+1.29%)
:arrow_up:

bluesky/callbacks/scientific.py
62.03% <0%> (-0.93%)
:arrow_down:

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 93d07af...a92b956. Read the comment docs.

This looks dead useful. Currently, I'm in the throes of a major cleanup/rewrite of bluesky.examples for 1.0, so I would like to hold off on merging this until that is merged. I certainly wouldn't impose on you the chore of rewriting this on top of that; I'll gladly take that on myself.
Just now, I elaborated on the motivation and plan for said cleanup/rewrite in https://github.com/NSLS-II/bluesky/issues/692.

Looks like this should go into ophyd.sim now.  Withdrawing this PR now.

Yep! I'm still willing to help refactor it, but would prioritize the AD debugging that I owe you. :- )

Ok, refactored version is stashed until ready.


[2.1.1] flakey test: HubConnectionTests.StreamDoesNotStartIfTokenAlreadyCanceled

Consider taking https://github.com/aspnet/SignalR/issues/2034 for 2.1.1

This fix is the same as https://github.com/aspnet/SignalR/issues/2341. Closing this.


Dephy forcings

@leifdenby @cemac-ccs : Here is some preliminary work on the DEPHY forcing implementation. It will need further improvements, cleaning, testing and documentation before it can go in.
Notes:

I have also worked on makefile compilation here. This works on my laptop now, but compiling with SOCRATES is temporarily disabled.
One commit addresses a bug in the interpolation scheme, which is of wider use.
The DEPHY routine replaces the profile initialisation done by the grid manager, as well as setfluxlook, coriolis, and the "forcing" (from mcf) routines. I have implemented some sanity checks to check for inconsistent initialisation.
The routine should be run when MONC is in "entire domain" mode, not "column mode". I am working on a sanity check for this.

PS: apologies for all the whitespace changes.

With gitk it is possible to filter out the whitespace, but I haven't been able to do this on github so far. I think number 2 (the bugfix for interpolation), will be quick to implement as a separate bugfix and I can do this tomorrow. Number 1 (the makefile compilation) could be a separate pull request as well. However, it breaks existing functionality as the moment, so I would not pull this into the master branch right now. On the other hand, I am currently using it to compile MONC, though I think fcm works as well.

Another issue I have been thinking about is how to read in forcings that are in the "normal DEPHY" format, rather than our "IDEPHYX" format (Implémentation de DEPHY avec des eXtensions ). The difference between the formats is that IDEPHYX has some additional time-dependent fields, which are characterised by their name having the subscript "_traj". There is also a quirk in DEPHY that I currently haven't dealt with, which are inputs which can be either a string or an integer (I have simply used integers for these).
I think eventually, we would want to check whether the "_traj" variable is present, and otherwise use the non-"_traj" ones in fortran. Throw a wobbly when none or both are present. This will make the code a bit ugly. As a first step, I could write a script that converts DEPHY to IDEPHYX.


Session Id specific recordings can incorporate some new efficiencies

Session Id specific recordings should close automatically on the image becoming unavailable.

They can also be stopped early if necessary by the normal API. This may be because multiple publishers are keeping it active when not wanted.

Renamed to open up any new semantics we can come up with.

Archive recording that are session specific can be marked as such and when the Image closes then the recording subscription can be removed by tracking this in the session.

Close this for now as not immediately required. Can be reopened in the future if required.


Varnish with PageSpeed module

If configuring the stack Nginx (SSL 443) -> Varnish (6081) -> Nginx with PageSpeed (8080), how to serve .pagespeed. rewritten files? As I see Varnish returning the page with URLs rewritten but it's not found physically. Looks like it stays at 8080 VirtualHost that unavailable from public net.
Any solutions to workaround this?

Have you configured pagespeed + dowmstream cache?
Here config for Varnish 3 and 4 and for nginx acting as a proxy server too.
I never used this config, so my advice maybe not accurate.
I see a main issue:
Most of the optimizations depents of the browser UA and part of the UA string is in the pagespeed cache key, so you need to assure that the  UA from the final user traverse all the chain and reaches pagespped.
And varnish need to store all the optimized variants for a resource. I have found a varnish config file, but I think is for a nginx (ssl443) + pagespeed -> varnish -> nginx config.
Hope this help.

I tried this config but it does not help in my case. All the things from Downstream docs are already done but still the problem with the rewritten assets. I don't know, maybe something configured wrong.
But as you described the flow, maybe only possible way to do it by putting PageSpeed in a front of Varnish (i.e. on SSL 443 VirtualHost)? I tried this and it's work, but cost too high. Varnish without PageSpeed is 2400 RPS in my project, but if PageSpeed turned on the front of it something like only 110 RPS...

Maybe this can help #787  In this thread yuou can found a varnish config, and update to the downstream cache config docs that are undocumented and a sugestion to use LoadFromFile in the pagespeed conf file

I already tried all of those things, but I'm still receiving rewritten assets .pagespeed. that unavailable for the client.

You can get the file w/o rewrite?
The server where the file is, is running pagespeed?
I will say, most people do a pagespeed Domain Other.domain.com but in this domain, no pagespeed module is running.
For example, suposse you domain is mydomain.com and the hml is served from here.
in pagespeed you have set:
pagespeed Domain mydomain.com; (This run pagespeed)
pagespeed Domain otherdomain.com; (This not)
and in the html you include a resource from otherdomain.com, this resource get rewrited, cause their domain is authorized, but the rewrited resource don´t exists, cause is not running pagespeed.

So, you trying to say that the only way to run PageSpeed is by putting it on SSL Termination Vhost (443) before Varnish? Because I did it only on non-SSL Vhost (8080), thinking that Varnish can cache the ".pagespeed." rewritten assets as it caches the main page? Or there is a misunderstanding of something?

You can get the file w/o rewrite?
With this I´m asking if you are able to request
https://somedomain.com/media/magiccart/magicslider/p/o/pomyslnaprezent.png
In this request ou bypass all the pagespeed stuff. If you are unable to do it, pagespeed is not the problem.
The server where the file is, is running pagespeed?
I will say, most people do a pagespeed Domain Other.domain.com but in this domain, no pagespeed module is running.
For example, suposse you domain is mydomain.com and the hml is served from here.
in pagespeed you have set:
pagespeed Domain mydomain.com; (This run pagespeed)
pagespeed Domain otherdomain.com; (This not)
and in the html you include a resource from otherdomain.com, this resource get rewrited, cause their domain is authorized, but the rewrited resource don´t exists, cause is not running pagespeed.

This all is not related with the config you are using. Most people do a bad config of pagespeed and put, for example:
pagespeed Domain mydomain.com;
pagespeed Domain youtube.com;
Then, url from youtube in the html code, in say index.html, is rewriten in some like https://www.youtube.com/someimage.jpg.pagespeed.ce.HASH.webp but youtube site respond with a 404.
So I´m asking if these image that get the 404 error is in a domain other than the main html, and if yes, if in these doamin pagespeed is running.

Image is upon domain I'm using, but PageSpeed is turned off on 443 Vhost and turned on 8080.

And are you able to request the image url?
https://somedomain.com/media/magiccart/magicslider/p/o/pomyslnaprezent.png

Yes.

Well, I see this a test subdomain, so not in production, rigth?
I will say, you can do changes w/o users disruption.
If I were you, I would divide the problem into other minors.
1.- You make public the web on port 8080, so that, as if you were an end user, you could send requests. Or try another way to do request on it, like wget or curl
This is where you would have nginx + pagespeed, and in principle it should work w/o problems.
2.- If this step is ok, then intruduce varnis: varnis -> nginx(8080)+pagespeed, and see what happens.
3.- when the previous step is ok, then introduce the ssl termination.

It works on http://test.onlinemagento.com:8080/media/favicon/default/xfavicon.ico.png.pagespeed.ic.fnp-lzaWlj.png
but in the https;//test.onlinemagento.com it requests the SSL termination domain, not 8080.
Is there a workaround to have it under public host?

So, on 8080 it works: http://test.onlinemagento.com:8080/?PageSpeedFilters=+debug
On direct Varnish: http://test.onlinemagento.com:6081/?PageSpeedFilters=+debug it returns domain (test.onlinemagento.com) is not authorized, while pagespeed Domain test.onlinemagento.com; is added.
On 443 it's not found: https://test.onlinemagento.com/?PageSpeedFilters=+debug

Uummm when you add pagespeed Domain yourdomain.com that defaults to the port 80 and if you use the https (pagespeed Domain https://mydomain.com) default to the 443 port.
Maybe adding the port is needed, I don´t know cause never used ports other than defaults. but:
pagespeed Domain http*://.domain.com:
I have found this:
https://github.com/apache/incubator-pagespeed-mod/blob/b4bf44cc56d8bbf17494c540dfb6ef20dfcf5073/html/doc/announce-sec-update-201603.html#L105
si the * is permited in the port side.

Thank you, now the domain is authorized, but :8061 Varnish side is requesting some assets over https.

Thank you, now the domain is authorized, but :6081 Varnish side is requesting some assets over https.

Maybe these assets are bypassing varnish? I will say, request that varnish send to the backend w/o any change on it.

UPD: It looks like http://test.onlinemagento.com:6081/?PageSpeedFilters=+debug is working, but on 443 still not found.

So now if you request directly the varnish it work, rigth?
Then the next step is debug the comunication between ngin ssl to varnish.
Whats happens if the nginx ssl become non ssl? For testing purposes.
Ummm I tried to fecht this:
https://test.onlinemagento.com/static/frontend/Sm/ogalo/pl_PL/css/A.print.css.pagespeed.cf.gWpod0Sgg1.css
and seems that is magento that tries to search a file:
Unable to resolve the source file for 'frontend/Sm/ogalo/pl_PL/css/A.print.css.pagespeed.cf.gWpod0Sgg1.css'
<pre>#1 Magento\Framework\App\StaticResource->launch() called at [vendor/magento/framework/App/Bootstrap.php:257]
#2 Magento\Framework\App\Bootstrap->run() called at [pub/static.php:13]
</pre>

P.D.: I´m new to this stuff, never have tried this config nor used varnish, so I´m blind in this issue and maybe I´m proposing things that don´t make sense. Sorry for that.

I think Magento can't find it just because of Varnish that didn't cache these assets to get it back to SSL Vhost.

But when you hit varnish the asset is here, so varnish cached it.... and when you hit nginx 8080 it manages to resolve the .pagespeed. thing, no magento is involved to return the asset.
Recap on this:
1.- The client send a https request to nginx SSL
2.- The nginx SSL do a http request to varnish (is a redirect or a proxy_pass?)
3.- The varnish do a http request to the nginx 8080.
You have tried:
1.- Hiting nginx 8080 and it work w/o issues
2.- Hiting varnish and it works w/o issue ( is rigth?)
3.- Hiting nginx SSL don´t work.
So the questions I see here:
1.- Is the nginx SSL doing a http request to varnish?  Seems that yes cause if not the non rewriten resources returns a 404 too.
2.- So why the .pagespeed. get the 404? Maybe these resources have https when go to varnish and/or varnish send it untouched to nginx 8080?
3.- Why magento try to find a file with .pagespeed. ? This request sould be catched by pagespeed module and do the stuff to search it in the pagespeed cache.

Whats happens if the nginx ssl become non ssl? For testing purposes.

Make the nginx SSL respond to the http 80 w/o redirect to the 443 and sending the request to varnish and see whats happens.

SSL host is using proxy_pass.
This is my Vhosts:
`server {
listen *:443 ssl http2;
server_name test.onlinemagento.com;
index index.php;
pagespeed off;
set $user 'ogalo';
set $base /home/users/$user/public_html/pub;
root $base;

location ~ \.php$ {
	set $php '71';
	include settings/fastcgi/use;
}

include settings/hosting/magento2;
include settings/hosting/locations;
include settings/hosting/ssl_certs;
include settings/security/security_headers;

location / {
    proxy_pass http://<IP_ADDRESS>:6081;
    proxy_set_header Host $http_host;
    proxy_set_header X-Forwarded-Host $http_host;
    proxy_set_header X-Real-IP $remote_addr;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    proxy_set_header Ssl-Offloaded "1";
    proxy_set_header X-Forwarded-Proto https;
    proxy_set_header X-Forwarded-Port 443;
    proxy_redirect http://test.onlinemagento.com:8080/ /;
    proxy_http_version 1.1;
}

}
server {
listen *:8080;
server_name test.onlinemagento.com;
index index.php;
#pagespeed off;
set $user 'ogalo';
set $base /home/users/$user/public_html/pub;
root $base;

location / {
	try_files $uri $uri/ /index.php?$query_string;
	add_header 'Access-Control-Allow-Origin' '*' always;
}

location ~ \.php$ {
	set $php '71';
	include settings/fastcgi/use;
}

include settings/hosting/magento2;

include settings/pagespeed/optimize.conf;
pagespeed Disallow https://test.onlinemagento.com/admin/*;
pagespeed DownstreamCachePurgeLocationPrefix http://<IP_ADDRESS>:6081;
#pagespeed DownstreamCacheRebeaconingKey "fO95w6H2V9vAy5RN";
pagespeed Domain http://test.onlinemagento.com:8080;
pagespeed Domain http://test.onlinemagento.com:6081;
#pagespeed MapOriginDomain http://<IP_ADDRESS> https://test.onlinemagento.com;
#pagespeed LoadFromFile http://test.onlinemagento.com:8080 /home/users/ogalo/public_html/;

}`

The problem is the forwarding of the assets from Varnish to SSL Vhost.
It looks like Varnish caches only the page you joined, but not static files.
This works:
http://test.onlinemagento.com:8080/static/frontend/Sm/ogalo/pl_PL/css/A.print.css.pagespeed.cf.gWpod0Sgg1.css
But this not: https://test.onlinemagento.com/static/frontend/Sm/ogalo/pl_PL/css/A.print.css.pagespeed.cf.gWpod0Sgg1.css

Maybe you know that, but.... I have read that varnish don´t store things with cookies and your static assets are server with cookies.
And I have found a site that maybe you need to take a look on it:
https://www.getpagespeed.com/
In their blog have articles about varnish like:
https://www.getpagespeed.com/server-setup/varnish-static-files-cache

The problem is the forwarding of the assets from Varnish to SSL Vhost.
It looks like Varnish returning back only the page you joined, but not static files

So if varnish don´t have stored the asset it must forward the request to the backend server and then this must reply . But seems that when a .pagespeed. request comes from varnish to the backend, magento tries to find it in filesystem before pagespeed catch it.

But seems that when a .pagespeed. request comes from varnish to the backend, magento tries to find it in filesystem before pagespeed catch it.

But why it works on the server w/o varnish?
.pagespeed. assets will work with Magento if I'll turn on PageSpeed on front SSL Vhost for example.

In their blog have articles about varnish like:
https://www.getpagespeed.com/server-setup/varnish-static-files-cache

This article is about caching static files on the hard drive instead of RAM, but it's not about how to cache it at all. Theoretically, it has to be cached automatically in RAM by default.

These assets will work with Magento if I'll turn on PageSpeed on front SSL Vhost for example and it >works here as you can see: http://test.onlinemagento.com:6081/static/frontend/Sm/ogalo/pl_P/css/A.print.css.pagespeed.cf.gWpod0Sgg1.css

I can´t see it, cause you have a redirection from http to https.

These assets will work with Magento if I'll turn on PageSpeed on front SSL Vhost for example and it works here as you can see: http://test.onlinemagento.com:6081/static/frontend/Sm/ogalo/pl_PL/css/A.print.css.pagespeed.cf.gWpod0Sgg1.css

Yes, but you can see in it how cookies are removed. The statics assest where served with cookies, are these added by nginx SSL?
Have yoy tried to remove the ssl? I wil say, not the nginx server in front of varnish, but w/o using ssl.
To discard the problem is the ssl himself.

I can´t see it, cause you have a redirection from http to https.
Oрen it in the incognito window.

EDIT: The rationale to try this (no ssl) is that I think the protocol (http/https) is part of the key that pagespeed cache uses.
I'll try.

As I have supposed...
Have you configured that:
pagespeed RespectXForwardedProto on;

As I have supposed...
Have you configured that:
pagespeed RespectXForwardedProto on;

I removed that, now the SSL Vhost trying to request backend with рort 8080 directly and by HTTРS.

See, the guy tried to do the same thing as me and got failed at the same рroblem, then just рutted it on the front SSL Vhost:
https://github.com/apache/incubator-pagespeed-ngx/issues/1547
But at the front, using РageSрeed with Varnish is nearly useless.

My bet is for a protocol issue.
Have you tried to do all http only?

My bet is for a protocol issue.
Have you tried to do all http only?

Yeah, I tried this. Got the same result with HTTP.

Btw: this regex location doesn't work in Nginx for some reason.
location ~ "\.pagespeed\.([a-z]\.)?[a-z]{2}\.[^.]{10}\.[^.]+" {
    	return 200;
}


Example is broken with Android emulator

The example here: https://pub.dev/packages/flutter_speech#-example-tab-
doesn't seem to work on the android emulator. I just see this in the debug log:
D/FlutterSpeechPlugin( 8158): onRmsChanged : -2.0
D/FlutterSpeechPlugin( 8158): onRmsChanged : -2.0
D/FlutterSpeechPlugin( 8158): onRmsChanged : -2.0
D/FlutterSpeechPlugin( 8158): onRmsChanged : -2.0
D/FlutterSpeechPlugin( 8158): onRmsChanged : -2.12
D/FlutterSpeechPlugin( 8158): onError : 2
I/flutter ( 8158): _platformCallHandler call speech.onSpeechAvailability false
I/flutter ( 8158): _platformCallHandler call speech.onError 2
I/flutter ( 8158): _MyAppState.activateSpeechRecognizer...
D/FlutterSpeechPlugin( 8158): Current Locale : en_US
D/FlutterSpeechPlugin( 8158): onError : 5
I/flutter ( 8158): _platformCallHandler call speech.onSpeechAvailability false
I/flutter ( 8158): _platformCallHandler call speech.onError 5
I/flutter ( 8158): _MyAppState.activateSpeechRecognizer...
D/FlutterSpeechPlugin( 8158): Current Locale : en_US

Same issue while running in the emulator:
D/FlutterSpeechPlugin(18048): Current Locale : en_US
I/flutter (18048): _MyAppState.start => result true
D/FlutterSpeechPlugin(18048): onError : 9
I/flutter (18048): _platformCallHandler call speech.onSpeechAvailability false
I/flutter (18048): _platformCallHandler call speech.onError 9
I/flutter (18048): _MyAppState.activateSpeechRecognizer...
D/FlutterSpeechPlugin(18048): Current Locale : en_US

Update: Just set the "microphone" options in this emulator to allow the host mic input:

I'm not sure emulator is supported by the android API itself, but as real devices are ok I'll not spend time debugging emulator sorry


Documented Azure AD access token usage

The approach of omitting the "User ID" and using an Azure AD access token in the "Password" property is supported per PG but not documented, want to ensure customers understand this is a valid and supported mechanism.

#hold-off
Pending review.

Moving to private.


Java: Checker Framework, OWASP Dependency Check and SonarQube

Added three java static analysis tools

Thank you @levonk.
While SonarQube is written in Java it supports multiple languages such as C, PHP, Python and Javascript. Could we move that to the Multiple Languages section before I merge it? :smiley:

Decided to merge this in and move SonarQube with the next commit.
Thanks for the contribution @levonk 🚀


Any way to get full muted users list?

Before submitting an issue, make sure you have:

[x] Updated to the latest version
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones

Purpose of your issue?

[ ] Bug report (encountered problems/errors)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

The following sections requests more details for particular types of issues, you can remove any section (the contents between the triple ---) not applicable to your issue.

Describe your issue
Is there a way to get a list of users I've muted (both stories and posts)?
api.send_request('friendships/muted_reels') - gets a list of users with muted stories only.
I suggested that there should be an endpoint 'friendships/muted' similar to 'friendships/blocked' and 'friendships/blocked_reels' pair, but it's not.

I'm trying to address this in #1080 Not sure how to get list by muted posts? or by both stories or posts.


Plugin doesn't fix more than one copy of React issue

I have a react library and build it like this
const esbuild = require("esbuild");
const { externalGlobalPlugin } = require("esbuild-plugin-external-global");

esbuild
  .build({
    entryPoints: ["index.ts"],
    outdir: "dist",
    bundle: true,
    logLevel: "info",
    minify: true,
    sourcemap: true,
    splitting: true,
    format: "esm",
    platform: "browser",
    plugins: [
      externalGlobalPlugin({
        react: "window.React",
        "react-dom": "window.ReactDOM",
      }),
    ],
  })

And I have a react project with serve script
const esbuild = require("esbuild");

esbuild
  .context({
    entryPoints: ["./src/index.tsx"],
    bundle: true,
    outdir: "public/www/js",
    platform: "browser",
    format: "esm",
    sourcemap: true,
    splitting: true,
  })
  .then(
    (ctx) =>
      ctx.serve({
        servedir: "public",
        port: 8000,
      })
  )

When I try to start a project I see react Invalid hook call error

I assumed that this plugin should replace copies of react to global window.React. Or am I wrong?
How should I use this plugin?

You probably want to externalise React (and other packages), see https://github.com/evanw/esbuild/issues/1567#issuecomment-910276732
Adding the external option (👇) to your esbuild config should fix the issue:
external: ['./node_modules/*'],


createMutation doesn't do default values correctly

in createMutation we have the following code.
    // Merge in default Values here
    const item = {
      ...arrayToObject(
        this.fields.filter(field => field.getDefaultValue() !== undefined),
        'path',
        field => field.getDefaultValue()
      ),
      ...data,
    };
    ...
    const resolvedData = await resolveAllKeys(
      mapKeys(data, (value, fieldPath) =>
        this.fieldsByPath[fieldPath].createFieldPreHook(value, context, createdPromise.promise)
      )
    );

We notice that resolvedData is calculated based on the original supplied data, rather than the object with defaults included, item.
This means that the default values are silently dropped. We should write a test case which exposes this behaviour and then fix the issue.

This is fixed.


可以添加上RMBG-1.4模型嗎Can we add the RMBG-1.4 model

可以添加上RMBG-1.4模型嗎
Can we add the RMBG-1.4 model

We currently have no plan of doing so.


fix: removing unsupported modal sizes

SUMMARY

There were a few instances of react-bootstrap using bsSize of medium which isn't actually supported by React Bootstrap. This sets those to sm and locks down TS/PropTypes accordingly.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] 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

Codecov Report

Merging #10625 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10625      +/-   ##
==========================================
- Coverage   64.22%   64.21%   -0.02%
==========================================
  Files         778      778
  Lines       36709    36708       -1
  Branches     3461     3460       -1
==========================================
- Hits        23578    23572       -6
- Misses      13022    13024       +2
- Partials      109      112       +3

Flag
Coverage Δ

#cypress
54.12% <ø> (-0.34%)
:arrow_down:

#javascript
60.62% <ø> (-0.01%)
:arrow_down:

#python
59.78% <ø> (+0.03%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../src/SqlLab/components/EstimateQueryCostButton.jsx
17.39% <ø> (ø)

...tend/src/SqlLab/components/ScheduleQueryButton.jsx
8.62% <ø> (ø)

superset-frontend/src/components/DeleteModal.tsx
92.30% <ø> (ø)

superset-frontend/src/components/Modal.tsx
94.44% <ø> (-0.30%)
:arrow_down:

superset-frontend/src/components/ModalTrigger.jsx
100.00% <ø> (ø)

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

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

...rontend/src/SqlLab/components/QueryAutoRefresh.jsx
65.90% <0.00%> (-6.82%)
:arrow_down:

...rontend/src/SqlLab/components/TabbedSqlEditors.jsx
77.27% <0.00%> (-5.20%)
:arrow_down:

...rontend/src/SqlLab/components/SqlEditorLeftBar.jsx
44.00% <0.00%> (-4.00%)
:arrow_down:

... and 14 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 692266f...334d758. Read the comment docs.


Sorting by video length

What feature or improvement do you think would benefit Files?
Add sorting by video length
Requirements
Ability to sort files by video length
Files Version
<IP_ADDRESS>
Windows Version
11
Comments
No response

Thanks for the feedback, We're tracking this in #8366 . You can check there or click subscribe in the right sidebar for updates.

To expand on what Josh shared, we're planning to add support for all the columns and sorting options available in File Explorer.


Verify iTunes Receipt failed.

For background, iTunes Receipt is purchased receipt in iTunes(macOS/iOS). It's a binary file which encoded with pkcs#7.
I try to use this package to parse iTunes Receipt like:
b, err := ioutil.ReadFile("./test_receipt")

d, err := base64.StdEncoding.DecodeString(string(b))

pkcs, err := pkcs7.Parse(d)

pkcs.Verify()

And it always return crypto/rsa: verification error when verify. Even I can parse data with pkcs.Content through asn1.
I can verify the file with openssl OK.
Is there anything I'm wrong or missing when verifying?
Attached test file: test_receipt.zip

Hey, looks like you're doing everything right. I was able to reproduce the issue using your sample receipt.
It looks like we don't handle cases where the AuthenticatedAttributes are omitted from the signature. Simple enough to fix.
Thanks for the report!

@googollee I had to push a quick fix since I missed a hunk in the git commit. If you tried it out before the last 5 minutes or so, try again now.
Side note, I've now setup Travis CI to prevent this in the future.

Great. Thank you for quick fix. I just try and it's works now.


alt-date widget doesn't support yearsRange option.

Looks like the cdn link given in the documentation doesn't contain the fix, I can see that the https://github.com/mozilla-services/react-jsonschema-form/pull/917 is already merged and this feature is already there in master branch.
It is also not working on the live playground athttps://mozilla-services.github.io/react-jsonschema-form/

@rohitbishnoi that's because no new release on this lib has been done after the merge or #917, while documentation is always "up-to-date" (although this can be confusing indeed).

Can we do something about this?
Looks like that the package version is also not updated, https://github.com/mozilla-services/react-jsonschema-form/blob/e0192b75d24024370d52a43293931c8ddd769752/package.json#L3
The current version is 1.0.3 in the master branch which contains the fix but the CDN link for the same version is not updated https://github.com/mozilla-services/react-jsonschema-form/blob/e0192b75d24024370d52a43293931c8ddd769752/package.json#L3


Add basic authentication capabilities to Pulsar SQL

Motivation
Allow the presto connector to authentication with a Pulsar Cluster.

rerun cpp tests

rerun integration tests

@sijie PTAL

rerun java8 tests

run cpp tests

run cpp tests

run cpp tests

mark this as 2.4.1 for conflict #4890


Checked some tests, added TrueColor internal field ,transparency can …

…be changed TODO: Stream I/O, add test images and exception strings to resources

Nice!


Bump ipnetwork to 0.20

Similar to #1426, bumps the ipnetwork dependency to 0.20.
Diff: https://github.com/achanda/ipnetwork/compare/v0.19.0...v0.20.0

Hello, is there anything else to do from my side?

This has already been done on 0.7-dev.


gulpfile.babel.js does not work in subdirectories

Before you open this issue, please complete the following tasks:

[x] use the search bar at the top of the page to search this repository for similar issues or discussions that have already been opened.
[x] if you are looking for help from the gulp team or community, open a discussion.
[x] if you think there is a problem with the plugin you're using, open a discussion.
[x] if you think there is a bug in our code, open this issue.

What were you expecting to happen?
gulpfile.babel.js in a subdirectory should be able to use ES6 imports.
What actually happened?
"SyntaxError: Cannot use import statement outside a module"
Please give us a sample of your gulpfile

import { getAssetsSubdir } from '@internal/build-utils';

const gulp = require('gulp');

Terminal output / screenshots

$ pwd
/cygdrive/c/Projects/ourProject/packages/shared/viewer-component
$ gulp copyAssets
C:\Projects\ourProject\packages\shared\viewer-component\gulpfile.babel.js:2
import { getRnAssetsSubdir } from '@internal/build-utils'; //
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at wrapSafe (internal/modules/cjs/loader.js:988:16)
    at Module._compile (internal/modules/cjs/loader.js:1036:27)
    at Module._compile (C:\Projects\ourProject\node_modules\pirates\lib\index.js:99:24)
    at Module._extensions..js (internal/modules/cjs/loader.js:1101:10)
    at Object.newLoader [as .js] (C:\Projects\ourProject\node_modules\pirates\lib\index.js:104:7)
    at Module.load (internal/modules/cjs/loader.js:937:32)
    at Function.Module._load (internal/modules/cjs/loader.js:778:12)
    at Module.require (internal/modules/cjs/loader.js:961:19)
    at require (internal/modules/cjs/helpers.js:92:18)
    at requireOrImport (C:\Users\Remie\AppData\Roaming\npm\node_modules\gulp\node_modules\gulp-cli\lib\shared\require-or-import.js:19:11)

Please provide the following information:

OS & version [e.g. MacOS Catalina 10.15.4]: Windows 7
node version (run node -v): 12.20.1
npm version (run npm -v): 6.12.x
gulp version (run gulp -v): 4.0.2
gulp-cli version: 2.3.0

Additional information
gulpfile.babel.js works great at the root of the project, but fails in subdirectories in our monorepo. Any way to make this work? Thank you!

Moving to GitHub Discussions. Issue tracker is not used for questions.


Fix list from cluster

Issue #, if available:
Description of changes:
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.11

/approve
/lgtm


Remove iam-token-service from .status.service

The iam-token-service was removed in #588, so it no longer needs to be retrieved for reporting the health of IM.

/retest

/retest


Unit test TestHealRemoteNSE is unstable

Expected Behavior
TestHealRemoteNSE is stable
Current Behavior
TestHealRemoteNSE is unstable
Failure Logs
build_69220_step_113_container_0.txt
Steps to Reproduce
Run TestHealRemoteNSE 1e3 times

reproducible https://circleci.com/gh/networkservicemesh/networkservicemesh/82791

@edwarnicke , @nickolaev  can you please reopen this?


Make it package manager agnostic

Currently yarn required, what not suitable for pnpm monorepos

thank you, i've implemented it in different way using https://www.npmjs.com/package/detect-package-manager


python3Packages.hstspreload: 2021.6.21 -> 2021.6.28

Motivation for this change
Update to latest upstream release 2021.6.28
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.hstspreload
python39Packages.hstspreload


Branded Anaconda

We want to change the logos on the installer

My current idea is the add an intermediate step to the bib processs to cahnge the URL for the anaconde rpms to our so that it downloads our branding.
/usr/bin/bootc-image-builder manifest ghcr.io/centos-workstation/achillobator:latest --type iso in the contianer is the first step toward that I think the next is the run osbuild

need to swap out sources.org.osbuild.curl.items "sha256:0d66d826222fac96739c0b4ed7127f090cd24598f68c9a82df52f23f9c0e996e": {
"url": "http://mirror.freedif.org/CentOS/10-stream/AppStream/x86_64/os/Packages/anaconda-gui-<IP_ADDRESS>-1.el10.x86_64.rpm"
}
    with our URL and SHA


Kevintahn1993

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

/spam

/spam


Open Telemetry drops logs when renamed

One possible solution to support Open Telemetry's filelogreceiver would be to have a mode more like go-cronowriter by writing directly to the current "backup file" and having Filename be a symlink to it.  If this were done, then Open Telemetry could watch the backup files by a glob and would neither lose any lines written to the file which were not consumed before rotation, nor double-count lines if including the Filename in the glob.

https://github.com/natefinch/lumberjack/issues/115#issuecomment-739835573


add Swift 5.5 job

@swift-nio-bot test this please

There are no nightly 5.5 docker images at the moment. We need to either get them to come into existence or disable the 5.5 job until they do.

Disabled the 5.5. job

I'm seeing problems compiling the tests with a recent official trunk snapshot build of the toolchain on linux:
> ~/src/swift-DEVELOPMENT-SNAPSHOT-2021-05-26-a-centos8/usr/bin/swift build --build-tests -j 2
/home/butta/swift-nio/Tests/NIOHTTP1Tests/TestUtils.swift:276:25: error: value of type 'sockaddr_in' has no member 'addressDescription'
            return addr.addressDescription()
                   ~~~~ ^~~~~~~~~~~~~~~~~~
/home/butta/swift-nio/Tests/NIOHTTP1Tests/TestUtils.swift:279:25: error: value of type 'sockaddr_in6' has no member 'addressDescription'
            return addr.addressDescription()
                   ~~~~ ^~~~~~~~~~~~~~~~~~

Swift 5.4 builds the exact same commit 43fa768 without a problem, so there appears to have been a regression somewhere.

This appears to be a compiler error, as that method is made available in an extension. Can you report this at bugs.swift.org?

Sure.

Looks like @weissi already reported it: SR-14268 and SR-14590.

@yim-lee I believe the 5.5 builds should be able to be turned back on with the subsequent changes made in this patch.

@swift-nio-bot test this please

The 5.5 job was enabled but didn't get triggered for some reason. Should be ok now.


first build

{package} {version} {:snowflake:}
Destination channel: {Snowflake | defaults}
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_609kgkq8iw/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:15: avoid_noarch: noarch: python packages should be avoided
clone/recipe/meta.yaml:39: missing_description: The recipe is missing a description

===== ERRORS =====

clone/recipe/meta.yaml:39: missing_dev_url: The recipe is missing a dev_url
clone/recipe/meta.yaml:39: missing_license_family: The recipe is missing the about/license_family key.
clone/recipe/meta.yaml:39: missing_documentation: The recipe is missing a doc_url or doc_source_url
===== Final Report: =====
3 Errors and 2 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_01z0cvojnf/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:15: avoid_noarch: noarch: python packages should be avoided
clone/recipe/meta.yaml:41: missing_description: The recipe is missing a description

===== ERRORS =====

clone/recipe/meta.yaml:41: missing_license_family: The recipe is missing the about/license_family key.
===== Final Report: =====
1 Error and 2 Warnings were found


Update SuqsQuestState.cpp

Fixes #2  - Explicit cast to stop editor from throwing an error when using launcher version.

(P.S: I wasn't sure whether casting to the expected type or using the ::Failed from the no-need-to-be-casted original type was the right solution as I didn't go through all the codebase)

Oops, looked like that bug has been silently ignored for ages. Thanks!


respect text encoding specified in argument (#20795)

If explicitly specified, respect encoding parameter of AioHttpTransportResponse#text method to resolve the issue #20795.

Thank you for the contributing. We are reviewing the change and will keep you updated.

/check-enforcer override

/check-enforcer evaluate

/check-enforcer reset

/check-enforcer evaluate

Thanks for the contribution.
If you don't mind, I will close this one and use #20999 to track.

OK, I agreed.
Thanks to your review.


Sourcemaps not generated for jsx files. No ability to change SourceMapDevToolPlugin test option.

This makes handling jsx annoying, especially in Karma.  See
https://github.com/webpack/karma-webpack/issues/104

+1

As mentioned in the linked issue, the problem is at https://github.com/webpack/webpack/blob/5b5775f9e2fc73fea46629f2d6a3ed7a1f8424d3/lib/SourceMapDevToolPlugin.js#L35 . By the looks of it you should pass custom test that includes jsx somehow.

This issue was moved to webpack/webpack.io#151


YAML front matter

Gosh, a Rubocop complaint.
lib/github_changelog_generator/generator/generator_generation.rb:11:13: C: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
Other than that, it looks merge-ready to me.


Support http://www.smpte-ra.org/schemas/2067-2/2016

I noticed Photon does not support/sees as invalid this ApplicationId:
<cc:ApplicationIdentification xmlns:cc="http://www.smpte-ra.org/schemas/2067-2/2016">http://www.smpte-ra.org/schemas/2067-21/2016</cc:ApplicationIdentification>
But it does support:
<cc:ApplicationIdentification xmlns:cc="http://www.smpte-ra.org/schemas/2067-2/2013">http://www.smpte-ra.org/schemas/2067-21/2016</cc:ApplicationIdentification>
If it is because it is seen as invalid, I think it is an error.

<cc:ApplicationIdentification xmlns:cc="http://www.smpte-ra.org/schemas/2067-2/2013">http://www.smpte-ra.org/schemas/2067-21/2016</cc:ApplicationIdentification>

It looks like this combination is not possible since ST 2067-21:2016 requires conformance with ST 2067-2:2016.

I did some more testing and it seems the message that 2067-2/2016 & 2067-21/2016 combination was not supported may have been caused by something else. I will try to investigate a bit more.


Add event metadata support for JNI

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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

Project coverage is 16.27%.  Consider uploading reports for the commit 26a0771 to get more accurate results

Files
Patch %
Lines

...mazonaws/kinesis/video/producer/jni/Parameters.cpp
0.00%
60 Missing :warning:

...s/video/producer/jni/KinesisVideoClientWrapper.cpp
0.00%
32 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1160      +/-   ##
===========================================
- Coverage    16.48%   16.27%   -0.21%
===========================================
  Files           50       50
  Lines         7019     7109      +90
===========================================
  Hits          1157     1157
- Misses        5862     5952      +90

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


”Bălteni”, ”Bălteni-Deal”, ”Chetrești” are listed in the drop-down menu

Description:
When user writes ”Bălteni”, ”Bălteni-Deal”, ”Chetrești” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Bălteni"

Expected results
The location appears in the drop-down menu as "Sat Bălteni, VASLUI (Bălteni)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bălteni-Deal"

Expected results
The location appears in the drop-down menu as "Sat Bălteni-Deal, VASLUI (Bălteni)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Chetrești"

Expected results
The location appears in the drop-down menu as "Sat Chetrești, VASLUI (Bălteni)".


react gojs api + tests

based on this example
packaging configuration (npm)
travis configuration
unit testing with enzyme

The documentation will be done in a next PR

@maurelio1234 my latest commit fixes all the requested changes


what's the backend db of this?

what's the backend db of this?
can use git or mysql?

I don't quite sure I understand your question:

This is only a API wrapper that help you extract data from Wikipedia database, so itself doesn't have a database
Yes you can use Git or any NoSQL you wnat for your project
Hope this answer your question


Add rel="canonical" support

Checklist

[X] I have read and accepted the contributing guidelines.

Is your feature request related to a problem? Please describe
I can't link reposts of articles I wrote elsewhere
Describe the solution you'd like
Add an option to the metadata for canonical links to enable reposting of content I have created elsewhere.
Describe alternatives you've considered
No response
Additional context
No response

@stemarks Please refer to jekyll-seo-tag documentation as this possibility seems to be available already.


Basic Hello World code in react js

Also added bolierplate code to run ReactJS

Hii, we are currently only adding Hello world and other programs from any language. React.js is not a language but a framework, so it can't be included here!
Thanks for the effort, you can create a PR for adding other programs.

I understand how we have to add languages so I have created a request for
the language Algol.
On Fri, 14 Oct 2022 at 10:22 AM, Priyanshu Singh @.***>
wrote:

Hii, we are currently only adding Hello world and other programs from any
language. React.js is not a language but a framework, so it can't be
included here!
Thanks for the effort, you can create a PR for adding other programs.
—
Reply to this email directly, view it on GitHub
https://github.com/GDSC-SGGSCC/Hacktoberfest-22/pull/131#issuecomment-1278481026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVHZMDJDAOYUSAQMC2ONJTDWDDRIZANCNFSM6AAAAAAQ7TA2TY
.
You are receiving this because you were mentioned.Message ID:
@.***>


[Off-Chain] Add support to REST RPCType

Summary
This PR uses shannon-sdk RPCType detection and error formatting to support REST.
It also improves RelayMiner and AppGateServer internal errors formatting (based on content type) and forwarding to the client
It can be tested on LocalNet by enabling ollama in localnet_config.yaml and issuing:
make send_rest_relay_sovereign_app

Issue

#511
#221
#572

Type of change
Select one or more:

[x] New feature, functionality or library
[ ] Bug fix
[x] Code health or cleanup
[ ] Documentation
[ ] Other (specify)

Testing
Documentation changes (only if making doc changes)

[ ] make docusaurus_start; only needed if you make doc changes

Local Testing (only if making code changes)

[x] Unit Tests: make go_develop_and_test
[x] LocalNet E2E Tests: make test_e2e
See quickstart guide for instructions

PR Testing (only if making code changes)

[ ] DevNet E2E Tests: Add the devnet-test-e2e label to the PR.

THIS IS VERY EXPENSIVE, so only do it after all the reviews are complete.
Optionally run make trigger_ci if you want to re-trigger tests without any code changes
If tests fail, try re-running failed tests only using the GitHub UI as shown here

Sanity Checklist

[x] I have tested my changes using the available tooling
[x] I have commented my code
[x] I have performed a self-review of my own code; both comments & source code
[x] I create and reference any new tickets, if applicable
[x] I have left TODOs throughout the codebase, if applicable

Merging despite of a failing E2E test that seems to happen only on CI.