0.14.2 overhaul

This pull request modifies the cidata and nginxns primitives to use the cloudcix.rcc 0.14.2 API.

Note: dhcpns is still missing. I'm holding off on that until I'm done with the various helper functions  in utils.py.

Closing this one because I split it up into separate pull requests instead of rebasing:
https://github.com/CloudCIX/primitives/pull/38
https://github.com/CloudCIX/primitives/pull/39
https://github.com/CloudCIX/primitives/pull/40


NSDate types need to be updated in ios.d.ts

From @speigg on January 25, 2016 20:1
Due to auto-marshalling, many of the api's defined in ios.d.ts are now incorrectly declared as returning NSDate objects when they should instead be declared to return Date objects. I'm not sure if this is also needed the other way around (presumably apis that are declared as accepting NSDate objects should be changed to declare that they accept Date objects?).
Copied from original issue: NativeScript/NativeScript#1436

The ios.d.ts file you refer to was generated a long time ago by an external tool and has now fallen somewhat out of sync with the runtime.
The NativeScript Objective-C metadata generator can generate typings that describe native APIs and it handles NSDate correctly. You can see how to generate typings here.


Update app-approval.md

There should be information where you can edit the list of application approvers' email addresses.
Summarize the change in the pull request title
Describe your change, specifically why you think it's needed.
Fixes #Issue_Number (if necessary)

@PanuSaukko : Thanks for your contribution! The author, @aczechowski, has been notified to review your proposed change.

Thanks @PanuSaukko for the contribution. I'm checking with engineering about one detail, and then will have some feedback.

@aczechowski  Currently we have one issue with editing emails addresses in Subscription dialog UI. When admin makes changes in Subscriptions UI where multiple email addresses are specified, the email to approve/deny request will always contain corpnet links despite CMG is configured. The workaround is to create a new deployment with multiple email addresses. Issue doesn't repro if single email address is specified and related only to CMG scenario. This bug will be fixed in 1902. Can we add this to the doc?


BREAKING swap order of parametrization of GammaLikelihood

To simplify dispatch on the type of invlink - which is what we'll normally want to dispatch on - whether \alpha is a Float32 or Float64 should generally be irrelevant.

@devmotion @st-- what's the status of this PR?


[Bug] Candlestick width not increasing on zoom with time xAxis and 'empty' filterMode

Version
5.5.1
Link to Minimal Reproduction
https://codepen.io/xpatt/pen/RwzZwYR
Steps to Reproduce
set xAxis type to 'time'
set dataZoom filterMode to 'empty'
use series type 'candlestick'
Current Behavior
Candlesticks do not widen on zoom:

Expected Behavior
Candlesticks should widen on zoom:

Environment
- OS: Windows 11
- Browser:Chrome & Mozilla
- Framework: JS

Any additional comments?
No response

Is there a reason to try filterMode:'empty' or it's just an experiment ? Obviously chart does not look good.
Also note that default data order is OCLH, not OHLC.  And the colors might need adjustment:
const upColor = '#26a69a';
const downColor = '#ef5350'; //red
itemStyle= {
  color: upColor, // bull
  borderColor: upColor, // bull
  color0: downColor, // bear
  borderColor0: downColor // bear
}

Alright, just changed it to OCLH. Yes, I want to keep the filterMode: empty because it is important for my use case that when zooming in horizontally the vertical height stays the same.

important for my use case that when zooming in horizontally the vertical height stays the same.

Another way of doing this is yAxis.max - Demo
📌 please close issue if problem solved.

Yes, but I also have a vertical zoom slider.

You can insert a hook function into the dataZoom event, where you can query the current data of the chart.
// Listening to the zoom event and dynamically increasing the candlestick width
myChart.on('dataZoom', function(params) {
  const startValue = myChart.getModel().option.dataZoom[0].startValue;
  const endValue = myChart.getModel().option.dataZoom[0].endValue;
  // Find indexes from dateData by startValue and endValue
  const startIndex = dateData.findIndex(item => new Date(item) >= new Date(startValue));
  const endIndex = dateData.findIndex(item => new Date(item) >= new Date(endValue));

  // Calculate a visible item count by indexes
  const visibleDataCount = (endIndex !== -1 && startIndex !== -1) ? endIndex - startIndex : 0;

  // Dynamically calculate new width, use grid-width, gap, candlestick visible count

  // Get the chart container width
  const chartContainerWidth = dom.clientWidth;
  // Retrieve grid configuration
  const gridConfig = myChart.getOption().grid[0] || {};
  const gridLeftPercent = parseFloat(gridConfig.left) || 0;
  const gridRightPercent = parseFloat(gridConfig.right) || 0;
  // Convert percentages to pixels
  const gridLeftPx = (gridLeftPercent / 100) * chartContainerWidth;
  const gridRightPx = (gridRightPercent / 100) * chartContainerWidth;
  // Calculate the width of the plotting area
  const gridWidth = chartContainerWidth - gridLeftPx - gridRightPx;

  // Gap between data
  const gap = chartContainerWidth / visibleDataCount / 3

  const MAXIMUM_BAR_WIDTH = 50
  // (GRID_WIDTH - SUM_GAP_WIDTHS) / SUM_BAR_WIDTHS
  const currentBarWidth = (gridWidth - (visibleDataCount - 1) * gap) / visibleDataCount
  // Maximum or Current width will min
  const newBarWidth = Math.min(MAXIMUM_BAR_WIDTH, currentBarWidth)

  // Update the candlestick width
  myChart.setOption({
    series: [{
      type: 'candlestick',
      barWidth: newBarWidth // New candlestick width
    }]
  });
});

Source: ECharts: Candlestick width not increasing on zoom with time xAxis and 'empty' filterMode - StackOverflow answer


DefaultShardManagerBuilder, wrong counting

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[ ] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report

Notification for building shards (DefaultShardManagerBuilder) displays wrong count (starting at 0 for the first shard, should start at 1)
http://prntscr.com/px01ki
Expected Behavior

Should count shards like 1/n, 2/n ...  n/n instead of 0/n ... n-1/n
Code Example or Reproduction Steps

Exception or Error

It counts as ID/Count, and IDs start at zero

That seems quite confusing to me to use a construct more likely describing an x of y ratio for that purpose.

JDA starts at 0 because Discord requires starting at 0; it's not a construct designed for JDA.

Shards are 0-indexed meaning they start at 0, while the total shard count is the size of shards.
Like jagrosh mentioned is this something from Discord and not JDA.


python-tools was unable to find your machine's python executable.

Suddenly the python in atom does not work. The error is:
Please try set the path in package settings and then restart atom.
Error: spawn python ENOENT
I didn't change the path in the environment settings. The path in system variables:

Thanks

Got the same error.
Was running before, no change to the path was made.

My solution: https://github.com/MichaelAquilina/python-tools/issues/38#issuecomment-216195217

Fixed in python-tools 0.6.8 thanks to @go2sh for the fix


External Account Connection

If I have a service like Intercom or PagerDuty which have a "corporate service accounts" that have multiple users in the account and I want the "corporate service account" to interact with Slack how do I associate a corporate service account to a completed OAuth flow?
I understand how to initiate the OAuth flow with Bolt and how it saves the data in the Installation, Bot and State tables - what I seem to be missing is how to make the association with the account that initiated the OAuth flow.
I looked in the BoltRequest and the BoltResponse but there is no way to pass in the data of the "corporate account ID".
Effectively, I would like to be able to associate the corporate account row in my database to an installation ID but I cannot see where in Bolt I would get that information to be able to make that association.
I am more than happy to provide more information or submit code examples of what I am looking to achieve.
Thanks!

How about doing as below?
As I answer to your question before, you can use callback functions to do extra when an installation is completed. The success function associates the installation with the corresponding row in your existing database tables.
BoltRequest represents an incoming request (and its payload) from the Slack API servers. So, the object does not contain any data that is specific to your backends. Instead, you can attach any data into BoltContext object. The bind_corporate_account method is a simple global middleware example. You can do the similar in your apps.
def success(args: SuccessArgs) -> BoltResponse:
    # associate the installation with the corresponding corp account:
    account_service.associate_slack_account(args.installation)
    return args.default.success(args)

def failure(args: FailureArgs) -> BoltResponse:
    return args.default.failure(args)

app = App(
    oauth_settings=OAuthSettings(
        callback_options=CallbackOptions(success=success, failure=failure),
    ),
)

@app.use  # or @app.middleware
def bind_corporate_account(next, context: BoltContext):
    account = account_service.find(context.enterprise_id, context.team_id, context.user_id)
    context["corporate_account_id"] = account.get("corporate_account_id")
    next()

@app.command("/my-corp-account")
def run_ops(ack, context):
    corp_id = context.get("corporate_account_id")
    if corp_id:
        ack(f"Here is your corp account id: {corp_id}")
    else:
        ack("Your Slack account is not associated with your corp account yet. Visit: https://www.example.com")

Thank you for your detailed response. I am understanding the part on how to pull out the data using the context. However the difficulty still remains in the account_service.associate_slack_account(args.installation) portion.
At that stage of the code, there is no available information to indicate the current corporate account that needs to be associated.
I have been pulling the requests apart in Intercom and it appears that they transmit the information in the state - as an example state 102579:wbqeymqr:3824235 - There are two random numbers on either side of the wbqeymqr which is my Intercom instance.
I understand that Bolt is crafting UUID4 for the state generation and validation. I am wondering if it would be a good place to either add to the end of the UUID in a parseable format or just allow the developer to override the state generator during the OAuth startup so that it would potentially be available at the time of state consume.
Below is an entirely hypothetical example - notice the inclusion of the new property state:
def success(args: SuccessArgs) -> BoltResponse:
    # associate the installation with the corresponding corp account:
    account_service.associate_slack_account(args.installation, args.state)
    return args.default.success(args)

This would allow for the developer to do whatever they wanted with the state object as it was originally generated by allowing the passing in of a crafted state in the oauth_flow.handle_installation method.
Perhaps passing it back from https://github.com/slackapi/bolt-python/blob/main/slack_bolt/oauth/oauth_flow.py#L214 and then injecting into https://github.com/slackapi/bolt-python/blob/main/slack_bolt/oauth/oauth_flow.py#L269
That said, I am totally open to any other means of deriving the "corporate account id" during the success method.
Thanks again!
(FYI I will be in the webmeeting on Nov 11)

You can implement your own OAuthStateStore and it should work. With this, your success callback handler can access the value in BoltRequest's data by args.request.query.get("state") (by the way, strictly speaking, my previous comment was incorrect. in this case, the request is an end-user's browser request. So, you can set values in cookies or state param. Sorry about the misinformation).
Another option would be setting another cookie at the timing of /slack/install's redirection for associating other external service (Intercom here) specific data and accessing the data in success callback handler. Either is fine.

FYI I will be in the webmeeting on Nov 11

Thanks for joining the webinar! As I live in Asia, I won't be there live but hope you will enjoy the session.

