AES-NI implementation using intrinsics for win/msvc/x64

Description
The PR adds AES-NI support using intrinsics for msvc/x64 on Windows. Also adds bn_mul optimization and implements rdtsc for timing.
Firstly, I'm interested if there's a wish by the maintainers to incorporate this PR. If so, I will begin agreement acceptance with our legal. Also any comments are very welcome.
Status
IN DEVELOPMENT/HOLD
Requires Backporting
NO
Migrations
NO
Additional comments
Addresses issue #855.
Todos

[ ] Changelog updated

Steps to test or reproduce
Run tests compiled with Visual Studio under Windows/x64 (needs enabling tests for MSVC in ./CMakeLists.txt)

--none--

Hi @orlx Thank you for your contribution!
As you already know, we cannot accept contributions without a valid CLA signed.

Instead, I'm opening a fresh PR with the same functionality but with a some code refactoring and cleanup

This PR has been superseded by PR #1901.


VS Code not able to detect python files

Issue Type: Bug
Whenever I create a python file, it automatically goes to the binary mode. I manually have to select the language mode.
Pls fix this ASAP !!
VS Code version: Code 1.65.2 (c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1, 2022-03-10T15:36:26.048Z)
OS version: Linux x64 5.13.0-37-generic
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Celeron(R) N4000 CPU @ 1.10GHz (2 x 2553)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: disabled_offoop_rasterization: disabled_offopengl: enabled_onrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
7.59GB (1.48GB free)

Process Argv
--unity-launch --crash-reporter-id d2e5fbb0-7926-4fb6-88bb-794f4dffa3f5

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasma

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

Extensions (20)

Extension
Author (truncated)
Version

html-snippets
abu
0.2.1

vscode-css-formatter
aes
1.0.2

doxdocgen
csc
1.4.0

dart-code
Dar
3.36.0

flutter
Dar
3.36.0

es7-react-js-snippets
dsz
4.4.3

copilot
Git
1.11.5453

better-cpp-syntax
jef
1.15.13

firebase-explorer
jsa
0.3.3

python
ms-
2022.4.0

vscode-pylance
ms-
2022.3.4

remote-containers
ms-
0.231.1

remote-ssh
ms-
0.76.1

remote-wsl
ms-
0.64.2

cmake-tools
ms-
1.10.5

cpptools
ms-
1.9.7

cpptools-extension-pack
ms-
1.1.0

vetur
oct
0.35.0

LiveServer
rit
5.7.5

cmake
twx
0.0.17

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vsclayoutctrc:30451274
pythonvsnew555:30457759
vscscmwlcmt:30438805
vscgsvid2:30447481
vscaat:30438848
pynewfile477:30463512

/gifPlease

As you can see here, after creating a python file, I manually have to select the language mode before working on it.
Please resolve this ASAP as this is seriously affecting my workflow.
PS: This issue is there for python only ig.

What version of Python have you installed on your Windows workstation?

Its Python 3.8 and it's a Linux machine

Any update on this issue 🤔🤔

I’m suspicious of an extension causing this behavior. Can you open the Command Palette (⌘/Ctrl+Shift+P) and run the extension bisect command: Help: Start Extension Bisect and follow the prompts? (More information about Extension Bisect here)

I'm also suspicious of the files.associations user setting. Can you double check that and make sure nothing is overwriting Python to Binary?

Hey @TylerLeonhardt .
Seems like you were right. The issue is with the files.associations user settings.

What is the correct alternative to this ??

@amalthomas-exe What is it that you are trying to achieve with that setting? Why is it set to code-text-binary instead of leaving the default (remove that line) which is to detect .py files as Python?

I didn't do that. Maybe some faulty extension setting ig.
Anyways thanks for your help.

The issue has been resolved. Closing it.


On Unix, PowerShell should define the SHELL environment variable on startup

POSIX-like shells such as bash define the SHELL environment variable on startup as the full path of the current user's default shell (e.g., /bin/bash). (They do so even if they're not the default shell themselves - unless the variable is already defined.)
This variable is mandated by POSIX:

SHELL
This variable shall represent a pathname of the user's preferred command language interpreter.

If PowerShell itself functions as the default shell, SHELL is never set, because PowerShell doesn't set it.
Unix utilities may rely on the presence of this environment variable.
Therefore, PowerShell, like POSIX-like shells, should define SHELL on startup, unless it is already defined.
Environment data
Written as of:
PowerShell 7.0

Unix utilities may rely on the presence of this environment variable.

Interesting, how would we test that defining this variable will have the desired effects and do not break these utilities?

how would we test

