ghc does not support Unicode, reports locale encoding ASCII.

Issue description
I installed ghc via Nix and also via Stack. With the ghc binaries provided by Stack, I can easily do Unicode IO. But the Nix ghc errors out and pretends to know nothing about Unicode.
Steps to reproduce
% ghc -e 'putStrLn "x"'
x
% ghc -e 'putStrLn "λ"'

<interactive>:0:11: error:
    lexical error in string/character literal at character '\56526'
% stack ghc -- -e 'putStrLn "λ"'
λ

% ghci
GHCi, version 8.6.5: http://www.haskell.org/ghc/  :? for help
Prelude> import System.IO
Prelude System.IO> localeEncoding
ASCII

% stack repl
...
Configuring GHCi with the following packages:
GHCi, version 8.6.5: http://www.haskell.org/ghc/  :? for help
Loaded GHCi configuration from /run/user/1000/haskell-stack-ghci/2a3bbd58/ghci-script
Prelude> import System.IO
Prelude System.IO> localeEncoding
UTF-8

% echo $LANG
en_US.UTF-8
% which stack
/home/kindaro/.nix-profile/bin/stack
% which ghci
/nix/store/67g6vwr5mx26h5mickgw17k2irdx1c0d-ghc-8.6.5/bin/ghci

Technical details

system: "x86_64-linux"
host os: Linux 5.1.16-arch1-1-ARCH, Arch Linux, noversion
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.2.2
channels(kindaro): "nixpkgs-19.09pre184803.d567c486ca5"
nixpkgs: /home/kindaro/.nix-defexpr/channels/nixpkgs

Does /usr/lib/locale/locale-archive exist? I wonder if arch has some custom locale location. The difference between how Nix and Stack works here is that Nix uses a Nix-built Libc while Stack uses the system's Libc.

You can always do export LANG=C.UTF-8 to get non-localized unicode support though. See https://github.com/NixOS/nixpkgs/pull/58009 and https://github.com/NixOS/nixpkgs/pull/61202 for info on that

Yes, this file does exist.
If I set LANG as you say, something strange happens. I cannot quite explain, let me rather show.

 % export LANG=C.UTF-8

 % ghc -e 'putStrLn "<ce><bb>"'
 λ
 % stack ghc -- -e 'putStrLn "<ce><bb>"'
 <interactive>:0:11: error:
     lexical error in string/character literal at character '\56526'

 % export LANG=en_US.UTF-8

 % ghc -e 'putStrLn "λ"'
 <interactive>:0:11: error:
     lexical error in string/character literal at character '\56526'
 % stack ghc -- -e 'putStrLn "λ"'
 λ

So, whenever neither Zsh not Stack can do anything with Unicode, ghc can. And the other way around. (I have no idea why Zsh cannot deal with C.UTF-8, but that is a whole other question.)
To clarify:

Zsh is installed via Arch package system.
ghc is installed via Nix.
stack ghc runs another ghc that is installed via Stack installed via Nix.

I can live with one terminal set to C.UTF-8 and another to en_US.UTF-8, but it can hardly be called life.

So I gather the bug is in the libc? How can I diagnose it further?

We are running into this at work.
When running ghc on NixOS, it correctly determines that the encoding should be UTF-8.  However, when running ghc on Ubuntu, it incorrectly thinks the encoding should be ASCII.
Here is an example of running it on NixOS:
$ which locale
/run/current-system/sw/bin/locale
$ locale
LANG=en_US.UTF-8
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=en_US.UTF-8
$ nix-shell -p ghc --command 'which locale'
/nix/store/22h3f311fjymkvp683kb657jycs7i5pn-glibc-2.27-bin/bin/locale
$ nix-shell -p ghc --command 'locale'
LANG=en_US.UTF-8
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=en_US.UTF-8
$ nix-shell -p ghc --command ghci
> import System.IO
> System.IO.localeEncoding
UTF-8
> import GHC.IO.Encoding.Iconv
> GHC.IO.Encoding.Iconv.localeEncodingName
"UTF-8"

Here is what happens on Ubuntu:
$ which locale
/usr/bin/locale
$ locale
LANG=en_US.UTF-8
LANGUAGE=
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC=ja_JP.UTF-8
LC_TIME=ja_JP.UTF-8
LC_COLLATE="en_US.UTF-8"
LC_MONETARY=ja_JP.UTF-8
LC_MESSAGES="en_US.UTF-8"
LC_PAPER=ja_JP.UTF-8
LC_NAME=ja_JP.UTF-8
LC_ADDRESS=ja_JP.UTF-8
LC_TELEPHONE=ja_JP.UTF-8
LC_MEASUREMENT=ja_JP.UTF-8
LC_IDENTIFICATION=ja_JP.UTF-8
LC_ALL=
$ nix-shell -p ghc --command 'which locale'
/nix/store/rjsymbdxlwmfbpasn0jik1w97wgfk3qj-glibc-2.27-bin/bin/locale
$ nix-shell -p ghc --command 'locale'
LANG=en_US.UTF-8
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC=ja_JP.UTF-8
LC_TIME=ja_JP.UTF-8
LC_COLLATE="en_US.UTF-8"
LC_MONETARY=ja_JP.UTF-8
LC_MESSAGES="en_US.UTF-8"
LC_PAPER=ja_JP.UTF-8
LC_NAME=ja_JP.UTF-8
LC_ADDRESS=ja_JP.UTF-8
LC_TELEPHONE=ja_JP.UTF-8
LC_MEASUREMENT=ja_JP.UTF-8
LC_IDENTIFICATION=ja_JP.UTF-8
LC_ALL=
$ nix-shell -p ghc --command 'ghci'
> import System.IO
> System.IO.localeEncoding
ASCII
$ nix-shell -p ghc --command 'env LC_ALL=C.UTF-8 ghci'
> import System.IO
> System.IO.localeEncoding
UTF-8

As above, you can see that explicitly setting LC_ALL=C.UTF-8, GHC picks up the encoding correctly.  However, be aware that there seems to be some weirdness with locales, and locales you may think exist do not actually exist.  On Ubuntu again:
$ nix-shell -p ghc --command 'env LC_ALL=C.UTF-8 ghci'
/nix/store/cinw572b38aln37glr0zb8lxwrgaffl4-bash-4.4-p23/bin/sh: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
> import System.IO
> System.IO.localeEncoding
ASCII

I haven't done a whole lot of testing of this, but this problem appears to have come about recently.
Here's an old GHC (from 19.03) on Ubuntu again:
$ cat /nix/var/nix/profiles/per-user/root/channels/nixpkgs/.version
19.03
$ cat /nix/var/nix/profiles/per-user/root/channels/nixpkgs/.git-revision
c2b8270fb8789af290da3f11bd6174a0ba7698f1
$ NIX_PATH=nixpkgs=/nix/var/nix/profiles/per-user/root/channels/nixpkgs nix-shell -p ghc --command 'ghci --version'
The Glorious Glasgow Haskell Compilation System, version 8.6.3
$ NIX_PATH=nixpkgs=/nix/var/nix/profiles/per-user/root/channels/nixpkgs nix-shell -p ghc --command 'ghci --version'
> import System.IO
> System.IO.localeEncoding
UTF-8

Here's a new GHC from the current nixpkgs-unstable channel, again on Ubuntu:
$ cat ~/.nix-defexpr/channels/nixpkgs/.version
20.03
$ cat ~/.nix-defexpr/channels/nixpkgs/.git-revsion
895874d2145862249df3f78335f4dcf62ef01626
$ NIX_PATH=nixpkgs=$HOME/.nix-defexpr/channels/nixpkgs nix-shell -p ghc --command 'ghci --version'
The Glorious Glasgow Haskell Compilation System, version 8.6.5
$ NIX_PATH=nixpkgs=$HOME/.nix-defexpr/channels/nixpkgs nix-shell -p ghc --command 'ghci'
> import System.IO
> System.IO.localeEncoding
ASCII
$ NIX_PATH=nixpkgs=$HOME/.nix-defexpr/channels/nixpkgs nix-shell -p ghc --command 'env LC_ALL=C.UTF-8 ghci'
> import System.IO
> System.IO.localeEncoding
UTF-8

Basically, if someone is willing to do a git bisect between c2b8270fb8789af290da3f11bd6174a0ba7698f1 (known-working) and 895874d2145862249df3f78335f4dcf62ef01626 (known-failing), we might be able to figure out what is the problem here.
I might do this.

Also, just in case you're curious, here is an explanation of text encoding stuff for GHC:
https://www.stackage.org/haddock/lts-7.14/base-<IP_ADDRESS>/System-IO.html#g:23
Here's the localeEncoding function I use above:
https://www.stackage.org/haddock/lts-7.14/base-<IP_ADDRESS>/System-IO.html#v:localeEncoding
Under the hood, this appears to be using iconv:
https://www.stackage.org/haddock/lts-7.14/base-<IP_ADDRESS>/src/GHC-IO-Encoding-Iconv.html

