g++ cannot find sys/auxv.h on ubuntu 18.04

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
Without conda, and desired behaviour, albeit with system g++
ubuntu-18.04$ which g++
/usr/bin/g++

ubuntu-18.04$ g++ -V
g++: error: unrecognized command line option ‘-V’
g++: fatal error: no input files
compilation terminated.

ubuntu-18.04$ g++ --version
g++ (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

ubuntu-18.04$ echo '#include <sys/auxv.h>' | g++ -c -x c++ -

ubuntu-18.04$

With conda
ubuntu-18.04$ conda activate base
(base) ubuntu-18.04$ conda update conda
Collecting package metadata (current_repodata.json): done
Solving environment: done

# All requested packages already installed.

(base) ubuntu-18.04$ conda -V
conda 22.11.1

(base) ubuntu-18.04$ conda create -n gcc5
Collecting package metadata (current_repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /home/awf/miniconda3/envs/gcc5

Proceed ([y]/n)?

Preparing transaction: done
Verifying transaction: done
Executing transaction: done
#
# To activate this environment, use
#
#     $ conda activate gcc5
#
# To deactivate an active environment, use
#
#     $ conda deactivate

(base) ubuntu-18.04$ conda activate gcc5

(gcc5) ubuntu-18.04$ conda install cxx-compiler
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /home/awf/miniconda3/envs/gcc5

  added / updated specs:
    - cxx-compiler

The following NEW packages will be INSTALLED:

  _libgcc_mutex      conda-forge/linux-64::_libgcc_mutex-0.1-conda_forge
  _openmp_mutex      conda-forge/linux-64::_openmp_mutex-4.5-2_gnu
  binutils           conda-forge/linux-64::binutils-2.39-hdd6e379_1
  binutils_impl_lin~ conda-forge/linux-64::binutils_impl_linux-64-2.39-he00db2b_1
  binutils_linux-64  conda-forge/linux-64::binutils_linux-64-2.39-h5fc0e48_11
  c-compiler         conda-forge/linux-64::c-compiler-1.5.2-h0b41bf4_0
  cxx-compiler       conda-forge/linux-64::cxx-compiler-1.5.2-hf52228f_0
  gcc                conda-forge/linux-64::gcc-11.3.0-h02d0930_11
  gcc_impl_linux-64  conda-forge/linux-64::gcc_impl_linux-64-11.3.0-hab1b70f_19
  gcc_linux-64       conda-forge/linux-64::gcc_linux-64-11.3.0-he6f903b_11
  gxx                conda-forge/linux-64::gxx-11.3.0-h02d0930_11
  gxx_impl_linux-64  conda-forge/linux-64::gxx_impl_linux-64-11.3.0-hab1b70f_19
  gxx_linux-64       conda-forge/linux-64::gxx_linux-64-11.3.0-hc203a17_11
  kernel-headers_li~ conda-forge/noarch::kernel-headers_linux-64-2.6.32-he073ed8_15
  ld_impl_linux-64   conda-forge/linux-64::ld_impl_linux-64-2.39-hcc3a1bd_1
  libgcc-devel_linu~ conda-forge/linux-64::libgcc-devel_linux-64-11.3.0-h210ce93_19
  libgcc-ng          conda-forge/linux-64::libgcc-ng-12.2.0-h65d4601_19
  libgomp            conda-forge/linux-64::libgomp-12.2.0-h65d4601_19
  libsanitizer       conda-forge/linux-64::libsanitizer-11.3.0-h239ccf8_19
  libstdcxx-devel_l~ conda-forge/linux-64::libstdcxx-devel_linux-64-11.3.0-h210ce93_19
  libstdcxx-ng       conda-forge/linux-64::libstdcxx-ng-12.2.0-h46fd767_19
  sysroot_linux-64   conda-forge/noarch::sysroot_linux-64-2.12-he073ed8_15

Proceed ([y]/n)?

Downloading and Extracting Packages

Preparing transaction: done
Verifying transaction: done
Executing transaction: done

(gcc5) ubuntu-18.04$ which g++ # good outcome
/home/awf/miniconda3/envs/gcc5/bin/g++

(gcc5) ubuntu-18.04$ g++ --version  # good outcome - version 11.3.0
g++ (conda-forge gcc 11.3.0-19) 11.3.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

(gcc5) ubuntu-18.04$ echo '#include <sys/auxv.h>' | g++ -c -x c++ - # bad outcome
<stdin>:1:10: fatal error: sys/auxv.h: No such file or directory
compilation terminated.

With conda, sys/auxv.h is not found.
.condarc is:
(gcc5) ubuntu-18.04$ cat ~/.condarc
channels:
  - conda-forge
  - defaults
channel_priority: strict

Installed packages
gcc5) ubuntu-18.04$ conda list
# packages in environment at /home/awf/miniconda3/envs/gcc5:
#
# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                 conda_forge    conda-forge
_openmp_mutex             4.5                       2_gnu    conda-forge
binutils                  2.39                 hdd6e379_1    conda-forge
binutils_impl_linux-64    2.39                 he00db2b_1    conda-forge
binutils_linux-64         2.39                h5fc0e48_11    conda-forge
c-compiler                1.5.2                h0b41bf4_0    conda-forge
cxx-compiler              1.5.2                hf52228f_0    conda-forge
gcc                       11.3.0              h02d0930_11    conda-forge
gcc_impl_linux-64         11.3.0              hab1b70f_19    conda-forge
gcc_linux-64              11.3.0              he6f903b_11    conda-forge
gxx                       11.3.0              h02d0930_11    conda-forge
gxx_impl_linux-64         11.3.0              hab1b70f_19    conda-forge
gxx_linux-64              11.3.0              hc203a17_11    conda-forge
kernel-headers_linux-64   2.6.32              he073ed8_15    conda-forge
ld_impl_linux-64          2.39                 hcc3a1bd_1    conda-forge
libgcc-devel_linux-64     11.3.0              h210ce93_19    conda-forge
libgcc-ng                 12.2.0              h65d4601_19    conda-forge
libgomp                   12.2.0              h65d4601_19    conda-forge
libsanitizer              11.3.0              h239ccf8_19    conda-forge
libstdcxx-devel_linux-64  11.3.0              h210ce93_19    conda-forge
libstdcxx-ng              12.2.0              h46fd767_19    conda-forge
sysroot_linux-64          2.12                he073ed8_15    conda-forge

Environment info
(gcc5) ubuntu-18.04$ conda info

     active environment : gcc5
    active env location : /home/awf/miniconda3/envs/gcc5
            shell level : 2
       user config file : /home/awf/.condarc
 populated config files : /home/awf/.condarc
          conda version : 22.11.1
    conda-build version : not installed
         python version : 3.8.15.final.0
       virtual packages : __archspec=1=x86_64
                          __glibc=2.27=0
                          __linux=5.4.0=0
                          __unix=0=0
       base environment : /home/awf/miniconda3  (writable)
      conda av data dir : /home/awf/miniconda3/etc/conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/conda-forge/linux-64
                          https://conda.anaconda.org/conda-forge/noarch
                          https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/awf/miniconda3/pkgs
                          /home/awf/.conda/pkgs
       envs directories : /home/awf/miniconda3/envs
                          /home/awf/.conda/envs
               platform : linux-64
             user-agent : conda/22.11.1 requests/2.28.1 CPython/3.8.15 Linux/5.4.0-64-generic ubuntu/18.04.6 glibc/2.27
                UID:GID : 2753:501
             netrc file : None
           offline mode : False

You need to install sysroot_linux-64>=2.16 to get sys/auxv.h.

Thanks, that helps me.
Would it be possible for the install to warn if the system libc is newer than the cxx-compiler one?


Fix dedimania

Has multiple issues, for example when having multiple players online and playing.

(https://github.com/ManiaJS/plugins)


Remove deprecation symfony 6.3

This is to remove deprecation notices on the Symfony 6.3 CI and adding this type should not break the bundle because all of them are on internal elements

Thanks !
Inside the Lexik\Bundle\JWTAuthenticationBundle\Security\Authenticator
Symfony\Component\Security\Http\Authenticator\Passport\PassportInterface does not exists anymore since SF 6.0
You should use directly Symfony\Component\Security\Http\Authenticator\Passport\Passport instead

@LaYoance Yes but it's only used for the compatibility of createAuthenticatedToken method. This is not a problem in this configuration, but if we change it, it will break the compatibility with the interface

Well, I understand that.
An old class in our project was using it. I've updated it on my side to match the new one.
Thanks

Good catch, thanks Maxime.

Symonfy 6.3.0-BETA1 has been released, any chance this change could also make it to a release?


DllNotFoundException: Msacm32.dll when I use NAudio in Unity of Mac OS

Hi,
when I want to use NAudio to convert mp3 into wav, the error message jumped out on my Unity log window. The message is in the following lines.
DllNotFoundException: Msacm32.dll
NAudio.Wave.Compression.AcmStream.SuggestPcmFormat (NAudio.Wave.WaveFormat compressedFormat)
NAudio.Wave.AcmMp3FrameDecompressor..ctor (NAudio.Wave.WaveFormat sourceFormat)
NAudio.Wave.Mp3FileReader.CreateAcmFrameDecompressor (NAudio.Wave.WaveFormat mp3Format)
NAudio.Wave.Mp3FileReader..ctor (System.IO.Stream inputStream, NAudio.Wave.FrameDecompressorBuilder

I found someone has the same problem as mine.
https://answers.unity.com/questions/1224167/dllnotfoundexception-msacm32dll.html

hi, NAudio contains many wrappers for Windows APIs, and the ACM APIs which are found in msacm32.dll are part of that. It means you can't use them from non-Windows operating systems.
By default the Mp3FileReader tries to use ACM, but you can plug in a fully managed MP3 decoder if you prefer. See my article here

I can confirm that the linked work-around works (although the article is from 2017, which does not inspire me with confidence). As of writing, the sample code works with NLayer.NAudioSpport 1.13.0, and looks similar to the article code:
var fileName = @"C:\Users\markh\Desktop\input.mp3";
var builder = new Mp3FileReader.FrameDecompressorBuilder(wf => new Mp3FrameDecompressor(wf));
using (var reader = new Mp3FileReader(fileName, builder))
{
    WaveFileWriter.CreateWaveFile(@"C:\Users\markh\Desktop\output.wav",reader);
}

From the comments above and no mention on the prerequisites page the library is not supported on Mac OSX?

Any news to be supported on Mac OSX?

Afraid I have no plans to support Mac OSX. I do not own a Mac computer. Someone else could always make an NAudio extension that implements interfaces like IWavePlayer and make the Mac equivalents of things like MediaFoundationReader.

I am also interested in using NAudio in a Mac computer.
Can you detail exactly which interfaces should be implemented to get a full mp3/wav decoding and audio playback specific to a target OS?
I need to know up to which point NAudio is generic and where we need to create concrete stuff for the target OS.
Thank you very much for the great job you did for Audio support in .NET core :) your library is simply amazing

You can read WAV files no problem cross platform as they almost always contain raw PCM.
NLayer can still be used for MP3 file reading - just use Mp3FileReaderBase instead as in this example
For more comprehensive Mac support I'd implement the equivalent of MediaFoundationReader that uses whatever Mac APIs support reading files of various types

I did all the above, also changed to Mp3FileReaderBase but the error still persists. Any idea why?
I'm using it in a .NET Core project. The error was "Unable to load shared library 'Msacm32.dll' or one of its dependencies".

Are you using WaveFormatConversionStream as well? If so, you don't need to use that


Allow GLES worker to be renamed

It would be convenient if the GLES worker could be renamed without requiring a re-install of the APK, or if there was a convenient way to do the required reinstall.
Perhaps having a script that uninstalls and then installs the APK with a given worker name could be a good solution, if that's possible?

How about:

have the app read worker and server info from a file
revert to the pop-up for this to be entered manually if file not found
have a script that takes apk, worker name and server URL as arguments and (a) uninstalls existing app if present, (b) installs the app, (c) makes file with worker name and server info available.

Too low priority for fixing.


No option available to logout and again login to app with same account

Bug Description
No option available to logout and again login to app with same account
Steps to Reproduce

Click on Log off icon on at top right corner
Only reset option is displayed

Actual Result
No option available to logout and again login to app with same account
Expected Result
When user clicks on log off button it should display two option 1. Reset Account 2. Logout from app
And when user log out user should have login option with same account
OS version
Windows
Screenshot/Recording

This is part of the import/export profile discussion.


httpd: WebSockets

WebSockets implementation on top of httpd. Binary and text modes are supported, opening multiple sockets is possible. Continuation frames are not implemented. Example code demonstrates streaming ADC readings, and controlling LED.

Thank you, great contribution!
Will merge it tomorrow if no objections.

Thanks. It just so happens that I've added large frames support recently, so I'd appreciate if you merge my next commit.


Bug: Runtime Type Error in Cache-Parser Where Header Token Contains no ,

What happened?
This code: https://github.com/arthurfiorette/tinylibs/blob/main/packages/cache-parser/src/util.ts#L18 , which this library seems to depend on, has a bug.
(I was very relieved to see you manage both libraries, so thank you for the work you do :). )
My very quick glance is the problem is the code seems to rely on the fact that , is in the header of the string that is passed to it. I dont think that assumption can always hold true, so sometimes the token ends up being null or a type other than string, so when .split is called on it, there becomes an uncaught runtime error. The split on line 22 is the culprit.
I don't know much more of the context of the code here and there might be some other reason, but that is my suspicion.
Example is https://github.com/dotnet/vscode-dotnet-runtime/issues/1734.
I don't have a repro, this info is garnered from customers.

It is causing some issues for us over in VS Code land and it is one of our highest causes of failure, causing roughly .01% out of several million customers to fail, with a high report rate on mac os sonoma silicon m1.
axios-cache-interceptor version
^1.0.1
Node / Browser Version
16
Axios Version
^1.3.4
What storage is being used
Memory Storage
Relevant debugging log output
Failed: n[r].split is not a function. Aborting. Stack: TypeError: n[r].split is not a function
    at /usr/local/google/home/USER/.vscode-insiders/extensions/ms-dotnettools.vscode-dotnet-runtime-2.0.5/javascript/esm|c:\Users\USER\vscode-dotnet-runtime\vscode-dotnet-runtime-library\node_modules\cache-parser\dist\index.mjs:1:425
    at parse (/usr/local/google/home/USER/.vscode-insiders/extensions/ms-dotnettools.vscode-dotnet-runtime-2.0.5/javascript/esm|c:\Users\USER\vscode-dotnet-runtime\vscode-dotnet-runtime-library\node_modules\cache-parser\dist\index.mjs:1:327)
    at Function.headerInterpreter (/usr/local/google/home/USER/.vscode-insiders/extensions/ms-dotnettools.vscode-dotnet-runtime-2.0.5/javascript/dynamic|c:\Users\USER\vscode-dotnet-runtime\vscode-dotnet-runtime-library\node_modules\axios-cache-interceptor\dist\index.cjs:2:1324)
    at n (/usr/local/google/home/USER/.vscode-insiders/extensions/ms-dotnettools.vscode-dotnet-runtime-2.0.5/javascript/dynamic|c:\Users\USER\vscode-dotnet-runtime\vscode-dotnet-runtime-library\node_modules\axios-cache-interceptor\dist\index.cjs:2:5445)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Pt.request (/usr/local/google/home/USER/.vscode-insiders/extensions/ms-dotnettools.vscode-dotnet-runtime-2.0.5/javascript/dynamic|c:\Users\USER\vscode-dotnet-runtime\vscode-dotnet-runtime-library\node_modules\axios\dist\node\axios.cjs:3872:7)
    at Pt.request (/usr/local/google/home/USER/.vscode-insiders/extensions/ms-dotnettools.vscode-dotnet-runtime-2.0.5/javascript/dynamic|c:\Users\USER\vscode-dotnet-runtime\vscode-dotnet-runtime-library\node_modules\axios\dist\node\axios.cjs:3877:41)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

cc @baronfel

Hey @nagilson, can you setup a small reproducible example or paste here a Cache-Control header exemple value that throws the above error?

Thank you so much for your very fast reply.
May you help me gather that information for you? The header seems to be generated from Axios-Cache-Interceptor, so I'm not sure where it's generated. Is there something we could type in the developer console to access that information?
The web request this is for is one for: https://dot.net/v1/dotnet-install.sh
with these options: https://github.com/nagilson/vscode-dotnet-runtime/blob/a77568a7b983d3565bbb571d31ec550442c953b5/vscode-dotnet-runtime-library/src/Utils/WebRequestWorker.ts#L168.
It does not always happen, in fact I have not been able to get it to happen.

Source:
const tokens = headerStr.toLowerCase().replace(/\s+/g, '').split(',');

for (const token of tokens) {
  const split = token.split('=', 2) as [string, string];
  headers[split[0]] = split[1] ?? true;
}

Transpiled:
var a = {},
  r = e.toLowerCase().replace(/\s+/g, '').split(',');

for (var t in r) {
  var s,
    n = r[t].split('=', 2);
  a[n[0]] = null == (s = n[1]) || s;
}

r is the result of e.toLowerCase().replace(/\s+/g, '').split(',') which shouldn't be something else than an array, then it gets iterated in a for (var t in r) loop.
Normally, r[t] cannot be undefined:

Am I missing something or this seems to be a bug in the js runtime?

@arthurfiorette The more I am looking at the code, I am not seeing what value would cause this to happen yet either. One sec...

Maybe String.prototype.split was modified by another library?


Fix 'Unsupported log file format' error by updating Fake.DotNet.cli

Summary
I just did a fresh clone and tried to build using README command
dotnet fake build -t Dev

And I ran into this error

Updating Fake.DotNet.cli resolved the issue
Changes

Update Fake.DotNet.cli package to 5.20

dotnet paket update Fake.DotNet.cli -g fakebuild

I also updated Fable.Elmish.Debugger that updated Thoth.Json to resolve the build error below
FSC : error FS0193: The module/namespace 'Encode' from compilation unit 'Thoth.Json' did not contain the val 'ValLinkagePartialKey(generateEncoder)' [D:\code\fsharp\github\fable-elmish-electron-material-ui-demo\src\Renderer\Renderer.fsproj]

My advice would be to 'Build' the solution in the default target to avoid merging code that don't compile

Thanks! Not sure what you meant by that last comment. AppVeyor already builds PRs, and the build for this PR seems to compile. Are we good to merge then, or did you mean something else?

As a side note, I have just updated Fable.Electron for Electron 9, so I will be updating all other packages (npm and nuget) soon, too.

I meant running dotnet build on the solution, before my last commit dotnet build was failing but appveyor build didn't see anything

Ok, I'll let the build script be the boss, that's the supported way to build. :) Thanks a lot!


