Update nodemon SemVer to ^3.0.0

Tried it and everything seems to work with the updated nodemon version.

Hi @pontusva, thanks for letting me know. I guess you are talking about create-vite-express templates, yeah, their dependencies are a little bit outdated so I'll try to update them.

@szymmis Hey!
Yes, that was what i was referring to. I think i got the repos mixed up?

Hi @pontusva, thanks for letting me know. I guess you are talking about create-vite-express templates, yeah, their dependencies are a little bit outdated so I'll try to update them.

No no, you got the repo right. The create-vite-express is a subpackage of this repo.
Maybe you want to make a contribution and update the dependencies? If you do, I'll be happy to answer any questions you'll have and provide guidance! If not then no problem and as I said, I'll try to find some free time to update them.
Either way, thanks again for letting me know!

No no, you got the repo right. The create-vite-express is a subpackage of this repo.
Maybe you want to make a contribution and update the dependencies? If you do, I'll be happy to answer any questions you'll have and provide guidance! If not then no problem and as I said, I'll try to find some free time to update them. Either way, thanks again for letting me know!

Oh okay, thank you for letting me know :)
I'd really  like to contribute and update the dependencies. I would be my first contribution and this seems like a good one to start with.
Is there a specific workflow for contributing?

Happy to hear that!
There is no official guideline at the moment. That means all you have to do is to:

fork the repository
make changes on your fork
create a PR onto this repository.

As soon as the PR shows up I'll try to check it out, and we can discuss details there.
In terms of requirements: I'd like you to update every dependency you can in every template.
You can run npm run test to check out if your changes work. I know that there are problems with tests on Windows. In that case you can simply create a local project and check it there manually. Whatever workflow suits you.
And as I said earlier, when you are ready, just open the PR.
Feel free to ask if anything is still unclear.

Ok, I'll look into this :) and reply here if i have any further questions

Hey @szymmis
I'm a bit new to forking and contributing in general. But i've made a fork and updated all dependecies. All tests passed. But i noticed this: https://vitejs.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated
And also i pushed to  branch on the fork i made but it seems that it is connected to this repo? Maybe that¨s normal behaviour :)

Okay, that link looks interesting, definitely something I need to check out.
Are you getting some kind of warnings or how did you stumbled upon this?
Yeah, your fork is connected to this repository, that's how they work. It's different from just creating a new repository on your account and pasting all the files there. Forks are generally used as a way to implement some changes to public repositories that you don't have write access as this one. It is de facto a standard of contributing.
You can open a PR now merging your branch onto this one.
That way I'll be able to check it out and test myself.


is a default toArray() method for IPP Objects worth thinking about?

I'm not sure if it's an accident or a feature, or simply a result of the conversion from Java originally, however IPP Objects are very strict, and also don't have a default array conversion.
We're using Laravel for our app, and have to coerce the IPP objects into arrays before passing them to our illuminate response component, which is fine, and probably lazy on our part to expect not to have to do that work... but I wondered if it was a worthwhile question.
Is it worth looking into how that might work?

I'm a goose.
My error message was The Response content must be a string or object implementing __toString(), "object" given.
I guess it's going to be hard to decide if it should be JSON or XML... so maybe this conversation can be closed.


1.0 release

Event loop 0.4 is very stable and there aren't any big BC breaks planned (maybe #30) so what is stopping us from putting a list of small things together we want in 1.0 and work towards that so we can tag 1.0 and push any major redesigns to 2.0?

:+1: I would like to see 1.0 tagged the way it is now. If there is to be more discussion on #30, let it wait until the next minor release where it can be deprecated until 2.0. The react libraries - especially event-loop and stream are great libraries that I use in production regularly. I would hate to think that people don't use them simply because they are still pre v1.

:+1: for 1.0 release!

@cboden, @clue  - what is needed to get to 1.0?

:+1:

:+1:

See #101 for new roadmap to v1.0.0 :+1:


added config for irt to build pytest-inmanta-extensions

Description
See inmanta/irt#690
closes inmanta/irt#682
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches master in 4101f96bcddfefa71ad3ea2f61d667e27d9dfda7

Processing #2898.


Reorder Dockerfile for better use of cache

Every change to the scripts was causing yum install to run over and over.
@bparees PTAL

[test]

[test]


python310Packages.pycontrol4: 0.3.1 -> 1.1.0

Description of changes
Diff: https://github.com/lawtancool/pyControl4/compare/v0.3.1...v1.1.0

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 216980 run on aarch64-linux 1

3 packages built:

home-assistant-component-tests.control4
python310Packages.pycontrol4
python311Packages.pycontrol4

@ofborg build home-assistant-component-tests.control4


feat: provide with an alias

Current coverage is 100% (diff: 100%)

Merging #13 into master will not change coverage

@@           master   #13   diff @@
===================================
  Files          14    14
  Lines         272   256    -16
  Methods        56    56
  Messages        0     0
  Branches        0     0
===================================
- Hits          272   256    -16
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 0c22289...c133929


Themes/Plugins updates installer

Added list of all themes and plugins
Added option for update, delete and install plugins/themes via repos
Added PHP and extensions checkup
added clearer code comments
added ability go to directly to “settings->general, security etc..)
removed themes and plugins links in modal footer
Added themes/plugins caching

Awesome work guys 🚀! I am absolutely loving this new theme/plugin installer.
I think that this will greatly increase the usability of WonderCMS for the less-experienced users.

@StephanStanisic, this one was highly inspired by you! @slavenstancic amazing work 🚀
What's awesome about this is: it includes a caching system that runs once every 24hours the admin is logged in (or it can be run manually through the Settings -> Themes or Plugins. It takes about 10 seconds on my end to fully fetch all versions and screenshots when I click the "Update". However, when it's cached, everything loads in less than a second.
Great job all around everyone, it's an absolute joy working with you.


NodeCanvas destroyed after returning from Play Mode

Unity version: 5.1.2p3
If I open the Node Editor, enter the Play mode, stop the Play mode and click on the Node Editor window again it stops responding and I get a lot of repeated errors in the console with the following error:
MissingReferenceException: The object of type 'NodeCanvas' has been destroyed but you are still trying to access it.
Your script should either check if it is null or you should not destroy the object.
NodeEditorFramework.NodeEditorWindow.DrawSideWindow () (at Assets/Editor/Node_Editor/NodeEditorWindow.cs:111)
NodeEditorFramework.NodeEditorWindow.OnGUI () (at Assets/Editor/Node_Editor/NodeEditorWindow.cs:104)
System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) (at /Users/builduser/buildslave/mono-runtime-and-classlibs/build/mcs/class/corlib/System.Reflection/MonoMethod.cs:222)

Strange, last time I chcked the canvas was simply unloaded, which is a minor issue... Thanks for the report, I'll take a look at fixing that today:)

Adressed that too in my latest commit (2dd6eb8a3e49fe9e1853685282cac4d98dd06522) next to other things, simply added a check for the canvas;)


Noice not working well with winbar

Describe the bug
Having both noice and winbar (with default configrations on both) causes visual issues when going to command or search mode.
Which version of Neovim are you using?
Nightly: NVIM v0.9.0-dev-16-g56998feeb
To Reproduce
Steps to reproduce the behavior:

Use packer to install both winbar and noice
Open up any file and type ":" and you will see the winbar script ontop of the popup and it will change the look of the popup. I will include a an image below.

Screenshots

https://github.com/fgheng/winbar.nvim#configuration
Have you added noice to the exclude_filetype of winbar?

I swear I thought I included that but I just tried that now and that fixed the issue, sorry that was a very silly issue on my end. Thanks for the help though!


Shogun 5.0.0 compile/linking error PythonModular

Hi,
I am trying to install Shogun 5.0.0 from source on a Debian 8.7 server / kernel <IP_ADDRESS>.amd64 inside a dedicated Conda environment (env packages in this gist )
The following cmake command
cmake
-DPythonModular=ON
-DBUILD_META_EXAMPLES=OFF
-DCMAKE_PREFIX_PATH=/TL/epigenetics2/work/pebert/conda/envs/comikl
-DCMAKE_INSTALL_PREFIX=/TL/epigenetics2/work/pebert/tools/shogun_5.0.0
-DSWIG_EXECUTABLE=/TL/epigenetics2/work/pebert/conda/envs/comikl/bin/swig
-DGENERATE_EXAMPLES=OFF

