[integration_test] BUG: Taps are not propagated correctly to AndroidViews under v2.8

Hello, with the recent update of 2.8 (and 2.8.1), integration tests that have been running fine under 2.2 and 2.5 are now breaking for me.
I'm working on an app that displays a Mapbox map using the mapbox_gl package. This is implemented inside the package using AndroidView on Android. (See here for the actual implementation.)
I'm running Integration Tests on an Android 10 phone (or emulator) using the SDK's integration_test package in which a tap on a symbol is made and a UI change is verified.
Under Flutter 2.5, everything ran just fine.
With the upgrade to Flutter 2.8 (and 2.8.1), the tests fail. The tap on the AndroidView is not progressed to the underlaying code and thus no UI change is triggered.
Note: This only applies to integration tests in particular, a release or debug build of the app works just as expected when tested manually.
This issue is a major blocker in our development pipeline
Steps to Reproduce
I created a guide to create a minimal app that uses the mapbox_gl package. It openes a map and places two symbols on the map. When tapping a symbol, the title in the app bar changes with a name given to the symbol.
When you create the app using Flutter 2.5, the test will pass. Unter Flutter 2.8, the test will fail.
(Unfortunately, I'm not able to provide an app that uses the AndroidView directly. That's why I have to use the mapbox_gl package. However, the package depends on two tokens that have to be generated on mapbox.com that I cannot provide.)

Create an app using flutter create demo

Run flutter pub add mapbox_gl

Add integration_test to the pubspec.yaml:
 dev_dependencies:
  flutter_test:
    sdk: flutter
  integration_test:
    sdk: flutter

Generate SDK_REGISTRY_TOKEN and MAPBOX_ACCESS_TOKEN from mapbox.com. See mapbox_gl repo for details.
Export the SDK_REGISTRY_TOKEN in your shell and restart your IDE. The token is needed to download the Mapbox SDK during the build stage. The access token will be used directly in the code.

Set the Android minSdkVersion to 29

Replace the main.dart file and add the map_widget.dart file with the following code

main.dart
import 'package:demo/src/map_widget.dart';
import 'package:flutter/material.dart';

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

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

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      onGenerateRoute: (RouteSettings routeSettings) {
        return MaterialPageRoute<void>(
          builder: (BuildContext context) {
            return const MapWidget();
          },
        );
      },
    );
  }
}

lib/src/map_widget.dart
import 'package:flutter/material.dart';
import 'package:mapbox_gl/mapbox_gl.dart';

class MapWidget extends StatefulWidget {
  const MapWidget({Key? key}) : super(key: key);

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

class _MapWidgetState extends State<MapWidget> {
  String title = "Initial title";
  late MapboxMapController _controller;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(title),
      ),
      body: MapboxMap(
        accessToken:
            'pk.XXXXXXXXXXX_PLACE_ACCESS_TOKEN_HERE_XXXXXXXXXXX',  // <-- Place your Access Token here
        initialCameraPosition: const CameraPosition(
          target: LatLng(0, 0),
        ),
        onMapCreated: (controller) {
          _controller = controller;
          _controller.onSymbolTapped.add(_onSymbolTapped);
        },
        onStyleLoadedCallback: onStyleLoaded,
      ),
    );
  }

  void onStyleLoaded() {
    _controller
      ..addSymbol(
        const SymbolOptions(
          geometry: LatLng(0, 0),
          iconImage: 'airport-15',
          iconSize: 10,
        ),
        {'name': 'Large Airport'},
      )
      ..addSymbol(
        const SymbolOptions(
          geometry: LatLng(0, 100),
          iconImage: 'airport-15',
          iconSize: 5,
        ),
        {'name': 'Small Airport'},
      );
  }

  void _onSymbolTapped(Symbol symbol) {
    debugPrint("onSymbolTapped ${symbol.id}");
    setState(() {
      title = symbol.data!['name'];
    });
  }
}

Add the app_test.dart file for the integration tests.

integration_test/app_test.dart
import 'package:demo/main.dart' as demo;
import 'package:demo/src/map_widget.dart';
import 'package:flutter/foundation.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:integration_test/integration_test.dart';

void main() {
  IntegrationTestWidgetsFlutterBinding.ensureInitialized();
  group('Integration test', () {

	// This test passes
    testWidgets(
      "should have initial title",
      (WidgetTester tester) async {
        demo.main();
        await tester.pumpAndSettle();

        expect(find.text('Initial title'), findsOneWidget);
      },
    );

	// This test fails under Flutter 2.8.1, but passes under 2.5.3
    testWidgets(
      "should have 'Large Airport' title after tap on airport icon",
      (WidgetTester tester) async {
        demo.main();
        await tester.pumpAndSettle();

        // We tap the center of the widget because we know that the large airport is there
        final center = tester.getCenter(find.byType(MapWidget));
        debugPrint(center.toString());

        await tester.tapAt(center);
        await tester.pumpAndSettle();
        await Future.delayed(const Duration(seconds: 3));
        await tester.pumpAndSettle();
        await Future.delayed(const Duration(seconds: 5));

        expect(find.text('Large Airport'), findsOneWidget);
      },
    );
  });
}

Logs
Flutter 2.8.1
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.0.1 21A559 darwin-x64, locale
    en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version
    31.0.0-rc4)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] Android Studio (version 2020.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.3)
[✓] VS Code (version 1.63.2)
[✓] VS Code (version 1.64.0-insider)
[✓] Connected device (2 available)

• No issues found!

Flutter 2.5.3
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, 2.5.3, on macOS 12.0.1 21A559 darwin-x64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc4)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] Android Studio (version 2020.3)
[✓] Android Studio (version 2020.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.3)
[✓] VS Code (version 1.63.2)
[✓] VS Code (version 1.64.0-insider)
[✓] Connected device (2 available)

• No issues found!

flutter analyze did not find any issues.

@darshankawar
I tried to replicate the behavior myself, but wasn't successful, i.e. my implementation using AndroidView with a native Android button passed the integration test. The problem with mapbox_gl persists unfortunately, so there was definitely a breaking change in that regard between 2.5 and 2.8. As far as I know, mapbox_gl uses AndroidView, so I don't know about SurfaceAndroidView.
I will investigate further at a later point, but at least you're aware of this problem in case you/your team get similar reports or find issues like this.