ci: conventional-commits validation

add conventional commits to GitHub workflows

accidentally open PR to main instead of 1.3, will close


standardize test macro parameters 'where', 'condition', 'from_condition', 'to_condition', ...

Describe the feature
Standardize names of kwargs for conditions to select rows in test macros, and use either explicit argument or kwargs.
Describe alternatives you've considered
Additional context
Currently, there are several test macros that allow the user to implement tests only for a subset of rows, which is really helpful. Only when writing my own macros that do that, I found it difficult to pin down the right naming convention, since macros follow different conventions, including:

The macro test_expression_is_true has condition as an explicit macro variable
The test_*_where macros define these conditions as kwargs, but the naming is not universal: most have where, but test_relationships_where has from_condition and to_condition.

It would be helpful if there were a general consensus around whether kwargs or explicit variables are the way to define these conditions, and how to name them in a way that makes it obviously generalizable to tests involving multiple models
Who will this benefit?
Users of test macros can more easily guess which optional parameters may be present if the names are standardized.
Are you interested in contributing this feature?
Yes, though I would definitely need input since it involves naming conventions

@marianneke this'd be great! It could probably even be backwards compatible by switching all other arguments to be where (for the sake of this conversation, but I agree picking a standardised name would benefit from wider input) and then deprecating the old argument names in a similar way to the unpivot macro


Port to Net Standard 2

Hi,
I think these extensions are still pretty useful and i would like to use them in net core apps.
The Following references are troubling:
Gridsum.DataflowEx.Utils  ->
Generic Types from mscorlib
System.Data.DataColumnCollection (used by Databases)
System.Diagnostics.Stackframe with Common.Logging.LogManager
Gridsum.DataflowEx.Databases ->
System.Data.SqlClient
Gridsum.DataflowEx.AutoCompletion -> System.Timers (should be easy)
(C5 -> recently updated for net standard but nuget out of date)
@karldodd any updates from you #3 ?

@w9n Thanks for your comments and the list for troubling references.
Yes, I do have the plan to migrate to .Net Core and is somehow waiting for .net standard 2.0, which I guess will make the migration a lot easier. I will also look at whether some old stuff should be replaced by modern libs.
Are u OK with this library targeting net standard 2, or u need it to be 1.x?