results in this cmake output - looks fine to me...
However, make does not finish w/o errors - see make output.
[ 78%] Linking CXX executable variational_approx_example
../../../src/shogun/libshogun.so.17.2: undefined reference to `dsyev_'
../../../src/shogun/libshogun.so.17.2: undefined reference to `dorgqr_'
../../../src/shogun/libshogun.so.17.2: undefined reference to `dstemr_'
../../../src/shogun/libshogun.so.17.2: undefined reference to `dsyevr_'
../../../src/shogun/libshogun.so.17.2: undefined reference to `dsygvx_'
../../../src/shogun/libshogun.so.17.2: undefined reference to `dgesvd_'

[...]

collect2: error: ld returned 1 exit status
examples/undocumented/libshogun/CMakeFiles/transfer_multitaskleastsquaresregression.dir/build.make:125: recipe for target 'examples/undocumented/libshogun/transfer_multitaskleastsquaresregression' failed
make[2]: *** [examples/undocumented/libshogun/transfer_multitaskleastsquaresregression] Error 1

[and so on]

Maybe related to #3213 ?
Thanks for your help.
Best,
Peter

hi @mpi-pebert thnx for reporting this. in case you just wanna use shogun on your jessie i'd suggest using the apt repository with the nightly packages of shogun: https://github.com/shogun-toolbox/docs/blob/master/INSTALL.md#debian-
the python interface is precompiled as well: simply install the python-shogun package.
i'm successfully using these binaries with a jupyter notebook server on http://cloud.shogun.ml (which i suggest you to try in case you just want to use shogun in a python project); here's the docker image for it:
https://hub.docker.com/r/shogun/cloud/
and here's the Dockerfile:
https://github.com/shogun-toolbox/shogun-cloud/blob/master/docker/cloud/Dockerfile

Hi Viktor,
thanks for the quick reply - sorry, I see that I have left out essential information:
I have no admin privileges on the server (also, no Docker available), and I need to install Shogun with Python interface for a third-party tool for a somewhat large analysis task, so a (public) cloud setup is unfortunately not an option.
+Peter

@mpi-pebert mmm okey... in that case you have still some options:

do you have the dependencies (not the build but the runtime dependencies) for the libshogun17 and python-shogun package on the system? because in that case you can just simply unarchive the deb packages and use the shared libraries inside :)
i suppose you are missing some lapack dependencies... so in that case i'll have to investigate a bit. or probably the lapack detection needs some love in shogun

well, it's worth a shot - I found libshogun17_5.0.0_amd64.deb in the Trusty PPA, but I don't seem to find python-shogun, where can I get the correct deb package?
any hint on what might be missing is of course appreciated

@mpi-pebert if i were you i'd use the packages in the nightly ppa... trust me it's really that much better :) + stable does not have python-shogun package yet
regarding 2.) would it be possible that you send me the output of your dpkg -l on your system? drop it via mail and i can replicate things in a docker container and maybe even fix the cmake

@mpi-pebert btw just realised, you mentioned Trusty? why trusty? isn't it debian jessie 8.7 that you are using atm? if that's the case (debian jessie) use the apt.shogun.ml repository

@mpi-pebert and another btw: have you actually tried to compile develop branch of shogun? if not could you please try it and let us know if it causes the same bug?

I managed to get my hands on libshogun18_5.0.0+1SNAPSHOT201703060902-0_amd64.deb and python-shogun_5.0.0+1SNAPSHOT201703060902-0_amd64.deb using my laptop (where I am root, otherwise same Debian in-house installation). I extracted the contents and linked the libs in my environment (is that the correct procedure? I have never done it that way before...). I just tried the Python interface, which gave me this:
>>> import modshogun
_mod = imp.load_module('_modshogun', fp, pathname, description)
ImportError: [...]../lib/libshogun.so.18: undefined symbol: cblas_dgemm

Also, I cloned the repo and tried to build the dev branch of Shogun - this is the CMake call
cmake
-DPythonModular=ON
-DBUILD_META_EXAMPLES=OFF
-DCMAKE_PREFIX_PATH=/TL/epigenetics2/work/pebert/conda/envs/comikl
-DCMAKE_INSTALL_PREFIX=/TL/epigenetics2/work/pebert/conda/envs/comikl
-DSWIG_EXECUTABLE=/TL/epigenetics2/work/pebert/conda/envs/comikl/bin/swig
-DBUILD_EXAMPLES=OFF

Again, did not result in obvious problems, yet the make resulted in the following:
SGBase.i:311: Error: Unable to find 'swig_typemaps.i'

src/interfaces/python_modular/CMakeFiles/_python_modular.dir/build.make:120: recipe for target 'src/interfaces/python_modular/modshogunPYTHON_wrap.cxx' failed

make[2]: *** [src/interfaces/python_modular/modshogunPYTHON_wrap.cxx] Error 1

CMakeFiles/Makefile2:325: recipe for target 'src/interfaces/python_modular/CMakeFiles/_python_modular.dir/all' failed

I put the full log here: make dev error
I'll send you the dpkg -l output via mail as requested.

@mpi-pebert thnx. i'll look into the dpkg list but the
SGBase.i:311: Error: Unable to find 'swig_typemaps.i'

is very alarming because that's basically saying that you are missing a file that is in the repository :(
could you actually check whether the file exists in src/interfaces/python_modular/swig_typemaps.i

Ok, I have identified a problem - I started again from scratch and confirmed that swig_typemaps.i exists; cloned the repo and compiled again; this time, it worked. That was a bit unexpected, so I played around and found that I can reproduce the Unable to find 'swig_typemaps.i' error by doing a make clean and then trying to rebuild the whole thing (i.e., swig_typemaps.i is now indeed missing from the folder, I presume that is expected behavior?)
So, technically, to get a running Shogun w/ Python interface:

set up Conda environment as specified in gist
clone repo
build Shogun from source, set CMake paths as appropriate for environment

In case something needs to be changed

delete everything
clone repo again
build again

The error about the 5.0.0 release version is still unsolved.
In principle, this could be closed. But if you want to track down the source of the release version error, I am happy to run some more tests/provide some more info if you need.

mmm yeah the lapack/cblas problem with release 5.0.0 are actually known... i was just hoping that we got it right finally in current develop. the current develop is inches away to be the next 6.0.0 release so i'm closing this now.
but i'll open a bug regarding the make clean story. thnx


README should contain documentation on how to build recipe locally

Most of the build system seems to be exclusively documented in the github action itself currently:
https://github.com/emscripten-forge/recipes/blob/main/.github/workflows/build_recipes.yaml
It would be nice to provide a script similar to https://github.com/conda-forge/staged-recipes/blob/main/build-locally.py and document the step involved.

Seems like:
python builder.py  build explicit path/to/recipe

do the trick!

--emscripten-32 flag will be needed, ie:
python builder.py build explicit recipes/recipes_emscripten/libffi --emscripten-32

nothing provides requested emscripten_emscripten-32

do I need to build and install emscripten_emscripten-32 in the build environment first?

what os are you on? for linux you can just add emscripten-forge as channel to your .condarc.
For MacOS you need to build these https://github.com/emscripten-forge/recipes/tree/main/recipes/recipes on your machine.
Note that for these compiler packages you do not need the flag -emscripten-32.
Ie:
python builder.py build explicit recipes/recipe/emscripten_emscripte32

Once that's done you  need to add the conda-build folder of the env from where you built to your .condarc

For reference there are now some instructions for building locally: https://github.com/emscripten-forge/recipes?tab=readme-ov-file#local-builds
Although not sure if they work and/or are up-to-date: https://github.com/emscripten-forge/recipes/issues/811


Provide documentation for packages/changelog-github

Affected Packages
pkg:changelog-github
Problem
Lack of documentation for installation and usage
Proposed solution
Would love to understand how to use this package. We've been using Changesets in our repository, and would like to automate this usage if possible. Please let us know if you're able to update your documentation so that this package may be externally consumed.

This is how you can use it:
https://github.com/emotion-js/emotion/blob/e09fb216180063f1cad4fafe390e770e7398f811/.changeset/config.json#L3-L6
I agree though that documentation is lacking in this regard - all config options should be documented. They even kinda are in a form of JSON schema that is available here but it's rather hard to reach this when skimming through the documentation (as it's not even part of the actual docs).

Thank you for your fast response @Andarist.


No LICENCE

Could this project please be licensed? I am preferably looking for something similar to MIT / Apache 2 or even Public Domain

Yeah sure, I'll add a license tonight.  Don't forget though that the actual software inside (offlineimap, etc) each has it's own license.

Thanks, that would be great. Will be using a fork of this as a submodule in a helper script.
Interesting point, does something that aids the install of another piece of code need to respect the license of the code it installs (with respects to using a license compatible for the inclusion/use of that application). GNU GPL2 which OfflineIMAP is licensed under wouldn't be a disaster for me, I just have a preference for less restrictive licensing on small projects.