Thanks for your contribution at https://github.com/slackapi/python-slack-sdk/pull/871 . If I understand the current state of the issue correctly, we do not have further action items here. Let me close this now but if you have anything to follow up, please feel free to write in here or create a new issue.


use new inventory topic

We now use the same inventory  topic for both delete and archive
messages. Since we are now pulling both messages off the same topic,
we added an additional check on archive messages to ensure we are
processing messages of the correct type before proceeding.

Codecov Report

Merging #92 into master will increase coverage by 19.55%.
The diff coverage is 89.23%.

@@             Coverage Diff             @@
##           master      #92       +/-   ##
===========================================
+ Coverage   50.14%   69.69%   +19.55%
===========================================
  Files          13       13
  Lines         357      396       +39
===========================================
+ Hits          179      276       +97
+ Misses        178      120       -58

Impacted Files
Coverage Δ

historical_system_profiles/deleter.py
0.00% <0.00%> (ø)

historical_system_profiles/listener_logging.py
0.00% <ø> (ø)

historical_system_profiles/archiver.py
65.90% <83.33%> (+65.90%)
:arrow_up:

historical_system_profiles/views/v1.py
98.63% <96.15%> (+31.96%)
:arrow_up:

historical_system_profiles/config.py
100.00% <100.00%> (+6.06%)
:arrow_up:

historical_system_profiles/db_interface.py
79.06% <100.00%> (+12.40%)
:arrow_up:

historical_system_profiles/listener_metrics.py
100.00% <100.00%> (+100.00%)
:arrow_up:

historical_system_profiles/models.py
98.03% <100.00%> (+27.66%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Column could identify the TextAlign and atomatically adjust Text to match it

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
It's common to put a Text inside a Column and then needed for the text to center aligned, but we don't want the rest of the Columns children to be centered as well, so the CrossAxisAlignment.start could be used. Setting the textAlign parameter of Text won't work, because the Text isn't taking the whole width of the Column, this can be hard to understand/debug for new devs specially.
Proposal
The solution is simple, we can just wrap the Text inside a Center, but I wonder if the Column widget could be a bit smarter and check if one of it's children is a Text and it has a textAlign, then the Column could make that Text take the whole width, so the property of the Text will work as expected.

Please do provide a code sample along with relevant images for better understanding. Thanks!

Hi @feinstein. Thanks for proposing this. Not too sure I understand your proposal.
Using TextAlign.center in Text widget in a Column seems to work as expected
Code Sample

import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(title: 'Bug', home: HomePage());
  }
}

class HomePage extends StatelessWidget {
  const HomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: const Padding(
          padding: EdgeInsets.only(bottom: 50),
          child: Column(
            children: [
              Text(
                "Hello World! Welcome to TutorialKart for this awesome Flutter Tutorial on Text widget.",
                textAlign: TextAlign.center,
              ),
            ],
          )),
    );
  }
}

Screenshot

Wrapping the Text with Center also seems to work fine. Can you please provide more information on the proposal?

Sorry, I forgot to say "when you set the column to CrossAxisAlignment.start.

Hi @feinstein. Thanks for your response. Adding crossAxisAlignment: CrossAxisAlignment.start to the code sample above still gives same behaviour as in the screenshot.

Use a small text. I will submit an example later today.

Try this (it's your code, but with a string smaller than the screen width):
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(title: 'Bug', home: HomePage());
  }
}

class HomePage extends StatelessWidget {
  const HomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: const Padding(
          padding: EdgeInsets.only(bottom: 50),
          child: Column(
            children: [
              Text(
                "Hello World!",
                textAlign: TextAlign.center,
              ),
            ],
          )),
    );
  }
}

Wrap your Text / Column inside SizedBox
Column(
  children: [
    SizedBox(
      width: double.infinity,
      child: Text(
        "Hello World!",
        textAlign: TextAlign.center,
      ),
    ),
  ],
)

Please read my original proposal. I am not reporting a bug, I know how to fix it, I say on the original post how to fix it. I am asking for a new feature where your fix, or my fix (using Center) isn't event necessary, as the Column will be smart enough to fix this by itself.

Sounds like a reasonable proposal.

In which cases the layout could be different from expected? I can only imagine the cases where the text isn't wide enough, and when we make it wider it works.
I thought about the argument that it might not be desirable for having a layout decision being made based on a parameter inside a widget, but I am not so sure if this is so negative, as this is a very common use case. But I would love to hear more of your thoughts about it.


instances of /etc/awx replaced with /etc/tower, resolves #46

Newer versions of tower use /etc/tower instead of /etc/awx. This is only relevant if --scope=global is set to begin with, and if that folder exists. It is a straightforward change. After this, the older version will no longer be supported.

Looks great, thanks @AlanCoding!


Handle coverity scan

Codecov Report

Merging #173 (da6f9f4) into main (9ed8fc0) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #173      +/-   ##
==========================================
+ Coverage   95.25%   95.27%   +0.01%
==========================================
  Files          43       43
  Lines        2762     2749      -13
==========================================
- Hits         2631     2619      -12
+ Misses        131      130       -1

Impacted Files
Coverage Δ

test/test.cpp
100.00% <ø> (ø)

math/wide_decimal/decwide_t.h
92.22% <100.00%> (+0.09%)
:arrow_up:

examples/example004_bessel_recur.cpp
98.30% <0.00%> (-0.11%)
:arrow_down:

examples/example002c_pi_quintic.cpp
100.00% <0.00%> (ø)

examples/example007_catalan_series.cpp
100.00% <0.00%> (ø)

examples/example008_bernoulli_tgamma.cpp
100.00% <0.00%> (ø)

examples/example010_hypergeometric_2f1.cpp
100.00% <0.00%> (ø)

examples/example010a_hypergeometric_1f1.cpp
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Tuneup migration guide

@radiantcapsule thanks for valuable feedback, it is appreciated. I created an issue https://github.com/paypal/Illuminator/issues/28 and will make sure to address all your ideas.


[vcpkg_fixup_pkgconfig] Large simplification, handle spaces in path.

Also add end-to-end test for zlib with spaces in path.

What does your PR fix?
Fixes #13103
Fixes #13166

This may also fix other issues as well due to significant logical simplification.
@Neumann-A for review to ensure this still covers all the important cases -- For example, this currently leaves off handling of -pthread and instead ignores unrecognized arguments

The path used in #13105 is the appveyor standard path of C:\tools\vcpkg (no spaces). On testing, this PR unfortunately does not fix #13105:
Starting package 1/1: zlib:x86-windows-port-debug
Building package zlib[core]:x86-windows-port-debug...
Could not locate cached archive: C:\Users\appveyor\AppData\Local\vcpkg/archives\3f\3f2d86020d02f900828dce6c44b7411765b79d0c.zip
-- Downloading http://www.zlib.net/zlib-1.2.11.tar.gz...
-- Extracting source C:/tools/vcpkg/downloads/zlib1211.tar.gz
-- Applying patch cmake_dont_build_more_than_needed.patch
-- Applying patch 0001-Prevent-invalid-inclusions-when-HAVE_-is-set-to-0.patch
-- Applying patch add_debug_postfix_on_mingw.patch
-- Using source at C:/tools/vcpkg/buildtrees/zlib/src/1.2.11-63309e48e5.clean
-- Configuring x86-windows-port-debug
-- Building x86-windows-port-debug-dbg
-- Building x86-windows-port-debug-rel
-- Downloading https://repo.msys2.org/mingw/i686/mingw-w64-i686-pkg-config-0.29.2-1-any.pkg.tar.xz...
-- Downloading https://repo.msys2.org/mingw/i686/mingw-w64-i686-libwinpthread-git-<IP_ADDRESS>06.c9a21571-1-any.pkg.tar.zst...
-- [DEBUG] Using pkg-config from: C:/tools/vcpkg/downloads/tools/pkg-config/0.29.2-1/mingw32/bin/pkg-config.exe
-- [DEBUG] Release Files: C:/tools/vcpkg/packages/zlib_x86-windows-port-debug/lib/pkgconfig/zlib.pc
-- Fixing pkgconfig file: C:/tools/vcpkg/packages/zlib_x86-windows-port-debug/lib/pkgconfig/zlib.pc
-- [DEBUG] Checking package (RELEASE): zlib
-- [DEBUG] pkg-config returned:0
-- [DEBUG] pkg-config output:
-- [DEBUG] pkg-config error output:
-- [DEBUG] pkg-config returned:0
-- [DEBUG] pkg-config output:-LC:/tools/vcpkg/packages/zlib_x86-windows-port-debug/lib -lzlib
-- [DEBUG] pkg-config error output:
-- [DEBUG] pkg-config processing '-LC:/tools/vcpkg/packages/zlib_x86-windows-port-debug/lib'
-- [DEBUG] pkg-config processing '-lzlib'
-- [DEBUG] STATUS;CHECK_LIB_zlib_RELEASE=C:/Tools/vcpkg/packages/zlib_x86-windows-port-debug/lib/zlib.lib
-- [DEBUG] Debug Files: C:/tools/vcpkg/packages/zlib_x86-windows-port-debug/debug/lib/pkgconfig/zlib.pc
-- Fixing pkgconfig file: C:/tools/vcpkg/packages/zlib_x86-windows-port-debug/debug/lib/pkgconfig/zlib.pc
-- [DEBUG] Checking package (DEBUG): zlib
-- [DEBUG] pkg-config returned:0
-- [DEBUG] pkg-config output:
-- [DEBUG] pkg-config error output:
-- [DEBUG] pkg-config returned:0
-- [DEBUG] pkg-config output:-LC:/Tools/vcpkg/packages/zlib_x86-windows-port-debug/lib -lzlibd
-- [DEBUG] pkg-config error output:
-- [DEBUG] pkg-config processing '-LC:/Tools/vcpkg/packages/zlib_x86-windows-port-debug/lib'
-- [DEBUG] pkg-config processing '-lzlibd'
CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:128 (message):
  CHECK_LIB_zlibd_DEBUG=CHECK_LIB_zlibd_DEBUG-NOTFOUND
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:224 (vcpkg_fixup_pkgconfig_check_files)
  ports/zlib/portfile.cmake:48 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:79 (include)
Error: Building package zlib:x86-windows-port-debug failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: zlib:x86-windows-port-debug
  Vcpkg version: 2020.06.15-nohash
Additionally, attach any relevant sections from the log files above.

Writing out package directory contents...

@Hoikas I've pushed an improvement for the error message as well as a sleep shim -- could you retry using
ce6564d?

Sadly the build continues to fail.

(Probably) Fixes #13105

nope since the error is not in the check but in the replacement before. Prefix was somehow not correctly replaced.

For example, this currently leaves off handling of -pthread and instead ignores unrecognized arguments

that is potentially dangerous since LFLAGS might contain gcc specific linker flags. (-Wl,something,someotherthing) It won't affect build with autotools but meson and cmake builds will most likely choke on it on Windows.