If I were to try to bisect this, I'd look for some change in how glibc or iconv is being handled that has occurred in the past couple months.  Or maybe even some direct change to ghc.

I think I figured out what is going on here.
Here's the explanation from the manual:
https://nixos.org/nixpkgs/manual/#locales

To allow simultaneous use of packages linked against different versions of glibc with different locale archive formats Nixpkgs patches glibc to rely on LOCALE_ARCHIVE environment variable.
On non-NixOS distributions this variable is obviously not set. This can cause regressions in language support or even crashes in some Nixpkgs-provided programs. The simplest way to mitigate this problem is exporting the LOCALE_ARCHIVE variable pointing to ${glibcLocales}/lib/locale/locale-archive. The drawback (and the reason this is not the default) is the relatively large (a hundred MiB) size of the full set of locales. It is possible to build a custom set of locales by overriding parameters allLocales and locales of the package.

My guess as to what is happening is as follows:
On Ubuntu, with older versions of nixpkgs, there was no locale archive provided by default, so GHC (really, iconv) falls back to the system locale archive in /usr/lib/locale/locale-archive.  The system locale archive has support for many different locales by default.  With newer versions of nixpkgs, there is a locale archive provided by default, so GHC (really, iconv) uses it.  However, it is very small and only has support for the C.UTF-8 locale.
On NixOS, with older versions of nixpkgs, there is a locale hardcoded somewhere with a bunch of locales provided by default.  With new versions of nixpkgs, NixOS explicitly sets the LOCALE_ARCHIVE env var pointing to somewhere with a bunch of locales available.
So @kindaro, the solution to your problem is to do one of the following things:

Set LOCALE_ARCHIVE to point to either ${glibcLocales}/lib/locale/locale-archive or your system locale archive at /usr/lib/locale/locale-archive (if you want to live dangerously).
Make sure you set LC_ALL=C.UTF-8 before you run GHC.

@matthewbauer Is this explanation about right?

@cdepillabout Awesome research, thank you. Setting LOCALE_ARCHIVE works, and it is a better solution than resetting LC_ALL because it does not affect other installations of ghc, such as stack's.

I'm going to go ahead and close this, since it seems to be "working as intended", and I listed some workarounds in https://github.com/NixOS/nixpkgs/issues/64603#issuecomment-551419489.

FYI: I think one can make the program locale independent by calling setLocaleEncoding utf8. (compare https://hackage.haskell.org/package/base-<IP_ADDRESS>/docs/GHC-IO-Encoding.html#v:setLocaleEncoding)


Request: New release containing fix for #83

I stumbled upon #83 when trying to use this library to fix https://github.com/tomasbjerre/violations-lib/issues/95.
I need this fix: https://github.com/java-diff-utils/java-diff-utils/commit/dbe601c2fcb766dda91b33d1a4db15dc8a056959
Would be nice if you can make a release with this included.

just released


Validation within a nested step

I have a nested directive with a form as a part of a step. I want to validate on exit using 'canexit'. What are the options to communicate with the nested directive?

@marcingrabda Can you give me a code example of JSFiddle?  It's hard to say how to pass scope around without knowing more information.  Is this nested directive inside angular-wizard another wizard or is it a custom directive?

The question is actually quite generic, but maybe it will influence the framework design.
<wz-step title="Sponsor Market" canexit="validateStepONe">
            <wizard-step header="Step One">
                <step-one>
                </step-one>
            </wizard-step>
</wz-step>

I have something like this and I see that it's not possible to currently call a method from 'stepOne' directive as 'canexit' handler. Not sure, if it's possible to provide any alternative at all.
Just sayin'.


chore: release 0.2.0

This release is too large to preview in the pull request body. View the full release notes here: https://github.com/aepfli/lifecycle-toolkit/blob/release-please--branches--main--release-notes/release-notes.md

:robot: Release is at https://github.com/aepfli/lifecycle-toolkit/releases/tag/v0.2.0 :sunflower:


Installation

I would like to pick this up

are you working on this? @vamsikrishnareddybevara

I would like to pick this up

Hey @g-savitha, I would like to pick this up.


Depend on other addons

I have a private addon, that have ember-cp-validations in dependencies but in my consumer app ember-cp-validations does not work as it should, and then I'm forced to install the addon also in my app.
Reference: https://dockyard.com/blog/2015/03/22/tips-for-writing-ember-addons
https://github.com/BrianSipple/ember-stagger-swagger/issues/1#issue-145557246

#334

@villander did the solution from #334 solve your issue?

Closing as this issue has been abandoned. Feel free to reopen if needed.


Unable to build client

I am struggling to build the client.
Issues:

Need CMake 3.17, not CMake 3.14+
Had to manually disable SDL2 from CMake (maybe there was a command for this?)
CMake wanted to build example that does not exist (CMake indicated it should have been in examples, but I could not find it.)
CMake errors out during linking something about CMake -E (I just stopped at this point)

I am not a CMake guru, I disabled OpenSSL in my CMake 3.21 build from source. I changed it to use direct-draw-on-buffer, but it then complained about image.bmp. I tried commenting that out. This is on Linux.
I am trying to test my library against this code base. I found two bugs in server.
Issues:

Possible race condition using mFrameCount to switch buffers inside thread (rare)
Pixels per panel is not computed correctly: width * width (easy fix)

I am able to build the client here: https://github.com/jaygarcia/network-rgb-matrix-display

Hi @daveythacher !

I am struggling to build the client. Issues:

Need CMake 3.17, not CMake 3.14+
I'll update the instructions accordingly.

Had to manually disable SDL2 from CMake (maybe there was a command for this?)
I will make a PR to do this.  It's likely an artifact from an improper commit. Thank you!

CMake wanted to build example that does not exist (CMake indicated it should have been in examples, but I could not find it.)
CMake errors out during linking something about CMake -E (I just stopped at this point)

I am not a CMake guru, I disabled OpenSSL in my CMake 3.21 build from source. I changed it to use direct-draw-on-buffer, but it then complained about image.bmp. I tried commenting that out. This is on Linux.

I am trying to test my library against this code base. I found two bugs in server. Issues:

Possible race condition using mFrameCount to switch buffers inside thread (rare)
Pixels per panel is not computed correctly: width * width (easy fix)
Thank you for these. I'll make a PR as well.


Facebook share shows none of the text

Here is a snippet of the code.  I replaced a few things, but here it is.

await CrossShare.Current.Share("I'm using (my new app) to get great seats to awesome shows, check it out! appStoreURL", "Hey, Check out (my new app)!");

I am not seeing any other apps where this actually works.

This is on iOS?

I tested on simulator and did a test on clipboard and that seemed to work... hmmmm https://github.com/jguertl/SharePlugin/blob/master/Share/Share.Plugin.iOSUnified/ShareImplementation.cs#L42-L63
Looks pretty accurate and has worked in the past... Perhaps it should be:
https://developer.xamarin.com/api/type/MonoTouch.UIKit.UIActivityViewController/
  var item = UIActivity.FromObject (textField.Text);
    var activityItems = new NSObject[] { item };
    UIActivity[] applicationActivities = null;

    var activityController = new UIActivityViewController (activityItems, applicationActivities);

will test today and push package.

It actually looks like Facebook broke it: http://stackoverflow.com/questions/29890747/ios-how-to-share-text-and-image-on-social-networks
I have added a way to remove specific UIActivityTypes, in the iOS project:
call...
ShareImplementation.ExcludedUIActivityTypes = new List<string>{ UIActivityType.PostToFacebook};

Additionally, if you are calling only from iOS then you can use an overloaded constructor. Will be available in the next version.


Code Refactory - Modified clear_data function

Included clear_data, clear_data_dummy function.

clear_data_dummy function is not working at all (used delete_many - similar to remove in MongoDB).
clear_data function is not working as expected (used update_one).

Required changes proposed via the slack channel.


Understanding

Hello,
Not a proper issue. I just want to understand some aspect missing or confusing for me in the documentations.
Does More that one thread can be achieve without creating two worker.thread.js file.
I tried to create two variables
let thread1= new Thread('/path/to/mythread) let thread2= new Thread('/path/to/mythread)
but I am not sure of the result.
My second point is about integration of package.
How can i include react-native-contact package within threads?

Hey, I created an Update fork.
You can check it here: https://github.com/irohitb/react-native-multithreads#working-with-existing-thread-passed-from-native-side


this adds a ✅ or a ❌ depending on if the candidate did or didn't clear the hurdle

Motivation
It's nice to see at a glance if a particular dump did or didn't clear the hurdle.
A bit of discussion in #156
Changes

[X] Ensured that you have rebased your branch with this repo's latest master branch.
[X] Ensured that relevant issues are linked, if this PR resolves any outstanding.
[X] Added a screenshot for all UI changes (you can drag the file into this edit box and it will be uploaded).
[X] Ensured that changes to auto-generated files have not been committed; in particular, *.html, *.csv and *.json files.

Nice. Good visual indicator.

Closed, see discussion on #156


Update tdr-aws-utils to 0.1.55

Updates uk.gov.nationalarchives:tdr-aws-utils from 0.1.52 to 0.1.55.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "uk.gov.nationalarchives", artifactId = "tdr-aws-utils" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "uk.gov.nationalarchives", artifactId = "tdr-aws-utils" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

This library is being removed in favour of using separate libraries.


Add damegender

Hi @davidam, thank you for adding information about a project that can help with demographic information.
It looks like the DCO sign-off failed for the second commit. Would you mind fixing it or creating a new pull request with the correct DCO sign-off? For information on the DCO sign-off, please review  https://github.com/chaoss/governance/blob/master/CONTRIBUTING.md#code-or-document-change-contributions-github-interface
Also, the Diversity and Inclusion metrics are not maintained in the /metrics repository. The D&I Working Group is working on D&I metrics in the following repository: https://github.com/chaoss/wg-diversity-inclusion/
Maybe the damegender reference would fit well in this document: https://github.com/chaoss/wg-diversity-inclusion/tree/master/demographic-data


Dublicated graph item

Hi, can you submit a simple project that reproduces this problem?

Close this issue because it cannot be reproduced and the requester has not responded for a long time.


Openshift CI docker file clean up

What type of PR is this?
/kind cleanup
What does this PR do / why we need it:
Cleaning up CI docker file
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes - N/A
How to test changes / Special notes to the reviewer:
CI pr https://github.com/openshift/release/pull/18588 should pass after this pr is merged

This pr is a clean up pr for https://github.com/redhat-developer/kam/pull/234

/approve
/lgtm


(test) Snyk path vulnerability detection algorithm

(test) Snyk path vulnerability detection algorithm

/hold

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 71.81%. Comparing base (bafb859) to head (7cf7436).
Report is 9 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #988       +/-   ##
===========================================
- Coverage   95.26%   71.81%   -23.46%
===========================================
  Files          54       53        -1
  Lines        2134     2164       +30
===========================================
- Hits         2033     1554      -479
- Misses        101      610      +509

Files
Coverage Δ

ols/app/endpoints/ols.py
81.08% <0.00%> (-15.52%)
:arrow_down:

... and 26 files with indirect coverage changes


can't use my micro anymore

Microsoft PowerToys version
0.53.1
Running as admin

[X] Yes

Area(s) with issue?
Video Conference Mute
Steps to reproduce

enable Video Conference Mute
Toogle Camera and micro mute (Win+N)
Toogle again (disable) (Win+N)

Now all my micros doesn't work anymore and I can't disable it.
Steps I try to solve:

reboot
disable the entire feature
reenable it some times
windows problem solver
quit powertoys
Windows Support
save mode

✔ Expected Behavior
It should enable the mute and than diable again.
❌ Actual Behavior
But it enabled, but I can't disable it.
Other Software
No response

Are they built-in or external? Tried disabling (and enabling) them in Device Manager?

/bugreport

We toggle the Windows global mute so big question is your microphone muted?  Win-Shift-A would just target that.
In Win11, just go to system->sound and you'll see the input dialog

Are they built-in or external? Tried disabling (and enabling) them in Device Manager?

They are external I am using a PC
I disabled and reenabled them in the Device Manager but it still doesn't work

We toggle the Windows global mute so big question is your microphone muted? Win-Shift-A would just target that.
In Win11, just go to system->sound and you'll see the input dialog

I've also tried Win+Shift+A but this also not works
I have Windows 10, it says I have to wait to get Windows 11

I suppose the question "Did yu check (unplug and put in) the cables?" would be a no-brainer? (you never know...)

It is the same path for Win10 settings.  Clicking the microphone should unmute you

Can you take a screenshot?  you shouldn't need to do anything with device manager.

I suppose the question "Did yu check (unplug and put in) the cables?" would be a no-brainer? (you never know...)

Yes I did I even tested it on another PC thereit works
I also switched the USB Ports

@crutkas do you mean this?
Sorry I'm German

Maybe this would be a possibility? Very difficult to tell, this way, through tekst on a screen...

Okay, I deaktiviated it and than reactivated it but it still does not work

I also tested it and it showed 0% Sound

Looking on a Win10 machine, looks like the Setting UX doesn't update like Win11's does.
You may have to click twice on that icon.

Also in PowerToys settings, by unchecking "Hide toolbar when both cameras are unmuted", you can toggle the mic & camera when the camera isn't in use.

It's showing that the mic is enabled but it doesn't work.
I can press the microphone button as many times as I want but nothing changes

@crutkas I don't know about Win 11, but the icon you circled on the Win 10 image, is not a button, just a static icon. (at least for me, what are the odds it's different for you?)
@domx4q Are yuo sure the external microphone doesn't have some sort of physical switch?