I have no idea about the interaction between licenses at that level.  If you find anything about it I'd be interested in what you find though!

See latest commit

Wonderful, thanks.


Bump eth-scan to 3.5.3

Bumps eth-scan to 3.5.3.
Fixes #8

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (cc6dd2d) compared to base (df22c24).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##            master        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines           77        77
  Branches        22        21    -1
=========================================
  Hits            77        77

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


VS is automatically hiding most of my code

Type: Bug
While editing my code,  '>' and 'V' characters spontaneously appear in the left margin that collapse my code, making it unreadable.
I never selected anything like this to happen.
When I attempt to un-collapse my code, the code beneath the uncollapsed section re-collapses.
I do not need these nonsense functions, and I do not need them coming on without my agency.
How can I turn this off?
VS Code version: Code 1.86.2 (903b1e9d8990623e3d7da1df3d33db3e42d80eda, 2024-02-13T19:41:37.860Z)
OS version: Linux x64 6.5.0-18-generic snap
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 7 5800H with Radeon Graphics (16 x 3552)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: disabled_off

Load (avg)
0, 0, 0

Memory (System)
30.76GB (28.48GB free)

Process Argv
--no-sandbox --force-user-env --unity-launch --crash-reporter-id f9e7df47-36f3-4bde-9432-6d503247ed4b

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
wayland

Extensions: none
A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
pydisww2:30959801
edj9j583:30969295
fegfb526:30961430
7j2b6412:30969109

/gifPlease
Please provide more details (starting from language) and samples of code that are collapsed unexpectedly.
Usually VS Code collapses imports at the top, but shouldn't do that "randomly"

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

You can turn off code folding in the settings, Settings id: editor.folding


IDVA6-1941: Updated to latest email-producer-java

Short description outlining key changes/additions
Updated to latest email-producer-java. Also eliminated all suppressions.
JIRA Ticket Number
IDVA6-1941
Type of change

[ ] Bug fix
[x] New feature
[ ] Breaking change
[x] Infrastructure change

Pull request checklist

[x] I have added unit tests for new code that I have added
[x] I have added/updated functional tests where appropriate
[x] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: Security warnings found!

CI: No security warnings found


Deneb BuilderBid.MarshalSSZ causes panic

When trying to use MarshalSSZ for deneb's BuilderBid type there is a panic while handling the *uint256.Int field.  If you modify it to match the capella SSZ method for value you can successfully Marshal the struct, but then unmarshaling from that output will fail with incorrect offsets in the BlobKZGCommitments field.
I've created some tests here that can be used to replicate the problem:  https://github.com/benhenryhunter/go-builder-client/pull/1

Should be fixed in #25 please reopen if not.  Thanks.

Awesome!
Is there an official release including this on the horizon?

A new release with these changes has been made.


Add a tiny cosmetic change to be more explicit

I'm reading kagome-dict to understand how double-array works. I found a tiny improvement so that the code is more explicit.

Nice catch! Thank you.


Use MS architectural principles link in DI topic

Fixes #8592

We have a few link destinations in the Architectural Principles topic that we can use here.
This isn't a UE pass ... just a link replacement PR.
We perform link updates on UE passes. It isn't feasible for us to perform a sweep of the entire repo for this type of update. I suggest we continue to do this work on a topic-by-topic basis.

Heh. Still linking to my words, but at least they're hosted on docs... :)


Why is Player UIViewContoller?

What is the reason why Player is not UIView? :grinning: :thought_balloon:

hey @entotsu thanks for the inquiry. i like to keep a lot of view logic thin and without smarts. if Player expanded on it's core feature set in the future and added more advanced features such as other decoding techniques or caching, then that code should more commonly be in a VC.

Great!!
Thank you.


Drafting

Allow users to pick cards, save them, and display the picked cards on screen

Redownloading yew and trunk were a much bigger headache this time for some reason. Finally got it to work by:

downloading openssl with these steps
setting a certain environment variable per this issue

Downloading the pre-compiled binaries and then setting the env var is probably enough though

Can make calls to backend API on a button click and display the results
Now to:

clean up the component code
add the picked cards to a new component that shows the decklist
probably need some CSS to fix layout for easier viewing

Component for individual cards is moved to its own file, slight cleanup of on-submit callback code

more cleanups to do (replacing strings???, minor error handling)
pts 2-3 from previous

Finished pts 1 & 3
Now for the final bit of functionality: adding picked cards to a new component showing the decklist

Picked cards are now added to deck list
Now to:

add "main/extra deck" choice differences
BONUS: fix CSS to make deck list extend w/o affecting draft area

spent too long trying to figure out a clever way of programmatically knowing which deck to draft for next (main vs extra), but eventually just settled on a static array
Leaving CSS and refactoring/cleanup to next task


Processed events with no matches produce a lot of missed images

@erikarenhill, curious to hear your thoughts on this as well.
I've been testing some of the recent code changes with untrained detectors. One thing I've noticed is that if SAVE_UNKNOWN is set to true, then a lot of images will be produced for each event. Each retry per detector could produce an image if a person is found. For example, if I'm running compreface and deepstack and the snapshot and latest retries are set to 10 each, then I could in theory have 40 images saved for just a single event.
Maybe just taking a couple of the best results from the unknown images instead of saving out every single one to display on the UI.

Maybe instead of having each detector pull the image, I write the image first and put that image through each detector. It could take a little longer, but would reduce the amount of images in general and Frigate API hits.

I went ahead and made the changes I mentioned above. Rather than each detector pulling images from Frigate on their own, the image is pulled then passed to each detector. This allows me to reduce the amount of images that are written. Hope this may help with the Frigate API issues too.

cool! I actually wrote something similiar about only saving the best matches in #32

I'll switch over to beta again and help to try it out

Awesome, thanks! One thing to note is the UI still displays the same image that was used per detector and if you delete one of the images that was used by another it'll delete both of them. I do want to clean this up still. Maybe displaying one "card" per image and then the results of each detector on that image.

@erikarenhill let me know what you think about this. Since the same image is now passed to the detectors during each processing loop, I think it makes sense to display all the detection results on the card. It removes all the duplicate looking images when it was previously displayed by detector. Probably need to add a better way to determine which bounding box belongs to which result.


boost159: fix build on aarch64-darwin

