FxCopAnalyzer fix for Wox.Core (PR#3of3)

Summary of the Pull Request
Enabled FxCopAnalyzers for Wox.Core
Note: This is the final PR, the commit is on top of commits for PRs https://github.com/microsoft/PowerToys/pull/7211 and https://github.com/microsoft/PowerToys/pull/7209
PR Checklist

[x] Applies to #5129
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed

Successfully built Wox.Core after setting all warning to errors
Built PowerToys.sln
Ran all unit tests

PR has been updated to include merged changes from the 1st and 2nd PR


refactor: change connect wallet texts to english

Refactor Request
Describe the Refactor Request
The connect wallet feature is in full Portuguese language.
Definition of Done:

Change the following texts from Portuguese into the English language.

Describe Preferred Solution
Use an AI to translate the texts. High recommend Deepl. Then verify if it looks right.

@0xneves The "connect wallet" feature appears to be operating in the same language set by the user's system preferences.

We're using a library for that feature. If it is still a problem, we can do some research to see if it's possible to set the language to be always english


[WIP] badge42 ver2

🚧 work in progress ...

📌 TODO

[x] Auth System
[x] Stats Badge
[ ] Project Badge
[ ] Create a previous API deprecated guide

🛠 Stack

Framework - Next.js v12
Language - TypeScript
Auth - NextAuth.js
Database - PlanetScale
ORM - Prisma
Hosting - Vercel
Styling - TailwindCSS

It is a very great work !
For the user interface :
In /me page, you might add a "Edit badge link" to got back to / because when we login/register we go to /me page, and it is not obvious how to edit our badge, I randomly remove me in the browserbar to go back at the root page (/).
(Also on my badge I got my pool background, instead of 42 background and the level is wrong I got 5.17 instead of 4.2)

It is a very great work !
For the user interface :
In /me page, you might add a "Edit badge link" to got back to / because when we login/register we go to /me page, and it is not obvious how to edit our badge, I randomly remove me in the browserbar to go back at the root page (/).
(Also on my badge I got my pool background, instead of 42 background and the level is wrong I got 5.17 instead of 4.2)
EDIT : Also when we are already sign in, maybe redirect the user in this page : /auth/signin?callbackUrl=https://badge42.vercel.app/me&error=SessionRequired

Thank you for your good opinion!
Could you tell me the url that is working incorrectly?

@GlaceCoding
When SVG is changed in Safari browser, there is an error in re-rendering strangely...
I don't like Safari browser. 🥲

@GlaceCoding
When SVG is changed in Safari browser, there is an error in re-rendering strangely...
I don't like Safari browser. 🥲

I forget to tell you my config it was on Firefox/Windows, but it is maybe the same problem

Thank you for your good opinion! Could you tell me the url that is working incorrectly?

It works fine but the UI integration could be improved :)
When I retry to visit a new time your website, my browser suggest me this page : https://badge42.vercel.app/auth/signin?callbackUrl=https://badge42.vercel.app/me&error=SessionRequired I got this:

It is fine, but I am already connected, you should make a HTML redirection when the user is already connected (the redirection should also rewrite the browser history if i'm not wrong)

@GlaceCoding

https://github.com/JaeSeoKim/badge42/pull/67#issuecomment-1073214593

I'll take it back, I don't like cross-browsing. 😂
Anyway, it will work normally now.

https://github.com/JaeSeoKim/badge42/pull/67#issuecomment-1073221711

I am aware of this problem. This seems to be caused by adding an additional authentication system along with next-auth.

#68

@all-contributors please add @raccoman for review


when in only pronounciation mode, we should still underline the word, even if it's not translated

pronouncing skriver leaves it completely unmarked:

given that we currently the semantics of dotted underline is pronounce in the reader, it should look like this:

Shouldn't this be moved into the web repository?
I can change the visual feedback when using hear pronunciation to just highlight the word.

yes, it's a web issue. let me see if i can move it!


Deep link to a custom team tab from the personal app not working on iOS

We have two Angular applications where one is the personal app in the and the second one is team tab app. We ran on the issue of deep links to a team tab are not working when linked from the personal app in iOS devices. Deep links are working on Windows, macOS, Android phones and tablets, but not on the iOS devices. For this purpose we are using microsoftTeams.executeDeepLink() method for opening deep link. Format of the link is:
'https://teams.microsoft.com/l/entity/{teamTabAppId}/{teamTabEntityId}?webUrl=' + encodedWebUrl + '&context=' + encodedContext + '&groupId=' + groupId'
When trying to execute this link with pure JavaScript (window.open()) link is opened successfully.
Callback onComplete method of the microsoftTeams.executeDeepLink() is returning state true with no reason of not properly executing the deep link.
Previously there were some issues reported regarding the deep links not working on iOS not sure if this introduced again or something is missing in our implementation.
Additional info:
iOS version - 15.1.1
MS Teams app version - 3.21.1 Build <IP_ADDRESS>1216401/0111 (general)
@microsoft/teams-js version - 1.11.0

@mlsBlue - We are investigating your query. We will let you know once we have any update.

@mlsBlue - We tried to repro your issue. We have tested on desktop,web,android and IOS client. But its working fine at our end. We have tested on IOS 15.1.1 and Teams version 3.20.0.
Could you please check your deeplink in browser first. Its correct or not?

@Nivedipa-MSFT Deep link in the browser is working fine, as well as in the Teams app and on Android. Same link is used in all cases.

@mlsBlue - Could you please share your deeplink to test at our end?

@Nivedipa-MSFT here is the one of the deep links that works in the browser, Teams desktop app and Android device but not in iOS.
https://teams.microsoft.com/l/entity/083693d6-7398-405b-b8a6-aab0566b3c5c/askmewhy-seamless-tab?webUrl=%2Fteams%2Fworkspace&context={"channelId"%3A"19%3A2TTv5mzYEfcV4ndqEeJUoh_JxLZu_w_N1Bjahkx-y581%40thread.tacv2"%2C"subEntityId"%3A""}&groupId=5a49bf49-daee-46d0-87bc-c518e369389d
This is the link to one of our own test public rooms in one of our tenants. Not sure will you be able to access the actual room.

@mlsBlue - Not able to access the deeplink.

@Nivedipa-MSFT like I said it's on our environment. Can you send me the format of the deep link you use for iOS, or can you check our format and see if it's the correct one.

@mlsBlue - We will check your format and will let you know the updates.

@mlsBlue - We have tried the following deeplink
https://teams.microsoft.com/l/entity/afe898b5-008c-4daa-82a6-a2a5b06b254d/request?webUrl=https://teams.microsoft.com/l/entity/afe898b5-008c-4daa-82a6-a2a5b06b254d/TestTab&context=channelId:19%25253aWjkrH4gV5JJ5Lq64oued47OVmrCEHzhsNcoINtTiGmw1%252540thread.tacv2&subEntityId:6dc098d0-0b73-4fde-9bf4-be847e9cf05a&groupId=9b3de624-cafd-40dc-9a6f-23a80f79f9e8
We have tested on IOS 15.1.1 with Teams version 3.20 and we get the You can't opent this link on the mobile app. Please open in the desktop or web app. and for Teams web client its working.

We have raised the bug for this issue. We will get back to you once we have any update.

@mlsBlue - This bug has fixed now. We have tested in IOS 15.3 and Teams version 3.22.0. Could you please update the Teams version and check again.

@Nivedipa-MSFT after checking the deep links functionality on earlier iOS version I can confirm that this is fixed with Teams version 3.22.0! Will close this one.


Type Hinting

We could use mypy (http://mypy-lang.org/) and add type hinting.
I've been wanting to try and do it for a whole project for a bit. We could add it to the testing suite.

I intended to do that originally but had a hard time figuring out how.  Now that mypy is stable it's probably a good time.
I'm a big fan of strong typing, I find it helps avoid most programming errors.  Since this module is all about typing and data transformation, it'd be a cool chance to try it out.


Bidirectional dismissible causes a framework exception when routing away after it is dismissed

Steps to Reproduce

Execute flutter run on the code sample
Dismiss the dismissible by fully swiping either left or right
Tap on the "Trigger... " text and exception occurs whilst popup is opened
Close the popup and see the crash report

Expected results:
No error.
Actual results:
The error message is

A dismissed Dismissible widget is still part of the tree.

StackOverflow answers recommended to use a UniqueKey for the dismissible, which we did (no change).
The widget tree after dismissing shows, that there is one residual Container with one of the icons in it, which prevents the dismissible from being completely removed from the widget tree.

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

void main() => runApp(const BiDiDisMiss());

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

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      routes: {
        '/': (context) => const BiDiDisMissLayout(),
      },
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(backgroundColor: Colors.grey[200], body: const BiDiDisMissWidget());
  }
}

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

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

class _BiDiDismissState extends State<BiDiDisMissWidget> {
  @override
  Widget build(BuildContext context) {
    return Padding(
        padding: const EdgeInsets.only(top: 50),
        child: Column(mainAxisSize: MainAxisSize.min, children: [
          Dismissible(
              key: UniqueKey(),
              child: const ListTile(title: Text("Swipe this to dismiss first")),
              direction: DismissDirection.horizontal,
              background: Container(
                  color: Colors.green,
                  padding: const EdgeInsets.symmetric(horizontal: 20),
                  alignment: AlignmentDirectional.centerStart,
                  child: const Icon(
                    Icons.check,
                    color: Colors.black54,
                  )),
              secondaryBackground: Container(
                  color: Colors.orange,
                  padding: const EdgeInsets.symmetric(horizontal: 20),
                  alignment: AlignmentDirectional.centerEnd,
                  child: const Icon(
                    Icons.not_interested,
                    color: Colors.black54,
                  ))),
          TextButton(
              child: const Text("Tap to trigger after dismissing"),
              onPressed: () {
                Navigator.of(context).push(MaterialPageRoute(builder: (context) => const Popup()));
              })
        ]));
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        body: GestureDetector(
            child: Container(
                height: 300,
                alignment: Alignment.center,
                child: const Text("POPUP\ntap to redeem the error message", textAlign: TextAlign.center)),
            onTap: () {
              Navigator.of(context).pop();
            }));
  }
}

Logs