yes I am sure that my microphone has no switch (is a camera)
I test this but now again for safety on my laptop

on my Laptop

IT WORKS
For some reason it works fine now.
I don't know why, because I had unplugged the camera before and tested it in another device.
Sorry for the trouble, maybe the problem had nothing to do with Powertoys and was just a hardware defect.
But thanks for the help you gave me.

we got you :)


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1759 ms

Resolved: Bofrost.be performance has improved in f5cb3d3 after 6 minutes.


Add OpenCovidTrace by @shiaulis

[x] Project URL:
[x] Update contents.json instead of README
[x] One project per pull request
[ ] Screenshot included
[ ] Avoid iOS or open-source in description as it is assumed
[x] Use this commit title format if applicable: Add app-name by @github-username
[x] Use approved format for your entry

Not sure I understand what mistake did I make…

Looks good, will be able to merge today. Thx!


ci: Lint with flake8

Closes #25
Just a basic annotator.
PR also contains a .vscode which contains settings appropriate for this repo (black autoformatting and flake8 settings). --> #38 and #39

It should be notated that this uses an action that uses a version of Node that will be deprecated soon:

Last commit is 2022 so not sure if the author will update it. I guess we can use it for now and in the worst case just replace or disable it again should it cause issues once support for Node 16 has ended.


create webpack loader

I was wondering to create another module, but this is a file with 2 lines of code. 😄
There are many benefits using graphql-tag as webpack loader:

Saves GraphQL ASTs processing on client-side
Optimizes bundle size
Enable queries to be separated from script over .gql files.

That's amazing! By the way, I'd say it would be better to have .graphql as the recommended file extension, since that's what most tools support these days.
Do you mind adding a page under "recipes" to the docs site as well? http://dev.apollodata.com/react/

Sure thing! I will do it ASAP 🍵

Published in 0.1.15!


Does NVS works well?

I have tried your program, I found my esp32 could not relink my keyboard after rebooting.

Hello there!
First of all, make sure line line #119 is commented out in the source, as is presented here. Just in case you have enabled that line by mistake.
NVS flash only takes a rol when using Bluetooth Classic keyboards with code entry pairing. Newer BLE keyboards don't need it, as the keyboard itself is responsible for reconnecting to the module.
In order to further assist you, I'll need more info and these questions precisely answered:

Is your keyboard BT Classic (code pairing) or BLE? (Exact brand and model will be perfect)
By "rebooting" do you mean powering on and off the host system, or the keyboard?
If talking about the host system, does the PS2 port power off and on? (Does it power-cycle the port?)


udocker installation fails

Following the installation guide at https://github.com/indigo-dc/udocker/blob/master/doc/installation_manual.md udocker installation currently fails: the proper version of udocker 1.1.1 seems not to be present in the download repository:
$ ./udocker install
Info: creating repo: /home/sustr4/.udocker
Info: installing 1.1.1
Info: installing from: http://repo.indigo-datacloud.eu/repository/indigo/2/centos7/x86_64/tgz/udocker-    1.1.1.tar.gz
Error: in download: HTTP/1.1 404 Not Found
Info: installing from: https://owncloud.indigo-datacloud.eu/index.php/s/AFImjw8ii0X72xf/download

I thanks, I'm waiting for Cristina to add the artifacts to the INDIGO repository, hopefully this week.
Meanwhile it can download automatically from the other locations, in your case it downloaded
from the INDIGO owncloud.


Visitor finish early

Suppose a use case to find a Transaction in a Block, once found the parsing of the block should be interrupted early.
Like returning a bool from visit_transaction indicating to continue to parse or not?

https://doc.rust-lang.org/std/ops/enum.ControlFlow.html was made for this
let users return ControlFlow<T> and return Break when a user returned Break (with the branch operator), Continue otherwise (like in the basic tree traversal example)
this also allows users to return a value like can be done with break some_value; in a for loop

That's interesting, however, I don't think in this case the Continue case , or having a value inside the Break variant is useful, as in PR #27

i would prefer this to a bool, a bool does not tell if it needs to continue to the next tx or stop, it needs to be written in the comments that true means stop and false means continue (or the other way around)
i tried to implement returning a value with ControlFlow::Break(some_value) but this was way too much effort. this is indeed not very useful considering they can be set on the visitor and extracted afterwards (like FindTransaction::tx_found) in the PR
ControlFlow<()> acts like a bool (2 variants, size of 1 byte, align of 1 byte, etc) so there is no performance difference, and describes what it does (continue and break instead of true and false)

