Update defaultProps.js

A more reliable defaultSortMethod. Works with uppercase/lowercase, negative to positive numbers, etc.

How does it work for strings? If I do "abc"-"def" in my console I get NaN.
Also where does it put null and undefined's?

This doesn't properly handle the ordering of null's and undefined's, which would be a clear regression.


New package: Hashicorp.Boundary version 0.13.1

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/114545&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


Run kakship asynchronously

When working in a large project like Chromium, starship git modules
like git_status can take a long time (I've experienced up to a second
of delay). Previously, the kakship call was run synchronously, which
blocks the editor while the command is running and prevents user
input. This change makes the kakship call async so movement is still
smooth while the prompt is being created.

Thanks for your contribution tom


Keybase public files (and maybe your own private files?) should be searchable with Spotlight on OS X

I'm not sure if this is even possible, given KBFS' reliance on an internet connection and lack of local cache - and of course the issues related to the encryption. However - I think that there should be an optional feature to allow KBFS files to be searchable via OS X's Spotlight search. This could be done by using an MDS extension to index the /keybase/ folder by storing an unencrypted cache when the user is logged in (and delete it when the user logs out). It would be useful to search the public folders you have loaded up in KBFS, and also your private files, at the very least to search by filename.

Hello.
Does anyone know if it's is possible to index my keybase folder with spotligth now ?