that is potentially dangerous since LFLAGS might contain gcc specific linker flags. (-Wl,something,someotherthing) It won't affect build with autotools but meson and cmake builds will most likely choke on it on Windows.

My current thinking is that this is just acceptable tradeoffs, since we can't really intend to keep up with every compiler's command line arguments and pkg-config allows users to reasonably include compiler-specific flags.

root@5178487134bb:/vcpkg# cat "/packaG es/zlib_x64-linux/lib/pkgconfig/zlib.pc"
prefix=${pcfiledir}/../..
exec_prefix=${prefix}
libdir=${prefix}/lib
sharedlibdir=${prefix}/lib
includedir=${prefix}/include

Name: zlib
Description: zlib compression library
Version: 1.2.11

Requires:
Libs: -L${libdir} -L${sharedlibdir} -lz
Cflags: -I${includedir}
root@5178487134bb:/vcpkg# PKG_CONFIG_PATH=/packaG\ es/zlib_x64-linux/lib/pkgconfig pkg-config --libs zlib
-L/packaG es/zlib_x64-linux/lib/pkgconfig/../../lib -lz

:( it looks like ${pcfiledir} here is sortof fundamentally broken for many libraries because it isn't escaped and they don't wrap their -L calls in quotes

@ras0219 Regex replace [[-I(${[^}]+})]] to  [[-I"\${\1}"]]and the same for the -L flags?

This fixes #13166.

Regex replace [[-I(${[^}]+})]] to [[-I"${\1}"]]and the same for the -L flags?

Yeah, this appears to work:
        string(REGEX REPLACE " -L(\\\${[^}]*}[^ \n\t]*)" " -L\"\\1\"" _contents "${_contents}")
        string(REGEX REPLACE " -I(\\\${[^}]*}[^ \n\t]*)" " -I\"\\1\"" _contents "${_contents}")
        string(REGEX REPLACE " -l(\\\${[^}]*}[^ \n\t]*)" " -l\"\\1\"" _contents "${_contents}")

Ok, so on Windows I believe there's a double-escaping bug that does the right thing if there aren't quotes, but if you actually quote things it causes pkg-config to spit out stuff like -LC:/vcpkg\\\ spaces/installed.
I can hack and hardcode things to unescape triple-slash correctly, but the real question is who's going to consume this? Meson? Autoconf in msys2? How do they handle this?

I ran a simple test with meson
project('simple', 'c')
zdep = dependency('zlib', version : '>=1.2.8')
exe = executable('zlibprog', 'prog.c', dependencies : zdep)

and it is able to successfully find zlib and pass it to the compiler with these changes:
C:\src\vcpkg space\foo>..\downloads\tools\python\python-3.8.3-x86\python.exe ..\downloads\tools\meson\meson-0.55.1\meson.py build
The Meson build system
Version: 0.55.1
Source dir: C:\src\vcpkg space\foo
Build dir: C:\src\vcpkg space\foo\build
Build type: native build
Using 'PKG_CONFIG_PATH' from environment with value: 'C:\\src\\vcpkg space\\installed\\x86-windows\\lib\\pkgconfig'
Using 'PKG_CONFIG_PATH' from environment with value: 'C:\\src\\vcpkg space\\installed\\x86-windows\\lib\\pkgconfig'
Project name: simple
Project version: undefined
C compiler for the host machine: cl (msvc 19.27.29111)
C linker for the host machine: link link 14.27.29111.0
Host machine cpu family: x86
Host machine cpu: x86
Found pkg-config: C:\src\vcpkg space\downloads\tools\pkg-config\0.29.1-3\mingw32\bin\pkg-config.EXE (0.29.1)
Using 'PKG_CONFIG_PATH' from environment with value: 'C:\\src\\vcpkg space\\installed\\x86-windows\\lib\\pkgconfig'
Run-time dependency zlib found: YES 1.2.11
Build targets in project: 1

Found ninja.EXE-1.8.2 at "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\Ninja\ninja.EXE"