on the sense of the bool you convinced me, it's pretty confusing what true or false means, even though it's also used in standard lib for example in the predicate of the filter method https://doc.rust-lang.org/std/iter/trait.Iterator.html#method.filter

https://github.com/RCasatta/bitcoin_slices/pull/27


OBP on clifford circuit produces a NoneType backpropagated circuit

What is happening and why is it wrong?
If I pass in a Clifford circuit to the OBP method, it can backpropagate over the entire circuit (since the number of terms in the operator does not increase). The truncated/backpropagated effective circuit will only be a trivial circuit with all qubits initialised in |0>. However, passing in such a clifford circuit through the OBP structure produces a NoneType object for the backpropagated circuit instead of a trivial one, thus restricting the following steps of execution.
How can we reproduce the issue?
from qiskit.quantum_info import random_clifford
from qiskit_addon_obp.utils.simplify import OperatorBudget
from qiskit.quantum_info import SparsePauliOp
from qiskit_addon_obp import backpropagate

cliff = random_clifford(num_qubits=6, seed=10)
clifford_circuit = cliff.to_circuit()
op_budget = OperatorBudget(max_qwc_groups=8) # exact value doesn't matter here
observable = SparsePauliOp("Z"*clifford_circuit.num_qubits)
bp_obs, remaining_slices, metadata = backpropagate(observable, slices, operator_budget=op_budget)

bp_circuit = combine_slices(remaining_slices, include_barriers=True)
bp_circuit.draw('mpl')

The draw function itself throws error because bp_circuit is not a QuantumCircuit object, but is NoneType
Any suggestions?
For such a scenario, where backpropagation leads to a trivial backpropagated circuit, the output should be a QuantumCircuit object with the same number of qubits. The simples bypass can be if remaining_slices is [], change the NoneType bp_circuit to QuantumCircuit(num_qubits).

To address the issue where the backpropagation of a Clifford circuit results in a NoneType object instead of a trivial circuit, we can modify the backpropagate function to ensure that it returns a valid QuantumCircuit object even when all slices are absorbed during backpropagation. Specifically, if remaining_slices is an empty list, we should create and return a trivial QuantumCircuit with the appropriate number of qubits initialized to the |0⟩ state.
Here are the proposed changes to the backpropagate function in qiskit_addon_obp.backpropagation.py:

Modify the return statement to check if remaining_slices is empty and return a trivial QuantumCircuit if so.

from qiskit import QuantumCircuit

# ... (rest of the code remains unchanged)

def backpropagate(
    observables: SparsePauliOp | list[SparsePauliOp],
    slices: Sequence[QuantumCircuit],
    *,
    truncation_error_budget: TruncationErrorBudget | None = None,
    operator_budget: OperatorBudget | None = None,
    max_seconds: int | None = None,
) -> tuple[list[SparsePauliOp], Sequence[QuantumCircuit], OBPMetadata]:
    # ... (existing code)

    # Return the slices which were not backpropagated
    slices_out = slices[: -metadata.num_backpropagated_slices]

    # Specify the output observables on all qubits in the circuit
    obs_out = [
        to_global_op(obs, qargs, num_qubits) for obs, qargs in zip(observables_out, qargs_out)
    ]
    # If the input was a single observable, return a single observable
    if isinstance(observables, SparsePauliOp):
        assert len(obs_out) == 1
        obs_out = obs_out[0]

    # If all slices are absorbed, return a trivial QuantumCircuit
    if not slices_out:
        num_qubits = slices[0].num_qubits if slices else 0
        trivial_circuit = QuantumCircuit(num_qubits)
        return obs_out, [trivial_circuit], metadata

    return obs_out, slices_out, metadata

Reasoning:

Trivial Circuit Handling: By checking if slices_out is empty, we can determine if all slices were absorbed during backpropagation. In such cases, we create a new QuantumCircuit with the same number of qubits as the original circuit (or zero if there are no slices), initialized to the |0⟩ state.

Compatibility: This change ensures that the function always returns a valid QuantumCircuit, preventing downstream errors when attempting to combine or draw the resulting circuit.

This approach maintains the integrity of the backpropagation process while ensuring compatibility with subsequent operations that expect a QuantumCircuit object.

This seems to be a perfect solution. No further comments on this.

Great! I'm glad the solution meets your requirements. If you have any more questions or need further assistance, feel free to ask. Happy coding!


Data: project addition by Ajay-Dhangar

Add a new project to the list
Add a new project to the list
I want to add my project named GitHub User Profile Viewer.
Here is the project https://javascript-mastery.github.io/javascript-mastery/Projects/github-api/
Describe:
This project allows users to enter a GitHub username and view the profile information of the corresponding user. The project utilizes the GitHub API to fetch data such as the user's profile image, name, bio, website link, location, followers, following, and public repositories. The fetched data is then displayed in a visually appealing format on the web page. This project serves as a simple and interactive way to explore GitHub profiles and learn more about users and their contributions. The project is implemented using HTML, CSS, and JavaScript and can be easily deployed.
Demo:

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

/assign

Successful completed: Live Preview


Deep ar session

Supplementary notebook for the DeepAR session. Notebook was tested using the btsa Docker image.

great! thanks!


[webapi] Change designed test to approved for WebDatabase

Impacted tests(approved): new 13, update 0, delete 0
Unit test platform: [Android]
Unit test result summary: pass 0, fail 0, block 13
Block Reason: not support openDatabaseSync() method

XWALK-2768 bug need to test them, these tests are changed to designed by PR#1173, it's my mistake, so reset them.

Prefix bug id in commit.


Redirect to homepage after login

Currently, if I login in the Browse page, I get directed back to homepage. Can we have it redirect back to Browse?

i would like to work on this


SAK-49757 Samigo: fix attachment to a question response and cleanup

Hi @bgarciaentornos, I spent a bunch of hours in here. It's really difficult code.
This reverts a lot of the code in SAK-48067 from Vincent. I tested paging, and it works with this code.
This also reverts some code from the Dynamic Rubrics: S2U-5 <IP_ADDRESS> Tests & Quizzes: Dynamic Rubrics - MASTER (#12241)
My main question: is the Dynamic Rubrics code very important? It scares me because I don't understand it. And it seems to touch lots of code. Will it be maintained going forward?
Note that I also revert a bit of code from Fernando in SAK-47090 Test & Quizzes - Set a bigger number of students in Sakai Pager and set it as default selected value (#10374). I don't think that code had much of an effect as the number gets overridden via the setter.

Hi, Sam, thanks for taking a look into this.
Dynamic rubrics is an important piece, part of the Unidigital project and it was also added very recently so it's not only alive, it's a newborn! From what I see the only "revert" you do is getting rid of the boolean param, but I'm not 100% sure why? Does it fail with it? We can test an alternative proposed solution but just removing that bit is not ideal.
I also added Jesus as a reviewer as he might know more about Fernando's changes.

Thing is the questionScores method inside getAllAgents uses that param too and it was the main reason for that override. It was introduced for a specific case within the dynamic rubrics, not sure what would happen now with your change. Let me think about it and I'll come back to you, hopefully we can find a middle point that supports it all. We can do the testing related to S2U-5 ourselves but hopefully we can find a way of solving this without breaking that feature. (We can do a demo of sorts at some point in the future, but I guess it's not worth it for you to delve deeper into it right now.)
Also on 22.x you won't find that param, so it might be one of those "different PR for each version" situations while we're discussing this...

Hi @bgarciaentornos did you have a chance to test this against dynamic rubrics?

Sorry, I was bloated with other tasks and also we had a big local holiday. I have a modification in mind that might avoid regressions while respecting your changes. Hopefully I can work on it and test it today and come back to you asap.

I have just added a commit but please wait until I can verify the functionality in a Linux env. Thanks!

Thanks! Note that if you push your commit to this branch, it will spin up a Linux server in Github Actions that is available via a Cloudflare tunnel for quick testing ;)

Hi! I can't see the Github Actions/Cloudfare, is it because permissions or am I missing something? Thanks!

I was going to verify this, but I found something else in the S2U-5 that is broken and is preventing me from testing this part. Let me find when did that happen and if I can fix it quickly.

It was broken here -> https://github.com/sakaiproject/sakai/pull/12329
I think I can do a temp local fix to verify this and will fix it better whenever I have more time


Provide an option in the SqlNNNScriptGenerator classes to preserve comments

Is your feature request related to a problem? Please describe.
Some use cases for the Sql*ScriptGenerator classes are actually for code "pretty printing" / formatting. However, using Sql150ScriptGenerator to re-generate the SQL text when provided with an input TSqlFragment, leads to comments (single-line and multi-line) both being excluded from the output.
Describe the solution you'd like
Provide an PreserveComments property in SqlScriptGeneratorOptions with the default being true in the 160 version, and false in the 150 or lower versions for backward compat.
Describe alternatives you've considered
Token re-writing: substituting a faux print statement instead of the comment token is a crude workaround. It fails in case the comment is a trailing suffix of a line within a multi-line T-SQL statement.

