SetIdempotencyKey does not support multiple goroutines

Describe the bug
The SetIdempotencyKey function on APIClient sets a header that is shared between all goroutines that might share the Adyen client. In a scenario where for example a http Handler will do an Adyen call, this can potentially distort what IdempotencyKey is being used between multiple goroutines.
To Reproduce
https://gist.github.com/aleksen/1ee7edf57a940be8fc1c0c361d3c7126
This code will first output some requests that have either missing or wrong idempotency-key, and then crash with "concurrent map writes", stemming from /src/common/configuration.go:104 where the DefaultHeader is being set.
This gist has been tested with commit 255a560aa36b9868273e13a80c4ef65f043df032.
Expected behavior
Expected behaviour is that the correct idempotency key is sent with the correct request. Perhaps the SetIdempotencyKey function on the client is not the best approach.

+1. Agree that SetIdempotencyKey at client level may not be the right approach.

Agree. As a user of the library I would very much appreciate if the library is thread safe and does not depend on modifying its internal state. I would rather pass the idempotency key explicitly as a parameter or part of a config struct for each request.

Thanks for raising this issue and I agree that this should be fixed, I didn't think about this when I added it to the client. Are any of you comfortable creating a PR with the fix? If not, I'll try to pick this up asap.

I can pick it up on Monday if you don't beat me to it, @KadoBOT

I can pick it up on Monday if you don't beat me to it, @KadoBOT

Great! Thanks @aleksen

@aleksen just released v3 which includes your changes to the idempotency key.


⚠ Bofrost.at has degraded performance

In 76ffc0a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 918 ms

Resolved: Bofrost.at performance has improved in 9ca0250.


Can't add feed without title

I'm trying to add this feed which seemingly doesn't have a title, but I'm getting a 500 Internal Server Error whenever I try.

Logs
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: 2022-12-27 15:13:45 - NoMethodError - undefined method `gsub' for nil:NilClass:
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/stringer/app/utils/content_sanitizer.rb:3:in `sanitize'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/stringer/app/commands/feeds/add_new_feed.rb:12:in `add'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/stringer/app/controllers/feeds_controller.rb:40:in `block in <class:Stringer>'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1675:in `call'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1675:in `block in compile!'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1013:in `block (3 levels) in route!'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1032:in `route_eval'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1013:in `block (2 levels) in route!'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1061:in `block in process_route'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1059:in `catch'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1059:in `process_route'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1011:in `block in route!'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1008:in `each'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1008:in `route!'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1129:in `block in dispatch!'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1101:in `block in invoke'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1101:in `catch'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1101:in `invoke'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1124:in `dispatch!'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:939:in `block in call!'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1101:in `block in invoke'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1101:in `catch'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:1101:in `invoke'
Dec 27 15:13:45 vps stringer-web.1.service[1524212]: #011/home/stringer/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/sinatra-2.1.0/lib/sinatra/base.rb:939:in `call!'
 Is this as easy of a fix as just adding a default / checking for existence of a title?

@guidopetri thanks for reporting! Can you give it another try? I've updated the new feed logic to fall back to using the URL for the feed name.

I don't have the easiest way of testing (I'm transitioning from a VPS to a local server setup with Docker) but I'll definitely try it out when I can. Thanks for the quick fix!

@guidopetri I'm curious, is it important to you to host your own instance? I'm thinking of setting up a service people can subscribe to so they don't have to set up Stringer themselves.

For me? yeah. I really value being in control of my data myself, even for something tiny like this :)
That being said I've seen quite a few services that are similarly able to self-host, but also subscribable for an easy setup (e.g.: Redash used to be that way, as is Monica CRM currently). I think it'd be a good value-add, and an easy way to infuse some money into the repo to fund e.g. bug bounties or new features.

Thanks, that's helpful. At any rate, if you do manage to get things updated on your end, any updates to the docs would be much appreciated.


exclude does not work correctly

Library Affected:
workbox-sw, workbox-build, etc.
Browser & Platform:
all browsers
According to the docs:

This allows you to specifically omit assets matching any of the provided criteria from being included in the precache manifest. It provides a filename-based approach to filtering.

exclude: [
  /main.(?:.*?).css$/gim,
  /\.map$/gim,
  /manifest.*\.js(?:on)?$/gim,
  /icon_.*.png/gim
]

resulting precache manifest:
self.__precacheManifest = [
  {
    "url": "/static/js/main.8acd832abdad30f2360c.js"
  },
  {
    "url": "/static/js/runtime.8acd832abdad30f2360c.js"
  },
  {
    "revision": "545cb59b4526ecbf2e3921d126a68c03",
    "url": "/static/index.html"
  },
  {
    "revision": "472b65ed38ed9e0c7830923d23a5644b",
    "url": "/static/favicon.ico"
  },
  {
    "revision": "3e55c1cea767df1fb708d5fdd28c53b9",
    "url": "/static/favicon.png"
  },
  {
    "revision": "02828d2baf4148969294dc8e18d18958",
    "url": "/static/icons/icon_24x24.png"
  },
  {
    "revision": "2feeb8718a1653ec48136415840610fb",
    "url": "/static/icons/icon_256x256.png"
  },
  {
    "revision": "36025a4911dc4ee6205f40a9fb921148",
    "url": "/static/icons/icon_512x512.png"
  }
];

I was expecting ALL files that has icon_ in the file name to not be included. It managed to remove ONLY SOME but not all.
Here's the catch:
After making changes, webpack updates the precache file resulting to this:
self.__precacheManifest = [
  {
    "url": "/static/js/main.9039b0aceb45eff07d28.js"
  },
  {
    "url": "/static/js/runtime.9039b0aceb45eff07d28.js"
  },
  {
    "revision": "94234b2318acfc82f1306f773bd9c94e",
    "url": "/static/index.html"
  }
];

Now the icons regex worked, BUT notice the favicon png and ico are missing. Their file name does not match the regex, right?

@aprilmintacpineda you say this, but then you don't mention what the changes where, can you elaborate?

Here's the catch:
After making changes, webpack updates the precache file resulting to this:

@jeffposnick any idea why their icon_ regex wouldn't be working here?

Here's the catch:
After making changes, webpack updates the precache file resulting to this:

Changes to any file that's preloaded. I'm running webpack on watch, it detects the main file changed, the precache-file changes as well, the service worker changes as well because the hashes has changed.

How are the /icons/ assets included as part of your webpack build? Are they referenced via a loader, are the included via copy-webpack-plugin, or something else?
If you could share the part of your webpack config that end up pulling in those icon assets, that would be useful.

I use https://github.com/webpack-contrib/copy-webpack-plugin to copy the whole public/ directory to build/.

And is copy-webpack-plugin listed before or after workbox-webpack-plugin? Could you share the plugins section of your webpack config?

I believe this is a duplicate of #1782.


the empty string should be used in delegation Paths to indicate a role can sign everything

Signed-off-by: David Lawrence<EMAIL_ADDRESS>(github: endophage)

LGTM pending CI

Add a test, and we're good to go. ;)
LGTM


Auto clear the offline cache if it exceeds a certain treshold

I ran into an issue where local storage was ... filled. And when I inspected, I saw that logdna was taking most of it. It happened when I was doing a massive refactoring and there were tons of warnings along the way.

I deleted it manually and I'm good to go now since that only happened in my local environment and also disabled logdna in local.
But I'm worried this can happen overtime in production if there is a surge of errors or warnings for any reason.

@haikyuu Thank you for reporting this.  I have seen this before and there is an open PR to remove the offline storage from the client due to this potential issue and due to the way our ingestors handle older timestamps.  I am hoping that this gets merged shortly and I will update you when it does.


Wrong error message when include file is missing (on Windows)

Error message when an included file doesn't exist is wrong - actually is being replaced by another raised exception, therefore covering the original message.
The error that is being printed is the following:
ERROR   -  Error reading page 'xyz.md': path is on mount 'W:', start on mount 'C:'

instead of telling of file not existing.
By delving into the code, I saw that the script correctly realizes that the file doesn't exist, but the issue is when the exception is raised.
I'm talking about the block at event.py:136:
        if not file_paths_to_include:
            lineno = lineno_from_content_start(
                markdown,
                directive_match_start,
            )
            raise PluginError(
                f"No files found including '{raw_filename}'"
-->             f' at {os.path.relpath(page_src_path, docs_dir)}'
                f':{lineno}',
            )

On Windows, the call os.path.relpath(page_src_path, docs_dir) fails and raises another exception when the two files are on different drives, because there is no relative path between them.
As a test I commented the line and the error message is what I would expect.

What is the error message if you remove the os.path.relpath and leaves just the variable page_src_path?

ERROR   -  Error reading page 'page.md':
ERROR   -  No files found including './non_existent_file.md' at W:\path\to\page.md:20

I guess the right message.

Fixed on v6.0.7.


SkeletonModification2DTwoBoneIK throws errors and warnings when used

Godot version
4.0.stable
System information
Windows 10, Vulkan, NVIDIA GeForce GTX 1070
Issue description
When using a SkeletonModification2DTwoBoneIK in a modification stack for a Skeleton2D, some errors and warnings are thrown to the console.

Steps to reproduce
Just execute the project below to see errors and warnings.
Minimal reproduction project
skeleton_ik_test.zip

Duplicate of https://github.com/godotengine/godot/issues/73247 (same cause).


Support for IRC colors