======== Exception caught by widgets library =======================================================
The following assertion was thrown building Dismissible-[#dff9a](dirty, dependencies: [_EffectiveTickerMode, Directionality], state: _DismissibleState#adbb5(tickers: tracking 2 tickers)):
A dismissed Dismissible widget is still part of the tree.

Make sure to implement the onDismissed handler and to immediately remove the Dismissible widget from the application once that handler has fired.

The relevant error-causing widget was:
  Dismissible-[#dff9a] Dismissible:file:///E:/test/test/lib/main.dart:40:11
When the exception was thrown, this was the stack:
#0      _DismissibleState.build.<anonymous closure> (package:flutter/src/widgets/dismissible.dart:571:11)
#1      _DismissibleState.build (package:flutter/src/widgets/dismissible.dart:580:8)
#2      StatefulElement.build (package:flutter/src/widgets/framework.dart:4782:27)
#3      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4665:15)
#4      StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:4840:11)
#5      Element.rebuild (package:flutter/src/widgets/framework.dart:4355:5)
#6      BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2620:33)
#7      WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:882:21)
#8      RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:319:5)
#9      SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1143:15)
#10     SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1080:9)
#11     SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:996:5)
#15     _invoke (dart:ui/hooks.dart:166:10)
#16     PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:270:5)
#17     _drawFrame (dart:ui/hooks.dart:129:31)
(elided 3 frames from dart:async)
====================================================================================================

E:\test\test>flutter analyze
Analyzing test...
No issues found! (ran in 0.9s)

[√] Flutter (Channel stable, 2.5.3, on Microsoft Windows [Version 10.0.19043.1288], locale en-AT)
    • Flutter version 2.5.3 at C:\Users\dlatikay\Documents\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (3 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

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

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

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] Connected device (4 available)
    • BE2013 (mobile)                    • 1cec7f13      • android-arm64  • Android 10 (API 29)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 95.0.4638.54
    • Edge (web)                         • edge          • web-javascript • Microsoft Edge 95.0.1020.30

• No issues found!

Widget tree before crash:

Widget tree after crash, vestigial remains highlighted

Hi @dlatikay-myneva, I took a look at your logs and it seems the error message has provided you with a solution, ie, you need to implement the onDismissed of the Dismissible before the item can be removed from the Widget tree.
Kindly see https://api.flutter.dev/flutter/widgets/Dismissible-class.html for more information on the Dismissible widget.
PS: I combined your sample with the example from the docs and no exception was thrown after navigating away

Thanks for taking your time to answer.
The documentation states,
“If the Dismissible is a list item, it must have a key that distinguishes it from the other items and its onDismissedhttps://api.flutter.dev/flutter/widgets/Dismissible/onDismissed.html callback must remove the item from the list.”
My Dismissible is not (part of) a list item.
I had been under the wrong assumption that in such cases, the Dismissible takes care of removing itself, which is obviously not the case.
I’ll do some more investigation and will likely close the issue.
BR
Daniel Latikaynen
From: Daniel Agbemava @.>
Sent: Thursday, 4 November 2021 09:27
To: flutter/flutter @.>
Cc: Latikaynen, Daniel @.>; Mention @.>
Subject: Re: [flutter/flutter] Bidirectional dismissible causes a framework exception when routing away after it is dismissed (Issue #92976)
Hi @dlatikay-mynevahttps://github.com/dlatikay-myneva, I took a look at your logs and it seems the error message has provided you with a solution, ie, you need to implement the onDismissed of the Dismissible before the item can be removed from the Widget tree.
Kindly see https://api.flutter.dev/flutter/widgets/Dismissible-class.html for more information on the Dismissible widget.
PS: I combined your sample with the example from the docs and no exception was thrown after navigating away
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/flutter/flutter/issues/92976#issuecomment-960546928, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWKRHGFOIMRZISUSMDNKMP3UKI7XDANCNFSM5HIZOJ7A.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

The typical use case of the swipe to dismiss feature is a list so that was my immediate assumption, however, if you want are using it outside of a list, you'll need to find a way to remove it from the widget tree after the Dismissible is done.
I used a boolean value to decide whether the Dismissible item should be in the tree afterwards and updated it's state after the widget it dismissed, kindly let me know if that works for you, however, if you have multiple items that you need to use as Dismissible outside of a list, it might not scale well.

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

void main() => runApp(const BiDiDisMiss());

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

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      routes: {
        '/': (context) => const BiDiDisMissLayout(),
      },
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        backgroundColor: Colors.grey[200], body: const BiDiDisMissWidget());
  }
}

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

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

class _BiDiDismissState extends State<BiDiDisMissWidget> {
  bool visible = true;

  @override
  Widget build(BuildContext context) {
    return Padding(
      padding: const EdgeInsets.only(top: 50),
      child: Column(
        mainAxisSize: MainAxisSize.min,
        children: [
          if (visible)
            Dismissible(
              key: UniqueKey(),
              onDismissed: (DismissDirection direction) {
                setState(() {
                  visible = false;
                });
              },
              child: const ListTile(title: Text("Swipe this to dismiss first")),
              direction: DismissDirection.horizontal,
              background: Container(
                color: Colors.green,
                padding: const EdgeInsets.symmetric(horizontal: 20),
                alignment: AlignmentDirectional.centerStart,
                child: const Icon(
                  Icons.check,
                  color: Colors.black54,
                ),
              ),
              secondaryBackground: Container(
                color: Colors.orange,
                padding: const EdgeInsets.symmetric(horizontal: 20),
                alignment: AlignmentDirectional.centerEnd,
                child: const Icon(
                  Icons.not_interested,
                  color: Colors.black54,
                ),
              ),
            ),
          TextButton(
            child: const Text("Tap to trigger after dismissing"),
            onPressed: () {
              Navigator.of(context)
                  .push(MaterialPageRoute(builder: (context) => const Popup()));
            },
          )
        ],
      ),
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: GestureDetector(
        child: Container(
            height: 300,
            alignment: Alignment.center,
            child: const Text("POPUP\ntap to redeem the error message",
                textAlign: TextAlign.center)),
        onTap: () {
          Navigator.of(context).pop();
        },
      ),
    );
  }
}

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Add an entry into kernel picker for Add Remote Jupyter Server Connection

Behind an experimental flag: jupyter.showOnlyOneTypeOfKernel


Add search to MudSelect control

Feature request type
Enhance component
Component name
MudSelect
Is your feature request related to a problem?
Is it possible extend MudSelect by adding search funsctionals
For example:

Describe the solution you'd like
I need to use that functionals for single select and multiselect
What do you think?
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
Or extend MudAutocomplete control for multiselect functionals...
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Duplicate of #1282


similar example for cli/bin environment?

Would be useful to me. Where a project would normally use something like:
  "bin": {
    "my-app": "bin/my-app.js"
  },

You're not using nodemon or an npm run script. Would you just not compile the entry point script?
We could add to this README or add another example-node-cli or something of the sort.

Also some detail about npm publishing and how the .npmignore would go would be great. I don't know these answers yet, but could help the write up if you want.


Bump Dependencies #4576406625

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/4576406625

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


Add some features of TweakIn

Spigot: https://www.spigotmc.org/resources/tweakin-1-16.93444/
Because vane has a better customization and modulation system,I'w like to use vane instead of TweakIn,but vane is lacking some features that TweakIn have,I have to install both of them.I do hope to see that vane can add some features of TweakIn.
Anyway,thank you for creating such an wonderful plugin.

Thanks for the suggestion, I'll definitely have an in-depth look at it after updating to 1.17 (which unfortunately is still blocked by waiting on a stable Paper release). It certainly has some promising ideas, and some features like the in-inventory openable shulkerboxes were already on the wish-list. Are there any features you are especially keen on having?

Now tracked by #120 #121 #122 #123 (mainly).


Virtual Host details

When using virtual hosts, is there a way to see what virtual host the info belongs to?
For example, in the Top Request (URLs) panel, I can not see to which virtual host the top urls belongs to.

Good question. You have to do some pre-processing. For instance, for the following log line, we can do:
vhost.com:80 <IP_ADDRESS> - - [02/Mar/2016:08:14:04 -0600] "GET /shop/bag-p-20 HTTP/1.1" 200 6715 "http://www.vhost.com/shop/prod-c-4" "Mozilla/5.0 (Windows NT 6.1; WOW64; rv:44.0) Gecko/20100101 Firefox/44.0" 7861

You can parse it with goaccess as:
awk '$8=$1$8' log | goaccess --log-format='%v:%^ %h %^[%d:%t %^] "%r" %s %b "%R" "%u" %D' --date-format='%d/%b/%Y' --time-format='%T'

which will output this:

BTW, seems you are using the wrong log format. Feel free to post a few lines from your log and I can take look.

Hi:
Thank you for your time...
I'm using nginx, a sample of the log file is:
vhost.com 18X.7X.1X.13X - - [02/Mar/2016:20:19:39 -0600] - "GET / HTTP/1.1" 301 178 "-" "Mozilla/5.0 (compatible; Baiduspider/2.0; +http://www.baidux.com/search/spider.html)" "-" "-" 0.182 - .

anothervhost.com 20X.0.14X.17X - - [02/Mar/2016:20:18:12 -0600] - "GET / HTTP/1.1" 301 178 "-" "Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.2)" "-" "-" 0.000 - .

My nginx log format is:
log_format  main  '$host $remote_addr - $remote_user [$time_local] $upstream_cache_status "$request" '
                      '$status $body_bytes_sent "$http_referer" '
                      '"$http_user_agent" "$http_x_forwarded_for" "$gzip_ratio" '
                      '$request_time $upstream_response_time $pipe';

And the goaccess.conf  formats:
time-format %H:%M:%S %z
date-format %d/%b/%Y
log-format %v %h - - [%d:%t] %^ "%r" %s %b "%R" "%u" "%^" "%^" %^ %T %^

After a lot of head scratching, I came up with that log-format line.
BTW: I see in a lot of examples the following for date:  %^ [%d:%^]
what does it means? I think it means "ignore" date field, is it so?
Thank you for you valuable help.

OK, the following should work:
awk '$9=$1":"$9' log | goaccess --log-format='%v %h %^[%d:%t %^]%^"%r" %s %b "%R" "%u" "%^" "%^" %T %^' --date-format='%d/%b/%Y' --time-format='%T'