I don't think you can, but I don't think we have to:
Even non-POSIX shells such as csh set the SHELL variable, and even the behavioral variations among POSIX-like shells (except if they're in compatibility mode) mean that you can't make blind assumptions about how, specifically, to invoke the shell reported by SHELL (other than without arguments).
In other words: no existing utility can make fixed assumptions about specific behaviors exhibited by the user's default shell, as reflected in SHELL.
However, knowing what shell is the user's default shell is important information, which PowerShell should provide too.
An important use case is that installer utilities may decide what shell-specific initialization files (profiles) they need to modify and how.

I have a proposed fix for this issue, but again I am not sure were to add the tests.  My planned main test is to unset the SHELL env. variable, make sure it is not defined, and then run pwsh and verify that SHELL is set (and if possible also verify its value)

I don't know if this has since been fixed or whether I'm missing something, but I just tried with PowerShell Core 7.2.0-preview.4 as the default shell on both macOS and Linux, and $env:SHELL is defined accordingly.
@davidBar-On, do you still see it not being set, and if so, in what version (and on what platform)?

I am using WSL (Ubuntu 20.4 Linux) and building from PowerShell 7.2.0-preview.3 master branch. $env:SHELL is not defined by this version.  (Initially it was defined as /bin/bash from Linux, but after unset it and running pwsh again it was not defined.)

I don't know if this has since been fixed or whether I'm missing something

We don't set the env variable in pwsh.

Thanks, @iSazonov.
I've come to realize that the POSIX-compatible shells also do not set this variable [as an environment variable - see below] - they rely on the system to set it:

On macOS, it seems that setting $env:SHELL happens via the login utility that the terminal applications use behind the scenes, given that every shell created in a terminal is launched as a login shell (-l)

On Linux, judging by my Ubuntu 18.04 machine, it seems that $env:SHELL is set once, when a user's OS session starts.

Therefore, from what I can tell, using PowerShell as a user's default shell should work correctly already - also via ssh.

Among POSIX-compatible shells, zsh and dash do not set $SHELL at all, whereas bash and ksh only set it as a shell-only variable, not as an environment variable - all this suggests that shells aren't expected to define this variable as an environment variable themselves.
Due to PowerShell having distinct syntaxes for environment- vs. shell-only variables, the bash / ksh approach wouldn't work; as a courtesy, we could set the environment variable, but I don't think it's necessary, given that it should be defined by the system, and given that POSIX-compatible shells do not (fully) try to compensate for its absence.

@davidBar-On, can you confirm that starting a new shell after chsh -s /usr/bin/pwsh indeed has $env:SHELL defined accordingly?

can you confirm that starting a new shell after chsh -s /usr/bin/pwsh indeed has $env:SHELL defined accordingly, after logging out and back in or rebooting?

After setting pwsh as the default shell using chsh and logging in the $env:SHELL is set to the new shell (I used su to login).
Also, under Linux bash I unset SHELL and checked which shell sets it (without login).  /bin/bash does set SHELL if it is not set already. /bin/sh and /bin/csh does not.

Thanks for checking, @davidBar-On (using su $env:USER is a great idea to avoid the need for a logout / reboot - hadn't occurred to me).
Note that /bin/bash sets a shell variable named SHELL, not an environment variable (verify with declare -p SHELL); On Ubuntu, /bin/sh is dash (which, as you state, takes no action at all). As stated, the bash approach (ditto for ksh) isn't really an option for PowerShell, given that environment variables use a different namespace ($env:SHELL) from shell variables ($SHELL).
Just to be clear: Are we in agreement that no action is needed?

If we don't know broken scenarios with SHELL environment variable I suggest to archive the issue.


Add support for downloading to seekable files

This pull request does the following:

Add a DownloadOutputManager that gives us flexibility in a similar fashion as the UploadInputManager
Add support for seekable files.

Here is some comparisons:
Seekable file download
Code:
import time

import boto3
from s3transfer.manager import TransferManager

c = boto3.client('s3')
key = '100MB'

start_time = time.time()
manager = TransferManager(c)
with open(key, 'wb') as f:
    future = manager.download('mybucketfoo', key, f)
    queue_time = time.time()
    print("Time to queue download: %s" % (queue_time - start_time))
    future.result()
    end_time = time.time()
    print("Time to complete download: %s" % (end_time - start_time))

Results:
Time to queue download: 0.00941491127014
Time to complete download: 3.51214289665

Filename
Code
import time

import boto3
from s3transfer.manager import TransferManager

c = boto3.client('s3')
key = '100MB'

start_time = time.time()
manager = TransferManager(c)
future = manager.download('mybucketfoo', key, key)
queue_time = time.time()
print("Time to queue download: %s" % (queue_time - start_time))
future.result()
end_time = time.time()
print("Time to complete download: %s" % (end_time - start_time))

Results
Time to queue download: 0.00508308410645
Time to complete download: 3.74164819717

CLI Comparison
$ time aws s3 cp s3://mybucketfoo/100MB 100MB
download: s3://mybucketfoo/100MB to ./100MB

real	0m3.738s
user	0m0.844s
sys	0m0.491s

cc @jamesls @JordonPhillips

Pretty straightfoward, no major issues for me.  I agree with @JordonPhillips's feedback about seekable and test improvements.  Otherwise :shipit:

@jamesls @JordonPhillips
I updated the following based on feedback:

Created a seekable() function in compat.py and ported uploads and downloads to use it.
Updated docs for get_final_io_task and made the CompleteDownloadNOOPTask is_final=True by default.

Let me know what you think.

:shipit:, had a question, but otherwise looks good to me.

:shipit:

                       _____|\
                  _.--| SSt |:
                 <____|.----||
                        .---''---,
                         ;..__..'    _...
                       ,'/  ;|/..--''    \
                     ,'_/.-/':            :
                _..-'''/  /  |  \    \   _|/|
               \      /-./_ \;   \    \,;'   \
               ,\    / \:  `:\    \   //    `:`.
             ,'  \  /-._;   | :    : ::    ,.   .
           ,'     ::   /`-._| |    | || ' :  `.`.)
        _,'       |;._:: |  | |    | `|   :    `'
      ,'   `.     /   |`-:_ ; |    |  |  : \
      `--.   )   /|-._:    :          |   \ \
         /  /   :_|   ;`-._;   __..--';    : :
        /  (    ;|;-./_  _/.-:'o |   /     ' |
       /  , \._/_/_./--''/_|:|___|_,'        |
      :  /   `'-'--'----'---------'          |
      | :     O ._O   O_. O ._O   O_.      ; ;
      : `.      //    //    //    //     ,' /
    ~~~`.______//____//____//____//_______,'~
              //    //~   //    //
       ~~   _//   _//   _// ~ _//     ~
     ~     / /   / /   / /   / /  ~      ~~
          ~~~   ~~~   ~~~   ~~~


loadImage with cors in v3

how to pass corssOrigin = "Anonymous" in the new api that's using fetch underneath? Also isn't falling back to blob and createObjectURL slowing things down the moment options are provided?
Here is function i'm trying to achieve
function loadImage(url) {
    return new Promise((resolve, reject) => {
      const downloadedImg = new Image();
      downloadedImg.crossOrigin = "Anonymous";
      downloadedImg.addEventListener("load", () => {
        resolve(downloadedImg);
      });
      downloadedImg.addEventListener("error", reject, false);
      downloadedImg.src = url;
    });
  }

So currently in v3, every loadFn function is accepting the options object for fetch as the second argument:
So one of these might work:
await loadImage(urlOrOpts, { mode: "cors" }) // or no-cors or same-origin
// or to set the property on the image
await loadImage({ url, crossOrigin: "anonymous" })
// or a combination of both

I have benchmarked and it is indeed faster to not use a blob and pass url directly to src. Now the question is one of consistency and API:
// 1. Only url as first and opts as second, no blob url
await loadImage(url, imageOpts)

// 2. url or opts as first argument only
await loadImage(urlOrOpts)

// 3. use fetch as blob only if options are passed
await loadImage(urlOrOpts, fetchOptions)

// 4. add more parsing options eg. no-parse/"blob"/"base64"
await loadImage(urlOrOpts, parse?)

Actually turns out that i was stuck at v2 and after upgrading to v3 i don't have problems with cors tinting. Closing for now until i run into same problem again.


Multiple conflicting command-line tools called ulid exist

Both this project and https://github.com/oklog/ulid/ build executables called ulid. The purpose of this issue is to open a discussion about possibly renaming one or both of them so that they can co-exist without workarounds.
This is motivated by the Fedora Linux case; we already have a golang-github-oklog-ulid package that installs /usr/bin/ulid, and since https://github.com/pydantic/pydantic-extra-types added a dependency on https://github.com/mdomke/python-ulid/, we are now working on a python-python-ulid package as well. A quick survey suggests that we may be the first Linux distribution to package both projects. We have some guidelines about how to deal with binary name conflicts, and approaching the relevant upstreams about a possible renaming is the first step.
Ideally, one or both projects would rename the ulid binary to something more unique, so that distributions don’t have to deal with conflicting packages or choose their own way to rename binaries. The issue https://github.com/mdomke/python-ulid/issues/13 may or may not also be relevant, depending on the response.

https://github.com/oklog/ulid/issues/111

Based on discussion in https://github.com/oklog/ulid/issues/111, the conflict in Fedora is resolved beginning with Fedora 40 by renaming the other package’s binary from /usr/bin/ulid to /usr/bin/ulid-go in https://src.fedoraproject.org/rpms/golang-github-oklog-ulid/pull-request/1. However, I’m leaving this issue open because ulid is still a pretty generic name considering how many implementations of ULIDs exist, and there may still be a case for renaming – especially considering https://github.com/mdomke/python-ulid/issues/13. Feel free to close this issue if no change is planned.


Can I show the current revision/version of the branch in the prompt?

Posh shows the current branch in the commandprompt but it would be nice if it could also show the current branch revision/version/shorthash.
Maybe this is already possible but I'm not a powershell wizard yet.

See issue #345.  I think a better approach to such niche customization is to make it easier for folks to customize the content of the Git status summary text rather than adding more & more options to posh-git.


ENYO-5742: Add unit tests of voice control for moonstone components

Checklist

[x] I have read and understand the contribution guide

[ ] A CHANGELOG entry is included

[ ] At least one test case is included for this feature or bug fix

[x] Documentation was added or is not needed

[ ] This is an API breaking change

Issue Resolved / Feature Added
Components should be checked voice control related props
Resolution
Add unit test to components
Additional Considerations
Links
ENYO-5742
Comments
Enact-DCO-1.0-Signed-off-by: Changgi Lee<EMAIL_ADDRESS>

@ryanjduffy It depends on what kind of intent used. For example, Select intent should be positioned same node with spottable because Web Engine will be simulate click event to target node. But another case, the node is used for only gateway to receive voice event from Voice Framework.
it be better to generalize the node lookups <= Can you provide more info about this? Thanks~

Instead of the following which assumes a particular DOM node or subcomponent
button.find('div').at(0).prop('data-webos-voice-intent');
expandableItem.find('LabeledItem').prop('data-webos-voice-disabled');

Implement tests that use generalized requirements like "the node with role='button' has the voice intent" or "the spottable node has the voice intent"

@ryanjduffy Thanks for your info. I also think generalized requirement is more clear way and try to apply it to Button component.(I did update TC code) In other case, for example TimePicker, my intention is to check voice disabled prop successfully delegated to each child(hour, meridiem, minute) pickers.(So, I hope it will be checked in current way)


fix: Make ModalStandardBottomSheet collapsed by default

GitHub Issue: fixes https://github.com/unoplatform/Uno.Gallery/issues/137
PR Type
What kind of change does this PR introduce?

Bugfix

Description
Modal sheet was initially visible when the window was resized.
PR Checklist
Please check if your PR fulfills the following requirements:

Commits must be following the Conventional Commits specification
[x] Tested UWP
[ ] Tested iOS
[ ] Tested Android
[x] Tested WASM
[ ] Tested MacOS
[x] Contains No breaking changes

@carldebilly Seems I don't have permission to merge here, can you merge the changes please?

/azp run

/azp run

/azp run


Take care of PDO false and DSN tailing semicolons.

@phanalpha You need to run cargo  fmt --all -- --check to fix format problem, the problem report from https://github.com/apache/skywalking-php/actions/runs/3268126418/jobs/5377222949

Also, could you commit your unit test?


Startup error in get_linux_directories

When starting LBRY-0.33.2 the first time on Ubuntu 19.04, I get the following message during startup:

The daemon may have encountered an unexpected error, or another daemon instance is already running.

The log file does not exist, but the console has the following output:
Starting SDK
Sandbox listening on port 5278.
Daemon: [17050] Failed to execute script cli

Daemon: Traceback (most recent call last):
  File "lbrynet/extras/cli.py", line 322, in <module>
  File "lbrynet/extras/cli.py", line 229, in main
  File "lbrynet/conf.py", line 400, in create_from_arguments
  File "lbrynet/conf.py", line 570, in __init__
  File "lbrynet/conf.py", line 582, in set_default_paths
  File "lbrynet/conf.py", line 621, in get_linux_directories
AttributeError: 'NoneType' object has no attribute 'group'

I tried manually creating the ~/.config/LBRY/user-dirs.dirs according to the code that tries to read it, but the exception is still the same.
I'm running Python 3.7.3, which has FileNotFoundError inherit from OSError instead of EnvironmentError, but this was apparently changed in 3.3 so already a long time ago. Does LBRY bundle it's own python interpreter?

Sorry to hear and thanks for opening this issue! I've run this successfully on 19 so it's probably a local issue with permissions or some strange directory config . Can you give us any more info?

The problem was that xdg.read() returns an empty string if the file does not exist, which causes re.search to return None.

@shauser if the file doesn't exist, it would fail with a different error
can you tell me how to reproduce this issue?

Sorry, I was busy with other stuff in the last few months. I just checked back with this issue...
The error actually occurs when the file ~/.config/user-dirs.dirs exists, but is empty. I don't know why this happened though, but after removing the file or recreating it with xdg-user-dirs-update lbrynet works flawlessly.

Thanks for the feedback! Can we show you some appreciation for the info ?
@eukreign should we try to handle this scenario or mark this closed as a environment specific issue. I haven't seen it happen elsewhere.


fix(dependencies): Update dependency @gitbeaker/rest to v39.10.3

This PR contains the following updates:

Package
Type
Update
Change

@gitbeaker/rest
dependencies
patch
39.10.2 -> 39.10.3

Release Notes

jdalrymple/gitbeaker
v39.10.3
Compare Source
🔨 Technical Debt

@gitbeaker/requester-utils, @gitbeaker/rest

Embedding the request options normalization #3374 (@jdalrymple)

↕ Dependencies

Bump eslint-config-prettier from 8.8.0 to 9.0.0 #3372 (@dependabot[bot])

Authors: 2

@dependabot[bot]
Justin Dalrymple (@jdalrymple)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore(notes): update release notes for 2019.04

Deploy preview for pf-next ready!
Built with commit 7ad7b69e379ce9a6e1c726eb50fd7952c85c2d7b
https://deploy-preview-2109--pf-next.netlify.com

Closing in favor of #2111 .


Count-Sort

Counting sort is a sorting algorithm that sorts the elements of an array by counting the number of occurrences of each unique element in the array.

@py3-coder let me know if any changes to make. I hope this would count for Hacktoberfest 2022 PR. Kindly merge it soon as possible from your side. Thankyou.

Read the Readme.md  file of CPP.
Guideline how to add CPP file read it again
PR accepted @Aviace
Need to Star this repo incase not done.
Just Update the required will merge to main
🚀🚀👍

You mean updating the readme file for cpp(Step 7) and add my cpp program with link there? But I can't see the update option for readme file. I have understood what to do, but where do I start or do it on? can you just elaborate a bit.


feat: cip68

few Qs in https://github.com/blockfrost/openapi/pull/264#issue-1498655379
prereqs:

https://github.com/blockfrost/openapi/pull/264
https://github.com/blockfrost/blockfrost-utils/pull/26

resolves https://github.com/blockfrost/blockfrost-backend-ryo/issues/6

Is it possible to add the integration test?


Game.ini settings on an instance specific base

I am trying to change the settings of only one of the servers that I am running. I want to disable friendly fire on one of my maps in the cluster. I read somewhere here about duplicating the entire ark server files. Is there any way to change it without doing this? I do not have the space to duplicate ark on my system.

If you don't want to use a staging directory, then you can try using bind mounts:
mount --bind /path/to/common/ark/directory /path/to/instance/directory
mount --bind /path/to/instance/save /path/to/instance/directory/ShooterGame/Saved
mount --bind /path/to/instance/mods /path/to/instance/directory/ShooterGame/Content/Mods

How would I use this to have different game.ini files?

Using bind mounts would allow you to use the same game files while keeping the saved files (including Game.ini and GameUserSettings.ini) separate between instances.

another question then as i am not super fluent in linux yet, how would i do that? just by copying and pasting the command above?
That seems so complicated, is ASM not able to edit Game.ini like it can for GameUserSettings.ini? I know for GameUserSettings.ini i can do ark_ arkflag_ or arkopt_ (i add all 3 but dont actually know which one works) for changing settings based on the instance but it does not work for lines in Game.ini

In the mount commands I gave, replace:

/path/to/common/ark/directory with the path to the common ark directory
/path/to/instance/save with the path to the directory where the instance save files should be stored
/path/to/instance/mods with the path to the directory where the instance mods should be stored
/path/to/instance/directory with the path to the directory from which you want the instance to run.

arkmanager does not itself modify GameUserSettings.ini - rather arkmanager provides the command-line options, and the game server uses these command-line options to modify the config and then saves the modified config.
ark_optionname=value options are those specified in the option string mapname?option1=value1?option2=value2
These are the ?optionname=value options specified in https://ark.gamepedia.com/Server_Configuration#Command_line_arguments
arkopt_optionname=value options are those specified with a leading dash with a value
These are the -optionname=value options specified in https://ark.gamepedia.com/Server_Configuration#Command_line_arguments
arkflag_optionname=true options are those specified with a leading dash without a value
These are the -optionname options specified in https://ark.gamepedia.com/Server_Configuration#Command_line_arguments

Can you tell me what I'm doing wrong that I don't want the test instance to work?
mount --bind /home/arkserver/ARK /home/arkserver/instance/ARK.center
mount -- bind /home/arkserver/instance/ARK.center.mods /home/arkserver/ARK/ShooterGame/Content/Mods
mount --bind /arkserver/instance/ARK.center.save /home/arkserver/ARK/ShooterGame/Saved
And I changed the root instance /home/arkserver/instance/ARK.center
IN Main.cfg

It looks like you meant to:
mount --bind /home/arkserver/ARK /home/arkserver/instance/ARK.center
mount -- bind /home/arkserver/instance/ARK.center.mods /home/arkserver/ARK.center/ShooterGame/Content/Mods
mount --bind /arkserver/instance/ARK.center.save /home/arkserver/ARK.center/ShooterGame/Saved

Thank you I changed but still the same problem does not start stops at 2.34 GB of RAM consumption

Yes, each instance is assigned a different one.  As I use on these given folders umount -A works fine

S_API FAIL] SteamAPI_Init() failed; SteamAPI_IsSteamRunning() failed.

S_API FAIL] SteamAPI_Init() failed; SteamAPI_IsSteamRunning() failed.

This is normal - it just means that you don't have the Steam client running, which is normal on a server.

 It was enough to restart the whole server to start everything working.  Thanks for the help

Hi,
i put the mount commands in "/etc/arkmanager/instances/servername.cfg" right??
best regards

No.
If you want to go with persistent bind mounts, then the best place to put
the mounts is in /etc/fstab.
/home/arkserver/ARK /home/arkserver/instance/ARK.center bind defaults 0 0
/home/arkserver/instance/ARK.center.mods
/home/arkserver/ARK.center/ShooterGame/Content/Mods bind defaults 0 0
/home/arkserver/instance/ARK.center.save
/home/arkserver/ARK.center/ShooterGame/Saved bind defaults 0 0
To mount these without restarting, run the following command:
mount -a
On Fri, 3 Jan 2020, 12:50 AM Soldize<EMAIL_ADDRESS>wrote:

Hi,
i put the mount commands in "/etc/arkmanager/instances/servername.cfg"
right??
best regards
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/FezVrasta/ark-server-tools/issues/1035?email_source=notifications&email_token=AABD5LCATQCWHDYZV46EHFLQ3XWHZA5CNFSM4IMZCMGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH6MBHY#issuecomment-570212511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABD5LFNAKMGXV3PYZU3F33Q3XWHZANCNFSM4IMZCMGA
.


[terra-clinical-result] Modified Icon pushed out of cell

Bug Report
Description
The modified icon is pushed out of the cell when "View Results" is present with the multiple result indicator. This icon can also be pushed out when a blood pressure cell contains a value with three digits for systolic/diastolic, both with normalcy icons and the multiple value ind.
Steps to Reproduce

Create a modified Flowsheet Result Cell whose value exceeds the width of the cell to get "View Results" to display.
Add an additional result to get the multiple value ind to display.
May need to shrink browser window to reproduce the issue.

Additional Context / Screenshots

Expected Behavior
The modified icon should not be hidden beneath the cell next to it.
Possible Solution

Environment

Component Name and Version<EMAIL_ADDRESS>Browser Name and Version: IE 11
Node/npm Version: Node 8 / npm 6.13.4
Webpack Version: ^4.41.5
Operating System and version (desktop or mobile): desktop

@ Mentions

Additional context:  This needs to be consistent in IE10, IE11, and modern browsers.  IE10 screenshot is shown above.


Not able to get authenticated for certain password encryption types.

Server version - Build from master
Access control is enabled using "openLDAP".
When password encryption type is set to "k5key"/"sha512"/"ext_des" , authentication fails with valid password.
Following errror is seen in logs:
2015-12-11 01:51:51,058 INFO  [:] [] [] [] [qtp26934713-599] [i.a.a.i.l.O.OpenLDAPIdentityProvider] Failed to login to ldap user:qa-santest3 cn=santest3 sanlasttest3,ou=cupertino,dc=rancher,dc=ioOriginal cause: Failed to login ldap User: [LDAP: error code 49 - Invalid Credentials]

Wanted to confirm , that this issue might be caused because of not having configured LDAP to support for these encyption types and this error has got nothing to do with how ldap auth is invoked.

This is due to openldap failing and is not something that can be fixed on our end. If ldap says invalid credentials all we can do is log that.


[BUG] wrong package import

Version
npm installed
Description
There are two wrong imports in the index js. The correct one should be like:
import { modulatorSources } from './soundfont/read_sf2/modulators.js';
import { trimSoundfont } from './soundfont/basic_soundfont/write_sf2/soundfont_trimmer.js';
Reproduction steps
Describe the steps to reproduce the behavior.
Expected behavior
Provide a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
MIDI files and SoundFonts
If the issue only persists with specific MIDIs or SoundFonts, add them here too. (preferably .zip)
Environment
If the issue only persists on a smartphone, or only on a PC, or only on Chrome, etc. please specify.
Additional context
Add any other context about the problem here that may be helpful.

Sorry, my bad! Fixed in 3.16.4
Also make sure to update worklet_processor.min.js in your project :-)

Sorry, my bad! Fixed in 3.16.4
Also make sure to update worklet_processor.min.js in your project :-)

Thanks a lot for your quick reply!


"flynn run" returns "httpclient: unexpected status 400"

When I run one of the following commands, as explained in the docs:
flynn run rake db:migrate
flynn run -r RELEASE_ID rake db:migrate
flynn run -r RELEASE_ID -l -- rake db:migrate
I always get:
POST https://controller.flynn.DOMAIN/apps/APP_NAME/jobs: httpclient: unexpected status 400
Is this a bug or do I missed something during the setup? Where can I find the log-file, so I could debug the request further? Tailing /var/log/upstart/flynn-host.log doesn't gave any answer.
Thanks for any information and further help!

@nlsrchtr check the output of flynn -a controller log for any errors around the time you get the 400 response.
If you can't see anything obvious, run sudo flynn-host collect-debug-info on a host in your cluster and post the resulting gist link here and we'll take a look.

Hi @lmars, I think my setup (running flynn behind a NGINX proxy) is the reason for this behaviour. I change the setup and now it works as expected. Thanks for your response!


Just a suggestion. Not a issue: face_recognition.load_img_file can use cv2.imread than current scipy.misc for fast responses

I am working to improve the reponse time and i found that scipy.misc is taking around 1sec to load huge image files (1mb or more). Whereas cv2 performs best with 200ms or so,
Is it something that can be changed ? Just my 2 cents.
https://www.kaggle.com/zfturbo/test-speed-cv2-vs-scipy-vs-tensorflow

The reason I don't want to include cv2 is because it's a huge dependency that is hard to compile for a lot of people. That's a lot to require be installed just to load images. Also, cv2 stores images in a different pixel format, so you have to know to convert from BGR to RGB. It's a big pain for new users.
 As long as you apply BGR->RGB conversion like in the examples, it should work fine. The only reason face_recognition even has a load_image_file() function is just to make it easy for new users to learn.
As far as a speed test, it's kind of weird to compare scipy vs. cv2 because scipy's imread() function is really just a wrapper around PIL or Pillow (depending on your version of Python). So you could get different results depending on how you have scipy installed and what version of PIL it is using.
When I have a minute, I will remove the scipy dependency and just use PIL/Pillow directly to read the images instead.

Thanks @ageitgey. Yes, you are right. its huge dependency just to read images.
Actually, Even without conversion from BGR to RGB, it is working fine.
I just replaced scipy.misc function with cv2.imread(file). However, when i plot the images. I can see the difference.
Does color conversion effect for detecting facial landmarks to identify faces ?
Thank you

Just a update, when i use cv2 library, even after converting images to rgb, sometimes, face_locations are not identified with model=hog. However, they are identified with model="cnn".
I tried to use PIL directly. However, after reading the image with PIL like below,
def load_image_PIL(file):     Img = numpy.asarray(PIL.Image.open(file))     return I
Howver, at identifyng face_locations step, i get error at _raw_face_locations saying
return face_detector(img, number_of_times_to_upsample) RuntimeError: Expected writable numpy.ndarray with shape set:
Any ideas ?

PIL is best than cv2. PIL has thumbnail function which comes with the aspect ratio for perfect scaling. For far captures better to use high resolution camera.
Load scale times for PIL or cv2 is more or less same. Face locations indetification for both changes. In that PIL thumbnail scaling is much better than manula scaling. This will help in finding the face location even for 300x300 shrinked images.


Add sloped sprites to wallwd8, wallwd16, and wallwd32

The next of my sloped wall PRs. As before, these sprites were made by ripping existing sprites and hand-editing in Paint.NET. Pinging @ZeeMaji for review.

Comparison with RCT1 terrain edge (for texture; color can't be matched). This one is slightly different as the diagonal crossbeam textures don't exist on sloped terrain, only for the full-face texture. Not sure if that is worth trying to replicate here.

Probably fine to forgoe matching it with the terrain edge if the textures & colors are different.

Looks good to me.


OnLaunch=Hide doesn't seem to function well with Browser

Hello,
I'm trying to implement a regular internet browser shortcut to this awesome launcher.
Below is the code I have in config.ini
Entry2=Live Sport;.\assets\icons\sports.png;"C:\Program Files (x86)\Microsoft\Edge\Application\msedge_proxy.exe" --start-fullscreen https://totalsportk.org/nflstreams
When I launch the browser shortcut in the launcher, the browser pops up underneath the launcher despite OnLaunch=Hide in config.ini and I can't see the browser window unless I Alt-tab.
All other implementations seem to work fine, just the internet browser that's causing the issue.
Is this something that can be fixed?
Thank you.

The browser focusing problem is a known issue on Windows. I implemented a workaround for it, but it only runs for a specific list of process names. I just added msedge_proxy.exe to the list. Please try this modified build and let me know if it works for you. If so, I'll merge the change for a new formal release.
As a side note: when using Edge with Flex Launcher, make sure the "Startup boost" option in Edge is disabled.

The browser focusing problem is a known issue on Windows. I implemented a workaround for it, but it only runs for a specific list of process names. I just added msedge_proxy.exe to the list. Please try this modified build and let me know if it works for you. If so, I'll merge the change for a new formal release.
As a side note: when using Edge with Flex Launcher, make sure the "Startup boost" option in Edge is disabled.

It works great! Thank you.
I discovered another issue while testing this, should I close this thread and open a new one?

I discovered another issue while testing this, should I close this thread and open a new one?

Yes.

Actually, keep this one open until I merge the fix into master. Make a new issue for the other problem.


NTR - Add support for ChangeDetection if the article of a variant cha…

…nges

Coverage decreased (-0.02%) to 42.297% when pulling 209183da9a3f67ae9b209f1487eceb3c76a2a756 on ntr/fix-event-listener-for-variant-article-change into 499fff6cd7c857e72c155a98bf757d64320ced2c on master.


python310Packages.awscrt: 0.13.14 -> 0.14.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.awscrt is: "Python bindings for the AWS Common Runtime"
meta.homepage for python310Packages.awscrt is: "https://github.com/awslabs/aws-crt-python"
meta.changelog for python310Packages.awscrt is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.14.0 with grep in /nix/store/36icr3z0hg811lp27x5sjmy0c2k8a099-python3.10-awscrt-0.14.0
found 0.14.0 in filename of file in /nix/store/36icr3z0hg811lp27x5sjmy0c2k8a099-python3.10-awscrt-0.14.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.awscrt
python39Packages.awscrt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/36icr3z0hg811lp27x5sjmy0c2k8a099-python3.10-awscrt-0.14.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/36icr3z0hg811lp27x5sjmy0c2k8a099-python3.10-awscrt-0.14.0
ls -la /nix/store/36icr3z0hg811lp27x5sjmy0c2k8a099-python3.10-awscrt-0.14.0/bin

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

2 packages built:

python310Packages.awscrt
python39Packages.awscrt

Maintainer pings
cc @davegallant for testing.

Result of nixpkgs-review pr 186366 run on x86_64-darwin 1

2 packages built:

python310Packages.awscrt
python39Packages.awscrt


[v8] Express - Custom JS per form

In order to do advanced customization for forms to make them more user-friendly, I'd love to be able to use custom jQuery on the form.  I envision this as being a HTML include per Express Object form where we can include custom js, css, etc as necessary.  Alternatively, automatically include JS from /application/express/forms/js/[objecthandle-formname].js (same for CSS).  I know the goal is to have Express Object eliminate the need for custom code, but giving developers the ability to do more advanced coding will make them even more powerful.

Surely this would be achieved through some custom templates, in applications?


Exercises: Further resource for RSpec mock

Complete the following REQUIRED checkboxes:

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this issue follows the location for request: brief description of request format, e.g. Exercises: Add exercise on XYZ

The following checkbox is OPTIONAL:

[x] I would like to be assigned this issue to work on it

1. Description of the Feature Request:
I think the explanation and linked resources for stub and mock in Exercise 13_ input_output are a little too technical and non-intuitive for the uninitiated beginner. It was difficult for me to understand why I might even need stub and mock and where they should be used when I first took the exercise.
For a simpler explanation -- both in words and in code, I found a resource online The ‘no problemo’ basic guide to doubles and stubs in RSpec that illustrated the concept of mock in a manner that's much more intuitive (and fun). I suggest adding it to the list of stub/mock resources in the exercise.
2. Acceptance Criteria:

3. Additional Information:

Hey @Imfae
Yeah I'm on board with this. A gentler introduction is a good thing. I'll assign you.


#654 asset structured

Related Issue
Closes #654

Description

Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@rohansx don't merge this i will be adding more commits in this for more categories
tell me how many books we should keep for each category?
i think 10 would be enough
so that the website is also light-weight
also deleted unused images and structured them

@k-deepak04 ok, do according to you

@rohansx can merge this
work is done


Westworld host attribute matrix

Hi DeepMind.
No issue here; just came here to say that this project scaringly reminds me od Westworlds host attribute matrix 😄
Even some attributes are similar, such as exploration == curiosity.

Cheers!

That is funny - and maybe this will give us some more spokes for future bsuite tasks.
For example, could Schmidthuber's formal theory of fun allow us to add a "humour" spoke to the radar plot?
http://people.idsia.ch/~juergen/creativity.html
Thanks for pointing out!


4.1 support

Can you add support for v4.1 of the Esri JS API?

Hi @MarcBate. I've had a quick look at upgrading to 4.1. Unfortunately - as you may have noticed if you tried to upgrade as well - it seems you can no longer inherit from esri classes in typecscript just using a simple extends statement on the custom class as you could in 4.0.
Something to do with changes they've made to the accessor class and how it's implemented/inherited in their classes. That's all I've had to time to figure out for now...think I'll need to dig into the transpiled javascript and compare it to some samples esri has for creating custom classes, although I couldn't find any samples for 4.1 - do you know of any? Also might just hit up esri support or forums to see if they can let me know how to get around it.
That's the first hurdle anyway, there'll no doubt be other issues once that's been sorted out.  I need to upgrade to 4.1 for one of my projects so it's on my radar...when that can be done I don't know though, sorry. If you manage to work it out please let me know.

Look at supporting upgrade for 4.1. Might be an all or nothing upgrade as far v 4.x is concerned though. I don't think 4.0 and 4.1 will work together in one code base...could create two separate versions for 4.0 and 4.1. Keep 4.0 in a branch and hopefully upgrade to 4.1 support in master.

now upgraded for 4.1.


Progression overhaul (and new stations)

TODO: Write down when advancements should trigger
Assembly station overhaul
The assembly station is currently too expensive to get very early in the game, due to it requiring deepslate and a schematic, which requires paper.
Makeshift assembly station
A more early game friendly alternative to the assembly station is required that costs a lot less and only works on wooden and stone parts.
Assembly station
The midgame assembly station's recipe will also need to be rebalanced, I think it could use being a bit more expensive and requiring iron or something, to properly add in progression.
An anvil could possibly turn it into a multiblock structure that can also repair the tools as well as create them in a single UI.
Possibly by making the anvil repair window appear below the existing window.
Maybe it should only take in tools as well, and disallow repairing tools with a normal anvil that is not part of a multiblock structure.
New part crafting stations
Supposed to be replacements to using the crafting table.
Possibly opt-in as Sigmund suggested.
Woodworking station
Allows you to create tool parts from wood, and if a masonry station is placed next to it it turns into a multiblock (masonry station) that can be used to craft both wooden and stone tool parts.
The UI should have a single input and a single output, with a scrollable button determining which schematic is used.
Only basic schematics can be used in this station, and you do not need to make them.
Forging station
This station is used to craft metal tool parts.
The UI should have a single input and a single output, with a scrollable button determining which schematic is used (selecting from schematics in your inventory, and in the future possibly a schematic chest attached to the forging station).
All schematics can be used in this station.

I think we also need to rethink the assembly station as a whole. It tries to do two things, combine assembly and disassembly, which has lead to loads of bugs, and general confusion. I think it's a good idea to introduce disassembly as a separate station. I like the idea of tiering the station, so that you start out with a wooden one, upgrade it to stone, then metals and soon, to be able to disassemble parts of a higher material tier. Hopefully that would lead to a natural progression that would be easy to tie into advancements

Also like the idea of a the woodworking + masonry station, that's neat

Glad to hear you like the ideas! I agree, though I think the disassembly issue can be fixed with better UI/UX. We'll have to experiment though!

Progression idea:
have a "primary" workbench that in the endgame can do everything: upgrading, swapping components, assembly, disassembly. UI will give a nice feeling for it. Proposal for UI: have a slot for a schematic. Once a schematic is placed, the UI updates to show custom item slots based on that schematic for all parts that can be combined using the given schematic.
For the intermediate progression steps, just restrict the capabilities.

Restrict the materials that can be used for crafting to the "hardest" material that was used in crafting the workbench. (so if it's made from wood and stone, can't use it for iron or diamond and so on)
have the workbench quality restrict how many parts can be combined. Good schematic has for example 5 slots, but in a bad bench you can only provide handle and head. In a better bench you can also include a binding and for even higher benches, all slots will be unlocked.
just placing a tool allows for swapping out individual parts of the tool. All parts that aren't accessible through this bench (due to poor quality) cannot be taken out and thus would be lost.

Rethinking the progression strategy
Note: I think this could work well in addition to newer stations mentioned above.
Problem
Forgero does not take the natural progression of the game into account. The gameplay is scattered at different places with no real goals in mind, except making it possible to craft and upgrade custom tools.
Strategy
Create an overview of the goals a player is trying to achieve at different points in the game, and see how Forgero's strengths can be used to help the player progress to new goals.
Forgero's strengths
Combine anything, use everything
Forgero makes it possible to attach attributes and gameplay features to items. It also provides custom tools/parts that can be upgraded and modified with parts and items.
Goals?
The gameplay loop in vanilla Minecraft is based on crafting items that can be used to solve a problem. These problems might be small. "I need a stone pickaxe to mine iron.", and they might be big: "I need really good weapons to beat the wither." They can also take a long time: "I need to find another cow so I have a solid food supply."
The purpose of defining clear gameplay goals is to identify areas where Forgero's unique ability to combine tools and materials can provide a satisfactory way to solve a problem.
Example situations from the first day in a new survival world
By combining a goal with a time frame, it is easier to find unique situations where Forgero could have been used to enhance the progression of a player. The first day is pretty simple, find everything you need to survive the first night.
Wood -> Stone pickaxe
The wooden pickaxe is pretty useless, you discard it after having mined three stone blocks. It is usually left in the inventory or thrown away so it does not consume space. By surrounding a wooden pickaxe with three stones, it could be turned into the stone variant of a tool, without the use of a crafting table. The pickaxe is then upgraded to the next tier without having to craft a separate pickaxe and use more sticks. This is faster and less resource intensive than doing it the vanilla way
Wooden tools + coal
Upgrading a wooden tool with coal can set it on fire. The tool slowly burns away, but can be a powerful weapon against enemies, as it will set them on fire.
Right clicking a bee will make your tools poisonous
Bees are poisonous, and interacting with them can make your tool inflict more damage to enemies for a given number of hits
A single iron ore can be used to upgrade the tip of a stone pickaxe
Upgrading your stone pickaxe with a single iron ore will increase the durability, speed and mining level, without taking too many materials. This would be a less resource intensive way of improving your tool if materials are limited.
An iron tool should create fire when interacting with flint
Instead of crafting a flint and steel, simply put the flint in the off-hand and an iron tool in the right hand, then right click.
Upgrading with snowballs
Packing snowballs on your tools can provide freezing effects on mobs.

There are of course many other good ideas, and I would love to hear them!
Identifying new features new features like these
I knew I wanted to find early game scenarios where Forgero could be useful, so I made my goal to start a new save, and do what I needed to be set up for the first night. Every time I encountered a new situation, I paused and looked around to see if there was something in the world or inventory that could help me overcome it by thinking about how Forgero could have been used to craft something interesting or upgrade my tools. These could be things like encountering ore, which I need to be able to mine, a hill that needs to be climped, lakes I need to cross, or monsters appearing from caves. The goal here is not to force players into using Forgero to solve their problems, but to make the player feel smart for discovering a new/better way to solve their problem using Forgero.
Next steps!
It's not possible to implement everything. There is no time. There are also plenty of solutions that would be better served as being a separate mod, but first we need to identify the possible new features. I think trying to split the gameplay progression of a complete vanilla game into separate steps can help us see where Forgero could be used to provide new paths for reaching the next level.
Here are some examples, but I would ideally like to create some sort of map of the entire Minecraft experience to see where we could fill in possible gameplay gaps. These are only ideas to get the mind going, there are not planned features. For the most part I need to sit down and play more to get a better understanding of the different steps you are going through and which new features can improve upon the vanilla experience without feeling like cheating.
First cave exploration

Make it possible to upgrade tools in place using new materials so the player does not have to resurface
Different weapons are better suited against different enemies. Maces deal more damage to skeleton like mobs
Using the flint and iron weapon can set fire to enemies
Wrapping your handle with leather can give a slightly higher ore drop chance.
Creating a makeshift bow can make it possible to use the arrows dropped from skeletons
Using gunpowder from creepers creates a small explosion
Looting cave chests can give unique upgrades that can be used on your current tools/weapons
Shoes made of wool decreases detection range of mobs
Redstone on weapons/arrows will trigger a creeper explosion no matter how far away
Sabers can instamine cobweb

Finding food sources

Knives can be crafted to increase the amount of meat dropped from animals
Maybe there is an accessible upgrade to increase the drop chance of seeds from grass during the early game

There are plenty other gameplay scenarios we can focus on, and I think we should try to categorise them as best as possible so we have a good understanding of which part of the game we are trying to improve. Some more examples are: Exploring the nether for the first time, Beating the ender dragon, finding your first village, beating a village raid, building a proper base, exploring new biomes and more.

I did CTRL + F did not find smithing I am dissapointed :kekw:

I'd love to have a fling back armor which you get by dipping your armor into slime. Only negative side effect, it increases the duration for which you burn and once the time is over, slime is burnt away. The slime also is washed away when entering water.
Oh, the positive effect is a certain chance to reflect projectiles back to their sender and redirect melee attacks to other surrounding creatures


Upgrade Qiskit & Django

Summary
Upgrade the Qiskit used version.
Upgrade Django version.
Details and comments
#130

[x] update requirements
[x] update requirements-dev
[x] update requirements-doc
[x] update setup.py
[x] update get version fonc
[x] update guides import
[x] update readmes

#137

[x] Update requirements
[x] update readme
[x] update CI

Closes #130
Closes #137

@IceKhan13 @AnsahMohammad ready for your review :)


devcontainer: replace VAULT_HOST with AWS_ROLE_ARN

Description

This PR is replacing the VAULT_HOST variable with AWS_ROLE_ARN. This is required to use the new token service to get AWS credentials.
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

LGTM, merging branch-24.10 back in, just to make sure there's no conflicts with other changes.


SQL Editor Background Color

 Issue by Eric-Da
Monday Jan 06, 2020 at 14:33 GMT
Originally opened as https://github.com/dbeaver/dbeaver/issues/7540

There's an issue since upgrading to 6.3.2 where I can no longer change the SQL editor background color. When I hit the apply button, it temporarily changes until I close the preferences window, then it goes back to grey.

 Comment by Eric-Da
Monday Jan 06, 2020 at 15:08 GMT

Looks like this only happens when Dark Mode is chosen

 Comment by serge-rider
Monday Jan 06, 2020 at 17:11 GMT

It seems to be an Eclipse bug (https://bugs.eclipse.org/bugs/show_bug.cgi?id=464226)

 Comment by Eric-Da
Monday Jan 06, 2020 at 18:11 GMT

Thanks. Hopefully they fix it soon.

 Comment by Eric-Da
Monday Mar 23, 2020 at 12:01 GMT

Do you have any idea when this will be fixed? Looks like the eclipse bug (https://bugs.eclipse.org/bugs/show_bug.cgi?id=464226) isn't getting any attention


git pull hangs

Expected behavior
As a IDE developer, I want to be able to test and debug our code so that I can analyze what is going on and trace down bugs.
Actual behavior
It seems as if git pull when called on our ide-urls repo hangs if we call it from Java esp. in debug mode.
It does not produce any output and also observing the process and its file handles in the task manager / process explorer does not shed any light into the problem.
Once when I waited for a very long time (more than 20 minutes) some timeout seemed to have been reached and git pull ended without any output or error message and exit code 1:
Running command 'git' with arguments 'remote' ...
Running command 'git' with arguments 'pull' ...
Running command 'git' with arguments 'pull' failed with exit code 1!
Failed to update git repository at D:\projects\_ide\urls
See above error for details. If you have local changes, please stash or revert and retry.

Steps to reproduce (bug) / Use Case of feature request (enhancement)

Run ide command in debug mode (not help but something that requires clone or pull of urls such as ide mvn -v).

Comments/Hints:
Currently I am clueless why this error happens.
When I run git pull in e.g. git-bash this never happens so IMHO it is something related to Java and Process.
Usually redirection of output seems to work fine but maybe even git pull actually writes something on standard out or error but some strange bug in ProcessContext hides this important information.
Actually I have never tested redirection of standard error yet. So maybe this should be the first thing to verify...
Affected version:

OS: Windows

I just stumbled over this one that seems to be a perfect match to our problem:
https://stackoverflow.com/questions/41745520/how-to-use-git-commands-without-human-interaction

From git v2.3.0, you can set the environment variable GIT_TERMINAL_PROMPT=0 to avoid your script to hang on the git command expecting manual authentication.

So it is not only required auth but any kind of interaction that git will not be waiting for.
IMHO we should try this and see if the "git hangs" error disappears (and maybe we will then find out the true reason why it was hanging such as some TLS confirmation, merge conflict resolution, or whatever).
PR #155 contains a great feature that is not directly related to this story so I would suggest that we create a new story for that feature (force git update for urls) and switch #155 to that new story while we create a small new PR implementing the non-interactive git invocation to fix this story.


geometric distribution stats.geom returns negative values if p is very small

For very small success probability p, stats.geom returns negative numbers.
Reproducing code example:
from scipy import stats
import numpy as np

np.random.seed(1234)
stats.geom(np.exp(-20)).rvs(size=5) # ok, large positive numbers
stats.geom(np.exp(-25)).rvs(size=5) # all equal to -2147483648
stats.geom(np.exp(-30)).rvs(size=5) # all equal to -2147483648

np.random.seed(1367)
stats.geom(np.exp(-25)).rvs(size=5) # all equal to -2147483648, except the last<PHONE_NUMBER>

Scipy/Numpy/Python version information:
<<Output from 'import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)'>>
1.0.0 1.14.1 sys.version_info(major=3, minor=5, micro=4, releaselevel='final', serial=0)

https://github.com/scipy/scipy/blob/v1.1.0/scipy/stats/_discrete_distns.py#L262 :
 vals = ceil(log(1.0-q)/log(1-p)) should probably use log1p(-q) / log1p(-p)

this looks like integer overflow in numpy.random
>>> np.random.geometric(np.exp(-25), size=5)
array([-2147483648, -2147483648,   998287931, -2147483648, -2147483648])

In my case, there is no negative returns.
>>> print(stats.geom(np.exp(-20)).rvs(size=5))
[103145472 472137975 279343132 746565655 734548744]

>>> print(stats.geom(np.exp(-25)).rvs(size=5))
[<PHONE_NUMBER>6<PHONE_NUMBER>6<PHONE_NUMBER>09<PHONE_NUMBER>72<PHONE_NUMBER>17]

>>> print(stats.geom(np.exp(-30)).rvs(size=5))
[<PHONE_NUMBER>898<PHONE_NUMBER>982<PHONE_NUMBER>1979<PHONE_NUMBER>0112
<PHONE_NUMBER>382]

numpy verseion is 1.14.5 and scipy version is '1.2.0.dev0+b7865dd'

I guess this is one of the int32/int64 cases that are platform specific that Robert Kern mentioned in the numpy random refactoring and enhancement discussion.

When I checked what you said, my result was int64.
@chrisb83 In your case, what is the result of int32/int64?

I can replicate the overflow in 1.14.4 and 1.15.2 Win10 but cannot on 1.14.5 and 1.15.2 on Ubuntu 18.04.

on python 2.7 it looks ok on my box (non dev env):

import numpy as np
np.random.seed(1234)
stats.geom(np.exp(-20)).rvs(size=5) # ok, large positive numbers
array([103145472, 472137975, 279343132, 746565655, 734548744])
stats.geom(np.exp(-25)).rvs(size=5) # all equal to -2147483648
array([<PHONE_NUMBER>6,<PHONE_NUMBER>6,<PHONE_NUMBER>09,<PHONE_NUMBER>72,
150269801017])
stats.geom(np.exp(-30)).rvs(size=5) # all equal to -2147483648
array([<PHONE_NUMBER>898,<PHONE_NUMBER>982,<PHONE_NUMBER>1979,<PHONE_NUMBER>0112,
4940962322382])

Lucass-MacBook:~ rlucas$ python -V
Python 2.7.15
and python3 also looks ok:
Lucass-MacBook:~ rlucas$ python3
Python 3.7.0 (default, Jun 29 2018, 20:13:13)
[Clang 9.1.0 (clang-9<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

from scipy import stats
import numpy as np
np.random.seed(1234)
stats.geom(np.exp(-20)).rvs(size=5) # ok, large positive numbers
array([103145472, 472137975, 279343132, 746565655, 734548744])
stats.geom(np.exp(-25)).rvs(size=5) # all equal to -2147483648
array([<PHONE_NUMBER>6,<PHONE_NUMBER>6,<PHONE_NUMBER>09,<PHONE_NUMBER>72,
150269801017])
stats.geom(np.exp(-30)).rvs(size=5) # all equal to -2147483648
array([<PHONE_NUMBER>898,<PHONE_NUMBER>982,<PHONE_NUMBER>1979,<PHONE_NUMBER>0112,
4940962322382])
np.random.seed(1367)
stats.geom(np.exp(-25)).rvs(size=5) # all equal to -2147483648, except the last<PHONE_NUMBER>
array([156454800182,<PHONE_NUMBER>,<PHONE_NUMBER>5,<PHONE_NUMBER>3,
1670264715])
import sys, scipy, numpy; print(scipy.version, numpy.version, sys.version_info)
1.1.0 1.15.1 sys.version_info(major=3, minor=7, micro=0, releaselevel='final', serial=0)

@akahard2dj, @ev-br, I investigated a little more in the process of addressing requested changes from @chrisb83 on PR #9309 I b/c the negative values will still show up on int64 by for a smaller value of the probability. I also tried the switch to log1p() method, and you do still get some negative values from under/overflow issues:
(scipy-dev) Lucass-MacBook:stats rlucas$ python
Python 3.6.6 |Anaconda, Inc.| (default, Jun 28 2018, 11:07:29)
[GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

import sys, scipy, numpy; print(scipy.version, numpy.version, sys.version_info)
<module 'scipy.version' from '/Users/rlucas/scipy-dev/scipy/scipy/version.py'> <module 'numpy.version' from '/Users/rlucas/anaconda3/envs/scipy-dev/lib/python3.6/site-packages/numpy/version.py'> sys.version_info(major=3, minor=6, micro=6, releaselevel='final', serial=0)
from scipy.stats import geom
import numpy as np
stats.geom(np.exp(-25)).rvs(size=5)
Traceback (most recent call last):
File "", line 1, in
NameError: name 'stats' is not defined
geom(np.exp(-25)).rvs(size=5)
array([<PHONE_NUMBER>1,<PHONE_NUMBER>83,<PHONE_NUMBER>3,<PHONE_NUMBER>52,
32707250124])
geom(np.exp(-50)).rvs(size=5)
array([-9223372036854775808, -9223372036854775808, -9223372036854775808,
-9223372036854775808, -9223372036854775808])
type(geom(np.exp(-50)).rvs(size=1)[0])
<class 'numpy.int64'>

@ev-br
if I change vals = ceil(log(1.0-q)/log(1-p)) to vals = ceil(log1p(1.0-q)/log1p(1-p)), test is failed.
self = <scipy.stats.tests.test_distributions.TestGeom object at 0x6fff6673550>

    def test_ppf(self):
        vals = stats.geom.ppf([0.5, 0.75, 0.875], 0.5)
        expected = array([1.0, 2.0, 3.0])
>       assert_array_almost_equal(vals, expected)
E       AssertionError:
E       Arrays are not almost equal to 6 decimals
E
E       (mismatch 66.66666666666666%)
E        x: array([1., 1., 1.])
E        y: array([1., 2., 3.])

expected   = array([1., 2., 3.])
self       = <scipy.stats.tests.test_distributions.TestGeom object at 0x6fff6673550>
vals       = array([1., 1., 1.])

I am figuring out this problem. I will put the PR.

@akahard2dj The alternative for log(1-x) is log1p(-x), not log1p(1-x) (which is log(1 + 1 - x))

The implementation stats.geom.rvs is now:
    def _rvs(self, p, size=None, random_state=None):
        return random_state.geometric(p, size=size)

and it has relied on NumPy to generate random variates since 2013 (at least), so the discussion above about ppf was a separate issue that seems to have been fixed in gh-9320.
There are two problems here, only one of which is SciPy.
If RandomState is used to generate the random variates, we have a problem that we can't fix in SciPy.
np.random.seed(0)
np.random.geometric(np.exp(-25))  # -2147483648
type(np.random.geometric(np.exp(-25))) # int
np.random.geometric(np.exp(-25), size=5).dtype # dtype('int32')

If Generator is used to generate the random variates, NumPy is fine:
rng = np.random.default_rng(0)
rng.geometric(np.exp(-35))  #<PHONE_NUMBER>351774
type(rng.geometric(np.exp(-35))) # int
rng.geometric(np.exp(-35), size=5).dtype # dtype('int64')

but SciPy converts it to int32 when size != (), at least on some platforms:
stats.geom.rvs(np.exp(-25), size=5, random_state=rng).dtype # dtype('int32')

The relevant code is:
https://github.com/scipy/scipy/blob/7abf27f2fe9046814a0eb6e618ad310bd29a15c4/scipy/stats/_distn_infrastructure.py#L1104-L1111
I think it is undesirable for the dtype behavior to be platform specific, so I think we should consider SciPy's conversion to int32 a bug and just fix it by specifying np.int64 instead of int. With that one change, the problem is fixed, and all tests pass. (If we think that np.int32 is preferred in some cases, then it seems that we should start with asking NumPy to offer dtype as a parameter for all Generator rvs methods. I don't think we should just "fix" NumPy's choice here.)
I'll submit a PR.


CS:CZ Installing wrong APP ID

Bug Report
System Information

Operating System (Including distribution name and version number)
AMP version and build date (Always use the version number, 'latest' is not valid!)
Which AMP release stream you're using (Mainline, Nightly or FastTrack)
Ubuntu 20.04 LTS
AMP Release "Europa"
v2.0, built 10/07/2020 21:34

I confirm:

[x ] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x ] that my operating system is up-to-date.
[x] that my checkboxes above look like the one on the left (remove this line when done)

Symptoms

Installing Counter Strike: Condition Zero, the installer is installing the wrong app id. It is actually installing Counter Strike: Source.
Condition Zero is app id 90 + it also needs "+app_set_config "90 mod czero" since it is off of hlds . It is instead installing 232330 (which is cs:s).
I tested the install for cs 1.6 and it works well, so currently cs:cz is the only issue as far as I know
Reproduction

Enter reproductions steps here.
Start clean install.
Update server
Server installs cs:s instead of cs:cz

Further investigation in the apps.json, I noticed this:
  {     "AppID": 232330,     "ClientAppID": 80,     "Server": "Counter-Strike: Condition Zero Dedicated Server",     "SteamCMD": true,     "AnonymousLogin": true,     "ExecutableWin32": "hlds.exe",     "ExecutableLinux": "hlds_run",     "GameName": "czero",     "Args": "+app_set_config \"\"90 mod czero",     "Srcds": true   },
Just need to change the Appid to 90

Alright, after a few hours I have found the solution that should help and fix this current issue. The Apps.json located in the instance needs to be updated to the following for condition zero install:

{
"AppID": 90,
"ClientAppID": 80,
"Server": "Counter-Strike: Condition Zero Dedicated Server",
"SteamCMD": true,
"AnonymousLogin": true,
"ExecutableWin32": "hlds.exe",
"ExecutableLinux": "hlds_run",
"GameName": "cstrike",
"ModName": "czero",
"Args": "+app_set_config ""90 mod czero",
"SettingMaps": { "srcdsModule.SRCDS.Map": "de_dust2_cz" },
"Srcds": true
},

@PhonicUK  I've posted the solution for this, if you could update the json with this, that would be awesome. Thanks!

Fixed in the latest build :)


not entering sleep mode properly

Hi, as the SleepMode is not entered most of the times with the current setMode void i suggest the following changes:
put the modifyRegister: https://github.com/autowp/arduino-mcp2515/blob/master/mcp2515.cpp#L167
inside the while loop so it will rewrite the register.
also set the timeout from 10ms to 200ms as the sleep mode entering can take up to 100ms
i also suggest to add the sleepWakeup bit (CANINTF_WAKIF) into the reset void: https://github.com/autowp/arduino-mcp2515/blob/master/mcp2515.cpp#L50
so it will wake up on can messages, or maybe make an extra function to enable or disable that bit so everyone can set if needed.

Current code works for me for entering the sleep mode.
Do you mind enlighting me where the specification of that entering sleep mode can takes 100ms? I didn't find it in the datasheet.

I tested the time it took to bring it to sleep mode with the edited function, and sometimes it was up to 100ms, it was not from the datasheet.
With the stock function i was not able get it into standby mode by calling it one time, even with disabled timeout.


Ignore broken definition when running flow-typed install

If there is a broken definition, how can you set it to be ignored when you run this command?

Try adding express and next as devDependencies, and installing their flow-typed definitions. You should then get the correct types within next.


with-reasonml example - state variables not persisting in runtime

Examples bug report
Example name
with-reasonml
Describe the bug
counter is not shared like the example documentation (copied from shared-modules-app) states it should be: "That Counter component maintain the counter inside its module. This is used primarily to illustrate that modules get initialized once and their state variables persist in runtime" ~ https://github.com/zeit/next.js/tree/master/examples/with-reasonml
Note that I have updated the dependencies for this example in this pull request:
https://github.com/zeit/next.js/pull/5048
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Increment the counter on index page
Click on about link in header
The counter resets to 0 (in the shared-modules-app example it maintains value between pages)

Expected behavior
I would expect the counter to be shared between index and about page and maintains 1 value.
The reasonml example seems like it was adapted from the shared-modules-app example where the counter is shared between pages.  I assume the reasonml example should function exactly the same and the transpiled reason->javascript is causing next.js to reinitialize component for each page.  I would really appreciate next.js advice on this.
System information

OS: Windows 10
Browser: Chrome
Version of Next.js: 6.1.1

Additional context
I am just getting up to speed with react/next/reason and really appreciate your help understanding if things are actually working as expected and reason components can't be shared between pages, or if the reason example is incorrect and there is some way to write it better for next.js.

@timneutkens At your convenience could I get your advice on this?  Ideally you might know someone with next/reason experience that might know how the example needs to be modified to behave as documented. I know reason is not yet very standard so the next best thing could be to identify what in the transpiled javascript causes next.js to behave different from shared-modules-app and perhaps we get reason team resource in to help improve the example code to transpile to next.js friendly js.  Or perhaps you can tell that next.js could be improved to handle this transpiled code better.  Worst case, this example is just the best we can expect from next/reason combination and we can change the docs to indicate why shared modules can't be expected.
transpiledjs.zip

Hey @rase-, I noticed you made some excellent changes to this example in the past.  Would you be able to give your advice on if there is anything that can be done to make the counter shared between pages just like the non-reasonml example works?  I am a little afraid to continue with next.js/reasonml combo without understanding the performance severity of this issue.

I am going to hope and assume this isn't a big deal.


z/OSMF Lite - Closed beta and delivery

Execute closed beta of z/OSMF Lite (internal and external users) to validate design

Closing as individual members are participating and this does not need to be tracked at the ZLC


Upgrade to a more recent version of markdownlint-cli

From 0.31.1 to 0.38.0
We have not upgraded the version used in CI for a long time, and the new version includes some additional checks such as MD051 link-fragments (Link fragments should be valid), which helps avoid deadlinks in the documentation (internal links in a given document).
Contributors can install the new version of markdownlint-cli using npm or Brew (macOS).

/skip-all


Experimental: builds with dual FAT MSC support.

Highly experimental mass-storage supporting builds of Pico firmwares. Uses a dual device setup with one readable to the host and one readable to the device.
:warning: Don't flash these builds on your Pico unless you've backed up and run flash_nuke.uf2! They will break your filesystem and do terrible awful things.

Kind of like how Circuit Python does it? I can see this ability as a bonus on devices like the Pico Lipo 16mb, or the Tiny 8mb.

Kind of like how Circuit Python does it?

Yes, but with none of the careful consideration the CircuitPython team put into making it work at least somewhat well, and documenting how to use it :laughing:
Though I don't think they make a habit of splitting the filesystem like this- it's always readable to the host and write-locked for the device by default, IIRC, and can be changed from Python if you need to write log files or something?


Implement GLV scalar multiplication algorithm

Some preliminary bench results:
bls12_377: g1_mul_assign
                        time:   [87.016 µs 87.050 µs 87.091 µs]
                        change: [-47.347% -47.283% -47.229%] (p = 0.00 < 0.05)
                        Performance has improved.
bls12_377: g1_is_in_correct_subgroup
                        time:   [53.454 µs 53.485 µs 53.516 µs]
                        change: [-11.952% -11.651% -11.381%] (p = 0.00 < 0.05)
                        Performance has improved.
bls12_377: g2_mul_assign
                        time:   [268.61 µs 268.80 µs 269.00 µs]
                        change: [-56.031% -55.960% -55.888%] (p = 0.00 < 0.05)
                        Performance has improved.

^Rebased here.


GitHub Action can't find Tighten sniff

I keep getting the following error for my PHPCS GitHub Action.
ERROR: Referenced sniff "Tighten" does not exist

I've only made minor changes to the files. (Update to PHP 8.0, adding composer auth.json, ignoring a few rules)
Here is my liny.yml and my .phpcs.xml.dist files:

lint.yml
name: Lint

on:
    push:
        branches: [ main ]
    pull_request:

jobs:
    phpcs:
        name: PHPCS

        runs-on: ubuntu-latest

        steps:
            - uses: actions/checkout@v2

            - name: Setup PHP
              uses: shivammathur/setup-php@v2
              with:
                  php-version: '8.0'
                  extensions: posix, dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick
                  coverage: none

            - name: Add HTTP basic auth credentials
              run: echo '${{ secrets.COMPOSER_AUTH_JSON }}' > $GITHUB_WORKSPACE/auth.json

            - name: Install dependencies
              run: composer install --no-interaction --no-suggest --ignore-platform-reqs

            - name: PHPCS lint
              uses: chekalsky/phpcs-action@v1
              with:
                phpcs_bin_path: './vendor/bin/phpcs'

    tlint:
        name: TLint

        continue-on-error: false

        runs-on: ubuntu-latest

        steps:
            - uses: actions/checkout@v1

            - name: Setup PHP
              uses: shivammathur/setup-php@v2
              with:
                  php-version: '8.0'
                  extensions: posix, dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick
                  coverage: none

            - name: Add HTTP basic auth credentials
              run: echo '${{ secrets.COMPOSER_AUTH_JSON }}' > $GITHUB_WORKSPACE/auth.json

            - name: Install dependencies
              run: composer install --no-interaction --no-suggest --ignore-platform-reqs

            - name: Tlint Lint
              run: vendor/bin/tlint

.phpcs.xml.dist
<?xml version="1.0"?>
<ruleset>
    <file>app</file>
    <file>config</file>
    <file>database</file>
    <file>public</file>
    <file>resources</file>
    <file>routes</file>
    <file>tests</file>

    <rule ref="Tighten"/>

    <rule ref="PSR1.Files.SideEffects">
        <exclude-pattern>public/index.php</exclude-pattern>
    </rule>
    <rule ref="PSR1.Methods.CamelCapsMethodName">
        <exclude-pattern>*</exclude-pattern>
    </rule>
    <rule ref="Squiz.Classes.ValidClassName">
        <exclude-pattern>REDACTED.php</exclude-pattern>
    </rule>

</ruleset>

No worries if you guys can't help. Been hacking on this for a while and can't seem to figure out the issue.
Thanks in advance!

@driftingly Looks like that fixed it right up. Should it be added to the installation instructions for duster?


[Bug]: Cannot mix different versions of joi schemas w/ SvelteKit + Vite and pnpm - Request to export types through Core

Current Behavior
Runtime error:
Error: Cannot mix different versions of joi schemas
    at new module.exports (node_modules/@hapi/hoek/lib/error.js:23:19)
    at module.exports (node_modules/@hapi/hoek/lib/assert.js:21:11)
    at Object.exports.isSchema (node_modules/joi/lib/common.js:132:5)
    at Object.internals.schema (node_modules/joi/lib/compile.js:66:16)
    at Object.exports.schema (node_modules/joi/lib/compile.js:17:26)
    at internals.Base.$_compile (node_modules/joi/lib/base.js:646:24)
    at node_modules/joi/lib/types/keys.js:262:92
    at Object.exports.tryWithPath (node_modules/joi/lib/common.js:176:16)
    at internals.Base.method [as keys] (node_modules/joi/lib/types/keys.js:262:32)
    at Object.args (node_modules/joi/lib/types/keys.js:49:23)

Expected Behavior
Page should work (it does with the following package versions):

@web3-onboard/common: 2.2.3
@web3-onboard/core: 2.14.1
@web3-onboard/injected-wallets: 2.6.2
@web3-onboard/walletconnect: 2.2.2

Steps To Reproduce

Clone https://github.com/beeb/lp-rescue-frontend
Run pnpm build and pnpm preview to see that everything works
Run pnpm update to get the update to the latest version of web3-onboard packages
Run pnpm build and pnpm preview to see a 500 error

What package is effected by this issue?
@web3-onboard/core
Is this a build or a runtime issue?
Runtime
Package Version

@web3-onboard/common: 2.2.3
@web3-onboard/core: 2.15.4
@web3-onboard/injected-wallets: 2.8.1
@web3-onboard/walletconnect: 2.3.1

Node Version
16.19
What browsers are you seeing the problem on?
No response
Relevant log output
Error: Cannot mix different versions of joi schemas
    at new module.exports (node_modules/@hapi/hoek/lib/error.js:23:19)
    at module.exports (node_modules/@hapi/hoek/lib/assert.js:21:11)
    at Object.exports.isSchema (node_modules/joi/lib/common.js:132:5)
    at Object.internals.schema (node_modules/joi/lib/compile.js:66:16)
    at Object.exports.schema (node_modules/joi/lib/compile.js:17:26)
    at internals.Base.$_compile (node_modules/joi/lib/base.js:646:24)
    at node_modules/joi/lib/types/keys.js:262:92
    at Object.exports.tryWithPath (node_modules/joi/lib/common.js:176:16)
    at internals.Base.method [as keys] (node_modules/joi/lib/types/keys.js:262:32)
    at Object.args (node_modules/joi/lib/types/keys.js:49:23)

Anything else?
Seems like a regression of https://github.com/blocknative/web3-onboard/issues/1287
Sanity Check

[X] If this is a build issue, I have included my build config. If this is a runtime issue, I have included reproduction steps and/or a Minimal, Reproducible Example.

Hey @beeb I am unable to recreate the error described with the steps provided.
Try removing the @web3-onboard/common package from your package.json as it is not needed and is imported by the packages that require it.

Hi @Adamj1232 thanks for the reply! I'm using the Chain type from the common package to make my interfaces typesafe. What's an alternative way I could import that type without the common package?

I confirm that removing @web3-onboard/common solves the issue. If I could get the Chain type from somewhere else it would be helpful. Does any of the other packages re-export it?

@beeb you should still be able to access @web3-onboard/common after removing from your package.json as it is a dep of @web3-onboard/common package. Are you seeing an error when importing that type?

I checked which version is used by @web3-onboard/core and it appears right now it's<EMAIL_ADDRESS>so I pinned the version to 2.2.3 and it works for now. But you guys should ideally fix this!
At least have your packages depend on the latest version of common.

Ah makes sense, we will update Core to export all the types from common so this wont be required in the future.
Agreed it isnt an ideal fix but common doesnt change often and we will have the types exported through Core shortly

After testing with a different system (WSL2), I still have the error even if I pin the version of the common package. I was testing on macOS and I thought for some reason I fixed it but it might have been a fluke.
Anyway, although it's not a fix for the issue, exporting the types from Core will help me in my particular use case.

@Adamj1232 You are right, problem only happens with pnpm.

Upon thinking on this issue a bit more of the importing of @web3-onboard/common for type usage...that package should be imported within devDependencies in the package.json rather than the dependencies object.

Yes anyway with sveltekit all packages are in devdependencies since vite does all the bundling in the end.

We have pinned the joi versions so all the latest packages will employ the same versions of joi and remove this error.


Update pymatgen to 2023.10.4

mp-api 0.37.0 has requirement emmet-core>=0.69.2, but you have emmet-core 0.68.0.


perlPackages.Future: 0.48 -> 0.50

Description of changes
https://metacpan.org/release/PEVANS/Future-0.50/source/Changes#L3-25

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build perlPackages.Future


NodeJS Cluster Module Incompatibility

Currently this module does not work with the node:cluster module (Node 17.2.0 being used)
The issue is that we cannot bind an IP if the cluster is being spawned the IP defaults to null in this case.
Please advise, or possible additional option for the IP_ADDRESS: Number()

Fixed, User error


[2405] Show no-organisations page if user has no user record

Context
The backend returns 401 if there is no user account. The last cleanup of
the status handling code left this user journey showing the "something
went wrong" unauthorized page.
Changes proposed in this pull request
Show no-organisations page if user has no user record
This change means that the same page is shown regardless of whether you
have no account in our database (401) or have an account but no access
to any providers (403), but the response codes to the browser are
reflective of which code the backend responded with (401/403).
Guidance to review
:ship:?
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Users with no record in our database seeing "something went wrong" 401 page

Updated test coverage to match new behaviour.
It didn't fail before because both error pages had the same data-qa value. Oops!

But wait, there's more!....

Added feature specs for rest of scenarios now I understand it all better


Cache issues

Hi,
When the error in the image appears , the table does't get rendered.When cache and browser history is cleared, the table is rendered but error remains.
Any help will be appreciated.Thanks!

This is basically the same as the issue you brought in #376 right? As I've said there, the issue seems to be due to other scripts you're running (like sap-ui-core.js or something like that)
Are you able to reproduce this problem with the example at http://nicolas.kruchten.com/pivottable/examples ? If not, I would tend to say that the issue comes from somewhere else in your page

Hi Nicolas,
In #376, the issue was about running my applications in different machines.
In #379 issue, the problem is rendering in the same machine.
My application does not throw this error when i do not use pivot.js. Also, i'm not able to reproduce the error with examples on your site.

I'm unclear about how clearing browser history would apply to running an application on different machines...

Hi Nicolas,
The issue got resolved when i called .pivotUI inside setTimeout.
So..the issue was the rendering time and not cache!! :+1:

Great! Does that resolve #376 also?

Yes!! #376 too is resolved! :+1:

Super


Mockoon DOWN

🚨 Service Alert: Check the details below
Monitor Details

Monitor Name: Mockoon
Incident Status: TRIGGERED
Severity: critical
Monitor Status: DOWN
Monitor Health Checks: 1
Monitor Failure Threshold: 1

Actions

View Monitor

[start_datetime:1732978980]

The incident has been auto resolved
Total downtime: 1 minutes


[API review] Refactor work in queue

SAS->Sas
IPRange->IpRange
Remove getClazz
getIdentifier->getId
Reference: https://apiview.azurewebsites.net/Assemblies/Review/2cefcabd89fe4b008f29c7300ba1c3c2

Remove getClazz.  The issue included in builder hierarchy issue :https://github.com/Azure/azure-sdk-for-java/issues/5608


ColorPicker: fix invalid hue bar click when saturation is zero (#5080)

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English).
[x] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

噗 明天改....睡了- -


Inconsistent Checkbox Spacings

There are a couple of forms in inventory that use checkboxes, but they look out of place on the page with their labels. For example, on the "+ new request" page, the "fulfill" checkbox seems to have a lot more whitespace then the rest of the page. When it's selected, a new "consumed" checkbox appears that also looks out of place in terms of element padding. The labels for the checkboxes seem to be too far away from the checkboxes as well.
This seems like it could be solved by either removing the labels for checkboxes, by adjusting the checkbox row height to be only half the other rows' heights, or by removing the bottom margins for the checkbox labels.

@jglovier, I see what's being referenced. Unless you disagree, I'm going to add this to the design list. Could we put this in "Help Wanted"?

Yeah, this def needs cleaned up. :+1: for help-wanted

Hello, I am an undergrad in computer science and MacEwan University in Edmonton. For one of my course we have to participate in an open source project and contribute to it in a few ways.
If no one has solved this issue I was wondering if I could take it on, @tangollama?

Swing away.
Typed or dictated imperfectly on my iPhone<PHONE_NUMBER>)

On Mar 15, 2016, at 5:40 AM, mvanstaalduinen<EMAIL_ADDRESS>wrote:
Hello, I am an undergrad in computer science and MacEwan University in Edmonton. For one of my course we have to participate in an open source project and contribute to it in a few ways.
If no one has solved this issue I was wondering if I could take it on, @tangollama?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/HospitalRun/hospitalrun-frontend/issues/322#issuecomment-196801678

Marketing as In Progress w/ @mvanstaalduinen working on this.

This seems like it could be solved by either removing the labels for checkboxes

:+1: for solving by removing the labels for checkboxes.

@jglovier, I'm thinking this needs to move back to Help Wanted, yes?

thinking this needs to move back to Help Wanted, yes?

@tangollama 👍 . Added back.

This looks like a good one for my first contribution.  I'm on it @tangollama. 🎯


Bump pgserver missing change

bump pgserver to 0.0.8 which works in windows.
expanduser()  change which was pending from last review.

This change is

ignore this. different pull request without conflict will bump


[C#] Incorrect snippet initialization type for repeated resource-name fields

Snippets are being generated with incorrect initialization code for repeated resource-name fields.
Example: IEnumerable<BookNameOneof> optionalRepeatedResourceNameOneof = new List<string>();
This should be: IEnumerable<BookNameOneof> optionalRepeatedResourceNameOneof = new List<BookNameOneof>();

InitCodeTransformer.cs line 415 is where the wrong type is used to generate this snippet initialization code.
Instead of item.getType(), the TypeRef needs to be a repeated of the resource-name type.
It's not clear to me if this TypeRef is available or can be constructed.
Or if an alternate way of building/representing repeated resource-name types is required.

@garrettjonesgoogle, @michaelbausor if you or someone else who is familiar with this InitCodeTransformer code could have a quick look I would appreciate it. Thanks.

I think @michaelbausor should take a look first since he is the most familiar with this at the moment.

@chrisdunelm You are right, the existing logic is not correct for C# - in Java we use new ArrayList<>() so the type is not required in the repeated field case.
I will have a look now and propose a fix for the C# case.