@karldodd I was kind of confused of the net standard 2 process, i need a .net core/4.6 compatible version but experienced that this complicated atm. I hacked together a branch with net standard 1.6, with minor api changes and also had to delete all Database related code to make it work for my purposes.
For now we should wait as standard 2 seems to be at 96% progress :)

@w9n This is almost done now with .Net Core preview 2 ! Please try out the preview release:
https://www.nuget.org/packages/Gridsum.DataflowEx/2.0.0-alpha

When can we expect the merge of #13 and release availability of the package for .NET Core?
.NET Core 2.0 is available: https://blogs.msdn.microsoft.com/dotnet/2017/08/14/announcing-net-core-2-0/

@karldodd im currently not working with .net and cant properly check out the port. Feel free to close the issue if you think its ready.

@Lakritzator I've released a new version (2.0.0-beta) of the package today which now uses .net core 2 rtm and corresponding libraries/references are also upgraded. Please try it out if you need.
I think #13 is almost done and will be merged soon if 2.0.0-beta works smoothly.


Update vaulted files with new vault password

trying again to rekey vaulted files with new ansible vault password
Now using lastpass cli based on what PUL is doing; refer to their setup instructions here: https://github.com/pulibrary/princeton_ansible#automatically-pull-vault-password-from-lastpass
I haven't updated the docs yet and we should, but I'm going to modify this approach slightly when I split out the geniza credentials — I think we won't need the lastpass-ansible gem. LMK what you think needs to be in our local documentation
FYI: AFAICT, when you tell ansible to prompt for a password but have a default password file configured, if you give it a bad password but the default password file is valid, it will decrypt. I had to remove my env var for the password file to be able to test properly that I could decrypt things with the new key and not the old one. (It was decrypting unexpectedly when I entered the old key at the prompt, I presume because it was falling back to the default file.)

Confirmed that old password fails and new password works.


Performance profiling

tldr:

CIDs goes via multibase which a bit expensive
~Ipld allocated and dropped with short living time~ fixed in #8
creaton of PyString, PyDict, calling set_item is expensive. pyo3 part
decoding dag-cbor strings, bytes rust native libipld part

flame graph with debug (from #6):
SVG direct link (Gist)

new flame graph after fixes in #8: https://gist.github.com/MarshalX/542a87255bbd446dfafb6e74a517581c

codspeed has been integrated. flamegraph available for each pull request now.


simplify resnet definition by reusing struct template parameter

I realized that I was using the template parameter for bn_con and affine twice: once for the struct template and one for each template that needed a batch normalization layer inside the struct.
Now everything uses the top level template parameter, leading to more readable code :)

I was thinking also about doing something like this to allow using namespace dlib;:
namespace resnet
{
    using namespace dlib;
    template<template<typename> class BN>
    struct impl
    {
        // resnet definition here
    }
    using train_18 = impl<bn_con>::n18;
    using train_34 = impl<bn_con>::n34;
    using train_50 = impl<bn_con>::n50;
    using train_101 = impl<bn_con>::n101;
    using train_152 = impl<bn_con>::n152;

    using infer_18 = impl<affine>::n18;
    using infer_34 = impl<affine>::n34;
    using infer_50 = impl<affine>::n50;
    using infer_101 = impl<affine>::n101;
    using infer_152 = impl<affine>::n152;
}

Then, on the calling side it would be more convenient, too:
resnet::train_50 net;

I have already these changes locally, if you want I will push them.
The goal would be to have a generic resnet.h file that people can copy and use.
I've also noticed that it would be nice to add tagged layers after each resnet level... I need to check how to do it cleanly :)

Yeah that sounds like a good idea too.  Feel free to put that into this PR as well :)

Thanks for another nice PR :)


Error 500

Using the latest version (dev-master through Composer or the latest phar) I get the following error:
[GuzzleHttp\Exception\ServerException]
Server error response [url] /build?q=1&t=543b02a31d5da-php-5.3&nocache=0&rm=0 [status code] 500 [reason phrase] Internal Server Error

Above that, my console is flooded with PHP notices:
Notice: fwrite(): send of 5 bytes failed with errno=32 Broken pipe in phar:///myproject/jolici.phar/vendor/stage1/docker-php/src/Docker/Http/Adapter/DockerAdapter.php on line 96

I tried debugging, I moved into the temp directory and ran docker build . => it works (runs the tests just fine). By the way, the travis.yml is ultra basic. So it seems there's a problem connecting to docker.
I have installed Docker on OS X with boot2docker, using Homebrew. Docker works fine.
I also tried jolici run --docker-host="tcp://<IP_ADDRESS>:2375" but I get the same result ($DOCKER_HOST is correctly defined by the way).
Any idea?

Closing as this should be good with last release

Thanks for the report, don't hesite to reopen this issue if this error happens again


Easier settings handling with dotenv

Major settings moved to settings.py for central distribution
Integrated python-dotenv for ease of use, especially when working with non-UNIX OS :)

:)

very nice you make it better and better - thanks


What kind of infrastructure will this require if I wanted it to run AWS ?

Use AWS EC2 g5 instance.


[github] Add Deno publish to autorelease script