Motivation for this change
Fix build on aarch64-darwin (related: #136643).
The critical change is adding #include "output.h" to make.c so that the varargs function out_printf is not implicitly defined. I thought it best to backport an upstream fix than to include a local fix.
Requires #159990, so opening as a draft.
Getting a patch into boost-build was slightly interesting due to the structure here.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Ping on this? I've just hit it.

Ping on this? I've just hit it.

Please run nixpkgs-review pr 160000 and post the result or any build failures

Result of nixpkgs-review pr 160000 run on x86_64-linux 1

1 package marked as broken and skipped:

rapcad

6 packages failed to build:

amarok (amarok-kf5)
boost159 (boost15x)
cqrlog
mysql57
percona-xtrabackup_2_4
powerdns

@hjones2199 can you rerun?

nixpkgs-review is still running, but percona-xtrabackup_2_4 fails with:
[ 35%] Building C object storage/heap/CMakeFiles/heap_embedded.dir/hp_clear.c.o
In file included from /build/source/storage/innobase/include/ha_prototypes.h:32,
                 from /build/source/storage/innobase/api/api0api.cc:27:
/build/source/storage/innobase/include/univ.i:416:33: error: unnamed scoped enum is not allowed
  416 | #define byte                    unsigned char
      |                                 ^~~~~~~~
/build/source/storage/innobase/include/univ.i:416:33: error: expected identifier before 'unsigned'
In file included from /nix/store/1wbkdhc44n09f6mhdw7y0d8dlkck29m9-gcc-11.2.0/include/c++/11.2.0/pstl/glue_algorithm_defs.h:13,
                 from /nix/store/1wbkdhc44n09f6mhdw7y0d8dlkck29m9-gcc-11.2.0/include/c++/11.2.0/algorithm:74,
                 from /build/source/storage/innobase/include/ut0ut.ic:26,
                 from /build/source/storage/innobase/include/ut0ut.h:626,
                 from /build/source/storage/innobase/include/univ.i:589,
                 from /build/source/storage/innobase/include/ha_prototypes.h:32,
                 from /build/source/storage/innobase/api/api0api.cc:27:
/nix/store/1wbkdhc44n09f6mhdw7y0d8dlkck29m9-gcc-11.2.0/include/c++/11.2.0/functional:979:19: error: expected unqualified-id before ':' token
  979 |   enum class byte : unsigned char;
      |                   ^
[ 35%] Building CXX object storage/myisam/CMakeFiles/myisam_embedded.dir/ha_myisam.cc.o
[ 35%] Building C object storage/heap/CMakeFiles/heap.dir/hp_close.c.o
[ 35%] Building C object storage/heap/CMakeFiles/heap_embedded.dir/hp_close.c.o
In file included from /build/source/storage/innobase/include/ha_prototypes.h:32,
                 from /build/source/storage/innobase/api/api0misc.cc:27:
/build/source/storage/innobase/include/univ.i:416:33: error: unnamed scoped enum is not allowed
  416 | #define byte                    unsigned char
      |                                 ^~~~~~~~
/build/source/storage/innobase/include/univ.i:416:33: error: expected identifier before 'unsigned'
In file included from /nix/store/1wbkdhc44n09f6mhdw7y0d8dlkck29m9-gcc-11.2.0/include/c++/11.2.0/pstl/glue_algorithm_defs.h:13,
                 from /nix/store/1wbkdhc44n09f6mhdw7y0d8dlkck29m9-gcc-11.2.0/include/c++/11.2.0/algorithm:74,
                 from /build/source/storage/innobase/include/ut0ut.ic:26,
                 from /build/source/storage/innobase/include/ut0ut.h:626,
                 from /build/source/storage/innobase/include/univ.i:589,
                 from /build/source/storage/innobase/include/ha_prototypes.h:32,
                 from /build/source/storage/innobase/api/api0misc.cc:27:
/nix/store/1wbkdhc44n09f6mhdw7y0d8dlkck29m9-gcc-11.2.0/include/c++/11.2.0/functional:979:19: error: expected unqualified-id before ':' token
  979 |   enum class byte : unsigned char;
      |                   ^

Result of nixpkgs-review pr 160000 run on x86_64-linux 1

2 packages marked as broken and skipped:

gnome.gnome-documents
rapcad

1 package failed to build:

percona-xtrabackup_2_4

48 packages built:

amarok (amarok-kf5)
boost159 (boost15x)
cinnamon.cinnamon-common
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.nemo
cqrlog
dropbox-cli
eiciel
exempi
gnome-console
gnome-photos
gnome.eog
gnome.file-roller (pantheon.file-roller)
gnome.gnome-books
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-terminal
gnome.gnome-user-share
gnome.nautilus
gnome.nautilus-python
img2pdf (python39Packages.img2pdf)
mate.atril
mate.caja
mate.caja-dropbox
mate.caja-extensions
mate.caja-with-extensions
mate.engrampa
mate.eom
mate.mate-control-center
mate.mate-settings-daemon-wrapped
mate.mate-user-share
mate.python-caja
mysql57
nautilus-open-any-terminal
ocrmypdf (python39Packages.ocrmypdf)
pantheon.file-roller-contract
paperless-ngx
pdfarranger
phosh
python310Packages.img2pdf
python310Packages.ocrmypdf
python310Packages.pikepdf
python310Packages.python-xmp-toolkit
python39Packages.pikepdf
python39Packages.python-xmp-toolkit
tellico
tracker-miners

That's  preexisting failure
Ofborg built boost159 on darwin and it worked so ready to merge


Serde field attributes not written to TokenStream

Previously, I've piggybacked on serde_codegen_internals to get whether or not a field will be serialised, and if so, what it will be called.
With the new macros 1.1 stuff this doesn't work because the serde field attributes are being removed from the token stream output if my code runs after Serialize:
//No serde attributes for ElasticType
#[derive(Serialize, ElasticType)]
struct MyType { ... }

//All the serde attributes for ElasticType
#[derive(ElasticType, Serialize)]
struct MyType { ... }

Wouldn't this also affect attributes visible to Deserialize if you also derive Serialize first?

This is intentional - the crate would fail to compile if the attributes weren't stripped since they're unknown to the compiler. It's one of the known bugs on the tracking issue for macros 1.1: https://github.com/rust-lang/rust/issues/35900#issuecomment-252499766


Fix nil deref if no schema defined in serving GraphQL

will be fixed with #602
Was first confused, then annoyed by this issue and decided to fix it right away.


ci: use readonly only for simple types

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Code style update
[x] Build or CI related changes
[ ] Documentation content changes

What is the new behavior?
https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/array-type.md
Does this PR introduce a breaking change?

[ ] Yes
[x] No

@vladimirpotekhin please, review me)


BUG - fix ANSI colors in Windows

Following PR #392

Codecov Report

Merging #398 (eba49d5) into main (a804493) will decrease coverage by 0.03%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main     #398      +/-   ##
==========================================
- Coverage   55.38%   55.35%   -0.04%
==========================================
  Files          42       42
  Lines        2533     2538       +5
  Branches      476      477       +1
==========================================
+ Hits         1403     1405       +2
- Misses       1030     1032       +2
- Partials      100      101       +1

Actually, #310 was addressed thanks to PR #390. After that, I was able to run benchopt benchmarks without problems.


Update to Webpack 4

Awaiting for https://github.com/s-panferov/awesome-typescript-loader/issues/534 to resolve

Closed via f09dea1


Move builindg docker images to deploy process