As an exmaple, some IRC channels contain bridge bots that relay messages from other networks to IRC. They prefix the messages with the nicks in color to make it easier to distinguish the users.
The color information is currently lost by heisenbridge.
It would be nice if this could be translated to equivalent coloring in matrix messages.

This only needs a proper IRC color escape parser. The current one strips away colors, poorly.
There were some efforts at doing this in the Matrix room but no one finished the work to a PR.

I wrote an IRC color parser and converter in c++ to figure out parsing, but never got to finishing the python port. note that this is only for IRC->matrix and not the other way around, which is much harder

#225 fixed this.


Vagrant up fails with missing libssl1.1 package.

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 1.8.6
VirtualBox 5.1.22r115126
-bash: ansible: command not found
(Can't get ansible installed, but that has never been an issue before..)
Your OS

macOS (10.11.6)

Full console output
https://gist.github.com/jiisuominen/047065d0a0f497b5fdbba1344a222b2b
Summary
Just trying to create new VM with latest master branch and updated VirtualBox and the rest. Every time exits with same status, fails with missing libssl1.1. I tried installing that manually, but it seems to be missing from the repo altogether.
So I'm out of options / skills here, can't get any VM's created because of this missing libssl issue.

@jiisuominen can you try updating the base box vagrant box update, if you have the latest installed please post your config.yml

Yeah, I've run vagrant box update, current version being 1.1.4.
Here's my config.yml: https://gist.github.com/jiisuominen/41def13c38595f23df19920abb7a21e8

Should be fixed as of 2h ago https://github.com/oerdnj/deb.sury.org/issues/639

I had the same issue this morning but it's now fixed.

Great, I'll close this as it was fixed upstream.


[NeoVim coc-clangd] Too many errors omitted + linter isnt working right (read for more)

This only just started happening recently. But whenever I start editing my C++ programs, the linter says
Too many errors emitted, stopping now
[clang: fatal_too_many_errors]

at the top. Strangely enough, ALL of the errors produced are produced in lines that have std::string
the message reads as follows:
Use of undeclared identifier 'std'
[clang: undeclared_var_use]

It is insanely annoying that all the strings I work with produce an error for clangd. Of course its fine when I actually run it, but damn is this annoying and in order for me to tell if a new error appeared, I have to either scroll through my code or go to all my std::string usages and subtract them from the count.
I tried reinstalling coc-clangd via :CocInstall coc-clangd and it didnt fix it. So then I tried reinstalling LLVM which reinstalled clangd too and that didnt work.

Also i forgot to add,
#include <iostream> produces an error too.
'iostream' file not found
[clang: pp_file_not_found]

Do you have a compile_commands.json for your project?
The errors were came from clangd, coc-clangd just display them.


metrics service

exposing Metrics data by metrics service

Hi, could you please resolve the conflict first?

Codecov Report

Merging #3833 into master will increase coverage by 0.07%.
The diff coverage is 77.04%.

@@             Coverage Diff              @@
##             master    #3833      +/-   ##
============================================
+ Coverage     63.57%   63.64%   +0.07%
  Complexity       71       71
============================================
  Files           705      709       +4
  Lines         31128    31273     +145
  Branches       5048     5057       +9
============================================
+ Hits          19789    19904     +115
- Misses         9063     9081      +18
- Partials       2276     2288      +12

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/apache/dubbo/common/Constants.java
93.33% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../java/org/apache/dubbo/config/ReferenceConfig.java
61.23% <100%> (+0.14%)
0 <0> (ø)
:arrow_down:

...in/java/org/apache/dubbo/config/ServiceConfig.java
56.58% <100%> (+0.09%)
0 <0> (ø)
:arrow_down:

...bo/config/spring/schema/DubboNamespaceHandler.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...in/java/org/apache/dubbo/config/MetricsConfig.java
18.18% <18.18%> (ø)
0 <0> (?)

.../org/apache/dubbo/monitor/dubbo/MetricsFilter.java
82.4% <82.4%> (ø)
0 <0> (?)

.../apache/dubbo/qos/protocol/QosProtocolWrapper.java
69.23% <0%> (-12.83%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0%> (-4.35%)
0% <0%> (ø)

...dubbo/rpc/protocol/dubbo/CallbackServiceCodec.java
80.14% <0%> (-0.74%)
0% <0%> (ø)

... and 8 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 72020df...5ac3264. Read the comment docs.

@nzomkxia Would you please help to review the Metrics collecting part?

@nzomkxia Would you please help to review the Metrics collecting part?

metrics collecting part looks good. but  how to pass the metrics protocol and port to dubbo admin? they are not in the registry center under simplified URL mode

the metrics stuff can be stored in metadata center

the metrics stuff can be stored in metadata center

Seems to be the only way at present.


leapp_upgrade_type is set to satellite skip

there has been an instance where if you over write the default of leapp_upgrade_type to satellite (the default) with extra vars it skips the Register to leapp activation key task. with satellite_organization and satellite_activation_key_leapp defined.

wrong code version so satellite wasn't the correct values


Build failure: gotop

Steps To Reproduce
Steps to reproduce the behavior:

nix-env -i gotop

Build log
o: downloading gopkg.in/yaml.v2 v2.4.0
go: downloading github.com/mitchellh/go-homedir v1.1.0
# github.com/shirou/gopsutil/cpu
../go/pkg/mod/github.com/shirou/gopsutil@v3.20.12+incompatible/cpu/cpu_darwin_cgo.go:13:5: error: 'TARGET_OS_MAC' is not defined, evaluates to 0 [-Werror,-Wundef-prefix=TARGET_OS_]
#if TARGET_OS_MAC
    ^
1 error generated.
error: builder for '/nix/store/8iqqlbzyk6mflmmzh8cc89h4aynb7r6g-gotop-4.2.0.drv' failed with exit code 1;
       last 10 log lines:
       > go: downloading github.com/pkg/errors v0.9.1
       > go: downloading github.com/ghodss/yaml v1.0.0
       > go: downloading github.com/jaypipes/pcidb v1.0.0
       > go: downloading gopkg.in/yaml.v2 v2.4.0
       > go: downloading github.com/mitchellh/go-homedir v1.1.0
       > # github.com/shirou/gopsutil/cpu
       > ../go/pkg/mod/github.com/shirou/gopsutil@v3.20.12+incompatible/cpu/cpu_darwin_cgo.go:13:5: error: 'TARGET_OS_MAC' is not defined, evaluates to 0 [-Werror,-Wundef-prefix=TARGET_OS_]
       > #if TARGET_OS_MAC
       >     ^
       > 1 error generated.
       For full logs, run 'nix log /nix/store/8iqqlbzyk6mflmmzh8cc89h4aynb7r6g-gotop-4.2.0.drv'.

Additional context
Happy to provide more info if needed.
Notify maintainers
@zowoq

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
- system: `"x86_64-darwin"`
 - host os: `Darwin 22.3.0, macOS 10.16`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.13.2`
 - channels(<user_redacted>): `"nixpkgs, nixpkgs-22.11-darwin-22.11-darwin"`
 - channels(root): `""`
 - nixpkgs: `/Users/<user_redacted>/.nix-defexpr/channels/nixpkgs`

Maybe we can apply https://github.com/xxxserxxx/gotop/pull/234 as a patch.

@aldafu can you try latest master ? gotop 4.2.0 should aleady contain patch mentioned by @dotlambda
https://github.com/xxxserxxx/gotop/releases/tag/v4.2.0


Problem with Existential?

Unsure about why this is failing (Property_Valid_4):
property contains(int[] xs, int x)
where some { i in 0..|xs| | xs[i] == x }

function id(int[] xs, int x) -> (int[] ys)
requires contains(xs,x):
    return xs

public export method test():
    int[] ys = id([0],0)

It can't seem to decide that contains([0],0) holds.

This is the minimal test case that fails then:
property contains(int[] xs, int x)
where some { i in 0..|xs| | xs[i] == x }

public export method test():
    assert contains([0],0)

Should try to confirm this in Dafny to see whether it;s really a bug or not.

See also this example from #54:
type arr is ({int f}[] xs) where some { i in 0..|xs| | xs[i].f == 1 }

public export method test() :
    arr x = [{f: 1}]


Fixed Broken Link in Readme.md

Currently in the section- Develop with BoCoEL

API reference can be found here.

Results in a broken link -

Fixed the link and now redirects correctly -

Hey @rentruewang, please review the PR

@Shreyas0410 Thank you. Good catch.


Make it easier to clear the search

[ ] Add "clear search" button
[ ] Save searches in the hash so that the back button works

👍
For clear search we can use the 'x', is that what you were thinking or something more like 'Clear' or 'Reset' links?


Bug Text widget when set foreground for stroke text on Platform IOS

Is there an existing issue for this?

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

Steps to reproduce
1.Use Text widget with foreground to stroke text on Platform IOS

Note: Platform Android its okay but not with IOS

Expected results
1.Show stroke on text
Actual results
1.No stroke on text
Code sample
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 MaterialApp(
      home: Scaffold(
        body: Container(
          color: Colors.blue[200],
          child: Center(
            child: Stack(
              children: [
                Text(
                  'Hello World!',
                  textAlign: TextAlign.center,
                  style: TextStyle(
                    fontFamily: 'ToySans',
                    fontSize: 36,
                    foreground: Paint()
                      ..style = PaintingStyle.stroke
                      ..color = Colors.white
                      ..strokeWidth = 10
                      ..strokeMiterLimit = 1,
                  ),
                ),
                const Text(
                  'Hello World!',
                  style: TextStyle(
                    fontFamily: 'ToySans',
                    fontSize: 36,
                    color: Colors.brown,
                  ),
                ),
              ],
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
![Simulator Screen Shot - iPhone 11 Pro - 2023-05-17 at 15 41 21](https://github.com/flutter/flutter/assets/32564261/bbe7e9b8-5096-4f7e-beba-a6421e71b3ba)

Logs
Logs
No log error!

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.0, on macOS 13.2 22D49 darwin-arm64, locale
    en-VN)
    • Flutter version 3.10.0 on channel stable at
      /Users/mac/Desktop/Flutter/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 84a1e904f4 (8 days ago), 2023-05-09 07:41:44 -0700
    • Engine revision d44b5a94c9
    • Dart version 3.0.0
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/mac/Library/Android/sdk
    • Platform android-33, build-tools 33.0.2
    • ANDROID_HOME = /Users/mac/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.12.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.78.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] Connected device (3 available)
    • iPhone 11 Pro (mobile) • 654CEC57-0ED1-4E6D-BD84-D5FE4DFB6980 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-2 (simulator)
    • macOS (desktop)        • macos                                •
      darwin-arm64   • macOS 13.2 22D49 darwin-arm64
    • Chrome (web)           • chrome                               •
      web-javascript • Google Chrome 113.0.5672.92

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Thanks for the report! This is a known issue being tracked at https://github.com/flutter/flutter/issues/126010.
Closing this issue as a duplicate, so please follow up on it for further updates.


Error in Demo Z2UI5_CL_DEMO_APP_090

Hi gens,
first thanks to all of you managing this fantastic framework.
My current problem concerns the demo popup of Z2UI5_CL_DEMO_APP_090. I want to make use of the "Open P13N Dialog"  :

When I use it only once it works fine. The popup appears and I can change values on it. But when I press the open button once again in the same session, I get an internal server error :

It complains that I am using two different mapper.
I guess it belong to this line in the code:

custom_mapper = z2ui5_cl_ajson_mapping=>create_lower_case( )
Can anybody help in understanding the problem. I thought the popup gets destroyed after closing and is not reusing anything.
Thanks in advance.
Oliver

thanks for finding this, there war a bug, should be fixed now with:
https://github.com/abap2UI5/abap2UI5/pull/1265

Thanks for for the correction. The demo runs fine again.
Oliver


⚠ Bofrost.be has degraded performance

In f7554ed, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1084 ms

Resolved: Bofrost.be performance has improved in ce387b2 after 29 minutes.


feat: cla setup

The goal of this pull request is to verify that the CLA workflow works correctly.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[SPARK-24185][SparkR][SQL]add flatten function to SparkR

What changes were proposed in this pull request?
add array flatten function to SparkR
How was this patch tested?
Unit tests were added in R/pkg/tests/fulltests/test_sparkSQL.R

Test build #90238 has started for PR 21244 at commit bfccf18.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2941/
Test PASSed.

Merged build finished. Test PASSed.

Test build #90238 has finished for PR 21244 at commit bfccf18.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90238/
Test PASSed.

Please also add [SQL][SparkR] in the PR title.

@huaxingao Isn't the correct Jira number SPARK-24185?

Test build #90261 has started for PR 21244 at commit fe769df.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2954/
Test PASSed.

cc @HyukjinKwon

Test build #90261 has finished for PR 21244 at commit fe769df.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90261/
Test PASSed.

Merged to master

@viirya @mn-mikke @felixcheung @HyukjinKwon Thanks all for your help!
@HyukjinKwon I will fix the two small things in my next PR.


sql,parser: Add experimental database check SQL statement

PR: https://github.com/cockroachdb/cockroach/pull/19161

Covered by https://github.com/cockroachdb/docs/issues/2043.


postgresql11Packages.pgrouting: 3.0.0 -> 3.0.1

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/pgrouting/versions.
meta.description for postgresql11Packages.pgrouting is: "A PostgreSQL/PostGIS extension that provides geospatial routing functionality"
meta.homepage for postgresql11Packages.pgrouting is: "https://pgrouting.org/"
meta.changelog for postgresql11Packages.pgrouting is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.0.1 with grep in /nix/store/i915fcwgfv7779yv6lmy82ywlyqms5y9-pgrouting-3.0.1

directory tree listing: https://gist.github.com/8a766da5c543b9bd7276a17041b6294d

du listing: https://gist.github.com/6d1d07a7fd7a1efd566773e3dbf5ec46

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
postgresql11Packages.pgrouting
postgresqlPackages.pgrouting

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i915fcwgfv7779yv6lmy82ywlyqms5y9-pgrouting-3.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A postgresql11Packages.pgrouting https://github.com/r-ryantm/nixpkgs/archive/14414670153913edfbb8ed041dc357265c983a78.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i915fcwgfv7779yv6lmy82ywlyqms5y9-pgrouting-3.0.1
ls -la /nix/store/i915fcwgfv7779yv6lmy82ywlyqms5y9-pgrouting-3.0.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- postgresql11Packages.pgrouting

Maintainer pings
cc @steve-chavez for testing.

@r-ryantm LGTM! The new pgrouting version compiles and installs successfully.


feat(injectManifest)!: optimize workbox load

Description

This PR includes:

use readFileSync  to load version
add ./package.json to package exports
update debug  to ^4.3.6
load workbox when using injectManifest strategy only when there is injection point

Linked Issues

closes #749
Additional Context

[!TIP]
The author of this PR can publish a preview release by commenting /publish below.

/publish


waiter.py: Error in wait(): raise WaiterError - RedshiftDataApi DescribeStatement FAILED status causes AttributeError

I am not sure whether this belongs here or if an update to botocore is preferable to pass the expected output to waiter.py but here goes:
I am calling RedshiftDataApi in a Python3.8 Lambda and everything works fine until my statement hits the FAILED state (presumably ABORTED would cause this issue as well in my code). I am defining a custom_waiter as outlined in this article. I am submitting queries to RedshiftDataApi via execute_statement() then using a custom_waiter to poll describe_statement() until finished state is reached. I am using Status [FAILED, ABORTED] to go to the failure state but when FAILED is reached RedshiftDataApi's describe_statement() returns a dict() with the item "Error" as expected by waiter.py but the Error (as defined in the docs) returns a String value instead of the dict() value for "Error" expected by waiter.py. Currently I have coded around this with a tertiary except clause to catch the AttributeError being raised by the raiseWaterError() code block when it attempts to do: response['Error'].get('Message', 'Unknown') but it sure would be neater to get the intended WaiterError.
Steps to reproduce
I cannot include my full working example as it is proprietary but here is my waiter config:
def get_custom_waiter(self):         delay = 1         max_attempts = 15         waiter_model = WaiterModel({             'version': 2,             'waiters': {                 'DataAPIExecution': {                     'operation': 'DescribeStatement',                     'delay': delay,                     'maxAttempts': max_attempts,                     'acceptors': [                         {                             "matcher": "path",                             "expected": "FINISHED",                             "argument": "Status",                             "state": "success"                         },                         {                             "matcher": "pathAny",                             "expected": ["PICKED", "STARTED", "SUBMITTED"],                             "argument": "Status",                             "state": "retry"                         },                         {                             "matcher": "pathAny",                             "expected": ["FAILED", "ABORTED"],                             "argument": "Status",                             "state": "failure"                         }                     ],                 },             },         })         return create_waiter_with_client(waiter_name="DataAPIExecution", waiter_model=waiter_model, client=self._redshift_data_client)
Using this waiter when I:

Submit a SQL query with a syntax error
Use the waiter to wait for the results of that query OR attempt to access data on a schema my IAM role is not authorized for
The Error in the debug logs below is encountered. As a workaround I have to implement a manual loop to call describe_statement and log out the string result of describe_statement()['Error'] once FAILED state is encountered.

Expected behavior
WaiterError should be raised instead of the AttributeError being raised in the raise WaiterError block.
Again, either string handling could be added here in waiter.py or in the RedshiftDataApi client code the error type needs to be changed to return 'dict()' as expected here instead of 'str'; but that seems like a bigger and more breaking change.
Instead the more useful error from the RedshiftDataApi client should be surfaced to assist in debugging.
Debug logs
Full stack trace by adding
[ERROR]	2021-02-24T21:21:22.525Z	d79b5d8c-af76-4f63-af89-b27390bf67a1	'str' object has no attribute 'get'
Traceback (most recent call last):
  File "/opt/python/aca/customerreportsclients/redshift_history_client.py", line 62, in get_query_results
    self._custom_waiter.wait(Id=query_id)
  File "/var/runtime/botocore/waiter.py", line 53, in wait
    Waiter.wait(self, **kwargs)
  File "/var/runtime/botocore/waiter.py", line 312, in wait
    reason=response['Error'].get('Message', 'Unknown'),

We also have this problem and it is very annoying.

Hi @adam-kellam-ACA, we’re looking into this. Can you confirm what version you’re seeing this with? Thanks!

@kdaily sure, it's a Python 3.8 Lambda I am getting this error deployed in our dev environment in us-east-1; here is what I get from the following code snippet in a test lambda:
from pip._internal.utils.misc import get_installed_distributions
Code:
from pip._internal.utils.misc import get_installed_distributions
def lambda_handler(event, context):
print_log = ''
for module in sorted(get_installed_distributions(), key=lambda x: x.key):
print_log +=  module.key + '~=' + module.version  + '\n'
print(print_log)
Result:
Function Logs
START RequestId: 90dbf718-095b-4013-aff6-5955fc6d45b6 Version: $LATEST
boto3~=1.16.31
botocore~=1.19.31
jmespath~=0.10.0
pip~=20.2.3
python-dateutil~=2.8.1
rapid-client~=0.0.0
s3transfer~=0.3.3
setuptools~=49.2.1
six~=1.15.0
urllib3~=1.26.2

@adam-kellam-ACA . We are looking into resolving this issue.

Hi @adam-kellam-ACA,
There is a PR to address this:
https://github.com/boto/botocore/pull/2314

Cool! OK so not sure whether to put myself out there and put this on the review for that PR or comment here; looking over the PR this appears to resolve the issue of getting an unexpected error from my custom_waiter, but I do not believe it will surface the underlying Error string being presented on the RedshiftDataAPI response; instead it appears I will get a waiter_error with the reason being "terminal failure state encountered". Once I get that error I would then have to finagle my code again to manually loop and poll RedshiftDataAPI for the response of describe_statement until it fails to see whether my issue was SQL syntax, permissions issue etc.
Is this because the onus of presenting a valid Error object should lie on the developers of the RedshiftDataAPI and not on waiter.py as the behavior here in waiter.py is absolutely correct for any valid Error response?

@adam-kellam-ACA thank you for following up. So we have actually found a potential discrepancy in the custom waiter config. We believe this is why you will get a WaiterError with the reason being "terminal failure state encountered". We are working with them to get this fixed. We recommend custom polling logic for this case, which seems like you have already implemented. The existence of the Error key in the response does not by default mean that an Error object is due, in some cases the key exists and the API call is successful.

@zdutta Makes sense! At any rate for our use case we ended up switching from the Data API to direct queries against Redshift as the performance on the Data API was substantially slower (3-600%) and our volume is such that we need the faster turnaround time.
Good to know in future other developers working with Data API will have the full response surfaced so they can read the Error string via custom_waiter instead of having to write their own polling loop. Thanks all!

Closing, as fix has been merged.  Let us know if you have any questions!


fix clients to use https://bsd.ac

Just "bsd.ac" doesn't work anymore.
Signed-off-by: Érico Nogueira<EMAIL_ADDRESS>Closes #6
That said, was it expected that HTTP would stop working completely?

That said, was it expected that HTTP would stop working completely?

Yes, given that there is no plain http listener anymore.
Thanks for the PR!


Added my name to the list

Added Enosejolagbon to the list of names

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.

Hello Enosejolagbon!
Please open a new PR! Remember to make a branch as you work on your changes! 🙏
We look forward to merging your next PR! :partying_face:


Mod Settings do not have a scroll bar

If a mod has a lot of settings, no menu scrollbar is present.

This is fixed


Two-Character Unicode Support

Is your feature request related to a problem? Please describe.
I'm almost certain one cannot render two-character unicode symbols using any tags supported in MathJax.  Some sample codes are those found for national flags, here.  In every case where the double-character is rendered, it is rendered as two separate elements.  E.g.,

Describe the solution you'd like
It would be great if \verb, or \unicode or some other TeX tag could render the double character.  E.g., 🇦🇩, or U+1F1E6 U+1F1E9.
Describe alternatives you've considered
I am building a browser based formula editor based on MathJax, and it uses TeX as a basis for constructing elements.  I had hoped there was a TeX command that might work, but after two days of reading and experimentation, I am unable to find anything.
Additional context
The output format used is SVG, and the TeX is converted to MathMl using MathJax.
Thank you!

I think https://github.com/mathjax/MathJax/issues/2595 and https://github.com/mathjax/MathJax-src/pull/676 should help.

Thank you very much!  Most of the discussion was over my head, but if I got the gist I can now use \symbol{🇦🇩, }, and that this capability is in a new version that is not part of a CDN distribution.  Do I have that right?  If so, would I need to follow the instructions in the manual for "Hosting Your Own Copy", using the version in the pull request?

if I got the gist I can now use \symbol{🇦🇩, }, and that this capability is in a new version that is not part of a CDN distribution.

Not quite.  There is no \symbol macro, but there are ones for the different variant forms, like \symbf for bold, \symsfit for sans-serif-italic, and so on.  The reason Peter pointed to this is that these macros also will group multiple characters into a single MathML element, whereas each character normally is put into its own MathML element.  So ab is usually translated as <mi>a</mi><mi>b</mi>, but \symbf{ab} would produce <mi mathvariant="bold">ab</mi> with both letters in the same <mi>.
Unfortunately, that does not solve the problem.  The multi-character flags seem to be handled as ligatures, which means the two character have to be next to each other in the same DOM node in the final output.  When MathJax encounters a character that is not in its math fonts (like these), it places them in separate <mjx-utext> elements (for CHTML output) or <text> elements (for SVG output), and so they are not next to each other in the DOM, and are not elided.
Fortunately, there is a way to get what you want.  That is to use \text{🇦🇩} and to configure MathJax to use either the surrounding font for text elements (via the mtextInheritFont: true property), or to use a specific font for text elements (via mtextFont: 'Times' for example).  When one of these properties is set, MathJax will render <mtext> MathML elements as a single DOM node, and that will allow the two characters to be combined by the browser.  So
MathJax = {
  chtml: {
    mtextInheritFont: true
  }
}

together with \text{🇦🇩} in the TeX input should provide the result you are looking for in CommonHTML output (change chtml to svg if you are using SVG output).
The current behavior of putting each unknown character into its own DOM element could probably be improved to combine them into a single one, in which case Peter's suggestion would have worked.  But that's not the case right now.
As for v3.1.3, it will be release either later today or tomorrow, so no need to install your own.

Thanks to both of you!
Sent from my iPhone
On Apr 22, 2021, at 12:24 PM, Davide P. Cervone @.***> wrote:

if I got the gist I can now use \symbol{🇦🇩, }, and that this capability is in a new version that is not part of a CDN distribution.
Not quite. There is no \symbol macro, but there are ones for the different variant forms, like \symbf for bold, \symsfit for sans-serif-italic, and so on. The reason Peter pointed to this is that these macros also will group multiple characters into a single MathML element, whereas each character normally is put into its own MathML element. So ab is usually translated as ab, but \symbf{ab} would produce ab with both letters in the same .
Unfortunately, that does not solve the problem. The multi-character flags seem to be handled as ligatures, which means the two character have to be next to each other in the same DOM node in the final output. When MathJax encounters a character that is not in its math fonts (like these), it places them in separate  elements (for CHTML output) or  elements (for SVG output), and so they are not next to each other in the DOM, and are not elided.
Fortunately, there is a way to get what you want. That is to use \text{🇦🇩} and to configure MathJax to use either the surrounding font for text elements (via the mtextInheritFont: true property), or to use a specific font for text elements (via mtextFont: 'Times' for example). When one of these properties is set, MathJax will render  MathML elements as a single DOM node, and that will allow the two characters to be combined by the browser. So
MathJax = {
chtml: {
mtextInheritFont: true
}
}
together with \text{🇦🇩} in the TeX input should provide the result you are looking for in CommonHTML output (change chtml to svg if you are using SVG output).
The current behavior of putting each unknown character into its own DOM element could probably be improved to combine them into a single one, in which case Peter's suggestion would have worked. But that's not the case right now.
As for v3.1.3, it will be release either later today or tomorrow, so no need to install your own.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/mathjax/MathJax/issues/2672#issuecomment-824988397, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAMKXO5CBAI6GGNWHLMG2CLTKBEJHANCNFSM43LE2OUA.

That worked great for MathMl, but unfortunately not for saving as SVG.  (See screenshot, below).
This is my MathJax setup:
MathJax = {
chtml: {
mtextFont: 'arial unicode ms',
// mtextInheritFont: true,
},
svg: {
mtextFont: 'arial unicode ms',
// mtextInheritFont: true,
},
loader: {
load: [
'input/tex-full',
'input/mml',
'output/svg',
'ui/menu',
'ui/safe',
'a11y/semantic-enrich',
'a11y/complexity',
'a11y/explorer',
'a11y/assistive-mml',
'[tex]/ams',
'[tex]/braket',
'[tex]/boldsymbol',
'[tex]/require',
'[tex]/html',
'[tex]/unicode',
'[tex]/verb'
]
},
};

On Apr 22, 2021, at 1:13 PM, David Quinn-Jacobs @.***> wrote:
Thanks to both of you!
Sent from my iPhone
On Apr 22, 2021, at 12:24 PM, Davide P. Cervone @.***> wrote:

if I got the gist I can now use \symbol{🇦🇩, }, and that this capability is in a new version that is not part of a CDN distribution.
Not quite. There is no \symbol macro, but there are ones for the different variant forms, like \symbf for bold, \symsfit for sans-serif-italic, and so on. The reason Peter pointed to this is that these macros also will group multiple characters into a single MathML element, whereas each character normally is put into its own MathML element. So ab is usually translated as ab, but \symbf{ab} would produce ab with both letters in the same .
Unfortunately, that does not solve the problem. The multi-character flags seem to be handled as ligatures, which means the two character have to be next to each other in the same DOM node in the final output. When MathJax encounters a character that is not in its math fonts (like these), it places them in separate  elements (for CHTML output) or  elements (for SVG output), and so they are not next to each other in the DOM, and are not elided.
Fortunately, there is a way to get what you want. That is to use \text{🇦🇩} and to configure MathJax to use either the surrounding font for text elements (via the mtextInheritFont: true property), or to use a specific font for text elements (via mtextFont: 'Times' for example). When one of these properties is set, MathJax will render  MathML elements as a single DOM node, and that will allow the two characters to be combined by the browser. So
MathJax = {
chtml: {
mtextInheritFont: true
}
}
together with \text{🇦🇩} in the TeX input should provide the result you are looking for in CommonHTML output (change chtml to svg if you are using SVG output).
The current behavior of putting each unknown character into its own DOM element could probably be improved to combine them into a single one, in which case Peter's suggestion would have worked. But that's not the case right now.
As for v3.1.3, it will be release either later today or tomorrow, so no need to install your own.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/mathjax/MathJax/issues/2672#issuecomment-824988397, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAMKXO5CBAI6GGNWHLMG2CLTKBEJHANCNFSM43LE2OUA.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/mathjax/MathJax/issues/2672#issuecomment-825036190, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS2JKH4NKQYBGVLQY5CYHMDTKBKLVANCNFSM43LE2OUA.

Oops!  My bad.  Setting mtextInheritFont: true worked for SVG as well.  Please consider this closed.

Your original configuration with mtextFont also works for me.
BTW, there was not attached screen image.  Also, you don't need to load the [tex]/... packages explicitly because they are already included in input/tex-full.
I'm going to leave this open for now, to remind me to look into whether combining adjacent unknown characters into the same container element would be feasible.

Thank you, @dpvc.  My image was in an email reply, so it probably didn't make it.  Here it is in github:


Dynatrace alert is not being captured in keptn

Hi team,
Facing some issues related to dynatrace alert is not triggered to keptn. Created new project and checked as well, still there is an issue in keptn. I have also raised it in slack help channel. Please check and let us the issue to get resolve in keptn asap.

This was fixed over our slack channel.


Add support for google analytics v4

The Google analytics template doesn't work with Google Analytics v4 (gtag.js) #7954. v3 analytics.js is not depreciated so we should support both using hasPrefix to tell the difference.
It should be noted that gtag uses the async attribute. Which is more like google_analytics_async.html. I'm not sure if we should move the changes to that template instead. The naming of the template is also confusing. See #8261.
Privacy settings anonymizeIP and respectDoNotTrack are supported while useSessionStorage is not. I could only find one document with instructions for session storage but in my testing it still set cookies.

This looks like a good PR! I haven't built/tested it, but glancing over the code it works how I would have expected it to. We'll have to update the Docs too.
We'll have to mention that if a user has a GA4 property ID then they should use the "_internal/google_analytics.html" template as the "_internal/google_analytics_async.html" won't render the new gtag template you added.

OK, @djatwood could you add a couple of lines to the relevant docs section below /docs in this repo?

I'm happy to update the docs. I could also add gtag support to google_analytics_async.html I just wasn't sure. What do you think about that?

I could also add gtag support to google_analytics_async.html I just wasn't sure. What do you think about that?

No, let's drop that, that creates too much stuff to maintain.
I suggest that we mention that in the docs is that "for v4, use that other template".

Alright I believe everything should be ready for review now.

@djatwood, can you help me? I'm trying to use ga v4. So I'm setting the config .toml file as the docs, the baseof.html  it's using the {{ template "_internal/google_analytics.html" . }} but still not tracking. May I miss something because I don't realize how to change this variable doNotTrack , when I render the site:

var doNotTrack = false;
if (!doNotTrack) {
	window.dataLayer = window.dataLayer || [];
	function gtag(){dataLayer.push(arguments);}
	gtag('js', new Date());
	gtag('config', 'G-xxxxxxxxxx', { 'anonymize_ip': false });
}

It's always false and doesn't execute the script.
Sorry if I sound stupid, but I'm a noobie in all that stuff.

I don't know if I have enough information. doNotTrack needs to be false for tracking to run. It's a double negative. Do you have any errors in your console? Are you using any browser other than chrome? Safari and Firefox will block GA

@natalianunez I'm having the same issue as you. Everything looks fine, and the request goes through with Status: 200 in the browser. However, still nothing is getting sent to google analytics. Is anybody else still having an issue with this? I raised the issue on discourse.hugo.io as well.


Windows Authentication Display a Temporary 500 Error Page

Issue / Steps to reproduce the problem

Enable Windows AUthentication using this link using IIS integration and Krestrel.
http://docs.identityserver.io/en/dev/topics/windows.html

When logging in, using Windows Authentication, a temporary 500 error page is displayed which then automatically gets refreshed and then logs the user in.
this issue is also mentioned in the following ticket logged by another user.
https://github.com/IdentityServer/IdentityServer4/issues/650

Can you share a screen shot of the error page?

<EMAIL_ADDRESS>From: Brock Allen<EMAIL_ADDRESS>Sent: Monday, March 6, 2017 11:50 AM
To: IdentityServer/IdentityServer4<EMAIL_ADDRESS>Cc: Vicky Malhar<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [IdentityServer/IdentityServer4] Windows Authentication Display a Temporary 500 Error Page (#879)
Can you share a screen shot of the error page?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_IdentityServer_IdentityServer4_issues_879-23issuecomment-2D284457035&d=DwMCaQ&c=ItdvPXPJc-zW5sqOZ-7AtXIt0TnvciGmwfBte4XFNFM&r=YjFV-Uuj5frMz78y99GWlpjNlKvwiQeqvBrs5JrsZak&m=YJRiNlJ7G99vVpFYijgrRf4hqCvEZ3CkWmFFTweDtgk&s=UC3uoNkW07KkJN2_5VPM_wp_PlaYQ9AMrt9-szLMsa0&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGoZGzmWjBfdOW4UPKrUIDKSwjPXx7uks5rjDk6gaJpZM4MUS-2De&d=DwMCaQ&c=ItdvPXPJc-zW5sqOZ-7AtXIt0TnvciGmwfBte4XFNFM&r=YjFV-Uuj5frMz78y99GWlpjNlKvwiQeqvBrs5JrsZak&m=YJRiNlJ7G99vVpFYijgrRf4hqCvEZ3CkWmFFTweDtgk&s=BQix8_NWHOvvmVhWkMRM3kzQjdTxEVqR-O9SbhbXRrA&e=.

This message is intended only for the designated recipient(s). It may contain confidential, privileged or proprietary information. If you are not a designated recipient,you may not review, copy or distribute this message. If you receive this message in error, please notify the sender by reply email and delete this message.
WARNING: All email sent to or from the First Reserve corporate email system is subject to archiving, monitoring and/or review by First Reserve personnel.

Yes, I've seen this same issue. If you notice other browsers don't show this, so I feel like it's more of a chrome problem.

Any update on the issue? closing for now - feel free to re-open if it needs further discussion.

This was recently re-tested on the Chrome v59 and was no longer the behavior.


Do not display dApps that have been added to predefined lists on Strapi

Actual: Do not display dApps that have been added to predefined lists on Strapi
Expect: Replace rtk query with fetch to get data

Version test: PR 1185 19:05 27/11/2023
Device test: Samsung S20 FE 5G

[ ] Crash app in case open dApp tab the first time https://konistudio-my.sharepoint.com/:v:/g/personal/nhungphung_koni_studio/Ef9_W7gW1MpEjoB7yOEfJdQBas0YD9pC9tQ8PHA5DzLrOQ?nav=eyJyZWZlcnJhbEluZm8iOnsicmVmZXJyYWxBcHAiOiJPbmVEcml2ZUZvckJ1c2luZXNzIiwicmVmZXJyYWxBcHBQbGF0Zm9ybSI6IldlYiIsInJlZmVycmFsTW9kZSI6InZpZXciLCJyZWZlcnJhbFZpZXciOiJNeUZpbGVzTGlua0RpcmVjdCJ9fQ&e=aWx3Z0

Version test: v1.1.12(242)
Device test: Samsung S20 FE 5G
Validation passed.


The markdown cannot correctly parse comments with line breaks in the input text

Describe the bug
The markdown is unable to correctly parse comments with line breaks in the input text.
To Reproduce
Steps to reproduce the behavior:

Go to Ask page
Type

<!--
hello world
-->

See hello world -->

Expected behavior
Show nothing.

<


ScienceNearMe → generic issue ("My Activity Log" page)

In the "My Activity Log" page (https://beta.sciencenearme.org/my/science), the "Report Your Science1" tab is missing a space before the number.

That 1 should be styled as a notification...

And is, when I look at it...

This was fixed in css-refine-12-22 but just never made it into master


Importing the PiGPio to the Pi Pico on micropython on MacOS

Dear reader,
I want to control a LED strip with my pi pico.
A ran into this library and succesfully tested it with my pi 4. Does wat it should.
For the next step I want to write a python code and run this on my pi pico.
The problem I run into is: having downloaded the repository to my macbook and trying to run make gives me:
pigpio-master % make gcc -O3 -Wall -pthread -fpic -c -o pigpio.o pigpio.c pigpio.c:59:10: fatal error: 'sys/sysmacros.h' file not found #include <sys/sysmacros.h>          ^~~~~~~~~~~~~~~~~ 1 error generated. make: *** [pigpio.o] Error 1
which raises the question: How would I install this library to my mac?
What I want to do is to run a python code telling:
from machine import pigpio
To be able to run the commands on my pi pico.
Any help would be appreciated.

You can't run pigpio on the pico.  It only runs on Raspberry Pis.
I understand your confusion, just accept that the Raspberry Pico is not a Raspberry Pi.
I have written a simple daemon for the pico, but it is not compatible with micropython.
See http://abyz.me.uk/picod/ or https://github.com/joan2937/picod

Thanks for your quick reply.
Then I am still wondering: is it possible to import the pigpio library on macOS using the thonny software? How would you do it?
Same for the pico software:
I can download the library but will I get a similar error running 'make'?

when i execute below command on my mac
user@XYZuser pigpio-master % make
i get the below error:
pigpio.c:59:10: fatal error: 'sys/sysmacros.h' file not found
#include <sys/sysmacros.h>
^~~~~~~~~~~~~~~~~


fix #1536, fix storm at least once support

the previous discussion is at #1540

The previous solution is flawed and the new idea is illustrated with the graph below.

The previous solution only keeps one pending message, acks the following messages (to avoid spout OOM) and acks the pending message when the global minAckTime exceeds its timestamp. This is incorrect since KafkaSpout will advance the store offset to the latest message when the pending message is acked (remember we already ack the messages behind the pending message) but we are not sure messages after pending message have been processed by the system.
The new idea is to add a next pending message when the pending message's timestamp is exceeded by the global minAckTime and only ack the pending message when the next pending message's timestamp is exceeded. Then KafkaSpout will advance the stored offset to that of next pending message. This is safe since we know that messages before next pending message have been processed by the system.
Then, next pending message becomes the new pending message and the whole procedure loops.

+1


Travis build fails with: LRU: key must be a string or number.

Travis build fails on abacus-usage-meter-itest with the following error in the logs:
TypeError: LRU: key must be a string or number. Almost certainly a bug! undefined
    at typeCheckKey (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/lru-cache/lib/lru-cache.js:20:19)
    at LRUCache.get (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/lru-cache/lib/lru-cache.js:230:3)
    at Object.get (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/abacus-lrucache/lib/index.js:45:20)
    at cached (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/abacus-metering-config/lib/index.js:104:18)
    at /home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/abacus-metering-config/lib/index.js:121:13
    at /home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/abacus-lock/lib/index.js:16:5
    at lock (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/lock/index.js:69:7)
    at lockcb (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/abacus-lock/lib/index.js:14:3)
    at config (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/abacus-metering-config/lib/index.js:116:3)
    at /home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/thunkify/index.js:43:12
    at next (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/co/index.js:90:21)
    at /home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/co/index.js:45:5
    at next (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/co/index.js:90:21)
    at /home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/co/index.js:45:5
    at next (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/co/index.js:90:21)
    at /home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/co/index.js:93:18
    at Immediate._onImmediate (/home/travis/build/cloudfoundry-incubator/cf-abacus/node_modules/co/index.js:52:14)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

Abacus is building ok locally with:
$ etc/versions
OS Darwin SOFM33790380A 15.2.0 x64
Node v5.4.1
Npm 3.3.12

Looks like this is now fixed. The build was broken for some time as we were in the middle of refactoring the Abacus stubs/plugins.


Update README.md

Correct hyperlink for http://adamschwartz.co/

Thanks for the correction! :)

You're welcome :)
Thanks for the great resource, just added it to the references of my web design courseware: http://collectifwp.github.io/cours-web/css/typographie#rfrences


bclient exit status

bclient should exit with a non-zero status if there was an error of any kind.
If the -wait option is used, this includes exiting with an error if the transaction ended up in the StatusError state.

Starting on this..

Done.


Error connecting to Mongo with production environment

I'm trying to start the Strapi container passing the NODE_ENV var to production but after everything is created I have got this error and the container continues to restart:
I NETWORK  [conn14] Error receiving request from client: SSLHandshakeFailed: SSLHandshakeFailed. Ending connection from ...

Is there something I'm missing? Everything working normal with NODE_ENV=development.

This is just a wild guess, but are you using self-signed certificates by chance?
Maybe some middleware (e.g. mongoose) handles certificates differently in development mode.

Hi @msgeissler, thank you for your response.
Nope, I have simply launched the docker-compose with NODE_ENV=production trying to simulate a production situation. I have CloudFlare pointing to my machine'IP, so the certificate is served by them, but I think the problem is likely more "internal".
Another thing I noticed is that I was able, just one time, to switch the previously started container in developed mode to production mode and managed it to work, but after a simple restart the error came up again.

Hi @LasaleFamine, have you try to run it outside the docker environment ?

Actually I had not the time to test outside Docker. Anyway I can confirm I have the problem also without the use of docker-compose. Launching two separate containers for Strapi and Mongo gave the same problem.

@Aurelsicoko : is there a specific behavior on the database connection when you set NODE_ENV=production ?
This bug is more related to Strapi.

No, but it will use the configurations available in the ./config/environment/production/*.json files.

Hi @LasaleFamine could you verify your production configuration ? are you using also SSL authentication for mongodb ?

I'm getting the error just setting NODE_ENV=production, so I suppose I'm using the bundled production Strapi configuration.

Unfortunately, the actual Strapi release used for this docker image (3.0.0-alpha.12) doesn't have the ability to connect to mongodb within SSL.
We'll do our best to build the image with the version 3.0.0-alpha.12.2 asap.

Hi @LasaleFamine , could you retry with the last docker image released ?

Awesome! Thank you so much for your constant updates @lucaperret.
The issue seems solved now with a fresh clone and docker-compose up with NODE_ENV=production specified for the Strapi container.
Close whenever you want guys, for me it's working. 👍

Hey, hello guys !
I've same problem with Strapi and Docker for environment production variable. Can't connect to mongodb :/

Hi @stephane-r, could you explain here how did you resolve it yesterday ?


Consume multiple-script-data endpoint of AST (#17619)

This PR is to deal with the following items

Connections quickly get exhausted when AST single-script-data endpoint is invoked several times.
Users face internal server errors when there's no connection available to call the AST API
The multiple-script-data endpoint is supposed to take multiple bindings in one single request and to serve the need for multiple connections through one connection where applicable
Now RTS server is accessed through <IP_ADDRESS> instead of localhost to avoid the domain name resolution before every API call (Related SO post)
Max connections are increased to 1000 which we can reduce later when we have a permanent solution to this problem
Earlier for each action inside a JS collection the updateLayout would've been called. As a result, the AST API was called more than 38K times just for changing one character in the JS editor for a moderately complex app
We have ensured one updateLayout call for a JS collection in this API and that reduces the number of API calls from 38k to 1.5k (Approximately)
This also helps improve the response time of the update JS collection API

Disclaimer
The PR is not supposed to solve the problem entirely i.e. it will not prevent it from ever happening again. But it will increase the chances of not getting internal server error while editing large JS objects.
Description
RCA and possible solution on Notion
Fixes #17619
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual
JUnit

Test Plan

Add Testsmith test cases links that relate to this PR

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=49b87ca

There are several factors/issues which are hindering the potential benefit that we could gain over this PR

For a moderately complex app it requires thousands of connections to be available for consuming AST APIs. The binding values are quite distinct and there are only a few cases where multiple binding values are multiplexed into one single request (/ast/multiple-script-data). Thus we couldn't exploit the multiple-script-data endpoint.
The AST API is consumed more by the appsmith-defined binding values e.g. {appsmith.theme.colors.primaryColor, appsmith.theme.boxShadow.appBoxShadow etc}. Can we work on that particular area so that AST API is consumed to get possible entity references for the user-defined binding values or can we introduce caching in the webclient?
The debounce time is currently configured as 100ms in the front-end. For every small change the update JS collection API is triggered and that requires a lot of connections to be available for consuming the AST API. Can we increase the debounce time to something larger or can we introduce a button for saving the JS object explicitly by the user?

/ok-to-test sha=74e4ce0

/ok-to-test sha=c115cb0

Tested this fix  by trying the following scenarios -

Cut / copy paste a code within a JS Object to a different area- works fine
Git connected apps JS Object editing - works fine
Import and export flow of git connected apps - works fine
Editor - adding lines in the js editor - works fine
Create
New JS object - works fine. (code from highlighted jsobject is copied, but this is being handled as part of different bug)
Delete JS object
 - works fine
Copy Js object  to same or different page - works fine
Move JS object
 to a different page - works fine
The internal server error was no longer seen.
The JS Object editor works fine, without hanging or crashing in any of the above scenarios.


Add ZipCode Service to main Branch

In this PR, we add an service that wraps the ZipCode API from
"http://api.zippopotam.us/us/{zipcode}";
Mutation Testing and jacoco both are 100% for ZipCode test and mvn compile works perfectly
Closes #13

looks good to me, can merge pull request


when to release the code?

when to release the code?

Same question.

Thanks for you interest. Now, we have released the code. Any problems, please contact me freely.

Same question.

Thanks for your interest. Now, we have released the code. Any problems, please contact me freely.

why not put the code in github?

why not put the code in github?


MySQL Database

Hello,
Do you support MySQL DB?
Thanks,
Sudhanshu Garg

We hopefully will in the near future.


minimal improvement of README.md

Thanks.

You are welcome. Great to see assertj-db getting some updates. :+1:

;-)
I will try to work on it.


inference explainability

Added wording on inference explainability

Docs Build status updates of commit bdfcccc:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/98726/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[4.0.0.-beta.13] ion-toggle always returns "on" as its value

Bug Report
Ionic Info
"@ionic/core": "4.0.0-beta.13"

Describe the Bug
ion-toggle always returns "on" as its value, even when switching it off.
Related Code
<ion-toggle onIonChange={e => console.log(e.detail.value) />

Expected Behavior
I'd actually prefer if the returned value was a boolean (currently the boolean is in event.detail.checked and that behaves correctly). But there must be some reason to use "on" and "off" for the toggle value?

The value of a ion-toggle or a ion-checkbox might be only used when it's participating in a form. It can be any value. Going to update docs to make that clear


Task 376-378 Word length tasks

task376_reverse_order_of_words |  Reverse the order of words in the given sentence | Answer Generation
task377_remove_words_of_given_length | Remove all words of a given length in the sentence | Answer Generation
task378_reverse_words_of_given_length | Reverse all words of a given length in the sentence | Answer Generation

@kurbster windering if you can review this task?

@kurbster windering if you can review this task?

Yes! I can start right now.

For Task 376, when I originally read the definition I thought the order of characters would be reversed (like task 378), maybe the definition could be more clear by saying "reverse the order that words appeared in the given sentence." You should add a negative example for this case, where each word is reversed, i.e "the quick brown fox" => "eht kciuq nworb xof". Also, in the current negative example the word parking needs a quote after it to remain consistent.
For Task 377, I feel the definition could be worded better, "Count of letters determine ..." => "The number of letters in a word determine its length." For your negative example I think you should add more explanation "The words 'in' and 'of' are length 2 however, the word 'of' is not removed. The answer should have been ..." Another negative example should be added where instead of removing all words of length 2, it removed 2 characters from all words, i.e "the quick brown fox" => "e ick own x"
For Task 378, same comment about the definition as Task 377. For your negative example you should explain why this is a negative example, "The word 'are' is reversed even though it is of length '3'. The answer should have been ..." Another negative example should be added where instead of reversing all words of length 2, it reversed 2 characters from all words, i.e "the quick brown fox" => "hte uqick rbown ofx"

Also, @danyaljj is it desirable to repeat the task in every instance? Every instance has "Reverse the order of words in a given sentence." at the end of the input. (Or whatever the prompt is for the respective task).

Also, @danyaljj is it desirable to repeat the task in every instance? Every instance has "Reverse the order of words in a given sentence." at the end of the input. (Or whatever the prompt is for the respective task).

Unless necessary, we prefer to keep all the task definitions/prompts in the "Definition" field.

For Task 376, the explanation for the second negative example include something like The characters of each word are reversed, instead of the order of words being reversed.
For Task 377 per danyljj, you should remove the definition Remove all words of length ... from the input. For the second negative example, the explanation should include with something like Two characters were removed from every word, instead of removing words of length 2.
For Task 378, you should also remove the definition Reverse all words of length... from the input. For the second negative example, the explanation should include with something like 'However, every word was reversed`
Basically we want to explain why those negative examples are negative. You can say this however you feel is best, you just need to provide context as to what specifically made the negative examples bad.

In Task 377 & 388 , there is a parameter involved. So, eliminating the sentence " Remove all words of length '{len}' in the given sentence." from the input would not the input incomplete.

@nrjvarshney that makes sense. I'm wondering if we could phrase it better.  @danyaljj what do you prefer for the input to the instances?

Length: '2'.
 Remove all words of length '2' in the given sentence.

Either way is fine. As long as it is clear to an average human reader, we're good!

Ok, then everything is good. Ready to merge!

Thanks to both of you!


A potential NPE bug

Hi all.
Again, our bug scanner reports a NPE bug.
The variable target is checked for whether being null at DatabaseMappingContainer.java#L126 while not checked at DatabaseMappingContainer.java#L136.
Thanks.

Thank you for the report, but it is expected.


Defender for Endpoint: clarify Note blob contents

Description:
From issue ticket #8810 (Confusing text):

The Note about server licensing is confusing. some words are repeated and acronyms are not defined. Please review and edit this so that people that are not familiar with server licensing terminology can understand what is needed. The whole topic of Defender for Endpoint in regards to servers is very confusing and needs to be simplified.

Thanks to @DeanGross for reporting the issue and for pointing out the duplication typo.
Changes proposed:

expand and clarify acronyms and initialisms in the details of the Note blob
convert the list within the sentence to a bullet point list for better overview
remove a duplicated word (Microsoft Defender Defender for Endpoint)

Whitespace changes:

remove 3 blanks at end-of-line

Ticket closure or reference:
Closes #8810

Looking forward to having this reviewed by the MS Docs team.

#sign-off

Thank you for the added corrections. 👍🏻


API call to check if the repo is a GTO repo

There is no clear definition what is "a GTO repo", but I assume:

If there is a .gto config file - it's a GTO repo
If there is an artifacts.yaml file - it's a GTO repo
If there are some git tags that register/promote artifacts, it's GTO repo

Although all signals may have False Positives (e.g. you created some tags of GTO format with some other tool and didn't mean GTO; or you created artifacts.yaml without intention to use it with GTO), I think it's safe enough to expect it's a GTO repo and try to parse it accordingly (at least it's what GTO CLI&API will do).
cc @amritghimire

Why can't we assume that every repo is a GTO repo? ... some of them are empty for now, that's fine.


in docs image doesn't have max-width?

http://tachyons.io/docs/general/images/
I'm still learning the framework, so not sure if it is a "class" missing or what, but figured I'd let you know. :)

Sorry - the docs were behind. This has been updated - you were right, there was a class missing:
http://tachyons.io/docs/general/images/
Thanks for bring this to our attention


Create codebooks for download

See https://d-place.org/static/downloads/

Could be a recipe in the cookbook - maybe even the CLDF cookbook.


😎 Dark mode and cool menu hover effect

This PR changes the site to a default dark theme that is easier to read.  It also adds a "terminal cursor" effect when menu items are hovered upon.

The preferable solution for this problem would be to write some JS that takes into account the prefers-color-scheme media query and bases the theme on that.  The downside is that any Linux users will simply have to deal with light theme, which is why I made dark the default.

The preferable solution for this problem would be to write some JS that takes into account the prefers-color-scheme media query and bases the theme on that. The downside is that any Linux users will simply have to deal with light theme, which is why I made dark the default.

We could use cookies to store the color scheme

We should also probably change the cursor color when it's on light theme. At least on my machine, the light theme on Terminal is accompanied by a gray cursor

The preferable solution for this problem would be to write some JS that takes into account the prefers-color-scheme media query and bases the theme on that. The downside is that any Linux users will simply have to deal with light theme, which is why I made dark the default.

We could use cookies to store the color scheme

Agreed, though localStorage would be preferable as it doesn't expire.

Good Idea

I can make these changes 😊 Give me a few minutes.

Also, could we add the hacktoberfest tag to this repo? That would definitely help with any participator's PR count.

Don't we have to sign up?

Nope, just add the tag to the repo!

Also, could we add the hacktoberfest tag to this repo? That would definitely help with any participator's PR count.

Don't we have to sign up?

Nope, just add the tag to the repo!

done

Should we merge soon?

Should we merge soon?

@micahlt hasn't changed anything yet. (As far as his fork goes.)

Should we merge soon?

@micahlt hasn't changed anything yet. (As far as his fork goes.)

ok

I'm almost done - I've made the CSS start using variables, but I just need to add a toggle and some logic to follow the prefers-color-scheme media query.

Ugh, curators.json always has such annoying conflicts!

Is there any reason we can't just gitignore it?

Can we not merge the .replit file?

It's my bad - while trying to resolve the merge conflicts I accidentally merged the PR into main instead of main into the PR.

I think the changes are fine


Windows export removes relevant icon sizes from .ico

Godot version
3.6.beta3 (but I assume 3.5 and 4.x are also affected)
System information
Windows 11
Issue description
When exporting a Godot game for Windows, you can specify a .ico file that is used for various icons in the final game (desktop icon, taskbar, ...).
As far as I can tell, Godot only seems to let certain sizes through and does not include the rest in the final application. The only sizes that seem to be acceptable are: 16x16, 32x32, 48x48, 64x64, 128x128 and 256x256
If you create and use an .ico file that includes additional sizes, such as 72x72 or 96x96, they do not seem to make it into the application. I confirmed this using Resource Hacker, which is an application that lets you inspect an exe and see which icons it includes.
These additional icon sizes are relevant, as - depending on the windows display scale factor - windows would prefer to use those sizes. Here's the (official) list of relevant sizes for Windows 11 (and 10 I believe): https://learn.microsoft.com/en-us/windows/apps/design/style/iconography/app-icon-construction#icon-scaling
I tested this using the following .ico file, which includes a lot of different sizes (like 96x96): https://github.com/maximiliancsuk/LFSNightlyBuild/blob/main/logo4_square.ico
Is there a specific reason these get removed? I found this file in the source code:
https://github.com/godotengine/godot/blob/3.x/platform/windows/export/export.cpp#L58
That's still present in the 4.x codebase:
https://github.com/godotengine/godot/blob/e5bacbc4716ce5b54bc2ab8b3053cd5e22ffe446/platform/windows/export/export_plugin.cpp#L50
I think that's it! Then it should be a quick fix and I can provide a pull request. But I wanted to first confirm with you if I'm even looking at the right place :)
I know that Windows automatically uses other sizes and rescales them if it cannot find an exact fitting icon. But that leads to very blurry, unprofessional looking icons, especially when your icon is meant to be pixel exact
Steps to reproduce