[x] I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested
Adds publishing to openai-deno-build (and by proxy, https://deno.land/x/openai) a part of our release pipeline. This means every new release will automatically be published to Deno.
I tested this works by creating a manual workflow to publish to Deno, and testing it works.
Additional context & links

@athyuttamre FYI I've incorporated these changes and your changes from the other PR in this release https://github.com/openai/openai-node/pull/520

Sounds good, will close this PR and merge that instead.


Allow the upscale but block the downscale in clusterQueue

What would you like to be added:
We should :

Allow the upscale in clusterQueue, like

[ ] Add new resources/flavors
[ ] Scale up the size of flavors

Block the downscale in clusterQueue, like

[ ] Remove resources, including flavors unless the flavor is not in use.
[ ] Scale down the size of flavors unless the flavor is not in use.

We can check that in cq.UsedResources.
Why is this needed:
More scaleable.
Completion requirements:
This enhancement requires the following artifacts:

[x] Design doc
[ ] API change
[ ] Docs update

The artifacts should be linked in subsequent comments.

I don't think we should block downscale, or at least it should be more flexible.
Once we have preemption support, when downscaling, we could start suspending some workloads if the quota is being violated.
Maybe we could look at the current state of the cache and only allow dowscale if the quota wouldn't be violated with the active workloads, as you say. However, we would be limiting the ability of a cluster administrator to quickly scale down resources if there is some sort of emergency.

Well, it's just a startup and based on the current implements. Maybe we can delay this until we implement the preemption.

/lifecycle frozen
/priority backlog


Diseases Restored Still Broken - Crash Before Main Menu with a Patching Exception Error

Here's the crash log.  Player.log file also attached.
Exception while loading mod DiseasesReimagined at D:/Media/Documents/Klei/OxygenNotIncluded/mods/Steam/1911357229.
HarmonyLib.HarmonyException: Patching exception in method null ---> System.ArgumentException: Undefined target method for patch method static System.Void DiseasesReimagined.BaseMinionConfig_CreatePrefab_Patch::Postfix(UnityEngine.GameObject __result)
  at HarmonyLib.PatchClassProcessor.PatchWithAttributes (System.Reflection.MethodBase& lastOriginal) [0x00047] in <6dcb326e4f6442999f701f1e67d0b5a0>:0
  at HarmonyLib.PatchClassProcessor.Patch () [0x00068] in <6dcb326e4f6442999f701f1e67d0b5a0>:0
   --- End of inner exception stack trace ---
  at HarmonyLib.PatchClassProcessor.ReportException (System.Exception exception, System.Reflection.MethodBase original) [0x00127] in <6dcb326e4f6442999f701f1e67d0b5a0>:0
  at HarmonyLib.PatchClassProcessor.Patch () [0x00093] in <6dcb326e4f6442999f701f1e67d0b5a0>:0
  at HarmonyLib.Harmony.<PatchAll>b__10_0 (System.Type type) [0x00007] in <6dcb326e4f6442999f701f1e67d0b5a0>:0
  at HarmonyLib.CollectionExtensions.Do[T] (System.Collections.Generic.IEnumerable`1[T] sequence, System.Action`1[T] action) [0x00014] in <6dcb326e4f6442999f701f1e67d0b5a0>:0
  at HarmonyLib.Harmony.PatchAll (System.Reflection.Assembly assembly) [0x00006] in <6dcb326e4f6442999f701f1e67d0b5a0>:0
  at KMod.UserMod2.OnLoad (HarmonyLib.Harmony harmony) [0x00007] in <44bf0e3611d3428cb5e183976b74cab9>:0
  at DiseasesReimagined.DiseasesPatch.OnLoad (HarmonyLib.Harmony harmony) [0x0007d] in <9e51f691995b4d03a0756436151e8355>:0
  at KMod.DLLLoader.LoadDLLs (KMod.Mod ownerMod, System.String harmonyId, System.String path, System.Boolean isDev) [0x00222] in <44bf0e3611d3428cb5e183976b74cab9>:0

  at UnityEngine.Debug.LogError (System.Object message, UnityEngine.Object context) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
  at Debug.LogError (System.Object obj, UnityEngine.Object context) [0x00000] in <4e08f3a60ed4421b99a6798f854897c4>:0
  at DebugUtil.LogErrorArgs (UnityEngine.Object context, System.Object[] objs) [0x00000] in <4e08f3a60ed4421b99a6798f854897c4>:0
  at DebugUtil.LogException (UnityEngine.Object context, System.String errorMessage, System.Exception e) [0x00000] in <4e08f3a60ed4421b99a6798f854897c4>:0
  at KMod.DLLLoader.LoadDLLs (KMod.Mod ownerMod, System.String harmonyId, System.String path, System.Boolean isDev) [0x00000] in <44bf0e3611d3428cb5e183976b74cab9>:0
  at KMod.Mod.Load (KMod.Content content) [0x00000] in <44bf0e3611d3428cb5e183976b74cab9>:0
  at KMod.Manager.Load (KMod.Content content) [0x00000] in <44bf0e3611d3428cb5e183976b74cab9>:0
  at Global.Awake () [0x00000] in <44bf0e3611d3428cb5e183976b74cab9>:0
  at UnityEngine.Object.Internal_InstantiateSingleWithParent_Injected (UnityEngine.Object , UnityEngine.Transform , UnityEngine.Vector3& , UnityEngine.Quaternion& ) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
  at UnityEngine.Object.Internal_InstantiateSingleWithParent (UnityEngine.Object data, UnityEngine.Transform parent, UnityEngine.Vector3 pos, UnityEngine.Quaternion rot) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
  at UnityEngine.Object.Instantiate (UnityEngine.Object original, UnityEngine.Vector3 position, UnityEngine.Quaternion rotation, UnityEngine.Transform parent) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
  at UnityEngine.Object.Instantiate[T] (T original, UnityEngine.Vector3 position, UnityEngine.Quaternion rotation, UnityEngine.Transform parent) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
  at Util.KInstantiate (UnityEngine.GameObject original, UnityEngine.Vector3 position, UnityEngine.Quaternion rotation, UnityEngine.GameObject parent, System.String name, System.Boolean initialize_id, System.Int32 gameLayer) [0x00000] in <4e08f3a60ed4421b99a6798f854897c4>:0
  at Util.KInstantiate (UnityEngine.GameObject original, UnityEngine.GameObject parent, System.String name) [0x00000] in <4e08f3a60ed4421b99a6798f854897c4>:0
  at LaunchInitializer.Update () [0x00000] in <44bf0e3611d3428cb5e183976b74cab9>:0
Build: U53-644960-SCD

Player.log

Should be resolved by https://github.com/skairunner/sky-oni-mods/pull/117


Calling reset raises TypeError: expected str, bytes or os.PathLike object, not NoneType

Shouldn't reset function use the same signature as the __init__?
Current reset https://github.com/acheong08/EdgeGPT/blob/master/src/EdgeGPT.py#L263 uses:
self.chat_hub = ChatHub(Conversation())

However, __init__ https://github.com/acheong08/EdgeGPT/blob/master/src/EdgeGPT.py#L235 uses:
self.chat_hub: ChatHub = ChatHub(Conversation(cookiePath))

I think it should use the same path used on object creation; it's looking for environment variable (fallback maybe?).
Traceback:
raceback (most recent call last):
  File "/home/capitano/telegrambots/Edge-GPT-Telegram-Bot/venv/lib/python3.8/site-packages/telegram/ext/_application.py", line 1104, in process_update
    await coroutine
  File "/home/capitano/telegrambots/Edge-GPT-Telegram-Bot/venv/lib/python3.8/site-packages/telegram/ext/_handler.py", line 141, in handle_update
    return await self.callback(update, context)
  File "./edge.py", line 40, in new
    await ut.CONV[cid][0].reset()
  File "/home/capitano/telegrambots/Edge-GPT-Telegram-Bot/venv/lib/python3.8/site-packages/EdgeGPT.py", line 263, in reset
    self.chat_hub = ChatHub(Conversation())
  File "/home/capitano/telegrambots/Edge-GPT-Telegram-Bot/venv/lib/python3.8/site-packages/EdgeGPT.py", line 133, in __init__
    f = open(os.environ.get("COOKIE_FILE"), encoding="utf-8").read()
TypeError: expected str, bytes or os.PathLike object, not NoneType

Yes. Needs to be updated. Thanks


AWS High Availability Architecture

Closes #25
Checklist:

[x] Added activities for learners
[x] Didn't give the solutions to the added activities straight away
[x] Followed a LearnByDoing approach
[x] Clean and well documented code (if source code files are involved)
[x] Added images for better understanding if deemed necessary

Re - Earlier Issues:
Hey @Nitesh-thapliyal!
You have made most of the changes requested by me, but you might have missed out a few so I am gonna reiterate it for you:
In Prerequisites, you are assuming that it is a Windows user. What if he uses Linux/Mac? Provide links for these too.
You still have not mentioned what you need in the HTML in activity 7. Tell the learner how big the html should be (Just a simple boilerplate or does it need to have anything)
You got rid of the part in the previous version when you were launching the website in terms of activity 11 (Add that part but in Activity 10).
Apart from all these changes, very good work, man.
Makes these changes as soon as possible.

@all-contributors kindly add @Nitesh-thapliyal for:

[x] code
[x] content
[x] ideas
[x] doc


[SPARK-18485][CORE] Underlying integer overflow when create ChunkedByteBufferOutputStream in MemoryStore

What changes were proposed in this pull request?
There is an underlying integer overflow when create ChunkedByteBufferOutputStream in MemoryStore. This PR provide a check before cast.
How was this patch tested?
add new unit test

Test build #68759 has started for PR 15915 at commit 8cd01d1.

Test build #68759 has finished for PR 15915 at commit 8cd01d1.

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

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

Merged build finished. Test PASSed.

Test build #68776 has started for PR 15915 at commit ba8ad2c.

Test build #68776 has finished for PR 15915 at commit ba8ad2c.

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

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

Merged build finished. Test PASSed.

To clarify, does this overflow only occur when spark.storage.unrollMemoryThreshold has been manually set to a huge value which doesn't fit into an int?
Setting the chunk size to Integer.MAX_VALUE probably isn't a good idea because it'll leave us OOM-prone. Instead, we should probably set some reasonable upper bound on the chunk size based on either the unsafe page size or something else.

@JoshRosen sounds like you don't disagree with this change because anything more than  Integer.MAX_VALUE is going to cause an outright failure, but mean it could impose a significantly smaller limit. Should we do that here? you probably know better, is the right limit to impose the page size? or just punt on that and fix the corner case error only here?

@JoshRosen I find some other usage of "chunkSize" in other code path, and we set it with 1024 * 1024 * 4. Is there any reason？ Now, I use it as default value for "chunkSize"

Test build #69068 has started for PR 15915 at commit 5210ce3.

@JoshRosen The main cause for this issue is the mix-up of "chunkSize" and other "size"

Test build #69068 has finished for PR 15915 at commit 5210ce3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #3435 has started for PR 15915 at commit 5210ce3.

Test build #3435 has finished for PR 15915 at commit 5210ce3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

@SparkQA  retest again

retest this please

Test build #69098 has started for PR 15915 at commit 5210ce3.

Haven't forgotten about this; it's on my list to review post-Thanksgiving.

Test build #69098 has finished for PR 15915 at commit 5210ce3.

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/69098/
Test PASSed.

Test build #69382 has started for PR 15915 at commit ad85da0.

Test build #69382 has finished for PR 15915 at commit ad85da0.

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/69382/
Test PASSed.

Is there any update? @JoshRosen

Test build #69460 has started for PR 15915 at commit fffd5f5.

Test build #69461 has started for PR 15915 at commit 7c3e2c7.

Test build #69461 has finished for PR 15915 at commit 7c3e2c7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #69460 has finished for PR 15915 at commit fffd5f5.

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/69460/
Test PASSed.

This introduces a whole new config property. I think the suggestion was to base it off of some existing property, like page size? @JoshRosen would be best positioned to comment on that. We can more narrowly fix the problem of exceeding Int.MaxValue as you did before if that is a non-trivial problem in real use.

@srowen +1 to your comment.

I think the suggestion was to base it off of some existing property, like page size?

Yes, introducing a new config property is a practice of lazy. As you said, we can reuse existing property. @JoshRosen can give some suggestion?

I'd use SparkEnv.memoryManager.pageSizeBytes (which is calculated at https://github.com/apache/spark/blob/bec0a9217b896a596c0a0919888da34589fc720d/core/src/main/scala/org/apache/spark/memory/MemoryManager.scala#L215).

@uncleGen can you follow up on this so we can finish this out?

@srowen Sorry for the delay, I will update it as soon as possible on the basis of comments.

Test build #70001 has started for PR 15915 at commit b021557.

Test build #70001 has finished for PR 15915 at commit b021557.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #70007 has started for PR 15915 at commit 45aeddb.

Test build #70008 has started for PR 15915 at commit 8551892.

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

Generally, there are three ways to fix this issue:

Check if chunkSize exceed Int.MaxValue to narrow this issue.
Provide a new config to set chunkSize.
Reuse existing config as chunkSize, like pageSize.

Basically, I don't like option 3, because users will never know the low level details and the effect to chunk size when modify spark.buffer.pageSize. Just like @viirya said:

spark.broadcast.blockSize has special meaning. I don't think we should replace it with pageSizeBytes

Besides, SparkEnv.get.memoryManager.pageSizeBytes returns Long, there is still underlying integer overflow issue.
Introducing a new config is also a good idea. So, I'd like to do some check for exceeding Int.MaxValue to narrow this issue.
Any suggestions will be appreciated.
@JoshRosen @srowen @viirya

I'd like option 1 that to add some checks and throw error message for users.

Test build #70063 has started for PR 15915 at commit 041992f.

Test build #70065 has started for PR 15915 at commit c48b8f9.

Merged build finished. Test FAILed.

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

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

Merged build finished. Test FAILed.

Test build #70083 has started for PR 15915 at commit b66b277.

Test build #70083 has finished for PR 15915 at commit b66b277.

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

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

Merged build finished. Test PASSed.

Test build #70115 has started for PR 15915 at commit d31d74f.

Can you also add a test into MemoryStoreSuite?

Test build #70118 has started for PR 15915 at commit 1567c73.

Discovery starting.
Discovery completed in 37 seconds, 984 milliseconds.
Run starting. Expected test count is: 10
MemoryStoreSuite:
- reserve/release unroll memory
- safely unroll blocks
- safely unroll blocks through putIteratorAsValues
- safely unroll blocks through putIteratorAsBytes
- set unrollMemoryThreshold a huge value larger than Int.MaxValue
- PartiallySerializedBlock.valuesIterator
- PartiallySerializedBlock.finishWritingToStream
- multiple unrolls by the same thread
- lazily create a big ByteBuffer to avoid OOM if it cannot be put into MemoryStore
- put a small ByteBuffer to MemoryStore
Run completed in 39 seconds, 91 milliseconds.
Total number of tests run: 10
Suites: completed 2, aborted 0
Tests: succeeded 10, failed 0, canceled 0, ignored 0, pending 0
All tests passed.

Test build #70115 has finished for PR 15915 at commit d31d74f.

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

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

Merged build finished. Test PASSed.

Current change looks good to me. cc @JoshRosen @srowen to check it again.

Test build #70118 has finished for PR 15915 at commit 1567c73.

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/70118/
Test PASSed.

It seems to do the job. It seems a little funny that if you specify a huge chunk size like 1GB, that's accepted, but if you specify ~2GB, it becomes 4MB. Is the logic that this probably happens when something else is left to a default value, so this is like a default value?
it seems more natural to clamp to Int.MaxValue if we believe that the caller really means to specify a huge chunk size. But it sounds like the argument is that this can never be sensible. Is there a lower hard limit to enforce?
That is: is it too much to just cap this at 4MB?

@srowen The truth is that chunk size is passed from different code path with different value. There is not a reasonable value when chunk size as a invalid value. Experiencedly, chunk size does not need a large value, although it can be. Besides, we can give it a special config, and restrict it a reasonable value, like not larger than 16MB or something else. But you said no to add new config.

You could also cap it at 16MB here. Or 128MB or whatever. That is unrelated to introducing a new config.
It seems funny that setting to allow Int.MaxValue but map anything else to something orders of magnitude smaller. Why not just cap at Int.MaxValue like in the original proposal? that's also less behavior change.

OK, I will give an update as soon as possible.

Test build #70191 has started for PR 15915 at commit 22b81cf.

Test build #70191 has finished for PR 15915 at commit 22b81cf.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #3500 has started for PR 15915 at commit 22b81cf.

Test build #3500 has finished for PR 15915 at commit 22b81cf.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

@srowen Looks like it is not very convenient to test this case, with java.lang.OutOfMemoryError: Requested array size exceeds VM limit in jenkins. Do you think it is necessary to test Int.MaxValue chunk size or we can add a unit test in the follow-up PR if necessary?

Because the actual change is far from complicated, if we can't have proper test case for it, can we skip the added test? @srowen @JoshRosen

There's still one comment outstanding, but other than that, yes I think it's fine to make this change without another additional test, because it's hard to test and because the change is such a clear, narrow fix.

OK, we reach a consensus, I will give a update as soon as possible.

Test build #70245 has started for PR 15915 at commit 2e23dd0.

Test build #70245 has finished for PR 15915 at commit 2e23dd0.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #70248 has started for PR 15915 at commit 6b892af.

Test build #3503 has started for PR 15915 at commit 6b892af.

Test build #3503 has finished for PR 15915 at commit 6b892af.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

There are some unrelated failures, retest it please.

Test build #70248 has finished for PR 15915 at commit 6b892af.

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/70248/
Test PASSed.

@srowen All tests passed,  take a look please.

LGTM

Merged to master


Update wakemeonlan.json - fix persisting per #6179

Persisting is presuming a directory not a file, needs this workaround to make it work, as per: https://github.com/ScoopInstaller/Scoop/issues/6179

Thanks for the PR!


(RGBA) Borders are just slightly larger shapes with a full fill

Regression?
No
Hyprlock Info and Version
0.4.1
 Hyprlock config

shape {
    monitor =
    size = 360, 60
    color = rgba(17, 17, 17, 0.15)
    rounding = -1
    border_size = 8
    border_color = rgba(0, 207, 230, 0.5)
    rotate = 0
    xray = false # if true, make a "hole" in the background (rectangle of specified size, no rotation)

    position = 0, 80
    halign = center
    valign = center
}

****
Compositor Info and Version
 System/Version info
Hyprland, built from branch  at commit 918d8340afd652b011b937d29d5eea0be08467f5  (flake.lock: update).
Date: Tue Jun 25 12:06:02 2024
Tag: v0.41.2, commits: 4886

flags: (if any)

System Information:
System name: Linux
Node name: joao-archlinux-lenovo
Release: 6.10.3-arch1-2
Version: #1 SMP PREEMPT_DYNAMIC Tue, 06 Aug 2024 07:21:19 +0000

GPU information:
63:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Phoenix1 [1002:15bf] (rev c7) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Description
The above snippet of my config draws an oblong shape that is meant to have a translucent border and a translucent background. However, it seems like the border is not actually a border but a slightly scaled up version of the defined shape and hence, the both colours end up mixing. Hence, borders do not work when using rgba.
How to reproduce
Include the shape widget into your config and lock the screen.
Crash reports, logs, images, videos
No response

bump

do we want gradient as well?

do we want gradient as well?

it doesn't fit in with my tastes, but why stop innovation just for me ^^

Yeah same. personally i wouldn't use it.
My plan is just to port the border shader from hyprland to hyprlock. That shader works with gradients, so it would be easy to just add them to hyprlock as well.

@fybx wanna test my pr?


Automatically update stable

When I run fvm use stable stable is not automatically updated to the latest stable version if it has been previously checked out.
I'll have to do something like fvm flutter upgrade to actually pull the latest stable version.
If the flutter version is set to stable I would expect it to always be the latest stable version and thus automatically update itself if this not the case.
I haven't checked, but I assume the same counts for beta and dev.

@rhalff That is a good suggestion, however this will probably not be implemented as it changes the expected behavior of the Flutter CLI, and the dev should be explicit about when upgrading..


Restyle the Slides

Do you have an eye for design?
Help us improve the look of the slide deck.
If you need help, check out the Appendix in the training manual.

I am new to this space.. I have no idea what I have gotten myself into


URL fix in the RDP article

One of the links in the article was broken due to the .md suffix in the link.

Thanks for your contribution @HankiDesign, it looks like this has already been update in issue: https://github.com/MicrosoftDocs/windowsserverdocs/issues/6940. Closing this PR.
Thanks once again,
Robin


0.15.7 changelog

Pull Request Checklist
Thank you for taking the time to improve Arrow! Before submitting your pull request, please check all appropriate boxes:

[ ] 🧪 Added tests for changed code.
[ ] 🛠 All tests pass when run locally (run tox or make test to find out!).
[ ] 🧹 All linting checks pass when run locally (run tox -e lint or make lint to find out!).
[ ] 📚 Updated documentation for changed code.
[x] ⏩ Code is up-to-date with the master branch.

If you have any questions about your code changes or any of the points above, please submit your questions along with the pull request and we will try our best to help!
Description of Changes
Prep 0.15.7 release.

Codecov Report

Merging #810 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #810   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines         1740      1742    +2
  Branches       295       300    +5
=========================================
+ Hits          1740      1742    +2

Impacted Files
Coverage Δ

arrow/_version.py
100.00% <100.00%> (ø)

arrow/arrow.py
100.00% <0.00%> (ø)

arrow/parser.py
100.00% <0.00%> (ø)

arrow/locales.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


#495 Run format lint in pre commit

Running format-lint first, and then run format-lint by per-commit hook before every committing. you will get error if any lint error happened.
After this, developers won't need to worry about forgetting format-lint before committing by manually.

 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.Raymans Peng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

It's better to review and merge this first, due to format-lint entire files, easily get conflicts if other changes happened.

This is awesome. Can't believe we had so many files escape linting. I hadn't even thought about linting test files, but that'll help a lot too.

I updated my node_modules and attempted to make a commit to this branch. Got the following message. Am I missing a step?
$ git commit -a -m "testing commit hook"
: No such file or directory 2: ./node_modules/pre-commit/hook
: integer expression expected: [: 127
: numeric argument required 5: exit: 0

@Raymans Thanks, awesome pull request, this will be really nice, we should also have something similar for the new app.

I've used a new fresh branch to install, build and commit which still worked perfectly, not sure what you went through.


Laravel 9

Deprecated: ZeroDaHero\LaravelWorkflow\Events\AnnounceEvent implements the Serializable interface, which is deprecated. Implement __serialize() and __unserialize() instead (or in addition, if support for old PHP versions is necessary)
runtime [php 8.1.2]
laravel components [9]
https://www.php.net/manual/en/migration81.deprecated.php

miss


Bug: PUT /teams/{id}/policies accept policies from other organizations

The endpoint should prevent us from adding policies from other organization to a team

Also add org as an header for all routes  (swagger will not show it otherwise)


Readme - Nuget Badge Addition

Readme - Nuget Badge Addition.
build.yml - OS tag update, ‘windows-latest’ -> ‘windows-2019’

Codecov Report

Merging #26 (cda47d0) into main (99acd92) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #26   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           80        80
  Branches         6         6
=========================================
  Hits            80        80

Impacted Files
Coverage Δ

ShadowRoot.Digger/ShadowRootAssist.cs
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Error could not find driver on productive instance

It works locally but as soon as I deploy it I get this error when calling the /cp/translations path:

The add-on requires SQLite, the server doesnt seem to have support installed for it

The add-on requires SQLite, the server doesnt seem to have support installed for it

Enabling sqlite as database driver worked for me.
@ryanmitchell how can one enable MySQL or generally the eloquent driver.

This add on uses an internal SQLite database. Eloquent support isn’t possible.
if you are asking how to use the eloquent driver generally see: https://github.com/statamic/eloquent-driver

I went through the code and found it's using Sushi hence the need for sqlite. I'm trying to see if it's possible to use a different storage for this. i.e. MySQL, Redis etc.

To be honest, I wouldn't be interested in supporting eloquent on this add-on. The purpose of using Sushi was it does a real time flat tile to simple eloquent transition. I don't think requiring SQLite is a big issue, its a very common PHP addon available on most installs.


Can goaccess output two or more output one time?

I want to get html output and csv output  one time.
How can I do it ?
Does goaccess support it?

This is not possible, it uses the standard output to print out a report. However, you should be able to run goaccess twice. What's the use case for this?

Thanks.
We want to see the result by html, and also want to get the data from csv to analyze by our data analyze server.
@allinurl

If you are using the on-disk storage, you could persist the data and parsing it again with goaccess should be pretty fast if you don't pass a new dataset. This is probably the fastest approach right now without consuming a lot of resources.
v1.0 is taking a different approach on the HTML output, it parses JSON raw data directly so it might be possible to generate a csv and output that as a blob.

Thanks.
Please close.

Added the ability to output multiple formats in one run.
In addition, it standardizes the output per issue #53. For instance,
it now supports the following.
goaccess -f access.log -o /www/file.html
goaccess -f access.log -o /www/file.csv
goaccess -f access.log -o /www/file.json
For backwards compatibility, it still supports the following format:
goaccess -f access.log > /www/file.html
goaccess -f access.log -o csv > /www/file.csv
goaccess -f access.log -o json > /www/file.json


Remove superfluous local variable setting.

The variable result was being set and returned, but the variable isn't being used elsewhere. That means it doesn't need to be set with ||=, just returned with ||.
Obvious fix.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@dreeve

Please sign the CLA here.

@jkeiser ?

Thanks for the contribution.  For future contributions "obvious fix" needs to be pat of the commit message - https://docs.chef.io/community_contributions.html#how-does-the-obvious-fix-rule-work
For this pull request, the existing assignment isn't harming anything, so unless there are strong feeling from @jkeiser or @btm, I'm going to close this.

@smurawski ok! No worries. Is there a better place to put "obvious fix"? It was in the commit message, but not on the first line; should it be on the first line?

@dreeve it was fine in the commit message, that's right. @smurawski probably missed it.

@dreeve Yeah, my bad.  I missed it.


✅ [11.x] Ask About secret for application DownCommand

This PR will assist you in entering maintenance mode without the need for commands such as php artisan down, while also inquiring about secrets.

Previously, you were limited to passing the secret argument for establishing a secret. With the growing popularity of the new prompts, this enhancement will facilitate the setup of a secret.

This is probably a bad idea to do this on a minor release, because there are probably many applications which do php artisan down and php artisan up in their deployment script. Which would now require interaction which will not happen and therefor breaking the deployment script. (At least this is my understanding of the prompting for any argument)

This is probably a bad idea to do this on a minor release, because there are probably many applications which do php artisan down and php artisan up in their deployment script. Which would now require interaction which will not happen and therefor breaking the deployment script. (At least this is my understanding of the prompting for any argument)

It seems like a lot of folks rely on the secret option for the down command since you can run php artisan down --secret=laravel or simply php artisan down and follow the prompts. Personally, I often forget to include the secret when I use the down command, which can be a bit of a hassle.

While I agree with the interactive context and asking for an optional secret.
The problem are deployment scripts without interaction, where now after an update all deployment scripts which simply use php artisan down without any further parameters would break. This should not simply happen on a minor release.

Theoretically a lot might rely on the secret, but for those who don't and/or have it integrated into a deployment pipeline or such this will break it, and in some cases in a very unpleasant way.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!


Restore account

Hello. can anyone help restore your account? I can't loginning in https://www.codewars.com/users/clooosha

Can you change my mail from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>


[Proposal] Add Button knob

I'm running into a few situations where I need a button knob that executes a function on press.
Specifically, for state transitions and transient states such as animations.
Having a button knob would allow me to trigger and test animations and state transitions, among other functionality.
I'm willing to make a PR if you'd be willing to accept a button knob.

KK, sounds good!


Deprecate support for Gaia v4 in favor of Gaia v7

Closes: #1986
Description

Per the acceptance criteria listed in #1986:

[x]  Remove legacy option from the chain-upgrade CLI
[ ] Remove all the references to gaia-v4 and upgrade the CI test suite to use gaia-v7

PR author checklist:

[ ] Added changelog entry, using unclog.
[ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[x] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

Shouldn't the test-end-to-end-current-gaia be updated to Gaia v6 and test-end-to-end-future-gaia be update to v7?
We should probably also remove the v5 integration-test.

Shouldn't the test-end-to-end-current-gaia be updated to Gaia v6 and test-end-to-end-future-gaia be updated to v7?

This makes sense. Alternatively, with Sean we discussed the following scheme:

Current → rename to Legacy (should be Vega, v6)
Future → rename to Current (now is Theta, v7)
Ref: https://github.com/informalsystems/ibc-rs/discussions/2030

Using the scheme "legacy/current" is more appropriate than "current/future" because v7 is already live. We will rarely be ahead of the release cycle (hence no need for "future").

Sounds good!


Remove support of custom cookbooks and private supermarkets

@dougireton Does it make sense to continue supporting custom cookbooks and private supermarkets? We can't really test these since we don't use this feature. Are these being used?

I think we should go ahead with this for v2.

I'm all for simplifying. It'd be great to have a deprecation notice though.

Part of the 2.0.0 release


[RESTEASY-2268] ResteasyHttpHeaders does not follow JAX-RS spec concerning modifiability

ResteasyHttpHeaders.getRequestHeaders() and ResteasyHttpHeaders.getRequestHeader() must return not modifiable collection
Signed-off-by: NicoNes<EMAIL_ADDRESS>

Hey @asoldano,
Since I never succeed in running all the tests  on my computer I always push first than I let Travis let me know what's wrong...One day I'll have to find out why local tests are not running
Anyway, I did the fix and  everything should be OK now. :+1:

@NicoNes thanks!


Update nodejs.yml

👋 Thank you for submitting this!  The team is pretty swamped at the moment, but we'll give this a review as soon as we can.

Hi @jpelaezClub! Can you describe what you're looking to achieve with this pull request, please? We don't have any top-level npm projects, so I'm guessing this has been opened accidentally. Given this, I'm going close it for now, but if it wasn't a mistake then please let us know and we can reopen it!


feat(async): introduce more async helper functions

Signed-off-by: azjezz<EMAIL_ADDRESS>

@veewee stuff you like ;)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

30 of 30 (100.0%)  changed or added relevant lines in 8 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 99.97%

Files with Coverage Reduction
New Missed Lines
%

src/Psl/Async/first.php
1
85.71%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
3304

Relevant Lines:
3305

💛  - Coveralls

Nice stuff!
Looks like amp is taking a slightly different approach in v3. But at the moment, I prefer the helpers in here over launching futures. It might be habit though 🙂

well, the good thing is that they can be used together!
use Psl\Async;
use Psl\Shell;
use Psl\Result;
use Amph\Future;

/**
 * @var Result\ResultInterface<string> $result1
 * @var Result\ResultInterface<string> $result2
 */
[$result1, $result2] = Amp\Future\all(
  Amp\launch(Async\reflect(fn() => Shell\execute('foo'))),
  Amp\launch(Async\reflect(fn() => Shell\execute('foo'))),
);


Resize window does not work in Firefox v63

What is your Test Scenario?
I'm trying to test elements being resized in Firefox v63 and I'm getting this warning:

Therefore, the test is not passing.
What is the Current behavior?
Any resize test does not work in latest Firefox version (63). This is working with version 62.
What is the Expected behavior?
Resize test must work correctly.
Your Environment details:

testcafe version: v0.22.0
node.js version: v8.9.0
browser name and version: Firefox Version 63 (64-bit)
platform and version: MacOS Sierra 10.12.6

Hey that is happening to me right now! Is there any solution for this?

Fixed in<EMAIL_ADDRESS>


I cannot build my VSCode extension project in VSCode after update to 1.76

VS Code Version: 1.76.0
OS Version: macOS Monterey 12.6

I'm making a vscode extension.
When I use version 1.75.0, In the project root path, I could build and run [Extension Development Host] by pressing F5.
However, after updating to 1.76, I cannot build my VSCode extension project in VSCode. No alert, No OUTPUT, No DEBUG CONOLE.
So now, I'm using 1.75.0, It works very well.
I think version 1.76 has some bugs.
Steps to Reproduce:

Update VSCode to version 1.76
Press F5

Can you give some more details? What happens? What does your build task do? Can you share the project?
Any error messages in devtools (Help > Toggle Developer Tools)?

Related: https://stackoverflow.com/questions/75619530/f5-not-working-in-vscode-extension-development/75619753#75619753

i have the same issue. Just created a Vscode extension. Didnt change any files. F5 dont work.
Related: https://stackoverflow.com/questions/75619530/f5-not-working-in-vscode-extension-development

Have the same issue

Can you give some more details? What happens? What does your build task do? Can you share the project?
Any error messages in devtools (Help > Toggle Developer Tools)?

@StephaneAdeso Can you provide the above additional information?

Today i decided to start an extension project. I installed the dependencies following the https://code.visualstudio.com/api/get-started/your-first-extension guide.
After creating the extension, i pressed F5 and nothing happened. So i suppose there is a bug by default on the yo-code extensions because i didnt modified yet any files.
I also have to say that i tried an old theme project ( 8 month old) and i was able to launch it with F5 without any problem.
I also tried the launch.json that you shared in Stackoverflow https://stackoverflow.com/questions/75619530/f5-not-working-in-vscode-extension-development/75619753#75619753 and it works perfectly.
So i am pretty sure there is something wrong in the launch.json that is generated by default.
You can see my files and version in the stackoverflow post.

when i press F5, this is the console.log output i get:

there you have the files that was autogenerated and dont work. i have changed anything since Yo-code created the project.
launch.json
// A launch configuration that compiles the extension and then opens it inside a new window
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Run Extension",
            "type": "extensionHost",
            "request": "launch",
            "args": [
                "--extensionDevelopmentPath=${workspaceFolder}"
            ],
            "outFiles": [
                "${workspaceFolder}/out/**/*.js"
            ],
            "preLaunchTask": "${defaultBuildTask}"
        },
        {
            "name": "Extension Tests",
            "type": "extensionHost",
            "request": "launch",
            "args": [
                "--extensionDevelopmentPath=${workspaceFolder}",
                "--extensionTestsPath=${workspaceFolder}/out/test/suite/index"
            ],
            "outFiles": [
                "${workspaceFolder}/out/test/**/*.js"
            ],
            "preLaunchTask": "${defaultBuildTask}"
        }
    ]
}

