chore: migrated to vitest

Description
As discussed on Slack, we're moving to vitest.
What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[x] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[x] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Close #2297
Mobile & Desktop Screenshots/Recordings

Steps to QA
The test should pass as we're just moving to another testing framework.
Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Just sorting out one failing test.

I'm getting this error which seems to be a pretty recent issue. (see https://github.com/vitest-dev/vitest/issues/4726)
><EMAIL_ADDRESS>test
> vitest run

/home/runner/work/app/app/node_modules/vite/node_modules/rollup/dist/native.js:87
		throw new Error(
		      ^

Error: Cannot find module @rollup/rollup-linux-x64-gnu. npm has a bug related to optional dependencies (https://github.com/npm/cli/issues/4828). Please try `npm i` again after removing both package-lock.json and node_modules directory.
    at requireWithFriendlyError (/home/runner/work/app/app/node_modules/vite/node_modules/rollup/dist/native.js:87:9)
    at Object.<anonymous> (/home/runner/work/app/app/node_modules/vite/node_modules/rollup/dist/native.js:96:48)
    ... 3 lines matching cause stack trace ...
    at Module._load (node:internal/modules/cjs/loader:[10](https://github.com/open-sauced/app/actions/runs/7173511276/job/19533013351?pr=2296#step:6:11)13:12)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:202:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:195:25)
    at async ModuleLoader.import (node:internal/modules/esm/loader:336:24)
    at async main (file:///home/runner/work/app/app/node_modules/vitest/dist/cli-wrapper.js:45:5) {
  [cause]: Error: Cannot find module '@rollup/rollup-linux-x64-gnu'
  Require stack:
  - /home/runner/work/app/app/node_modules/vite/node_modules/rollup/dist/native.js
      at Module._resolveFilename (node:internal/modules/cjs/loader:[11](https://github.com/open-sauced/app/actions/runs/7173511276/job/19533013351?pr=2296#step:6:12)34:15)
      at Module._load (node:internal/modules/cjs/loader:975:27)
      at Module.require (node:internal/modules/cjs/loader:[12](https://github.com/open-sauced/app/actions/runs/7173511276/job/19533013351?pr=2296#step:6:13)25:19)
      at require (node:internal/modules/helpers:177:18)
      at requireWithFriendlyError (/home/runner/work/app/app/node_modules/vite/node_modules/rollup/dist/native.js:69:10)
      at Object.<anonymous> (/home/runner/work/app/app/node_modules/vite/node_modules/rollup/dist/native.js:96:48)
      at Module._compile (node:internal/modules/cjs/loader:[13](https://github.com/open-sauced/app/actions/runs/7173511276/job/19533013351?pr=2296#step:6:14)56:[14](https://github.com/open-sauced/app/actions/runs/7173511276/job/19533013351?pr=2296#step:6:15))
      at Module._extensions..js (node:internal/modules/cjs/loader:1414:10)
      at Module.load (node:internal/modules/cjs/loader:1[19](https://github.com/open-sauced/app/actions/runs/7173511276/job/19533013351?pr=2296#step:6:20)7:[32](https://github.com/open-sauced/app/actions/runs/7173511276/job/19533013351?pr=2296#step:6:33))
      at Module._load (node:internal/modules/cjs/loader:1013:12) {
    code: 'MODULE_NOT_FOUND',
    requireStack: [
      '/home/runner/work/app/app/node_modules/vite/node_modules/rollup/dist/native.js'
    ]
  }
}


[CSL-1586] /healthcheck/route53 endpoint

This single-commit PR is essentially the same as https://github.com/input-output-hk/cardano-sl/pull/1552 but rebased against 1.0 and with the port being configurable.
We default to 3030 (taken arbitrarily) and localhost if no argument is passed on the command line, to smooth out deployments. Happy to take suggestions if the above behaviour is not desirable.

@domenkozar Oh! Appveyor is failing it seems:
--  While building package node-sketch-<IP_ADDRESS> using:
      C:\s\setup-exe-cache\x86_64-windows\Cabal-simple_Z6RU0evB_<IP_ADDRESS>_ghc-8.0.2.exe --builddir=.w\dist\ca59d0ab build lib:node-sketch exe:discovery exe:ping-pong --ghc-options " -ddump-hi -ddump-to-file"
    Process exited with code: ExitFailure 1
    Logs have been written to: C:\w\.w\logs\node-sketch-<IP_ADDRESS>.log
    Preprocessing library node-sketch-<IP_ADDRESS>...
    <command line>: cannot satisfy -package-id statistics-<IP_ADDRESS>-vN6s2qb86Z5lmqDDU9KzO
        (use -v for more information)
Sorry, we tried running command for 5 times and all attempts were unsuccessful!

Is there any hope of restarting a build there? I couldn't find any button, but maybe I haven't looked hard enough 😁

Done

Done.


How to select an Item onClick or Ontap

I would like to use this, but need support for onClick and onTap event. When clicked that item should be selected. I see a option focusOnselect but it not seems to work

like in following image

Tap or click on slide is working now

Thats great. Thank you.

Hi, focusOnSelect doesn't function for my case. I'm using react.0.13.3 and react-slick.0.8.0
sliderSettings:
{
  "centerMode": true,
  "focusOnSelect": true,
  "infinite": false,
  "dots": true,
  "arrows": false,
  "speed": 500,
  "slidesToScroll": 1,
  "slidesToShow": 5,
  "responsive": [
    {
      "breakpoint": 1024,
      "settings": {
        // same as defaults but with different `slidesToShow`
      }
    },
    {
      "breakpoint": 600,
      "settings": {
        // same as defaults but with different `slidesToShow`
      }
    },
    {
      "breakpoint": 480,
      "settings": {
        // same as defaults but with different `slidesToShow`
      }
    },
  ]
}

initializing:
<Slider {...sliderSettings}>
  {
    this.props.items.map((item) =>

              <Item {...item} />

    )
  }
</Slider>

where <Item /> is:

    {this.props.title}

    {this.props.description}


feat(curriculum): add exercise tracker lab

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes https://github.com/freeCodeCamp/CurriculumExpansion/issues/563

Blocking this while we figure out if we want to use the same or new challenge ID's


which proto is for generate_tsv.py？

I want to use your model to get features, but I don't know what proto do I need for generate_tsv.py.
Following the "test part", I choose "models/vg/ResNet-101/faster_rcnn_end2end_final/test.prototxt", but it report error :
cudnn.hpp:122] Check failed: status == CUDNN_STATUS_SUCCESS (3 vs. 0)  CUDNN_STATUS_BAD_PARAM
I will run this model in XMedia Wikipedia and Pascal. I want more introduction about this part please.
btw, do I need to resize the img shape to 224,224,3 ?
Thank you

I've solved this problem by add"engine:CAFFE" for every convolution layer，but I still don't know which model I should use for generate_tsv.py

I have the same problem, have you solved it?

Any README for running generate_tsv.py? Actually, I am interested in figuring out the issue#79 in lxmert.

Any README for running generate_tsv.py? Actually, I am interested in figuring out the issue#79 in lxmert.


stale read with raft-kv2 returns unexpected region error occasionally

Bug Report

client-go maybe receive following error when send stale-read request.
[2023/10/11 15:35:54.523 +08:00] [INFO] [region_request.go:1497] ["send request got region error"] [conn=3967811628] [session_alias=] [err-message="\"[components/raftstore-v2/src/operation/query/mod.rs:139]: PeerMsg::RaftQuery should not get stale read requests\""] [err-string="message:\"\\\"[components/raftstore-v2/src/operation/query/mod.rs:139]: PeerMsg::RaftQuery should not get stale read requests\\\"\" "]

What version of TiKV are you using?

nightly: 58253e8b7cea59b414511753b75dd7fc980d99af
What operating system and CPU are you using?

Steps to reproduce

What did you expect?
What did happened?

https://github.com/tikv/tikv/blob/b4e0bf7bab6ad395b74c0be938119d82ded4cd2a/components/raftstore-v2/src/operation/query/local.rs#L315-L321
The root cause is if try_get_snapshot returns Redirect due to ReadDelegate is not exist, raft-v2 will always do try_to_renew_lease which will triggers a readIndex. This is not true for staleRead.

Close this issue temporarily since Dynamic Regions project is paused.


This lesson has faulty instruction

Describe your problem and how to reproduce it:
The instruction says " Skew the element with the id of bottom by 24 degrees along the X-axis by using the transform property."
while in the body of code the red color is applied to the top div. The bottom div doesn't have color.
The instruction should instead say to apply Skew to the top div not the bottom.
the instruction should change to see the actual skew effect.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/responsive-web-design/applied-visual-design/use-the-css-transform-property-skewx-to-skew-an-element-along-the-x-axis
Tell us about your browser and operating system:

Browser Name: Chrome
Browser Version:
Operating System: windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

it is correct. i made a mistake

it is correct. i made a mistake


Chore: Update to latest drizzle-builder

This PR finishes out addressing #53.
Changes include:

Update to point to latest drizzle-builder
Update templates to use body/main instead of wrapper/body

cc: @tylersticka @erikjung @saralohr @nicolemors

👍


Dockerfile: "sh: 1: ulimit: error setting limit (Operation not permitted)" error fixing

I cannot run container on the latest Docker (1.12.0, build 8eab29e) because of "sh: 1: ulimit: error setting limit (Operation not permitted)" error. The latest docker supports setting ulimits through the command line --ulimit nofile=262144:262144.

Ok, thanks!


Bookmarks list: label overflow

Related issue: https://github.com/readium/readium-desktop/issues/723


[Relay][Parser] Support slash in identifier.

Variables from tensorflow may contains '/' in name (x/y/z).
Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.
@tqchen @jroesch please help to review this change, which found in tensorflow model
Thanks a lot :)
Regards,
Zack

cc @areusch @tkonolige @comaniac
Thanks a lot.

Could someone help to merge? Thanks :)


Add a shell.nix to modules for easier testing

As Doom continues to grow, so have incoming bug reports for modules I do not use and toolchains I know little about, like :lang clojure or :tools ein, and so I struggle to support them.
If you know a little nix and use Doom, consider contributing a shell.nix file to the module(s) you use the most. Particularly :lang modules, so that I can get a test environment up and running to test against.
These shell.nix files should set up a minimum viable environment, installing the language/tools I need to test that module as comprehensively as possible (excluding Emacs and Doom itself). For example, this is what lang/clojure/test/shell.nix might look like:
with import <nixpkgs> {};
stdenv.mkDerivation rec {
  name = "lang-clojure";
  buildInputs = [
    clojure
    joker
    leiningen
  ];
}

Understandably, some languages/tools may have more complex needs (such as multiple versions of python, or different environment managers for python, etc). For these, feel free to add arguments to the derivation to be passed via --argstr, like I have with Doom's test/shell.nix (installs Doom + Emacs 26.3) and test/shell.head.nix (installs Doom + Emacs 28+).
If you want to modularize them, place additional nix files in that module's test/ subdirectory.
Also: I'm still new to the nix language and am open to suggestions!

This issue has been mentioned on Doom Emacs Discourse. There might be relevant details there:
https://doomemacs.discourse.group/t/add-a-shell-nix-to-modules-for-easier-testing/157/1


install on Amazon Linux AMI

I remove old varnish
    sudo yum remove varnish

install new varnish
   cd ~/src/Varnish-Cache
   git pull
   ./autogen.sh
    ./configure
    make
    sudo make install
    sudo ldconfig

then reinstall libvmod-geoip2
    cd ~/src/libvmod-geoip2
    git pull
    ./autogen.sh

and get the error message:
    No package 'varnishapi' found

     Consider adjusting the PKG_CONFIG_PATH environment variable if you
     installed software in a non-standard prefix.

from fgsch
@dbsdsun Is pkg-config installed? Assuming it is, do you have a varnishapi.pc file under /usr/local/lib/pkgconfig/ ?
@fgsch
pkg-config is installed, I can see varnishapi.pc file under /usr/local/lib/pkgconfig/.
[ec2-user@ip-172-31-11-96 ~]$ sudo yum search pkg-config
Loaded plugins: priorities, update-motd, upgrade-helper
714 packages excluded due to repository priority protections
========================================================================================================== N/S matched: pkg-config ==========================================================================================================
mingw32-pkg-config.x86_64 : A tool for determining compilation options for the win32 target
mingw64-pkg-config.x86_64 : A tool for determining compilation options for the win64 target
perl-ExtUtils-PkgConfig.noarch : Simplistic interface to pkg-config
Name and summary matches only, use "search all" for everything.
[ec2-user@ip-172-31-11-96 ~]$ ls /usr/local/lib/pkgconfig
geoip.pc  libmaxminddb.pc  varnishapi.pc

Can you try installing pkgconfig instead via yum install pkgconfig?

Actually, I'd expect a different error if pkg-config is missing so I guess /usr/local/lib/pkgconfig/ is not in the default list of directories to search for .pc files.
Try running env PKG_CONFIG_PATH=/usr/local/lib/pkgconfig ./configure.

wget http://pkgconfig.freedesktop.org/releases/pkg-config-0.18.1.tar.gz
then unzip ...
./configure
got this error:
checking how to run the C++ preprocessor... /lib/cpp
configure: error: C++ preprocessor "/lib/cpp" fails sanity check
See `config.log' for more details.

in config.log
|                    Syntax error
configure:4894: /lib/cpp  conftest.cc
cpp: error trying to exec 'cc1plus': execvp: No such file or directory
configure:4900: $? = 1
configure: failed program was:
| /* confdefs.h.  */

I seems I don't have cc1plus

Rather than trying that please see my other comment first. You can contact me in gitter.im too.

run the following commands
env PKG_CONFIG_PATH=/usr/local/lib/pkgconfig ./autogen.sh
./configure
make

got error again
make[2]: Entering directory `/home/ec2-user/src/libvmod-geoip2/src'
/usr/bin/python2.7  ../src/vmod.vcc
  File "../src/vmod.vcc", line 27
    $Module geoip2 3 Varnish GeoIP2 Lookup Module
    ^
SyntaxError: invalid syntax

Sorry, you need to do that for both autogen.sh and configure:
env PKG_CONFIG_PATH=/usr/local/lib/pkgconfig ./autogen.sh
env PKG_CONFIG_PATH=/usr/local/lib/pkgconfig ./configure

the follow commands are passed:
env PKG_CONFIG_PATH=/usr/local/lib/pkgconfig ./autogen.sh
env PKG_CONFIG_PATH=/usr/local/lib/pkgconfig ./configure
make
sudo make install

Thanks for the help!!!

No problem.   Glad it's building and installing now.
I will update the installation details and try to address the SyntaxError issue in configure.

In retrospective, I can't really explain nor reproduce the SyntaxError when PKG_CONFIG_PATH was not set.
I get an error at configure time.

@fgsch maybe, I was still using old ./configure result.
./configure     I missed the error message
make             I was using old configure result generated with previous commit.
I think so.

@dbsdsun I considered that but did not know whether you have received an error or checked the output at configure time. Thanks for letting me know.


Register a Shipit/Grunt task that is able to be called by shipit.start and Grunt

Ran into this problem here:
https://github.com/timkelty/shipit-shared
Basically, I want to be able to:

run grunt shipit:development shared manually,
Have it the shared automatically run on the published event (from shipit-deploy).

In this example, the task gets correctly called on published, but I'm unable to run it manually as grunt shipit:development shared:
module.exports = function (gruntOrShipit) {
  var shipit = getShipit(gruntOrShipit);
  require('./create-dirs')(gruntOrShipit);
  require('./link')(gruntOrShipit);

  registerTask(shipit, 'shared', [
    'shared:create-dirs',
    'shared:link',
  ]);

  shipit.on('published', function () {
    shipit.start('shared');
  });
};

If I pass gruntOrShipit to registerTask instead, I can run the grunt command manually, but the task is not run on published and I get the error: Task 'shared' is not in your shipitfile
Is there a good solution for this?
Might be more applicable in https://github.com/shipitjs/grunt-shipit, but I'm not sure.

In you example, "publishd" is not correctly spelled.
I think that you can assume that gruntOrShipit is shipit, because you don't want to be compatible with the old grunt plugin. So you don't need getShipit, etc..
If you do that it should work:
module.exports = function (shipit) {
  require('./create-dirs')(shipit);
  require('./link')(shipit);

  registerTask(shipit, 'shared', [
    'shared:create-dirs',
    'shared:link',
  ]);

  shipit.on('publshed', function () {
    shipit.start('shared');
  });
};

If you want to be compatible with grunt, I think that you have to test if you are in grunt or in Shipit mode because it's not the same command to start tasks. grunt.tasks.run(['shared']) VS shipit.start('shared')

You can also abstract the method to start the task, it's cleaner :).

I think that you can assume that gruntOrShipit is shipit, because you don't want to be compatible with the old grunt plugin. So you don't need getShipit, etc..

I'm not trying to be compatible with the old (pre-orchestator-based) grunt plugin, just https://github.com/shipitjs/grunt-shipit.
If I use your example from grunt (grunt shipit:development deploy), I get:
Loading "grunt-task.js" tasks...ERROR
>> TypeError: Object #<Object> has no method 'on'

Running from shipit works fine: (shipit development deploy or shipit development shared).
I was just following your lead from shipitjs/shipit-deploy with the registerTask/gruntOrShipit stuff.

@timkelty yes I know, it's simple, when you have gruntOrShipit you need sometimes to test if it's grunt or Shipit to run the right command.

Ok, sorry for the confusion :)
So it seems I just need to have a method to run/start from gruntOrShipit, as you suggested.

Yes.


source_id registration AWI-ESM-1-REcoM

Placeholder issue awaiting publication of AWI-ESM-1-REcoM data on https://esgf-data.dkrz.de/search/cmip6-dkrz/ - Toggling the "Registered" status to "Published" will occur at that point and close out this issue
@chrisdane ping
Ref #1216 #1215

Awaiting publication of https://aims2.llnl.gov/search?project=CMIP6&activeFacets={"source_id":"AWI-ESM-1-REcoM"}

Hi Paul, we published data (https://aims2.llnl.gov/search/cmip6/?mip_era=CMIP6&institution_id=AWI&source_id=AWI-ESM-1-REcoM, more to come though).
Please see PR #1233
Thanks a lot and cheers,
Chris

@chrisdane while we're cleaning things up AWI-ESM-2-1-LR has never had any data published against it, so I suggest we deregister, at the same time, ok?

@chrisdane I am unable to access the index of data published on the DKRZ node, it seems https://esgf-metagrid.cloud.dkrz.de/search/cmip6-dkrz/ is not returning any datasets, and the shared index locally (https://aims2.llnl.gov/search/cmip6/?mip_era=CMIP6&institution_id=AWI&source_id=AWI-ESM-1-REcoM) is also not returning results, so I will await your feedback before merging the changes - particularly I want to know the first date that AWI-ESM-1-REcoM data was made available on ESGF, if 2024-07-04 is not right.
And the separate question about AWI-ESM-2-1-LR

Hi Paul
For me the data appears on both pages, aims2.llnl.gov and esgf-metagrid.cloud.dkrz.de. Maybe it took some time until everything was settled on the servers?
The version stamp v20230314 was used by helping people from our local ESGF node (DKRZ). It is correct that the data was added rather recently, i.e. I think 2024-07-04 is correct. Is that a problem? Apparently it was not a problem for the DKRZ people who in the end decide if data can be uploaded or not. Changing this would require extra work. Thanks for advice.
I cannot say anything about AWI-ESM-2-1-LR, sorry.
Thanks for your help,
Chris

It appears 2024-07-04 is the first publication date for AWI-ESM-1-REcoM - see here (blocked IP ranges).
@chrisdane I can process the AWI-ESM-1-REcoM update. For the AWI-ESM-2-1-LR as this has remained open without comments for a while, so I'll deregister this to close out the stale dialogue.
Pinging @christian-stepanek tp close out the discussion

@chrisdane @christian-stepanek those changes (including the ocean/seaIce native_nominal_resolution tweak) are now in CMIP6_source_id.html. Please take a peek and let me know if there are any remaining issues to resolve


Improve screen reader support

To fix the issue at https://code.google.com/p/chromium/issues/detail?id=499426
Are the alt-texts good enough? Thanks.

Not very descriptive, but certainly better than before. :-)


Broken in Nuxt ^3.1.0

Doesn't work at all in either Nuxt 3.1.0 nor 3.1.1

While investigating this error, I encountered another error (reported on  nuxt/nuxt(#18563)).
I'm not sure if these errors are related, but if so, it may take a longer time to fix.

I'll try to do more tests on my free time tomorrow. In the meanwhile I willy just not update Nuxt.

+1

Working, good enough until the final release.

@e-chan1007 I still have an issue with a clean 3.1.0 nuxt install and v.1.1.3 of your module:

This is happening again with the new Nuxt version


README

README
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

Made an attempt to have a professional looking Readme


Document a gotcha with additionalProperties

Relates to #848 #902 #992 (and maybe others).
Reasons for making this change
It seems like a lot of people are running into the same problem, this will help them avoid it earlier and be more clear about the current limitations of the library.
Checklist

[x] I'm updating documentation

[x] I've checked the rendering of the Markdown text I've added
[ ] If I'm adding a new section, I've updated the Table of Content

[ ] I'm adding or updating code

[ ] I've added and/or updated tests
[ ] I've updated docs if needed
[ ] I've run npm run cs-format on my branch to conform my code to prettier coding style

[ ] I'm adding a new feature

[ ] I've updated the playground with an example use of the feature

Thanks for submitting this, and sorry for taking so long to respond to it. This reads like a pretty mild warning but it almost sounds like it's describing a bug. Is it actually a bug? Is using additionalProperties: false incorrect somehow? If not, is there a workaround -- something else that should be used?

#848 has a pretty good example of the behaviour. Using additionalProperties: false will produce incorrect results and break things. There may be a fix for this that would delete additional properties from the output, but would be hard to work.
Either way, it is a limitation that seems to be commonly encountered, so it should be more clearly documented.


More focused description

@pablohoffman , @eliasdorneles ,
does it look better?

+1

Yeap, lovely!
We should update the phrase in GitHub header too: https://github.com/scrapinghub/extruct


PPD: look for h-feed anywhere on page

use mf2util.find_first_entry to BFS the page for h-feed, instead
of only allowing the root to be h-feed.
fixes #584

nice! i <3 mf2util. thanks!

nice! i <3 mf2util. thanks!


dotnet custom tool task picks wrong framework version

Description
We are using a dotnet tool as part of our build pipeline to create swagger files out of the compiled code.
The simplified list of steps are:
UseDotnet task with .net8 as target
Restore nugets
Restore dotnet tools from nuget
build projects
run dotnet tool.
All build steps seem to use .net8 fine, but the DotNetCoreCLI seems to not honor net8 from path correctly.
Details (shortened):
What else should I look for to provide as information?
Same task with build works perfect, writing binary files to .../net8/... directories.
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[X] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Current image version: '20231115.8.0'
https://dev.azure.com/project-argos/Sorento/_build/results?buildId=745471&view=logs&j=9dd51b93-2435-51c6-9534-da9ac09c666e&t=e7d3c925-197a-4661-8bb7-39aaebc64659
Is it regression?
no
Expected behavior
Build would use dotnet tool from correct framework
Actual behavior
Error from logs:
Starting: Generate 2022-12-01 swagger file
Task         : .NET Core
Description  : Build, test, package, or publish a dotnet application, or run a custom dotnet command
Version      : 2.221.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/build/dotnet-core-cli
Info: .NET Core SDK/runtime 2.2 and 3.0 are now End of Life(EOL) and have been removed from all hosted agents. If you're using these SDK/runtimes on hosted agents, kindly upgrade to newer versions which are not EOL, or else use UseDotNet task to install the required version.
/opt/hostedtoolcache/dotnet/dotnet swagger tofile --serializeasv2 --output ./service/AzureOpenAI.Api/Swagger/Reference/2022-12-01/azureopenai.json ./service/AzureOpenAI.Api/bin/Release/net8.0/linux-x64/AzureOpenAI.Api.dll 2022-12-01
You must install or update .NET to run this application.
App: /home/vsts/work/1/s/.\packages/swashbuckle.aspnetcore.cli/6.5.0/tools/net7.0/any/dotnet-swagger.dll
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '7.0.0' (x64)
.NET location: /opt/hostedtoolcache/dotnet/
The following frameworks were found:
8.0.0 at [/opt/hostedtoolcache/dotnet/shared/Microsoft.NETCore.App]
Learn more:
https://aka.ms/dotnet/app-launch-failed
To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=7.0.0&arch=x64&rid=linux-x64&os=ubuntu.20.04
##[error]Error: The process '/opt/hostedtoolcache/dotnet/dotnet' failed with exit code 150
Info: Azure Pipelines hosted agents have been updated and now contain .Net 5.x SDK/Runtime along with the older .Net Core version which are currently lts. Unless you have locked down a SDK version for your project(s), 5.x SDK might be picked up which might have breaking behavior as compared to previous versions. You can learn more about the breaking changes here: https://docs.microsoft.com/en-us/dotnet/core/tools/ and https://docs.microsoft.com/en-us/dotnet/core/compatibility/ . To learn about more such changes and troubleshoot, refer here: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/build/dotnet-core-cli?view=azure-devops#troubleshooting
##[error]Dotnet command failed with non-zero exit code on the following projects : [ '' ]
Finishing: Generate 2022-12-01 swagger file
Repro steps

task: UseDotNet@2
displayName: 'Install .NET Core sdk v8.x'
inputs:
packageType: sdk
version: 8.x
installationPath: $(Agent.ToolsDirectory)/dotnet
includePreviewVersions: true

task: DotNetCoreCLI@2
displayName: 'Restore nuget packages'
inputs:
command: custom
custom: restore
projects: |
**/*.csproj
arguments: >-
--configfile:nuget.config
--runtime $(BuildRuntime)

task: DotNetCoreCLI@2
displayName: 'Locally restore dotnet tools'
inputs:
command: custom
custom: tool
arguments: >-
restore
condition: succeeded()

task: DotNetCoreCLI@2
displayName: 'Generate 2022-12-01 swagger file'
inputs:
command: custom
custom: swagger
arguments: >-
tofile --serializeasv2 --output ./service/AzureOpenAI.Api/Swagger/Reference/2022-12-01/azureopenai.json ./service/AzureOpenAI.Api/bin/Release/net8.0/linux-x64/AzureOpenAI.Api.dll 2022-12-01
condition: succeeded()

Hey @bexxx. Thank you for your report, we will try to investigate this. Please keep in mind that recent Ubuntu 20.04 image update includes new version of .NET Core (8.0.100), which may affect you workflow. Also there might be issues with either DotNetCoreCLI@2 or UseDotNet@2 Azure task related to the update of .NET Core.
Unfortunately, your project is private and we can't access it. Could you please provide pipeline logs in more convenient format (e.g. attach a file)?
Also could you please elaborate whether you have this pipeline working recently or you are trying to create a new one and it doesn't work from the very beginning? If it was working recently, could you please attach logs of successful run as well so we can examine it?

This is the raw log of this build:
57.txt
Our global.json which is checked in too:
global.json
It works for .net7 builds. This is the first time we try to get it work with our code using .net8. All other builds and packaging with .net 8 works, just not the dotnet tool step.
If this turns out to be a task issue, could you please point me to the place where those can be reported?

@bexxx , please run dotnet --list-sdks before and after UseDotNet@2 (before all of them and after all of them).

Full logs with info und list-sdks:
79.txt

This should be reported to dotnet developers, it is not the image bug because UseDotnet downloads dotnet distros to the /opt/hostedtoolcache/dotnet/ and this is not the directory where we install dotnet (we install system dotnet to /usr/share because not everybody using the task). Going to close this ticket, feel free to reach us out again if questions left.

Just for reference if someone with same problem find this.
I can confirm this works for me using an Azure DevOps build pipeline now.
I've added
variables:
DOTNET_ROLL_FORWARD: LatestMajor
to the build and it started to work.


Adds smog readability test

I wanted to set the smog grade to nil or a string The text has less than 30 sentences but the report method doesn't allow it.
So I settled on 0.

Please be sure to include specs. Thanks for this!

There are no existing readability specs or I couldn't find any.
Do you want me to create a readability.cr specs file ?
And I think I'd better create one single PR for the added tests. It'll be much clearer. WDYT ?

hmm I thought I had readability tests. I'll add some so that you can just add to the file per PR. I'd rather have tests and features merged at the same time.

OK, nice.


Long lived connections stale/become unusable

Version of react-native-iap
7.5.0
Version of react-native
0.65.1
Platforms you faced the error (IOS or Android or both?)
Android
Hi, we migrated from 6.2.4 to 7.5.0 a month ago, from that moment our error rates started sky-rocketing, we have been trying to debug the issues for some days now, but we couldn't find any issue with our code but there is nothing wrong, however, we did notice somethings:

We initialize the connection once on app start, and we let the object alive through the app's lifecycle
On the 6.3.0 release, the logic changed to keep only one client alive (https://github.com/dooboolab/react-native-iap/commit/1409b8f13cb4d3eb60b4c5eac17bab8edaf3cd7d)
We think a combination of us keeping the billingClient alive for a long period of time + now the handling of a single client instance, causes the client at some point to disconnect and never to reconnect again, causing elevated error rates

Any ideas if this is correct or if something else could be going on?

Hi @ospfranco
I face the same problem as you (I mentioned my issue that is, I think, related).
What did you do to solve this? Did you downgrde to ^6.0?
I'm using hooks, do you?

No, I'm using redux sagas, I added a new method (android only) isReady, if that returns false, you can kill and re-initialize the connection

No, I'm using redux sagas, I added a new method (android only) isReady, if that returns false, you can kill and re-initialize the connection

Ok thanks, I don't quite know where to add the call of isReady in my situation since I'm using hooks.

you can kill and re-initialize the connection

How do I do that?
In my root container
const {
  currentPurchase,
  finishTransaction,
} = useIAP();

useEffect(() => {
  const checkCurrentPurchase = async (purchase): Promise<void> => {
    if (purchase) {
      const receipt = purchase.transactionReceipt;
      if (receipt)
        try {
          await finishTransaction(purchase);
          await dispatch(Purchase.action(purchase))
          // display something
          showMessage({
            message: t('shop.successMessage.message'),
            description: t('shop.successMessage.description'),
            type: 'success',
          })
        } catch (ackErr) {
          console.warn('ackErr', ackErr);
          showMessage({
            message: ackErr.message,
            type: 'danger'
          })
        }
      }
  };

  checkCurrentPurchase(currentPurchase);
}, [currentPurchase, finishTransaction]);

And in my Shop container
  // ...
  const {
    products,
    getProducts,
    requestPurchase
  } = useIAP()

  // ...

<ShopCardItem
    key={item.productId}
    onPress={() => requestPurchase(item.productId)}
    title={title}
    description={item.description}
    highlight={'🚫'}
    price={item.localizedPrice}
    color={Colors.gold}
/>

Do you know if I have to call init method with hooks?

The method is not intended/integrated with hooks, it's meant for manual handling of the client
Cannot help you with your issue

Do you know if purchaseUpdatedListener and purchaseErrorListener needs to be re-initialized when endConnection and initConnection is called again?

Don't think so, the callbacks are handled on the iap package and not the iap instance, shouldn't be too difficult to test it yourself though

True! I'll have to do some testing. I just wondered whether it was something you had already thought about in your implementation or if it isn't needed.

Going to close this issue for now, but here is some info for those looking for some answers:

Our (temporary) solution was to revert to ~6.x.x version, because that version internally re-initializates the internal iap object whenever some of the methods are called (which causes memory leaks but at least is guaranteed to work)
The next step is to upgrade again and use the newly exposed method (isReady) to check if the instance is alive before doing any operation, if not alive then initConnection needs to be called again
isReady is meant to be used only with manual handling of a IAP client, the usage of hooks was not part of my problem, therefore you will need to debug any issues with the hook yourself, but since hooks are tied to component lifecycles, I think you need to unmount/remove your component for a new instance to be initialized

This is all the info I can give

Thanks for the info. I still think it would be nice to keep an issue open for this though, as the problem apparently still exist so it needs to be fixed or documented officially how to properly handle it. Else it's a very sneaky bug that may not appear when testing, but will appear later in production and is hard to reproduce. Shall I create a new issue?

Did you by any chance find a way to reproduce the problem locally? I tried:

turning off wifi
airplane mode
force stopping google play services

but isReadyAndroid still returns true

I didn't do further testing nor did I upgrade to the latest version as I switched jobs. But we had reverted versions and that worked fine

It should be fixed in 8.4.0. Please take a look. Reopen if the issue comes back. Thank you


[Docs] What are acceptable ranges for a) hyperparameters b) performance values?

The only thing I could conclude from he example is that you are maximizing performance. It also seems that he example prefers everything 0-1 normalized, is this required/preffered, are there issues with not doing this?

Both the hyperparameters and performance values should be in $[0-1]$. Any values outside of this range will trigger an exception (see).


Error when pushing to server with TLS certificate trusted in windows certificate store

When using the NuGet push build step targeting the on premise NuGet DevOps server library which uses a TLS certificate from a chain that is trusted on the build agent by the windows certificate store i get the following error:
2019-04-11T19:20:37.4087671Z ##[section]Starting: NuGet push
2019-04-11T19:20:37.4096761Z ==============================================================================
2019-04-11T19:20:37.4096915Z Task         : NuGet
2019-04-11T19:20:37.4097027Z Description  : Restore, pack, or push NuGet packages, or run a NuGet command. Supports NuGet.org and authenticated feeds like Package Management and MyGet. Uses NuGet.exe and works with .NET Framework apps. For .NET Core and .NET Standard apps, use the .NET Core task.
2019-04-11T19:20:37.4097156Z Version      : 2.0.50
2019-04-11T19:20:37.4097245Z Author       : Microsoft Corporation
2019-04-11T19:20:37.4097359Z Help         : [More Information](https://go.microsoft.com/fwlink/?LinkID=613747)
2019-04-11T19:20:37.4097462Z ==============================================================================
2019-04-11T19:20:38.2629494Z [command]C:\WINDOWS\system32\chcp.com 65001
2019-04-11T19:20:38.2629692Z Active code page: 65001
2019-04-11T19:20:38.2632775Z SYSTEMVSSCONNECTION exists true
2019-04-11T19:20:38.2640408Z Detected NuGet version <IP_ADDRESS>0 / 4.1.0
2019-04-11T19:20:39.8625717Z ##[error]Error: Error: unable to get local issuer certificate
2019-04-11T19:20:39.8628016Z ##[error]Packages failed to publish
2019-04-11T19:20:39.8726155Z ##[section]Finishing: NuGet push

It would be nice to know HOW i can add trusted root certificates to provide them to the DevOps Nuget Task.

I figured it out.
Add trusted root certificates
Add an environment variable
NODE_EXTRA_CA_CERTS=file

When set, the well known "root" CAs (like VeriSign) will be extended with the extra certificates in file. The file should consist of one or more trusted certificates in PEM format. A message will be emitted (once) with process.emitWarning() if the file is missing or malformed, but any errors are otherwise ignored.
Note that neither the well known nor extra certificates are used when the ca options property is explicitly specified for a TLS or HTTPS client or server.
This environment variable is ignored when node runs as setuid root or has Linux file capabilities set.

Node Environment Variables

@andyjlewis maybe see if the identified solution needs inclusion in whatever documentation we have

There is a Q & A section at Microsoft docs: Nuget taks where it could fit.

@thomps23 @andyjlewis has the information made it into any documentation?

@ramiMSFT : Can you take a look?

@lennybacon My apologies for the delayed response, I missed this in my inbox. Thank you for your contribution, the changes have been made and the updated content will go live soon.


Pagination last pages empty

Describe the bug
In many categories the last few pages are empty. E.g. mod most positive.
Screenshots

OS
Windows
Wanna have the bug fixed quickly?
Visit Issue hunt...

I noticied this when I was creating the pagination and it seems to be an api issue as the api reports that there are more pages and items than there actually are. It might require an api change to correct this.

This will probably be an elide.io issue we need to report then

can we have an example query?
On Fri, 28 Aug 2020, 14:45 Alexander von Trostorff, <
<EMAIL_ADDRESS>wrote:

This will probably be an elide.io issue we need to report then
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FAForever/downlords-faf-client/issues/1868#issuecomment-682507322,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBYJJK4W3IF4O66ROI4UXLSC6RFDANCNFSM4QOAXUGA
.

on it

http://api.faforever.com/data/mod?sort=latestVersion.reviewsSummary.lowerBound&page[size]=20&page[totals]&page[number]=64

Can somebody check if it's fixed now?

well the given url now returns data so I would say yes :D

@axel1200

seems fixed


ttb parameter in matlab output

The ttb parameter written to matlab output does not look right.  The first 2 time points look correct, but the subsequent time points  are much larger than the expected values.

Hi @jc-brandeis . Sorry for the late reply. Does this happen to all of your datasets? How big is the jump? Can you print out the values that you have?

The ttb problem happens to another dataset as well, whether using cosmos or cosmos+hmm model for fitting.
in matlab, ttb(1:5,:)-ttb(1,1) gives:
0          832
1709         2536
1.4562e+09   1.4562e+09
2.9123e+09   2.9123e+09
4.3685e+09   4.3685e+09
The         0          832      are correct and match the glimpse header ttb data.
1709         2536

This has been fixed in the version 1.1.16


Fixing the logo link in Reading and Talking therapy Page

I would like to fix the logo link issue in Reading and Talking therapy Page

@Surya8991 Start working!

Ok

closing issue as the changes have been made. And I couldn't do because i was busy.


About graphics card

Can you provide how number of graphics card you need, and about the type (Nvidia Geforce 2080Ti or NVIDIA Tesla P100 or others)?

The whole model is implemented by Tensorflow v1.9 and trained on the NVIDIA TITAN V (our graphics card) and Intel(R) Xeon(R) E5-2690 CPUs of our machine. Specifically, it needs 1064 MB for training the gait encoding model and 616 MB  for the recognition network model. Since our graphics card possesses 11.78GB, we only need one card to train and test. We believe that it is the same for other cards like NVIDIA GeForce 2080Ti or NVIDIA Tesla P100. Thank you.


Undefined global for dofile attached functions

Undefined global check doesn't takes into account using dofile.
All functions defined in my lua libraries and attached via dofile marked as undefined golobal.

Is the file in your workspace or can be searched by workspace.library ?

@sumneko 当 lua 被嵌入使用时，如 openresty, hammerspoon 有支持提示的方式吗？

@ueaner This is determined by the editor, as far as I know, VSCode does not support this function.

@sumneko Thanks for the quick response.
Maybe we can extract the relevant definitions of Lua functions in openresty, organize them into a separate lua file, and configure the "workspace.library" option for "lua-language-server". What do you think?

OpenResty supports content_by_lua_file, or you can do it like this:
content_by_lua_block {
    require('script.tcp').accept()
}

eg: from, to, err = ngx.re.find(subject, regex, options?, ctx?, nth?)
Can be auto-complete and parameter prompts for definitions beginning with "ngx." in the lua file.

You can make a definition file for it, refer to the definition file of the built-in standard library

@sumneko thx, I will try.

I register some functions in c++ so they are globally available in my lua file. Is there any way to declare just the function definition (name and parameters) for good intellisense support without defining the actual function?

You could use EmmyLua, see https://github.com/sumneko/lua-language-server/wiki/EmmyLua-Annotations


Spinning (moving?) a grid unbuckles everyone on grid

Description

Equip a grid with gyroscopes/thrusters
Start rotating with the shuttle console
#. Everyone currently buckled will be unbuckled, and attempting to buckle will teleport and unbuckle you.

Screenshots
N/A.
Additional context
Discovered on a random Saltern shift.

can't reproduce locally anymore

Trying to buckle yourself while the station is spinning is still buggy. Not sure about it un-buckling people who are already buckled though.

Around Jan 10th I was reproducing this. I'm assuming the radial velocity of the player from the grid spinning is messing with the "too far away, unbuckle the player" code.

Cannot repro, eshuttle regularly moves great distances with buckled players.


Azure LogAnalytics Documentation references Sku not included in enumeration

Link to article:
https://docs.microsoft.com/en-us/azure/templates/Microsoft.OperationalInsights/2015-11-01-preview/workspaces#Sku
Problem:
The Workspace properties.retentionInDays description states:

The workspace data retention in days. -1 means Unlimited retention for the Unlimited Sku. 730 days is the maximum allowed for all other Skus.

However, the SkuNameEnum does not contain a definition for 'Unlimited'

@bwren - can you take a look at this description from the swagger file? The SKUs in the enum do not seem to match the description of retention.
The published REST reference shows the result.


merge upstream v211.1.0

Changes proposed in this pull request:
Merge upstream changes for version 211.1.0
security considerations
None

Updates for cg-deploy-logsearch are in a branch: https://github.com/cloud-gov/cg-deploy-logsearch/tree/v211.1.0. While I am sure there are things to work out, I think this is ready to be merged/built so it can begin to be tested end to end.


Warning for add overflow

Tiny PR to make sure that addition has an overflow message. Not perfect or as helpful as I would like, but something more robust would require time that is not available at the moment.

Closing this without merging since I'm fixing up this in tandem with some other stuff on a separate branch


Response::new_with_opt_u8_array_and_init unexpected array contents

Summary
I've raised this as a question as not sure if it's a Cloudflare worker issue or WASM issue on my side or WASM web_sys bug.
I seem to have issues with web_sys::Response::new_with_opt_u8_array_and_init returning unexpected output.
Take the below function:
pub fn response_as_bytes(s: String) -> Result<Response, JsValue> {
    let mut init = ResponseInit::new();
    let mut headers: HashMap<&str, &str> = HashMap::new();
    headers.insert("Content-Type", "text/html; charset=utf-8");
    headers.insert("Cache-Control", "no-cache");

    let headers = wasm_bindgen::JsValue::from_serde(&headers).unwrap();
    init.headers(&headers);

    let mut body = s.into_bytes();

    web_sys::Response::new_with_opt_u8_array_and_init(Some(&mut body), &init)
}

I expect the function called with Hello World! to return a response with an ArrayBuffer containing the utf8 string Hello World!.
The output of this response is corrupt in that it does not contain a utf8 string Hello World!. I get the characters nn displayed in a browser.
If I debug the response from Javascript I get the below:
const resp = await execute(request)
const body = await resp.arrayBuffer()
console.log(body)

[[Uint8Array]]: Uint8Array(12)

    0: 16
    1: 109
    2: 19
    3: 0
    4: 16
    5: 109
    6: 19
    7: 0
    8: 32
    9: 0
    10: 0
    11: 0

The array looks to be of the correct size, 12, but the contents are not correct.
The below is the expected contents:
const encoder = new TextEncoder()
const view = encoder.encode('Hello World!')
console.log(view)

int8Array(12) [72, 101, 108, 108, 111, 32, 87, 111, 114, 108, 100, 33]

To verify I am correctly encoding the string I created the below function:
pub fn response_as_bytes_debug(s: String) -> Result<Response, JsValue> {
    let mut init = ResponseInit::new();
    let mut headers: HashMap<&str, &str> = HashMap::new();
    headers.insert("Content-Type", "text/html; charset=utf-8");
    headers.insert("Cache-Control", "no-cache");

    let headers = wasm_bindgen::JsValue::from_serde(&headers).unwrap();
    init.headers(&headers);

    let mut body = s.into_bytes();
    let body = serde_json::to_string(&body).unwrap();

    web_sys::Response::new_with_opt_str_and_init(Some(&body), &init)
}

When I run this function I get the below response, which is what I expect matching the contents of new TextEncoder().encode('Hello World!').
[72,101,108,108,111,32,119,111,114,108,100,33]

Looking Response::new_with_opt_u8_array_and_init calls the js Response object with a ArrayBuffer constructor argument?
The below looks to work in Javascript:
const encoder = new TextEncoder()
const view = encoder.encode('Hello World')
const resp = new Response(view)
resp.text().then(console.log)

> Hello World

The following repo contains a full poc test case https://github.com/forficate/cloudflare_wasm_bug_testcase
With the above I'm not sure if I'm misunderstanding Response::new_with_opt_u8_array_and_init or if there's a bug with the bytes being passed through not matching the bytes received on the js end.
I've only started looking in to WASM so not quite sure where next to go debugging.

Looks like the content is the same, just DEC representation of ASCII values. https://www.asciitable.com/
'H' => DEC 72,.... If other presentation format is chosen for the contetn, then it should be "Hello!"

I've done some more debugging.
In my Javascript entrypoint I have adjusted it to the below adding a custom my_response shim method:
addEventListener('fetch', event => {
  event.respondWith(handleRequest(event.request))
})

const { execute } = wasm_bindgen;
const instance = wasm_bindgen(wasm)

function my_response(body, init) {
  return new Response(body, init)
}

async function handleRequest(request) {
    await instance;
    const resp = await execute(request)
    return resp
}

On my WASM side I have the two functions:
// 1) Use the built in web_sys::Response::new_with_opt_u8_array_and_init func
pub fn response_as_bytes(s: String) -> Result<Response, JsValue> {
    let mut init = ResponseInit::new();
    let mut headers: HashMap<&str, &str> = HashMap::new();
    headers.insert("Content-Type", "text/html; charset=utf-8");
    headers.insert("Cache-Control", "no-cache");

    let headers = wasm_bindgen::JsValue::from_serde(&headers).unwrap();
    init.headers(&headers);

    let mut body = s.into_bytes();

    web_sys::Response::new_with_opt_u8_array_and_init(Some(&mut body), &init)
}

// 2) Use my custom my_response shim
pub fn response_as_bytes_custom(s: String) -> Result<Response, JsValue> {
    let mut body = s.into_bytes();

    let mut init = web_sys::ResponseInit::new();
    let mut headers: HashMap<&str, &str> = HashMap::new();
    headers.insert("Content-Type", "text/plain; utf8");
    headers.insert("Cache-Control", "no-cache");

    let headers = wasm_bindgen::JsValue::from_serde(&headers).unwrap();
    init.headers(&headers);
    init.status(200);

    Ok(my_response(&mut body, init))
}

And an extern declaration for my_response:
#[wasm_bindgen]
extern "C" {
  fn my_response(body: &mut [u8], init: web_sys::ResponseInit) -> web_sys::Response;
}

The below fails using the standard web_sys::Response. In the browser I see the characters 'HH'.
#[wasm_bindgen]
pub async fn execute(req: Request) -> Result<Response, JsValue> {
    response_as_bytes(String::from("Hello World!"))
}

Swapping the above out for my custom shim works as expected, I get the text "Hello World!" in the browser:
#[wasm_bindgen]
pub async fn execute(req: Request) -> Result<Response, JsValue> {
    response_as_bytes_custom(String::from("Hello World!"))
}

Many details, are Response interface of the Fetch api ?
https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API

I seem to be hitting a similar issue. I'm calling a javascript function with a &[u8] and it seems like the Uint8Array actually passed on the other side contains 8 bytes of garbage before the actual data.

Using wee_alloc instead of the default allocator makes it so that only the first 4 bytes are changed.

Same issue here. Seems to be Cloudflare specific. With wee_alloc, first 4 bytes are mangled (ie. transformed into garbage), and without it sometimes last bytes are mangled.
I recommend using new_with_opt_str_and_init instead for now, dealing with small performance overhead of transforming &[u8] into &str

I haven't had the time to look into this more deeply but did encounter the same problem. Converting a byte slice to Uint8Array and using Response::new_with_opt_js_u8_array_and_init which was added in the recent 0.3.72 web-sys release does work.

If I'm not wrong, this is a duplicate of #423.
Basically the constructor takes an ArrayBuffer that is still based on the Wasm memory. E.g. in OPs example the String is dropped at the end of the function and because we can't really say when this memory will be reused, this is UB and explains the random behavior in some of the debugging efforts above.
Indeed the solution for now is to use Response::new_with_opt_js_u8_array_and_init to give away owned memory that won't be overwritten.


How does it help with Model Interpretability and Explainability for Black-box model architecture

Checklist

[ ] I've searched the project's [issues](https://github.com/Giskard-AI/giskard/issues?q

❓ Question

Is Giskard model-agnostic and what is the performance on black-box models?
How does  Giskard assist in model interpretability concerning fairness, security and privacy considerations?
Is it possible to [...]?
📎 Additional context

I was unable to fully explore Giskard due to computational resource constraint. However, Giskard provides tools for evaluating models before serving in production. The experimental nature of Giskard would need further exploration for the inclusion of advanced model debugging features that pertain to network and infrastructure resource usage and adversarial activities in a production environment.

Hey @mrbestnaija here are our docs: https://docs.giskard.ai/en/stable/
Debugging is something that's available in our enterprise product only. Hope that answers the question!


ent won't work with gqlgen and federation

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
I tried following this repository and everything worked fine until I tried to use a feature from the Apollo Federation (that works on gqlgen) that is using the @external directive. It throws me an error saying:

creating entgql extension: schema.graphql:48: Undefined directive external.
exit status 1
ent\generate.go:3: running "go": exit status 1

Expected Behavior 🤔
It should generate without any errors. The directive is actually created and exists on the generated.go gqlgen file.
Steps to Reproduce 🕹
You can use the repository mentioned above and just include the @external in a field or use the ent + gqlgen example, activate the federation, and use the @external directive.
 Your issue looks like another one that I fixed via this commit https://github.com/ent/contrib/pull/199, So, what is your ent-contrib version?

It worked, thank you @giautm

Hey @a8m, can you release a new version of ent-contrib?


Font clutter

When we change the language of the application and change it to Persian, the fonts are in the middle

Please take action to resolve this issue

same issue

same issue in show Arabic characters


Create Diamond.cpp

Diamond Pattern

@bhattanimesh I'm closing the PR as it is not fulfilling the requirement...you can send a PR again after maintaining all the steps and eligibility criteria for a valid PR...


Improve Audio Generation Speed

Hi,
first of all thank you very much for your contribution!
I'm trying to build a realtime voice assistant, for which I use different tools for stt, nlp and tts. I would love to use your voice for this, but the on the fly audio generation is a bit slow with your tacotron2 model.
I found this comparison https://github.com/coqui-ai/TTS/discussions/522
Is there any way to speed up the audio generation to similar values of the english models?

@domcross and i are working on new/better models using HifiGAN vocoder. Samples available on Thorsten-Voice project website. These models might be faster than the current one available. But maybe you should check work by @synesthesiam with larynx. My voice is available there too and it's really fast.

Thanks for your quick reply and for pointing me in the right direction!
I just used your model with the parameters specified in the readme: tts-server --model_name tts_models/de/thorsten/tacotron2-DCA

I tried following pip3 instal tts==0.5.0 and run tts-server --model_name tts_models/de/thorsten/tacotron2-DCA . Got an RTF around 0,6 - 1 on my notebook cpu which i think isn't too bad. What RTF do you have?
Just if you're interested in:
https://www.thorsten-voice.de/2022/03/20/vergleich-thorsten-aktuell-mit-dem-neuen-modell/

I just did that with the addition of --show_details SHOW_DETAILS and my RTF is about 0,6:
 > Processing time: 3.101564407348633
 > Real-time factor: 0.5756691513639508

I use a 12 Core Ryzen 3000 Processor.
But the Processing time of 3s is extremly high given my use case of generating just in time responses for my voice Assistant.
I build a workaround wich caches most wav files, but if I generate Responses with variable in the text this doenst work.
Also i would love to use your new model, is there a way to use it?

The new model is not released yet. I'll keep community updated on release date on Twitter or my Youtube channel.
I'd recommend you taking a look larynx as it's designed for small compute power (like a raspberry) and my german voice is available too.

@r4nc0r Keep watch for the release of Mimic 3 (samples), which should be this month. You should get a 8-10x speedup with it; I typically get an RTF of 0.03, but I'm also on a Ryzen 5950X.

Also i would love to use your new model, is there a way to use it?

Hi @r4nc0r ,
you can download model and config on @coqui-ai prerelease 0.7.0 here: https://github.com/coqui-ai/TTS/releases
Easy pip based installation will follow when final 0.7.0 will be released.
In addition you can play around with beta of Mimic 3 with my german voice (and some more german voices) as mentioned by @synesthesiam .
https://mycroft.ai/blog/mimic-3-preview/

As Mimic 3 is already released you can easily use this. You can watch this video on how to set it up and use it and/or check official doc.
If you want to use Coqui TTS (little bit slower, but better quality) you can do this by:

pip install tts==0.7.1
tts-server --model_name tts_models/de/thorsten/vits

I close this issue for now, but feel free top reopen if you have further questions.


Add replicate demo

Hi there! Really awesome work here. I played around with it and added a demo on replicate which can be hit via an API.
If you prefer I get this demo moved to your own user account on Replicate, I can make that happen too, just lmk :)

Awesome! Thanks for your contribution! It would also be nice if the demo can be re-located :-)


feat(security): update to use chainguard images

@howethomas updated Dockerfile to use Chainguard images from Docker Hub. Tested build and vcon-admin runs.


Restructuring of repo and introducing inspec-k8s

Any chance to merge #9 in here and then update the README and walkthrough for the new approach?

Any chance to merge #9 in here and then update the README and walkthrough for the new approach?

Suggest we merge #9 first into master, rebase and update the walkthrough after merging this PR.

Merging into dedicated branch for now


Update create-wip-policy-using-intune.md

There is a bug when creating WIP policies using for instance the Swedish language Intune management portal and the policies simply do not apply in that scenario. So to prevent users from creating WIP policies and wondering why they simply do not work (I struggled hours with this before finding out the language bug) you should add this text to this article.

Closing the pull request since you're working directly with the product team to figure out if this is a bug. If this ends up being more than a bug, please open another pull request and we'll get the content updated


Plans to generate schemas for *Input types ?

Hello,
Just wanted to know whether it's planned to support generation of input schemas too.
Prisma generates a lot of types like 'ModelCreateInput, ModelUpdateInput....' so they can be quite useful for validating requests. The problem with current solution is that properties aren't marked as required so schemas can't be used for validating user input.
Thanks

There are no plans to integrate further features because of a full-time job, studying, and a side project :(. But a PR is very welcome :)

@MikeYermolayev I will try to work on this soon-ish. Can you maybe describe in details, which kind of types would be useful for the first iteration?


Fix for filtered modules in SelectModulesScreen [Issue #3050]

Contains
Small fix for the Issue #3050.
The issue was that getExplicitlySelectedModules() method use sortedModules, and filterText() method do the same. In this case, when we call refreshSelection() it works with "filtered/searched" modules which might be are not explicitly selected.
How to test

Create new game
Activate JoshariasSurvival module.
Search "help". You should get list from 3 items (In Game Help, In Game Help API, and WorkstationInGameHelp)
Activate and deactivate "In Game Help API".
Check that search result is still yellow and presented.

Hooray Jenkins reported success with all tests good!


Fix: Add missing icon and fix upper casing of tags

Fixes
This PR fixes

The tags on the self-assessment page. It now only upper cases the first letter, but it leaves dates alone
It adds the missing icon on the recommendation page

Haven't checked the design, but would it be better to put the img within the a?


Remove force--consent class from html after accept

The class stays even after accepting and overflow-x: hidden!important; can cause unexpected issues later.

This will be addressed in the next release!


STY: fix lint rule UP031

Reference issue

#21972
What does this implement/fix?

This PR aims to tackle the issue above by replacing printf strings with either f-strings or using the .format() method.
Additional information

There are over 200 more errors left. I will try to fix them over the upcoming week.

Sorry I should have mentioned - you can ignore any failures in directories which are ignored in tools/lint.toml:
exclude = [
    "scipy/_build_utils/tempita/*",
    "scipy/_lib/_docscrape.py",
    "scipy/datasets/_registry.py",
]

You could add these to your ruff command with --exclude, or try to use a command based on the existing config file with --config-file=tools/lint.toml

Sorry I should have mentioned - you can ignore any failures in directories which are excluded in tools/lint.toml:
exclude = [
    "scipy/_build_utils/tempita/*",
    "scipy/_lib/_docscrape.py",
    "scipy/datasets/_registry.py",
]

You could add these to your ruff command with --exclude, or try to use a command based on the existing config file with --config-file=tools/lint.toml

no worries! I already did changes in the first two, is it fine if we keep them for now? I'll skip the registry.py one.
Also, im about to make some more updates today.. it would be nice if you could do a quick sanity check, just to make sure im in the right direction :).

feel free to leave them in for now, we can always revisit before merge. Looks like your changes are along the right lines so far, yep!

@lucascolley I just finished fixing all the errors!
Running ruff check . --target-version=py310 --select UP031 now yields 0 errors :).
There was one thing I noticed.. for some changes I was putting everything on one line and they didn't get formatted properly when committing :(. Is there a command you use for automatic formatting (specifically for line too long issues).

Nice! We don't have an autoformatter you can use sadly (see e.g. https://github.com/astral-sh/ruff/discussions/7310?sort=top#discussioncomment-7266904 if you're interested). We're not too picky on formatting generally, but we do enforce a line length limit of 88 via the linter.

Nice! We don't have an autoformatter you can use sadly (see e.g. astral-sh/ruff#7310 if you're interested). We're not too picky on formatting generally, but we do enforce a line length limit of 88 via the linter.

I am not too familiar with the linter, is there a specific command I can run?
Also, not sure why some of the CI is failing, does it have something to do with the changes I've made?

The lint job has failures relevant to your changes: https://github.com/scipy/scipy/actions/runs/12218999555/job/34091634758?pr=21987. Let's look at any others after you fix those up. You can run python dev.py lint to check your changes locally.

I am getting the following error when running python dev.py lint
Traceback (most recent call last):
  File "/workspaces/scipy/dev.py", line 121, in <module>
    from doit.cmd_base import ModuleTaskLoader
ModuleNotFoundError: No module named 'doit'

Not sure how I can fix that. Also if the PR in #22304, doesn't have these linting issues, I don't mind closing this PR in favor of the other one :).

No module named 'doit'

How have you set up your development environment? If you follow the instructions at https://scipy.github.io/devdocs/building/index.html#building-from-source-for-scipy-development you should get doit installed.

FYI, if you merge main, you should now be able to run python dev.py lint --all to check all files at once against our config file tools/lint.toml

config file tools/lint.toml. (There should be 6 Python errors from main that are unrelated to your changes)

Thank you! The new command was very helpful :).
I just finished fixing all the remaining linting issues, could you please have a look, thanks!

for the CI errors, you've accidentally updated the scipy/_lib/array_api_extra submodule while merging main. You can fix it by checking out main, ensuring you are up to date with upstream, running git submodule update, then switching back to this PR branch and committing the submodule update

unfortunately, still receiving some errors (probably due to _generate_pyx.py). will investigate later today.

The mmio changes were quite complicated. Fortunately, that implementation only powers a deprecated submodule (scipy.io.mmio), so getting the tests passing is satisfactory for me.

The mmio changes were quite complicated. Fortunately, that implementation only powers a deprecated submodule (scipy.io.mmio), so getting the tests passing is satisfactory for me.

ok sounds good, let me know if I can still be of any help! Thanks a lot for your reviews and feedback :). P.S. I really hope this PR doesn't cause any subtle breaks in the future.


Email "0" does not comply with addr-spec of RFC 2822 when sending notification

Hello,
I found an error on the plugin. The problem is that the $properties["mail_recipients"] is an array, due to the fact that the value is registred with datatable widget (with old magic forms is was registered as a simple string).
The idea to change the widget type is good, but you have to create an associative array with the email as key. Actually the key is [0].

Thank you,
Dorian

Hi @fossadierDorian,
Using the old format for the "mail_recipients" property will result in this error, below is an example of the old format:
[genericForm]
mail_recipients[] =<EMAIL_ADDRESS>mail_recipients[] =<EMAIL_ADDRESS>
Update these to use the email as the key and the name of the recipient as the value:
[genericForm]
<EMAIL_ADDRESS>= "Blake Jones"
<EMAIL_ADDRESS>= "Example Person"

Please let me know if this clears things up or not. If I've misunderstood, please elaborate further.
Updated documentation that documents this, as well as a full upgrade guide are coming soon.
Thank you!


Library seems to resize toggles

It works great, in terms of bindings but, when applying, it seems to stop the toggles from resizing properly, so they're all the default width (and text overflows).
Thanks

This appears to be beacuse line 23 reapplies the bootstrapToggle:
$(element).bootstrapToggle();

Can you provide a minimal working example for this?


"Ubuntu" is misspelled in book/getting-started/setup.md

book/getting-started/setup.md says "Ubunutu setup". It should say "Ubuntu Setup".

Covered in #1420.


Create a middleware to sanitize the request body

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from starlette.middleware.base import BaseHTTPMiddleware
from starlette.types import Message
from fastapi import Request
import bleach
import json

class SanitizeMiddleware(BaseHTTPMiddleware):

    def __init__(self, app):
        super().__init__(app)

    @staticmethod
    def __sanitize_array(array_values):
        for index, value in enumerate(array_values):
            if isinstance(value, dict):
                array_values[index] = {key: bleach.clean(
                    value) for key, value in value.items()}
            else:
                array_values[index] = bleach.clean(value)
        return array_values

    async def set_body(self, request: Request):
        receive_ = await request._receive()
        async def receive() -> Message:
            return receive_

        request._receive = receive

    async def dispatch(self, request: Request, call_next):
        await self.set_body(request)

        if request.method == 'POST' or request.method == 'PUT':
            json_body = await request.json()
            sanitize_body = {key: self.__sanitize_array(value) if isinstance(
                value, list) else bleach.clean(value) for key, value in json_body.items()}
            request._body = json.dumps(
                sanitize_body, indent=2).encode('utf-8')
            await self.set_body(request)

        response = await call_next(request)
        return response

Description
Hello guys, I'm facing an issue while trying to sanitize the request body from POST and PUT methods. The idea is to sanitize the request.body() before it even reachs the view. For instance, I would like to pass bleach on it to avoid security issues that might appear under the body that is sent.
I've built a middleware that inherit from BaseHTTPMiddleware, to sanetize the body, but I've notived that I'm not changing the original request element:
Operating System
Linux
Operating System Details
No response
FastAPI Version
0.73.0
Python Version
3.8.10
Additional Context
No response

You never return your request from set_body. Try this:
class SanitizeMiddleware(BaseHTTPMiddleware):

    def __init__(self, app):
        super().__init__(app)

    @staticmethod
    def __sanitize_array(array_values):
        for index, value in enumerate(array_values):
            if isinstance(value, dict):
                array_values[index] = {key: bleach.clean(
                    value) for key, value in value.items()}
            else:
                array_values[index] = bleach.clean(value)
        return array_values

    async def set_body(self, request: Request):
        receive_ = await request._receive()
        async def receive() -> Message:
            return receive_

        request._receive = receive
        return request

    async def dispatch(self, request: Request, call_next):
        await self.set_body(request)

        if request.method == 'POST' or request.method == 'PUT':
            json_body = await request.json()
            sanitize_body = {key: self.__sanitize_array(value) if isinstance(
                value, list) else bleach.clean(value) for key, value in json_body.items()}
            request._body = json.dumps(
                sanitize_body, indent=2).encode('utf-8')
            request = await self.set_body(request)

        response = await call_next(request)
        return response

You never return your request from set_body. Try this:
class SanitizeMiddleware(BaseHTTPMiddleware):

    def __init__(self, app):
        super().__init__(app)

    @staticmethod
    def __sanitize_array(array_values):
        for index, value in enumerate(array_values):
            if isinstance(value, dict):
                array_values[index] = {key: bleach.clean(
                    value) for key, value in value.items()}
            else:
                array_values[index] = bleach.clean(value)
        return array_values

    async def set_body(self, request: Request):
        receive_ = await request._receive()
        async def receive() -> Message:
            return receive_

        request._receive = receive
        return request

    async def dispatch(self, request: Request, call_next):
        await self.set_body(request)

        if request.method == 'POST' or request.method == 'PUT':
            json_body = await request.json()
            sanitize_body = {key: self.__sanitize_array(value) if isinstance(
                value, list) else bleach.clean(value) for key, value in json_body.items()}
            request._body = json.dumps(
                sanitize_body, indent=2).encode('utf-8')
            request = await self.set_body(request)

        response = await call_next(request)
        return response

I tested it here and it continues with the same previous request

Hmm I did not expect that.. 🤔 I am tinkering around with this and I notice that there is a custom request being send to the next ASGI app, but the body is unsanitized. If I include a custom key in the request.scope, then that key (and value) is available in my endpoint.
Somewhere, Starlette (or FastAPI but I am thinking Starlette) is unsanitizing the body. Which is very weird!

I think that maybe the request element isn't the same at all, I've noticed that the id for both are different, not sure if that tells much but I'm assuming that they might be addressing to different elements in the memory, but is pure speculation at this point.


process.server use paypal sdk

use paypal node sdk https://github.com/paypal/Checkout-NodeJS-SDK
I want to redirect or store commit, I can't succeed.
How do I get a url and jump?
async asyncData({redirect, store}){
    if(process.server){
        const paypal = require('@paypal/checkout-server-sdk');

        let clientId = "<<PAYPAL-CLIENT-ID>>";
        let clientSecret = "<<PAYPAL-CLIENT-SECRET>>";
        let environment = new paypal.core.SandboxEnvironment(clientId, clientSecret);
        let client = new paypal.core.PayPalHttpClient(environment);
        let request = new paypal.orders.OrdersCreateRequest();
        request.requestBody({
            "intent": "CAPTURE",
                "purchase_units": [
                     {
                      "amount": {
                          "currency_code": "USD",
                          "value": "100.00"
                      }
                  }
              ]
          });
         (async ()=>{
             let response = await client.execute(request);
             let url = response.result.links.find(item => { item.rel === 'approval_url' });

             // I want to redirect or store commit, I can't succeed.
             // redirect(url.href);
             // store.commit('setUrl', url.href);
         })();
    }
}

This question is available on Nuxt community (#c9622)

use paypal node sdk https://github.com/paypal/Checkout-NodeJS-SDK
I want to redirect or store commit, I can't succeed.
How do I get a url and jump?
async asyncData({redirect, store}){
    if(process.server){
        const paypal = require('@paypal/checkout-server-sdk');

        let clientId = "<<PAYPAL-CLIENT-ID>>";
        let clientSecret = "<<PAYPAL-CLIENT-SECRET>>";
        let environment = new paypal.core.SandboxEnvironment(clientId, clientSecret);
        let client = new paypal.core.PayPalHttpClient(environment);
        let request = new paypal.orders.OrdersCreateRequest();
        request.requestBody({
            "intent": "CAPTURE",
                "purchase_units": [
                     {
                      "amount": {
                          "currency_code": "USD",
                          "value": "100.00"
                      }
                  }
              ]
          });
         (async ()=>{
             let response = await client.execute(request);
             let url = response.result.links.find(item => { item.rel === 'approval_url' });

             // I want to redirect or store commit, I can't succeed.
             // redirect(url.href);
             // store.commit('setUrl', url.href);
         })();
    }
}

This question is available on Nuxt community (#c9622)

What if you change this line:
let url = response.result.links.find(item => { item.rel === 'approval_url' });
To:
let url = response.result.links.find(item => (item.rel === 'approval_url'));


Ability to merge application pubspec.yaml with flavor specific versions

Use case
A white labeled application needs to be published for different package IDs, version numbers and assets.
While basic building of the application flavors can be accomplished by using flutter build ... --flavor xxx, a key component like pubspec.yaml can not be customized as detailed as needed. Only the version name and build number can be specified on the command line.
The assets part is not easy to maintain for separate applications.
Proposal
flutter --flavor xxx should resolve the following pubspec.yaml files by default:

pubspec.yaml
pubspec.xxx.yaml

The keys from pubspec.xxx.yaml should be merged into pubspec.yaml.
Existing non-array keys (name, version) should be overwritten.
Maps and Array keys should be extended.
E.g.:
If pubspec.yaml contains:
flutter:
  uses-material-design: true
  assets:
    - assets/icons/
    - assets/sounds/

And pubspec.xxx.yaml contains:
flutter:
  assets:
    - assets/fonts/xxx/
    - assets/icons/xxx/

Then the resulting flutter.assets path should be parsed as:
flutter:
  uses-material-design: true
  assets:
    - assets/fonts/xxx/
    - assets/icons/
    - assets/icons/xxx/
    - assets/sounds/

This would be a extension on how flutter handles pubspec.yaml and should probably not become part of the generic pub command.

https://github.com/flutter/flutter/issues/21682
https://github.com/flutter/flutter/pull/25487

@robinbonnes thank you for linking to these other issues. We could close this one in favour of #21682.

Duplicate of  https://github.com/flutter/flutter/issues/21682


Am getting the following error in the latest version of IdentityServer4 in ASP Core, "Constants variable cannot be accessed due to its protection level" at Constants.BuiltInIdentityProvider. I get the error while using the latest package of IdentityServer4 but not getting it in "IdentityServer4": "1.0.0-beta4-update2" package? Has the functionality of accessing the identity provider of a claim type changed? Sorry for the poor formatting.

public override async Task SignInAsync(TUser user, AuthenticationProperties authenticationProperties, string authenticationMethod = null)
{
var userPrincipal = await CreateUserPrincipalAsync(user);
        userPrincipal.Identities.First().AddClaims(new  []
        {
            new Claim(JwtClaimTypes.IdentityProvider, authenticationMethod ?? **Constants.BuiltInIdentityProvider**),
            new Claim(JwtClaimTypes.AuthenticationTime, DateTime.UtcNow.ToEpochTime().ToString())
        });

        await Context.Authentication.SignInAsync(options.Cookies.ApplicationCookieAuthenticationScheme,
            userPrincipal,
            authenticationProperties ?? new AuthenticationProperties());
    }

IdentityServerConstants


feat: support inverting header matches for rate limit

What type of PR is this?
Feature changes for supporting inverting header matches for rate limiting.
What this PR does / why we need it:
Implements API changes proposed in #4176
Which issue(s) this PR fixes:

Ref #2193

the PR looks good, can you add some tests in https://github.com/envoyproxy/gateway/tree/main/internal/gatewayapi/testdata & https://github.com/envoyproxy/gateway/tree/main/internal/xds/translator/testdata

can you make linter happy?

can you make linter happy?

here are the errors

internal/xds/translator/testdata/in/xds-ir/ratelimit.yaml
  Error: 74:13 [indentation] wrong indentation: expected 10 but found 12
  Error: 75:17 [indentation] wrong indentation: expected 14 but found 16
  Error: 86:9 [indentation] wrong indentation: expected 6 but found 8
  Error: 87:13 [indentation] wrong indentation: expected 10 but found 12
  Error: 88:26 [new-line-at-end-of-file] no new line character at the end of file

@arkodg @zirain done!


Move dynamic pdb-redo data loading to StructuresTable

[demo PR, not to be merged] @jvfdunkley

I've moved all logic from PdbCell to StructuresTable, as now we want to filter by PDB validations.
I've removed useLocalStorage altogether and created a CacheRepository instead (to be called from use cases or other repositories). This way, we decouple the caching from React infrastructure, which is always nice.
useRenderedRows restores some code we had and need again, as we load the validations on the parent StructureTable only for the rendered/visible rows, not within the PDB cell (where we were certain it was going to be rendered).
Now the workflow is: 1) Load the covid19 data (without PDB validations), we can already render 2) Load PDB validations on the background and re-render rows when finished, and 3) Filter the data (which contain the validations, so you can now add the new filter there).

Feel free to refactor it as you see fit.

@jvfdunkley if I am not wrong you merged this code into https://github.com/EyeSeeTea/3DBIONOTES/pull/123, right? Can we then close this PR?

@adrianq yes we can close it, I merged the code


accessibility: increase color contrast ratio to >7

this fixes #93
and increeses the lighthouse score!
Lighthouse requires a color contrast ration of >7. The current value is 3.38.
I'll changed the clolor blue to be above 7 (Level AAA) - and still look good :wink:.
see TR/WCAG21#contrast-minimum

Looks ok and fixes the contrast ratio. Thanks a lot!


URL-decode the template

If the template is received in response to an API request it will be URL-encoded - https://host/api/%7b?foobar%7d which this library apparently cannot handle.
It would be much more convenient to use a URL-encoded template directly without having to decode it first.
Current usage:
var link = entryPoint.Links.Single(l => l.Rel == "foo-bar");
var href = new UriTemplate(HttpUtility.UrlDecode(link.Href))
    .AddParameters(new
                   {
                       foobar = "foobar"
                   })
    .Resolve();

Desired usage:
var link = entryPoint.Links.Single(l => l.Rel == "foo-bar");
var href = new UriTemplate(link.Href)
    .AddParameters(new
                   {
                       foobar = "foobar"
                   })
    .Resolve();

Thanks for the suggestion.
My first observation is that this is an orthogonal problem - RFC6570 uri templates themselves don't support this "external encoding".
My second concern is that HttpUtility.UrlDecode() allocates strings. We wouldn't want the standard constructor to pay the cost of decoding (and the consequent allocations).
If you are using this code a lot, have you considered something like
/// <summary>
/// Helpers for dealing with a URL-encoded RFC6570 uri template.
/// </summary>
public static class UrlEncodedUriTemplate
{
    /// <summary>
    /// Decodes a URL-encoded RFC6570 uri template, and creates a <see cref="UriTemplate"/> from the resulting string.
    /// </summary>
    /// <remarks>
    /// Note that this trades allocations for convenience.
    /// </remarks>
    public static UriTemplate Create(string urlEncodedUriTemplate)
    {
        return new UriTemplate(HttpUtility.UrlDecode(urlEncodedUriTemplate));
    }
}

You could then write code like:
var link = entryPoint.Links.Single(l => l.Rel == "foo-bar");
var href = UrlEncodedUriTemplate.Create(link.Href)
    .AddParameters(new
                   {
                       foobar = "foobar"
                   })
    .Resolve();

Thanks for considering this and the insights. You are right, decoding the template probably shouldn't be built into this library for the reasons you outline above even if it would make the consumption much more convenient.

I've got another comment in #68 which might elaborate on this a little bit now I've read both of them!


Adding perceptional precision support

This PR adds support for the great work that Eric has completed: https://github.com/pointfreeco/swift-snapshot-testing/pull/628
png support only for now. I'll be working on updating the HEIC library next.

Been following along on the creation of this feature upstream and looks super interesting. Need to play with it more but I'm happy with what's here.


Broken Sexp2dict

As per my conversation with @ktonga
There was an issue in ProjectConfig's parse method's sexp2dict method where incase of list of lists in the loaded sexp, only the first element of the list was considered.

Always suspected that was too easy 😄  Thanks!


Differentiable way of constructing an SE3 from SO3 and translation

📚 The doc issue
What is the properly differentiable way to construct an SE3 object from an SO3 and 3-dim tensor?
For example:
R = pp.rand_SO3()
b = torch.randn((3,))

Can we construct a SE3 like:
T = pp.SE3(R, b)

such that gradient with respect to T can be propagated to R and b?
Suggest a potential alternative/fix
Documentations or a new constructor of SE3

Currently, we can do
>>> t = torch.randn((3,), requires_grad=True)
>>> r = pp.randn_SO3(requires_grad=True)
>>> T = pp.SE3(torch.cat((t, r.tensor()), dim=-1))
>>> T.sum().backward()
>>> t.grad
tensor([2., 2., 2.])
>>> r.grad
tensor([1., 1., 1., 1.])

Here .tensor() is to bypass a User warning, but it can still run without calling it.
We can discuss your expected use case of pp.SE3(R, b), but we cannot promise right now.

Thanks, I would use this way for now.


Fixed Image and Test Links

why u remove the twitter tags? they indetify what is to be tweeted?

The image was not showing due to the tag.


APIM Store generated tokens are not shown

Description:
When refreshing the Store Application Production key generation page, Keys and Tokens are not shown
Affected Product Version:
APIM 3.0.0 M8

Fixed with following PR: https://github.com/wso2/carbon-apimgt/pull/4800 , Hence closing the issue


adding cluster connection check

https://github.com/sustainable-computing-io/susql-operator/issues/38

Merging this, since it's straightforward.


chore: investigate whether special support for CSI ephemeral volume or generic ephemeral volume is required

What happened:

It needs to be further investigated whether CSI ephemeral volume or generic ephemeral volume need to inline AzVolume creation, like inline Azuredisk volumes.

What you expected to happen:

If determined required, AzVolume instances need to be created for inline CSI ephemeral volume and generic ephemeral volume in Pod Controller.

How to reproduce it:
Anything else we need to know?:
Environment:

CSI Driver version:
Kubernetes version (use kubectl version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

CSI ephemeral volumes aren't supported by azure disk driver
For generic ephemeral volumes, create volume and controller publish volume calls are made like any other volume created using pvc, so AzVolume and AzVolumeAttachment CRIs get created as expected. There shouldn't be a need to do any special handling for this. Tested it by deploying the changes from PR: https://github.com/abhisheksinghbaghel/azuredisk-csi-driver/pull/346 and by using this example: https://github.com/abhisheksinghbaghel/azuredisk-csi-driver/blob/master/deploy/example/nginx-pod-ephemeral.yaml. The CRIs get deleted upon the deletion of pod.

Awesome. Thank you for looking into this in detail! 👍

Given the discussion in kubernetes-sigs/azuredisk-csi-driver#1189, I do not believe we should implement support for CSI Ephemeral Volumes in the Azure Disk CSI Driver.


Reproducability

Fixes the reproducability issue by introducing a seed as parameter in 'training.yml' that goes into the 'torch.manual_seed' method (see docs) within the instructor init.

Hello @stroblme! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/quantum_siren/pipelines/training/nodes.py:

Line 42:89: E501 line too long (173 > 88 characters)

Quick result from two runs with identical parameters resulting in identical metric values. Intermediate values match as well; plot omitted.


Support for multiple accounts under the new authentication scheme

This is a big thing missing when moving to centralised CREST SSO. Scouts have multiple accounts and alternate between them: having to logout and relogin constantly is a massive hindrance to usability.
We need a way to associate a Pathfinder login with multiple accounts.

Yeah this is indeed a Problem. It took me some time to finish https://github.com/exodus4d/pathfinder/issues/131#issuecomment-214016064 first :(. All new features are now implemented for v1.0.0RC3. Now I´ll focus on this Issue and some smaller bugs before it is ready to release it.

That's awesome -- thanks again for all the hard work, I hope you don't
think I'm giving you a hard time!
On 24 April 2016 at 11:29, Mark Friedrich<EMAIL_ADDRESS>wrote:

Yeah this is indeed a Problem. It took me some time to finish #131
(comment)
https://github.com/exodus4d/pathfinder/issues/131#issuecomment-214016064
first :(. All new features are now implemented for v1.0.0RC3. Now I´ll
focus on this Issue and some smaller bugs before it is ready to release it.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/exodus4d/pathfinder/issues/138#issuecomment-214020092

Oh no, I appreciate our feedback! Unfortunately I don´t have the time to play a lot, therefore user feeback is very important for me!
How it works

An user has to login via CCP´s SSO when using Pathfinder for the first time with this character (this should always work on the current dev branch.
Users get a sort Cookie notification, which can be accepted (or not). Cookies will not be send to clients that have not accepted this hint. (EU legislation on cookies)
If login was successful a login-cookie is send to the client
When he comes back, Pathfinder will check for existing cookies and provide character login for valid characters.

Technical information
After reading a lot about "long term User authentication" and security concerns, I really liked they way described in this article "Implementing Secure User Authentication in PHP".(awesome article!)

Any Cookie is stored on a new DB table character_authentication which is used to verify the validity of each Cookie (Client side manipulation e.g. changing the "expire date" will result in an invalid Cookie,...)

There is no "blank" data stored on a Cookie (like DB ids,...). So there is no chance of "guessing" other cookie data. No way to get additional information about the App (e.g. number of users/characters which would be the case when we use DB ids,...)
Expire time can be set in pathfinder.ini (default: 30days)
Each character has its own Cookie. The Cookie name is unique ("char_" . md5([CHARACTER_NAME]). The name does not have to be secure, it just should be unique :)
The Cookie value of it consists of two parts
A unique selector generated out of an random initialization vector is used to identify the validity of the Cookie against the DB
A validator which represents the character name. The hashed (sha256) version of this validatoris stored in the DB column and is used for an additional layer of security

Even if somebody breaks into the DB, it is not possible to generate valid Cookie information just from the DB tables
Cookies are set with httponly = true setcookie(); to protect against JS manipulation.

Cookie verification process

Read all character log in cookies
Check Cookie format (selectorand validator must be present)
Select DB column from character_authentication table where selector data matches appropriate column
Check if Cookie expire time matches DB expire time in character_authentication
Check if character "exists" is "valid" and "active"
Check if validator column  character_authentication table matches sha256-hashed  version of Cookie validation data
Try to update Character over CREST (update access_token, e.g. update corp/ally,...)
Check if character is authorized to log in (e.g. corporation has changed/not on "white list", or App access was changed/limited to specific corps/allys,...)
Login :)

I'm glad the article was helpful; from a security standpoint your implementation looks solid but I'll have a closer look today as it's not currently working for me (possibly because you set the cookie via javascript instead of PHP and I have some addons running).
Would it be possible to set a default character so that when you land on the homepage you get automatically logged in? You could then use the character selector at the top to switch accounts as well.

Hmm, I might be crazy, but I only see the EU legislation cookie actually being set in your commit? None of the char_ cookies appear to being set at all.

The first cookie (which is set when you confirm the "set cookie" hint at the bottom) is set by javascript.
The character cookies are set by PHP when you log in by SSO: sso.php#L208 in this function
[setLoginCookie();] (https://github.com/exodus4d/pathfinder/blob/develop/app/main/controller/controller.php#L164)
The characters you see on the homepage are already "verified" and updated by CREST, by checking your cookies. This is done by Ajax requests for each character cookie on document.onload. I implemented it that way, because it takes about 100ms to perform CREST calls and DB updates for each character. -> Otherwise a user with 10 characters would wait about 1s before the page gets initial loaded.
Just click on one of the character images and you are logged in.
But you can still click the SSO button, for log in a fresh character. This will also update cookie data of an existing character you already have access to (e.g. you click the SSO button instead of the character image,..)
I don´t think we need a "default character", it is just one click to select a char.
I´m not completely done with this task. There are still some things missing:

When a character was sold -> all existing character_authentication relations must be deleted
The getCookieCharacters(); handles all the verification process of character cookies (invalid cookies are deleted at the end). But we need a new cronjob as well which deletes expired cookies (just to keep the DB clean). This should be done on EVE server downtime
You are right, a character "logout" should also delete cookie information from DB
The character switch (on the map page) is currently not "in line" with the one on the homepage (and not using those cookie information). This will be unified.

Because Pathfinder gets more depending on CCPs CREST API by the time. It would be helpful to see/check CREST status and EVE server status information. In case CREST is down, you will not  be able to log in and use the app.
Therefore a implemented a small notification panel on the login page:

https://github.com/exodus4d/pathfinder/commit/dfd1e8501d17acb55a38b2cfb823c2425a3e4572 Added new "cookie logout" functionality. -> If you manually logout (click on the menu entry), all server-side stored cookie information are deleted, too.

This will prevent anyone else to login this character with a valid cookie
This will delete your cookie for this character
This will not logout any character from the active user

If there is already someone else logged in (with an active SESSION), he will logged out as soon as PATHFINDER.TIMER.LOGGED (default: 240 minutes -> pathfinder.ini) expires. This may can be improved in the future but requires some additional programming to work.

I woulds suggest making the Accept cookies notice a bit bigger and more obvious.


make 1.0 pass unit tests for mosek 8.0

This solves issue #449, I tested with two mosek versions, 8.1.34 and 8.1.41

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.tao.luo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverage decreased (-0.005%) to 83.346% when pulling b4eeac96075106b89d5e9be9361b9835448dfcb1 on mickeydonkey:1.0 into d9a2bcc8ccebde28497da5c42566c2e08dfb0eef on cvxgrp:1.0.

As suggested by @rileyjmurray in #449 , check basic feasible solution with 10 decimal places for higher precision.

Ok looks good.


CONCERNING HOW TO INSTALL THE "COCOA PODS"

I had a problem installing - $ gem install cocoapods and $ pod install. Is there a way you could please, help me solve this bug ?

do sudo gem install cocoapods


Incremental.

@mzhurovich -- merging for your post-review =)

FTR, this was the prerequisite to https://github.com/dkorolev/c5t_popen2_example/pull/3/files, after which I can create the (ugly!) PR with the logic of graceful shutdown.


Watermark Model unnecessarily requires create_time

Expected Behavior

create_time is an optional field
Current Behavior

create_time is a required field in the Watermark model https://github.com/amundsen-io/amundsendatabuilder/blob/master/databuilder/models/watermark.py#L23
Possible Solution

Make create_time not required in the watermark model on the backend, or in the watermark type on the frontend.
Steps to Reproduce

Publish data to the model without create_time

Screenshots (if appropriate)
Context

I'm passing in Snowflake min/max ts data in the watermark model. There are no physical partitions in Snowflake, so create_time does not apply for our models
Your Environment

Amunsen version used: Databuilder 4.0.3
Data warehouse stores: Snowflake
Deployment (k8s or native): Docker

@instazackwu would gladly take a PR to fix this, if you want!