Yours was pretty close though. Seems you have an extra %^ before %T.
About your question %^[%d:%^] .
For the first %^[ is telling the parser to ignore everything until it finds a [. This pattern is better than having %h - - [ since one of those fields can actually be something else other than -, in that case, goaccess will treat this request as invalid.
For the second part (within the brackets), before, goaccess did not care about the time field, however, now it is required, so you must have %t.

Tank you for your help, everything is ok now.

Glad that worked!


Cache-aware choice of TopNParams.numValuesPerPass

Currently, the available processing buffer is always used in full:
https://github.com/apache/incubator-druid/blob/566dc8c719489283f9190cefd6346bbb3f12955f/processing/src/main/java/org/apache/druid/query/topn/PooledTopNAlgorithm.java#L255
But this may not be optimal in certain cases, as demonstrated in this post by @lemire.

If someone who knows the code wants to work on this, I'd love to help out.


Add simple proposal voting example

This PR adds a basic voting smart contract with the following features:

Allows voting on multiple proposals
Each proposal is configurable by:

Timelock
Target approval rate
Maximum number of participants

Only designated admin is able to create proposals

This is a simplified version of https://github.com/eigerco/smart-proposals

@geofmureithi just added rust docs and checked add. See last 2 commits.
Requesting re-review !

If there is a specific reason not to run creation in init, please highlight it. Otherwise, LGTM

Thanks for review @geofmureithi . This contract has the capability of tracking multiple proposals. The parameters we store in the init function are only intended as "defaults", for later use in create_proposal function, that way its not needed to specify all proposal parameters on each call, just the id of the new proposal, which i think is convenient.
If an user, wants different parameters for an specific proposal during the lifecycle of this contract, it can invoke directly the create_custom_proposal as that function will require all proposal parameters, bypassing defaults.

But once you deploy you get a contact id, shouldn't we be using that?

But once you deploy you get a contact id, shouldn't we be using that?

@geofmureithi
With the current example, we can have multiple proposals in the same smart contract. So for voting we would need the contract id + the proposal id.
An example of how to invoke the contract can be checked here: https://github.com/eigerco/nebula/issues/5#issuecomment-1645208546

I have not seen any other examples approach it that way. Especially because everything else is already hardcoded hence it wont create a new proposal but rather a clone of the same proposal just with different id.

I have not seen any other examples approach it that way. Especially because everything else is already hardcoded hence it wont create a new proposal but rather a clone of the same proposal just with different id.

I was probably influenced by this . Although is true eth is a different environment.
There are other examples, like the multi swap one thought: https://github.com/stellar/soroban-examples/blob/v0.9.2/atomic_multiswap/src/lib.rs  .
Anyway, if you think there is something wrong with the current design, let me know and we can change it.

I think the problem you ate trying to solve is already solved by wasm. I see how you can see it that way but instead of your approach we would rather have a single proposal per contract. I think there is no objectively better one because it matters in use case.
Yet we are building for developers. You created a two part process where one is needed. If someone wants to reuse then they can write a bash script.
IMO lets not reinvent the wheel.


String values become integer values after replacement

Having the following template.json file:
{
  "sampleProperty": "{{placeholder}}"
}

and the substitutions.json file:
{
  "placeholder": "123"
}

running jsonPlaceholderReplacer like this:
ts-node index.ts template.json substitutions.json

gives me the result:
{
  "sampleProperty": 123
}

Considering examples from README, and the statement that jsonPlaceholderReplacer keeps original variable types, which follows these examples, it appears to be an unexpected behavior.
Quickly written unit test for this case gives me red. Is there actually an error? If yes, I would like to provide a PR with a fix and some tests.

Great.
Yes, I think it's an error.
Since you don't mind, please send a PR with the fix and its tests.
Thanks.

There's one thing I didn't take into account – it's replacing placeholders inside string context, e.g.
Before:
{
  "greeting": "hello {{someone}}!"
}

Variable map:
{
  "someone": "world"
}

After:
{
  "greeting": "hello world!"
}

I think that it is a very important one, so I've fixed current version to support this scenario added a test for it in PR https://github.com/lopidio/jsonPlaceholderReplacer/pull/20

Thanks for the PR.
I realized there was something wrong and just committed a fix which creates conflicts with PR #20 .
Please let me know if this commit fix the issue.
If it doesn't, I'd love to merge the PR with the conflicts solved.
Thanks.

Just applied your changes. Sadly, the test shows that it doesn't fix the problem.
The "should keep string numbers as string" test is redundant as this case is covered in "should keep original type of string values".
Anyway, I've merged your commit, kept your style changes and also adjusted my code to be consistent, so it's ready to be pulled.


Update controltimer.ts

Restart timer if a start is requested an we are in PAUSE mode

Hi @jftheoret!
Thank you for taking the time to implement the changes and open a pull request. Your effort is greatly appreciated!
By default, I believe controltimer shouldn't continue the countdown with the START action. This would make the CONTINUE action irrelevant and could potentially disrupt the expected functionality for other users who have built their flows with the assumption that controltimer in a paused state does not resume with the START action.
However, I acknowledge the value of having the option to enable this additional functionality. That's why I incorporated your suggested change with the addition of a configuration option in the PR for the v0.4.0 release.
Thank you for the contribution and I hope you find controltimer useful.

Hello Robert!
Thank you very much for adding this as a configured option!
Your node is indeed a very helpful addition to my software (I use it in
conjunction with the node-red-contrib-persistent-fsm node to enable
transitioning from states; it also allows me to pause the state machine and
continue processing my state, hence the need for the continue on start)
Great work!!
Jean-François
On Sat, May 27, 2023 at 4:03 PM Robert @.***> wrote:

Hi @jftheoret https://github.com/jftheoret!
Thank you for taking the time to implement the changes and open a pull
request. Your effort is greatly appreciated!
By default, I believe controltimer shouldn't continue the countdown with
the START action. This would make the CONTINUE action irrelevant and
could potentially disrupt the expected functionality for other users who
have built their flows with the assumption that controltimer in a paused
state does not resume with the START action.
However, I acknowledge the value of having the option to enable this
additional functionality. That's why I incorporated your suggested change
with the addition of a configuration option in the PR for the v0.4.0
release https://github.com/Writech/node-red-contrib-controltimer/pull/28
.
Thank you for the contribution and I hope you find controltimer useful.
—
Reply to this email directly, view it on GitHub
https://github.com/Writech/node-red-contrib-controltimer/pull/26#issuecomment-1565663031,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKBDH46VUZFQPJLTBFBAEDXIJMYZANCNFSM6AAAAAAW7LPHGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Consumer  client.id is named ureplicator-null-null-null-1

I debuged  the module which is Work-3.0 and found the name of client.id in console logging，It is strange .
2019-11-07 18:51:03  [ org.apache.kafka.common.config.AbstractConfig:279 ] - [ INFO ]  ConsumerConfig values:
auto.commit.interval.ms = 5000
auto.offset.reset = latest
bootstrap.servers = [vm01.com:9093]
check.crcs = true
client.id = ureplicator-null-null-null-1
connections.max.idle.ms = 540000
enable.auto.commit = false
exclude.internal.topics = true
...
Here is the code:
com/uber/stream/ureplicator/worker/WorkerInstance.java:128
String clientId = String.format("ureplicator-%s-%s-%s", srcCluster, dstCluster, routeId);
com/uber/stream/ureplicator/worker/WorkerStarter.java:82
controllerWorkerHelixHandler = new ControllerWorkerHelixHandler(helixProps,
helixClusterName, workerInstance);
com/uber/stream/ureplicator/worker/WorkerConf.java:303
public static Options constructWorkerOptions() {
final Options workerOptions = new Options();
workerOptions.addOption("help", false, "Help")
.addOption(FEDERATED_ENABLED, true, "Whether to enable federated uReplicator")
.addOption(CONSUMER_CONFIG_FILE, true,
"Embedded consumer config for consuming from the source cluster.")
.addOption(PRODUCER_CONFIG_FILE, true, "Embedded producer config.")
.addOption(HELIX_CONFIG_FILE, true, "Embedded helix config.")
.addOption(CLUSTER_CONFIG_FILE, true, "Embedded cluster config.")
.addOption(TOPIC_MAPPING_FILE, true,
"Path to file containing line deliminated mappings of topics to consume from and produce to.")
.addOption(ENABLE_TOPIC_PARTITION_COUNT_OBSERVER, true,
"Configure the uReplicator to observe destination topic partition count")
.addOption(OFFSET_COMMIT_INTERVAL_MS, true, "Offset commit interval in ms")
.addOption(ABORT_ON_SEND_FAILURE, true, "Configure the uReplicator exit on a failed send.")
.addOption(ENABLE_FILTER, true,
"Configure the uReplicator to filter message to send to dst cluster.")
.addOption(HOSTNAME, true,
"hostname for this host")
.addOption(REGION, true,
"region for worker instance")
.addOption(METRICS_PREFIX, true,
"metrics prefix")
.addOption(GRAPHITE_HOST, true, "graphite host")
.addOption(GRAPHITE_PORT, true, "graphite port")
//
.addOption(GRAPHITE_REPORT_FREQ_IN_SEC, true, "graphite report frequency in seconds")
.addOption(ENABLE_JMX_REPORT, true, "enable jmx report")
.addOption(ENABLE_GRAPHITE_REPORT, true, "enable graphite report")
//
.addOption(WORKER_PORT, true, "worker port");
return workerOptions;
}
How to get these value if these are not defined in options ， It's better to make it more evident for people to understand according to the definition.

What's your config for worker?

looks like a bug,  we will comment after discussing

@Technoboy-  I cannot clearly understand the  ConsumerFetcherThread Constructor (this.kafkaConsumer = new KafkaConsumer(properties)),   Serialization in kafkaconsumer configurations parameter are key.deserializer=org.apache.kafka.common.serialization.ByteArrayDeserializer and value.deserializer=org.apache.kafka.common.serialization.ByteArrayDeserializer .
I created a producer to write data，the configured serialization parameters are
properties.put("key.serializer", "org.apache.kafka.common.serialization.StringSerializer");
properties.put("value.serializer", "org.apache.kafka.common.serialization.StringSerializer");
I don't quite understand why ureplicator consumers can poll data, Their serialization is different.
producer  StringSerializer   -->  consumer  ByteArrayDeserializer
I hope you can understant what i mean.  I'm looking forward for your reply.

all the data will be transport via byte array, right ?
so ,if producer uses StringSerializer, the underlayer channel(socket) will make it to byte array with easy way like : String.getBytes("UTF-8");
and consumer uses ByteArrayDeserializer , but consumer does not known what the byte array mean, what the byte array can deserialize  to Json, to xml, to Object and etc. so producer and consumer have to negotiate before producing message and deserialize message.
you know that all the storage store data via byte array.

Thank you for solving my doubts.  I forgot the relationship between the generics of kafkaconsumer and serialization.

reference to https://github.com/uber/uReplicator/pull/296
@xinzhuxiansheng


Private page to connect to my server of MultiOgar

Hello, I wonder how I could have my own website to connect to my server agario.
I explain?
Let's say between the page and choose a mode and I can connect.
I want to make a agario with about 3 or 4 servers but not how.
Example: http://agariov.com/

I have not explained well:
I need a client that I can install on my machine that has minimap mode selector (Multiple Servers), switch skins.

Client sid- I'm just not going to say it anymore. 😞

Is there some Ogar forum?  If this many people are going to make an issue with general question like these, there should be one

Just skid this http://72.k.vu/out.html

?

http://72.k.vu/out.html, is pretty easy to take and modify


Naming collision: sg is a standard linux tool

I know there was a recent discussion about semgrep, but neither tool should use sg. Sg is a standard linux binary, see https://man7.org/linux/man-pages/man1/sg.1.html
This has been around for decades.

Please see this issue https://github.com/ast-grep/ast-grep/issues/56


How to check for alarm stops when stopping from notification stop button? and onTap of Notification.

Alarm plugin version
4.0.0-dev.3
How i check for is my alarm is stopped from notification stop button.
In my app to stop alarm i am using Alarm.stop(id); to stop the alarm so here after i called it i can do something in my code,
but what about stopping alarm from notification button? here i have to call a function when i stop alarm from notification
as me, you can provide onStop method when stop from notification stop button.
If there is method like this then everyone can perform some task on alarm stop from stop button of notification.
And also need onTap of notification
If there is onTap for notification then user can perform task accordingly.
Additional context
For my app i have to show non dismissible notification when user stops the alarm. This is possible when i use  Alarm.stop(id); but need to do from notification stop button. I used flutter_local_notifications for non dismissible notification.
And also want to open Alarm screen when tap on notification, so for it onTap is useful.

Hi @dhavalmnjtech
Thanks for your interest in the package.
I'll consider adding a onStop callback when the alarm is stopped from the notification and a onTap when app is opened from the notification. I'll keep you updated here.

Hello @gdelataillade

I was working on it but faced a technical limitation on Android. If someone with Kotlin experience could help me this PR #275 it would be great !


LBRY Crashes on watching any video after upgrade to 0.52.2

Bug
Upgraded from 0.39.1 to 0.52.2.
Trying to watch any video causes LBRY 0.52.2 to crash after a second or two, though I can still hear the video playing in the background.
I ran LBRY in command-line mode and got the following:
C:\Program Files\LBRY>lbry /?

C:\Program Files\LBRY>
SDK already running
Sandbox listening on port 5278.
[10:36:55.446] [info] Checking for update
[10:36:56.164] [info] Update for version 0.52.2 is not available (latest version
: 0.52.2, downgrade is disallowed).
[10760:0212/103711.355:ERROR:gl_surface_presentation_helper.cc(260)] GetVSyncPar
ametersIfAvailable() failed for 1 times!

System Configuration

LBRY App version: 0.52.2
LBRY Installation ID: 4CndnGTUR7uN5KbGudPJ7LsasV1S8KGZ3CX9T2zd5BbJ53yEqhFpG8fpNC
TfbBcx1i
Operating system: Windows 7 Pro x64

Thanks for the report - can you make sure the daemon version under help is 0.106?

Thanks for the report - can you make sure the daemon version under help is 0.106?

Yes, seems to be:
Daemon (lbrynet) | 0.106.0

I'll see if I can make a build for you to test this week.

https://github.com/electron/electron/issues/32317 looking at this now.

I have the same problem, audio runs behind a Windows appcrash message. Running 0.52.2 in Win7 64-bit. Lbry has been working fine since long, this happened after accepting a recent in-app offer for an update.
Went back to 51.1. It now gave message about "incompatible daemon" and continue anyways, which I did. Worked fine, no appcrash when playing a video.
I then accepeted the in-app offer for 52.5, now it crashes again when pressing play.

I've noticed that adding --disable-cpu to the command line when launching LBRY.exe will produce a black screen while the app crashes, and --disable-gpu will produce video and audio playing in the background but the app still crashes.

Hello!
I also faced this problem. Version v0.52.5, Windows 7 x64.
But I've noticed that the app doesn't crash, if the video is initially without sound - I uploaded a video made with sound off and it played fine and the app didn't crash. Maybe this will help...

https://www.electronjs.org/releases/stable?version=15#15.3.5 Thanks for finding this - there seems to be a patch in electron 15.3.5 that fixes media crashes in older windows version. we're on 15.3.2. will try asap!

patch in electron 15.3.5 that fixes... we're on 15.3.2. will try asap!

Great, thanks hope it fixes the issue, personally I wish there was an option to render the video using libVLC with electronjs, which is possible, though it is more readily convenient for developers to use a BrowserWindow to render video from webkit, I guess that's what is being done here as I saw this same error being discussed for other chromium\webkit based projects.
Check out the wcjs-player & wcjs-prebuilt modules if you want to go this way, they are node wrappers for the webchimera.js video player, which uses VLC in the background. (I'm not affiliated with them, just currently using the modules for something similar in one of my projects with success so far.)

Also I wanted to say VLC has support for MKV and AVI formats which webkit doesn't support, so that'd be nice too because I have to open some videos on LBRY in VLC now even before it started crashing every time.

I can confirm 0.52.6-alpha-2 currently fixes this crash.
I was able to watch videos with no crashes using it, thanks.

Yes 0.52.6-alpha-2 confirmed working here too, thank you B)


!! Add sharding support !!

!!!!!

https://github.com/jonas747/yagpdb/commit/b932f2392f4df262d42b336da0eec8cd0f948bac added this


landrush-ip is installed in all virtual machines under libvirt

This happens even if landrush is not enabled.

This is a regression as for previous versions we need to explicitly enable landrush in the Vagrantfile.

This is a regression as for previous versions we need to explicitly enable landrush in the Vagrantfile.

You still have to enable Landrush explicitly. Installing landrush-ip is actually not harmfull, but in case Landrush is not enabled it seems odd. See also https://github.com/Werelds/landrush-ip/blob/master/lib/landrush-ip/plugin.rb.


Improvement on adblocker

Please read here for reference:https://www.kaushik.net/avinash/adblock-tracking-google-analytics-code-metrics-reports/

This is not necessary.  The id="ad-container", id="ad" and class="banner" is enough for different AdBlockers to detect and hide or remove this emulated ad banner.  After that it will be removed from the DOM, so no needs to set the display and negative top.

I'm testing this with uBlock and it's not tracking...

That's why I created the "ins" element that is used by Adsense, probably the most used ad platform.
var node = dom.appendChild(dom.document.body, dom.createElement('ins'));
Apart from this do you know how can I test if Komito is correctly tracking adblocks? I don't see anything in the debug...

uBlock filters the ins tags only for this case:
publicdomainq.net##ins.adsbygoogle
https://github.com/uBlockOrigin/uAssets/blob/master/filters/unbreak.txt#L775
Please post an issue to add support for uBlock and we will implement it a soon as possible.

To enable AdBlockers tracking, you need to enable this feature, because it is disabled by default:
'trackAdblock': 0,    // Tracks page views with blocked ads. (Experimental)


Fix/stderr

Decided Approach:
I noticed that exceptions thrown and eventually caught in mlem/cli/main.py utilize the cli_echo context manager and the echo function to print exceptions to the console.
The cli_echo context manager sets the echo function to print to stdout through a Console instance provided by the Rich package. By default, Console will write to stdout, hence the reason why exceptions are printed to stdout.
I did some research and discovered that the Rich package supports consoles that can print to stderr by passing stderr=True when creating the Console. With this knowledge, I created a Console instance called error_console with stderr=True. If an error needs to be printed to stderr, we can utilize the error_console and if we need to print to stdout, the existing console which prints to stdout can be used.
I then introduced a new context manager called stderr_echo that will switch the echo function’s underlying print function to error_console.print when an error needs to be printed. If printing to stdout is required, then we will utilize the exisitng cli_echo context manager that will switch the echo function’s underlying print function to console.print.
With this approach, we use the existing Rich package for printing to both stdout and stderr thus providing consistency. Additionally, it allows us to keep the signature of the echo function the same and let Rich manage printing to different streams under the hood.
Alternative Approaches:
I thought about other approaches to implement printing to stderr, for example, using sys.stderr.write directly. The problem with this approach is that the echo function currently excepts multiple parameters including a Text instance (along with other configurations) specific to the Rich package, however sys.stderr.write requires only a string parameter.
If we were to use this approach, we’d need to write code to only accept a string parameter (but not a Text instance) if we’re printing an error. It is possible through function overloading, however, it’s definitely a cleaner implementation to handling this through the Rich package.
Related Tasks:
If this approach is accepted and merged, I will update the contribution guide to explain the usage of the cli_echo and stderr_echo context managers along with the echo function.
Testing:
With the ability to print to both stdout and stderr, it is important that we are able to develop tests that are capable of reading and differentiating outputs from both these streams.
In the current cli test module, the CliRunner instance writes both stdout and stderr streams to result.output. If a test needs to differentiate where the output came from, it is unable to do so.
I’ve performed a couple updates to the cli test module.

I’ve updated the CliRunner instance by using mix_stderr=False, which outputs the stderr stream to result.stderr and outputs the stdout stream to result.stdout.
I’ve updated all tests in the cli test module to use result.stderr and/or result.stdout for their assert statements.
I’ve update cli tests so that error messages for assert statements will now print the stdout stream, stderr stream and exceptions by using (result.stdout, result.stderr, result.exception).
I’ve implemented tests to ensure that MlemError and Exception are printed to stderr.

close https://github.com/iterative/mlem/issues/404

Hi @aguschin @mike0sv, I saw codecov threw a warning because the code coverage was 88.88%, however, the target amount is 89.39%. It suggested that stderr_echo should be covered in the unit tests. I've updated the PR to include a test for stderr_echo.

Ok, I see the test of DVC extensions fails. Fixing that test. Since most of CI jobs were passing except for that test and few jobs that didn't have time to complete, I'm merging this.


Feature/fe 179/enhance mobile web app

The camera view might be a bit smaller on large mobile screens (only if the device is in portrait orientation), but I think it's okay since we inform the user to rotate to landscape for a better experience.

I have some layout issues while testing.
First on Android Expo App:

Second on Android web App:

Hmmm, I know why it happens.. will fix it right away 🐛 🐛 .

Hmmm, I know why it happens.. will fix it right away 🐛 🐛 .

Mobile browser layout issue
The first issue is fixed in the latest commit, but the second one is not a bug, perhaps I made it like that on purpose:
When we're on Portrait orientation, and due to the browser address bar provided by IOS we can't get the useful width of the view port(known issue in the community), which gives us the ability to set the right width to the mask and camera as well:

And if we try to use the css 100vh value or Javascript window.innerHeight both solutions will give us the whole height including the address bar height, which will result like following:

So that's why I made the mask and the camera smaller only on mobile browser and Portrait orientation.


Change file naming convention for PLOS 'annotation' DOIs

Right now the doi_to_path() etc transformations require XML parsing for annotation DOIs to get the journal name. Need to change this so that the transformations don't rely on anything outside of the DOI string.

Working on this in https://github.com/PLOS/allofplos/tree/annotations

This was merged, if you include the words "closes #28" in the pr notes or commit message, it will automatically close the issue when merged.

Oh that is clever! Thanks @mpacer
closes #28.


File could not find the specified path

I have the same issue here, I can't install the python extra files
I have this error code:
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for fairseq
Failed to build fairseq
ERROR: Could not build wheels for fairseq, which is required to install pyproject.toml-based projects
Sadly the project is so far not made for windows I believe, it's mac/ linux friendly

I found a solution:
run cmd this way

(or just cd you're way there)
then run this command "python infer-web.py" and it should work

So is there a file named python.exe in runtime directory？


GitHub Test Project Separation

As part of organizational improvements, there is a need to create a dedicated repository specifically for the CS Internship test project. This will help in better organization, version control, and collaboration on the test project.
Proposed Implementation:
1. Create Dedicated Repository:
  - Establish a new GitHub repository with an appropriate name for the CS Internship test project.
2. Repository Structure:
  - Organize the repository with clear directory structures, considering different components or stages of the test project.
  - Set up a README file providing essential information about the project, including setup instructions and guidelines for contributors.
3. Access Permissions:
  - Define and configure access permissions to ensure that only relevant team members have appropriate access to the repository.
Expected Outcome:

A dedicated GitHub repository specifically for the CS Internship test project with a well-organized structure and proper documentation.

Additional Considerations:

Collaboration Guidelines: Include guidelines on collaboration, branching strategies, and code review processes in the README file.
Issue Tracking: Utilize GitHub Issues to track tasks, enhancements, and bugs related to the test project.
Continuous Integration: Integrate with a CI/CD system if applicable to automate testing and deployment processes.

Use Case:
The dedicated repository will streamline collaboration among CS Internship team members, providing a centralized and organized space for managing the test project.

Create the test repo under cs-internship.
After the base test repo was created, this folder which is used as a sample for tests should be moved to the newly created base test repo.
Replace new URLs in test methods.

@BSeyfi

I found the document a bit challenging to decipher, but if I’ve grasped it correctly, it seems this repository is intended to serve as an example for a "CS Organization". It will include sample documents and badges. If my understanding aligns with the purpose, how about we consider the name sample-org-contoso? It has a nice ring to it! 😊
@hootanht @BSeyfi

Agree with sample-org-contoso.
@mehrandvd

It's now available at: https://github.com/cs-internship/sample-org-contoso
@BSeyfi

Thanks a lot.
@mehrandvd


please add an OSS license to the repo

Howdy! There are some commercial products that want to package this repo, but cannot until there is an explicit license. It's legally dangerous for anyone to use your code without some form of license on the code. Most CF projects are Apache 2 licensed.
Thanks!

Taken care of by this merged PR: https://github.com/cloudfoundry/capi-k8s-release/pull/30


Manage Soft Credits page does not allow mix of percent and amount

Both the Manage Allocations page and the Manage Soft Credits page distribute parts of an Opportunity among other entities, in this case GAUs and Soft Credits. I think our users will expect a consistent experience when manipulating Opportunities in this way.
Manage Soft Credits does not allow a mix of percentage and amount, but Manage Allocations does.
Updated to reflect https://github.com/SalesforceFoundation/Cumulus/issues/2233
Updated to reflect https://github.com/SalesforceFoundation/Cumulus/issues/2242

👍

**lurch: add

Tracking W-011083

Nice catch @chrismcmahon

Should we separate these into two items? One for the percent/amount, one for the error message display revision needed for the Manage Allocations page?

yep, two completely separate and unrelated items.

W-7781131

Please follow this Known Issue in the Trailblazer Community for updates.


Subscribe mandatory webhook

https://shopify.dev/apps/webhooks/configuration/mandatory-webhooks

This has been.


Add pgBouncer into Docker image

Details
Using osm2pgsql for PgOSM Flex creates a relatively large number of connections to Postgres based on the number of tables created by the Flex output. This was discovered here (https://github.com/rustprooflabs/pgosm-flex/pull/226#issuecomment-1059809664) and answered on the osm2pgsql discussion here.  Now documented on osm2pgsql.org:  https://osm2pgsql.org/doc/manual.html#number-of-connections
It is unlikely this behavior will change in osm2pgsql so the plan is to bake pgBouncer into the Docker image, example standalone pgbouncer image here.  Until this is in place, running --append mode will require upping the max_connections to at least 210, preferably 300 to give headroom.

It's unlikely at this point that PgBouncer will be directly added into this project's Docker image.  First challenge is that as of now, pgbouncer does not actually provide any benefit.  See https://github.com/openstreetmap/osm2pgsql/discussions/1650#discussioncomment-2570706.
Further, putting pgbouncer into the docker image means it needs to be maintained then forever, and I don't see the great benefit to adding that overhead to this project at this time.

If osm2pgsql changes their connection handling behavior to not hold onto idle connections this can be reconsidered.  Though, if they do that, the biggest portion of the problem this was trying to solve would be removed too... either way: closed

It looks like using pgbouncer in transaction mode might be an option again.  Updated discussion on osm2pgsql.  The PR was merged in pgbouncer recently, not in tagged version yet.

Once again, closing... Recent work in osm2pgsql to reduce idle connections has removed my reasoning to work further on this. See https://github.com/rustprooflabs/pgosm-flex/pull/376 for details. The initial import w/ default layerset dropped from 43 connections down to 10. Replication updates dropped from 207 connections down to 17.


Upgrade actions/checkout v3.0.2 -> v3.1.0

This PR contains the following updates:

Package
Type
Update
Change

actions/checkout
action
minor
v3.0.2 -> v3.1.0

Release Notes

actions/checkout
v3.1.0
Compare Source

Use @actions/core saveState and getState
Add github-server-url input

[ ] If you want to rebase/retry this PR, click this checkbox.

Suggested commit message:
Upgrade actions/checkout v3.0.2 -> v3.1.0

See:
- https://github.com/actions/checkout/releases/tag/v3.1.0
- https://github.com/actions/checkout/compare/v3.0.2...v3.1.0

Build failed with:
2022-10-05T01:01:28.5902491Z Requested labels: ubuntu-22.04
2022-10-05T01:01:28.5902525Z Job defined at: PicnicSupermarket/error-prone-support/.github/workflows/deploy-website.yaml@refs/pull/277/merge
2022-10-05T01:01:28.5902541Z Waiting for a runner to pick up this job...
2022-10-05T01:01:29.0058201Z Job is waiting for a hosted runner to come online.
2022-10-05T01:01:33.1965146Z Job is about to start running on the hosted runner: Hosted Agent (hosted)
2022-10-05T01:01:36.3681720Z Current runner version: '2.297.0'
2022-10-05T01:01:36.3721500Z ##[group]Operating System
2022-10-05T01:01:36.3722187Z Ubuntu
2022-10-05T01:01:36.3722543Z 22.04.1
2022-10-05T01:01:36.3722872Z LTS
2022-10-05T01:01:36.3723168Z ##[endgroup]
2022-10-05T01:01:36.3723626Z ##[group]Runner Image
2022-10-05T01:01:36.3724072Z Image: ubuntu-22.04
2022-10-05T01:01:36.3724419Z Version: 20220925.1
2022-10-05T01:01:36.3725114Z Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20220925.1/images/linux/Ubuntu2204-Readme.md
2022-10-05T01:01:36.3725886Z Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20220925.1
2022-10-05T01:01:36.3726491Z ##[endgroup]
2022-10-05T01:01:36.3726852Z ##[group]Runner Image Provisioner
2022-10-05T01:01:36.3727277Z <IP_ADDRESS>-main-20220916-1
2022-10-05T01:01:36.3727678Z ##[endgroup]
2022-10-05T01:01:36.3728517Z ##[group]GITHUB_TOKEN Permissions
2022-10-05T01:01:36.3729237Z Contents: read
2022-10-05T01:01:36.3729831Z Metadata: read
2022-10-05T01:01:36.3730254Z Pages: write
2022-10-05T01:01:36.3730712Z ##[endgroup]
2022-10-05T01:01:36.3735078Z Secret source: Actions
2022-10-05T01:01:36.3735649Z Prepare workflow directory
2022-10-05T01:01:36.4869704Z Prepare all required actions
2022-10-05T01:01:36.5096917Z Getting action download info
2022-10-05T01:01:36.6765178Z ##[error]The operation was canceled.

That seems to be a step before execution of this action. Also, the Build and verify action did pass. Will trigger a re-run.


P.adjust needs to be done individually for each estimate (not intercepts)

currently adjusts non-intercept p values all at once


Hello world in Kannada

Issue #351 Added Hello World in Kannada

ISO: correct
Translation: correct
Approved

@avinashupadhya99 Please comment your account number so I can send you coins! If you don't have an account you can create one by downloading the app.

My account number - 94cf53921e9c5ddf48dc5589b7cdf111f7acdc93dc3073b94880416d4676b9ae . Thank you.

@odetolataiwo 500 coins!


Repo sync for protected CLA branch

The pull request is created from master637606719993900995 to master to fix git push error for protected CLA branch

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


The Alpha channel cannot be change between different themes

Describe the bug
When i set a token like this, at Light Theme:
backgroundHover: 'rgba(0,0,0,0.9)',
and switch change it in Dark Theme:
backgroundHover: 'rgba(255,255,255,0.5)',
The color does change, but the alpha channel stuck at 0.9 between themes
Your minimal, reproducible example
none
Steps to reproduce
None
Expected behavior
The alpha change actually change base on theme
How often does this bug happen?
None
Screenshots or Videos
No response
Platform
window
tailwindcss-themer version
4.0.0
Additional context
No response

Hiya! Thanks for opening the issue. Let me look into it.

This is a legitimate issue and here is a reproduction example https://codesandbox.io/p/devbox/117-repro-7yqchp
My wife says I've been working too much today, so I'll start looking into potential solutions later, but for your amusement, here was my "maintainers' 5 stages of grief" in response to seeing this issue.

😏 Denial: This should work just fine; I have a e2e test written that covers this, but let's just be sure
🤬 Anger: Crap. I was able to reproduce it https://codesandbox.io/p/devbox/117-repro-7yqchp and the snapshot output of that test is messed up
😅 Bargaining: Oh well, this is probably an easy fix
😰 Depression: Oh darn, this isn't straightforward
😌 Acceptance: Oh well, I'll look into it soon

This hasn't left my mind!
The fix for this will be tricky and I'm currently evaluating options.

I have a proof of concept made using an approach that parameterizes the alpha value using css vars. This won't support the old opacity approach (e.g. classes like bg-opacity-50), but those are deprecated anyway.
For example a class like bg-primary that outputs the following css before the change...
:root {
  --colors-primary: ...
}

.darkTheme {
  --colors-primary: ...
}

.bg-primary {
    background-color: rgb(var(--colors-primary) / 0.5); /* 0.5 hardcoded because of this bug */
}

...would now output a class like...
:root {
  --colors-primary: ...
  --colors-primary-alpha: ...
}

.darkTheme {
  --colors-primary: ...
  --colors-primary-alpha: ...
}

.bg-primary {
    background-color: rgb(var(--colors-primary) / var(--colors-primary-alpha));
}

Reference discussion for background on alpha support: https://github.com/RyanClementsHax/tailwindcss-themer/discussions/95

I haven't forgotten about this. I plan on working on it soon.

Hi @RyanClementsHax, first of all thank you for your great work! I have to ask if we can expect a solution to this problem here? If not, we will have to consider discontinuing the use of your near-perfect library. We would really appreciate a positive response :)

Hi @RyanClementsHax,
We've been encountering the same issue, not only with opacity but also with the theme function. To help illustrate the problem, I created this CodeSandbox. When we define a color with opacity at default extend theme (using hsla in my case), it correctly applies to the div, which does not happen in the tailwindcss-themer.
A similar issue occurs with the with-theme: theme("colors.destructive") function - it applies the definition from example_1 but fails to update when switching to defaultTheme.
If you have any insights on why this happens and whether the issue could be split into smaller tasks, I'd be happy to assist. Thanks for your awesome work!
Here's the sandbox for reference: https://codesandbox.io/p/github/Matozinho/tailwindcss-themer-example/main

So I finally got some time to look at this. I apologize for keeping you all waiting. Initially, this was supported by using tailwind's function syntax for creating color values, but since this ticket was opened, it is being removed in v4. I looked into trying to support this without using that syntax but it seems like tailwind has a hard requirement to want to configure its own alpha values.
I could attempt to contribute upstream for v4, but I won't be able to prioritize the time for that.
The closest I got to supporting it is found in this branch https://github.com/RyanClementsHax/tailwindcss-themer/tree/fix/alpha-values. I got the main case working, but the problem comes when tailwind wants to override that value in a class like bg-primary/75. Tailwind doesn't let us declare that there is a defined alpha value unless it tries to give us one.
I believe with v4 coming out, part of this may be moot or needs to be rethought anyway. I'll think about it a little more, but I will admit defeat in the meantime and formally request for help on this.


FFMpeg Extra Arguments Only Rotate Preview And Not Stream

Hi,
I noticed you added extra arguments for ffmpeg which is great because I was trying to figure out how to rotate a camera without physically rotating it. However, it appears to only rotate the preview image and not the actual stream. Is this intended behaviour or have I configured this incorrectly?
Screenshot attached.
Thanks

Turn off stream for camera and restart, it will rotate also stream if you are not using stream.
It has some disadvantages and advantages, depending on your environment nad HW. https://github.com/JurajNyiri/HomeAssistant-Tapo-Control#troubleshooting--faq


Recent removal of Play Services dependency from Firebase-auth

Welcome to FirebaseUI and thanks for submitting an issue!
Please take a look at open issues, as well as resolved issues, to see if your issue is either already being addressed, or has been solved by someone else.
If not, please feel free to fill in the following info so we can help faster!
Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a GitHub issue.
For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: Any but Fire tablet would be a good case in point
Android OS version: N/A
Google Play Services version: N/A
Firebase/Play Services SDK version: N/A
FirebaseUI version: 6.4.0

Step 3: Describe the problem:
The latest release of Firebase-auth has removed the dependency on Google Play Service and yet FirebaseUI-auth still has this dependency. Is there any prospect of this being removed in the near future.
Steps to reproduce:

Observed Results:

What happened?  This could be a description, logcat output, etc.

Expected Results:

What did you expect to happen?

Relevant Code:
// TODO(you): code here to reproduce the problem

@RPCarter53 please don't file empty issues, there's no way I can know what you're asking.
If you're wondering if we can remove the Play services dependency from FirebaseUI: maybe in the long term, but not immediately.  The reason is that Google Sign In still requires Play services and untangling that dependency is going to be a pretty big project!
But I do hope we can get there eventually.

Thanks for the prompt response. The issue we have with Play Services is always the Amazon Fire tablets which is my some way our biggest market place. I was rather hopeful when I saw that the dependency had been removed from the underlying Firebase library. Is there any possibility of the dependency being linked just to Google Sign which I believe we could do without in the short term?

  Here's a quick glance at all the places the  play-services-auth library is used within our code:
https://github.com/firebase/FirebaseUI-Android/search?q=com.google.android.gms.auth
We need to isolate that usage to just a few files and then guard the usage of those files on some runtime checks for certain classes, otherwise we risk crashing a lot.

Thanks Sam for the update. I think for now I will go with the underlying firebase-auth and just link with facebook login. I will keep an eye on firebaseUI and pick up when this dependency is gone.


fix: update docker.elastic.co/elasticsearch/elasticsearch docker tag to v7.17.25

This PR contains the following updates:

Package
Update
Change

docker.elastic.co/elasticsearch/elasticsearch (source)
patch
7.17.14 -> 7.17.25

Release Notes

elastic/elasticsearch (docker.elastic.co/elasticsearch/elasticsearch)
v7.17.25: Elasticsearch 7.17.25
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.25.html
v7.17.24: Elasticsearch 7.17.24
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.24.html
v7.17.23: Elasticsearch 7.17.23
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.23.html
v7.17.22: Elasticsearch 7.17.22
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.22.html
v7.17.21: Elasticsearch 7.17.21
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.21.html
v7.17.20: Elasticsearch 7.17.20
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.20.html
v7.17.19: Elasticsearch 7.17.19
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.19.html
v7.17.18: Elasticsearch 7.17.18
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.18.html
v7.17.17: Elasticsearch 7.17.17
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.17.html
v7.17.16: Elasticsearch 7.17.16
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.16.html
v7.17.15: Elasticsearch 7.17.15
Compare Source
Downloads: https://elastic.co/downloads/elasticsearch
Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.17/release-notes-7.17.15.html

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (7.17.25). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


fix: problem on node target engine introduced on last version

Fix a problem on node target engine introduced on last version

:tada: This PR is included in version 1.1.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add functionality to calculate power of a number

Name
Abhishek
Describe your Issue
Add new functionality to calculate the power of a number.

I can take this up

@bhattabhi013 @Susmita-Dey please review PR #52

hi @ays14,
Thanks for joining the community. I'm just a fellow contributor to this project.
As I can see you've already created a PR, so you can take up this issue, however, next time please get the issue assigned to yourself before working on it.

hi @ays14, Thanks for joining the community. I'm just a fellow contributor to this project. As I can see you've already created a PR, so you can take up this issue, however, next time please get the issue assigned to yourself before working on it.

Hi @bhattabhi013 , as you mentioned, I thought you're a maintainer and hence went ahead with the PR. Will take care next time onwards. Thanks.


Accessor where field name ends with a number doesn't get mutated value

Laravel Version: 6.3.0
PHP Version: 7.2.0
Database Driver & Version:

Description:
When the model has the field which ends with the number ($user->photo_1 as an example), then getPhoto1Attribute($value) doesn't mutate the value and returns $user->photo_1 as it is.
Steps To Reproduce:
Create a model with fields like 'something_number' ('something_1').
Then try to test getSomething1Attribute()
    public function getSomething1Attribute($value)
    {
        return 'test';
    }

When you get your model from eloquent you will see the field is not mutated.

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to report back and I'll gladly help you out and re-open this issue.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


Config menu in Anki 2.1?

It doesn't appear that the configuration options are available for the IR3 add-on in Anki 2.1 (beta 36).
The 'Read' menu doesn't show any choice to set config options, nor to modify keyboard shortcuts.
The Tools > Add-ons menu has a button for 'Config options' but when selecting 'Incremental Reading' in the list and then pressing the 'Config options' button, Anki returns the error 'Add-on has no configuration.' (This happens for all add-ons though, so might not be specific to accessing IR's options through the Add-ons menu.)
What am I missing here?

Many add-ons don't have a config file and haven't registered a config dialogue with Anki, so that doesn't necessarily indicate a general problem with your installation. However, if the "Read" menu is missing "Options...", that's not something I can explain. What happens when you press Alt+1?


Error installing the Helm Charts for Armo collector pods | Readiness probe failed: HTTP probe failed with statuscode: 503

Installed the Helm chart to explore the image scanning in my K8s environment. However, as part of installation there seems to be some failure with "helm upgrade --install armo" command as seen below
Release "armo" does not exist. Installing it now.
Error: failed post-install: timed out waiting for the condition

Another attempt as part of helm upgrade command gives
Error: UPGRADE FAILED: post-upgrade hooks failed: timed out waiting for the condition
A quick status check on armo pods
kubectl -n armo-system get pods
NAME                                         READY   STATUS      RESTARTS   AGE
armo-collector-5bf574d698-mgddf              0/1     Completed   1          10m
armo-collector-6b448cccdc-gwcdv              0/1     Running     3          17m
armo-notification-service-845466b457-27zcb   1/1     Running     0          10m
armo-vuln-scan-6c5b9955bb-jl9wt              1/1     Running     0          10m
armo-web-socket-66f7dc766d-5nc7q             0/1     Running     0          10m
armo-web-socket-fbf96f9f5-q6t8b              0/1     Running     0          17m

kubectl -n armo-system get pods
NAME                                         READY   STATUS             RESTARTS   AGE
armo-collector-5bf574d698-mgddf              0/1     CrashLoopBackOff   3          20m
armo-collector-6b448cccdc-gwcdv              0/1     CrashLoopBackOff   4          27m
armo-notification-service-845466b457-27zcb   1/1     Running            0          20m
armo-vuln-scan-6c5b9955bb-jl9wt              1/1     Running            0          20m
armo-web-socket-66f7dc766d-5nc7q             0/1     Running            0          20m
armo-web-socket-fbf96f9f5-q6t8b              0/1     Running            0          27m

kubectl describe pod for the armo-collector indicates
Events:
  Type     Reason     Age                  From               Message
  ----     ------     ----                 ----               -------
  Normal   Scheduled  30m                  default-scheduler  Successfully assigned armo-system/armo-collector-6b448cccdc-gwcdv
  Normal   Pulling    30m                  kubelet            Pulling image "quay.io/armosec/k8s-ca-dashboard-aggregator-ubi:185"
  Normal   Pulled     29m                  kubelet            Successfully pulled image "quay.io/armosec/k8s-ca-dashboard-aggregator-ubi:185" in 34.639658932s
  Normal   Created    29m                  kubelet            Created container armo-collector
  Normal   Started    29m                  kubelet            Started container armo-collector
  Warning  Unhealthy  28s (x298 over 29m)  kubelet            **Readiness probe failed: HTTP probe failed with statuscode: 503**

The readiness probe is failing (as seen above) for the armo-collector pods. Any inputs / suggestions on what seems to be the issue and resolution will be appreciated?

Thank you @sprathod369 . I did not see this issue in the past. Is it possible your network is slow?
@Bezbran What are your thoughts regrading handling this is the future?

Hi @sprathod369, as @dwertent mentioned, this isn't a common failure we saw so far.
One thing you can check is the connection to *.armo.cloud servers is opened in your Firewall.
In addition, if you can attach the pod logs it will be very helpful. Just hide any sensitive data before attaching it here.

Here are the pod logs for the collector @dwertent
# kubectl -n armo-system logs armo-collector-5bf574d698-t4sxl
-include-namespaces=armo-system not exists.
Image version: 185
E0301 06:30:12.916725       8 podwatch.go:536] Found nil element in pdm
E0301 06:30:12.970719       8 podwatch.go:536] Found nil element in pdm
E0301 06:30:24.046450       8 websocket.go:72] x509: certificate signed by unknown authority
E0301 06:31:07.744920       8 websocket.go:72] x509: certificate signed by unknown authority
E0301 06:31:31.954473       8 podwatch.go:536] Found nil element in pdm
E0301 06:31:31.954582       8 podwatch.go:536] Found nil element in pdm
E0301 06:31:59.441696       8 websocket.go:72] x509: certificate signed by unknown authority
E0301 06:33:54.182791       8 websocket.go:72] x509: certificate signed by unknown authority
E0301 06:35:00.899340       8 websocket.go:72] x509: certificate signed by unknown authority
E0301 06:35:00.899394       8 websocket.go:76] cant connect to websocket after 5 tries
E0301 06:35:00.899401       8 websocket.go:91] cant connect to websocket after 5 tries
E0301 06:35:00.899408       8 main.go:64] cant connect to websocket after 5 tries