C:\src\vcpkg space\foo>ninja -C build -v
ninja: Entering directory `build'
[1/2] "cl" "-Izlibprog.exe.p" "-I." "-I.." "-IC:/src/vcpkg space/installed/x86-windows/include" "/MDd" "/nologo" "/showIncludes" "/W2" "/ZI" "/Ob0" "/Od" "/RTC1" "/Fdzlibprog.exe.p\prog.c.pdb" /Fozlibprog.exe.p/prog.c.obj "/c" ../prog.c
[2/2] "link"  /MACHINE:x86 /OUT:zlibprog.exe zlibprog.exe.p/prog.c.obj "/nologo" "/release" "/nologo" "/DEBUG" "/PDB:zlibprog.pdb" "C:/src/vcpkg space/installed/x86-windows/lib/zlib.lib" "/SUBSYSTEM:CONSOLE" "kernel32.lib" "user32.lib" "gdi32.lib" "winspool.lib" "shell32.lib" "ole32.lib" "oleaut32.lib" "uuid.lib" "comdlg32.lib" "advapi32.lib"

# installed\x86-windows\lib\pkgconfig\zlib.pc
prefix=${pcfiledir}/../..
exec_prefix=${prefix}
libdir=${prefix}/lib
sharedlibdir=${prefix}/lib
includedir=${prefix}/include

Name: zlib
Description: zlib compression library
Version: 1.2.11

Requires:
Libs: -L"${libdir}" -L"${sharedlibdir}" -lzlib
Cflags: -I"${includedir}"

This also works in an ubuntu docker container:
root@121cce60c572:/vcpkg/foo# ln -s /vcpkg /vcpkg\ space
root@121cce60c572:/vcpkg/foo# PKG_CONFIG_PATH=/vcpkg\ space/installed/x86-windows/lib/pkgconfig ../downloads/tools/meson/meson-0.55.1/meson.py build --debug
The Meson build system
Version: 0.55.1
Source dir: /vcpkg/foo
Build dir: /vcpkg/foo/build
Build type: native build
Using 'PKG_CONFIG_PATH' from environment with value: '/vcpkg space/installed/x86-windows/lib/pkgconfig'
Using 'PKG_CONFIG_PATH' from environment with value: '/vcpkg space/installed/x86-windows/lib/pkgconfig'
Project name: simple
Project version: undefined
C compiler for the host machine: cc (gcc 9.3.0 "cc (Ubuntu 9.3.0-10ubuntu2) 9.3.0")
C linker for the host machine: cc ld.bfd 2.34
Host machine cpu family: x86_64
Host machine cpu: x86_64
Found pkg-config: /usr/bin/pkg-config (0.29.1)
Using 'PKG_CONFIG_PATH' from environment with value: '/vcpkg space/installed/x86-windows/lib/pkgconfig'
Run-time dependency zlib found: YES 1.2.11
Build targets in project: 1

Found ninja-1.10.0 at /usr/bin/ninja

I've looked at the meson code and am unable to determine where/how they handle this; I see https://github.com/mesonbuild/meson/blob/1c403e20e70ac523216a31f977901fb815166b7a/mesonbuild/dependencies/base.py#L699 but based on skimming that would seem to only replace one slash.
Regardless, the tl;dr is that at least meson handles it correctly; we can consider injecting a shim if needed for autoconf later.

@ras0219: Check the meson.log. It might have used cmake. Or add  method : 'pkg-config' to the dependency call and make sure it uses pkg-config.

I did check the logs, they used pkgconfig :)

/azp run

The two failures in arm64-windows were caused by a download failure of powershell while trying to install the cross-triplet prerequisites (protobuf and boost-build).

A suitable version of powershell-core was not found (required v7.0.3). Downloading portable powershell-core v7.0.3...
Downloading powershell-core...
https://github.com/PowerShell/PowerShell/releases/download/v7.0.3/PowerShell-7.0.3-win-x86.zip -> D:\downloads\PowerShell-7.0.3-win-x86.zip
WinHttpReceiveResponse() failed: 12152

Therefore, this PR is ready to be reviewed and merged.

/azp run

I just asked @JackBoosY if his concerns are still outstanding

LGTM, @Neumann-A do you have other questions about this changes?

@JackBoosY No

Thanks for your help folks!

Thanks again for your contribution Robert!

TYVM for pushing it through :)


Program panics when serde_json is built with arbitrary_precision feature

Description
Program panics when serde_json is built with arbitrary_precision feature
thread 'actix-server worker 0' panicked at src\http_services\api\dashboard\summary.rs:11:14:
called `Result::unwrap()` on an `Err` value: Client(ClientError { message: "failed to parse JSON response from server", source: Some(reqwest::Error { kind: Decode, source: Error("invalid type: map, expected f64", line: 0, column: 0) }) })
stack backtrace:

To reproduce

cargo.toml

[dependencies]
serde_json = { version = "1.0.117", features = ["arbitrary_precision"] }
prometheus-http-query = { version = "0.8.3" }

Then try querying any metric via Client::query().await

Cause

This is due to the way serde_json handles numeric values with “arbitrary_precision” feature.
The current logic forces serde_json to cast the String type to Float64: https://github.com/puetzp/prometheus-http-query/blob/45422c0fce98b82dd745eb01a3dce7b297102aff/src/response.rs#L322

My opinion
This is a bug in the current crate.
I believe there is no need to convert String to F64, as that is the prerogative of those who really need it.
On the other hand, if contributors wish not to fix this, the correct operation of the crate with arbitrary_precision feature should be implemented.

Update: I changed the API a little to eliminate a "helper" enum in the deserialize_f64function. However this alone does not alleviate the issue. I suspect I would have to change more parts that help to deserialize Prometheus API responses, e.g. the internally tagged Rule enum. This enum deserializes responses such as:
{
  "groups": [
    {
      "rules": [
        {
          "alerts": [
            {
              "activeAt": "2018-07-04T20:27:12.60602144+02:00",
              "annotations": {
                "summary": "High request latency"
              },
              "labels": {
                "alertname": "HighRequestLatency",
                "severity": "page"
              },
              "state": "firing",
              "value": "1e+00"
            }
          ],
          "annotations": {
            "summary": "High request latency"
          },
          "duration": 600,
          "health": "ok",
          "labels": {
            "severity": "page"
          },
          "name": "HighRequestLatency",
          "query": "job:request_latency_seconds:mean5m{job=\"myjob\"} > 0.5",
          "type": "alerting",
          "evaluationTime": 0.000312805,
          "lastEvaluation": "2023-10-05T19:51:25.462004334+02:00",
          "keepFiringFor": 60
        },
        {
          "health": "ok",
          "name": "job:http_inprogress_requests:sum",
          "query": "sum by (job) (http_inprogress_requests)",
          "type": "recording",
          "evaluationTime": 0.000256946,
          "lastEvaluation": "2023-10-05T19:51:25.052982522+02:00"
        }
      ],
      "file": "/rules.yaml",
      "interval": 60,
      "limit": 0,
      "name": "example",
      "evaluationTime": 0.000267716,
      "lastEvaluation": "2023-10-05T19:51:25.052974842+02:00"
    }
  ]
}

In this instance members of the rules array are deserialized based on the value of the type parameters, recording or alerting. An internally tagged enum is the most straightforward way to deserialize such a response. Based on the various issues filed in serde_json and the failing unit tests in this crate when arbitrary_precision is enabled, this internally tagged enum seems to be part of the issue.
However I'm hesitant to change the deserialize procedure here because a different implementation that does not use an enum (but e.g. an intermediary HashMap instead) would almost certainly be way less efficient.
I will think this over again soon, but for now this issue seems to be blocked on this which in turn depends on this.


Radio buttons where there are multiple keys chosen show the first one alphabetically as selected

URL
https://www.openstreetmap.org/way/423881224
How to reproduce the issue?
Look at a node which has some form of radio button interface implemented via multiple keys, which appears to currently be just these two fields:
https://github.com/openstreetmap/id-tagging-schema/blob/bde8a416c80cbbb27a0ad36df3c6dcec3151ec8b/dist/fields.json#L2805-L2812
https://github.com/openstreetmap/id-tagging-schema/blob/bde8a416c80cbbb27a0ad36df3c6dcec3151ec8b/dist/fields.json#L1083-L1095
I know the second one has been somewhat divisive:
https://github.com/openstreetmap/iD/issues?q=is%3Aissue+toilet
So without wishing to re-open those debates, given a radio button is currently used to render these (and hence you can't select more than one choice), wouldn't it be better if no option was selected on the radio when more than one was chosen (so the blue highlighting was more obvious).
Logically I could imagine the second existing and indeed a bit of Overpass searching found some fords in tunnels, so they're both real world scenarios.
Example node:
Structure: https://www.openstreetmap.org/way/423881224
Toilet: https://www.openstreetmap.org/node/411286195
Screenshot(s) or anything else?
Structure:

Toilets:

Which iD Editor versions do you see the issue on?
Released version at openstreetmap.org/edit
Which browsers are you seeing this problem on?
Firefox

I think what you’re seeing is that the field checks the radio button that comes last visually, not first alphabetically. The radio buttons have the same name attribute, so the browser only allows one to be checked at a time, but this code checks each applicable radio button in order, regardless of the other radio buttons. That causes the browser to uncheck all the previously checked radio buttons:
https://github.com/openstreetmap/iD/blob/2f9285e0115e6bc0aebd46664e84c6a7e0e02755/modules/ui/fields/radio.js#L269-L274
The blue highlighting for multiple selection is subtle indeed. I wonder if that’s a side effect of implementing simultaneous editing of multiple features in #7306. It looks slightly different when selecting two features with different values for the same key, but none of the radio buttons is checked, which is good:

I think what you’re seeing is that the field checks the radio button that comes last visually, not first alphabetically.

Well spotted @1ec5 , I clearly hadn't tested well enough and just made assumptions.
Thanks for fixing it @tyrasd !


[DO NOT MERGE] Hotfix/infinite tracing no data

This is a PR intended to make it easy for customers to use the hotfix for Infinite Tracing that fixes an issue where enabling Infinite Tracing can sometimes lead to no data reported.
Do not merge!
This fix is already merged to dev and will be removed when 7.0 is released.

To use this hotfix, add the following to your Gemfile.
This is based off the 6.15.0 release and patches only Infinite Tracing:
gem 'newrelic_rpm',
  :git => 'https://github.com/newrelic/newrelic-ruby-agent.git',
  :branch => 'hotfix/infinite_tracing_no_data'

Are you sure it should be gem 'newrelic_rpm'? Previously I've gotten instructions to modify the newrelic-infinite_tracing, I would expect it to be the same this time as well? I.e. for us this would mean:
gem "newrelic_rpm", "~> 6.15"
gem 'newrelic-infinite_tracing',
  :git => 'https://github.com/newrelic/newrelic-ruby-agent.git',
  :branch => 'hotfix/infinite_tracing_no_data'

@rokcarl -- you're right.  The correct way to deploy this fix presently is:
gem "newrelic_rpm", "~> 6.15"
gem 'newrelic-infinite_tracing',
  :git => 'https://github.com/newrelic/newrelic-ruby-agent.git',
  :branch => 'hotfix/infinite_tracing_no_data'

This fix has been released in our 7.0.0 version of the agent. Please upgrade to this version for this fix.


Some log types use "type" field.

Describe the bug
Currently we should be using log_type as the field to define the type of log it is. We have HIPMATCH, USERID, GLOBALPROTECT logs that are still using "type".
Expected behavior
Change remaining logs to use "log_type"
Current behavior
Customers have to make searches based on type or log_type.
Possible solution
Update transforms.conf to use log_type on all logs. Then update dashboards and datamodels to reflect the change.
Context
This has been reported by customers. This was introduced in 7.1.0 of the app.

Fixed in version 8.0.0


dts: yaml: remove unused id field

Some new yaml bindings that got committed added back in the 'id:' field
which we have removed.  Remove it from those dts binding files.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

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

@@            Coverage Diff             @@
##           master   #10646      +/-   ##
==========================================
- Coverage   53.25%   53.09%   -0.16%
==========================================
  Files         214      214
  Lines       25823    25823
  Branches     5693     5693
==========================================
- Hits        13752    13711      -41
- Misses       9756     9803      +47
+ Partials     2315     2309       -6

Impacted Files
Coverage Δ

include/posix/posix_sched.h
0% <0%> (-100%)
:arrow_down:

lib/posix/pthread_barrier.c
0% <0%> (-100%)
:arrow_down:

lib/posix/pthread_cond.c
62.5% <0%> (-20.84%)
:arrow_down:

lib/posix/pthread.c
68.03% <0%> (-12.33%)
:arrow_down:

lib/posix/pthread_mutex.c
76.38% <0%> (-1.39%)
:arrow_down:

lib/posix/pthread_sched.c
62.5% <0%> (+20.83%)
:arrow_up:

Continue to review full report at Codecov.

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


Expose default serializer

Is your feature request related to a problem? Please describe.
Cosmos REST contracts are JSON.net annotated. For customers using streaming API a serialization implementation which can work with them is required.
Describe the solution you'd like
CosmosJsonSerializer.Default
Describe alternatives you've considered
Custom JSON.net code but it breaks once moved out of JSON.net FX.

I left my opinion in another thread. The core API should be serialization agnostic and implement "stream only" functionality. The serialization should then be implemented in extensions and abstractions in a manner similar to how aspnet mvc abstracted away JSON.Net.


Plugin Level configuration not validating in vscode

Describe the bug
When adding configuration per plugin, the schema is considered invalid

To Reproduce
Steps to reproduce the behavior:
Code sandbox doesn't seem to validate the schema, so no code sandbox applies

My GraphQL schema:

Not applicable

My GraphQL operations:

Not applicable

My codegen.yml config file:

overwrite: true
schema: "src/core/graphql/schema.graphql"
# These are used to automatically generate the various React hooks we use from documents
documents:
  - "src/**/*.graphql"
hooks:
  afterOneFileWrite:
    - prettier --write

generates:
  src/core/graphql/schema.ts:
    - typescript:
        declarationKind: "interface"
        enumsAsTypes: true
        defaultScalarType: unknown
        skipTypename: true
    - "typescript-operations":
        declarationKind: "interface"
        avoidOptionals: true
        defaultScalarType: unknown

Expected behavior
I would expect to be able to get autocomplete / validation in vscode as suggested in the documentation when inputting a valid config. The CLI command generates the correct output, so the config is technically correct, it's just the YAML schema that doesn't seem to support plugin level configuration
Environment:

OS: MacOS
@graphql-codegen/...:

"@graphql-codegen/cli": "~1.21.7",
"@graphql-codegen/typescript": "~1.23.0",
"@graphql-codegen/typescript-operations": "~1.18.4",

NodeJS: v14.17.0

Additional context
Using RedHat YAML plugin as suggested by docs. v0.21.1 which is the latest version at the time of this writing.

@dotansimha I'm willing to pick this up to get it fixed if you / someone can point me in the general direction of what I would need to change

@dotansimha I'm willing to pick this up to get it fixed if you / someone can point me in the general direction of what I would need to change

This might be tricky. We are generating a JSON-Schema (schema.json) file based on the TypeScript signatures, and then we have a script that knows how to build the entire tree. I think there might be a bug there.
The code for this script is here: https://github.com/dotansimha/graphql-code-generator/tree/master/packages/utils/config-schema/src

Note that root-level config isn't correct in the JSON schema either, there is no autocompletion and validation fails.

Fixed!


Search engine over the training data

Hello all,
Thanks a lot for releasing this dataset. I was wondering whether you were planning to release any form of "search engine" over your dataset which is something similar in spirit to what other people started doing for LLM data (e.g., ROOTS Search tool).
Many thanks,
Alessandro

No but we have a Nomic map: https://atlas.nomic.ai/map/obelics


Mega Man eye material transparency

Mega Man's eye materials are oddly translucent when seen in SSBHE but not ingame, exposing the inside of the mesh.

Certain animations are setting CustomVector0 to 0,0,0,0 and causing the eyes to become transparent. I'll do some testing to figure out how the game is handling the alpha in this case.

I double checked in an emulator and the alpha rendering is correct. There's a bug in the ssbh_wgpu renderer causing the post processing to not apply properly if the mesh has alpha set to 0.


merge: implement batched backwards sync strategy

Description
Besu currently implements a block-by-block backward merge sync strategy for instances when the consensus layer client sends newPayload and forkchoiceUpdated requests for blocks that besu does not yet know about.
Rather than incurring the network traffic of block-by-block requests, we should request a batch of blocks in reverse for a specified hash.  This should reduce the network load significantly and speed up besu's backward sync when it is substantially behind the current chain head.
Acceptance Criteria
backward sync:

requests headers in batches of 200, similar to foward sync behavior
uses block numbers to request the correct batch size when the number of headers is below 200

The BWS already supports batching.


New tag for master changes? Need bug fix for #264

There's several changes in master, but we don't pull from master.
The bug fix we need is https://github.com/jenkinsci/pipeline-aws-plugin/issues/264 and there's others.
Can we get a new tag created that includes all the changes in master?

Hey @sr75 did you resolve this?  Unsure how to get the latest so I can pull in this fix also.

Could you please release a new Version to the Jenkins Plugins Index.
I still have the problem from https://github.com/jenkinsci/pipeline-aws-plugin/issues/264. It was fixed more than 2 years ago, but the current version still doesn't contain the fix.


Op latency: weird spikes every 2 frames

We need to look why we have peeks every 31.5 seconds.
I can understand 16ms (assuming 60FPS system clock), but 2 ticks looks weird.
Primary suspect - DeltaScheduler, who does 20ms delays (that would translate into 2 ticks).
We should also re-evaluate once BatchManager is out, and implicit batching at JS turn are in.
We also need better view into all stages of this process:

How long ops are sitting on machine until they emitted to socket (that would catch/show batch manager impact)
Actual time to ack op (same as today but minus # 1 & # 2, i.e., only time to network and service time).
How long ops are sitting in inbound queue until it is processed (that would capture impact of DeltaScheduler & ScheduleManager).

Office_Fluid_FluidRuntime_Performance
| where Event_Time > ago(5d)
| where Data_eventName == "fluid:telemetry:OpPerf:OpRoundtripTime"
| where Data_duration >= 0
| summarize count() by bin(Data_duration, 1)
| render linechart
| where Data_duration <= 500

I see 531K "InboundOpsProcessingTime" events in one day.
So yes, DeltaScheduler seems like is being hit a lot.
One thing that I do not like is that we issue event only if we were processing over 2K ops.
I do not see why 2K or any number of ops is important here. It feels like once we set a timer in batchEnd(), we have to report an event that it happened, and ideally

how many ops we already processed till that moment
how many ops are remained in queue
Once done (reached idle), how many ops actually were processed (will be different from # 2, as ops are coming in).

@andre4i, I'll assign it to you, as I think most likely this is batch manager.
That said, I opened a bug (see above) for DeltaScheduler as well, and it is also a big suspect, given it uses 20ms delays, which will map nicely to 32ms on 60fps system clock system. Please work with Navin figure out next steps here.
Also note that I've opened couple issues to better track various parts of processing pipelines to quickly figure out where problems are.

#9505, #8909, #8912 are the issues tracking actual improvements and collection of more data to get more insights into the problem.
So, moving this issue to May to give enough time to have time to analyze data when it's available.

We still do not have data (latest changes are not deployed), i.e. "OpRoundtripTime" events do not have yet Data_durationInboundQueue property. Pushing it out.

We have some data.
let P=95;
union Office_Fluid_FluidRuntime_*
| where Data_eventName contains "OpRoundtripTime"
| where isnotnull(Data_durationOutboundQueue)
| summarize count(), percentile(Data_durationInboundQueue, P), percentile(Data_durationInboundToProcessing, P), percentile(Data_lengthInboundQueue, P)

count_
percentile_Data_durationInboundQueue_95
percentile_Data_durationInboundToProcessing_95
percentile_Data_lengthInboundQueue_95

7,874
364.18326118326115
25.154761904761905
1

Key observations:

durationInboundQueue is really hig.
durationInboundToProcessing: It's average is almost 2x worse than P95! Suggesting really heavy tail
Queue length is tiny, so it's not number of ops.

durationInboundQueue distribution:

durationInboundToProcessing distribution:

And duration:

So, looks like durationInboundQueue  is the main cause of the problems!

this.opPerfData.durationInboundQueue = this.opProcessingTimes.opStartTimeInboundPushEvent - this.opProcessingTimes.opStartTimeSittingInboundQueue;
opStartTimeInboundPushEvent - "push" time for inbound queue
opStartTimeSittingInboundQueue - "push" time for outbound queue
That eliminates DeltaScheduler as the problem.
Note that inbound queue is sorted, so if ops are coming out of order, we will not add them until we fill in the gap.
Outbound queue is filled in only if we are connected as "write", which should be fine, as switching from "read" to "write" will reset all telemetry and will not send data including transition to write.
I highly doubt that gaps in ops are causing it - we observer very few gaps in general once connection is established.
And client does not send ops (and thus does not measure these times) until it is fully caught up on connection.
We do not have signal data like that, but manual experiments show that signals are much smoother in terms of end-to-end feedback. So, it's very unlikely socket.io depending on system clock to process incoming messages.
This suggest the most likely cause is PUSH.

union office_fluid_ffautomation_*
| where Data_eventName contains "OpRoundtripTime"
| where isnotnull(Data_durationOutboundQueue)
| summarize count(), toint(avg(Data_durationInboundQueue)), toint(percentile(Data_durationInboundQueue, 95)), percentile(Data_pingLatency, 95) by Data_driverType

Data_driverType
count_
avg_Data_durationInboundQueue
percentile_Data_durationInboundQueue_95
percentile_Data_pingLatency_95

routerlicious
132,896
668
1,972
4,162

odsp
253,882
103
125
45

tinylicious
91,105
7
26

local
12,652
1
5

Ping times are collected from previous ping on socket, so it can be above rountrip times.
This data is very concerning. It confirms that op rountrip times high and have weird spikes in distribution most likely due to service processing (ordering service).
What is even more concerning, that FRS numbers are substantially worse than ODSP.
@GaryWilber , @znewton, I do not know how to drive this to resolution, but if you have service numbers in terms of how long it takes to process ops in ordering service, and if we can do distribution chart to see if we observe similar spikes as what we see from client, then we should look into this data.

We have actual ping data, but number of hits is extremely low (499 hits in last 2 weeks), and data exists only for FRS, so I'd not look into it that much:

union office_fluid_ffautomation_*
| where Event_Time > ago(14d)
| where Data_eventName contains "DeltaLatency"
| summarize count(), toint(avg(Data_duration)), toint(percentile(Data_duration, 95)) by Data_driverType

ODSP stress tests (run by IDC team - much bigger scale) - distribution of durationInboundQueue times.
Same observations - distribution has multiple spikes.

R11S data is really weird and does not match ODSP pattern. It has huge tail and humongous standard deviation.
At the same time, we do not observe same problem of multiple spikes in data, it's standard hockey-stick kind of distribution (with very long tail). It's pretty obvious from the fact that P90 < average!

union office_fluid_ffautomation_*
| where Data_eventName contains "OpRoundtripTime"
| where isnotnull(Data_durationOutboundQueue)
| summarize count(), toint(avg(Data_durationInboundQueue)), toint(percentile(Data_durationInboundQueue, 50)), toint(percentile(Data_durationInboundQueue, 90)), toint(stdev(Data_durationInboundQueue))
by Data_driverType

Data_driverType
count_
avg
P50
P90
stdev

routerlicious
136,867
653
104
322
2959

odsp
261,408
103
93
124
49

tinylicious
93,229
7
4
14
9

local
12,876
1
1
2
3

From Gary:
Random latency spikes when producing messages · Issue #2912 · edenhill/librdkafka (github.com)
Significant performance regression between 1.6.1 and 1.7.0 · Issue #3538 · edenhill/librdkafka (github.com)
librdkafka/CHANGELOG.md at master · edenhill/librdkafka (github.com) v1.9.0 - a lot of relevant changes:
FRS is linux based and Push is window's based. So the queuing/timer logic in the kafka library will differ across the two.


Show some basic statistics in the terminal

I find myself wanting to know some basic values often enough it might be worth displaying these in the terminal.  I'm kind of thinking along the lines of what ngrok does.

For Owncast it would be nice to surface:

Machine CPU usage
Machine memory usage
Number of current viewers
Stream duration
Video configurations currently in use

Maybe using something like https://github.com/gdamore/tcell or https://github.com/gizak/termui ?  This idea might be overkill, but it would be pretty user-friendly and useful.  Maybe I'll experiment sometime after the 0.0.1 release.

After hearing feedback from different people on #111 I've decided to change gears and go with the following game plan:

Drop the idea of a full terminal performance/configuration dashboard.
Add some simple error events based on hardware utilization to alert the service operator things aren't going well (cpu spikes mostly).
Plan for a future lightweight web dashboard that has more detailed statistics to fill the need I was trying to fill with the terminal version.  This would include creating a new metrics package.
Keep getting feedback on what values would be useful to expose in different ways.  @graywolf336 mentioned prometheus metrics as an option.  But I figure we can wait until we have our own little dashboard that works for everyone out of the box before we start doing specific implementations for more niche use cases.


Add version-info and up-time

Hi version info and up-time value is very usefull.

Hey there :wave:
Wouldn't this be solved using CustomGauge and CustomCounter? Or is there something missing in the lib that would allow you to register these without modifying the lib itself?
(Also you committed your .vscode directory)

Seeing as it's been a few months with no feedback, I'll close this PR. Feel free to reopen it or provide some more details :smile:


How to append extra info (like session info ,data from async request) to the event?

I want to append some info in event like session info, authToken , project key,etc.. to my data.
For this, i made a file in which there are async call to query the database.. the result should be appended to data .. and then aggregate.apply() method should be called..
I want to do this before event is created , so i think  aggregate.apply() method is right place.. what you think?Is this the right place to append data to the event?
Secondly,I was thinking about using meta, but don't know how could i use this?
In  domain.defineCommand() you have mention about meta that
// optional, if defined theses values will be copied to the event (can be used to transport information like userId, etc..)
  meta: 'meta'

By this do you mean that the values of meta should be come from command?

yes, you can pass extra infos via command.meta to the event

Is there any other way to achieve this??
i mean inside domain.defineCommand() i want to use my custom method to append data to event.. here is my code structure..
module.exports = require('cqrs-domain').defineCommand({
    name: 'createNarration',
    payload: 'payload'
}, function (data, aggregate) {
       require('/path/to/myfile.js').append(data, "createNarration", function(err, res){
        if(err){
           cosnole.log(err);
        }else{
           // Not working :Type Error is generated..
            aggregate.apply('narrationCreated', data);
        }
    });
})

If my code structure is like below, its working fine..
module.exports = require('cqrs-domain').defineCommand({
    name: 'createNarration',
    payload: 'payload'
}, function (data, aggregate) {
    //Working fine but i want to append info to data..
    aggregate.apply('narrationCreated', data);
})

But i am using my custom method to fill data before event created. Though data is filled with my custom info but,
The problem here i am facing is after appending data to events ,aggregate.apply can not apply data to event ..
Error is like
 aggregate.apply('narrationCreated', data);
                      ^
TypeError: Object #<Object> has no method 'apply'
    at /home/vishant/Documents/cqrs-sample-vishant/domain/domainFiles/masters/narration/commands/createNarration.js:12:23
    at module.exports.append (/home/vishant/BizOs/bizosprojects/commonfeatures/eventdenormalizerservice/utility/commandappenderutil.js:155:13)
    at _asyncMap (/home/vishant/BizOs/bizosprojects/commonfeatures/node_modules/async/lib/async.js:254:17)
    at async.eachSeries.iterate (/home/vishant/BizOs/bizosprojects/commonfeatures/node_modules/async/lib/async.js:157:25)
    at _asyncMap (/home/vishant/BizOs/bizosprojects/commonfeatures/node_modules/async/lib/async.js:251:21)
    at async.series.results (/home/vishant/BizOs/bizosprojects/commonfeatures/node_modules/async/lib/async.js:615:34)
    at module.exports.append.async.series.data.error (/home/vishant/BizOs/bizosprojects/commonfeatures/eventdenormalizerservice/utility/commandappenderutil.js:150:17)
    at async.series.results (/home/vishant/BizOs/bizosprojects/commonfeatures/node_modules/async/lib/async.js:610:21)
    at _asyncMap (/home/vishant/BizOs/bizosprojects/commonfeatures/node_modules/async/lib/async.js:249:17)
    at async.eachSeries.iterate (/home/vishant/BizOs/bizosprojects/commonfeatures/node_modules/async/lib/async.js:149:13)

Process finished with exit code 8

F.Y.I.  aggregate object has all the attribute & methods except 1 that is apply method

This is what i was trying to say.. I have replicate the issue ..
I have added a custom function to your cqrs-sample project.. Below is my commit to your repo..
https://github.com/vishant-dhandha/cqrs-sample/commit/20a0f2472366f44518f5dd122e20f20f9b6e9835
Here i have added async http call to fetch some info & append to the 'data'..
At aggregate.apply('itemCreated', data); data contains my desired info but some how server stopped with TypeError: Object #Object has no method 'apply'

this is not possible... the whole apply part is completely sync...
can't you extend your command before it enters the domain.handle?

I want to make a async call, if that async call successfully returned then and then only the event should be generated.
How could i make out this?
Saga is the solution for that???

asynccall(function (err, extenstion) {

  if (!err) {
    command.myExtension = extension;
    domain.handle(command);
  }

});

I have an idea:
Would it be ok for you, if you define a precondition for that command in which you make your async requests and save the resulting data in a sort of clipboard on the aggregate and afterwards in de command just get that saved data and extend your event?

Yes , this (precondition) solved my problem..
thank you


Does not show up in UI

I read the closed issue and tried to remove AGB (which never showed up in the UI as well) and also removed onnxruntime.
It looks as if onnxruntime is installed again after removing the folder, but then I get errors like this:
Error loading script: postprocessing_rembg.py
Traceback (most recent call last):
  File "C:\Users\***\stable-diffusion-webui\modules\scripts.py", line 248, in load_scripts
    script_module = script_loading.load_module(scriptfile.path)
  File "C:\Users\***\stable-diffusion-webui\modules\script_loading.py", line 11, in load_module
    module_spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "C:\Users\***\stable-diffusion-webui\extensions\stable-diffusion-webui-rembg\scripts\postprocessing_rembg.py", line 5, in <module>
    import rembg
  File "C:\Users\***\stable-diffusion-webui\venv\lib\site-packages\rembg\__init__.py", line 5, in <module>
    from .bg import remove
  File "C:\Users\***\stable-diffusion-webui\venv\lib\site-packages\rembg\bg.py", line 21, in <module>
    from .session_base import BaseSession
  File "C:\Users\***\stable-diffusion-webui\venv\lib\site-packages\rembg\session_base.py", line 4, in <module>
    import onnxruntime as ort
ModuleNotFoundError: No module named 'onnxruntime'

If I keep the existing onnruntime folder, I get no error messages, there is just nothing in the UI to activate rembgr.... The onnx already installed seems to be 1.13.1

Rembgr use rembg package that depends on onnxruntime, so it must be there. If extension exist in /extenstions dir and you dont see it in the UI it must be some kind of error. It is very strange if nothing was printed in the console... May be you can try it on fresh Web UI installation.

I was - after watching a video about this extension, expecting to find it under im2img and as a simple checkbox. What I do have is a menu under extras with different models like u2net. However, trying to use any of these models I get this:
Error completing request
Arguments: (0, <PIL.Image.Image image mode=RGB size=2160x2160 at 0x1B5A09B33D0>, None, '', '', True, 0, 4, 512, 512, True, 'None', 'None', 0, 0, 0, 0, 'u2net_human_seg', False, True, 240, 10, 10) {}
Traceback (most recent call last):
  File "C:\Users\****\stable-diffusion-webui\modules\call_queue.py", line 56, in f
    res = list(func(*args, **kwargs))
  File "C:\Users\****\stable-diffusion-webui\modules\call_queue.py", line 37, in f
    res = func(*args, **kwargs)
  File "C:\Users\****\stable-diffusion-webui\modules\postprocessing.py", line 56, in run_postprocessing
    scripts.scripts_postproc.run(pp, args)
  File "C:\Users\****\stable-diffusion-webui\modules\scripts_postprocessing.py", line 130, in run
    script.process(pp, **process_args)
  File "C:\Users\****\stable-diffusion-webui\extensions\stable-diffusion-webui-rembg\scripts\postprocessing_rembg.py", line 53, in process
    session=rembg.new_session(model),
  File "C:\Users\****\stable-diffusion-webui\venv\lib\site-packages\rembg\session_factory.py", line 66, in new_session
    ort.InferenceSession(
  File "C:\Users\****\stable-diffusion-webui\venv\lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 347, in __init__
    self._create_inference_session(providers, provider_options, disabled_optimizers)
  File "C:\Users\****\stable-diffusion-webui\venv\lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 395, in _create_inference_session
    sess.initialize_session(providers, provider_options, disabled_optimizers)
RuntimeError: D:\a\_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1069 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "C:\Users\****\stable-diffusion-webui\venv\lib\site-packages\onnxruntime\capi\onnxruntime_providers_tensorrt.dll"

Which is, what the other user had in the closed issue. However, the path to the solution there did not work for me.

Ok, so I deleted all folders in site-packages with onnx in the folder name. Then the installation of onnx by your extension installed 1.14.1 instead of 1.13.1 - and now it works...
Maybe it would be helpful to provide a screenshot of where to find the setting after installing the extension... :)

I will, thank you :)


Never-completing builds in certain situations

This is the issue mentioned in https://github.com/wojciech-kulik/xcodebuild.nvim/discussions/44#discussioncomment-8540610
First of all, my environment is using neovide  and dashboard.nvim.
If I open and build the project inside the shell using nvim or neovide commands from the root of project, it works fine.
But Run neovide and open the that project in the recently projects section of dashboard.nvim,
When I try to build it, the build never finishes and continues.
Even when using the nvim command in the shell, running neovim with commands like nvim {project root} on the parent root of the project causes problems.
Just in case, I created a new iOS sample project and tried the same thing, but the problem was the same.
The output of :lua print(vi.fn.getcwd()) in all situations is the same.
When running as a shell command from the project root

When running as Dashboard.nvim, etc.

I investigated the problem and it's caused most likely because of switching between projects via Dashboard. In general, the plugin doesn't support switching projects. However, there is some workaround for it, you can check this thread:  #7.
Here is a sample config for Dashboard that reloads the plugin:
return {
  "nvimdev/dashboard-nvim",
  event = "VimEnter",
  dependencies = { { "nvim-tree/nvim-web-devicons" } },
  config = function()
    require("dashboard").setup({
      config = {
        project = {
          action = function(path)
            vim.cmd("Telescope find_files cwd=" .. path)

            for name, _ in pairs(package.loaded) do
              if string.find(name, "xcodebuild%.") then
                package.loaded[name] = nil
              end
            end

            require("xcodebuild").setup({
              -- your options
            })
            require("xcodebuild.project.appdata").load_last_report()
          end,
        },
      },
    })
  end,
}

Please check it out and let me know. If you still encounter issues, please check out what shows the command :XcodebuildShowConfig make sure that the project is correctly set up. If you messed with the previous version without reloading the plugin it could break your project config, so setup it up again if necessary.

Thanks!
I reloaded the plugin as you suggested and it works fine.
I'm using Lazy.nvim, so Lazy reload xcodebuild.nvim also seems to work.
Thanks again for the awesome plugin.


Compaction and eviction are mutually exclusive

Mutually exclusive because of Timeline::layer_removal_cs. This means that for long compaction::create_image_layers we currently stall both kinds of evictions (disk usage based, eviction_task) once they try to evict anything of "long compaction tenant."
So far this has not been observed with typical sized tenants but it is very observable with 1TB database sizes.
Related PRs, work ideas:

silence std::io::ErrorKind::NotFound for eviction failure in #4737
we could use a JoinSet for per Timeline::evict_layers with cancellation to not wait for evictions taking longer than X min because the decisions what to evict will be stale very soon

similarly in eviction_task we could just deem the attempt a failure soon

I think the solution will be to:

get rid of RemoteLayer (internally managed residentness)
Arc::<dyn PersistentLayer>::drop handling the file deletion

It is possible the (2) will not require (1), but it would simplify the implementation. It also raises questions like "what if an to-be-evicted layer keeps on getting accessed, will the to-be-evicted flag be reset". Unsure what is the answer. The implemented struct Layer has "access always wins eviction". I will handle that in follow-up to #5108.
### Tasks
- [ ] https://github.com/neondatabase/neon/pull/4737
- [ ] https://github.com/neondatabase/neon/issues/5172
- [ ] https://github.com/neondatabase/neon/pull/4938
- [ ] https://github.com/neondatabase/neon/pull/5766
- [ ] https://github.com/neondatabase/neon/pull/5767
- [ ] https://github.com/neondatabase/neon/pull/5756
- [ ] https://github.com/neondatabase/neon/pull/5743
- [ ] https://github.com/neondatabase/neon/pull/5742
- [ ] https://github.com/neondatabase/neon/pull/5108
- [ ] https://github.com/neondatabase/neon/pull/5852
- [ ] remove Timeline::evict_batch, simplify disk usage based eviction, account for failed evictions

A bit on the edge if this is really a bug, but perhaps that is the closest one we have. Timeline::layer_removal_cs existence should be the bug. Related to evictions we do have race conditions right now as the layer_removal_cs is not held for every starting layer access, which means those could fail with same std::io::ErrorKind::NotFound if the fd was not already open (cache => randomness).

It is possible the (2) will not require (1), but it would simplify the implementation. It also raises questions like "what if an to-be-evicted layer keeps on getting accessed, will the to-be-evicted flag be reset". Unsure what is the answer.

Thinking about this more with (1 -- the internally managed residentness) it is very possible that it will keep happening. So (1) and (2) are mutually exclusive; the Arc::<dyn ...>::drop works only with GC when a layer goes away forever.
For allowing the eviction with (1) we should have "read interests" which manage the deletion for eviction purposes. Basically anytime when we will have to read the layer, we need to register a read interest while the interest is over:

is not yet uploaded queue
we have open key/value iterators
we have open get_reconstruct_value queries

Only the upload queue is difficult out of these; sadly no layers are being used there only paths which I've never understood the reason for.

Tests have been finally solved in https://github.com/neondatabase/neon/pull/5108 this week. The RemoteTimelineClient::shutdown might be good enough. Open problem hopefully to be discussed next week.
Timeline::evict_layers cleanup has been started.

status update:
@koivunej will open for review the PR, and will create a follow up back pressure issue

Merging it this week, hopefully today. Then moving on to eviction improvements.

this has been merged.


Auto increment slugs on per model basis

Fixes #10147 Adds ability to auto increment slug on per model basis

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[x] For new features: Has the documentation been updated accordingly?

Thanks! I just wanted to say how brilliant the solution you proposed was. It fixed the problem with minimal code change while also being generic enough to cater to other custom form requirements. Extremely elegant!

Yeah. I'm pretty pumped for this feature, I hope it will be really useful for 6.0 and beyond. Sometimes it's good to thrash around a few ideas and see what feels right before going all in for a PR.


linqpad6

Microsoft Reviewers: Open in CodeFlow

I'm struggling to get this app to run even though I install the .Net Core runtime.


Setup

I’m having issues with the initial setup. I’m not able to add/remove apps in settings.

Fixed in latest build. It will be on bigboss tmoorrow, and is available now at repo.ghostbin.co


Ensure the product name is "Google Cloud Tools for Visual Studio" everywhere

Ensure we are using the right product name in all visible places for the UI, including the Visual Studio Gallery.

It is in the title for the OAUTH flow as well.

This is now fixed.


Remove LBRY Prefix

A good tedious task to pass a gloomy Sunday afternoon.

Some of these are fine (LBRYSession -> Session, LBRYWallet -> Wallet) but I
would caution against changing LBRYFile to File. An LBRYFile is just one
particular method of breaking a regular plaintext file into chunks suitable
for being distributed via lbrynet and then recombining them. There are
other ways it can be done (for example, not encrypting the chunks, not
putting the encryption key directly in the stream descriptor, etc) and I
think it would be a good idea to design the system with that in mind.
LBRYFile is not a good name but I think File is a step in the wrong
direction.
On Sep 26, 2016 9:50 AM, "Jeremy Kauffman"<EMAIL_ADDRESS>wrote:

@kauffj commented on this pull request.
👏
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lbryio/lbry/pull/174#pullrequestreview-1537828, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdx5KLaQvXWVjtVdzlhmEOr3qJU_Y0mks5qt82HgaJpZM4KGEvE
.

I switched File to EncryptedFile

Probably better to get this in sooner rather then later as it touches 46 files and will be a pain to handle merge conflicts.
@jackrobison: thoughts?


Fail when zero events

https://github.com/thejevans/mla/blob/d7dee4ffd114df7672bcf568b9e148929208ae92/mla/test_statistics.py#L77
It will throw error(I think) when you pass in zero event. Either catch that during minimize_ts and evaluate_ts(return zero when no event at the beginning ) or try to deal with it during _preprocess

It will throw a warning for sure. where's the errror?

looks like the error will come from the background_dec_spline in event_model.
is ts = 0 for no events the behavior that we want? If so, I'll add it.

I think a lot of places will throw an error if we pass zero events to it. Yes, we do need to return a ts of 0 for zero events since it is normal for a short time window having no background events.

fix implemented


[Bug]: I cant define the Appwrite domain

Description
I have done a one-click installation of AppWrite, the problem comes when you want to define a domain for it, there is no possible way to do it, when you click on the appwrite container it throws error 500

Minimal Reproduction (if possible, example repository)
Install the appwrite app from the services and try define a domain
Exception or Error
Error 500
Error: Call to a member function databaseType() on null (View: /var/www/html/resources/views/livewire/project/service/show.blade.php) (View: /var/www/html/resources/views/livewire/project/service/show.blade.php)
Version
v4.0.0-beta.114

It has been fixed in beta.115. Sorry about it.


Add favicon to website

I propose that a favicon be added to the main website ( http://www.kiwijs.org/ ).

I second this proposal ;)

It's so rare to find sites which got no favicon! Please make 1 ASAP!  :P

Agreed :)
Sent from my iPhone

On 26/01/2015, at 11:54 am, GoToLoop<EMAIL_ADDRESS>wrote:
It's so rare to find sites which got no favicon! Please make 1 ASAP! :P
—
Reply to this email directly or view it on GitHub.

Here's a preview of 8 different KiwiJS favicons:

And here's the individual icons:

Very nice @thornyd! I be thinking the transparent Kiwi (also known as Kiwi Icon #6) looks the best. Do we have any other recommendations.

I prefer the black outline on kiwi-favicon-3 :+1:

I'm thinking the last one ( all dark blue ). It's sleek.

I like #6 and #8... I'd probably go with #8 since Zach thinks its sleek :)
Any other opinions?
On Mon, Jan 26, 2015 at 2:49 PM, Zachary W. Freiberg <
<EMAIL_ADDRESS>wrote:

I'm thinking the last one ( all dark blue ). It's sleek.
—
Reply to this email directly or view it on GitHub
https://github.com/gamelab/kiwi.js/issues/145#issuecomment-71405702.

--
Dan Milward
M +64 21 449 901
178 Willis Street – Wellington CBD – Wellington 6011 – New Zealand
www.kiwijs.org  -  www.gamefroot.com http://www.gamefroot.com/leveleditor

The black bird has been placed on the site since it seems it has the majority. Checks it out!
Thanks @GloverDonovan that :)

Cool :smile:


Fix typo in main.rs

Hello, I'm Hayashi-Yudai.
I use your repository and learn from the codes. This time, I found very tiny typo in help message. I'm grad if you merge my pull request.
Sincerely.

Codecov Report

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

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   73.62%   73.41%   -0.22%
==========================================
  Files           8        8
  Lines         345      346       +1
==========================================
  Hits          254      254
- Misses         91       92       +1

Remove return; to pass CI

Thanks :)


discv4: Don't let some task crashes propagate to run_task()

Some discv4 coroutines executed with run_task() could leak exceptions
causing the whole component to stop, even though a failure in any of
them do not affect the component's operation. These coroutines are now
wrapped and any exceptions they leak are logged as errors.
This was motivated by the following crash, which I saw today and didn't
give me any clues about what was the invalid address nor how it ended up
in our DB
Traceback (most recent call last):
  File "trinity/components/builtin/peer_discovery/component.py", line 124, in <module>
    trio.run(main)
  File "/home/salgado/virtualenvs/trinity-trio/lib/python3.7/site-packages/trio/_core/_run.py", line 1804, in run
    raise runner.main_task_outcome.error
  File "trinity/components/builtin/peer_discovery/component.py", line 120, in main
    await run_standalone_eth1_component(PeerDiscoveryComponent)
  File "/home/salgado/src/trinity/trinity/extensibility/component.py", line 185, in run_standalone_eth1_component
    await run_standalone_component(component, endpoint_type, APP_IDENTIFIER_ETH1, (Eth1AppConfig,))
  File "/home/salgado/src/trinity/trinity/extensibility/component.py", line 171, in run_standalone_component
    await component.do_run(boot_info, endpoint)
  File "trinity/components/builtin/peer_discovery/component.py", line 101, in do_run
    await async_service.run_trio_service(discovery_service)
  File "/home/salgado/src/async-service/async_service/trio.py", line 156, in run_service
    await manager.run()
  File "/home/salgado/src/async-service/async_service/trio.py", line 208, in run
    for _, exc_value, exc_tb in self._errors
  File "/home/salgado/src/async-service/async_service/base.py", line 299, in _run_and_manage_task
    await task.run()
  File "/home/salgado/src/async-service/async_service/trio.py", line 76, in run
    await self._async_fn(*self._async_fn_args)
  File "/home/salgado/virtualenvs/trinity-trio/lib/python3.7/site-packages/trio/_socket.py", line 753, in sendto
    args[-1] = await self._resolve_remote_address(args[-1])
  File "/home/salgado/virtualenvs/trinity-trio/lib/python3.7/site-packages/trio/_socket.py", line 564, in _resolve_remote_address
    return await self._resolve_address(address, 0)
  File "/home/salgado/virtualenvs/trinity-trio/lib/python3.7/site-packages/trio/_socket.py", line 539, in _resolve_address
    host, port, self._sock.family, self.type, self._sock.proto, flags
  File "/home/salgado/virtualenvs/trinity-trio/lib/python3.7/site-packages/trio/_socket.py", line 159, in getaddrinfo
    host, port, family, type, proto, flags | _NUMERIC_ONLY
  File "/usr/lib/python3.7/socket.py", line 748, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -9] Address family for hostname not supported

Fair enough, let's go with just catching OSErrors from socket.sendto(), then

I'm merging this to get it into the release (I think the fundamental point has been addressed). In case anyone is wondering CI is green, it's just that the session is still open at the time of merging.


Dark Mode Enabled

What does this PR do?
Dark Mode enabled With Toggle Button matching with wbsite theme. Also added Header component which contain logo, Discord Link, Dark Mode toggle. Fixes Banner Image For better UI/UX. Uses React Icons for Dark Mode Button.
Related Issue(s)
Issue #22
Changes
Added Header component which is reusable, Remove unnecessary margins which causing problem in dark mode. Fixes Some CSS issues.
Screenshots or GIFs (if applicable)

Checklist

[x] I have read and followed the contributing guidelines.
[x] My code follows the project's coding style and conventions.
[ ] I have added unit tests (if applicable) to ensure the code works as expected.
[x] I have updated the project documentation (if necessary).

Additional Comments
Please inform me if you want any changes.
By submitting this PR, I confirm that:

[x] I have reviewed my code and believe it is ready for merging.
[x] I understand that this PR may be subject to review and changes.
[x] I agree to abide by the code of conduct and contributing guidelines of this project.

Thanks for contributing 🚀


event in log is not send to http_output endpoint

Describe the bug
event in log is not send to http_output
I can see event info in falco pods log, but I can not receive log in http_output address

How to reproduce it

Expected behaviour

Screenshots

Environment

Falco version:

System info:

Cloud provider or hardware configuration:
OS:

Kernel:

Installation method:

Additional context

Did you enable the http_output? What are the logs in Falco? Could you please provide the Falco configuration (if you are using the helm charts, share the values.yaml file).

Does Falco complain that it cannot send output to the HTTP endpoint? Could you share the Falco logs?

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

any news?

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


LNURL-auth for non LNURL-auth websites (generate passwords)

LNURL-auth describes a way to create a unique signing key for each website and use that to sign a message to login.
Sadly most websites do not yet support that.
We could offer the user a way to use the same flow to generate a password for websites. The user would not need to use a password manager and the password can be derived from the lightning master/root key.

Oh this is a cool idea, we could even autofill the password field. This makes the extension a bit like an "identity wallet" / password manager so backing up this data will be important.
Few questions about edge cases

LNURL-Auth is passwordless, but also anon logins. What is inputed as the email address?
How do we deal with cases when the user changes their password on another device?

the extension still will hold no data as those passwords can be derived on the fly. - similar to the lnurl-auth flow right now.
The extension will hold no sensitive data (except of the connection to the lightning node as currently)
regarding email/username: I guess that's still up to the user.
changing passwords would not work. - at least then the generated password would not work anymore.

as you've mentioned lnurl-auth is specific to websites that have implemented it.
If you want to extend it to regular websites with email/password requirement and tie it to lnurl-auth, you could do:
1- since you already have implemented https://github.com/fiatjaf/lnurl-rfc/blob/luds/13.md you could derive pubkey from this privkey
2- use that as an email address with a domain<EMAIL_ADDRESS>3- for the same domain, getalby (and even other wallets that implement LUD-13) will derive same pubkey/privkey pair.
4- this may be too ambitious but you (or someone else) could run this email service and forward to users own email if they opt-in.

Do you know FIDO2 better? Maybe a LN node's priv. keys are compliant with FIDO specs. That could open up the whole non-LNURL space.


Trip Planner

Experience
Anytype.20231212.192355.71.zip
Title
Trip Planner
Screenshots

Description
Travel with half the hassle. Keep your tickets, bookings, and reference information all in one place, so you do not need to fuss over wifi while traveling.
Categories
Travel
Language
en-US

/gallery approve


Fix UI and add tutorial

There are some inconsistencies in the UI, those should be fixed.
Also add a tutorial.
The tutorial should be opened by the program the first time the user opens it.
It should contain pictures of a Rubik's cube in the lines.
Instructions on how to aim the camera and which colors to take picture first.
First image is white up, red left, blue right
Second image is yellow up, green front, orange right
The lines should match the lines on the cube.
The cube should be well lit from all sides.
If generate button won't work, there should be a popup for the user, to check and correct the squares.
The capture fragment shouldn't have a navbar, or a toolbar.
Delete the options menu. Its not needed.

Also take out the option to use images from phone. Those won't work like ever

Manage if there is no image or one image to decode


Remove show_report capabilities and cosmicqc dependency

Description
This PR removes show_report capabilities and the cosmicqc dependency. We decided this functionality wasn't as useful as creating plots by hand.
Closes #7
What kind of change(s) are included?

[ ] Documentation (changes docs or other related content)
[ ] Bug fix (fixes an issue).
[x] Enhancement (adds functionality).
[ ] Breaking change (these changes would cause existing functionality to not work as expected).

Checklist
Please ensure that all boxes are checked before indicating that this pull request is ready for review.

[x] I have read and followed the CONTRIBUTING.md guidelines.
[x] I have searched for existing content to ensure this is not a duplicate.
[x] I have performed a self-review of these additions (including spelling, grammar, and related).
[x] These changes pass all pre-commit checks.
[ ] I have added comments to my code to help provide understanding
[ ] I have added a test which covers the code changes found within this PR
[x] I have deleted all non-relevant text in this pull request template.

Thanks @jenna-tomkinson !


Split remote storage in its own go mod

This commit removes the dependency between Prometheus and influx.
Note: Go keeps adding the indirect dependencies in go.mod, I can't
remove them.
 This PR points to that commit: https://github.com/prometheus/prometheus/tree/rmremote_storage


add link to http://ibm.github.io to various places

Should be in the descriptions for both this project and the org.
Should also be in the README.md for this project.
If folks wander to this this project, somehow, we should give them a link to the actual github.io page - but I don't see that anywhere.

I think this has been resolved  - but thanks, Pat! (I was expecting others to more closely monitor the issues and PRs, so didn't see this until now).


"tabbed text" interface mode

The current Racer's interface is not well suited for programmatic interaction:

Outputs are comma/semicolon separated, however in some cases output may contain names of generic Rust types, which may contain commas.  This makes parsing of Racer output ambiguous.
Input commands in deamon mode are whitespace-separated and contain file paths, which is again ambiguous, since paths may contain spaces.

This change introduces a new "tabbed text" interface mode, in which all separators are replaced with tabs (i.e. \t), which will allow unambiguous data parsing in both directions.
This applies to both input and output (however does not affect command line parsing, as that is done by the OS).
For compatibility with existing clients, "tabbed text" mode is activated only when Racer is launched with the --interface=tab-text option.

Thanks @vadimcn, merged.


[Bug]Got "ValueError: not enough values to unpack (expected 2, got 1)" when BatchMatMul using two same tensors

I got a tf graph , imported it into TVM succesfully using api "relay.frontend.from_tensorflow"
Error occurs when :

Error log like this :

The two input tensors of  "batchMatMul" node are the same .both of their shapes are [1, 41, 16].
And i traces the soure code here:

i am deeply confused

Seems that only one placeholder is created. Would you mind sharing the Relay IR?

Seems that only one placeholder is created. Would you mind sharing the Relay IR?

i am sorry that the Relay IR which is generated from a online business model. But it seems u r right. i modified the sample batchmatmul in tvm and i got the same error hints.
The original smaple code is:
x = placeholder(xxxx),
y = placeholder(xxxx),
And what i did is :
x = placeholder(xxxx),
y = x,
However, my model works in online TF env, and what i should do if this case happened when i want to use TVM?
thanks bro!

You can print the prim func here and check if the placeholders are correctly created.
https://github.com/apache/tvm/blob/main/src/relay/backend/te_compiler_cache.cc#L125

On Sep 4, 2021, at 9:48 AM, Noob @.***> wrote:

Seems that only one placeholder is created. Would you mind sharing the Relay IR?
i am sorry that the Relay IR which is generated from a online business model. But it seems u r right. i modified the sample batchmatmul in tvm and i got the same error hints.
The original smaple code is:
x = placeholder(xxxx),
y = placeholder(xxxx),
And what i did is :
x = placeholder(xxxx),
y = x,
However, my model works in online TF env, and what i should do if this case happened when i want to use TVM?
thanks bro!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I don't think we can reuse the placeholder in this way. If your model results in this IR, then it is probably due to the incorrect Relay TF frontend, or unsupported behavior.

You can print the prim func here and check if the placeholders are correctly created. https://github.com/apache/tvm/blob/main/src/relay/backend/te_compiler_cache.cc#L125
…
On Sep 4, 2021, at 9:48 AM, Noob @.***> wrote:  Seems that only one placeholder is created. Would you mind sharing the Relay IR? i am sorry that the Relay IR which is generated from a online business model. But it seems u r right. i modified the sample batchmatmul in tvm and i got the same error hints. The original smaple code is: x = placeholder(xxxx), y = placeholder(xxxx), And what i did is : x = placeholder(xxxx), y = x, However, my model works in online TF env, and what i should do if this case happened when i want to use TVM? thanks bro! — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

My co-worker hacked the tf subgraph, and added a placeholder into the bmm node this morning, Tvm built the Relay IR well :)
Anyway. one thing i am gonna comfirm, if only one placeholder in bmm, i.e a matrix multiply itself, does tvm support this?
Thanks bro!

I checked the below example, seems Relay currently doesn't support this.
import tvm
from tvm import relay
a = relay.Var('a', relay.TensorType((1,8, 16)))
b = relay.nn.batch_matmul(a, a)
f = relay.Function([a], b)
with tvm.target.Target('llvm'):
    relay.build(f)

I checked the below example, seems Relay currently doesn't support this.
import tvm
from tvm import relay
a = relay.Var('a', relay.TensorType((1,8, 16)))
b = relay.nn.batch_matmul(a, a)
f = relay.Function([a], b)
with tvm.target.Target('llvm'):
    relay.build(f)

thx, it really helps


Option to show HealthKit dialog again after selecting "Don't allow"

Could it be possible to enable a re-prompt of the HealthKit permission? Once disallowed, it's unclear how to handle and perhaps show a UI error message to inform the user to open the Health app and enable it in "Sources".

Any time your app requests new permissions, the system displays a form with all the requested data types shown. — Source

You could experiment with asking for new permissions, and see if old ones show up too. I don't think so, as a long time Apple Health user. But maybe the project maintainer knows better :)


Pulling up http get call

Adding _get method to Command class

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 82.927%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
170

Relevant Lines:
205

💛  - Coveralls


how to configuration plain_acl when enable msg trace

If we need msgtrace function and acl function opened together, i need to give every account msgtrace topic sub permission, is there any way to configure the topic whitelist?
如果同时开启消息轨迹和acl鉴权功能, 需要对每个开启消息轨迹的账户都配置对应的内部topic的发布权限, 有没有topic级别的白名单配置呢

@chenyx7 当前还没有topic维度的白名单

Okay, I will implement this enhancement feature.

thx @duhenglucky @zongtanghu

This enhancement feature will be released in the 4.7 version.


Alerts just FLASH and Disappear in Firefox 45.0.1

In Firefox 45.0.1 Alerts flash on then off.....very annoying ;)

Please try adding the following css:
[data-notify="container"] {
  -moz-animation-iteration-count: 1;
  animation-iteration-count: 1;
}


Update Capsule Access Links

Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
[ ] Did you follow the contribution guidelines?

PR-Codex overview
This PR updates the API Key acquisition link for Capsule signers and provides clearer integration steps in the documentation.
Detailed summary

Updated API Key acquisition link to https://usecapsule.com/api
Clarified integration steps in the documentation for Capsule signers

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Hey @nityas, a quick heads up for y'all: we're going to be dropping support for aa-signers going forward with the v4 release of the SDK. This is largely due to it being hard to maintain when we don't have context and it's not a standard practice for most SDKs. We recommend instead publishing an aa package that y'all maintain that uses aa-sdk under the hood. We will be keeping a part of the site where 3rd party signers can link to their docs + packages when we release the v4 package


Mute command issue

I have created a mute command for my discord bot and when I mute the user it dm's the user this layout: https://cdn.discordapp.com/attachments/699880048504733747/737701640928362586/unknown.png
But I want it do this layout but for some reason, it is not working https://cdn.discordapp.com/attachments/699880048504733747/737701923263610930/unknown.png
I was wondering if anyone can help me
https://cdn.discordapp.com/attachments/708825001456828427/737707775941017610/unknown.png

Note unrelated to the issue 😄:
To add an image in markdown, use is syntax:
![Alt Text](url)
For example:
![user message](https://cdn.discordapp.com/attachments/699880048504733747/737701640928362586/unknown.png)

For more details check out Mastering Markdown
from Github Guides

Great tip @alrifay 👍 , thanks for the question @SalihGhauri 🤓
The original message will look like this with formatting...

I have created a mute command for my discord bot and when I mute the user it dm's the user this layout:

But I want it do this layout but for some reason, it is not working

I was wondering if anyone can help me

To get the layout, you need to use embeds https://discordjs.guide/popular-topics/embeds.html
From their documentation...
// at the top of your file
const Discord = require('discord.js');

// inside a command, event listener, etc.
const exampleEmbed = new Discord.MessageEmbed()
	.setColor('#0099ff')
	.setTitle('Some title')
	.setURL('https://discord.js.org/')
	.setAuthor('Some name', 'https://i.imgur.com/wSTFkRM.png', 'https://discord.js.org')
	.setDescription('Some description here')
	.setThumbnail('https://i.imgur.com/wSTFkRM.png')
	.addFields(
		{ name: 'Regular field title', value: 'Some value here' },
		{ name: '\u200B', value: '\u200B' },
		{ name: 'Inline field title', value: 'Some value here', inline: true },
		{ name: 'Inline field title', value: 'Some value here', inline: true },
	)
	.addField('Inline field title', 'Some value here', true)
	.setImage('https://i.imgur.com/wSTFkRM.png')
	.setTimestamp()
	.setFooter('Some footer text here', 'https://i.imgur.com/wSTFkRM.png');

channel.send(exampleEmbed);


My Application

Can I use the ideas of your work to create a free little software? Non-profit. Please.
Thank you for your reading . Best wishes to you .

Please . Can I ... ?

Yeah of course! All my work is public domain, Creative Commons Zero: https://creativecommons.org/publicdomain/zero/1.0/

Thank you . When I finish it , I will upload it to my github.