tasks.json
// See https://go.microsoft.com/fwlink/?LinkId=733558
// for the documentation about the tasks.json format
{
    "version": "2.0.0",
    "tasks": [
        {
            "type": "npm",
            "script": "watch",
            "problemMatcher": "$tsc-watch",
            "isBackground": true,
            "presentation": {
                "reveal": "never"
            },
            "group": {
                "kind": "build",
                "isDefault": true
            }
        }
    ]
}

package.json
{
  "name": "xxxxx",
  "displayName": "xxxxx",
  "description": "xxxxx",
  "version": "0.0.1",
  "engines": {
    "vscode": "^1.76.0"
  },
  "categories": [
    "Other"
  ],
  "activationEvents": [],
  "main": "./out/extension.js",
  "contributes": {
    "commands": [
      {
        "command": "xxxxx",
        "title": "Hello World"
      }
    ]
  },
  "scripts": {
    "vscode:prepublish": "npm run compile",
    "compile": "tsc -p ./",
    "watch": "tsc -watch -p ./",
    "pretest": "npm run compile && npm run lint",
    "lint": "eslint src --ext ts",
    "test": "node ./out/test/runTest.js"
  },
  "devDependencies": {
    "@types/vscode": "^1.76.0",
    "@types/glob": "^8.1.0",
    "@types/mocha": "^10.0.1",
    "@types/node": "16.x",
    "@typescript-eslint/eslint-plugin": "^5.53.0",
    "@typescript-eslint/parser": "^5.53.0",
    "eslint": "^8.34.0",
    "glob": "^8.1.0",
    "mocha": "^10.2.0",
    "typescript": "^4.9.5",
    "@vscode/test-electron": "^2.2.3"
  }
}