Problem
Circleci builds are failing on PRs (#49) because environment variables are not available in PR fork builds and missing AWS_ACCOUNT_ID is causing build command to fail.
Solution
Move building docker image to deploy process which will only happen for master branch builds.

Merging this. Also disabled build on Travis-CI for this repo.


initial research on Weblate

Fixes #4197
Changes proposed in this pull request:

Outlines understanding of Weblate internals

@antoine2711 @wetneb before continuing I wish to know if I am on the right track with what is requested in this issue

@Babi-B : good thing you ask. The objective of the issue, in the way I understand it, to document the translation process (which you did well from what I see), but rather to starting using a context string.
So right now, for translations, we have the key and the text. The question is to know: can we add a context string to help us know where the translation will be used? If so, can an example show how-to?
I think your documentation is good, but it's not what needed. The issue is about addind context to translation strings. Can we do it?
I hope my explanations are clear. Don't hesitate to ask more questions.
Regards, Antoine

Hello @antoine2711. I have redone the report based on my understanding of your comment. Before I continue I wish to know if I am now on par with what is needed

I think you are overdocumenting. Here's then task from the issue:

[ ] Figure out what Weblate actually supports to contextualize messages
[ ] Investigate how we could use those features for the OpenRefine project

The need here is not documentation or to write a file. It's interesting, but our real aim is to incorporate that in OR, if we can… ;-) So this issue is about having an answer: can we — or not — add a context string with Weblate…
I read your documentation, and I can't make for sure if we have the possibility now…
What I see with a first glance at the software is that I can't put a context string in translated strings, but, in the glossary, Weblate seams to have such a feature…
Translated string edit pane:

Glossary string edit pane:

In the second picture, the glossary « QuickStatements » string, I have put some context data in the « Explanation » field ("Software for mass transfert to Wikidata."). It would be great if we could do the same with translated string… Can you find out? That’s the issue. Not writing documentation.
Understanding an issue is never easy, and it's often a lot of discussions. If I'm not clear, feel free to ask again. I'm here for you. You can also email me<EMAIL_ADDRESS>Regards, Antoine

Thanks @antoine2711 for the thorough review! I agree this should not go in the repository itself. @Babi-B feel free to take notes in the issue itself and open a PR when you are ready to make actual changes.


[Bug] Install script fails if user doesn't have write permissions to working directory

Consider the following sequence of commands:
cd /restricted-folder
bash /home/myself/install.sh

Now, when the script asks for the installation directory, I provide /home/myself/appsmith. This will then fail with the error:
./install.sh: line 747: nginx_app.conf: Permission denied

This is because we are writing to current working directory, and so unintentionally assuming write access to the current directory.

@sharat87 I tried following the testing steps but got stuck at step 2. I have been continuously getting  option to enter email id.
I have closed one terminal tab so you will not be able to see few steps here. Please check the screenshot for reference.


Configurable HTTP listen address

Adds the ability to configure Jenkins' HTTP listen address - sometimes listening on <IP_ADDRESS> isn't practical for a variety of reasons.
The semantics are a bit different between Debianoids and Red Hat-oids due to differing init configs/scripts from the upstream Jenkins packages, so separate tasks/ files are used per-platform. More than happy to move things around if that defies convention / doesn't work for some other reason. It felt like a misuse of vars due to the need for conditional regex parts, and using task conditionals with when: had a confusing output presentation.
I've tested this manually on all the Dockerfiles available in tests/ (except CentOS 7, which incidentally is the platform I use the role for - already tested in production) to ensure that the socket is bound to the specified address.

Can you rebase on master? I had to make a few changes in similar places to get 2.x setup steps to complete.

Any ETA on getting this merged and released?

So sorry for the delay. I'll get this rebased ASAP.

Sounds good to me. Thanks for the change. Listen address is a really useful addition to a great play book. Much appreciated.

I hate to have to ask again... I think some of the other changes to support more robust testing have created a conflict again :(

Rebased again :)

Please merge this PR 🙏


🐛  fix hasDateChanged

no issue

i don't know if this never worked or has worked and something changed in bookshelf
but this fixes: saving the content (no change for published_at) of a scheduled post within the 2minutes window
add beforeWrite option to hasDateChanged helper, see comment
add a test and fix some other small issues in the scheduler tests

This was tested with #8249.
I need to port this fix over to LTS as well.

👍 tested and works 😄 will merge once Travis is ✅


Improve performance of fallback hash function

This code only runs on systems where the node crypto module isn't available, which I don't even know when happens, but if it does run it's basically a denial of service attack on the GC and is problematically slow. The imperative version isn't as fancy but is about 25 times faster.

Test script
function old(data) {
  const chars = data.split("").map(str => str.charCodeAt(0));
  return `${chars.reduce((prev, curr) => ((prev << 5) + prev) + curr, 5381)}`;
}
function fast(data) {
  let acc = 5381;
  for (let i = 0; i < data.length; i++) {
    acc = ((acc << 5) + acc) + data.charCodeAt(i);
  }
  return acc.toString();
}

let largeStr = "";
for (let i = 0; i < 50000; i++) {
  largeStr += Math.random().toString();
}

console.log(`Test string has ${largeStr.length} characters`);
console.log(`Old: ${measure(old)}`);
console.log(`Fast: ${measure(fast)}`);

function measure(func) {
  let start = Date.now();
  for (let i = 0; i < 200; i++) {
    func(largeStr);
  }
  return Date.now() - start;
}

Output
Test string has 913574 characters
Old: 41260
Fast: 1582

Need to undo submodule changes


Validation dataloader error in training: error each element in list of batch should be of equal size

Hi, thanks for your fantastic work.
In training, I encountered a data loader error for validation after 1 epoch.
The error message is:
raise RuntimeError('each element in list of batch should be of equal size')
RuntimeError: each element in list of batch should be of equal size
batch: 3it [00:36, 12.01s/it]
val:   2%|▎         | 1/40 [00:11<07:21, 11.33s/it]
epoch=0/35, loss=16.124535306149284
I wonder if the cause of the error is the test data format or something else? Thanks in advance.

Hello, @MIXIAOXIN.
Thank you for your interest in our research.
Please uncomment the changed code (line 32 to 51) on K-Lane/baseline/datasets/registry.py.
We confirmed that changing the batch_size of validation to 1 fixes this error when it happens.
However, this is only a temporary solution.
We will find and correct the main cause of the error.
Thank you for the report.

Hi, thanks for your fantastic work. In training, I encountered a data loader error for validation after 1 epoch. The error message is: raise RuntimeError('each element in list of batch should be of equal size') RuntimeError: each element in list of batch should be of equal size batch: 3it [00:36, 12.01s/it] val: 2%|▎ | 1/40 [00:11<07:21, 11.33s/it] epoch=0/35, loss=16.124535306149284
I wonder if the cause of the error is the test data format or something else? Thanks in advance.
me too

Hello, @MIXIAOXIN
We modified the code in K-Lane/baseline/datasets/klane.py (line 123-126) and K-Lane/baseline/engine/runner.py (line 132, 472).
Since the description information is required in the conditional evaluation, we made the flag to deal with the problem.
Thank you.


Change android build to work without having to "hack" the jar

Made changes to include files in the android jar, also added the standalone android dstu, dstu2 and shaded jar to distribution, instead having an almost empty distribution in 1.2 release

Thanks for the submission! Merging now.


Build OpenCV as library

This is an amazing project, but I think it misses one very important opportunity: Allow people would to build they're OpenCV based project for the browser.
How can I build OpenCV as a library (no bindings), that I can then link to from my own libs?
The bindings/JS-expoting will be done from my own code.
I don't what to program my algorithms in JS, but instead export my C++ code with OpenCV linked internally. How can I do that?

Currently working on a modified make.py that does exactly that. It's quite messy but maybe you want to take a look?
If nothing else this is an example of how to build and link opencv during the compilation of your own project.

Cool, where do I find it?

I'll make it public as soon as I'm back at my desk.

https://github.com/biggzlar/opencv_js_compiler
Here you go. As mentioned, this is still in rough shape and there has been almost no testing. The example.cpp file contains some basic image manipulation methods (blur, threshold). Simply run:
python n_make.py example/example.cpp

Once compiled they can be called from within the javascript part in index.html (there is a helper function for html canvas interaction, taking one of the exposed functions as an argument). Just see if you can compile the example and use it in index. Let me know if you hit a bump!

What platform do you use this on? Linux? OSX?

Usually MacOs, just ran it on Ubuntu Linux and it worked with the latest OpenCV.

On Ubuntu Linux, did you have to build Emscripten from scratch?
Sent from my iPhone

On 2 Jun 2017, at 23:19, Lennart<EMAIL_ADDRESS>wrote:
Usually MacOs, just ran it on Ubuntu Linux and it worked with the latest OpenCV.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I didn't have it on there so I did build it from scratch.

Works now! Thanks!


Add test to compare array with arguments

Currently function equal depends on arguments order and not works if pass array as first param and callback as second.

Thanks! I've fixed this in a9528ff71561c303e722eb5ae0a3b9b2ecae4a3b.


Non standard ID's (Mongo)

Hi,
I've not got a lot of time to go through the source at the moment, I'm using Mongo along with a custom build of Limoncello on Laravel 5 but having problems integrating Mongo models where the id is '_id'.
Is there any way to customise which field should be used for ID within the Schema currently?
Thanks

You should be able to do the following, if I'm understanding you correctly.
    public function getId($author)
    {
        /** @var Author $author */
        return $author->id;
    }

example from here: https://github.com/neomerx/limoncello-shot/blob/master/app/Schemas/AuthorSchema.php

@Daveawb it should solve your problem. If not don't hesitate to ask.
@jcloutz thank you for your help

@jcloutz Perfect, thank you.
@neomerx  It does solve the problem, thanks.


Adds guidance about CLI for Microsoft 365 context

Adds guidance about CLI for Microsoft 365 context
Best to test it by serving the docs web.
Linked Issue
Closes #3896
This PR should be closed after #4485

Great job @Adam-it, we'll have a look at it soon.

thanks @martinlingstuyl for your comments 👍 I updated the docs. Let me know if it is better now ?

Hi @Adam-it, added some further comments. We're getting there

Thanks for your awesome comments🤩 I am working on my English writing skills 😉.
Please double check some of the comments I do not agree with and we could revisit them 😉 All others I will try to resolve ASAP (I would love that the guidance on our docs would catch up with the already released blog post 👍)

I'm no expert either :) I'll check your comments 👍

@martinlingstuyl I resolved your comments. I think it is (I hope) ready 🙂
Thank you for your awesome review 🤩 and your patience with my English skills 😉

@martinlingstuyl 👍✅

Merged manually, thanks!! 👏


KAFKA-9506: Fix JmxReporter corner cases and optimize

Fix some inconsistencies in the case where multiple JmxReporter objects
attempt to claim a single mbean.  Avoid holding a global lock when
performing JMX operations.
Finally, handle close better. Some calling code continues performing operations after close, so close needs to clean up the internal state

ok to test

retest this please


motorway_junction node not show when node is dragged

When a node with highway=motorway_junction is dragged to a nearest node, the motorway_junction icon is not shown at the merged node. The data at the junction is not affected but the visually it appears like the data is absent.
When the modified data is uploaded, motorway_junction icon appears, but while editing the icon is not shown.
To reproduce - follow the steps per 👇 gif at this motorway_junction - OpenStreetMap Link

cc @bhousel

Thanks for reporting @srividyacb - that does look like a rendering bug 🤔


Load an hljs grammar's sublanguages

Arcdown's JIT hljs language registration omits any given language's "sublanguage" grammars.
For example, highlight.js's included JS grammar can highlight html strings that begin with "html`" -- and that html can in turn highlight css and javascript. Grammars all the way down!
But since Arcdown detects the code-fence is just javascript it only loads the javascript language for hljs.
The actual request: find an opt-in way to register multiple languages for a block so that the parent grammar can highlight "sublanguages".

added and documented: https://github.com/architect/arcdown#sublanguages-object


Review and Sign off of Project Charter(May 5)

#Privacy of Information and Protection:
Your opinion is your personal information. Please do not include any information which identifies you or others in your response.
#Description:
Review sign off with project sponsor to make final changes and sign off.
#Outcome:

[ ]  Complete and Signed Project Charter
[ ]
[ ]

#To Complete this ticket:

[ ]
[ ]
[ ]

@lagaultier I have a few more updates to make as it is getting too lengthy.  Some of the content will come out and be put in separate document.

@lagaultier There is a version 4 of the document now.  Please hold off until IT reviews it.  I will let you know.  Thx.


v1.10.0-rc1: RPi Zero W docker not working

I flash v1.10.0-rc1 to RPi Zero W.
Then I run docker info, I got the error.
But v1.9.0 is working.

Thanks. See also #271


add documentation for typescript function runtime and update runtimes

The typescript runtime was recently added to openwhisk but we do not yet have docs for the runtime and it is missing from runtimes.json.
We need a doc like https://github.com/apache/incubator-openwhisk/blob/master/docs/actions-nodejs.md for typescript functions.
Adding the runtime to the runtime manifest can be done per https://github.com/apache/openwhisk/blob/master/docs/actions-new.md#the-runtimes-manifest.

@rabbah Please assign this to me. Much obliged 🙏

@rabbah The typescript runtime seems deprecated, I cannot find the typescript counterpart as https://github.com/apache/openwhisk-runtime-nodejs

Hey, would like to know if this issue is resolved?.
If not, I would like to take it up and contribute.
Thanks:)