@sprathod369, thanks for the logs.
Is there any chance you have some MIDM whose change our original certificate to one of your own?
Here are the details of our certificate:

@Bezbran - Not sure if I understand the above point. I have successfully performed the cluster scan (using all 4 frameworks) from this same kubernetes environment & I can see the results published on the UI at https://portal.armo.cloud/configuration-scanning/ So is there still a possibility that the connection to *.armo.cloud servers are blocked in the Firewall while attempting the image scanning (beta) feature? I'll check with my networking team but if you have any specific pointers / suggestions please let me know. Thanks!!

Hi @sprathod369, can we set a call to check the situation?
Here is my email<EMAIL_ADDRESS>feel free to contact me.
Regarding the results on the UI, can you see the results of the daily scanning as well, or just your first scan?

I see no comment regarding this issue anymore, assuming it was solved.

@Bezbran - Sorry for the tardy reply, was OOO for the past few days.
Regarding the results on the UI for cluster configuration scanning, I can see the historical results of the past few runs that I had triggered
The latest update on Image scanning is that I can see the collector pod running now (no longer see "CrashLoopBackOff" status) - network team helped me address this .... maybe a certificate issue or some blocking firewall issue on my side.
Here's the latest snapshot status of the armo pods
# kubectl -n armo-system get pods
NAME                                         READY   STATUS      RESTARTS   AGE
armo-collector-5bf574d698-t4sxl              1/1     Running     1790       8d
armo-notification-service-845466b457-27zcb   1/1     Running     0          9d
armo-scan-scheduler-27443520-cd7rp           0/1     Completed   0          2d12h
armo-scan-scheduler-27444960-vczl9           0/1     Completed   0          36h
armo-scan-scheduler-27446400-z29qd           0/1     Completed   0          12h
armo-vuln-scan-6c5b9955bb-jl9wt              1/1     Running     0          9d
armo-web-socket-66f7dc766d-pn54b             0/1     Running     0          8d
armo-web-socket-fbf96f9f5-q6t8b              0/1     Running     0          9d