Would really appreciate if this issue was fixed, it will help greatly with using scriptdom to fix code issues. thank you.

This fix would be a huge help! Not only for me but many others. Thank you.

Completely agree that we need an option to preserve comments when using the script generator.


feat: support custom favicon

Fix https://github.com/slidevjs/slidev/issues/420

It's possible use now a custom favicon?

PR welcome :)


Implement CAN monitor mode / buffer dump

Even though the ELM327 can't monitor a high-speed CAN bus in real-time, it would be nice to have support for "streaming" all received data to the application.
According to the datasheet, either MA (monitor all) or BD (buffer dump) can be used for that purpose.
Together with CF (ID filter) and CM (mask) only the relevant messages can be filtered, so the chip has a chance to capture all relevant bus traffic.
This won't play nice with the way this library currently works: by definition, the monitor command will never complete (unless the buffer becomes full). But the library will only call the response handler if the command is complete, and the whole response has been received.
My idea is to add special handling when in monitoring mode - each received line is directly forwarded to the application, with no further processing in between.
I'm going to try and implement it in my fork and will send a PR.

Sounds good – I'd definitely like to see this in LTAutomotiveSupport.
With respect to the inner workings, here are some spontaneous ideas:

Either we distinguish on the LTCommand level whether we have a oneshot or a streaming command or we come up with a subclass LTStreamingCommand.
The command queue / protocol engine then needs to be prepared to handle commands that have the streaming attribute in a slightly different way than before.
With respect to the API, currently I would vote for leaving it as it is, i.e., we could just define that for certain commands the response handler may be called multiple times.
We can use cancelPendingCommand to stop a currently running streaming command.

These are just directly out of my head without thinking longer about it. Feel free to come up with an alternative proposal, if you think you have a better idea.
Good speed!

Due to time constraints, I have not finished my rework... right now I simply start the monitoring again as soon as a response has been received from the device.
Since bluetooth LE can't transmit very fast, the device quite often responds with "BUFFER FULL", but so far, I was able to sniff CAN traffic at 250kb/s (with filters set, it would be too much without them).


Use in another piece of software

Hi, I've recently discovered this software, and it's exactly what I need,
I've been developing a program called Sim Racing Launcher, which allows sim racers to launch racing titles, along with other necessary software in one click. One great feature I would like to add to my software is the ability to set a certain audio device as default on game launch, and back to another on game exit.
I have already got the appropriate events for game launch/close, and I was wondering if I could use your software to manage the audio switching, if I pass it calls from my software.
Since you've kindly distributed the source code, do I have permission to bundle this software with mine, and use it.
I can either host the windows form inside my WPF app, add a button to run the audio switcher software externally, or have the audio switcher silent, with the user controls being in my software. Even command line arguments could be a method.
Please could you recommend any hints and tips for the best practice,
Cheers,
Morgan

I pulled out the switching functionality a packaged it into a nuget package a while back.
Repository: https://github.com/xenolightning/AudioSwitcher
Package: https://www.nuget.org/packages/AudioSwitcher.AudioApi.CoreAudio/4.0.0-alpha5
Feel free to use that, to do all the switching you need.


[PostgreSQL] Ability to add column comment to view/materialized view

Ability to add column comment to view/materialized view through UI
Now there is no way to edit the columns of views/materialized views at all, so it's not a trivial task to make it possible to edit only comments.

version 7.3.3  not changed

version 7.3.3  not changed

21.0.3 added this future. Ok

The feature is not implemented yet.

Closed as duplicate of https://github.com/dbeaver/dbeaver/issues/16337


Design Submodule package namespacing? tbdex.Foo vs. tbdex.bar.waz.Foo

This needs more discussion, de-prioritizing for now.


Representing the Binned distribution directly through log probabilities

Co-authored with @canerturkmen
Description of changes: The Binned distribution is not too robust in its log_prob method. The reason is the internal representation being based on the probabilities of the bins, rather than directly the log probabilities. This can be seen with the following MWE:
import mxnet as mx

from gluonts.distribution import BinnedOutput

distr_out = BinnedOutput(bin_centers=mx.nd.array([1.0, 2.0, 3.0, 4.0, 5.0]))
args_proj = distr_out.get_args_proj()
args_proj.initialize()

network_out = 50*mx.nd.ones(shape=(10,))

args = args_proj(network_out)

distr = distr_out.distribution(args=args)

print(distr.log_prob(mx.nd.array([0.5, 1.5, 2.5, 3.5, 4.5, 5.5])))

Output before the fix:
[nan nan nan nan nan nan]
<NDArray 6 @cpu(0)>

Output after the fix:
[   0.       -113.20108   -54.23133   -65.08172   -16.988205  -16.988205]
<NDArray 6 @cpu(0)>

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #556 into master will decrease coverage by 1.55%.
The diff coverage is 92.85%.

@@            Coverage Diff            @@
##           master    #556      +/-   ##
=========================================
- Coverage   83.26%   81.7%   -1.56%
=========================================
  Files         178     176       -2
  Lines        9953    9894      -59
=========================================
- Hits         8287    8084     -203
- Misses       1666    1810     +144

Impacted Files
Coverage Δ

src/gluonts/distribution/binned.py
94.91% <92.85%> (+0.22%)
:arrow_up:

src/gluonts/nursery/sagemaker_sdk/defaults.py
0% <0%> (-100%)
:arrow_down:

src/gluonts/nursery/__init__.py
0% <0%> (-100%)
:arrow_down:

src/gluonts/nursery/sagemaker_sdk/log.py
0% <0%> (-100%)
:arrow_down:

src/gluonts/nursery/sagemaker_sdk/__init__.py
0% <0%> (-83.34%)
:arrow_down:

src/gluonts/dataset/split/__init__.py
0% <0%> (-80%)
:arrow_down:

src/gluonts/model/r_forecast/__init__.py
0% <0%> (-80%)
:arrow_down:

src/gluonts/nursery/sagemaker_sdk/model.py
0% <0%> (-53.34%)
:arrow_down:

src/gluonts/dataset/split/splitter.py
0% <0%> (-47.48%)
:arrow_down:

src/gluonts/nursery/sagemaker_sdk/estimator.py
0% <0%> (-43.75%)
:arrow_down:

... and 6 more


transformed documents

@Urigo this caused a regression in my application :( I have a collection that contains an array of objects, which are mapped to inputs on the page. With this change, every time I input something into a field, Angular seems to think that the entire array has changed or something similar.
I thought this could be a $$hashKey getting lost along the way, but rather it seems to be just the call to self.$$collection.findOne that does this, since it's enough for me to console.log that function call for Angular to react the way I described above

Interesting
@Nieziemski thoughts about our previous solution?

@fredrikekelund can you also share a repo that shows the problem?

and give @Nieziemski and myself write access so we can add our scenario and find the best solution to support both

I'm also gonna try using observe instead of observeChanges since it is supposed to give you the document after tranformation and not just the fields like observeChanges (not sure about that yet...). Needs to see how this will affect performance though.

interesting..  branch?

I reproduced this bug, but can't find the reason.
Till now I go with this in console:
angular.extend(temp1[0], {name: 'test1'});
temp1.$rootScope.$apply()
where temp1 is angular collection. And sometimes it reloads entire collection, but sometimes not. Little wierd. I will look at this tomorrow again.
@Urigo: I Can't read the code now, but looking at $$internalProps ("A list of internals properties to not watch for, nor pass to the Document on update and etc") I think there may be more issues with attaching anything that isn't mongo property to doc. So the whole idea with transformed documents may be a dead end. Sorry for not checking this arlier.

@Urigo do you still want me to share some code, even if @Nieziemski can reproduce the problem as well?
Maybe we can roll back this change until we've found a possible solution?

@fredrikekelund please do, it's too early to give up on the transform idea, I want to have a good repo that uses both of your use cases and then I can work on it

@Nieziemski @fredrikekelund I would also love to see a reproduction of your bug. I've tried reproducing and I didn't encounter a case where the whole collection seemed to update even though just one document was updated...
Where exactly did you console.log? how did you update the collection? manually or by auto bind?

@Nieziemski @fredrikekelund please check @netanelgilad's fork:
https://github.com/Urigo/angular-meteor/pull/209
and tell me if it works for you.
For me it looks like a really elegant solution.

@Urigo, tested the fork, and it works like a charm! Thanks @netanelgilad

Great, it's already merged but we are waiting for another performance improvement we added before releasing a new version.
@Nieziemski can you confirm as well?

@Urigo, yes, for me it also works very well in all cases. And looks really good :)

Great, I hope I will merge the performance update later today and maybe release a new version tomorrow, just before ng-conf!