@bjoernahrens
Thanks for the update. I looked at the release notes for 2.8 specific to changes related to AndroidView and found out this PR which changed id to viewId for AndroidViewController` and others: https://github.com/flutter/flutter/pull/90294
But this doesn't seem to have any impact on your case.
Since your scenario works with AndroidView without using the plugin, could it be that there's something in the plugin itself ?
Have you reached out to the plugin owner to about it and see if it could be at their end ?

@darshankawar
Exactly, there's nothing in the release notes, so I assume it's an unintended bug.
I don't think it's the plugin itself. I checked out multiple versions of mapbox-gl that were released over the recent months. Every version worked fine with Flutter 2.2 and 2.5. Likewise, all those versions failed under Flutter 2.8.

Thanks for the update. If we take an example of webview_flutter, the latest version if it has SurfaceAndroidView() as default hybrid composition. Maybe the mapbox-gl package may need to incorporate this change ?

@darshankawar
Thanks for the tip. I cloned the mapbox-gl repo and implemented the AndroidViewSurface in place of the AndroidView myself to try it out.
Unfortunately, it did not change anything regarding the behavior. Taps on the map are still not recognized in integration tests, but work fine in release builds.

Thanks for the update. Since there doesn't seem to be an easy reproducible code sample to verify this, I am unable to confirm the behavior. (The code sample provided by @bjoernahrens would need to have access tokens generated and used in order to replicate the behavior).
I am labeling it for further input from the team.
/cc @jiahaog  @dnfield

Hi, I am a colleague of @bjoernahrens, doing some tests on this topic I have seen that the event map#onMapLongClick is produced but not symbol#onTap when doing tapAt(offset)... does it tell you something @darshankawar? Thanks in advance.

This is probably related to #111110


can't compile game

so i am using the 1.6.1 version of kade. and when i try to start up the game it says Source path "C:/HaxeToolkit/haxe/lib/extension-webm/git/ndll/Windows64/extension-webm.ndll" does not exist. and so i tried doing the lime rebuild extension-webm windows. but then another error occurs saying Error: mkvwriter.cpp
./libwebm/mkvwriter.cpp(9): fatal error C1083: Cannot open include file: 'mkvwriter.hpp': No such file or directory. PLEASE HELP

I SWEAR TO GOD NO ONE READS THE FUCKING GUIDE
read. https://kadedev.github.io/Kade-Engine/building

so i am using the 1.6.1 version of kade. and when i try to start up the game it says Source path "C:/HaxeToolkit/haxe/lib/extension-webm/git/ndll/Windows64/extension-webm.ndll" does not exist. and so i tried doing the lime rebuild extension-webm windows. but then another error occurs saying Error: mkvwriter.cpp
./libwebm/mkvwriter.cpp(9): fatal error C1083: Cannot open include file: 'mkvwriter.hpp': No such file or directory. PLEASE HELP

do you have the windows building dependencies installed

you could take one mods extension-webm.ndll and on the HaxeToolkit/haxe/lib/extension-webm/ndll/Windows, copy the .gitignore.txt then make a folder called Windows64 then paste the 2 files, done!!!

you could take one mods extension-webm.ndll and on the HaxeToolkit/haxe/lib/extension-webm/ndll/Windows, copy the .gitignore.txt then make a folder called Windows64 then paste the 2 files, done!!!

IT WILL WORK, I DID THIS TOO AND IT WORKED


Feat: re-enable the hot-reloading when working on the model

Closes #884, by importing local json files in dev mode instead of relying on a local http server.
Changelog

Import the co2-*.json, supportedRegions.json, personas-*.json and contenu-ecrit.json files in dev mode.
Remove the http-server dependency.

The removal of the difference loading the rules between dev and prod was intentional, in order to be more confident about the fact that what works in dev works in prod.
On the other hand, the past complexity is brought back, but only partially. I agree it's less complex since the processing of rules is now shared in ./nosgestesclimat instead of duplicate code between rulesToJSON and a block of code in RulesProvider, and differences essentially came from there.
What do you think @Clemog ? Do we really need the model hot reload ? If yes, we can go with this and just be careful about potential problems arising.

The removal of the difference loading the rules between dev and prod was intentional, in order to be more confident about the fact that what works in dev works in prod.

I agree, however, the behavior should be substantially the same.

If yes, we can go with this and just be careful about potential problems arising.

Do you have any idea of the problems that could arise?

I think it is a really nice improvement even if, I agree, we reintroduce different behavior between dev and prod but:

Rules processing is the same
Risk concerns mainly differences between public folder and local files. But CI is here to highlight compilation problems no ?

What do you think @Clemog ? Do we really need the model hot reload ? If yes, we can go with this and just be careful about potential problems arising.

I think it's important as refreshing page manually usually skip the question we are working on or display again any skipped banner, which is quite annoying

Risk concerns mainly differences between public folder and local files. But CI is here to highlight compilation problems no ?

To me, the behavior is the same -- modulo the fetch one. However, the files fetched in the previous version and the ones which are now imported are the same : they all come from nosgestesclimat/public.

Do you have any idea of the problems that could arise?

No, I'm trying to be cautious but I don't see any yet.


bevy_reflect: Replace "value" terminology with "opaque"

Objective
Currently, the term "value" in the context of reflection is a bit overloaded.
For one, it can be used synonymously with "data" or "variable". An example sentence would be "this function takes a reflected value".
However, it is also used to refer to reflected types which are ReflectKind::Value. These types are usually either primitives, opaque types, or types that don't fall into any other ReflectKind (or perhaps could, but don't due to some limitation/difficulty). An example sentence would be "this function takes a reflected value type".
This makes it difficult to write good documentation or other learning material without causing some amount of confusion to readers. Ideally, we'd be able to move away from the ReflectKind::Value usage and come up with a better term.
Solution
This PR replaces the terminology of "value" with "opaque" across bevy_reflect. This includes in documentation, type names, variant names, and macros.
The term "opaque" was chosen because that's essentially how the type is treated within the reflection API. In other words, its internal structure is hidden. All we can do is work with the type itself.
Primitives
While primitives are not technically opaque types, I think it's still clearer to refer to them as "opaque" rather than keep the confusing "value" terminology.
We could consider adding another concept for primitives (e.g. ReflectKind::Primitive), but I'm not sure that provides a lot of benefit right now. In most circumstances, they'll be treated just like an opaque type. They would also likely use the same macro (or two copies of the same macro but with different names).
Testing
You can test locally by running:
cargo test --package bevy_reflect --all-features

Migration Guide
The reflection concept of "value type" has been replaced with a clearer "opaque type". The following renames have been made to account for this:

ReflectKind::Value → ReflectKind::Opaque
ReflectRef::Value → ReflectRef::Opaque
ReflectMut::Value → ReflectMut::Opaque
ReflectOwned::Value → ReflectOwned::Opaque
TypeInfo::Value → TypeInfo::Opaque
ValueInfo → OpaqueInfo
impl_reflect_value! → impl_reflect_opaque!
impl_from_reflect_value! → impl_from_reflect_opaque!

Additionally, declaring your own opaque types no longer uses #[reflect_value]. This attribute has been replaced by #[reflect(opaque)]:
// BEFORE
#[derive(Reflect)]
#[reflect_value(Default)]
struct MyOpaqueType(u32);

// AFTER
#[derive(Reflect)]
#[reflect(opaque)]
#[reflect(Default)]
struct MyOpaqueType(u32);

Note that the order in which #[reflect(opaque)] appears does not matter.

Several of the types touched here are missing trait reflection attributes, primarily for Debug. Your call if you want to change it here.

Done!

totally in favour of the direction! i think opaque is a good name but i would also like to suggest Atom as in "indivisible unit" i think it's more slightly descriptive as long as we don't think people will confuse Atoms for atomic values.

Hmm, I like both. Opaque makes more sense as an annotation, but Atom feels better for primitives. I'll let y'all bikeshed this for a week, but if a decision is not reached by the next merge train (not today!) I'm going to merge this as is.

totally in favour of the direction! i think opaque is a good name but i would also like to suggest Atom as in "indivisible unit" i think it's more slightly descriptive as long as we don't think people will confuse Atoms for atomic values.

This is pretty good too. Like Alice said, this would help clear up the confusion with primitives.
My only concern is in documentation like you mentioned. We might write "returns an atom type"/"takes an atom value", but readers might see "returns an atomic type"/"takes an atomic value", especially since "atomic" is the adjective form.
For that reason, I think I lean a bit towards "opaque", but I'd love to hear if anyone else has any opinions on the matter! We can definitely go with "atom" if people prefer it.

And merge conflicts. I thought that might happen lol.


[mobile-client-build] Add support for cordova builds

Jira:
https://issues.jboss.org/browse/AEROGEAR-2772
Verification:

Provision Mobile CI/CD and attach an iOS node.
Create a new cordova application
Create a new build (Platform: Android, Build Type: Debug), start the pipeline and verify it completes.
Create a new build (Platform: Android, Build Type: Release), start the pipeline and verify it completes.
Create a new build (Platform: iOS, Build Type: Debug), start the pipeline and verify it completes.
Create a new build (Platform: iOS, Build Type: Debug), start the pipeline and verify it completes.

This cordova application repo and branch can be used for testing purposes https://github.com/feedhenry-templates/helloworld-app/tree/5xjenkinsfile2

Closing for new PR https://github.com/aerogear/origin-web-console/pull/63. All changes according to review will be included here.


typo on the 01_preface.md page

Hi Michael,
There's a typo on the 01_preface.md page (extra word "to"):
There have been many different workarounds for to this problem
should be:
There have been many different workarounds for this problem
Thanks for putting together the article. This WSL is very useful.
It's great for running the "lftp" program, to speed up downloads
from remote servers by using multiple connections, e.g.
lftp -c "mirror --use-pget-n=4 --verbose sftp://username:passwword@hostname/pathname"
Cheers,
Ned.

Another couple of typos:
in it's POSIX environment
should be:
in its POSIX environment
"Its" is the possessive form of "it".
so it cannot run any programs that need one, IE: Browser
should be:
so it cannot run any programs that need one, i.e.: Browser
("IE" might be confused with "Internet Explorer")

Thanks for this! I've  just now had a chance to read through it. I will take a deeper look at this over the next two weeks and try to answer as many of these items as I can.
Since you still have context here would you be able to make PR that we can look over:
Yes on all the typos, and the password change looks good too, but I'll need to take a longer look at the rest. I also probably overuse sudo which is why you'll see it used incorrectly. If you understand how to use sudo well then please update those sections too.
Also, my knowledge about Linux and Windows isn't that deep yet and I may not be the best person to answer some of these questions, so I open them to the community to answer as well. If you are confident that you know the topic, please add it in the PR as well.
Thanks again for taking the time to look into this at that level! Let me know if you need anything or if there is anything I can do to help you.

Updating the title:
Before:
**typo on the 01_preface.md page **
After:
[P1][01_preface.md][Typos][Content] Content enhancements and Questions.
Is that an appropriate title?


Possible to patch version number?

Comment:
Hi. Is it possible to patch the version number of a release of a package? (e.g. there is an extra character that got into a SemVer package version that shouldn't be there and is confusing solvers). If so, it is unclear how to do it from the then section in the recipe/README.md.
I think I need a patch along the lines of the following, but I don't know the proper keyword to use to overwrite the version
if:
  name: sherpa
  version: v2.2.16
then:
  - replace_???:  # what key should be used here
      old: ${version}
      new: 2.2.16

No, it is not. The version number is in the package filename and so cannot be patched. Best to mark it broken and rebuild.


Different parsing in IE8 vs. Firefox v34.05

In Firefox, the head from my XML document will be parsed like this:
        "xmlns:xsi": {
          "_value": "http://www.w3.org/2001/XMLSchema-instance",
          "_ns": "http://www.w3.org/2000/xmlns/"
        },

in IE8 the head from the XML document will be parsed like this:
      "_attr": {
        "xmlns:xsi": {"_value": "http://www.w3.org/2001/XMLSchema-instance"},

It is not important for me to have the _ns attribute, but imho the parsing should be the same in all browsers!

I don't have a copy of IE8 laying around, so I can't test this behavior.  It looks as though the xmlns option is being set differently.  Can you verify that in each case, the xmlToJson configuration options are the same?

I've tested with the default values, so without any custom settings.

Try setting the xmlns option to false.  This should force consistent behavior between the two browsers.  It as though IE 8 doesn't properly support the namespaceURI attribute.  If so, it's forcing the evaluation to false.  Let me know.

After setting xmlns = fase the difference between Firebox and IE will be:
Firefox:
      "_attr": {
        "xmlns:xsi": {"_value": "http://www.w3.org/2001/XMLSchema-instance"},
        "noNamespaceSchemaLocation": {"_value": "../xsd/HTMLVideoMapper.xsd"}
      },

IE8:
      "_attr": {
        "xmlns:xsi": {"_value": "http://www.w3.org/2001/XMLSchema-instance"},
        "noNamespaceSchemaLocation": {
          "_value": "../xsd/HTMLVideoMapper.xsd",
          "_ns": "http://www.w3.org/2001/XMLSchema-instance"
        }
      },

So in fact setting the xmlns will not affect changes between those browsers

It looks as though the output is consistent.  IE added the _ns value, but the object naming and values are otherwise identical which should give consistent and predictable behavior between browsers.  I will update the documentation to indicate that the library is intended for IE9+ and close this issue.

Well, that's open source and you have the right to change requirements. But for me, that means a huge problem since I already started to implement your plugin today. Each XML2JSON-converter plugin will use different ways to handle attributes, texts, values and namespaces. My target browser is IE since it is a customer requirement. So for me that would mean, you are switching from IE7+ to IE9+ and killing the work.
For my understanding: What is the problem why there are such differences betwen IE8 and Firefox? Also, are there more things/features than namespaces that can be different in each browser?

The difficulty lies in Microsoft’s support for DOM standards in IE8.  The situation greatly improved with IE9.  IE10+ are significantly better.  Chrome, Firefox, and Safari implemented the DOM methods for XML much more rapidly and so support appears in earlier versions.
I updated the documentation to reflect that the library will work as expected for IE9+ .  I also stated in the docs that the library can be reasonably and predictably used with IE 7 and 8 if you set the xmlns option to false.
I hope this helps, and I apologize if it does not.  If you would like to submit a patch, I’d be happy to consider it.  For now though, I have no intent of supporting outdated and non-standard browsers.
Best luck with your endeavors.

On Jan 28, 2015, at 12:47 PM, Julian<EMAIL_ADDRESS>wrote:
Well, that's open source and you have the right to change requirements. But for me, that means a huge problem since I already started to implement your plugin today. Each XML2JSON-converter plugin will use different ways to handle attributes, texts, values and namespaces. My target browser is IE since it is a customer requirement. So for me that would mean, you are switching from IE7+ to IE9+ and killing the work.
For my understanding: What is the problem why there are such differences betwen IE8 and Firefox? Also, are there more things/features than namespaces that can be different in each browser?
—
Reply to this email directly or view it on GitHub https://github.com/metatribal/xmlToJSON/issues/6#issuecomment-71902348.

The only thing left I don't understand is why setting xmlns to false will fix the differences from IE8 to Firefox. For me setting the value to false means that there should not appear any namespace attributes. But in IE8 those still exist (see my last example).
What am I missing? Why is settin xmlns to false a solution in this situation?


Azure

Why do I need an account on azure to store the data that could be saved in a local SQL server?
This is what to open source when microsoft is buying the platform

This is a full block explorer, most project don't need it. This is too coupled to Azure to run on anything else for performance reason.
Use https://github.com/dgarage/NBXplorer instead which is lightweight and no cloud dependency.


Code example comments are longer than 120 columns

The code examples are displayed in boxes limited to 120 columns, some comments exceeds this limit which makes reading more difficult than it should be.

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

ID: 87f95d49-485e-1151-bd35-1c9d8741cb4f
Version Independent ID: eb522807-d55e-cef8-2077-6c896f1035e3
Content: Induce Chaos in Service Fabric clusters
Content Source: articles/service-fabric/service-fabric-controlled-chaos.md
Service: service-fabric
GitHub Login: @motanv
Microsoft Alias: motanv

Proposed fix:
https://github.com/MicrosoftDocs/azure-docs/pull/29066

Thank you very much for your contribution to the Azure docs.

@motanv @aljo-microsoft Please validate this PR.

This has been merged--thanks @cBrauge ! #please-close


Fixed ValueError not raised

Raised ValueError in wagtail/images/utils.py that was called but not raised
I work for Semmle and I noticed these issues using our LGTM code analyzer
https://lgtm.com/projects/g/wagtail/wagtail/alerts/?mode=tree&severity=error&ruleFocus=1505923886371

Do the tests still pass? (http://docs.wagtail.io/en/latest/contributing/developing.html#testing)
yes
Does the code comply with the style guide? (Run make lint from the Wagtail root)
yes
For Python changes: Have you added tests to cover the new/fixed behaviour?
no
For front-end changes: Did you test on all of Wagtail’s supported browsers? Please list the exact versions you tested.
not applicable
For new features: Has the documentation been updated accordingly?
not applicable

Thanks @damianosSemmle! Merged (with test added) in 20f47ff50e191695fe26041d089a253e61eb7196.


Replace Collection with List

Replace Collection<> with List<> in encoder classes as List<> supports initial capacity and improves performance

List<> is faster in debug mode, Collection<> in release. Closing.


onActivityResult deprecated....

i have a project with fragments. in the new appcompat the function:
@Override
protected void onActivityResult    was deprecated.
how i can get the result of mercadopago?????

You can still use the method even if it's deprecated, it will work.

You can still use the method even if it's deprecated, it will work.

Nop, the function didn't execute. the app is waiting for the response

Same issue here, can be nice if you add a callback for an Activity Result to your roadmap.


randCodeSnippet

Description
randCodeSnippet({ lang: 'html' })
One small/medium snippet for top languages.
// data

{
   html: `snippet..`,
   javascript: `snippet...`
}

Proposed solution
No response
Alternatives considered
No response
Do you want to create a pull request?
No

Assign to me please.

I struggle to find some time before weekend.   Feel free to reassign to someone else.

No rush

Hi, I'd be interested in picking this up

@jasiene you can take it if @michaelxvoelker doesn't mind.

Do it. 😊👍🏼

Thanks @michaelxvoelker 😄

@jasiene are you still working on this? @NetanelBasal can you reassign to me?

Hey @Parikshit-Hooda I am yes. I will submit a PR tonight


Update Dockerfiles and allows kestrel to listen for * domains

Fix "web" command mismatch with kestrel entrypoint by updating Dockerfiles
Allow multiple domains with server.urls *

Hi @emyann, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@emyann, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@glennc @ahmetalpbalkan ?

@Eilon FYI, the mismatch only exists into the folder called latest. The other ones are correct. Thank you for treating this PR :)

I don't want to depend on microsoft/aspnet in this sample. These samples are supposed to always work on the latest bits, and the aspnet image is static to released versions. So there is a reason it is that way.
The * change is fine.
Outside of that we will need to change this whole thing significantly when RC2 comes around.

@glennc any thoughts on this?

No longer relevant now that there are no samples here.


add category

Added AAAA as a category

What is AAAA?

Testing out using the GitHub editor pull-requests as a workflow for editing cards with a team.

Ah, hah sorry disregard my nosiness!


Double running / internal_

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[x] 3.1

On this operating system:

[x] MacOS, version:

Foxx
Apologies if this is just something I don't understand about ArangoDB/Foxx, but I was trying to integrate the session middleware and kept coming up with odd errors saying that my 'sessions' collection didn't exist.
I started placing console.logs in my code and pouring over the output of arangod and noticed that, for some reason, my code was being run twice, first as module.context.collectionPrefix set to internal_ then set to the actual value. I was able to work around this issue by using a if statement on the collectionPrefix to avoid setting up sessions on the first / internal_ version.
The relevant bit of my code:
const
  arango          = require('@arangodb'),
  createRouter    = require('@arangodb/foxx/router'),
  joi             = require('joi'),
  db              = arango.db,
  errors          = arango.errors,
  aql             = arango.aql,
  router          = createRouter(),
  sessionsMiddle  = require('@arangodb/foxx/sessions'),
  users           = module.context.collectionName('users'),
  DOC_NOT_FOUND   = errors.ERROR_ARANGO_DOCUMENT_NOT_FOUND.code;

console.log("Starting...");

console.log('PREFIX',module.context.collectionPrefix);

if (module.context.collectionPrefix !== 'internal_') {
  const sessions = sessionsMiddle({
    storage: module.context.collection('sessions'),
    transport: ['header', 'cookie']
  });

  module.context.use(sessions);

}
module.context.use(router);

Output in arangod:
2017-03-31T19:43:21Z [28640] INFO Starting...
2017-03-31T19:43:21Z [28640] INFO PREFIX internal_

.. some irrelevant logging ..

2017-03-31T19:43:21Z [28640] INFO Starting...
2017-03-31T19:43:21Z [28640] INFO PREFIX midcentury_dev_

If I remove the if around the sessions bit I get the following error:
Cause: AssertionError: No session storage specified
    at sessionMiddleware (/usr/local/opt/arangodb/share/arangodb3/js/server/modules/@arangodb/foxx/sessions/index.js:42:3)
    at Object.<anonymous> (/var/folders/_v/s9jvdb0x1753l8b1v4_yyf_m0000gn/T/arangod_dgqkbz/apps/tmp-28640-901218078/index.js:23:20)
    at Module._compile (/usr/local/opt/arangodb/share/arangodb3/js/common/bootstrap/modules.js:586:8)
    at Object.Module._extensions..js (/usr/local/opt/arangodb/share/arangodb3/js/common/bootstrap/modules.js:604:12)
    at Module.load (/usr/local/opt/arangodb/share/arangodb3/js/common/bootstrap/modules.js:539:36)
    at FoxxService.run (/usr/local/opt/arangodb/share/arangodb3/js/server/modules/@arangodb/foxx/service.js:435:14)
    at exports.routeService (/usr/local/opt/arangodb/share/arangodb3/js/server/modules/@arangodb/foxx/routing.js:153:40)
    at _validateService (/usr/local/opt/arangodb/share/arangodb3/js/server/modules/@arangodb/foxx/manager.js:1087:7)
    at Object.upgrade (/usr/local/opt/arangodb/share/arangodb3/js/server/modules/@arangodb/foxx/manager.js:1418:3)
    at Object.actions.defineHttp.callback.easyPostCallback.callback [as callback] (/usr/local/opt/arangodb/share/arangodb3/js/actions/_admin/foxx/app.js:146:26)

And in arangod
2017-03-31T19:47:07Z [28640] INFO Starting...
2017-03-31T19:47:07Z [28640] INFO PREFIX internal_
2017-03-31T19:47:07Z [28640] ERROR Cannot execute Foxx service at /internal: ArangoError: failed to invoke module
2017-03-31T19:47:07Z [28640] ERROR File: /var/folders/_v/s9jvdb0x1753l8b1v4_yyf_m0000gn/T/arangod_dgqkbz/apps/tmp-28640-901218078/index.js
2017-03-31T19:47:07Z [28640] ERROR Cause: AssertionError: No session storage specified

I'm 'restarting' my foxx app with foxx-manager as so:
foxx-manager upgrade [name] [mount point] --server.password [password] --server.username [username] --server.database [db]

What am I doing wrong / is there something wrong here?

You're right that this is a bug in the Foxx Manager. The service is currently being "validated" before each upgrade/replace to guarantee that it will work before the existing service is overwritten. Part of the validation is that the service is executed as if it would actually be mounted (i.e. the main file is loaded and executed). This is why you see errors about the "/internal" mount path -- that's where the service thinks it's mounted during the "validation".
Normally the main file does not perform any database access so this used to work fine throughout 2.x. However as you've noticed the session middleware we introduced in 3.0 performs a sanity check that the collection you're using actually exists, which throws an error at mount time.
We're already in the process of heavily rewriting the service management logic as part of the cluster resilience improvements for 3.2 so this problem should be fixed with the upcoming 3.2 release (note that alpha releases currently are still affected by this behaviour).
In the meantime you can continue using the if workaround to avoid running into this problem. I apologise for the inconvenience.

Thanks for verifying that I found a bug! I thought I was going crazy.

You're definitely not crazy. The "validation" logic will be removed in 3.2 and replaced with some less intrusive sanity checks. I'm keeping this issue open until the feature branch (vulpine-resilience) is merged.

Will be part of the next 3.2 release.

We're pleased to anounce that ArangoDB 3.2 containing this is now available for download - Closing.


Connection closed/query canceled on INSERT

Hi, thanks for your great work on this package. Experiencing a weird bug that I have no idea to fix. I also don't really know what the cause is.
I have a source_gen-powered ORM project I'm working, and it generates code that uses package:postgres. I have a method that inserts data into a database, and then returns data from the row, so that it can be deserialized and returned to the caller.
The relevant code:
var result = await connection.query(
  'INSERT INTO "categories" ("name", "description", "created_at", "updated_at") VALUES (@name, @description, @createdAt, @updatedAt) RETURNING ("id", "name", "description", "created_at", "updated_at");',
  substitutionValues: {
     'name': name,
     'description': description,
     'createdAt': createdAt != null ? createdAt : __ormNow__,
     'updatedAt': updatedAt != null ? updatedAt : __ormNow__
 });

However, this always fails with the following error:
PostgreSQLSeverity.error : Connection closed or query cancelled.

The resulting stack trace unfortunately offers no insight into what happened behind the scenes, since it uses a Completer.
So, my question is: why would an INSERT command cause Postgres to close the connection or cancel the query? I feel like there's a core concept I'm missing here. Thanks in advance...
NOTE: __ormNow__ is just an instance of DateTime that I used to make sure there are created_at and updated_at values.

why the author does not answer you? support for this repository is discontinued?

I assume that they are more busy with other projects, i.e. Aqueduct, and will get to this when time is available.
Coincidentally, though, I just encountered this same error again in another project.

More recent version of driver should give you the underlying exception. Even if not, set a break point for uncaught exceptions to see where it gets thrown from and include that here. My guess is that you have a data type mismatch for your time related columns and you should be explicitly indicated which column type they are.

Thanks so much. Really appreciate this, and all the work you've done for
the Dart ecosystem.
On Fri, Dec 8, 2017 at 11:38 AM Joe Conway<EMAIL_ADDRESS>wrote:

More recent version of driver should give you the underlying exception.
Even if not, set a break point for uncaught exceptions to see where it gets
thrown from and include that here. My guess is that you have a data type
mismatch for your time related columns and you should be explicitly
indicated which column type they are.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stablekernel/postgresql-dart/issues/22#issuecomment-350309408,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJiKPK70QwsqAO5oII2dVk_FB7gWwcGqks5s-WYFgaJpZM4OOnWh
.

Actually, it turns out that's not the problem. It turns out the rows are actually getting inserted into the database, which is good.
I'm not entirely sure where the issue is occurring. How can I set a break point for uncaught exceptions?


Bluetooth mesh samples with settings go into reset on nrf5340dk_nrf5340_cpuapp_ns with TF-M

Describe the bug
If run bluetooth\bluetooh mesh apps with default configuration (that includes enabled settings) then device goes into reset during ble host initialization.
That happens here: https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/bluetooth/host/hci_core.c#L3936 (didn't investigate what happens inside settings).
Seems TF-M with trustable storage cannot coexist with settings subsystem.
Target platform: netcore - nrf5340dk_nrf5340_cpunet, appcore - nrf5340dk_nrf5340_cpuapp_ns.
There were added separate configuration files to all mesh samples for nrf5340dk_nrf5340_cpuapp_ns with disabled settings as workaround.
To Reproduce
Steps to reproduce the behavior:

enbale settings for mesh sample (disable workaround):

diff --git a/samples/bluetooth/mesh/boards/nrf5340dk_nrf5340_cpuapp_ns.conf b/samples/bluetooth/mesh/boards/nrf5340dk_nrf5340_cpuapp_ns.conf
index c638a292c9..cceb398273 100644
--- a/samples/bluetooth/mesh/boards/nrf5340dk_nrf5340_cpuapp_ns.conf
+++ b/samples/bluetooth/mesh/boards/nrf5340dk_nrf5340_cpuapp_ns.conf
@@ -1,3 +1,3 @@
 # Known issue: non secure platforms do not work with settings subsystem.
-CONFIG_SETTINGS=n
-CONFIG_BT_SETTINGS=n
+#CONFIG_SETTINGS=n
+#CONFIG_BT_SETTINGS=n

build and flash net core sample (hci_rpmsg)

west build -b nrf5340dk_nrf5340_cpunet -- -DCONF_FILE="nrf5340_cpunet_bt_mesh-bt_ll_sw_split.conf

build ans flash app core sample (mesh)

west build -b nrf5340dk_nrf5340_cpuapp_ns

run and see error
infinite reset

*** Booting Zephyr OS build v3.4.0-167-g327eb119b6bc ***
Initializing...
*** Booting Zephyr OS build v3.4.0-167-g327eb119b6bc ***
Initializing...
*** Booting Zephyr OS build v3.4.0-167-g327eb119b6bc ***
Initializing...
*** Booting Zephyr OS build v3.4.0-167-g327eb119b6bc ***
Initializing...

Expected behavior
mesh samples work as expected with enabled tf-m and settings
Impact
Not possible to use bluetooth mesh and tf-m for real use cases.
Environment (please complete the following information):

OS: Linux
Toolchain: zephyr sdk 0.16.1
Commit SHA: 327eb11

Hi @henrikbrixandersen, I'm pretty sure the issue is not related to Bluetooth Mesh or just Bluetooth. The issue is related to coexistence TF-M with trustable storage and Zephyr's settings subsystem on nrf5340dk_nrf5340_cpuapp_ns.
I just reproduced it in bluetooth mesh samples, since I maintain them. I guess it is possible to reproduce with other samples too.

Checked main with this fix. Seems the issue has been fixed. No resets anymore.


Notify the user when the rift barrier unlocks.

Trigger a notification when the rift barrier's state goes from locked to unlocked. This is implemented carefully so that the notification should only trigger if the user is standing in front of the locked gate when it unlocks (and shouldn't trigger on world hop or login).
Should be useful if you're waiting for a game in progress to end, but you don't want to have to baby-sit the progress by peeking at the portal.

Friendly ping for review.

@DatBear any chance you could review this?


data: hariFED

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Hi @kumarsonsoff3  as you mentioned in the review the file name harifed.json is renamed to hariFED.json.


How to get the related user to a certain key?

Hello, thanks in advance.
How can I get the apikeyable id related to a certain key?
I'm about to modify the library but I would like to know if this is a feature that already exists or you are planning on implementing.
*The user is not logged in.
In all cases, as soon i finish I'll do PR.
Thanks!

Oh, found it, you are attaching the user to the request.
Misunderstood the documentation.
Maybe it is an idea to make it more clear for dummies like me?
Thanks a lot!


Add names of dogs for each image

Context
Name of the dog
Expected Behavior
Include text at the bottom of the picture that has the name of the dog that is currently being viewed
Acceptance criteria
Add label for text underneath picture icon, include a requirement for text to be entered upon submittal of picture
Screenshots
N/A
Additional information

I like this but I don't know if the dog ceo API provides names. It DOES provide breeds though! Maybe the name of the breed?


Fix build description for perf test

Description
Getting java.lang.ClassCastException for the mentioned if block. Removing it for now
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2605 (063dfe8) into main (c29e62e) will increase coverage by 5.58%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #2605      +/-   ##
=============================================
+ Coverage     94.41%   100.00%   +5.58%
=============================================
  Files           219         6     -213
  Lines          4462       105    -4357
  Branches         29        19      -10
=============================================
- Hits           4213       105    -4108
+ Misses          243         0     -243
+ Partials          6         0       -6

Impacted Files
Coverage Δ

src/sign_workflow/signer_pgp.py

.../jenkins/jobs/CreateTestResultsMessage_Jenkinsfile

src/sign_workflow/signer_windows.py

...teg_test_start_properties_opensearch_dashboards.py

src/manifests/manifest.py

src/test_workflow/integ_test/distributions.py

src/sign_workflow/sign_args.py

src/manifests_workflow/component_opensearch.py

...w/perf_test/perf_test_runner_opensearch_plugins.py

src/manifests/manifests.py

... and 203 more

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


Placing ion-label inside ion-grid in an ion-item fails to render the entire grid

If you have a ion-grid inside ion-item and placing ion-label together with some other components inside ion-grid is causing issue with rendering the entire grid.
Here is a post  that I have created in ionic forum.
Below is the markup from that post
<ion-list>
    <ion-item  *ngFor="let order of orders">
      <ion-grid>
        <ion-row>
          <ion-col>
            <ion-icon name="pie" item-left></ion-icon>
          </ion-col>
          <ion-col>
            <ion-label>{{order.number}}</ion-label>
            <ion-label>{{order.quantity}}</ion-label>
          </ion-col>
          <ion-col>
            <ion-label>{{order.productType}}</ion-label>
            <ion-label>
              <ion-icon name="calendar"></ion-icon>{{order.date}}</ion-label>
          </ion-col>
        </ion-row>
      </ion-grid>
    </ion-item>
  </ion-list>

As @brandyscarney pointed out in another post  regarding ion-item, it is actually ion-label that is actually causing the issue, and also pointed out that this issue could be something to do with issue  in content projection of angular 2.

Hello, thanks for opening an issue with us! Would you be able to provide a plunker that demonstrates this issue? Thanks for using Ionic!

Hello @jgw96
Here is the plunker showing the issue
Regards
Vijay

Hello @vijquick just to update you on what ive found. After some testing this looks to be a content projection issue. It looks like the ion-grid never even gets rendered, which points me towards content projection.

Hello @vijquick, after discussing this with my teammates it seems to be partly by design, and partly an angular issue that this does not work. As a workaround for this i would recommend not using ion-grid and just use css to do your layout. Sorry that i could not give you a better answer, and thanks for using Ionic!

I think this is still an issue with RC3. Will use the suggestion above.

avoiding the ion-grid is the best solution


KD loss function

https://github.com/jl749/knowledge-distillation-pytorch/blob/4a7c13d090554961f04b1b5f4ecc324cdca94d66/mnist/distill_mnist.py#L96-L97

why x2 ?

from /knowledge-distillation-pytorch/issues/10

why KLDivLoss instead of CrossEntropyLoss ?
pytorch CrossEntropy takes "one hot vector" as a target
which means we cannot use CrossEntropyLoss with soft target
THEREFORE, use KLDivLoss instead (it can take prob distribution as target)


[W7.7][W13-B4] ModU

Avoid using too detailed diagrams as they are hard to maintain as the code changes.
Section 3.2 Missing return lines in sequence diagram.

Missing Alternative 2?

Section 3.2.1 Pros: Less work. Can put it more effectively in terms of Software Engineering?
Cons: Hard for new developers to understand the event center mechanism. Repeated for 3 sections. Try thinking of different disadvantages.


Using github api to obtain git revision

Fixes https://github.com/debezium/debezium-ui/pull/332

Hey, @uidoyen can you list the testing steps.

Hey, @uidoyen can you list the testing steps.

@indraraj test steps are simple as below
Run ui on dev or prod mod. Click help icon on header top right corner you should be able to see build hash as below screenshot

Hey, @uidoyen can you list the testing steps.

@indraraj I am not sure what testing steps you are referring to but you can follow below steps:
Run ui on dev or prod mod. Click help icon on header top right corner you should be able to see build hash as below screenshot

I was talking about the UI build fail issue, but now that i realise this is not using git cmd at all it may not be needed

Hey @uidoyen I was thinking if we should not work directly from the main repo but from our fork, leave the main repo clean with some occasion required feature branches.


Feature: Profiles for deploying containers

In the UI I'd like to be able to save/recall a container configuration (as a profile), so that I can quickly start up containers without having to fill out the full container deployment options every time.
Does this sound like a feature that would be useful to others?

+1 for this

+1


Dependabot/GitHub actions/james ives/GitHub pages deploy action 4.1.7

Description
Please provide a meaningful description of what this change will do, or is for. Bonus points for including links to related issues, other PRs, or technical references.
Note that by not including a description, you are asking reviewers to do extra work to understand the context of this change, which may lead to your PR taking much longer to review, or result in it not being reviewed at all.
Type of Change

[ ] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[ ] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[ ] I have verified this change is not present in other open pull requests
[ ] Functionality is documented
[ ] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

Codecov Report

Merging #2 (58c0a66) into main (5bf1ed9) will decrease coverage by 6.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   50.97%   44.78%   -6.20%
==========================================
  Files          44       44
  Lines        1389     1389
  Branches       60       60
==========================================
- Hits          708      622      -86
- Misses        668      755      +87
+ Partials       13       12       -1

Impacted Files
Coverage Δ

src/Transactions/MsSqlTransactionWrapper.cs
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/Support/Setup/DatabaseSetup.cs
43.24% <0.00%> (-54.06%)
:arrow_down:

src/Connection/Factories/MsSql.cs
0.00% <0.00%> (-50.00%)
:arrow_down:

src/Connection/SqlConnectionWrapper.cs
0.00% <0.00%> (-33.70%)
:arrow_down:

src/Connection/DatabaseConnection.cs
78.94% <0.00%> (-11.58%)
:arrow_down:

src/Connection/Database.cs
81.96% <0.00%> (-6.56%)
: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 5bf1ed9...58c0a66. Read the comment docs.


UProbes not Working with Non-Zero Function Offsets

Hi,
I am experimenting with the malloc example. It works successfully with victim. I changed the UProbe offset from 0 to 135 (here). 135 is a valid offset in the system glibc:
   0x00000000000970a4 <+132>:	mov    %rbx,%rsi
   0x00000000000970a7 <+135>:	callq  0x94330 <_int_malloc>
   0x00000000000970ac <+140>:	test   %rax,%rax
The kernel eBPF (i.e., running without bpftime) shows probe hits, while running victim with bpftime causes no hit.
Any hint is appreciated!
Regards.

The offset was not touched by victim. The kernel eBPF traces all running processes. Some non-victim running processes were touching the offset, causing probe hits for the kernel eBPF.


Charts[Network Node]: Expose prometheus metrics from HAProxy

:tada: This issue has been resolved in version 0.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Optimized images are too big

I tried a lot of jpg image to optimize with image_optim.
It reduces the size but when I compare how the results of what facebook does are, these images are way smaller than what comes out from image_optim.
Is there a way to decrease the image size any further without reducing the quality?

By default image_optim is lossless, so using all the tools it will only remove all unnecessary data and will try to optimise compression. But you can allow lossy optimisations using allow_lossy: true / --allow-lossy, and if not enough with default options, configure workers to reduce image size more like jpegrecompress: {quality: 0} / --jpegrecompress-quality=0.
As a small test I've uploaded a 2048×1536 941KB image to facebook and got same size 324KB image, using image_optim --allow-lossy --jpegrecompress-quality=0 I've got a 220KB image.
Visual difference in my case was hardly noticeable.
Comparing original vs optimised using imagemagick: mean error per pixel (normalized mean error, normalized peak error)
for facebook optimisation: 5.29145e+09 (0.000184438, 0.203922)
for image_optim: 5.96241e+09 (0.000240838, 0.262745)
The jpegrecompress worker will soon get more options like #103, so more flexibility to get the desired result.

That is great thank you.


Turn on duplicate detection for all senders

Problem statement
Reportstream has an option to detect and ignore duplicate submissions. Turning this option on for senders will save processing time and resources in report stream in the event that errors occur in a sender system.
What you need to know
Option in sender settings:
This is the default: allowDuplicates: true
All should be changed to allowDuplicates: false
Acceptance criteria

[ ] All senders have allowDuplicates: false

Verify behavior of allowDuplicates: false: Does it strip out duplicate tests from within bulk submissions or does it reject the whole submission?
Be prepared for extra work if the whole submission is rejected.

allowDuplicates: false does not strip out duplicate tests from within bulk submissions. The entire CSV file or HL7 batch message will fail if there is a single detected duplicate row.
Interestingly, it also does not detect duplicates within a csv file or HL7 batch message in an inbound message, if those messages or rows have not already been submitted. This is because the way that duplicate checking is working is done by comparing the values in item_hash column on the item_lineage table in the database (fun: isDuplicateItem in DatabaseAccess.kt)

Decision was made to only turn on duplicate detection for senders who are sending a single message per file.


Error on building release

error D8016 : /clr and &/EHs command-line options are incompatible.

fixed with https://github.com/alza54/opensource2/pull/3


Invalid argument(s): Missing library: package:my_application/my_file.dart

Steps to Reproduce

updated dependencies (like flutter version, etc) no changes in codeenvironment:
  flutter: ^2.8.1
  sdk: '>=2.15.0 <3.0.0'
dev_dependencies:
  freezed: ^1.1.0
  flutter_lints: ^1.0.4
  analyzer: ^2.7.0
  build_runner: ^2.1.7
  json_serializable: ^6.1.0
  injectable_generator: ^1.4.1
  auto_route_generator: ^3.2.0
  retrofit_generator: ^2.0.1

Execute flutter pub run build_runner build --delete-conflicting-outputs

Expected results:
flutter build_runner executed successfully
Actual results:

Code sample

import 'package:json_annotation/json_annotation.dart';

part 'user_entity.g.dart';

@JsonSerializable()
class UserEntity {
  UserEntity({this.applicationId, this.jwtToken, this.userId, this.password});

  final String? applicationId;

  @JsonKey(ignore: true)
  final String? jwtToken;

  final String? userId;

  String? password;

  factory UserEntity.fromJson(Map<String, dynamic> json) =>
      _$UserEntityFromJson(json);

  Map<String, dynamic> toJson() => _$UserEntityToJson(this);

  @override
  String toString() {
    return 'userId : $userId, password: $password';
  }
}

Logs
Run  with flutter pub run build_runner build --delete-conflicting-outputs
[INFO] Generating build script...
[INFO] Generating build script completed, took 666ms

[WARNING] Invalidated precompiled build script due to missing asset graph.
[INFO] Precompiling build script......
[INFO] Precompiling build script... completed, took 9.4s

[INFO] Initializing inputs
[INFO] Building new asset graph...
[INFO] Building new asset graph completed, took 1.8s

[INFO] Checking for unexpected pre-existing outputs....
[INFO] Checking for unexpected pre-existing outputs. completed, took 1ms

[INFO] Running build...
[INFO] 1.0s elapsed, 23/39 actions completed.
[SEVERE] freezed:freezed on test/data/response_test.dart:

Invalid argument(s): Missing library: package:m_banking_back_office/domain/core/entities/user_entity.dart
Available libraries: [dart:typed_data, dart:convert, dart:math, dart:core, dart:_internal, dart:collection, dart:async, dart:isolate, dart:developer, dart:ffi, dart:_js_embedded_names, dart:_js_names, dart:_recipe_syntax, dart:_rt
i, dart:_foreign_helper, dart:_js_helper, dart:_interceptors, dart:_native_typed_data, dart:web_gl, dart:js_util, dart:_metadata, dart:html_common, dart:indexed_db, dart:svg, dart:web_audio, dart:html, dart:io, dart:_http, dart:js
, dart:_js, dart:mirrors, dart:nativewrappers, dart:cli, dart:_late_helper, dart:_dart2js_runtime_metrics, dart:_js_primitives, dart:_async_await_error_codes, dart:_js_annotations, dart:_spirv, dart:ui]
#0      LinkedElementFactory.createLibraryElementForReading (package:analyzer/src/summary2/linked_element_factory.dart:74:7)
#1      LinkedElementFactory.elementOfReference (package:analyzer/src/summary2/linked_element_factory.dart:128:14)
#2      LinkedElementFactory.libraryOfUri (package:analyzer/src/summary2/linked_element_factory.dart:177:12)
#3      ElementBuilder._selectLibrary (package:analyzer/src/summary2/element_builder.dart:1018:37)
#4      ElementBuilder.visitImportDirective (package:analyzer/src/summary2/element_builder.dart:598:31)
#5      ImportDirectiveImpl.accept (package:analyzer/src/dart/ast/ast.dart:5953:50)
#6      NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:7758:20)
#7      ElementBuilder.buildLibraryElementChildren (package:analyzer/src/summary2/element_builder.dart:60:21)
#8      LibraryBuilder.buildElements (package:analyzer/src/summary2/library_builder.dart:109:24)
#9      Linker._computeLibraryScopes (package:analyzer/src/summary2/link.dart:109:15)
#10     Linker._buildOutlines (package:analyzer/src/summary2/link.dart:88:5)
#11     Linker.link (package:analyzer/src/summary2/link.dart:74:5)
#12     link (package:analyzer/src/summary2/link.dart:33:10)
#13     LibraryContext.load2.loadBundle (package:analyzer/src/dart/analysis/library_context.dart:176:30)
#14     SetMixin.forEach (dart:collection/set.dart:142:30)
#15     LibraryContext.load2.loadBundle (package:analyzer/src/dart/analysis/library_context.dart:113:32)
#16     LibraryContext.load2.<anonymous closure> (package:analyzer/src/dart/analysis/library_context.dart:206:7)
#17     PerformanceLog.run (package:analyzer/src/dart/analysis/performance_logger.dart:32:15)
#18     LibraryContext.load2 (package:analyzer/src/dart/analysis/library_context.dart:204:12)
#19     AnalysisDriver._computeUnitElement.<anonymous closure> (package:analyzer/src/dart/analysis/driver.dart:1574:22)
#20     PerformanceLog.run (package:analyzer/src/dart/analysis/performance_logger.dart:32:15)
#21     AnalysisDriver._computeUnitElement (package:analyzer/src/dart/analysis/driver.dart:1572:20)
#22     AnalysisDriver.performWork (package:analyzer/src/dart/analysis/driver.dart:1126:35)
#23     AnalysisDriverScheduler._run (package:analyzer/src/dart/analysis/driver.dart:2140:24)
<asynchronous suspension>

flutter doctor -v
[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.19043.1083], locale en-US)
    • Flutter version 2.8.1 at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (12 hours ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\u066757\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Users\u066757\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\203.7784292\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

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

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Users\u066757\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\203.7784292
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[☠] IntelliJ IDEA Community Edition (the doctor check crashed)
    X Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about this issue at https://github.com/flutter/flutter/issues.
    X FormatException: Unexpected extension byte (at offset 5)
    • #0      _Utf8Decoder.convertSingle (dart:convert-patch/convert_patch.dart:1789:7)
      #1      Utf8Decoder.convert (dart:convert/utf.dart:318:42)
      #2      InputStream.readString (package:archive/src/util/input_stream.dart:207:30)
      #3      new ZipDirectory.read (package:archive/src/zip/zip_directory.dart:40:30)
      #4      ZipDecoder.decodeBuffer (package:archive/src/zip_decoder.dart:19:30)
      #5      ZipDecoder.decodeBytes (package:archive/src/zip_decoder.dart:14:12)
      #6      IntelliJPlugins._findPluginXml (package:flutter_tools/src/intellij/intellij.dart:130:44)
      #7      IntelliJPlugins._readPackageVersion (package:flutter_tools/src/intellij/intellij.dart:141:40)
      #8      IntelliJPlugins.validatePackage (package:flutter_tools/src/intellij/intellij.dart:63:35)
      #9      IntelliJValidator.validate (package:flutter_tools/src/intellij/intellij_validator.dart:103:15)
      #10     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:32)
      #11     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:109:18)
      #12     _rootRun (dart:async/zone.dart:1428:13)
      #13     _CustomZone.run (dart:async/zone.dart:1328:19)
      #14     _runZoned (dart:async/zone.dart:1863:10)
      #15     runZonedGuarded (dart:async/zone.dart:1851:12)
      #16     runZoned (dart:async/zone.dart:1782:12)
      #17     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
      #18     Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:197:9)
      #19     Doctor.diagnose (package:flutter_tools/src/doctor.dart:301:47)
      #20     DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:53:47)
      #21     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1290:12)
      <asynchronous suspension>
      #22     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1140:27)
      <asynchronous suspension>
      #23     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #24     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
      <asynchronous suspension>
      #25     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
      <asynchronous suspension>
      #26     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #27     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)
      <asynchronous suspension>
      #28     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
      <asynchronous suspension>
      #29     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #30     main (package:flutter_tools/executable.dart:94:3)
      <asynchronous suspension>

[√] VS Code, 64-bit edition (version 1.63.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension can be installed from:
       https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (3 available)
    • SM A715F (mobile) • RR8NB01G9DD • android-arm64  • Android 11 (API 30)
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 94.0.4606.81
    • Edge (web)        • edge        • web-javascript • Microsoft Edge 96.0.1054.57

! Doctor found issues in 1 category.

Screenshot of updated dependencies :

Hi @Dargoz
From what I can see, the issue is related to a 3rd party plugin rather than to Flutter itself. Please open the issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself.
If you disagree, please write in the comments,
providing your flutter doctor -v, your flutter run -v,  your pubspec.yaml
a minimal reproducible code sample that does not use 3rd party plugins,
and I will reopen it.
Thank you


added event grid namespace system events

Data Plane API - Pull Request

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[ ] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc:
Updated paths:

❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

Can we move these events and the SubscriptionValidation events under a new file called EventGridSystemEvents or something alike? @JoshLove-msft

PR with proposed changes - https://github.com/Azure/azure-rest-api-specs/pull/25526

Closing this out as https://github.com/Azure/azure-rest-api-specs/pull/25526 has been merged.


Exception: Bad point. Try increasing theta0.

I'm not really sure where this error results from -- I am not so familiar with the GP backend from sklearn.
Here's the stack trace. It happens at iteration number 86, but it's inconsistent.
Traceback (most recent call last):
...
bo.maximize(init_points = 15, n_iter=100)
File "build/bdist.linux-x86_64/egg/bayes_opt/bayesian_optimization.py", line 314, in maximize
File "/share/apps/python-2.7.9/lib/python2.7/site-packages/sklearn/gaussian_process/gaussian_process.py", line 352, in fit
raise Exception("Bad point. Try increasing theta0.")
Exception: Bad point. Try increasing theta0.

My function I'm trying to optimize looks like this
def crossValScore(c):
        scores = []
        for trainI, testI in StratifiedShuffleSplit(y, 10, test_size=.1,):
            lr = LR(C=c)
            Xtr = X[trainI,:]
            ytr = y[trainI]

            Xts = X[testI,:]
            yts = y[testI]

            lr.fit(Xtr, ytr)
            scores.append(f1_score(yts, lr.predict(Xts)))
        scores = np.array(scores)
        return np.mean(scores)

my call looks like
 bo = BayesianOptimization(crossValScore, {'c':(.00001, 100000)})
    bo.maximize(init_points = 15, n_iter=100)

This is sklearn's GP object complaining. I believe that due to the vastly different scales between lower and upper bounds it makes it a bit harder on the GP.
You can follow their instruction and increase theta0, or increasing the nugget can help sometimes as well. To so you may simply pass these as parameters to maximize, which will redirect them to the GP object.
The default value for theta0 is: theta0=numpy.random.uniform(0.001, 0.05, n_params)
Alternatively you can either narrow down the search space, or log-transform it, something along the lines of:
def crossValScore(log_c):
    c = exp(log_c)
    ...

and adjust the bounds accordingly.

I log-transformed it and am not getting the error -- thanks!


Update whats-new.md

November updates for QnA Maker

@prakul-bansal : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off approved

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Pin to specific version (0.0.6) of cisagov/postfix image

cisagov/postfix:0.0.6 is our latest and greatest version.

🗣 Description
This PR pins this composition to the current newest version of cisagov/postfix-docker, which is 0.0.6.  Previously, we only used cisagov/postfix:latest.

💭 Motivation and context
There are two goals here:

Pinning to a specific version moves us toward our goal of more reproducible build processes.
We want to pull in the recent improvement from: https://github.com/cisagov/postfix-docker/pull/33.

🧪 Testing
All automated tests pass.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Finalize version.

✅ Post-merge checklist

[ ] Add a tag or create a release.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/gophish_init/_version.py
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
45

💛  - Coveralls


Allow for a 0th GRIB2 message

grib2io is the successor to ncepgrib2 which was developed alongside pygrib and therefore takes on the pygrib convention that there is no 0th GRIB2 message.  As we refactor grib2io for v2.0, we should allow a 0th GRIB2 message.
Python (and C) indexing begins at 0, so when reading a file we read in units of bytes, starting at 0.  In grib2io, we will use that same idea for reading from files in units of GRIB2 messages.
GOAL:
g = grib2io.open(<file>)
g.seek(0) # Puts the file pointer at the beginning of the file
g.read(1) # Reads the first message (i.e. the 0th message)
g.tell() # Should return 1
msg = g[0] # msg is the first message on the file

This is done and working as expected.


Added some comments in code, fixed the random test failures, added in a few tests and my adoc

Closes #131


template updates for diagrams

I created a new jinja template for the diagram page and also created separate templates for individual diagrams used in the service template. I didn't make any changes to the service annotations, used the current for diagrams.
I used the basic flowchart and tried to add some different styles. Also, endpoints and events are ignored in diagrams if they have no consumers (so they have no role in the communication).
For now, all-services diagrams are added as a new page (Service Communications in the UI menu). Next, we should update the menu and homepage.
Below there are images to show how they look.
About checks
"Check template files" is failing. So, I updated the template files in a separate branch: https://github.com/ghga-de/ghga-devutil/pull/7

Diagrams are updated with new style. Screenshots below.

@lkuchenb You are right that would be more clear to understand. I quickly change the Produces text with event types, please check the below images how it will look like. I will look into a more pretty way to align nodes and lines in the larger diagrams.

@lkuchenb the branch is up-to-date with all the requested changes


fix documentation on the default TLS crate

The default TLS crate changed recently from native-tls to rustls but the documentation hasn't been updated entirely.
This should fix it.

Thanks


Adds DelistingEventsAlgorithm.py

Adds Python version of DelistingEventsAlgorithm

Thanks for the great PR's @AlexCatarino! Please fix the merge conflicts and we'll pull it in

Nice work @AlexCatarino, please resubmit with the conflicts resolved :)

Looks like this file didn't make it into the csproj. I'll add it.

Merged in https://github.com/QuantConnect/Lean/commit/37c6f1eb0e30553b7016884542f90f4db434b2f3


[shared_preferences] Replace AsyncTask class with AsyncHandler to handle async task for Android

Description
Replace AsyncTask class with AsyncHandler to handle async task for Android
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[ ] No, this is not a breaking change.

hi @matthew-carroll / @stuartmorgan , would you review this? else just close this pull.

We apologize for the long delay in triaging this PR. We’re in the process of overhauling our PR triage system to respond much more quickly, as well as working through the backlog.
If you're still interest in moving forward with this you're welcome to re-open it, and we'll get to it as we triage and prioritize the PR backlog. If not, I understand, and hope you'll consider contributing again in the future once the PR triage/review situation has improved.

Hi @stuartmorgan i believe the issue is not only about PRs that takes too long to be triage and reviewed.
I had done a quick check on some of the PRs and I'm not comfortable if these are going to be merged to the plugins:

Issues with AsyncTasks and thread pools https://github.com/flutter/plugins/pull/3481, https://github.com/flutter/plugins/pull/3244 Usually i have one thread pool to be used anywhere for efficiency and these do not cater to my need. also there seemed to be an issue with that MethodCallHandlerImpl.commitAsync  and I'm not sure why it is merged into the main branch. it is creating handler and executorService object every time the method is called, both should be field variables not local variables and I'm worried if my app will have memory and performance issue if user often change shared preferences value.

Unnecessary kotlin migration adding more dependencies https://github.com/flutter/plugins/pull/3139

I will have to fork and maintain different version for my app for a while. If you think this PR is good I will re-open, if not just close it.

I had done a quick check on some of the PRs and I'm not comfortable if these are going to be merged to the plugins

I wouldn't judge a plugin by unreviewed PRs. Anyone can open a PR, and the quality of those PRs vary significantly. One of those already had a comment explaining that we don't currently have any plans to accept it, and would require a formal proposal with discussion.
As for the merged PR, the best way to express feedback about a PR is to comment on it. If you think there are ways to improve it, you can note them there. Perhaps you have a suggestion for improvement that the reviewer missed, which could be addressed in a follow-up.

As for the merged PR, the best way to express feedback about a PR is to comment on it. If you think there are ways to improve it, you can note them there. Perhaps you have a suggestion for improvement that the reviewer missed, which could be addressed in a follow-up.

Yes if I'm doing something wrong letting me know would be better maybe I could improve it.


Onboarding feature: add styling and automatic repositioning of teaching bubbles

Included in this PR:

Styling: support high-contrast mode
Styling: differentiate primary and secondary buttons
Automated positioning

Checks whether teaching bubbles fit and repositions if necessary
This takes care of many screen-size issues

Here's an upload target: https://makecode.microbit.org/app/2f7a21a2fc5d243c9b32a0659fbceebb3fbb547b-48763d0d80
Still To Do:

Add more telemetry ticks
Screen-size issues

Move "Tour" to the cogwheel for small screens (just like other items from the help menu)
Have secondary query selector for elements that disappear/move for smaller screens (i.e. Share button)

Refactoring

Make cutout its own React component
Make the teaching bubble arrow its own React component
clean up code in general (feel free to offer suggestions)

Markdown parsing of content (instead of content from .tsx file)

re: positioning attempt
@jwunderl, yea the reason I don't include the center over element as an option is because I think that would really only be viable when it's being specified by the author. Otherwise, the bubble will likely cover most or all of the element.
Another thing to consider is whether it would be better to not include the bubble at all if it can't fit or the element is not being displayed. Although I think it would be better to have the info than not in most cases.


Edit Removal broken

When edit is active, it should remove the karma given out.
It doesn't do this properly anymore.

fixed in 1.11.4


Implement FOV in UE demo

What
Will try to implement FOV variability based on a user Z-coordinate. This might improve depth perception.

Update 29.03
Got this to work, but must be tweaked.


fix settings type: csharp.unitTestDebugingOptions -> csharp.unitTestDebuggingOptions

@zackschuster,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

Hi! Thanks for the contribution to Microsoft C# for Visual Studio Code! If you're able to sign the Contribution License Agreement linked above, we'll can get this merged.

@zackschuster, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

Thanks! This is a breaking change, but I suspect usage is low enough that it'll be OK.

@DustinCampbell @gregg-miskelly at the very least there aren't any workspace configs on github that use it.

Thanks for catching this!

@zackschuster: agreed. Thanks again for the contribution!

of course -- thanks for making such a quality extension! 😄


52 support for color functional notation

Close #52
Close #49

Has this feature been released or is it coming soon?


Adding?

I feel like you should add JATS XML, XLingPaper XML, XSL-FO, TeXML, and Adobe Indesign.

Greetings,
Reading the above suggestion made me think about it too.

JATS XML, if I understand it correctly, is just an XML schema for specific types of documents (and conversion to anything else relies on e.g. XSLT stylesheets). XLingPaper XML, I am not so sure, might be in the same case. So I am not sure there really belong here -- or any document-related XML schema would be suitable (say TEI XML, etc.)... It's kind of a different topic.

On the other hand, I do believe XSL-FO, with existing commercial processors but also an open source solution (Apache FOP) ought to be mentioned. FOP could be considered as a "a digital typesetting system (...) arranging text and media into digital documents". Yes, the "input" format is XSL-FO, which is not particularly human-friendly, but it is usually just a part of a broader solution. After all, FOP and Co are implementing line breaking, paragraph assembling, page layout, font selection etc., = the deep core of all typesetting systems.
There are several markets using this approach to produce lengthy documents in PDF and HTML --- for instance, one could mention DITA, a specification focused on developing technical documentation (in XML syntax or in LwDITA which is a specialized lightweight Markdown syntax) is quite used in the industry for software and hardware documentation; it has dedicated tools, either commercial (there are several ones) or open source (at least two implementations to my knowledge, the "official" DITA-OT and XMLMind's DITAC). It's sort of a targeted niche market, but the specific challenges met by technical product documentation are usually forgotten in other typesetting solutions.... Both DITA-OT and DITAC rely on XSL-FO, for conversion to other formats (usually HTML), and on XSL-FO processors such as FOP (for PDF generation).

Another solution which might be worth mentioning is Paged.js. Whatever one may think about ultimately delegating typesetting to a (headless) Web browser, it has some momentum and would tick several of the checkboxes here... The web stack is mentioned on the website (in "Separation of concerns"), so it's hard to avoid discussing it ^^

Regards.
(Disclosure: I am currently quite involved in using SILE. I used DITAC a few years ago in my professional life)

Thank you for your response. Thanks for the pointer @HughP and for the details @Omikhleia!

JATS XML is not a software, but an XML schema, as @Omikhleia said. So it is out of scope.
To my understanding, DITA is another XML schema.
XLingPaper XML is software. It is not FOSS, but I relaxed the inclusion criterion to “source code is accessible for reading”. This seems to be satisfied. So it is actually a candidate. Thank you!
DITA-OT is a candidate, I was not aware of. Thank you!
DITAC is a candidate, I was not aware of. Thank you!
True that. Software only targetting the web stack can also satisfy the criteria. But I see a problem in my definition I am taking out of the paged.js context:

If I write software which literally just replaces <body with <body data-format="html5" in some HTML5 file, I think it would also satisfy the criteria listed on the website (because they inherit the properties of HTML5). Or if we have some Javascript library, we have the same situation. But adding a <script/> element really was not my intention of what a typesetting system shall be. My inclusion criteria are not mature yet.
I wonder whether a certain notion of conversion shall be included. Specifically about Paged.js, I don't think it should count as typesetting system, but it is not terrible if it fits the criteria. In general, a typesetting system should do some typesetting tasks like arranging elements (obvious when you have paged output documents) or conversion of structure/metadata/serialized data (typical minimal task of a system generating reflowable documents).
So maybe we could extend “3. Produces HTML5, PostScript, or PDF 1.4 (or later) output” to “3. Produces HTML5, PostScript, or PDF 1.4 (or later) output and its input format must not be the same”. Any opinions?

By the way, another addition could be Lout. I got somewhat hyped when it came into being, and was advertised a new (La)TeX too as some of the other solution... it even has a Wikipedia page... I believe it's quite dead nowadays, or at best stuck in its time, but it had interesting concepts and it does tell something, also about the possible failure of such attempts. Will SILE, Typst, Patoline, etc. stand the test of time? Ah!


UNCAUGHT ERROR

Hello
I am trying to use this plugin into my application. But it gives the " angular.js:38Uncaught Error: [$injector:modulerr]"
I have already included in angular.module as well.
Is there else i have to do ?

I just updated the documentation and hope this helps. A good starting point is looking at how the demo app is wired up: http://niklr.github.io/angular-query-builder


Consider a .tmp extension for incomplete parts instead of .zpaq. This can improve compatibility with file sync applications.

Franco, I’m syncing .ZPAQ archives to a remote server using Syncthing. While new archive parts (archive0002.zpaq, etc.) are being created, Syncthing attempts to read them to generate hashes. It detects that they are currently being modified and starts over, leading to a repeated cycle.
Is there an option for ZPAQFRANZ to create files with a .tmp extension initially, and then only change the extension to .zpaq once the part is fully completed?
Syncthing doesn’t support exclusions based on date, and the developers refuse to work on such a feature.

60_9q.zip
You can try backup with -tmp
(not very tested)

please compile that version so I can test

zpaqfranz upgrade -force

looks good for now...

With this patch, it is now safe to add to archives and run tests simultaneously.

only if the are multipart

Not sure if I should file as a new issue.
-tmp doesn't work when using index files.
To reproduce:
./zpaqfranz a "/volume1/Netbackup/test-??????" "/volume1/homes" -nosynology -tmp -filelist -ssd

That creates a file test-000001.tmp and then renames it to test-000001.zpaq at the end.
Then I extract the index file:
./zpaqfranz x "/volume1/Netbackup/test-??????" -index "/volume1/Netbackup/test.index"

And then I run the backup again, but this time using the index file. The .tmp file is no longer created:
./zpaqfranz a "/volume1/Netbackup/test-??????" "/volume1/homes" -index "/volume1/Netbackup/test.index" -nosynology -tmp -filelist -ssd

The index file do nothing in extraction
It is meant to MOVE the .zpaqs on a different storage space
What and why you are doing?

I'm doing that because the backup files are stored in the cloud (BackBlaze) and downloading the data again costs lots of money (5 TB of data downloads every time doesn't make sense). So I use the index files.
I would like to avoid using rclone as it introduces another point of potential failure. So the index files are a perfect solution to that problem.

By default, now, the backup command use .tmp


Figure out the bad coregistration pattern on temporal fit map

See the output of scripts/main_experiment2_temporal.py.

This was a mistake in the script, that used the original DEMs instead of the coregistered ones...


Baseline implementations seem non-modular

I would just like to share my experience after working with the ppo2 PPO implementation. I happily welcome objections, in case I just haven't found the right way to work with the implementations.
The ppo2 interface is the learn() function, which seems to me very opaque. Why not offer some PPO agent to which you can just feed data, and which will use it to learn?
Example 1: When wanting to train Acrobot-v1 using ppo2, I wanted to plot the total reward for each episode. You can find the code in https://github.com/openai/baselines/issues/771 - there you can see that I had to (or just missed some easier solution?) make a wrapper around the environment to capture the rewards experienced by the agent.
Example 2: I wanted to try out RND (https://arxiv.org/abs/1810.12894) on the same environment. They use PPO in the paper, so I tried to use the ppo2 implementation. Also in this case (and maybe more justified than in Example 1), I found that I had to make a wrapper around the environment, in order to ignore the extrinsic reward (they don't do that in the paper but I'm not interested in it), and instead calculate the intrinsic reward. Here is the env wrapper:
class RNDEnv:
    def __init__(self, inner_env, agent):
        self.env = inner_env
        self.agent = agent
        self.observation_space = inner_env.observation_space
        self.action_space = inner_env.action_space

    def step(self, action):
        state_next, extrinsic, terminal, info = self.env.step(action)
        intrinsic = self.agent.visit(state_next)
        return state_next, intrinsic, terminal, info

    def reset(self):
        state = self.env.reset()
        return state

    def render(self):
        return self.env.render()

Where agent.visit(state) calculates the intrinsic reward in that state, that is, the prediction error between a target network and a predictor network as defined in the RND paper.
I also want to add epsilon exploration, but I have not yet found a way to do this.
I think this is a serious issue for the usefulness of this repository. In my opinion, it would be better to define agents which you feed data (next states, reward), rather than making monolithic functions that do everything for you. Such a change would be useful whenever one wants to make an agent that has for example PPO only as a part of its internal working, or if one wants to modify the reward (reward engineering or research on intrinsic reward). Or just for having more control over what happens, which is especially useful in research.

Hi @Ploppz !  While I agree with your general criticism that baselines should be more modular (in fact, we are working on making it so), I the idea of providing agents that one feeds the data is easily extendable to cover all use cases. Specifically, different algorithms require different data from the environment / agent. off-policy algorithms (DQN, DDPG) require only transition tuples (s_t, a_t, r_t, s_{t+1}). Policy gradient based algorithms require chunks of trajectories, along with negative log likelihood of actions taken, and the value function estimates of the states. The data may be coming from one or multiple agents interacting with respective environments, and the algorithm has to be aware of that (for instance if recurrent policies are used an the states of RNN cells have to be managed).
In a sense, to me it seems like the environment itself is the right abstraction for what you call "just feed the data" - the agent supplies actions, the environment returns reward and next observations. Another modular piece that can be modified / swapped (although not as easily as environment) is the Runner (see the comment to the issue #771 ) - so I would argue that that part is modular also.
Usage of PPO2 as a part of another algorithm - I suppose that depends quite a bit on a use case; however, what we can do (in fact it is rather straightforward), is split model setup, generating a rollout, training on a single batch of data and the entire training loop into separate functions that can be used independently... Is that the kind of modularity that you had in mind?
All of that being said, the right abstractions for RL are by no means a solved problem; so if you have ideas how to make things more modular (replaceable, testable - all those good SWE practices) feel free to post them or submit a PR directly.

I am gonna keep this open for now, but at the moment I don't think there is anything actionable wrt this issue

I see now that indeed it's a rather hard problem to make it modular, and that my idea is too simple.
Maybe indeed the environment is conceptually an okay wrapper. Especially keeping in mind that the boundary between the "agent" and the "environment" conceptually is rather fuzzy. (just think about how animals' reward system is in their brain)
I'm sorry I can't come up with much useful ideas, especially because I haven't looked too much into what is usually done in such an algorithm (e.g. batching, and as you said: "chunks of trajectories, along with negative log likelihood of actions taken, and the value function estimates of the states").
About what you said that policy-gradient algorithms require - "chunks of trajectories, along with negative log likelihood of actions taken, and the value function estimates of the states" - Please forgive me if the following is a naive remark:
I don't really see why collections of trajectories, calculation of negative log likelihood and all that, cannot be done inside a hypothetical PPOAgent class that only has an input stream of (s, a, s', r) (say, manifested as a member function step(s, a, s', r)).
Doesn't it then get all the information it needs? What more information is there?
Having multiple agents acting in parallel though, is still a problem.

Theoretically - you are right, just storing old actions should be sufficient; however, practically, computing neglog probability and action can be done in a single pass through neural net, whereas if you just store actions, you'll have to do the computation again.  Also, for ppo you need the negative log likelihoods of old actions at the time when the action was taken (i.e. with the policy parameters at the time of acting) - so if you don't store them outside of the ppo, you'll have to somehow either two multiple copies of the parameters - the one corresponding to the action time, and the current one, or do a separate pass computing neglogp's of actions before computing the gradients.


Code cleanup

@elaineo no, he didn't

Oh! @gagarin55 suggested https://standardjs.com/ as a standard. It's pretty different from the airbnb style, but it has automated tools that might make things easier. I'm fine with whatever we choose, as long as we all agree on something.

Ah, I see. eslint can fix style automatically too with --fix option, but it has more features and configuration options, so it seems it better suits for a large projects


spelling, extra words

Description
A couple minor grammar and spelling tweaks to the readme.


can not send coin

{"result":null,"error":{"code":-32603,"message":"-1: Rejected transaction 9d5a41c22cc57d74629fc1e754ec8eb391d30a1907e8088765bb44ef286fb508: orphan transaction 9d5a41c22cc57d74629fc1e754ec8eb391d30a1907e8088765bb44ef286fb508 references outputs of unknown or fully-spent transaction d148e279fa5b6df22f2d473623a140f6f1d9617cd4548c02713e7cac95081a24"},"id":"curltest"}

maybe you should send after the wallet rescanning the best block