Hey, would like to know if this issue is resolved?. If not, I would like to take it up and contribute. Thanks:)

Hi, I have a PR https://github.com/apache/openwhisk-runtime-nodejs/pull/227 that is still waiting for review. It's for JavaScript though. If a documentation were needed for TypeScript, I am not sure where to add that. As far as I know, there seems to be only one single Node runtime:  https://github.com/apache/openwhisk-runtime-nodejs

Typescript (ts) functions get compiled and run as node functions - there is some support to ship ts code as the action code and the action will compile it before it runs it (at init time). This will create a longer cold start. It's a nice feature for getting started but for production, the code should get compiled ahead of time rather than at runtime. As such we should close this issue as wont fix.


WIP: gRPC Server Middleware

Implements the server side of a client/server pair to allow
encrypted DNS over gRPC between two CoreDNS instances.
This is server-side only, not like the previous PR that was both client and server.
This still needs tests. Additionally, having a middleware that starts up another server that sits inside a server block that already defines a port seems weird:
.:53 {
  grpc :443 {
    tls cert.pem key.pem ca.pem
}

I would think it would look better like:
.:443 {
  grpc cert.pem key.pem ca.pem
}

And maybe this gRPC server should live alongside the existing udp/tcp server entries in the dnsserver.Server? Or replace them somehow?
Thoughts...?

GitHub won't do dependent changes. The client code is dependent on this PR because it needs the TLS stuff and it needs the gRPC protobuf files. So, to see the client only changes from this one, look here:
https://github.com/johnbelamaric/coredns/pull/1
John

Current coverage is 38.52% (diff: 5.31%)

Merging #463 into master will decrease coverage by 0.56%

@@             master       #463   diff @@
==========================================
  Files           113        115     +2
  Lines          5503       5597    +94
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2151       2156     +5
- Misses         3177       3266    +89
  Partials        175        175

Powered by Codecov. Last update c4ab98c...6468062

ah yes: tests...

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] WIP: gRPC Serve..." ]

johnbelamaric commented on this pull request.

g.server = grpclib.NewServer()

pb.RegisterDnsServiceServer(g.server, g)
go func() {

 g.server.Serve(ln)

}()
return nil
+}

+func (g *grpc) Query(ctx context.Context, in *pb.DnsPacket) (*pb.DnsPacket, error) {

msg := new(dns.Msg)
err := msg.Unpack(in.Msg)
if err != nil {

 return nil, err

}

//TODO(jbelamaric): this is junk

Not sure. I don't fully understand what it is used for. I could put the local address that gRPC is listening on and the remote address of the gRPC connection, I think. I think I can get that info from the context...but what's it for? can these be nil?

This is getting a bit icky - if we want this we need to make grpc a first class
citizen - i.e. CoreDNS should do the parsing and then just give the *dns.Msg
to a ServeDNS
I'm like how this actually seems to work, but it somewhat of a layer violation
to do this in a middleware.
I'm also somewhat serious about writing a draft about this as it is exactly DNS
over TLS, but using gRPC as a transport.

Doing this outside of a middleware might still be tricky, esp. since go DNS does not support grpc, so need to somehow know this came in via grpc and do the right thing when returns. Probably wrapping the ResponseWriter, with a gRPCResponseWriter that takes care of this.

I think it would have to take over the TCP connection rather than have the standard DNS over TCP connection running. That way we could use the port from the main server definition as described in the comment above: https://github.com/miekg/coredns/pull/463#issue-197007201
I don't know that we could multiplex that connection.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] WIP: gRPC Serve..." ]

I think it would have to take over the TCP connection rather than have the standard DNS over TCP connection running. That way we could use the port from the main server definition as described in the comment above: https://github.com/miekg/coredns/pull/463#issue-197007201

(can't reply to that commment). So I'll reply here:

This still needs tests. Additionally, having a middleware that starts up another server that sits inside a server block that already defines a port seems weird:

yes, that is weird

I don't know that we could multiplex that connection.

that's weird as well. You want something that tells what the protocol will be.
I.e. in the Corefile:
dns-tls://.:53 {...}

dns-grpc://.:53 { ...}

/Miek
--
Miek Gieben

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] WIP: gRPC Serve..." ]

I think it would have to take over the TCP connection rather than have the standard DNS over TCP connection running. That way we could use the port from the main server definition as described in the comment above: https://github.com/miekg/coredns/pull/463#issue-197007201
I don't know that we could multiplex that connection.

Might want to play with dns.DecorateReader to make grpc happen in a sane way on
the server side. No idea how viable that is though.
/Miek
--
Miek Gieben

That looks good. Can you have multiple protocols though? For example udp and tcp.

John Belamaric
(240) 383-6963
On Dec 22, 2016, at 12:05 PM, Miek Gieben <notifications<EMAIL_ADDRESS>wrote:
[ Quoting <notifications<EMAIL_ADDRESS>in "Re: [miekg/coredns] WIP: gRPC Serve..." ]

I think it would have to take over the TCP connection rather than have the standard DNS over TCP connection running. That way we could use the port from the main server definition as described in the comment above: https://github.com/miekg/coredns/pull/463#issue-197007201

(can't reply to that commment). So I'll reply here:

This still needs tests. Additionally, having a middleware that starts up another server that sits inside a server block that already defines a port seems weird:

yes, that is weird

I don't know that we could multiplex that connection.

that's weird as well. You want something that tells what the protocol will be.
I.e. in the Corefile:
dns-tls://.:53 {...}

dns-grpc://.:53 { ...}

/Miek
--
Miek Gieben

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/miekg/coredns/pull/463#issuecomment-268844004, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJB4s3cE-esd_qfeutZlAqSromyAcfEGks5rKq23gaJpZM4LTQ9W.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] WIP: gRPC Serve..." ]

That looks good. Can you have multiple protocols though? For example udp and tcp.

By default this always open a udp and tcp port.

Ok, I have updated this to set the remote address and the local address when we can. I have NOT updated it to try and make gRPC a first-class citizen. It is still nested inside another server block.
I also update https://github.com/johnbelamaric/coredns/pull/1 for your review comments.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] WIP: gRPC Serve..." ]

Ok, I have updated this to set the remote address and the local address when we can. I have NOT updated it to try and make gRPC a first-class citizen. It is still nested inside another server block.
I also update https://github.com/johnbelamaric/coredns/pull/1 for your review comments.

Ack. Note that I'm very hesitant to merge the server part of this. Client side
proxy is OK (but I'll want to review it again)

For the server side, what is the primary hesitation? That it should be a first-class citizen and not in a middleware?
FYI, the way it is written now the client PR is a patch on top of the server PR.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] WIP: gRPC Serve..." ]

For the server side, what is the primary hesitation? That it should be a first-class citizen and not in a middleware?

Yes, that gets super-weird...OTOH doing a first-class citizen may involve
patching Go DNS itself (and I'm totally nacking that as gRPC dns is nowhere near
to a standard).
So yes, I wish there was a cleaner way of implementing this...

FYI, the way it is written now the client PR is a patch on top of the server PR.

Ack. We should split this anyway IMHO - client side first and then discuss
serverside.
note: I'm well aware that this is nice to have (the serverside grpc stuff)

The only thing is that without the server the client is useless :)
Anyway, you are right this was intended as just a prototype for now, there is no pressing need for it.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] WIP: gRPC Serve..." ]