Pod logs for the collector
# kubectl -n armo-system logs armo-collector-5bf574d698-t4sxl
-include-namespaces=armo-system not exists.
Image version: 185
E0309 11:53:09.652733       8 podwatch.go:536] Found nil element in pdm
E0309 11:53:09.653745       8 podwatch.go:536] Found nil element in pdm
E0309 11:53:23.656736       8 podwatch.go:536] Found nil element in pdm
E0309 11:53:23.656761       8 podwatch.go:536] Found nil element in pdm
E0309 12:29:21.084744       8 secretwatch.go:98] Got unexpected secret from chan: &{ <nil>}
Log level: 2, set -stderrthreshold=INFO for detailed logs/k8s-ca-dashboard-aggregator -include-namespaces=armo-system 2>&1  exited with code 4
Image version: 185
E0309 12:29:33.653732      25 podwatch.go:536] Found nil element in pdm
E0309 12:29:33.654300      25 podwatch.go:536] Found nil element in pdm
E0309 12:29:38.657979      25 podwatch.go:536] Found nil element in pdm
E0309 12:29:38.658002      25 podwatch.go:536] Found nil element in pdm

However, I do not see any results on the for Image scanning at https://portal.armo.cloud/image-scanning.
What seems to be missing - anything potentially to do with the web-socket pod ?
Happy to talk to you if required. Thanks for the support!!