Added context support

to: @ljharb @goatslacker
Adds support for context with mount and shallow.
The API looks like:
const context = { name: 'foo' };
const wrapper = mount(<Component />, { context });

const context = { name: 'foo' };
const wrapper = shallow(<Component />, { context });

Additionally, there is a .setContext(newContext) prototype method on each wrapper class.
Fixes https://github.com/airbnb/enzyme/issues/49
Note: this will constitute a minor change (new feature, backwards compatible)

@ljharb tests added. comments addressed.

Changes LGTM - @goatslacker should weigh in on the context stuff tho since he's got more experience with it than I

Looking at the coverage info with the line numbers mismatched is a PITA. I'm going to submit a followup PR that fixes istanbul line numbers, as well as improve coverage where needed

LGTM

Super useful PR. May I ask what is the reason for render not having this functionality?


sendHttpRequest Url length

Hi,
i try to use your library for a small gps project. I call the sendHttpRequest with the following example url:
http://mydomain.net/api/InsertData.php?id=at1&lat=0&long=0&orga=test&plattform=arduino&version=1.0&capacity=100
The request doesn't work because of an truncated url. I think it's an timing problem. Can you fix it?
Regards,
Matthias

Hey, the url seems to be short enough to fit into the buffer, which is hardcoded to be 200 bytes long. Try playing with the buffer timeout values in this file.
core->writeCommand("AT+HTTPPARA=\"CID\",1", "OK", 2000); on line 53 - changing '2000' to a larger value might help.
Making the url shorter should improve the performance. This is more of a hardware issue, not something that can be fixed in the code.


Redirect history?

Does this library provide one?

@mzabriskie  @nickuraltsev ?

There's no support for the complete history but there's a way to get the last url. See #799.

Why is there no complete history, at least for Node.js? Controlled isomorphism?

@rubennorte ^

We use follow-redirects internally to implement redirections and it doesn't provide it. In case anyone is interested in this, it should be implemented there and then adapted here.


feat: modify the behavior of 2 buttons ; commit after a rebase master

Which issue(s) this PR fixes

Fixes #160
I replaced the "p" markup by the Link component.
Additional comments

Agreements

[X] I hereby declare this contribution to be licensed under the Apache License Version 2.0, January 2004.
[X] I hereby agree to grant TOSIT a copyright license to use my contributions.

I made another commit to change the type of the component, replacing it by a Button

I added the attribute to the button ^^


catfish: fix double wrapping resulting in wrong StartupWMClass

Description of changes
Wrapping the binary also changes the WMClass:

xprop

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Ah, that does work! Much better than my solution.

@ofborg build xfce.catfish


Add a Codacy badge to README.md

Codecov Report

Merging #45 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #45   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          64     64
  Lines        1357   1357
=====================================
  Hits         1357   1357

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 24a3ec8...2dde580. Read the comment docs.


Fix harmless Fargate error checking

Closes #40.
I've run across a slightly different error than the one that's currently being checked on the harmlessFargateProfileCreationErrors slice.
In the mentioned Issue, I've suggested something like checking with strings.Contains but I feel like that could ignore errors that aren't harmless in some cases (not quite sure though).
With this in mind, I've decided to simply add my variation of the error to the slice itself, this fixed my issue and now it works correctly.
I took the opportunity to fix a simple grammar typo as well if you don't mind.
Let me know if this needs to be changed in any way or if it simply isn't the best approach for it.

Any chance you can take a look at this @mumoshu?

@TiagoJMartins Thanks for the PR! I'll review this today.


feat: use @chainsafe/blst directly

Motivation
In order to do some optimizations, we want to remove the @chainsafe/bls abstraction layer. We know we won't be running the beacon node or validator in node only. In the future, we may relax this restriction if a usecase presents itself.
Description

Replace all uses of @chainsafe/bls throughout the monorepo, with the exception of the light client package
Notable changes:

.toBytes() -> .serialize()

serialize defaults to compressed
there is no longer a third param to do validation, it needs to be done afterwards with a .keyValidate() or .sigValidate()

.fromBytes() -> .deserialize()
Pubkey.aggregate, bls.aggregatePubkeys -> aggregatePubkeys

same for signatures

Signature.verifyAggregate -> fastVerifyAggregate
all verification functions into (msg(s), pubkey(s), sig(s)) order -- <- very important to get right

Metrics on feat2-mainnet do not look great.  They should be equal to unstable/stable but seems to be performing worse for some reason... Will try and swap for another feat group tomorrow and see if its the machine

After running for a while it seems like this branch is actually fine.  @tuyennhv would love your opinion as well but I'm going to approve

Looks good to me, had a suggestion for signatureFromBytes() util also signatureFromBytesNoCheck() should be moved to @lodestar/utils instead so that both could be consumed by different packages

Added them both to @lodestar/utils

@twoeths @philknows Spoke with @wemeetagain and we will merge this one after interop.  Should not be an issue but why add more complexity to an already moving target.  Can sit for the week and get merged when we get back.  All is ready and just waiting for reapproval after the last update for the comment @twoeths had

:tada: This PR is included in v1.19.0 :tada:


CMake error string REGEX MATCH

I get this CMake error while trying to compile example project (or any project that fetches this project)/ CMakeLists file is root folder of the project
CMake Error at CMakeLists.txt:74 (string):   string sub-command REGEX, mode MATCH needs at least 5 arguments total to   command.
Got this issue just recently. CMake version is 3.27.7

Same issue happens using cmake version 3.28.0-rc3

Thought that another cmake version fixes it, but same happens with 3.28.0-rc3

Hey, thanks for reporting this problem!
It seems like the curl command does not return any result.
Can you call the following command in your command line and tell me what response you receive:
curl -s -D - -X POST -d generator=c&api=egl%3Dnone&api=gl%3D4.5&profile=gl%3Dcore&api=gles1%3Dnone&profile=gles1%3Dcommon&api=gles2%3Dnone&api=glsc2%3Dnone&api=glx%3Dnone&api=vulkan%3Dnone&api=wgl%3Dnone&options=LOADER https://gen.glad.sh/generate
As a workaround you can set IGG_GLAD_DOWNLOAD to OFF. The script will then use the included glad loader sources (gl 4.5 core profile).

When I'm trying to run this command I get next error (my windows is in another language so I translated the error therefore it could not match with what is listed on the internet):
Invoke-WebRequest : Missing argument for "SessionVariable" parameter. Specify parameter with type "System.String" and try again.
line:1 sym:6
+ curl -s -D - -X POST -d "generator=c&api=egl%3Dnone&api=gl%3D4.5&prof ...
+      ~~
    + CategoryInfo          : InvalidArgument: (:) [Invoke-WebRequest], ParameterBindingException
    + FullyQualifiedErrorId : MissingArgument,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

And yep, making IGG_GLAD_DOWNLOAD as OFF worked perfectly fine, thanks!

In order for curl to work in the windows powershell, you have to use the program name with the extension:
curl.exe -s -D - -X POST -d "generator=c&api=egl%3Dnone&api=gl%3D4.5&profile=gl%3Dcore&api=gles1%3Dnone&profile=gles1%3Dcommon&api=gles2%3Dnone&api=glsc2%3Dnone&api=glx%3Dnone&api=vulkan%3Dnone&api=wgl%3Dnone&options=LOADER" "https://gen.glad.sh/generate"

Well, if I use it like this it outputs nothing. Seems like it could be a problem on my side but idk

That's strange, can you try it in the cmd?

I've released a new version (v4.1.0) with better error handling and reporting. If you want you can try this version and check the output of the cmake script. If it's unable to download the glad loader sources it will print the error code/error message reported by curl, it will also automatically use the included version in this case.

Sorry for writing so late, but cmd isn't working either.
The error that's being displayed is this:
CMake Warning at cmake-build-debug/_deps/imgui-glfw-glad-glm-src/CMakeLists.txt:80 (message):
  C:/Windows/System32/curl.exe returned: 35

But new version works fine, thanks! I think we can close this issue now

No problem!
According to curl, error code 35 means there is a problem with the SSL/TLS handshake. I cannot reproduce this behaviour on my machine by sending a simple GET request with curl to https://gen.glad.sh/.
I will close this ticket as it seems to be not a problem with the CMake script.


Write functional tests for Templates API

This commit adds a Templates API spec - https://github.com/gocd/ruby-functional-tests/commit/55eb6b247493d709253fe2d799c54cb268395876. More spec will be added when we have APIs for updating the Template authorization


Script error

[21:08:46] [SCRIPT SV] [info] SCRIPT ERROR: [SERVER] Entity class "CCrossbowBolt" doesn't match required class for this index or function call
[21:08:46] [SCRIPT SV] [info]  -> array<entity> weapons = attacker.GetMainWeapons()
[21:08:46] [SCRIPT SV] [info]
CALLSTACK
*FUNCTION [MeteorThermite_DamagedTarget()] weapons/mp_titanweapon_meteor.nut line [114]
*FUNCTION [CodeCallback_DamagePlayerOrNPC()] mp/_codecallbacks.gnut line [276]