The only thing is that without the server the client is useless :)

Yes I know :-( To be fair, this could be the first example of an out-of-tree
middleware. We just lack an automated way to build a coredns binary without
fiddling with the source. See #453 - but I think I want something go generate
like, that will do all-the-things correctly during build time.
This could be a nice intermediate solution - which allows us to think/code what
we need be needed to make this a first class citizen.
(It does create a somewhat weird schism that the client side can speak grpc by
default and the serverside needs an external middleware).
/Miek
--
Miek Gieben

Closing this in favor of https://github.com/infobloxopen/coredns-grpc
Still plan to include client though (and move .proto and pb files to CoreDNS) once #481 merges.


I want to add a new branch ie ECE-AI  in the B.Tech section.

I want to add a new branch called ECE-AI and its resources according to 1st, 2nd, 3rd, and 4th yr to the B.Tech section.
I want to work on this please assign it to me

Please close this issue @anmode


MSID: 92091 Version: 2 DOI: 10.1101/2<IP_ADDRESS>9569

MSID: 92091
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>9569
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=92091
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>9569

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/92091v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>9569
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/92091/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c15) has a publication-type='other'. Is that correct?
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c16) has a publication-type='other'. Is that correct?
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c17) has a publication-type='other'. Is that correct?
warn: [title-lower-case] Content of  element is entirely in lower-case case: Is that correct? '10x library preparation'
warn: [title-lower-case] Content of  element is entirely in lower-case case: Is that correct? '10x initial processing'

Fixing the above


AzureSupport ?

Does it support Azure Storage ?

Hi @ubxtydev,
Thanks for taking an interest in the package.
We use Laravel's file storage module as the underlying filesystem abstraction, which in turn uses flysystem. You can setup azure by doing the following:
Steps

Extend Laravel's filesystem to provide an azure driver.
https://laravel.com/docs/8.x/filesystem#custom-filesystems
https://flysystem.thephpleague.com/v1/docs/adapter/azure

Setup a disk to use the azure driver in config/filesystems.php

Update the media.disk setting in config/media.php to the name of the azure disk you configured.

Hope this helps,
Jack


Bump version and add myself to list of authors

@j-fu just to make sure (my first version bump). I added myself to the authors and after merge I simply comment the commit with the Julia registrator?

Yes, "@JuliaRegistrator register" should be sufficient to trigger everything.
As for the invalidation action: I think we can remove it at the moment, as
others do, see
https://github.com/julia-actions/julia-invalidations/issues/19

Comments on pull requests will not trigger Registrator, as it is disabled. Please try commenting on a commit or issue.

Ah it seems the registrator got triggered here :) Good he know his place.

Oops, a false positive trigger :)

I started the registration on the merged commit.


[Paranoid Durability] Add metrics and logs for paranoid durability writes.

This PR adds more log statements and metrics to help in evaluating and debugging the performance of the paranoid durability write feature:

A single metric for when not enough successful writes can be made to satisfy the requirements (i.e. not enough local or remote successful writes).

Log messages with additional information to assist in debugging.

A metric for when there are not enough replicas in total to attempt a paranoid durability write (log messages already exist).

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

Project coverage is 18.65%. Comparing base (52ba813) to head (88120c9).
Report is 33 commits behind head on master.

Files
Patch %
Lines

...bry/router/ParanoidDurabilityOperationTracker.java
0.00%
13 Missing :warning:

.../github/ambry/router/NonBlockingRouterMetrics.java
0.00%
4 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (52ba813) and HEAD (88120c9). Click for more details.
HEAD has 2 uploads less than BASE
| Flag | BASE (52ba813) | HEAD (88120c9) |
|------|------|------|
||3|1|

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2811       +/-   ##
=============================================
- Coverage     64.24%   18.65%   -45.59%
+ Complexity    10398     2922     -7476
=============================================
  Files           840      842        +2
  Lines         71755    72296      +541
  Branches       8611     8700       +89
=============================================
- Hits          46099    13487    -32612
- Misses        23004    57572    +34568
+ Partials       2652     1237     -1415

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


MGDCTRS-1214 Fix K8s resource requests and limits for infrastructure components (Fleetshard-Sync, Camel Operator, Debezium Operator, Camel-K, Strimzi)

closes https://issues.redhat.com/browse/MGDCTRS-1214

@lburgazzoli  @rinaldodev mind checking if those changes make sense and are correctly integrated in our release pipeline?
Is the integration into cos-manifests automated or do I have to make changes there as well to get those changes into stage/production?

Are changes in ./etc/ not triggering any tests?

@rk3rn3r the kubernetes.yml files inside /etc files are actually copied from the ones inside the specific projects, like this one: https://github.com/bf2fc6cc711aee1a0c2a/cos-fleetshard/blob/main/cos-fleetshard-operator-camel/pom.xml#L180-L189
So it needs to be changed there.
Camel K and Strimzi looks good to me.

Thx for the clarification. But I feel like people don't know that. They highly diverge for example in the case of Camel Operator's kubernetes.yml

@rinaldodev thx! I've updated the poms accordingly.

@rk3rn3r you can try to build with ./mvnw clean install -DskipTests just to see if the files are all correctly generated.


TypeError: Cannot read property '1' of undefined

The first argument of leftPad can be a number and a number can have the value NaN. The following TypeError occurs when NaN is passed as the first argument. I expect to throw an error or give a default value when the first argument cannot be left padded.
file:///path/left-pad/index.js:34
  if (ch === ' ' && len < 10) return cache[len] + str;
                                          ^