Make build.bat call-agnostic (fixes part of #5)

Currently, build.bat assumes that it's run from {cwd}/build
directory. This might not always be the case, so the present
fix proposes a solution without modifying any functionality.
set cd=%~dp0  -> set "cd" variable to the folder build.bat is in
set build=%cd%build -> set the build directory to be in working dir
pushd %cd% -> go to the working dir
IF NOT EXIST build (mkdir build) -> create build directory if none
pushd %build% -> go to the build directory
popd -> return to %cd%
popd -> return to wherever the call originated

As of the second second part of #5 , if compiling under 32-bit mode, adding WINAPI on line 673 fixes the compiler error:
673 DWORD WINAPI GetFolderContents(void *_filePath) {
However, doing so exposes a linker error unresolved external symbol __ftol2_sse. As by the guide here, you can use the following snippet to define _ftol2_sse:
#ifdef __cplusplus
extern "C"
#endif
__declspec(naked) void _ftol2_sse()
{
__asm
{
fistp dword ptr [esp-4]
mov   eax,[esp-4]
ret
}
}


v3 Snippets: Page Numbers

This is a small PR for Page Numbers (Manual & Auto):

Included the footnote element with the Auto Page Number snippet to match the Manual Page Number snippet.  Since many people may just always choose auto, they may not ever see the footnote bit as the snippet is currently setup.
changed both snippets to be a single line, or <spans>.  Reduces line count and is easier to read, IMHO.

(Is this PR too small?  I assume I shouldn't lump a bunch of unrelated snippet changes together but let me know if I should be dreaming bigger).

Small PRs are fine. Fixing one problem at a time means makes it much easier to review.


input[type=number] doesn't update when up/down arrow is clicked

html5 browsers render small up/down arrows for number input fields. if these arrows are used to update the input field, the view is updated immediately, but the change propagates to model only when the input field loses focus.
This is likely caused by us listening only on keydown and change events and something else is being fired by the browser when the small arrows are being used. In the worst case, we might need to listen on click and check if the view value changed and update the model immediately.

This number field example can be used to reproduce the problem: http://docs-next.angularjs.org/api/angular.inputType.number

fixed


Typo in Documentation (Writing Custom Middleware)

https://flareapp.io/docs/ignition-for-laravel/writing-custom-middleware
A use-statement in one of the code samples has the namespace Spatke, which I believe should be Spatie.

Fixed. Thanks!


[Suggestion for installation guide][Linux] Add MusicBot as a background service with autostart

This was done on Ubuntu 16.04 on a Raspberry Pi 2 model B
sudo nano /etc/systemd/system/
paste:
 [Unit]
 Description=MusicBot
 After=network.target

 [Service]
 Type=simple
 WorkingDirectory=/home/user/MusicBot
 ExecStart=/path/to/python3.5 /home/user/MusicBot/run.py
 #if you don't know where you have python 3.5 installed you can run 'which python3.5'

 [Install]
 WantedBy=multi-user.target

save as music.service
run
sudo chmod 644 music.service

then
sudo systemctl enable music

then
sudo systemctl daemon-reload

Now you can go ahead and summon the bot.
If the server ever restarts (ex. if you have it on a pc or a Pi) it automatically starts up.
if you want to look at the logs, simply run:
journalctl -u music

This was added to the Debian guide.

Legend, thanks!


Empty slugs can crash the site

If a news item has a title that is only charcters that are stripped away in the slug the api will crash.
For example:
ææææææ => None
æøå => None

Opened new issue #1304


Login view config doesn't respect enabled

I'm trying to implement my own login view and switching web.login.enabled = false doesn't disable the login endpoint.
{
  expand: {
    customData: true,
  },

  website: true,

  web: {
    register: {
      enabled:       true,
      autoAuthorize: true,
      fieldOrder: [
        'username',
        'email',
        'givenName',
        'surname',
        'password',
        'passwordConfirm',
      ],
    },
    login: {
      enabled: false,
    },
    logout: {
      enabled: true,
    },
    forgotPassword: {
      enabled: true,
    },
    changePassword: {
      enabled:   true,
      autoLogin: true,
    },
  },
}

@erbridge Thank you for reporting this issue.
The issue here is that website: true conflicts with the web configuration.
What website: true actually does it that it enables the web endpoints register, login, logout and me. What you can do meanwhile is remove website: true and add register and me to your list of enabled web endpoints.
I.e. so it would look like this:
{
    expand: {
        customData: true
    },
    web: {
        register: {
            enabled: true,
            autoAuthorize: true,
            fieldOrder: [
                'username',
                'email',
                'givenName',
                'surname',
                'password',
                'passwordConfirm'
            ]
        },
        login: {
            enabled: false
        },
        logout: {
            enabled: true
        },
        forgotPassword: {
            enabled: true
        },
        changePassword: {
            enabled: true,
            autoLogin: true
        },
        me: {
            enabled: true
        },
        register: {
            enabled: true
        },
    }
}

Let me know if this helped you.

There's now a PR in our config module that contains a fix for this issue. Once we have merged that in and have updated the integration to use that version, this issue will be resolved.

This should be fixed in 2.1.0, please re-open if that is not the case.


Expose a global bin

When I ran npm i -g pryjs, I expected to be able to run pryjs from the command line, just like I would with Ruby pry.
Thoughts on this?

What is the use case for needing this?
Most of the available commands are only useful for gathering context-sensitive things such as wtf or play. The benefit I can see is being able to switch modes from js to coffee, otherwise it'll be almost identical to just running node or coffee, right?
I believe Pry has a lot more useful commands for browsing code and documentation, which I've not found a great way to do in Node. This could make it much more useful without being in an actual breakpoint.
What do you think?

Well, given that 90% of my usage of pry for ruby is simply using it as a pluggable repl that's far more ergonomic to use than irb, I figured that whatever ergonomic (color, custom introspective commands like ls and so forth) you implemented for pry.js would carry over to being good reasons for improving on the (IMO) absolutely awful node REPL experience.

Fixed by 211dd65ee020ad9d6a33758860f8ac30defc2bef

Yay!

On Apr 20, 2016, at 23:13, Blaine Schmeisser<EMAIL_ADDRESS>wrote:
Closed #35.
―
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


Scrollview is not smooth

While scrolling through the details of each speaker , the scrolling experience is poor .

Hey @minervapanda .
Thanks for reporting this issue.
Would you like to work on it?

Sure. I will work on it , please assign me.

Sure, you can go ahead and get started :+1:
On Tuesday, July 26, 2016, Minerva Panda<EMAIL_ADDRESS>wrote:

Sure. I will work on it , please assign me.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/516#issuecomment-235147532,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGQtls1h4-bPMp0GbfKasRtpMdBN9zbMks5qZXTCgaJpZM4JT4bF
.

--
Best,
Harshit Dwivedi
Contact : +919968195588
GitHub https://github.com/the-dagger | LinkedIn
https://in.linkedin.com/in/hadwivedi
Google Play https://goo.gl/wl3loD | Facebook
https://www.facebook.com/daggerdwivedi

I am closing this issue because I found the once sessions are added .. the scrollview looks fine


Added support for comma-separated list for media queries for mq-layout

This PR adds support for passing a comma separated list of queries to the mq-layout component, targeting the provided queries only.
Used as such:
<mq-layout mq="sm,md,lg">
  <span> Display on sm, md and lg </span>
</mq-layout>

Readme and example updated as well, to reflect changes.
I can't see this breaking any backwards compatibility, as commas aren't used in property keys.

@tdolsen Thanks! Your work has been merged but not directly I wanted to support array syntax rather than string with commas.

Excellent! It's much better with array syntax anyway. Thanks for the excellent plugin! :tophat:


secondary-storage-vm-console-proxy-creation-failure

@rhtyd
ISSUE TYPE

Bug Report
Improvement Request
Enhancement Request
Feature Idea
Documentation Report
Other
Other

COMPONENT NAME

UI,etc

CLOUDSTACK VERSION

ACS 4.11

CONFIGURATION

ACS 4.11 , KVM , NFS
OS / ENVIRONMENT

CentOS 7
SUMMARY

I'm a newbie, and I've been trying to setup cloudstack with NFS and using KVM by following the official documents , everything goes smoothly, but when I get Dashboard , secondary storage is missing , i cant figure out where I'm going wrong.
STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

SSVM missing and even console proxy fails to start

@prashant2594 please share the logs, add a secondary storage and seed the correct version specific template.  Here's a quick tutorial on ubuntu/kvm: https://rohityadav.cloud/blog/cloudstack-kvm and you can also refer to http://docs.cloudstack.apache.org/en/<IP_ADDRESS>/
Kindly join the users ML and ask such questions there. What you're experiencing is a setup/install issue and not a bug or issue: http://cloudstack.apache.org/mailing-lists.html


Flutter tool is broken after updating to version 2.7

Steps to Reproduce

Run flutter upgrade
Try to run any flutter command

Logs

Logs

Building flutter tool...
Running pub upgrade...
../../../flutter/packages/flutter_tools/lib/src/dart/pub.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/localizations/gen_l10n.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:75:15: Error: Type 'Pub' not found.
    @required Pub Function() pub,
              ^^^
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:83:9: Error: Type 'Pub' not found.
  final Pub Function() _pub;
        ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:348:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:349:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.create,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:371:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:372:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPackage,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:406:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:407:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPlugin,
                 ^^^^^^^^^^
Error: Unable to create dart snapshot for flutter tool.
Building flutter tool...
Running pub upgrade...
../../../flutter/packages/flutter_tools/lib/src/dart/pub.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/localizations/gen_l10n.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:75:15: Error: Type 'Pub' not found.
    @required Pub Function() pub,
              ^^^
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:83:9: Error: Type 'Pub' not found.
  final Pub Function() _pub;
        ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:348:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:349:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.create,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:371:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:372:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPackage,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:406:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:407:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPlugin,
                 ^^^^^^^^^^
Error: Unable to create dart snapshot for flutter tool.
Building flutter tool...
Running pub upgrade...
../../../flutter/packages/flutter_tools/lib/src/dart/pub.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/localizations/gen_l10n.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:75:15: Error: Type 'Pub' not found.
    @required Pub Function() pub,
              ^^^
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:83:9: Error: Type 'Pub' not found.
  final Pub Function() _pub;
        ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:348:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:349:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.create,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:371:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:372:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPackage,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:406:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:407:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPlugin,
                 ^^^^^^^^^^
Error: Unable to create dart snapshot for flutter tool.
Building flutter tool...
Running pub upgrade...
../../../flutter/packages/flutter_tools/lib/src/dart/pub.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/localizations/gen_l10n.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:75:15: Error: Type 'Pub' not found.
    @required Pub Function() pub,
              ^^^
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:83:9: Error: Type 'Pub' not found.
  final Pub Function() _pub;
        ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:348:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:349:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.create,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:371:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:372:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPackage,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:406:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
eateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPlugin,
                 ^^^^^^^^^^
Error: Unable to create dart snapshot for flutter tool.
Building flutter tool...
Running pub upgrade...
^CDeseja finalizar o arquivo em lotes (S/N)? s
PS C:\Users\bruno\Documents\flutter\forked_projects\LearnUIUX\youdesign> flutter upgrade
Building flutter tool...
Running pub upgrade...
../../../flutter/packages/flutter_tools/lib/src/dart/pub.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/localizations/gen_l10n.dart: Error: The control character U+0000 can only be used in strings and comments.
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:75:15: Error: Type 'Pub' not found.
    @required Pub Function() pub,
              ^^^
../../../flutter/packages/flutter_tools/lib/src/flutter_cache.dart:83:9: Error: Type 'Pub' not found.
  final Pub Function() _pub;
        ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:348:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:349:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.create,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:371:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:372:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPackage,
                 ^^^^^^^^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:406:13: Error: The getter 'pub' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'pub'.
      await pub.get(
            ^^^
../../../flutter/packages/flutter_tools/lib/src/commands/create.dart:407:18: Error: The getter 'PubContext' isn't defined for the class 'CreateCommand'.
 - 'CreateCommand' is from 'package:flutter_tools/src/commands/create.dart' ('../../../flutter/packages/flutter_tools/lib/src/commands/create.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'PubContext'.
        context: PubContext.createPlugin,
                 ^^^^^^^^^^
Error: Unable to create dart snapshot for flutter tool.
Building flutter tool...
Running pub upgrade...

I had to reinstall flutter :/


Fix variable name typo (DOTNETCORE_VESRION -> DOTNETCORE_VERSION)

This is due to the original example being wrong, for which I have opened a PR as well: https://github.com/microsoft/vscode-dev-containers/pull/179

Closing since this got fixed and we've moved to a pre-built image.


Site and API optimization

Completed optimization for the following:

API optimization by using GZIP compression
Web optimization by lazy loading

/deploy

/deploy

/deploy

/deploy

/deploy


Feature/140 implement version of loading the results from the reports pitest

This was my idea for the Ticket #140
In order to get the coverage information I used logic from the MutationHTMLReportListener from pitest. I implemented a ResultListener I thought this is the easiest way to integrate it into the project's classpath (I mean here the project that is opened in IntelliJ and uses our plugin).
The new ResultListener creates a new xml file which contains the coverage information for each class and the total of all classes. And example can be found in this PR.
Now to use the mutation ResultListener you need to publish it to MavenLocal (like it is done for the override-plugin). Then you need to put the following line pitest("io.github.amosproj:coverage-plugin:1.0") in the dependecies of the build.gradle.kts:
dependencies {
    pitest("io.github.amosproj:coverage-plugin:1.0")
}

And the outputFormat is called "report-coverage" this also has to be set in the build.gradle.kts file:
pitest {
    outputFormats.set(setOf("HTML", "XML", "report-coverage")))
}

Finally I adjusted the XML Parser so it is able to extract the information from the new xml file as well and store it in the correct data structures. This ResultListener would make the HTML parsers obsolete.

This looks like a good approach especially since I could not find a way to add something to the classpath from withing the override plugin.
One concern I have though is with adding yet another plugin. Couldn't we add this to the override plugin and avoid another plugin we'd have to publish?

I wouldn't mind. I just thought maybe other people also want to use this mutation Listener. When I looked at example mutation listeners I noticed many people uploaded their MutationListener to maven. I would only leave the MutationListener and the override plugin seperated because then the code is more oragnized.

My main concern is having to publish another thing. But I agree that in terms of code organization and also the override-plugin being a gradle plugin and the coverage part being a pitest dependency having the coverage part separate is better and integrating it into the override-plugin makes no sense.
Maybe instead of publishing it we could build and bundle it with the intellij plugin?
Otherwise this is definitely the way to go

Why exactly would this concern you?
Building and bundling it with the Intellij Plugin sounds also good. How would you do this?

It just raises the amount of work required from us and whoever takes this over down the line...
I have added a PoC in #152 that demos how it could work without publishing.


Do not emit GEP instruction when pushing string literals to stack

I am able to crash bpftrace on one of asserts in llvm/Support/Casting.h when
building with LLVM8 and asserts enabled. Minimal repro I found is:
bpftrace -e 'BEGIN { printf("%s", "a"); }'

The crash occurs while creating GEP instruction when generating code pushing "a"
string literal to stack (see
https://github.com/iovisor/bpftrace/blob/master/src/ast/codegen_llvm.cpp#L46).
GEP expects its argument to be a Value of type PointerType, whereas type of
Constant pointed to by const_str is ArrayType. Proposed fix is to skip GEP
in this case, store opcode can handle arrays.
The issue might be hard to spot because with asserts disabled code seem to
work fine (fields of PointerType and ArrayType fortunately overlap in a way
that makes sense, PointeeTy in PointerType overlaps with ContainedType
added in ArrayType's superclass).
Test plan:
Build bpftrace with LLVM8 and asserts enabled and make sure that I can't repro
the crash anymore. In additoin pick  run:

biolatency.bt
biosnoop.bt
runqlat.bt
tcpconnect.bt

Seems to make sense. Since the tests didn't fail I guess this won't change the generated IR in most cases, correct?

Build bpftrace with LLVM8 and asserts enabled and make sure that I can't repro
the crash anymore. In additoin pick run

Is this a custom LLVM build with asserts enabled, or having asserts enabled on bpftrace is enough? I had problems in the past with LLVM debug build where we would hit some asserts as well.

Seems to make sense. Since the tests didn't fail I guess this won't change the generated IR in most cases, correct?

I think so. bpftrace -e 'BEGIN { printf("%s", "abc"); }' reports the same IR when I use preinstalled version (v0.9) and the one with this change and asserts enabled. After all getting address of the 0th element of an array is easy to optimize...

Is this a custom LLVM build with asserts enabled, or having asserts enabled on bpftrace is enough? I had problems in the past with LLVM debug build where we would hit some asserts as well.

My setup is a bit inconsistent. I build everything on my own, llvm with NDEBUG defined, bcc and bpftrace without. Because of that casts in llvm's *.cpp files did not crash, but casts performed in functions defined in headers did. I guess some of the weak symbols were pulled from bcc/bpftrace. There definitely are casts in llvm's *.cpp files that should crash if llvm is built in debug mode, for example this one: https://github.com/llvm/llvm-project/blob/master/llvm/lib/IR/Constants.cpp#L1990-L1993

I'll build LLVM in debug mode as well and give it a try. It would be a pity to trade one bad assert for another bad assert. Let's not merge until then ?

It would be a pity to trade one bad assert for another bad assert. Let's not merge until then ?

I don't think this would introduce another bad assert, but I'm almost sure there are other bad asserts out there if you use Debug LLVM.

This change makes the following assert fire: https://github.com/llvm/llvm-project/blob/release/8.x/llvm/lib/IR/Instructions.cpp#L1208. I'll do some casting before storing the value.

Now bpftrace -e 'BEGIN { printf("%s", "a"); }' does not crash with all asserts enabled, but there are still many type bugs remaining.

Now bpftrace -e 'BEGIN { printf("%s", "a"); }' does not crash with all asserts enabled, but there are still many type bugs remaining.

I tried this PR with a Debug build of LLVM 6 and got this error:
# ./src/bpftrace -e 'BEGIN { printf("%s", "a"); }'
Program aborted due to an unhandled Error:
Error value was Success. (Note: Success values must still be checked prior to being destroyed).
Aborted (core dumped)

I built clang/llvm with:
cmake ../llvm -DCMAKE_BUILD_TYPE=Debug -G Ninja -DLLVM_ENABLE_PROJECTS="clang;llvm" -DLLVM_TARGETS_TO_BUILD="X86;BPF" -DLLVM_BUILD_LLVM_DYLIB=ON -DLLVM_ENABLE_RTTI=ON

It still LGTM (maybe address https://github.com/iovisor/bpftrace/pull/667/files#r287598795), I don't think we need to fix all asserts in one PR.

Now bpftrace -e 'BEGIN { printf("%s", "a"); }' does not crash with all asserts enabled, but there are still many type bugs remaining.

I tried this PR with a Debug build of LLVM 6 and got this error:
# ./src/bpftrace -e 'BEGIN { printf("%s", "a"); }'
Program aborted due to an unhandled Error:
Error value was Success. (Note: Success values must still be checked prior to being destroyed).
Aborted (core dumped)

I built clang/llvm with:
cmake ../llvm -DCMAKE_BUILD_TYPE=Debug -G Ninja -DLLVM_ENABLE_PROJECTS="clang;llvm" -DLLVM_TARGETS_TO_BUILD="X86;BPF" -DLLVM_BUILD_LLVM_DYLIB=ON -DLLVM_ENABLE_RTTI=ON

It still LGTM (maybe address https://github.com/iovisor/bpftrace/pull/667/files#r287598795), I don't think we need to fix all asserts in one PR.

That argument should be a pointer now, but I am pretty sure I did not manage to fix all the tests (a lot of tests have up to 3 cases depending on LLVM version, I tested with llvm8). I'll rely on travis to catch tests I did not fix correctly.
Are there any tips on how to fix tests in cases like this? I modified all of them by hand incorporating feedback from bpftrace_test. That took a while.

Well, CI is happy. I'll run runtime tests and tooling tests later to see if there has been any changes there.

Are there any tips on how to fix tests in cases like this?

Unfortunately no. Ideally we should have this automated somehow, but right now this has to be done manually.
@ajor is this PR good to merge for you?

I think it was fine last time I looked at the code, but I'd still like to go through the changed tests just to make sure nothing unexpected is happening there, so not ready just yet. Normally it's not too bad to have to manually update the codegen tests, since the changes should be checked manually anyway, but I agree it is a bit annoying when it's so many like this.

FWIW I had a vim bind a while back that would execute the test with -d and paste it in the codegen file, but that's not helpful when dealing with multiple LLVM versions or several test files.

Ah I forgot about pseudo just returning 64 bit ints. I actually prefer the code before the final commit since it's simpler to just use the ints directly - can we back that one out or do you feel strongly about it?
I'd be happy to fix the conflicts in this pull request since it's my fault this didn't get in before the version freeze. I'll do that this weekend if you don't have time before then.

I won't be able to get back to this until later next week, if you feel like merging this sooner then please take over.

I dropped the last commit and fixed conflicts.


Incorrect DOI for vis-estuarine-monitoring-events

I accidentally republished vis-estuarine-monitoring-events under the same title as vis-freshwater-monitoring-events in v1.4. This was corrected in v1.5 and another section updated in v1.6.
On GBIF however, this mistake caused the DOI of estuarine: http://doi.org/10.15468/jhv16z to be redirected to freshwater: http://www.gbif.org/dataset/b2906713-07c4-41b1-887f-adfc0a24b3ab I contacted Jan Legind to solve this.
I now notice the reverse is happening: the DOI of freshwater: http://doi.org/10.15468/klsy8u is redirected to estuarine.

/cc @jlegind

now fixed

Awesome, thanks @cgendreau 👏


Valuta is missing

Hi, Valuta is missing in the list.
Valuta is a simple and fast currency conversion tool #libadwaita #GTK4 #Python 3
https://apps.gnome.org/de/Valuta/

Closed with #207.


[BUG] Only .scss files work.

Checklist before submitting an issue

[X] I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
[X] I have specifically verified that this bug is not a common user error
[X] I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Description of the bug
I read the docs, and it says I can use either an eww.scss file, or an eww.css file. Yet, when I try to use a .css file, it is completely ignored!
Reproducing the issue
Just use a CSS file instead of a SCSS file.
Expected behaviour
I want the CSS file to actually get read!
Additional context
I use eww-wayland if that helps.

As a temporary workaround try doing
ln -s eww.css eww.scss

hmm seems like I have the issue as well, the logs are telling something..
Action failed with error: Given SCSS-file doesn't exist! /home/amal/.config/eww/eww.scss
Caused by:
No such file or directory (os error 2)
Given SCSS-file doesn't exist! /home/amal/.config/eww/eww.scss
Caused by:
No such file or directory (os error 2)

I just tried and doing
ln -s eww.css eww.scss
does work, but the config does not reload automatically, I have to close and open again

First of all, scss is much better than css, and all css code is compatible with scss so I dont know why you'd wanna use a css file. for example -
.nowplaying {
  color: rgb(77, 158, 118);
  font-style: italic;
  font-weight: 600;

  border: 1px solid rgba(40, 172, 91, 0.034);
  border-radius: 6px;
  margin: 0px 5px;
  padding: 2px 20px;
  background-color: rgba(17, 16, 16, 0.363);

  /* &-icon means .nowplaying-icon */
  &-icon {
    margin-right: 14px;
    padding-right: 17px;
    border-right: 2px solid rgba(76, 80, 78, 0.096);
    opacity: 1;

   /* &-playing means .nowplaying-icon-playing */
   &-playing {
      color: rgb(95, 199, 95);
    }

    &-paused {
      color: rgba(72, 131, 207, 0.651);
    }
  }
}

Now as a temporary workaround try doing ln -s eww.css eww.scss

I don't like SCSS, it also does not do variables the same way CSS does, thus my theming system doesn't work with Eww. Do you think the linking solution will work with a Git repo?

CSS files should work, as far as I'm aware, are you on a relatively recent version of eww? try updating... otherwise I'll look into this again


can not remove "partially_running" stack

The stack state is "partially_running" and it has just one service that's "0 / 1      no       deleting"
Tried stopping, rm --forceing, redeploying...
Maybe caused by missing image (typo in image:)

#1873 should fix this.

#1873 scheduler rewrite is merged for 1.2.0. Remove of a partially_running stack works as expected:
kontena stack show test
test:
  created: 2017-03-29T13:45:53.733Z
  updated: 2017-03-29T13:45:53.736Z
  state: partially_running
  stack: test/test
  version: 0.0.1
  revision: 1
  expose: -
  services:
    foo:
      created: 2017-03-29T13:45:53.837Z
      updated: 2017-03-29T13:45:53.837Z
      image: redis:latest
      revision: 1
      state: running
      stateful: no
      scaling: 1
      strategy: ha
      deploy_opts:
        min_health:
      dns: foo.test.development.kontena.local
    bar:
      created: 2017-03-29T13:45:53.861Z
      updated: 2017-03-29T13:45:53.861Z
      image: redis:nonexist
      revision: 1
      state: stopped
      stateful: no
      scaling: 1
      strategy: ha
      deploy_opts:
        min_health:
      dns: bar.test.development.kontena.local

kontena stack rm test
Destructive command. To proceed, type "test" or re-run this command with --force option.
> Enter 'test' to confirm:  test
 [done] Removing stack test

kontena-agent
I, [2017-03-29T14:00:02.599285 #1]  INFO -- Kontena::ServicePods::Terminator: terminating service: /test.foo-1


Allows multiple comma separated packages to scan - Fixes #161

Is there a possibility to have this code at one location ? Because now it is a code duplication ,right ?
also - why not using java 8 streams ? ;-)

@ajoecker Now added a helper to remove duplication. ;)


Blog on PSP deprecation

The Comms team was requested to write up a pre-emptive look at this one. Here's the original request: https://kubernetes.slack.com/archives/C1TU9EB9S/p1611185981053300
It relates to this action taken: https://github.com/kubernetes/kubernetes/pull/97171#issue-535490768
Draft in progress is here: https://docs.google.com/document/d/1bFbHbWT2BR5Z_TWwZ11qmUcHQlWyxp7ul612DuuGhh8/edit#
Docs team is quite welcome to comment. Kaslin and Chris will get this in as a PR in the coming days: cc @bobbytables
/sig contributor-experience
/area contributor-comms
/assign @kaslin @chris-short

/triage accepted

/triage accepted

/area blog

/area blog

Cc @IanColdwater & @tabbysable (this is the issue for the PSP deprecation blog article)
/sig auth
/sig security
/priority important-soon

The PR is in for this at https://github.com/kubernetes/website/pull/27369, so this issue is all set from the contrib comms POV. Go team 🎉


Add missing FInsPacketType values

Hey,
There was 2 values missing for FInsPacketType so I'm making this PR to add them

official packets are just
dlgi2 #guri^603^1^ID #guri^603^0^ID 1807 1 othername
msgi2 0 1811 2 othername yourname
sayi2 1 ID 10 1811 2 othername yourname


License?

I'm building a similar teensy-based synthesizer and it would be great if I could include some of your code - could you put it under an open source license to make this possible?

Hello,
Feel free to use the code in your projects. I've created a license file and this project is under MIT license.
Please let me know if you happen to find any bugs in the code.


Ruby -> JavaScript regexp support

There are big differences between the regular expression capabilities of Ruby and JavaScript. It's not within the scope of this project to bridge the gap, but it does cause problems for certain typical uses of the format validator. See issue #5 for an example.
At the moment, I'm thinking that the best way to solve this is to write a plugin for XRegExp which will add some of the more commonly used Oniguruma features (character types and anchors, POSIX bracket syntax for character classes). We can then use XRegExp in place of the native RegExp in Judge's client-side format validator method.

For anyone reading this all these years later, check out https://github.com/slevithan/oniguruma-to-es to easily translate Oniguruma / Ruby regexes to JS.


论文中训练过程的y =αt * flip(ϵ) − σt *z't ， 这里y为什么要用噪声减去加噪后的图片，而且权重还不一样？diffusion的loss应该只是预测噪声吧，loss =mse(f - flip(ϵ))

SVD adopts v-prediction, please checkout this parts in the original paper for more details.

I found that in the training code, ref_model_pred was not used in the subsequent loss calculation, which seems inconsistent with the description in the paper.


feat(procdiscovery): make GetPidDetails public

They broke the helm chart.
Fix here: https://github.com/open-telemetry/opentelemetry-helm-charts/pull/1143
Until then you can add ref: e0c6b9c to the action/checkout to check out the last working commit.


Add CSS class with tag name to each tag

Adds a CSS class to each tag that includes the tag name. Can be useful for changing the CSS for an individual tag separate from other tags.
Handles the case of a tag being within a hierarchy by replacing / with - so the class name includes the whole tag name but remains valid.
Resolves #24.

Thanks :)  I merged by hand in the end as I needed to rebase and added one commit on the end as I prefer using explicit function calls rather than exposing them in the imports... apart from the exceptions of course ;) ..but that is mostly Html and a couple of Parser functions...
Thanks again - I'll do a beta release for BART to be able to pick up.  Have done a vast swathe of refactoring too!


Compatible xUDTs

Mainnet
https://github.com/nervosnetwork/ckb-explorer/blob/3feb70fed5012c15c453f46ab2a21edd1af45bf3/config/settings.mainnet.yml#L38-L42
Testnet
https://github.com/nervosnetwork/ckb-explorer/blob/3feb70fed5012c15c453f46ab2a21edd1af45bf3/config/settings.testnet.yml#L40-L43
TODOs

https://github.com/utxostack/typeid-contract-cell-deps/pull/6/
[ ] add other compatible xUDTs

References

https://github.com/Magickbase/ckb-explorer-public-issues/issues/775

Do we need to add all compatible xUDTs? Not all compatible xUDTs' deployment information is publicly available.
see: https://explorer.nervos.org/scripts

If the compatible xUDTs' deployment information can be found in the CKB Explorer, we can add it to the token list, otherwise, ignore the tokens.

agree.

This issue is just used for tracking.
As the previous discussion, let's use RUSD as a new UDT to practice the process of supporting new UDT for RGB++ protocol.
We can consider others on a case-by-case basis when necessary.


Mixins, part files and @patches are triggering OOM in gen_snapshot with compressed pointers

TLDR
As soon as number of (mixin applications + files with part directive + @patch annotations) exceed ~8000 and number of dart files exceed 16384 gen_snapshot with compressed pointers fails with
Error: Out of memory
Exhausted heap space, trying to allocate <number> bytes.

Description
A few month ago we faced in issue with Out of memory from gen_snapshot, recently we were finally able to reproduce it in example project without internal code.
It is mainly an issue for flutter, since compressed pointers don't seem to be a thing anywhere except arm64/x86_64 android, but it is caused by implementation of KernelLoader.
Every time KernelLoader is created call to ClassForScriptAt creates patch_classes_ and they don't seem to be ever cleared during aot compilation.
Steps to reproduce
example with reproduction: example project oom
dart ./tools/generate.dart && flutter build apk --release (same for --profile)

expected result: successful build
actual result:

Exhausted heap space, trying to allocate 83552 bytes.
Error: Out of memory
Dart snapshot generator failed with exit code 254
Exhausted heap space, trying to allocate 83552 bytes.
Error: Out of memory
Dart snapshot generator failed with exit code 254
Target android_aot_release_android-arm64 failed: Exception: AOT snapshotter exited with code 254
Target android_aot_release_android-x64 failed: Exception: AOT snapshotter exited with code 254
 > Task :app:compileFlutterBuildRelease FAILED

(actual number of bytes might be different between versions, but it is consistent on the same version)

Dart version: 2.19.6, 3.0.0, 3.0.6, 3.2.2, 3.3.0-197.0.dev (basically all the versions up to version used in flutter master channel)

What have we tried to do to mitigate the issue internally

disable compressed pointers: https://github.com/flutter/engine/compare/3.7.12...2ZeroSix:engine:feature/disable_compresed_pointers

it does work, but, as expected, it leads to slightly increased memory footprint, not critical, but sensible

reuse patch classes between KernelLoaders, quick implementation is here: https://github.com/dart-lang/sdk/compare/2.19.6...2ZeroSix:sdk:feature/reuse_patch_classes

We've settled on this solution for now, it's not perfect, but it works for now without any increase in memory consumption
ideally it could be tackled down with 1 allocation in aot mode in 3 (as far as i understand it is platform, vm_outline and app) allocations in jit mode. Probably it could be shared through the whole isolate(?), the only thing that stopped me from investigating it further is that I do not have enough understanding of what I could broke with it and quick solution was enough to postpone the issue for a quite some time

// @2ZeroSix thanks for the detailed reproduction instructions we will investigate.
// @alexmarkov @rmacnak-google

@2ZeroSix please make sure to include some valid open source license into your repo. thanks.

@2ZeroSix please make sure to include some valid open source license (MIT, GPL, BSD, etc) into your repo. thanks.

done

@2ZeroSix a fix went in at top of tree for this issue and the example reproduction you provided works, can you try your real application at top of tree and see if you are still running into the same issue.

@2ZeroSix a fix went in at top of tree for this issue and the example reproduction you provided works, can you try your real application at top of tree and see if you are still running into the same issue.

@a-siva I have replaced our current work-around with 448b9ef989ae4669f7a34f17019d05e0fc7e389a and it has worked flawlessly. Thank you!
I'll try to replace it in main and see if we'll found any issues with that.


Allow „Subsetting“ of InteractionValues

Allow to turn an InteractionValues object into another InteractionValues with only a subset of players part of the new one. This is helpful for plotting only a selection of interesting players and potentially other cool things down the line.

Would the objective be to create something similar to get_top_k_interactions, but instead supplying a subset of players and only retaining those values which belong to (at least) one of the players?

Yes, what I meant: So basically $C = A \setminus B$ where $A$ and $B$ are both sets of players, and $C$ is just the set-difference of both. Then all interactions with $B$ are removed and $C$ contains only interactions that can be made up of players in $A$.


Warning: Do not know how to convert MSVS attribute InheritedPropertySheets

I am trying to add my msvs property sheets to the binding.gyp configuration. Afaik this is the way to do it but the node-gyp configure keeps on throwing this warning and the generated .vcxproj does not come out the way it should.
Warning: Do not know how to convert MSVS attribute InheritedPropertySheets
I am expecting the generated .vcxproj to look like:
  <ImportGroup Label="PropertySheets">
    ...
    <Import Project="..\..\properties\default-release.props" />
  </ImportGroup>

But the PropertySheets label statys empty and my property sheets arent imported. Except for the default microsoft props file ofcourse.
  <ImportGroup Label="PropertySheets">
    ...
  </ImportGroup>

Here is a snippet of the respective importing of the msvs_props.
"conditions": [
  ['OS=="win"', {
    "defines": [
    ],
    "msvs_props": [
      "./properties/default-release.props"
    ]
  }]
]

node-gyp v3.4.0
Is the feature of importing property sheets in the binding.gyp supported in the current node-gyp version or am I doing something terrible wrong.

Did you manage to figure this out?  I think you may need to write this instead:
'msvs_props': ['$(SolutionDir)/properties/default-release.props']

Thanks for the reply! I havent solved it yet. Hacking around it by adding the settings in all property files manually to match the property sheets but that is far from ideal.
I tried your suggestion. Thought I am not sure if SolutionDir is a thing outside of visual studio, the warning stays the same.
Warning: Do not know how to convert MSVS attribute InheritedPropertySheets
After running configure and rebuild the property sheets arent listed in the project file in the build directory, so my guess is it doesnt work. I've been looking at the implementation as well for parsing the props files and my guess is the issue is there but I havent had the time to fix it, test it and do a pull request.
I might still be missing something so any other suggestions are more than welcome. Thanks again for your time and the reply!

I had a look and it seems GYP currently only knows about the following properties:

CharacterSet
ConfigurationType
IntermediateDirectory
OutputDirectory

So yes, it looks like InheritedPropertySheets won't work at this time.

Yea exactly, I am looking at doing stuff with the Additional Include Dirs,  Library - and Executable settings ect. I'll keep it like this for now. Thanks again!


Test search results with production data

Duplicated to https://github.com/thematters/developer-resource/issues/184.


Missing alternative for disallowed single quote

Disallow the following input characters
' Character data string delimiter
[Enter feedback here]
What is the proper alternative to disallow it, but still allow Mc'Donalds (surname) to include single quote in user's name?

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

ID: 971e2142-ecc0-357f-2c9f-2f9697a79d3f
Version Independent ID: 971e2142-ecc0-357f-2c9f-2f9697a79d3f
Content: SQL Server security best practices - SQL Server
Content Source: docs/relational-databases/security/sql-server-security-best-practices.md
Service: sql
Sub-service: security
GitHub Login: @dplessMSFT
Microsoft Alias: dpless

@LeszekKalibrate -- Leszek, thank you for your feedback.  Please see related issue #8473.
I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Security Warning when I open project in Visual Studio Community 2017

I get this warning when I open it saying that the build process is effected. Can you expand on why that is?
Thanks

I'm guessing you downloaded it as a ZIP?
Pretty sure that just means Windows tagged it as coming from the Internet. Same as the warnings you get if you download a Word or Excel file. There is a checkbox on the ZIP file properties you could disable. "Unblock" I think, I don't remember exactly what it says, I disable that faux-security on my machines.

Yeah thats correct. Thanks for your speedy response and for your fine work on this project.
Can't believe Azure haven't sorted DI yet!

One caveat I really should document -- the function-name attribute is global to your csproj. I didn't realize that until later. That means if you rely on the default "RegisterServices" trigger, you should only have one for the entire csproj.
This actually ends up working pretty well since Functions at the project-level should still be tightly focused and the handful of triggers you create will probably share all or most of the same services.


RFC: next-urql mergeExchangeOption should receive ctx

Summary
In a context of a jwt auth based application, best practices enforce using a short live token (jwt-token) and a longer live token (refresh_token).
A common practice si to store the refreshToken as a serverSide cookie. It's generally do in a refreshToken() method that could be happen clientSide or serverSide(on initial access)
To having a authExchange that take care of refreshing the token, we need to access ctx so authExchange can pass ctx to refreshToken when it happen on serverside so cookie header can be set.
Since it's not the case,  we need to use fetchOption to bind context to the refreshToken method
 return withUrqlClient(
    (ctx) => ({
      url: "/graphql",
      fetchOptions: {
        refreshToken: () => refreshToken(ctx),
      },
    }),

Proposed Solution
It would be simpler to also have ctx from the mergeExchange
(ssrExchange, ctx) => [
      dedupExchange,
      cacheExchange,
      ssrExchange,
      () => authExchange(ctx),
      fetchExchange,
    ]

@kitten wrote an authExchange example https://gist.github.com/kitten/6050e4f447cb29724546dd2e0e68b470#file-authexchangewithteardown-js
and an auth example is here https://gist.github.com/lionelB/65c0472304b18f9417665b5bce85991e

I’m starting to think that it may be interesting to attempt to change the API entirely, if possible.
The second mergeExchanges argument has lead to confusion and we could steer people in the right direction by simply making the entire HOC accept a single option, e.g.
withUrqlClient((ctx, ssrExchange) => ({
  /* ... options */
  url: ‘/api/GraphQL’,
  exchanges: [
    dedupExchange,
    cacheExchange,
    ssrExchange,
    fetchExchange
  ],
}))(Page);


Fix encoding of cloudinary URLs

Fixes https://github.com/OpenCollective/OpenCollective/issues/41

Merging this because needs to be on master to avoid the npm install cyclic dep

We'll also need to update the other thankyou templates.

Good point... https://github.com/OpenCollective/api/pull/294


fix(list-box): update CarbonX selection icon svg's height

Closes #2111
Changelog
Changed

adds height: 100% to the bx--list-box__selection svg so it is placed better. This is similar to what is done for the bx--list-box__menu-icon svg already 👍   (for comparison: https://github.com/IBM/carbon-components/blob/master/src/components/list-box/_list-box.scss#L456)

cc @IBM/carbon-designers do we use the chevron glyphs for dropdown variants, or should we use a 16x16 version?

@joshblack we are currently using the 16x16 version

@IBM/carbon-developers should this be merged in as-is or should we be updating the assets to use the 16x16 variant? (or both?) I believe the hbs is using the glyph variant of these icons right now.

this should be fine as-is right? I believe there are some other components using the chevron glyphs instead of 16x16 so we can cut a new issue for that

@emyarod if the spec has it at 16x16 icon we should be using that instead of the glyph so that it aligns with type

@jendowns Do you want to go ahead with the change that @joshblack suggests? Or would you feel better if we take care of choosing the sized icon? Thanks!

Just chiming in, I agree with @joshblack's suggestion

Thanks @joshblack @asudoh & @tw15egan! 🎉 I'll update the icon shortly


Inclusion of objects that are external to the exchange XML document

Prerequisites

[x] I have written a descriptive issue title
[x] I have verified that I am running the latest version of the ReqIFSharp
[x] I have searched open and closed issues to ensure it has not already been reported

Description
Currently the library does not provide support for extracting OLE objects or other objects that may be present in q reqifz file. This needs to be implemented. Question is what would be the best way to achieve this? Currently the reqif xml data is loaded into memory and a fully deserialized object graph is created and made available.
It doubt this is the best way forward for attachments. These might be large objects for which it does not make sense to load them into memory. Probably better to retrieve them (load via stream) on request.
Steps to Reproduce
NA
System Configuration

ReqIFSharp version:

[x] ReqIFSharp:         3.0.1

Environment (Operating system, version and so on):
.NET Framework version:
Additional information:

@LBRNZ can you let me know what kind of API would help you most for exposing objects? In case an external object is present, would you want to get it on request as a stream?

I created a fork and added this class: https://github.com/LBRNZ/reqifsharp_plus/blob/development/ReqIFSharp/EmbeddedObject.cs So i load every attached file into a memory stream and with the full name i can easily access them from the xhtml objects. My intention was that the files could not be manipulated once the reqif is loaded. The List of embeddedObjects is stored on the top level of the reqif:
/// <summary>
/// Backing field for the <see cref="EmbeddedObjects"/> property.
/// </summary>
private readonly List<EmbeddedObject> embeddedObjects = new List<EmbeddedObject>();

/// <summary>
/// Gets the optional preview images referenced in XHTML Attribute Values.
/// </summary>
[XmlIgnore]
public List<EmbeddedObject> EmbeddedObjects
{
	get
	{
		return this.embeddedObjects;
	}
}

thanks for the feedback, i am working on something for reqifsharp now. I do think that the implementation you have in your fork does have a memory leak and are you not worried that you run into a memory issue? If you have large embedded objects that you load into memory there might be a performance hit at some point (swapping).
I think the memory leak might appear here: https://github.com/LBRNZ/reqifsharp_plus/blob/6ca103694a5bfb92dbe547df8cda934ab841a760/ReqIFSharp/ReqIFDeserializer.cs#L206
What i am about to implement is something called ExternalObject which is the terminology used in the OMG spec with the 4 properties the spec states (data, type, width, height). The data property is a URI that (relative or absolute) that can be use to load the object from the reqifz file or from another resource in case of an absolute URI which could be something like http://www.example.com/bar.mp3.
I don't think intepreting the object is the responsibility of reqifsharp. with the uri and the type (mime-type) property a consuming app can work with it.

Yes I also noticed that problem already, but due to me not beeing an expert at all in these things :-) I have no Idea on how to dispose the MemoryStream after the file is unloaded. When I wrap it into a using command, the MemoryStream is already closed right after loading the file. Maybe I have to do a little research on this problem.
But I think your approach is a good one. You just have to give the user a handle to the file and he is responsible to interpret it

@LBRNZ i just created a pull request with a solution to read external objects. Can you have a look and let me know what you think? next step is to also support writing files to a reqifz file

@LBRNZ just released nuget version 3.1.0 which provides support for so-called External-Objects.
The AttributeValueXHTML now has a property called ExternalObjects that is a list of ExternalObjects found in the XHTML. The ExternalObject class exposes a method to determine whether the contents of the ExternalObject should be in the reqifz file or whether they are an external resource (http://www.somurl.org/image.png) that can be download using an httpclient.

use the IsDataLocal method to find out whether it is local or not.
in case it is local use the QueryLocalData method to extract the data, this method takes a Stream as argument which is populated with the content.

here you can find a test method that demonstrates how it should be used https://github.com/RHEAGROUP/reqifsharp/blob/ed80b983aeb903fee1aa6ad99141fd131ac7d400/ReqIFSharp.Tests/ReqIFDeSerializerTestFixture.cs#L132
Please let me know what you think and whether this works for you

Looks nice so far. I have to check if I can make it work with the html renderer I use, because it only gives me the url of the embedded png. So I would have to somehow search all Specobjects and its Attributes to find the external object. Thats why I created a global List of external objects. Another issue I see is that it only works with reqifz and not with reqif and a folder of files


Error dialog problems

On the current I18N branch, error dialogs do not show up.
I traced this down to a missing div with an ID of "errors".
Pull request #104 adds the div in, but there are still several issues.
You can reproduce this problem by going to the admin page and clicking the update questionnaire button at the bottom of the page.  You'll see several problems:

The update questionnaire dialog shows up even though there is an error
The error dialog is poorly formatted
There is no title on the error dialog (note: prior to the pull request, there was a title but it had the HTML formatting in it)

I'll fix the server error which allows us to duplicate this problem in a separate PR

Resolved with PR #117


fix error spam in HA log after update to ZM 1.36+

There's probably more that might need to be updated, however this fixes the immediate error spam in HA log after updating ZM to 1.36+ for the time being, while preserving the former functionality (as far as I can tell with what I am using the cameras in HA for)

did this ever get fixed?

@hugalafutro thank you for this PR, #56 has been merged and should cover this fault


Wrong links on demo page

If you navigate to the demo page, and then try to navigate back to the home page, you are greeted with a 404. This is because the links are relative and the demo is hosted on a different domain than the rest of the site.

Yeah, I need to edit the demo page since I broke it. I'll get around to that soon.

Should be fixed.


search doesn't return data if ranking data is set for dates that don't overlap data

I've just encountered this issue
ranking datrange =   '2015-01-01-00:00:00+00:00_2019-01-01-00:00:00+00:00'

And the datasource data keys
data_keys = {'2013-11-23-12:28:30+00:00_2013-12-30-04:51:30+00:00':'data/uuid/49967a55-aa96-4997-be42-642ceb1f0862/v1/2013-11-23-12:28:30+00:00_2013-12-30-04:51:30+00:00'
'2014-01-01-22:36:30+00:00_2014-01-07-09:17:30+00:00':'data/uuid/49967a55-aa96-4997-be42-642ceb1f0862/v1/2014-01-01-22:36:30+00:00_2014-01-07-09:17:30+00:00'
'2020-03-27-19:03:30+00:00_2020-06-24-09:41:30+00:00':'data/uuid/49967a55-aa96-4997-be42-642ceb1f0862/v1/2020-03-27-19:03:30+00:00_2020-06-24-09:41:30+00:00'}

As it looks for data keys that overlap the ranking data, if the data doesn't overlap then it breaks. Really it should just return the data it finds instead.

This was opened incorrectly, ranking functionality works correctly.


Oxidize synth_cnot_count_full_pmh

Summary
Closes #12227.
In a benchmark to synthesize 100 qubit linear networks 1000 times, the runtime is ~44 times faster:
(main) Took<PHONE_NUMBER>9648132s
(this PR) Took 3.920625925064087s

I would assume the main speedup is due to a more efficient circuit construction.
Details and comments

Does not address #12106, which is currently tackled in a different PR.
There certainly is some way to use iterators more by iterating over sections, but I couldn't yet wrap my head around it (and it might not give a significant speedup), suggestions are welcome 🙂

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

115 of 115 (100.0%)  changed or added relevant lines in 5 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 89.764%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
6
92.11%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
63489

Relevant Lines:
70729

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

115 of 115 (100.0%)  changed or added relevant lines in 5 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.06%) to 89.769%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
2
93.38%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
63491

Relevant Lines:
70727

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

113 of 113 (100.0%)  changed or added relevant lines in 5 files are covered.
11 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.02%) to 89.776%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
5
92.88%

crates/qasm2/src/parse.rs
6
97.61%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
63645

Relevant Lines:
70893

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

116 of 116 (100.0%)  changed or added relevant lines in 5 files are covered.
45 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.03%) to 89.724%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
5
92.11%

crates/qasm2/src/parse.rs
12
96.23%

qiskit/visualization/pulse_v2/device_info.py
28
39.71%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
63627

Relevant Lines:
70914

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

113 of 113 (100.0%)  changed or added relevant lines in 5 files are covered.
54 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.006%) to 89.762%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
94.02%

crates/qasm2/src/lex.rs
2
93.13%

qiskit/circuit/quantumcircuit.py
23
95.69%

qiskit/visualization/pulse_v2/device_info.py
28
39.71%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
63652

Relevant Lines:
70912

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

113 of 113 (100.0%)  changed or added relevant lines in 3 files are covered.
12 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.003%) to 89.843%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
94.02%

crates/qasm2/src/lex.rs
5
91.86%

crates/qasm2/src/parse.rs
6
97.61%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
65198

Relevant Lines:
72569

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

113 of 113 (100.0%)  changed or added relevant lines in 3 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 89.85%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
3
92.11%

crates/circuit/src/circuit_data.rs
9
90.73%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
65199

Relevant Lines:
72564

💛  - Coveralls


What is 'co-owners group'?

Azure communicates a schedule for planned maintenance by sending an email to the subscription owner and co-owners group.

Could you please make sure what does it mean 'co-owners group'? There is no role in built-in roles. If it means classic admin group, it should be 'Co-Administrator'.
https://learn.microsoft.com/en-us/azure/role-based-access-control/rbac-and-directory-admin-roles
I found same issue. https://github.com/MicrosoftDocs/azure-docs/issues/43304

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

ID: 289dc246-31cd-2705-45b2-69e8d7e66246
Version Independent ID: 9d55f15f-befc-762a-259b-f1b250d004cb
Content: Use the portal for maintenance notifications - Azure Virtual Machines
Content Source: articles/virtual-machines/maintenance-notifications-portal.md
Service: virtual-machines
Sub-service: maintenance
GitHub Login: @cynthn
Microsoft Alias: cynthn

I found similar issue.
https://github.com/MicrosoftDocs/azure-docs/issues/43304

@jyama4ta
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@cynthn
Can you please check and add your comments on this doc update request as applicable.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Parameter 'dimensions' are not using anymore  for embbedings.

Test:
curl https://api.openai.com/v1/embeddings \
-H "Content-Type: application/json"
-H "Authorization: Bearer AP”I_KEY
-d '{
"input": "Your text string goes here",
"model": "text-embedding-ada-002”,
}'
Please check : https://platform.openai.com/docs/api-reference/chat/create
My quick fix over : lib/langchain/llm/openai.rb:
   def embed(
     text:,
     model: defaults[:embeddings_model_name],
     encoding_format: nil,
     user: nil,
     dimensions: nil # if we expect it to be explicitly passed
   )
     raise ArgumentError.new("text argument is required") if text.empty?
     raise ArgumentError.new("model argument is required") if model.empty?
     raise ArgumentError.new("encoding_format must be either float or base64") if encoding_format && %w[float base64].include?(encoding_format)

     # Warning about the obsolete `dimensions` parameter
     if dimensions
       warn "Warning: The 'dimensions' parameter is obsolete and no longer used. The embedding dimension will automatically be determined based on the selected model."
     end

     parameters = {
       input: text,
       model: model,
       # The `dimensions` parameter is no longer included here
     }
     parameters[:encoding_format] = encoding_format if encoding_format
     parameters[:user] = user if user

@emezac I think here what we should do is:

Add the dimensions: param back in:

def embed(
  ...
  dimensions: nil
)

Use either what's passed in (an override) or use the defaults

      parameters = {
        input: text,
        model: model,
        dimensions: dimensions || default_dimension
      }

Yeap that is a better approach.

@emezac Would you be able to open a PR please? Please make sure to write tests as well.
@prdanelli Thought on this approach?

Yes I will

The dimension param now causes a error for the text-embedding-ada-002 model, however can be used with text-embedding-3-small and text-embedding-3-large.
I would suggest:

Making the dimension parameter nil by default, and just letting the API use its default.
Making text-embedding-3-small the default model, it's cheaper and better than text-embedding-ada-002.
Leaving the model param, as we do have options now.

Leaving the model param, as we do have options now.
@spikex What did you mean here? The rest of your changes were done in the linked PR btw.

There was a comment above suggesting removing the model param. All good.


Navbar needs some make over

The navbar could have an overlay like in the attached sample with orange background and an svg patten overlay that can be discussed with the maintainers. Contributors can be creative with their own ideas.
Search bar is not required.

Assigned to Debjit#8340 from Discord issues forum

@jay-io can u please assigne me this issue

is this for haktober fest?

@JASKIRAT11011 Sorry this has been taken. Someone suggested it in the forum earlier. Stay on the lookout for newer issues. And yes, this is for Hacktoberfest.

okay np

Assing me on this issue.

@thisisharsh7 We'll let debjitpurohit finish first. If you still have changes to suggest, we'll recommend discussing it with the other maintainers on our Discord forum. That is the standard way we're approving contributor suggestions for now.


E2E: go tag main config-ldflags-assets-tag slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v20.0.29
Run: https://github.com/slsa-framework/example-package/actions/runs/2539654879
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Trigger: push
Branch: v20.0.29
Date: Wed Jun 22 03:34:31 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/v20.0.30
Run: https://github.com/slsa-framework/example-package/actions/runs/2542187426
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Trigger: push
Branch: v20.0.30
Date: Wed Jun 22 12:27:49 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/v20.0.31
Run: https://github.com/slsa-framework/example-package/actions/runs/2542245061
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Trigger: push
Branch: v20.0.31
Date: Wed Jun 22 12:37:44 UTC 2022

verifier-e2e.go.tag.main.config-ldflags-assets.slsa3.yml creates a release with release assets that don't include the version number.
Ex: https://github.com/slsa-framework/example-package/runs/6996882250?check_suite_focus=true
e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml creates a release with release assets that have the tag attached to the name so you have conflicts like the runs above.

Repo: https://github.com/slsa-framework/example-package/tree/v13.0.44
Run: https://github.com/slsa-framework/example-package/actions/runs/2546520157
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets.slsa3.yml
Trigger: push
Branch: v13.0.44
Date: Thu Jun 23 03:33:48 UTC 2022
Tests are passing now. Closing this issue.

/cc @asraa who crated the verify workflow. I'll take a look next week too. Thanks for the pointers, @ianlewis it's really useful


pagination should allow minimalistic or open-end style

There are different pagination styles that currently doesn't seem to be possible with the pagination directive:

open-end: show the first-link but not the last-link
always show page number 1 instead of a a first-link
show the first-link only when we're not on the first page, same for last-link
show previous and next links only if they are active

I'm not sure, what would be the best way to expose this options in the directives attributes.

We've tried to keep our pagination component relatively simple so there are always going to be styles that it can't handle natively. If you need to implement a different style, you could always create your own templates.


Migrate to sbt-ci-release, remove Travis CI

Adds a release workflow, based on sbt-release-ci.

On tag, publishes a release to Sonatype. (Untested).
On master or series/*, publishes a snapshot to Sonatype.
On master, publishes the website project.
On master or series/* or tag, publishes the docs project.

This removes Travis CI and its associated cruft.  Our next release should be by pushing a tag.
Plugins that are transitive through sbt-release-ci are now managed through that, to reduce trouble and Scala Steward noise.  sbt-release is removed, so we had to refactor away from its Version to generate the version table on the project site.  That's most of the noise in Http4sPlugin.scala.

This will fail on release to master until I set up the secrets.  I've been testing this on my fork.


lucet-runtime: fix deb packaging by deleting ref to deleted file

unfortunately our CI doesnt cover this but i will vouch for it having passed in private CI


Implement the Subscription Substore for the Subscription module

Description
This is a step towards #50.
Subscriptions, as NFT tokens, are stored in a dedicated subsctore in the form of objects with the schema
subscriptionStoreSchema = {
  $id: 'subscription/subscription',
  type: 'object',
  required: ['price', 'consumable', 'streams', 'members', 'ownerAddress'],
  properties: {
    price: {
      dataType: 'uint64',
      fieldNumber: 1,
    },
    consumable: {
      dataType: 'uint64',
      fieldNumber: 2,
    },
    streams: {
      dataType: 'uint32',
      fieldNumber: 3,
    },
    members: {
     type: 'array',
      fieldNumber: 4,
      members: {
         dataType: 'bytes',
     }
    },
    ownerAddress: {
      dataType: 'bytes',
      fieldNumber: 5,
    },
  },
};

Attributes

price: The price of the subscription in Muzikie tokens. This value is defined at the creation time.
consumable:  This is the maximum aggregated stream cost and network fees paid using the subscription. The value starts from 80% of price. and every stream transaction sent my members of the subscription deduct the network and stream costs from it. consumption can not be less than zero.
streams: This value corresponds the number of streams accomplished using the current subscription. This number gets incremented by the stream transactions submitted by any of the members.
ownerAddress: The account who purchased the subscription. This is the only account who is able to update the members list.

Resolved by #67


Rename module to amqtt

closes #4

@HerrMuellerluedenscheid  I tried to keep it somewhat readable by splitting it up into multiple commits with "search and replace" style changes grouped into commits away from "actual changes"


[communication] type signature for shared CommunicationTokenRefreshOptions prevents mypy from checking

All of the packages under sdk/communication which contain the _shared directory are prevented from running mypy due to the typing in the init of CommunicationTokenRefreshOptions:
https://github.com/Azure/azure-sdk-for-python/blob/eeb2f3b8c7c115337f9d71166aca91db732c931e/sdk/communication/azure-communication-chat/azure/communication/chat/_shared/user_token_refresh_options.py#L13-L24
This is reproducible by running mypy==0.931 at the package level directory:
azure-sdk-for-python\sdk\communication\azure-communication-chat>python -m mypy ./azure
azure\communication\chat\_shared\user_token_refresh_options.py:20: error: Function has duplicate type signatures
azure\communication\chat\_shared\user_token_refresh_options.py:20: error: Type signature has too few arguments
Found 2 errors in 1 file (errors prevented further checking)

Tagging @turalf for visibility

This file no longer exists. mypy errors for communication-chat are now tracked in this issue: https://github.com/Azure/azure-sdk-for-python/issues/32833


Add XML documentation to interfaces and classes

This pull request adds XML documentation to the interfaces and classes in the codebase to improve code readability and maintainability.
This is a work in progress. I think this add value to the code.

Hi @bistok
I am not sure whether this kind of comments are needed. I assume they are generated, but most of the time code is self explanatory and these comments add no value. Moreover, they add noise and what is worse - need to be maintained, so there is a risk that will be out of sync with code.
I add comment only when:

I cannot express some concept in code or it is very hard to do that (rare case)
I need to make a hack (rare case)
it a public API (for instance REST API Requests)

Of course I will listen other opinions about that.

I agree with @kgrzybek, i think it only adds value if it gives additional information. Some times it is useful for an utiltity method to explain how to use it or add examples if the input and output (for complex algorithms).


Plugin Manager

Once we have https://github.com/kubeshop/monokle/issues/177 in place we should add a Plugin-manager to the UI where users can:

search for and add plugins from a central repository
add plugins from a local folder
enable/disable installed plugins
remove installed plugins
get notified if updates are available for installed plugins

Can we close this @devcatalin ?

closed since we have an initial plugin-manager in place


Added the title

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks! Next time please include all the changes to the same file in the same PR. You made two that even conflicted with each other, which is bad form.


Invert Binary Tree

Question: Given the root of a binary tree, invert the tree, and return its root.
I/P: [4,2,7,1,3,6,9]
O/P: [4,7,2,9,6,3,1]
@rudrakshi99 @raksha009 I would like to work on this issue using Java.

@OjusWiZard @tarun26091999 please allow me work on this issue. I will submit everything after it has been assigned

Please assign me this in Python.
Thank you.

I would like to contribute to the above problem in java as I am a participant of GSSOC'21. Kindly assign this to me.


chore: a more sane badge

Extension builds preview

Name
Link

Latest commit
476a5405a5ae26ca0149e17618353bbe52cc0e62

Latest job logs
Run #7428816329

DownloadDownloadDownloadDownload

radbot lol

Thank you!


Insufficient memory for config-watcher container

As it stays in the title the current memory request is not sufficient for the config-watcher container in case it is used.
On GKE it will be immediately killed with an OOMKilled error. Right now it is limited to 20M of memory which should be raised or made configurable.
https://github.com/l7mp/stunner-gateway-operator/blob/main/internal/config/dataplane.go#L36

Thanks. TBH, this was deliberately set to a very low value, just used for testing. The idea was that the Helm chart would create the default Dataplane resource at installation time and that resource would override the default set in this template. That being said, I'm OK with setting this to a higher value that will at least not result an OOM kill. Any suggestions?

When installing the charts separately (stunner and stunner-gw-op) the default value for the config-watcher is the following:
        resources:
          limits:
            cpu: 500m
            memory: 500Mi
          requests:
            cpu: 100m
            memory: 128Mi

The limit might be high, however, the request is sufficient.

Closing this, since with the introduction of the managed mode we no longer ship the config-watcher.


Should not skip users with empty bio and lack of mandatory_words in configuration

What happened:
If the mandatory_words is being omitted from the filters.yml, then bot skips the users with empty biography with a message like this:
@someparticularuser has an empty biography, that means there isn't any mandatory things that can be checked. Skip.

What you expected to happen:
Configuration should expect either a non-empty list of mandatory keywords or accept the lack of mandatory_words by accepting any bio.
How to reproduce it (as minimally and precisely as possible):
Comment the line with mandatory_words configuration and re-run the bot.
Anything else we need to know?:
Environment:

GramAddict version: 3.2.2
Device Model/Emulator Type: Android Studio
Android Version: 9.0
Instagram Version: -
Discord Ticket ID:           *(if submitting crash log)

Relevant Logs:
[06/27 15:44:14]     INFO | @someparticularuser: interact
[06/27 15:44:20]     INFO | @someparticularuser has an empty biography, that means there isn't any mandatory things that can be checked. Skip.

*Note: if you have a crash log, please do not attach the archive here as this is not a secure place to upload the sensitive data inside. Please open a ticket in Discord in #lobby and provide the ticket ID here.

Hello! It's mandatory THINGS not words :D
For example: mandatory words, bio language, specific alphabet
Probably not the best English in that message, what should I write instead?

I'm confused, docs say:
mandatory_words: [“cat”, “kitten”, “meow”]

Checks a biography for specified words. If anyone words is found in the bio, the user will be interacted with. Otherwise, the user will not be interacted with.

Ok, you said you didn't use mandatory words. Right? Mandatory words are not the only mandatory rule in the bot.
If you use other mandatory rules like the language of the bio or the specific alphabet and the profile doesn't have the biography, there is nothing to check and it's skipped.
if not cleaned_biography and (
            len(field_mandatory_words) > 0
            or field_bio_language is not None
            or field_specific_alphabet is not None
        ):


Remove version: ChrisBagwell.SoX version 14.4.2

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/sox/files/sox/14.4.2/sox-14.4.2-win32.exe/download
OK
(Automated message - build 923)

Close with reason: Package still available;


Add update live activity with alert config

Adds support for Alert Config, which causes the live activity to send notification ping/vibration/wake screen

Hi @charlesRmajor, thanks for your PR 👍
I will release a new update of the plugin including your feature soon.

Hi @istornz. Thanks a lot for the great package. When will you be releasing this feature?

@istornz — I think this hasn't been release yet? If that's true, maybe hold off. I'll submit a PR in a bit updating to use the new 'update' function. Apple has deprecated the old one ... keeping us on our toes, eh?
I'll include being able to set the staleIn the same was as the package currently has for 'create'

Hello 👋
This feature is already available with the latest version of the package 👌
Perfect @charlesRmajor I will release a new version once it will be done.
Thanks 🙏


Allow deferring starting sink until procedures are registered

Feature description (Mandatory)
If a procedure is used within a sink template query, and messages are queued up for consumption before the Neo instance is started (or during a restart), consumption may begin before procedures are registered. This results in an error in consumption and the sink will shut down and need to be restarted.
This could operate in a similar way to how APOC waits for procedures to be available before running initialization queries: https://github.com/neo4j-contrib/neo4j-apoc-procedures/blob/4.1/core/src/main/java/apoc/cypher/CypherInitializer.java#L56
Considered alternatives
No real alternatives other than hacking around to stop and start the sink during database startup
How this feature can improve the project?
This will allow using procedures within sink template queries without running into problems during startup

@ivangreene we already have this for apoc, do you want a more general approach?

@conker84 yeah I don't know of any way to wait until the procedures are available to start consuming from Kafka. I am using procedures in the Cypher template configurations, so will fail messages that were already on the topic when the server starts up. Is there a way I could make use of the apoc feature here?

Sorry I'm not sure what your issue is, do you need to wait for APOC to be registered or for other procedures?

@conker84 need to wait on some other procedures, but it works just as well to assume APOC needs to be registered. If you use an APOC procedure in a Cypher template query as a sink consumer, it will fail on any messages consumed during startup because it is not available

If you want to wait for apoc please check here:
https://github.com/neo4j-contrib/neo4j-streams/blob/3.5/doc/asciidoc/consumer/configuration.adoc
In particular:
streams.check.apoc.timeout=<ms to await for APOC being loaded, default -1 skip the wait>
streams.check.apoc.interval=<ms interval awaiting for APOC being loaded, default 1000>

@conker84 great! This should help me out, I had not seen those configuration options. It seems like the other procedures get loaded around the same time as APOC (we are calling them from the APOC initializer) so this should work for the other plugins as well


[flutter_conductor] Conductor should stop mirroring beta releases to dev

Starting from https://github.com/flutter/flutter/commit/481dfec00e96f01cf709ea196a92e4656ebe53ee, the conductor started mirroring all beta releases to the dev channel so that the dev channel would never be behind beta. However, as of https://github.com/flutter/flutter/pull/97028, users on dev have been auto-migrated to the beta channel when issuing flutter upgrade. There have been many dev releases since this change, so it is now safe to remove this code.
(Technically we could just flip the boolean at https://github.com/flutter/flutter/blob/master/dev/conductor/core/lib/src/globals.dart#L31; however, we have no future need of that flag so we should just remove it altogether to simplify the code.)

https://github.com/flutter/flutter/pull/100716 fixed this


Feature new repo structure

New Structure

new function for template creation: download_dataset_init()

new function for looping over the dataset ids: download_dataset()

all functions are now in R/

packages are loaded in R/import.R

with devtools::load_all(".") the packages and functions are loaded

We can discuss if the function names are ok ;). realised just now that they are almost the same ;).

wrong destination branch


Update icons and add instructions on how to do this

Had to fix scripts a bit to make them work again.
Couldn't update GNOME's icons due to https://gitlab.gnome.org/Teams/Design/icon-development-kit/-/issues/126
Resolves https://github.com/Relm4/icons/issues/23

I apologize for the lack of documentation regarding this, but I converted the icons from icon-dev-kit with Symbolic Preview so far, which does not rely on a script. That said, a script would be more convenient.

Well, in that case the commented-out code will not be helpful anyway, so this PR is still valid. If you can push here or separately updates to icon-development-kit too that'd be fantastic.
BTW what about stable release for this after that?

We should probably make this optional instead of commenting it out and document where to export the icons with SymbolicPreview.

This is exactly what was confusing to me. If it is not there, why not uploading icons to the target directory right away?

This is exactly what was confusing to me. If it is not there, why not uploading icons to the target directory right away?

There is still the analyze_dir function that goes into the subdirectories and extracts all the icons from there. Plus it renames the icons if there happen to be duplicates during this "flatten" operation. So it's more convenient to automate at least this part instead of extracting the icons in the file manager.

Okay, restored it with a warning if missing

Will you update it or should I do it?

I assume it would just be about changing the branch of the submodule. I can do that, but if you want to have this done right away and not in a couple of days, it'll be faster if you do it probably.