Both web-socket and vuln-scan are required for image scanning.
Both are required egress access.
We are working on some list of servers should be opened in FW in order to make it works.
Send me a mail if you want us to take a closer look.
<EMAIL_ADDRESS>


Toast deprecation error on iOS 12.1.2

Expected behaviour
Toast should display
Actual behaviour
Toast does not display, error message:
Error: The old format of this exec call has been removed (deprecated since 2.1). Change to: cordova.exec(null, null, 'Service', 'action', [ arg1, arg2 ]); iOSExec — cordova.js:906 execProxy — cordova.js:1048 showWithOptions — Toast.js:49 show — Toast.js:53 gpsTooltip — core.js:1340 onclick — index.html:642 gpsTooltip — core.js:1331
I'm seeing this behaviour on

[x] iOS device
[x] iOS sim
[ ] Android device
[ ] Android sim

Hardware models
iPhone 6 Plus, simulated iPhone 8
OS versions
iOS 12.1.2
I'm not a dummy, so I've checked these

[ ] It happens on a fresh Cordova CLI project as well.
[x] I'm waiting for deviceready to fire.
[x] My JavaScript has no errors (window.onerror catches nothing).

So how can we reproduce this?
I am using as such:
window.plugins.toast.show('Location services unavailable', 'short', 'center', '', '');
That snippet works on Android, just not on iOS.