TypeError: Cannot read property '1' of undefined
    at leftPad (file:///path/left-pad/index.js:34:43)

Could you submit a PR with the failing test?

@stevemao Please see #44

@stevemao Could you check the pull request and let me know what you think? Thank you.


ga4gh_benchmark server benchmarking

Here's a more substantive strawman for #46.
Open issues:

How many queries?
Which queries?

Just to be explicit: if you have the ga4gh-example-data (Wormtable) dataset installed, you can type 'ga4gh_benchmark' and get a number ~15 minutes later.
Running this verified for me that the memory bloat issue (#10) is still open.

+1

I like it @Naburimannu, we definitely want this. I wonder though, do we want to ship this as a first-class CLI program installed on every user's machine, or should it be a development tool (say, server_benchmark.py)? If it is to be a first-class program, then the input will need to be generalised and we'll need to provide a UI to set the queries and so on. If it's a dev script, then I'm happy for it to depend on example data living in a fixed location. What do you think?

Coverted to a standalone script with a header comment to tell you (briefly) what data we want where. Started parameterizing it for future plans.
Before the b2bd475 commit, it would take ~90 seconds to run on my desktop to time ~30 seconds inside Backend.searchVariants(), because the JSON parsing overhead is huge. Using a regexp instead feels fragile, but 3x speedup for a tool makes this a lot more feasible to consider running regularly, and this is explicitly the "dev script" branch of of @jeromekelleher's question so we might tolerate some fragility.

Looks great @Naburimannu. Other than a couple of tiny (optional) nits, I think it's perfect. I agree on the use of the regex - JSON parsing is a massive overhead, and just distracts from what we're really trying to measure.

Nits addressed and squashed; ready to review? (the CORS test was broken by a previous commit; filed https://github.com/ga4gh/server/issues/94 with my attempt at diagnosis)

+1, thanks @Naburimannu. @dcolligan, would you mind having a quick look over this and +1'ing if you're happy with it?
I'm looking forward to seeing memory profiling and the other great tools you've developed getting integrated into this in later PRs!

The only thing that is keeping me from +1'ing this is the file path of server_benchmark.py ... do we want it in the root dir?  I think we should keep a few things in the root dir as possible.  What about under scripts/ or perhaps creating a new dir dev_scripts/ ?

I have no objection to moving it to scripts/; it's where it is to mimic server_dev.py and client_dev.py.
I'll clean up the nits, and agree that the blank line shouldn't be there; will remove unless it's a PEP8 issue.

It's a lot easier to keep these scripts in the root, as you'll have to install the full ga4gh package if you want to run them otherwise (they import ga4gh stuff). The only simple way I found to do it (without installing) was to create a hard link from scripts/something.py to something.py in the root, and then  run python something.py. However, these hard links kept breaking when git changed files around, so I was never quite sure what version of the file I was working with. Unless someone's got a better workaround for the import problem, I'd much rather leave scripts that import ga4gh in the root.

@jeromekelleher hm, I hadn't considered that.
In that case, would it make sense to make it a part of the ga4gh package somewhere?  Hm, but even if we did that, we would still need some launching glue in the root, a la server_dev or client_dev.
Ok, I take it back... leaving it in the root is fine.

(Python does have a relative import syntax, but it only can be used within a package, not across them https://docs.python.org/2/tutorial/modules.html#intra-package-references )

Does anybody recognize this travis failure?
Searching for avro
Reading https://pypi.python.org/simple/avro/
No local packages or download links found for avro
error: Could not find suitable distribution for Requirement.parse('avro')
The command "python setup.py install" failed and exited with 1 during .

I can't just retry Travis jobs, I think because I don't have the Contributor bit.

Nope, haven't seen that one before. PyPI seems to be a weak link in general here, I've seen it refusing connections from Travis a few times.

I reran it there, and it works fine now. We need some bdist_wheels for our requirements we can download from somewhere reliable, this is going to get very tedious otherwise.

OK, I think this is ready to merge. @dcolligan, do you agree?

@Naburimannu , please squash your commits down to one commit and rebase on top of the current master before resubmitting the branch
After that, I'll merge it.  (or someone else can... here's my +1 after that: +1)

Rebased & squashed.
I notice that I was overly-literal when you suggested removing args, and didn't also remove the argument parser, but I'd like to go ahead and land this soonish; the next CL I'm planning in this area will either use or remove it.


npm install semantic-ui --save error

npm WARN excluding symbolic link tests/withsymlinks/.hidden -> ../shown/.hidden
npm WARN excluding symbolic link tests/withsymlinks/bar.txt -> ../shown/bar.txt
[11:51:10] Starting 'run setup'...
? Set-up Semantic UI Automatic (Use defaults locations and all components)
?
    We detected you are using NPM. Nice!

    Is this your project folder?
    /Users/dd/IdeaProjects/management/view

 (Use arrow keys)
  Yes
  No, let me specify

readline.js:991
            throw err;
            ^

TypeError: Cannot read property 'name' of undefined
    at Prompt.render (/Users/dd/IdeaProjects/management/view/node_modules/inquirer/lib/prompts/list.js:95:69)
    at Prompt.onSubmit (/Users/dd/IdeaProjects/management/view/node_modules/inquirer/lib/prompts/list.js:116:8)
    at AnonymousObserver.Rx.AnonymousObserver.AnonymousObserver.next (/Users/dd/IdeaProjects/management/view/node_modules/rx-lite/rx.lite.js:1535:12)
    at AnonymousObserver.Rx.internals.AbstractObserver.AbstractObserver.onNext (/Users/dd/IdeaProjects/management/view/node_modules/rx-lite/rx.lite.js:1469:31)
    at AnonymousObserver.tryCatcher (/Users/dd/IdeaProjects/management/view/node_modules/rx-lite/rx.lite.js:63:31)
    at AutoDetachObserverPrototype.next (/Users/dd/IdeaProjects/management/view/node_modules/rx-lite/rx.lite.js:5782:51)
    at AutoDetachObserver.Rx.internals.AbstractObserver.AbstractObserver.onNext (/Users/dd/IdeaProjects/management/view/node_modules/rx-lite/rx.lite.js:1469:31)
    at AnonymousObserver._onNext (/Users/dd/IdeaProjects/management/view/node_modules/rx-lite/rx.lite.js:4234:13)
    at AnonymousObserver.Rx.AnonymousObserver.AnonymousObserver.next (/Users/dd/IdeaProjects/management/view/node_modules/rx-lite/rx.lite.js:1535:12)
    at AnonymousObserver.Rx.internals.AbstractObserver.AbstractObserver.onNext (/Users/dd/IdeaProjects/management/view/node_modules/rx-lite/rx.lite.js:1469:31)
⸨               ░░░⸩ ⠦ install: info lifecycle<EMAIL_ADDRESS>Failed to exec install script

npm ERR! Darwin 16.3.0
npm ERR! argv "/usr/local/Cellar/node/7.4.0/bin/node" "/usr/local/bin/npm" "install" "semantic-ui" "--save"
npm ERR! node v7.4.0

Did you press the arrow key to select Yes or No? I think there is no option selected by default for some reason...

@Dreampie have you figured it out?

no, I download dist js file,maybe you can set default value to selected,thanks

@Dreampie, I have had the same problem, and found that pressing the arrow key to select Yes or No will make the error disappear, as @dominicjameskelly said.

@tarvos21 is right, the issue seems to happen when the user doesn't manually select an answer.
To fix the issue alone we could just add default: true, to https://github.com/Semantic-Org/Semantic-UI/blob/master/tasks/config/project/install.js#L291
But unfortunately, I was looking at the other choice questions as well and they all seem to not have a default option.

Same error here. First two times selecting via arrows did not cause any reactions, until eventually the installation process died by its own. On third try everything went smooth, I didn't make any changes in the meantime ... magic :smile: :star:

Has anyone checked and confirmed adding default value fixes the issue?

Hi, @jlukic, I myself encounter this problem when try to npm install semantic-ui --save in anempty directory, or any project directory, withou any tasks/ directory or *.json files.
When it asks for project direcotry:

The error will appear when I press 'Enter' like the previous step, but actually I need to use arrow keys to choose yes or no, but it's not so obvious for a really new user, who probably is used to directly just hit enter several times and everything is done. I think this may be a problem for really new users of Semantic-UI.

@jlukic sorry, no - I was wrong. It looks like the problem wasn't that we were not specifying a default choice but that the default option that we were specifying was wrong, see https://github.com/SBoudrias/Inquirer.js/issues/203.
The default used to be the project root, /home/ionut/PhpstormProjects/ui in my case, whereas the choice values are true and false. Just removing that default seems to have fixed it, as Inquirer always selects by default the first choice.
Made a PR with instructions on how to test it here: https://github.com/Semantic-Org/Semantic-UI/pull/5070.

Similar with npm init i like to hit Enter several times when init/install something or use the -y option, but when install semantic-ui, I hit Enter several times and I met the above mentioned error, google pointed me to this page, hope that the installation process could me more smooth. Thanks.

With "npm install semantic-ui --save", I had the same problem.
but once using -y option (npm install semantic-ui -y --save), the problem was solved.


Add a flag to disable ambient credential usage for cosign sign

Description
It'd be helpful to add a flag that prevents cosign sign from automatically picking up on ambient credentials. This would be useful for usage in environments that may have ambient credentials available, but the user might want to sign via the OAuth flow instead.
Right now, users need to manually determine which environment variables cosign is picking up on and unset/modify them when calling cosign sign to force it to use the OAuth flow.

sgtm!

IMO you should be able to force it both ways: disallow ambient credential usage, or require it (with the default remaining as it is today: use it if it's there).


How to know which Hugging Face Hub models can be used?

According to guide_models.md, many models in the Hugging Face Hub can be used by Semantra. But how to know which Hugging Face Hub models can be used?
How can we determine whether an error encountered when executing the "--transformer-model" command is due to the unavailability of the model or other factors such as computer configuration?

any pretrained Sentence Transformers model can be used

These are listed on https://www.sbert.net/docs/pretrained_models.html

any pretrained Sentence Transformers model can be used

These are listed on https://www.sbert.net/docs/pretrained_models.html
For instance, --transformer-model sentence-transformers/paraphrase-multilingual-mpnet-base-v2 works for me.

Yes, same question as in #53 , do others work?

You can use many HuggingFace transformers models — it's certainly not limited to Sentence Transformers at all. If you look at the models guide you will see some other HuggingFace models outside of Sentence Transformers (sgpt) that have proven to be effective. Anything branded for semantic similarity will likely work well with Semantra.
It's possible that almost any HuggingFace model will technically work, since they all have an inner layer that can be used for embeddings (and I coded it to be as flexible as possible), but these embeddings will not necessarily be any good for the purpose of semantic similarity. For instance, you can use gpt2 as a model, and the search results may even be relevant, but they might not be as effective compared to other models of similar sizes trained on the task of semantic similarity. If there is a HuggingFace model that you expect to work, and it has an error, please file an issue.


Tools section: add Contentful converter and reference Jekyll import command

Thank you!
Merged as ebc187b and 7857601 (master), and as 52f5ade and 458499c (v0.15.docs).


Add names for test failures

This PR adds the coordinate name to the test failure printout in assert_allclose() when cycling through the coordinates . Specifying the coordinate makes test failures much easier to diagnose just based on the message.

[ ] Closes #xxxx
[x] Tests added / passed
[x] Passes git diff upstream/master **/*py | flake8 --diff
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API

@shoyer I had to think a bit on your suggestion to figure out a way to make the name parameter work for the recursive cases, but now here's something to that effect. Also made me wonder how useful a test suite for testing test failures would be...