Create a new project
set the Windows .ico in the project settings, using the ico file I linked above
setup a windows export, again setting the .ico file
export the project
either
inspect the resulting .exe using Resource Hacker, seeing that it only contains a limited set of icons:

or

run the .exe on Windows with desktop scaling at f.e. 150%, which makes Windows prefer to use icon sizes that are not included, and then see that the taskbar icon is blurry (which hints at a rescaling), like this:

Minimal reproduction project
N/A

but I assume ... 4.x are also affected

Would need verifying, a lot of code has changed in 4.x, and how to approach this bug depends a lot on that

but I assume ... 4.x are also affected

Would need verifying, a lot of code has changed in 4.x, and how to approach this bug depends a lot on that

I just verified it in 4.1.3. Generated .exe is missing icon sizes, just as in 3.x. It also hardens my suspicion that the code I referenced above is to blame.

https://learn.microsoft.com/en-us/windows/apps/design/style/iconography/app-icon-construction#icon-scaling

Probably a new Windows 11. The included set of sizes was the full list of the required and supported sizes for ICO files. But if Window can use other sizes (list of sizes might be relevant for UWP only), it should be extended to match it.


Angular compiler implementation for super-initializer parameters

See https://github.com/dart-lang/sdk/issues/48055 for the main tracking issue.