Pretty sure you're missing the callback. That's whats causing the error.


Regular users (username only) are not able to access the portals when the email login is enabled

Description:
When the email login is enabled and the following configuration is used, both regular users as well as the email users should be able to access the web portals.
[tenant_mgt]
enable_email_domain = true
disable_email_domain_validation = true

This functionality is not working as expected currently in APIM 4.0.0. However, the same functionality works fine in IS 5.11.0
Steps to reproduce:

Enable email login for the users by following the below.

https://apim.docs.wso2.com/en/latest/install-and-setup/setup/security/logins-and-passwords/maintaining-logins-and-passwords/#maintaining-logins-and-passwords

Try to login to a web portal as a regular user and the below error message will be displayed at the login page.

Invalid username! Username should be an E-mail address
Affected Product Version:
API Manager 4.0.0

Optional Fields
Related Issues:
https://github.com/wso2/product-is/issues/10760

any idea how it will work in the current release of wso2is 5.11.0?


component button missing on website (PACMAN-410)

The Component Manager version

ESP-IDF Version

python Version

Operating System

Browser (for https://components.espressif.com  Issues)
chrome
Description
Broken link to https://components.espressif.com/component/espressif/button
error:
FAILED TO FETCH COMPONENT DETAILS
To Reproduce
-

Expected behaviour

Additional info
No response
I have checked existing issues and online Documentation

[X] I confirm I have checked existing issues and online Documentation.

Few seconds after posting it appeared, but before i tried to refresh few times with no result.


Metronic Admin Theme

Is there any way I can adapt this package to work on my Metronic Admin Theme, it would help me so much.

Sorry first time I listen to Metronic admin theme... It seems a lot of work but looking at source code you can see how Laravel Blade templates and layouts are used and you can adapt tto any template

Metronic theme makes me dizzy, although we've a problem acacha.
illuminate/support suggests installing jeremeamia/superclosure (Required to be able to serialize closures (~2.2).)
illuminate/support suggests installing paragonie/random_compat (Provides a compatible interface like PHP7's random_bytes() in PHP 5 projects (~1.1).)
illuminate/support suggests installing symfony/polyfill-php56 (Required to use the hash_equals function on PHP 5.5 (~1.0).)
illuminate/support suggests installing symfony/var-dumper (Improves the dd function (2.8.*|3.0.*).)
illuminate/filesystem suggests installing league/flysystem (Required to use the Flysystem local and FTP drivers (~1.0).)
illuminate/filesystem suggests installing league/flysystem-aws-s3-v3 (Required to use the Flysystem S3 driver (~1.0).)
illuminate/filesystem suggests installing league/flysystem-rackspace (Required to use the Flysystem Rackspace driver (~1.0).)

I close this issue because sorry but no support for metronic admin theme.


Add touch input methods to ManualInputManager

Split from #2925

[ ] Depends on #3266

Adds touch IInput methods in ManualInputManager for testing purposes:

Method
Summary

MoveTouchTo(pos, source)
Enqueues TouchPositionInput with provided touch source and provided position.

ActivateTouch(source)
Enqueues TouchButtonInput with provided touch source and set to pressed / down.

ActivateTouchAt(drawable, source)
Enqueues TouchPositionInput with position set to the center of provided drawable followed by a TouchButtonInput with provided touch source and set to pressed / down.

ActivateTouchAt(pos, source)
Enqueues TouchPositionInput with provided position followed by a TouchButtonInput with provided touch source and set to pressed / down.

DeactivateTouch(source)
Enqueues TouchButtonInput with provided touch source set to released / up.

Closing as packed up to one PR due to no possible way to test this at its separation: #3269


Image is not displaying in WidgetKit

WebImage(url: URL(string: item.imageUrl))                             .resizable()                             .scaledToFit()                             .cornerRadius(5)                             .frame(width: 80, height: 57)
This is my code to download the image. I've put this code in ForEach.
Simply I run my app and added the Widget. Other data are displaying but images are not appearing. After refreshing the Widget, some of the images appearing and after refreshing again, images get appear and disappear.
Using the latest version via CocoaPods.
Any help?

Found solution.
But this issue can not been solved by using iOS 13 deployment target. Which means, I have to remove iOS 13 support and only support iOS 14+ using the new APIs. This is limited by Apple:
https://stackoverflow.com/questions/63274763/is-there-a-way-to-conditionally-use-stateobject-while-targeting-ios-13
I'll release 2.0.0 and bump iOS 14+ soon.


[SE-0311] Update the table of contents

Alternatively:
https://github.blog/changelog/2021-04-13-table-of-contents-support-in-markdown-files/

Thanks, it's nice to have the ToC in the source so let's get that in even with the GH feature present. Thanks!


Start testing the interpreter

We should start testing the interpreter.
The tests should be written in a way that is possible to run in the bootstrapped compiler as well.
My current thinking is that we should have everything needed to evaluate output of a program in the same source as the program. The test runner should parse the comments in the file for: exit code, stderr and stdout outputs.
In the interpreter, we can probably use golden-tests or something similar.
In the bootstrapped compiler we can keep using the same library, or port it to Fir.

I've started adding some tests in https://github.com/fir-lang/fir/tree/main/tests, using golden-tests.
We should add more.

Two things that I think we should add to golden-tests:

We should make it possible to somehow process stdout/stderr before comparing, e.g. maybe with a line of shell.
This allows checking stderr in case of type error without comparing the stack trace printed by the error.
Of course type errors shouldn't be panics, but (1) preprocessing the program output before comparing would still be useful filter out unimportant details (2) panicking may be acceptable in the interpreter, which we want to keep simple.

We should make it possible to run a program multiple times, passing different inputs. For example:
fn main(input: Str) = ...

# input:
# ...
# expected stdout:
# ...

# input:
# ...
# expected stdout:
# ...

Here the program should be run twice, with the given inputs and expected outputs.
This is useful in cases like:

We can test example programs this way. Example programs will have a main and print stuff.
We could add a test mode and another entry for testing, but the program would have to be refactored to return values and we would have to check those values, instead of checking stdout.

When testing crashes/errors, we can't use a test function with assertions as the program will crash after an error.
Also, we don't have error/exception handling right now and it may take some time before we have the features and standard library functions in place.

It would also be helpful if we could run the tests without spawning a Fir process for each test, to be able to generate coverage.

We have a lot of tests in the tests dir now. We also test examples.


feat: Add tooltip for branch button in the Legend panel

This PR implements the tooltip feature for the branch button in the Legend panel, as discussed in issue #80

@NirmalScaria ,
Please, let me know if there is any issue with my PR.


Monitoring

using DiagnosticSource and Butterfly.OpenTracing Implementing distributed tracking

Coverage decreased (-2.1%) to 79.078% when pulling 9c7d562975c87684bf95a0fd2d3741d034f2961d on geffzhang:Monitoring into 6ff8c1cac4d80ed8d297f4c318cdfa4bfe4eed81 on TomPallister:develop.

@geffzhang thank you for this PR. I will check the code ASAP! I think we need to add some unit tests for this code as the coverage has decreased 2.1%! :o

@geffzhang looks good, I have left a few comments.
I still need to get this running with Butterfly to understand properly!
It might be worth writing some acceptance tests with a fake butterfly service so that if anyone refactors the code we will pick up an errors.

@geffzhang one more thing....Ocelot already has a diagnostic listener configured in OcelotMiddlewareExtensions line 320. How will the OcelotTracingDiagnosticListener effect this? Will asp.net just use both now! We might have to look into this.

Yes, my code hasn't been written yet, and the unit test hasn't been written yet.

@geffzhang OK cool, sorry I did not understand before!!!
I'm excited to see this PR when it is finished!! Thank you so much for your contributions! :D

@TomPallister Thank you for reviewing the code, I refactored it, merging and deleting the related code, and the rest of the work is writing some acceptance tests with a fake butterfly service

@TomPallister Please review my newest code

@geffzhang thank you! I will start review tomorrow!


Enable RegexMatchSpan with concatenates words by sep="(separator)" option

Description of the problems or issues
Is your pull request related to a problem? Please describe.
A clear and concise description of what the problem is.
A sentence "123 456 789" is parsed and gets three words "123", "456", and "789".
I'd like to match a number like
RegexMatchSpan(rgx=r"\d{9}", sep=" ")
but sep=" " has no effect
Does your pull request fix any issue.
Fix #270
Description of the proposed changes
Enable RegexMatchSpan with sep="(separator)" option.
It concatenates mention spans to one word and does RgexMatch without consideration of the separator.
Test plan
Add Test Code to 'fonduer/tests/candidates/test_matchers.py'.
A sentence "This is apple" is parsed and gets 2 2-grams "This is" and "is apple".
We can get "is apple" with following rgx and sep="(space)" option:
RegexMatchSpan(rgx=r"isapple", sep=" ")
Checklist

[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have updated the CHANGELOG.rst accordingly.

Some codes may be updated while creating #492. I'm now re-checking.

Codecov Report

Merging #492 into master will decrease coverage by 0.04%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master     #492      +/-   ##
==========================================
- Coverage   85.85%   85.81%   -0.05%
==========================================
  Files          88       88
  Lines        4568     4568
  Branches      851      853       +2
==========================================
- Hits         3922     3920       -2
- Misses        464      465       +1
- Partials      182      183       +1

Flag
Coverage Δ

#unittests
85.81% <71.42%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

...fonduer/candidates/models/implicit_span_mention.py
81.96% <66.66%> (ø)

src/fonduer/candidates/models/span_mention.py
82.24% <66.66%> (ø)

src/fonduer/candidates/matchers.py
97.31% <100.00%> (ø)

src/fonduer/parser/visual_linker.py
83.25% <0.00%> (-0.94%)
:arrow_down:

Something failure in installation of ubuntu. There would be nothing more I can.

Thanks for making this clear!


Move to home bug

Addresses #50

This is just a quick fix for preventing unwanted behavior of the real robot at startup. The real fix will come in a later pull request.


Inconsistent nested field logging with the human formatter

When logging an object with the human formatter, I see that the output is different in nested object keys and values, vs. root ones:

keys are quoted in nested objects, and not quoted in the root
string are not quoted if they're values of root fields
empty arrays are not logged if they're values of root fields

import log from 'bristol';
log.addTarget('console').withFormatter('human');

const object = {
  rootString: 'key name not quoted, string should be quoted',
  nested: {
    string: 'string is quoted, and so is the key',
    array: [],  // empty array correctly logged
  },
  array: [],  // nothing logged
};

log.info('Unquoted strings', object);

Output:
[2019-05-26 00:30:47] INFO: Inconsistent output (file:///.../bristol.mjs:13)
	rootString: key name not quoted, string should be quoted
	nested:
	    {
	        "string": "string is quoted, and so is the key",
	        "array": []
	    }
	array:

In the meantime you could try a different formatter such as palin @dandv

#plug 😉