[21:08:46] [SCRIPT SV] [info] LOCALS
[attacker] ENTITY (crossbow_bolt [1016] at <7842.53 -6291.02 2422.38>)
[damageInfo] INSTANCE
[target] ENTITY (player anthougil [2] (player "anthougil" at <7806.4 -6243.04 2409.99>))
[this] TABLE
[@ITERATOR@] 1
[callbackFunc] CLOSURE
[@INDEX@] 0
[damageSourceId] 40
[damageMultiplier] 1
[attackerIsNPC] false
[attackerIsTitan] false
[attackerIsPlayer] false
[inflictor] ENTITY (crossbow_bolt [1016] at <7842.53 -6291.02 2422.38>)
[attacker] ENTITY (crossbow_bolt [1016] at <7842.53 -6291.02 2422.38>)
[entIsNPC] false
[entIsTitan] true
[entIsPlayer] true
[damageInfo] INSTANCE
[ent] ENTITY (player anthougil [2] (player "anthougil" at <7806.4 -6243.04 2409.99>))
[this] TABLE

DIAGPRINTS

@Zanieon

This is a weapon error, i fail to see where it is related to Frontier Defense.

@liveskai as @Zanieon pointed out this error is unrelated to FD. It did not happen in any Frontier Defense related scripts but in weapons/mp_titanweapon_meteor.nut as indicated by the logs you posted.
Just because a script error happened while playing Frontier Defense does not necessarily mean that it was caused by script related to FD.
Please only ping Zanieon if you are completely certain and have checked that the error is in Frontier Defense related scripts.
If you're not certain you are of course still free to make an issue but in this case please do not ping Zanieon as he primarily focuses on FD related things.
Thanks <3

Closing due to inactivity and apparent mod issue rather than Northstar issue


Is there a workaround to use run-tests.sh --browser?

Issue Type

Support Request

Your Environment
Vagrant 1.8.7
VirtualBox 5.1.10r112026
ansible <IP_ADDRESS>
  config file = /Users/wt/.ansible.cfg
  configured module search path = Default w/o overrides

Your OS

macOS (10.11.6 El Capitan)

Full console output

Couldn't get a file descriptor referring to the console