Is there a detailed documentation of how to configure a launch.json ?  where you can consult all the available parameters and how they work.

I'm also hitting this and can confirm that nothing "dubious" is showing in the console of the development tools. As a matter of fact, there is almost nothing there. (Which is odd, because it's normally full of output from the extension host, extensions, etc.)
Additional information is that it appears (at least for our extension) that the extension activates (when it's not supposed to) and then seems to just "stop" being executed.
Here's "all" of the console output from the extension host developer tools console
[Extension Host] STOPPED on first line for debugging on port 53603
J @ localProcessExtensionHost.ts:366
languagesAssociations.ts:73 Overwriting extension <<.mk>> to now point to mime <<text/x-Windows.NMake>>
(anonymous) @ languagesAssociations.ts:73
languagesAssociations.ts:77 Overwriting filename <> to now point to mime <<text/x-Windows.NMake>>
(anonymous) @ languagesAssociations.ts:77
localProcessExtensionHost.ts:298 Extension Host
localProcessExtensionHost.ts:299 Debugger attached.
console.ts:137 [Extension Host] activating extension (at console. (c:\Users\gabrield\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:96:138784))
console.ts:137 [Extension Host] starting language server (at console. (c:\Users\gabrield\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:96:138784))

좀 더 자세히 알려주실 수 있나요? 무슨 일이야? 빌드 작업은 무엇을 하나요? 프로젝트를 공유할 수 있습니까?
devtools(Help > Toggle Developer Tools)에 오류 메시지가 있습니까?

I can confirm the issue in v1.76 that nothing happens when pressing the F5 key.
I have to use the menu option Run>Start Debugging, a bit tedious.

This is probably due to a misbehaving extension, please run the "Start Extension Bisect" command to figure out which one it might be

Possibly same issue as #175202. F5 build tasks seem to activate too many extensions

Yeah @mjbvz there is a fix out in https://github.com/microsoft/vscode/issues/176006, if you are seeing it please bisect to figure out which extension is breaking it, I think it would just be nice to know which common extensions are impacted

Same as https://github.com/microsoft/vscode/issues/175976. I have a little more info there.

Maybe there's a second one then.

i disabled the java extension, but it didn't work. so i evicted bisec again and it detected the Material icon extension. si i disabled it and it didn't work. i think there is something more behind all this.

i commented the preLaunchTask in my launch.json and now it works perfectly. Which seems strange to me, because JSON doesn't allow comments. But it has worked and now it boots with F5 without problems.
I had it like this :
// A launch configuration that compiles the extension and then opens it inside a new window
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
{
	"version": "0.2.0",
	"configurations": [
		{
			"name": "Run Extension",
			"type": "extensionHost",
			"request": "launch",
			"args": [
				"--extensionDevelopmentPath=${workspaceFolder}"
			],
			"outFiles": [
				"${workspaceFolder}/out/**/*.js"
			],
			"preLaunchTask": "${defaultBuildTask}"
		},
		{
			"name": "Extension Tests",
			"type": "extensionHost",
			"request": "launch",
			"args": [
				"--extensionDevelopmentPath=${workspaceFolder}",
				"--extensionTestsPath=${workspaceFolder}/out/test/suite/index"
			],
			"outFiles": [
				"${workspaceFolder}/out/test/**/*.js"
			],
			"preLaunchTask": "${defaultBuildTask}"
		}
	]
}

and now I have it like this:
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Run Extension",
      "type": "extensionHost",
      "request": "launch",
      "args": ["--extensionDevelopmentPath=${workspaceFolder}"],
      "outFiles": ["${workspaceFolder}/out/**/*.js"]
      /* "preLaunchTask": "${defaultBuildTask}" */
    },
    {
      "name": "Extension Tests",
      "type": "extensionHost",
      "request": "launch",
      "args": [
        "--extensionDevelopmentPath=${workspaceFolder}",
        "--extensionTestsPath=${workspaceFolder}/out/test/suite/index"
      ],
      "outFiles": ["${workspaceFolder}/out/test/**/*.js"],
      "preLaunchTask": "${defaultBuildTask}"
    }
  ]
}