cc @leonsenft

There are two ways that AngularDart might need to consider but I am not worried about this change.

The generated code part. The Angular compiler doesn't generate code like this pattern.
Clients' Angular Dart code source. The Angular compiler can rely on Dart analyzer to extract the right nodes.

Regarding (2), I think there are some unanswered questions around dependency injection annotations on super-initializer parameters. See my thoughts in b/213383778.

FYI, the flag has been flipped so these can be used without a flag internally now.

this will be so useful for Angular DI...
any public Angular repo that we can try this out?

This is still in progress.

The internal bug has just been wrapped up. All set to close this issue now.

Thanks!

we know that if Google was to publish their internal AngularDart it would mean lots of breaking changes, however, we are very ok with that.
Before we dig the compiler and other internal stuff and have many hours of rework, maybe we would come with a plan to start publishing AngularDart again?


Mention how to ssh first time we need to ssh

https://github.com/kelseyhightower/kubernetes-the-hard-way/pull/191

+1, this confused me the first time I ran through the tutorial, I had to search for how to do it.

I've updated all the docs. This should be fixed now.


Update locatable-camera.md

Docs Build status updates of commit f51f446:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/locatable-camera.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


compile bebop_driver failed

Hi everyone
There was a problem I cannot fixed them.
When I used 'catkin built'  as the last step to build bebop_autonomy, it gives me this error:
/usr/bin/ld: /usr/local/lib/libavcodec.a(mqc.o): relocation R_X86_64_32 against `.rodata' can not be used when making a shared object; recompile with -fPIC
/usr/local/lib/libavcodec.a: error adding symbols: Bad value
collect2: error: ld returned 1 exit status
make[2]: *** [/home/taochen/bebop_ws/devel/.private/bebop_driver/lib/libbebop.so] Error 1
make[1]: *** [CMakeFiles/bebop.dir/all] Error 2
make: *** [all] Error 2
cd /home/taochen/bebop_ws/build/bebop_driver; catkin build --get-env bebop_driver | catkin env -si  /usr/bin/make --jobserver-fds=6,7 -j; cd -
...............................................................................
Failed     << bebop_driver:make                [ Exited with code 2 ]
Failed    <<< bebop_driver                     [ 30.5 seconds ]
Abandoned <<< bebop_tools                      [ Unrelated job failed ]
[build] Summary: 2 of 4 packages succeeded.
[build]   Ignored:   1 packages were skipped or are blacklisted.
[build]   Warnings:  None.
[build]   Abandoned: 1 packages were abandoned.
[build]   Failed:    1 packages failed.
[build] Runtime: 33.0 seconds total.
It still no works  when I add the 'set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fPIC" )' in the CMakeList.txt file which was located in the  bebop_dirver folder.
I also have tried command 'set(Boost_USE_STATIC_LIBS OFF) set(CMAKE_POSITION_INDEPENDENT_CODE TRUE)'. but still get the same error.
Anyone know how to fix the error?
Many thanks
Tao

This issue has been fixed in #164. Maybe, such changes were not reported in the master.


Curry loop()

As with Iterable.fold() (#540).

Eeeek, I agree that this is much better. It's a breaking change, though....
WDYT, folks, is it OK to make this change?

Seems to me we don't have much choice, leaving it to be different from the rest is just ugly. Just put it in the bag of other breaking changes for 1.2

Well, what other breaking changes have we made? Off the top of my head I can't recall any. We have been trying very hard to avoid them. Even at the binary level.

I have merged it. I doubt that there is very much code depending on loop(), and surely people who are using it are going to prefer it this way. Better to break things sooner than later.

Thanks @gdejohn.


make README easier to follow

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Create chiragnahata.is-a-fullstack.dev

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description

Website Link / Preview

Please help.
Also please say me how to assign SSL certificates to it and subdomains to it ?

You need to setup an SSL certificate or use Cloudflare's proxy.


(MODULES-3949) Fill in metadata.json

This commit fills in information for metadata.json
so that it'll no longer be empty.

Should this also delete Modulefile so there isn't redundant information in two files?

Looks like it - done in latest commit.


How to use the ignorecase option in the diffChars method

Please specify how to use the method, I am unable to use the ignoreCase parameter
I am using it in the browser. ie without node
thanks

Was just wondering, some testing reveals:
Diff.diffChars("ME", "me", {ignoreCase : true})

Thanks for the reply, this issue is quite old, closing.


Update config.toml

Updated the link for "Get Started" to point to the right place

- What I did
Updated the top menu for the "Get started" link to point to the right link
- How I did it
updated the config.toml file
- How to verify it
just 2 lines of code. Added in the config file.
- Description for the changelog

@aannttoo this will change because of the  UI change.


Voting for freshness/validity of a Demanded Comodity

Subject of the issue
Currently we are not able to identify if a need cited 3 hrs back is still valid or not and hence mobilizing forces towards meeting that demand may prove futile in the end
Your environment

version of your python
which browser and its version
OS Type (iOS/Android/Linux/Windows)

Steps to reproduce
Tell us how to reproduce this issue. Please provide a working demo as a GIF, if possible.
Expected behaviour
We need to have a way to list a demand at a place and people to be able to publicly vote for it's relevance , with time the validity should decrease and people should be able to mark the demand as met or not met  ( increasing validity )
Actual behaviour
Tell us what happens instead or what went wrong

good idea and #621


Clicking post before image uploads results in blank expense comment

See https://opencollective.com/foundation/expenses/90942
There's a blank comment from me. I think I clicked post before the image had fully uploaded. I did it again on the comment below and waited for it to load, and then it worked.
If the image has not uploaded, the comment should wait until it has to be posted, or the post button should be inactive until it's ready, or it should fail with an error.

Your proposed solution to disable the submit button while a file is uploading seems like the simplest and most generic way to solve this. We should ideally do it in RichTextEditor, to make sure all places using it (collective long description, update, etc) will be fixed too.


Check product minimal quantity

The module doesn't check minimal quantity of product or product attributes when sends an email to suscribed customers.
This fix modifies this and now checks the minimal quantity instead of just checking if the quantity is greater than 0.

Hello @Rmagic96 , you got some feedbacks from the dev team, are you able to make the required changes ?
Thanks !

Hello @Rmagic96 , you got some feedbacks from the dev team, are you able to make the required changes ?
Thanks !

Sure, its my first commit here and didn’t know i had to make the changes.
I will make those changes asap (today/tomorrow) and will make a new commit.
Thanks for telling me!

Hello @Rmagic96
I'm closing this PR since we have no news from you and another user created PR which is already in QA validation:
https://github.com/PrestaShop/ps_emailalerts/pull/85
Thank you for your work and I hope to see you again in other contribution!


Exiting list mode is a difficult user experience

It is fairly unintuitive to end a list of items.
Current behavior: You must click the list button or use the keyboard command to exit list mode.
Possible better behavior: Like Google Drive and other document editors, if you click enter on an empty list item, it will end the list mode.
I've recorded how Google Drive and Slate Editor handle this scenario:
https://giphy.com/gifs/l0IsIiFTS3ubqqHbW/fullscreen

Hi @charlex!
We will treat this as soon as possible but, we really appreciate a PR too! 😃
If you want to challenge yourself to solve this bug, we will be very glad to review your PR! If you have any problem to setup your local environment, we can help you. 😉
We were really happy with feedbacks came from the community.
And, thanks for the improvement request!


refactor automations to use choose.

refactor automations to use choose.

Should merge #15 before this one


EKS SMP use gp3 storageClass

Thanks for contributing to the Harness Developer Hub! Please use GitHub Issues to report documentation issues or request documentation enhancements, and our code owners will review your issue.
For issues and feature requests for the Harness Platform and modules, use the Harness Ideas Portal.
Type

[ ] Issue (such as typos, broken images/links, or missing/malformed documentation)
[X] Enhancement (such as UX suggestions, new documentation requests, or suggestions for how content is organized/presented)

:bulb: Tip: You can select Edit this Page at the bottom of any HDH page to instantly edit that page and submit a PR. This is the fastest way to correct errors and publish other small changes.

Description
As we described internally, want to provide more context for our EKS SMP install to utilize gp3 storage class as the default to help users save money:

No, we have a placeholder for storageClassName, if it is unspecified it picks up the cluster’s default storageclass (which is usually gp2 in aws, standard in gcp, it differs if customer has explicitly set it different as well)
you can make the storageClass configurable by setting global.storageClassName and global.storageClass to both gp3 in your onverride files. If there is a storageclass with that name, it will assign it to that

@tuffacton will take it on, just wanted the ticket started

@tuffacton can we close this?

Yes, lets clean it up, I'll get a proper ticket if it comes up