when using
run-tests.sh --browser
Summary
I realise why this is happening - obviously if I run tests from the Mac itself it can easily launch my  browser when finished.  However certain tests (such as the core issue I was working on today, simply don't work properly when run in OS X, only in DrupalVM, and it's really helpful to use --browser and --verbose to look at individual pages.
Is there anyway to start tests in the VM (or possibly via Ansible??) - and thus filter by individual method, as well as class - but still get the full HTML reports? (even if I have to check what the URL is myself each time.)

Adding a few bits and pieces to make sure this is reproducible:

Install a Drupal 8 site.
Enable the Simpletest module.
vagrant ssh into Drupal VM.
cd into /path/to/[docroot]/scripts
Run php run-tests.sh --browser Test

It looks like the run-tests.sh script is looking to run either open [url] (macOS, most often), xdg-open [url] (Linux flavors), or start [url] (Windows flavors).
To get xdg-open working inside Drupal VM (these instructions assume Ubuntu, tested on 16.04 at least):

Install xdg-open: sudo apt-get install -y xdg-utils
Run a test: php run-tests.sh --browser Test

Then it will show an error like:
/usr/bin/xdg-open: 461: /usr/bin/xdg-open: links2: not found
/usr/bin/xdg-open: 461: /usr/bin/xdg-open: links: not found
/usr/bin/xdg-open: 461: /usr/bin/xdg-open: lynx: not found
/usr/bin/xdg-open: 461: /usr/bin/xdg-open: w3m: not found
xdg-open: no method available for opening 'http://localhost/sites/default/files/simpletest/verbose/results-487d9cd9-5d8a-474c-bad3-73fdb03bcedf.html'

I've tried installing and using links2 and lynx, but it seems the run-tests.sh script just pauses until stopped manually if it detects a CLI browser.
So in lieu of that, I just copied the link that was spit out at the end, replaced localhost with my Drupal VM's hostname (drupalvm.dev by default), then pasted it in a browser on my Mac/host, and voila!

And I was about to file a core issue to fall back to printing the URL alone if no valid OS helper was found (it searches for xdg-open, then open, then start)... but then realized that Ubuntu flavors (don't know about others) include /bin/open, which apparently is a wrapper around openvt and doesn't actually have anything to do with opening a file from the CLI.
So, I'll just leave it be, and recommend you install xdg-utils to get the URL to view the full test results.

thanks, that did the trick


Run-cleaner internal dns is incorrect

Summary
Run cleaner doesn't correctly find the API's service DNS entry

This is resolved in PR #84


How to integrate '-e UPTIME_KUMA_DISABLE_FRAME_SAMEORIGIN=1' option in docker-compose.yml

Hi,
I run uptime-kuma "out of the box" like the quickstart shows. Everything fine!
Now I created a docker-compose.yml like descibed here:
https://github.com/louislam/uptime-kuma/blob/master/docker/docker-compose.yml
and want to integrate the option that I use with docker run -e UPTIME_KUMA_DISABLE_FRAME_SAMEORIGIN=1
But I am not sure how to "translate" the -e argument for the docker-compose.yml
Any idea?
Thanks

OK, found a solution here:
https://github.com/AgP42/MMM-SmartWebDisplay/issues/34#issuecomment-1214423746
version: '3.3'
services:
uptime-kuma:
image: louislam/uptime-kuma:1
container_name: uptime-kuma
volumes:
- ./uptime-kuma-data:/app/data
ports:
- 3001:3001  # :
restart: always
environment:
- UPTIME_KUMA_DISABLE_FRAME_SAMEORIGIN=1


Getting google.rpc module error from opencensus library.

Package Name: azure-ai-ml
Package Version: 1.13.0
Operating System: Linux
Python Version:  3.10

Describe the bug
Tried to install azure-ai-ml and run the python code. It was giving google.rpc module error.
It comes from the opencensus/trace/status.py at line 15
Screenshots

Hi @DeepKariaX - did you get this error during the installation command or while trying to run azure-ai-ml code? If the latter, can you share the code you ran?


background property on the table contains invalid value

The following mjml:
<mjml>
  <mj-body>
    <mj-section>
      <mj-column padding="10px" background-color="red">
        <mj-divider border-color="#F45E43"></mj-divider>
      </mj-column>
    </mj-section>
  </mj-body>
</mjml>

creates HTML with a table, having property background which is equal to the specified color.
In this case <table background="red"/>. background property is used to specify the image, not the color. Color should be specified using the bgcolor property.
Is this a workaround for some issue or just by accident?

Hi,
Yeah this should be removed and the container-background-color one too.

It seems that it has been fixed in recent 4.2 release, so i'm closing


Mismatch between GPU and CPU simulations "pyridineDensOverlap" example

I ran the "pyridineDensOverlap" example with the GPU and CPU versions (https://github.com/Probe-Particle/ppafm/tree/main/examples/pyridineDensOverlap). Using the same input files and comparing the results, I noticed differences in the simulated images generated. For most simulations, there is a match; but some appear quite different (e.g., df_024.png or df_029.png). What are these differences due to? I expected much more similar results. Am I missing something? Thank you

The GPU version is using some faster approximations in various parts of the computation, which means that it's potentially less stable in some respects.
In this case it's a interpolation error that comes from resampling the charge density and Hartree potential to a coarser grid. If you set the grid density higher by putting pixPerAngstrome=14 (the original grid density) in the run_gpu.py script, then the rings should disappear, but this also makes the computation slower and requires significantly more memory.
I actually spotted this one before but forgot about it. We probably should fix this but I'm not sure how, because for example my laptop GPU runs out of memory if I set the grid density to that value.

We agreed that @NikoOinonen will take on the task of adding documentation explaining the areas of applicability of different methods/approximations. After that, the issue can be closed.


Add Upgrade Guide notes for various deprecations and removals

Fixes #21515

@bot-gradle test BD

OK, I've already triggered the following builds for you:

BuildDistributions build

@bot-gradle test BD

OK, I've already triggered the following builds for you:

BuildDistributions build

@bot-gradle test and merge


Update AndroidManifest.xml for android:usesCleartextTraffic="true"

Summary:
In order to use localhost:8081 for dev server, we need this to allow non HTTPs requests
Changelog:
[ANDROID][Fixed] Allow clear text traffic for the Android template app
Test Plan:
Create a new app and run TD

We can't accept this on the template as it will disable HTTPs for every app

oh. thanks!
then how to enable http://localhost:8081 for local debugging? any best practice?

It's already enabled for the Debug-only manifest:
https://github.com/facebook/react-native/blob/main/packages/react-native/template/android/app/src/debug/AndroidManifest.xml#L6


Added support to enlarge center option and keep fixed size for others

Before :

After :

Hey @b-ray  addressed suggested changes.


Question: PowerArgs.InvalidArgDefinitionException: 'The method must be static because it has been imported using [ArgActionType] or [ArgActions]'

Hi!
I have multiple classes that each implement some properties and an action. From my main method I try the following:
    [ArgActionResolver]
    public class Program
    {
        public static void Main(string[] args)
        {
            var api = Stuf.GetInstance("MyKey");

            Args.RegisterFactory(typeof(MyFirstType), () => new MyFirstType(api));
            Args.RegisterFactory(typeof(MySecondType), () => new MySecondType(api, "stuff!?"));

            Args.InvokeAction<Program>(args);

            Console.ReadKey();
        }
    }

[ArgActions]
    public class MyFirstType
    {
        private readonly IStuff _api;

        public MyFirstType(IStuff api)
        {
            _api= Api;
        }

        [ArgActionMethod]
        public void MyAction(
            [ArgRequired] string stuff,
            [ArgRequired] string stuff2)
        {
            // do stuff
        }
    }

PowerArgs finds the action but throws the exception from the title. I can't make my method static since I need to access the properties PowerArgs has parsed, as well. How can I achieve that?
Thanks in advance!

If you make it static you can still access the parameters on the method. I think your issue is that you're mixing the factory pattern that lets you initialize your MyFirstType class with the ArgActions pattern that requires the args to be static. If you like ArgActions then just find another way to inject the IStuff property into the code.  You could just put this line var api = Stuf.GetInstance("MyKey"); directly in the action method.


[WIP] Check consistency proofs while catch-up

@p5n please review new test for consistency proofs.

Moved to https://github.com/hyperledger/indy-sdk/pull/21


SONARMSBRU-85: Improve error message when trying to build multiple configurations at once

Error messages discussed offline.
LGTM


Commands don't seem to be working

So I'm trying to run a .bat file that just does this for now:
ECHO "hello world!"
ECHO %1
pause

This is my config.xml:
<watches>
    <logging>
        <!--
            The full path to the log file.

            Default: %TEMP%\fw.log
        -->
        <path>C:\log\fw.log</path>
    </logging>
    <!-- A folder watch-->
	<watch>
        <!--
            Path of the folder to watch. File and folder exclusions are relative
            to the value specified in this element.
        -->
        <path>C:\Users\seanm\Desktop\temp\fw</path>
        <!--
            The amount of seconds to wait for the path to exist before
            establishing the watch. This timeout can be used with paths that
            are on external hard drives that may not be connected for a time
            when FileWatcher is started.
        -->
        <!-- <timeout></timeout> -->
                <!--
            Perform actions when a change is detected.
        -->
        <actions>
            <action>
                <!--
                    What events will trigger the action.

                    Values:
                        Change
                        Create
                        Delete
                        Rename
                -->
                <triggers>
                    <trigger>Create</trigger>
                </triggers>
                <!--
                    The type of action to perform.

                    Values:
                        Copy
                        Move
                        Delete
                -->
                <type>Copy</type>
                <!--
                    The source path of the action.

                    Placeholders:
                        [exactpath] - full path of the changed file
                        [fullpath] - full path of the changed file without the path of the watch
                        [path] - the path of the changed file without the path of the watch and the file
                        [file] - the name of the file with the extension
                        [filename] - name of the file without the extension
                        [extension] - the file extension
                -->
                <source>[exactpath]</source>
                <!--
                    The destination path of the action.
                    This value does not apply to the Delete action.

                    Placeholders:
                        [exactpath] - full path of the changed file
                        [fullpath] - full path of the changed file without the path of the watch
                        [path] - the path of the changed file without the path of the watch and the file
                        [file] - the name of the file with the extension
                        [filename] - name of the file without the extension
                        [extension] - the file extension
                        [createddate:format]
                        [modifieddate:format]
                        [currentdate:format]
                -->
                <destination>C:\log\[file]</destination>
                <!--
                    Verifies the file after a copy or move.

                    Values:
                        true
                        false (default)
                -->
              <verify>true</verify>
            </action>
        </actions>
        <!--
            Run commands when a change is detected.
        -->
        <commands>
            <command>
                <!--
                    What events will trigger the command.

                    Values:
                        Change
                        Create
                        Delete
                        Rename
                -->
                <triggers>
                    <trigger>Create</trigger>
                </triggers>
                <!--
                    The path of the command to run.

                    Placeholders:
                        [exactpath] - full path of the changed file
                        [fullpath] - full path of the changed file without the path of the watch
                        [path] - the path of the changed file without the path of the watch and the file
                        [file] - the name of the file with the extension
                        [filename] - name of the file without the extension
                        [extension] - the file extension
                -->
                <path>C:\app\test.bat</path>
                <!--
                    The arguments of the command to run.

                    Placeholders:
                        [exactpath] - full path of the changed file
                        [fullpath] - full path of the changed file without the path of the watch
                        [path] - the path of the changed file without the path of the watch and the file
                        [file] - the name of the file with the extension
                        [filename] - name of the file without the extension
                        [extension] - the file extension
                -->
                <arguments>"[fullpath]"</arguments>
            </command>
        </commands>
    </watch>
</watches>

I'm running filewatcher from the cmd prompt. The action works just fine, but nothing happens either in the cmd prompt or the log to suggest it tried to run the command.
My log file only logs when the watch begins, and when actions are performed, not that a new file has been created.

2023-03-22 10:19:07 INFO Creating watch for C:\Users\seanm\Desktop\temp\fw.
2023-03-22 10:19:07 INFO Watch created for C:\Users\seanm\Desktop\temp\fw.
2023-03-22 10:19:20 INFO Copied C:\Users\seanm\Desktop\temp\fw\New Text Document.txt to C:\log\New Text Document.txt. Verify: True. Keep timestamps: False.
2023-03-22 10:19:28 INFO Copied C:\Users\seanm\Desktop\temp\fw\f - Copy - Copy (3) - Copy.txt to C:\log\f - Copy - Copy (3) - Copy.txt. Verify: True. Keep timestamps: False.

Is there something I might be doing wrong? thanks
Windows 10

OK, so I filled out the batch script with what I really want to do - activate python and run a script.
It seems like what's happening is that the console doesn't get passed through to filewatcher. This appears in the log now:
2023-03-22 11:07:50 INFO The execution 'C:\app\test.bat "C:\Users\seanm\Desktop\temp\fw\myfile.txt"' has exited. Exit code: 0.
I assume this means there was an error in either my bat file or python code? How can I let it print from the console for debugging? - especially if it could log it that would be ideal.

Perfect, works great!


Add !$ OMP critical around file opening for VTK

This is the preferred alternate approach to PR #2339.  If this works, this PR will be merged in and the other closed.
Feature or improvement description
We were having problems with the !OMP directives around high resolution file reading in AWAE.f90. Since the file reading starts with a call to GetNewUnit before starting the opening, parallel calls to GetNewUnit could result in the same unit number handed out to two processes.  The first process would open the file and start reading, but then the second process would open a different file with the same unit number causing read errors for both processes as they attempted to read the same file at the same time.
Adding $OMP critical around the GetNewUnit and following OpenFile... calls so that in theory these cannot be done in parallel.
Related issue, if one exists
#2339
Impacted areas of the software
VTK file reading within !$OMP loops
Additional supporting information
Test results, if applicable
@rthedin will report on testing with many turbines in a FAST.Farm simulation.

Tested on Kestrel on an old 5-turbine case that worked with 3.5.3 commenting out the !OMP directives as Andy mentioned above. It runs with no problem.
The following header is printed:
 FAST.Farm-v2.5.0-3190-g0e7774a6
 Compile Info:
  - Compiler: Intel(R) Fortran Intel(R) 64 Compiler Classic for applications running on Intel(R)
  64, Version 2021.10.0 Build 20230609_000000
  - Architecture: 64 bit
  - Precision: double
  - OpenMP: Yes, number of threads: 104/104

It seems like there is an issue with the version number. Also, note that I have not noticed any performance difference between omp-commented-out 3.5.3 and this branch. CPU use patterns are the same, estimated time for completion is within 2 minutes of each other (total expected runtime ~6 hours).

I don't have a test case ready with many (100+) turbines. I will test it, but might take a few days.

With a debug run, there may not be a big performance increase given all the other single threaded parts of AWAE.  Thanks for testing @rthedin!


[Epic] GitHubDevelopersTrainingTeacherScripts - Powershell module to manage clsss repos for GitHub for Developers training

https://github.com/rulasg/GitHubDevelopersTrainingTeacherScripts/releases


Tag v1.1.1

Hey. Thank you for your open source work. Could you please add a git tag for v1.1.1?
#44

@mightyiam , I have created a tag for v1.1.1 . Please let me know if I should stop pushing changes directly to master (if that would break something for you), and only push from a PR.

or let me release with tags, so that you could point to the tags instead of master directly

Thank you, @ganezdragon. I recommend automated releases. Consider taking a look at semantic-release.