I can confirm that after 1.76.1 update, F5 is still broke.

It sounds like there might be an issue with your build task then.

It sounds like there might be an issue with your build task then.

As I mentioned above, the project is made from scratch with the yo-code plugin. The task is the one that comes by default.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Reintroduce ability to specify custom Verification Method IDs in did:dht

Allow user-provided Verification Method IDs provided in the CreateOptions for did:dht

If not provided, the JWK's Thumbprint is used and not included in the DNS TXT Record encoding

NOTE: Due to KeyManager currently all of the kid fields within any JWK is the Key's Thumbprint whether you provide a custom Verification Method ID or not. This includes the Identity Key #0. This is not compliant with the did:dht spec, but will be handled in a separate PR in order to prevent breaking anything downstream when changing KeyManager to allow for user-provided a kid to be specified.

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

Project coverage is 90.87%. Comparing base (c0eba91) to head (6e628ba).

Additional details and impacted files
@@           Coverage Diff            @@
##             main     #637    +/-   ##
========================================
  Coverage   90.87%   90.87%
========================================
  Files         118      119     +1
  Lines       29744    30027   +283
  Branches     2214     2251    +37
========================================
+ Hits        27029    27288   +259
- Misses       2680     2703    +23
- Partials       35       36     +1

Components
Coverage Δ

agent
80.86% <ø> (ø)

api
94.00% <ø> (-0.15%)
:arrow_down:

common
98.68% <ø> (ø)

credentials
94.88% <ø> (-0.38%)
:arrow_down:

crypto
93.81% <ø> (ø)

dids
97.69% <88.88%> (-0.05%)
:arrow_down:

identity-agent
96.70% <ø> (ø)

crypto-aws-kms
100.00% <ø> (ø)

proxy-agent
96.70% <ø> (ø)

user-agent
96.70% <ø> (ø)

@thehenrytsai I disabled the test vector tests using xit and created the issue to track the needed changes https://github.com/TBD54566975/web5-js/issues/638
Because I hate checking in not-covered code, I also added an e2e DNS Packet test for a custom controller within a Verification Method and generally cleaned up the toDNSPacket and fromDNSPacket tests into the e2e.


Bug in miniBatchStdDev

As far, as I understand, the miniBatchStdDev layer has a bug in this line
There the group and batch dimensions swap so after that std outputs tiled incorrectly.
We can try this on this code sample:
a = torch.tensor([0, 0, 0, 0, 1, 2, 3, 4, 100, 200, 300, 400], dtype=torch.float32)
a = a.reshape(-1, 1, 1, 1)
miniBatchStdDev(a, 4)[:, -1, 0, 0]

The output would be
tensor([ 99.5100,  99.5100,  99.5100,  99.5100, 149.1763, 149.1763, 149.1763,
        149.1763, 188.8589, 188.8589, 188.8589, 188.8589])

wich is obviously incorrect and should be
tensor([99.5100,  149.1763, 188.8589, 99.5100,  149.1763, 188.8589, 99.5100,
        149.1763, 188.8589, 99.5100,  149.1763, 188.8589])

Am I right?

Indeed it seems that you're right. I'll correct it right away.
I'll need to test it but there should be a PR within a week.

Thanks for pointing the bug, it was well hidden.

https://github.com/facebookresearch/pytorch_GAN_zoo/pull/70


URLs with uncoded blanks are only partially recoignised by OpenRefine3.5 beta1

Current Results
URLs with uncoded blanks are only partially recoignised by OpenRefine3.5 beta1. The string up to the unencoded blank is recognised as URL and the string after the blank is not recognised. This is a regression compared to OpenRefine 3.4.1, which recongnises URL with unencoded blanks.

URL without unencoded blank is recognised: http://siibns.ch/german/display_before91.asp?cdu=Ebersberg&folder=bns\00023359&cursor=2
URL with unencoded blank is partially recognised: http://siibns.ch/german/display_before91.asp?cdu=Ellikon an der Thur&folder=bns\00023471&cursor=2

I tested on different browsers and operating systems. The result stays the same regardless of OS and browser.
Expected Behavior
URL with unencoded blanks should be completely recognised.
Screenshots
Both screenshots show one URL with unencoded blanks and one without.
OpenRefine 3.5 beta1 (incorrect)

OpenRefine 3.4.1 (correct)

Versions

Operating System:  macos 10.14.6, Windows 10 Enterprise 1909
Browser Version: macos: Firefox 90.0.2, Safari 14.02; win10: Edge 92
JRE or JDK Version: bundled version
OpenRefine: OpenRefine 3.4.1 and OpenRefine 3.5beta1

Datasets
01_Konkordanz-GEO_portioniert_E-F-(korr)-abgeschlossen-export.openrefine.tar.gz

This was a determined change we made.  Before, we treated the whole string in a Cell as a URL and didn't do any detection of a valid URL.  Whitespaces are not allowed in URLs. So just use a GREL transform on the column to encode for URL first.

Hi @thadguidry. Ok, thanks for the info! I close this issue.

@ToVie If you need a quick way to validate a column of URLs, I typically just use this short Java interop expression with Clojure as my expression language. It attempts to convert the cell value to a URI, and then asks if the converted value is a URI.  You can optionally check the box to Store the error if you want in a new column, or use this in a Custom Text Facet.
(uri? (.toURI (java.net.URL. value)))


Fix lvgl.h include

Fix the file headers to match convention:
#ifdef LV_LVGL_H_INCLUDE_SIMPLE
#include <lvgl.h>
#else
#include <lvgl/lvgl.h>
#endif

https://github.com/lvgl/lv_font_conv/blob/master/lib/writers/lvgl/lv_font.js#L63
--lv-include allows to set any value you need dirctly.
@kisvegabor does this need update and how compatible is that (if yes)?

How can this be used in the online converter tool though? Meaning https://lvgl.io/tools/fontconverter

No ideas. I don't know online wrapper state and would not recommend it. Probably, such question would be better on lvgl forum.

I like the idea, it clearly more flexible than the current #include <lvgl/lvgl.h> way.
For the sake of simplicity, the online converter doesn't support every option.

@kisvegabor can i use sample from first post as is or it needs some tweaks?

Use this please. (only <> is changes to "")
#ifdef LV_LVGL_H_INCLUDE_SIMPLE
#include "lvgl.h"
#else
#include "lvgl/lvgl.h"
#endif

Please check maser branch. If everything fine, i will publish.

Looks good!


⚠ Bofrost.com has degraded performance

In ab76fcd, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 900 ms

Resolved: Bofrost.com performance has improved in 6594e75 after 13 minutes.


Suggestion to fix dangling ghc process on double ctrl+c

Hi!
This PR is not meant to be merged. Instead, I'd like to open the discussion to find possible solutions to #257
Debugging the code, not fully looking at it, I noticed that when you press ctrl+c once it seems quit will be called, which in turn will call interrupt and then ghciInterrupt. However, if you press ctrl+c once again the process will shutdown immediately and a dangling ghc might happen.
Just to try things to improve this, I used installHandler to trap SIGINT (ctrl+c). When I did that, every time you press ctrl+c, no matter how many times, the handler call will run. So I tried putting quit there and it works fine. However, the second time it will print some failure (because we already called terminateProcess) but it at least doesn't leave a dangling ghc, so it's already slightly better than the current situation. Ideally the second ctrl+c would do nothing: just wait for the process to exit, or maybe just ignore the second ctrl+c, I don't know.
Thoughts?
The dangling ghc is not a blocker but I'd definitely like to prevent it.

Aren't you trading a process that leaks a GHC, for one you can't kill with Ctrl-C at all? Both seem like fairly unpleasant options...
The other issue is that ghcid is both a library and an executable, but I'm sure with some config we could work around that.

Aren't you trading a process that leaks a GHC, for one you can't kill with Ctrl-C at all?

It does get killed. You just get an error message when you press that second ctrl+c, but other than that it works well.

but I'm sure with some config we could work around that.

Yes! An option to make this configurable sounds great.

It does get killed. You just get an error message when you press that second ctrl+c, but other than that it works well.

I'm confused - can you explain the sequence of events? I hit ctrl-c, that starts terminating GHC how it does now. I hit ctrl-c again, then I go to the handler, and what happens? What if GHC exits cleanly, and what if it stays around? If I hit ctrl-c again, what happens then? (I'm a Windows guy, so signal handlers aren't my area of expertise!)

Oh, I don't know about signals either, much less in Haskell. This was just an experiment to see ig I could kill the underlying process no matter how many ctrl+c I did.
I'll investigate this with my team to see if we can come to a clean PR that works as expected.
Thank you!

Happy to keep this PR open while you do so (up to you) - I agree this seems like a reasonable approach. I'd also suggest you stick to Posix only to start with, since doing whatever you do cross platform will be additional complication. (Although very happy if you can do it cross platform, of course!)

Sounds good!

Hello,
I've experienced this issue recently, and I think that it would be very useful to fix it 🙂
Basically my computer was becoming unresponsive after using ghcid with a very heavy project. I realized ghci processes would start to become daemonic and consume lots of memory and CPU. Apparently ghcid was not able to stop them, so it would leave them hanging around. This is quite confusing, since as a user I would expect to ignore that there's an underlying ghci process, and clearly it should not be left running in the system without any notice.
I have a draft commit with a fix https://github.com/ndmitchell/ghcid/commit/9f1eb0fe622c0d95955760ddd0590fd73166a2f9 . My proposal is: don't exit ghcid unless ghci is dead. This means that a user may need to manually kill ghcid in case ghci hangs; but at least, user will be aware that there's something wrong with current state, and check what action needs to be taken.
Some notes about my commit:

Note that the Haskell runtime will simply kill the current app when receiving a second SIGINT, leaving a daemonic ghci running in all cases
I think we should ignore the second SIGINT altogether: the first SIGINT will try and stop ghci, if it won't die, then there's nothing else we can do. First call to installHandler does that.
The function I patched (Session.kill) is also used by ghcid to kill an unresponsive ghci and start a new one. So we need to be careful to restore the original signal handler in case this kill action does not originate from a SIGINT. Second call to installHandler does that.

@acondolu your solution sounds pretty reasonable to me, and has the nice advantage of being cross platform. Thoughts @asterite? A second set of eyes on such pieces is always useful.

Notes if we do go with @acondolu's code:

Would be better to use a bracket for disabling the interrupt, so we don't forget to add it back.
There's a lot of good comments in this thread and the message that would be useful to move the knowledge into the code as comments.

@ndmitchell I've revisited my commit with your observations ➡ see https://github.com/acondolu/ghcid/commit/d299f9a4cbdf7a2278d7c5b1be03adfeb4cc195b
(now using bracket + added a comment)
What do you guys think?

Any chance to get this going?

@asterite Sorry for giving no thoughts about this, but a lot of time has passed and I lost context on this, plus I won't have time to look at it... so don't wait for my opinion 😄  (in case you were doing so)

@acondolu does that mean if ghci doesn't exit, ghcid hangs?

Current state: pressing Ctrl+C twice will make ghcid exit but the underlying ghci dangling, running in the background.
Proposed state: after pressing Ctrl+C (once or multiple times) if ghci doesn't exit, ghcid hangs.


All PowerToys utilities disabled after update

Microsoft PowerToys version
0.72.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
When updating PowerToys (this has now happened at least 3 times), all utilities are disabled after the update and need to be enabled one by one. This is of course not a blocker, but somewhat annoying nonetheless.
Configuration of each utility seems to be restored on enabling it.
✔ Expected Behavior
The utilities I had enabled before the update would be enabled after the update too.
❌ Actual Behavior
All PowerToys utilities are disabled after an update has been installed.
Other Software
Microsoft Windows 10 Pro (Build 19045)

PowerToysReport_2023-08-02-14-55-59.zip

does this still happen ? /needinfo

@Lane-Schwartz does this issue still happen for you where things are disabled on upgrade?


Method does not behave as expected: org.assertj.core.api.AbstractAssert.as(String, Object...)

The codelines below should result in the same behavior in my opinion:
assertThat("A").isEqualTo("B").as("Error Message");
assertThat("A").as("Error Message").isEqualTo("B");

But the first code line does not print the error message.
Actually I prefer to write it like the first code line, but I am forced to write it like the second code line, since otherwise I won't see the error message when the assertion fails.

It would be nice indeed to be able to write the first line but it's not possible in the current design of AssertJ (and won't be unless a major refactoring is done).
The reason is that AssertJ evaluates assertion in order, in your first example isEqualTo("B") is evaluated before as("Error Message") and if it fails then an assertion error is thrown which explains why as("Error Message") is not evaluated at all.
Should AssertJ do lazy evaluation ? That would be nice but how AssertJ would know when to evaluate the chained assertions ? that is, in your first example, how AssertJ would know that is should evaluate the assertion after as("Error Message") ? No easy solution to that except adding a termination method to signal AssertJ to perform the assertion but that would be verbose and not fluent (ex assertThat("A").isEqualTo("B").as("Error Message")evaluate();) - not a nice solution IMO.
To conclude, this is the normal behavior and it can't be improved without rewriting a major part of AssertJ.

closing the issue as it works as expected.

Hi Joel,
the method as(...) could trigger evaluation so the method evaluate() at the
end would not be required in that case.
I don't know if it would be possible to determine by reflection
(getStacktrace()?) in the assertion methods the whole chain of assertions
in order not to raise a AssertionError before the error message is set (if
there is an as(...) method being used). But that would be costly in terms
of performance and tricky.
Maybe it is not pracicable what I suggested. But from a readers view I
still think it should not matter where i put the as(...) command, it should
be the same behavior.
On Fri, Aug 21, 2015 at 1:35 PM, Joel Costigliola<EMAIL_ADDRESS>wrote:

It would be nice indeed to be able to write the first line but it's not
possible in the current design of AssertJ (and won't be unless a major
refactoring is done).
The reason is that AssertJ evaluates assertion in order, in your first
example isEqualTo("B") is evaluated before as("Error Message") and if it
fails then an assertion error is thrown which explains why as("Error
Message") is not evaluated at all.
Should AssertJ do lazy evaluation ? That would be nice but how AssertJ
would know when to evaluate the chained assertions ? that is, in your first
example, how AssertJ would know that is should evaluate the assertion after as("Error
Message") ? No easy solution to that except adding a termination method
to signal AssertJ to perform the assertion but that would be verbose and
not fluent (ex assertThat("A").isEqualTo("B").as("Error
Message")evaluate();) - not a nice solution IMO.
To conclude, this is the normal behavior and it can't be improved without
rewriting a major part of AssertJ.
—
Reply to this email directly or view it on GitHub
https://github.com/joel-costigliola/assertj-core/issues/481#issuecomment-133380230
.

Just to be clear, I have closed this issue not because I disagree with you but because it is a major change and I haven't found an elegant way to address it (adding evaluate() is not nice option IMO).


Adds a new function to sanitize the components of a reaction like molecules

Trying to cram this into the existing sanitizeRxn() function just didn't make sense, so I created a new one
Fixes #7108

I think it might be confusing to know when to call this function and when to call SanitizeRxns.
Perhaps this could be a documentation update?

I think it might be confusing to know when to call this function and when to call SanitizeRxns.
Perhaps this could be a documentation update?

I think that makes sense. I will write something describing the difference between transforms and specific reactions, do a blog post with it, and then figure out how to incorporate that into the docs. I don't think it should hold up this PR though.

I'm going to reserve the right to call this FUNC via a flag in SanitizeRxns :)


update arabz tag#24982

I'm submitting a ...
[ ] bug report
[x ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

Please update the tag in this token
update arabz #24982

Please merge, thanks
@keone

@spacemandev-git
Could you please merge, thanks


Feature request: support for periodic boundaries

Low priority, but it would be nice for visualising cosmological simulations.

Excellent idea for the gui, also for spherical coordinates this can be useful. A workaround now is, to shift the 'center' is to use for instance:
(l + 180) % 360 - 180
To go from a range [0, 360) to [-180, 180)

Thanks, that certainly works for now!

Is there also a way to add a "ghost" region - e.g. a periodic box ranging from [-0.5, 1.5) where the box' "real" values run from [0,1)?
Application would be for vectors that cross the boundary and are now missed.

The only dirty way would be, say from the notebook, to copy the data multiple times with this offset, and add them together using ds.concat. If you then want to use it from the gui, you'd have to export it.. using ds.export_hdf5... as I said, quite dirty :)

ugh...

A more clean solution is explained in #76


GitHub Actions are broken

Data isn't updated since Nov 22, 2022.

This is FIXED, due to latest ubuntu image which does not support python 3.6. https://github.com/actions/setup-python/issues/544


Adding Flex Lexer based on issue #1893

We have added support for a Flex Lexer based on the issue https://github.com/pygments/pygments/issues/1893.
Any feedback regarding our implementation and correctness of our code would be much appreciated also. Multiple tests are also available for these purposes.

Based on the PR comment by Jean-Abou-Samra

What do you mean? When there are several reviews, you need to address all of them :-)