Bypass stdin in CLI using config file

It should be possible to set all parameters needed in a config file (e.g. json) and pass this config file to the CLI.

This feature will be included in the next release.


Load historical prior lake data into hydrocron

Depends on:
https://github.com/podaac/hydrocron/issues/193
May want to do prior to:
https://github.com/podaac/hydrocron/issues/194
Depending on timing, may not need historical if we choose to only process the new collection which will start reprocessing/forward in October.
Need to execute script which will run load data lambda for historical granules.

This is blocked by #210

10/15/24 - Ying created Run script and Tim used it to finish the bulk-loading of the historical granules for the SWOT_L2_HR_LakesSP_2.0 collection into the new Hydrocron SWOT prior lakes DynamoDB table per:
https://wiki.jpl.nasa.gov/display/PD/Hydrocron+v1.4.0+OPS+Services+Deployment+Procedure+-+2024-09-30


D10.2.3

On D10.2.3 on EditDirDblClick(
SelectDirectory (sDir, []); is not compatible with my Delphi
from embarcadero
FileCtrl.SelectDirectory(Dir, [sdAllowCreate, sdPerformCreate, sdPrompt],SELDIRHELP)

On D10.2.3 on EditDirDblClick( SelectDirectory (sDir, []); is not compatible with my Delphi from embarcadero FileCtrl.SelectDirectory(Dir, [sdAllowCreate, sdPerformCreate, sdPrompt],SELDIRHELP)

Add Uses
Vcl.FileCtrl.SelectDirectory see more  > http://docwiki.embarcadero.com/Libraries/Sydney/en/Vcl.FileCtrl.SelectDirectory

I think this no longer applies


Animation Controller for Dismissed Not work in iOS 9.3 & iOS 12

Thanks to the author for the great library, very easy to use, however
I can not run the dismissal effect on ios 9.3 and ios 12. Please check again.
Thank you.

Great. The funny thing is that the view hierarchy inspector looks fine:

I'll toss this to an iOS issue, filed radar 44790012.

Another reference to the issue: https://forums.developer.apple.com/thread/108568

Same issue

Check out the updates in c283745 to implement the fix.


How to navigate the same level pages ?

Like BottomNavigationView, Home & Deeplink are part of  the same level page, Deeplink should not have backstack. How to implement this feature?

I've posted a workaround over in https://github.com/googlecodelabs/android-navigation/issues/13#issuecomment-395256248

This was fixed in alpha087. You can read about the new AppBarConfiguration class in the documentation and it will be added to the codelab shortly via #40


walletconnect as web3 provider

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

Summary

I'm having issues with creating a web3 instance with wallet connect as a provider. Am I supposed to somehow use the connector.connection as a provider?
Would appreciate some guidelines on this, thank you 🙏

I found the answer. connector contains rpcUrl. So I just do:
const web3 = new Web3(connector.rpcUrl)
But I couldnt find any information about it, not even in walletconnect docs. Maybe would be good to include it somewhere (or maybe I just missed it).


Alternative solution

Why not just this code?

:+1:
Thank you, Andrea!

Please amend the commit message before merging it.
It should mention WICKET-6242.


Upstream the Vim plugin to nixpkgs

I would expect one of three things here:

Ideally, it's just there in pkgs.vimPlugins.himalaya or pkgs.vimPlugins.himalaya-vim.

The there's something in the Flake that exposes the Vim plugin (as an app or package?)

Or it's bundled with the main package and it could be added if needed:

# custom plugins
{
  himalaya = vimUtils.buildVimPluginFrom2Nix {
    pname = "himalaya";
    version = "VERSION";
    src = "${pkgs.himalaya}/share/vim"; # or some better location
  };
}

$ tree (nix-build --no-out-link '<nixpkgs>' -A pass)
/nix/store/yn90aa8viibbh439pxqnli5387yfn9ma-password-store-1.7.3
├── bin
│   ├── pass
│   └── passmenu
├── lib
│   └── password-store
│       └── extensions
└── share
    ├── bash-completion
    │   └── completions
    │       └── pass
    ├── emacs
    │   └── site-lisp
    │       └── password-store.el
    ├── fish
    │   └── vendor_completions.d
    │       └── pass.fish
    ├── man
    │   └── man1
    │       └── pass.1.gz
    └── zsh
        └── site-functions
            └── _pass

15 directories, 7 files
$ tree (nix-build --no-out-link '<nixpkgs>' -A himalaya)
/nix/store/sdfmnqaarc5ighhlahfbxkmd6bcl42yl-himalaya-0.3.0
├── bin
│   └── himalaya
└── share
    ├── bash-completion
    │   └── completions
    │       └── himalaya.bash
    ├── fish
    │   └── vendor_completions.d
    │       └── himalaya.fish
    └── zsh
        └── site-functions
            └── _himalaya

8 directories, 4 files

Here we can see that pass offers emacs stuff in `share/'. I'm a Nix newb and I'm having trouble figuring out the best way to pull this into my own config.

@srid what do you think?

This issue can be closed @toastal?

Yup.


Implement method specific task queues for MethodChannels

Description
If you have a project that is using MethodChannels and want to migrate some of the methods to execute on a background thread, not the platform thread, you have to split your usage between 2 different channels, one that handles on the platform thread and one on the background thread.  When developing federated plugins it isn't possible to change the channel communication scheme though without creating a breaking change to the platform_interface.
We have talked about removing channels from the platform_interface.  If we do that, this isn't a problem.  cc @stuartmorgan
Proposed Api
channel = new MethodChannel("com.example.foo", binaryMessenger);
taskQueue = binaryMessenger.makeBackgroundTaskQueue();
channel.setMethodTaskQueue("foo", taskQueue);

Implementation Details
If MethodChannel supplied its own TaskQueue to the BinaryMessenger, it would have the ability to dispatch to TaskQueues that it has stored locally to be associated with specific methods.  This only works though if the dispatch method is available to MethodChannel, currently it is private to the BinaryMessenger.

When developing federated plugins it isn't possible to change the channel communication scheme though without creating a breaking change to the platform_interface.

This is true only if the platform channels are part of the platform interface package; nothing about federation requires that, it's just something we did (and so is probably pretty common among 3P federated plugins as a result).

We have talked about removing channels from the platform_interface.

For 1P plugins, which are the only ones we control.
But I don't think we probably want to create an API that adds threading subtlety just to facilitate making threading changes in a model that we will probably not recommend once we write up more docs about federate plugins.

@gaaclarke Could you give a concrete example where we can see this happening?

@zanderso here @camsim99 ran into it: https://github.com/flutter/plugins/pull/4502#discussion_r751684525


How to Register Custom Scalars?

Is it possible to register custom scalar values? I would like to register ZoneId as a string scalar and deserialize using ZoneId.of(...) method. It's not included in mapped scalars here: https://github.com/leangen/graphql-spqr/blob/master/src/main/java/io/leangen/graphql/util/Scalars.java

Sorry, forgot to search first.  Looks like solution is here: https://github.com/leangen/graphql-spqr/issues/228


initial version of the adapter

Initial version of the google analytics adapter.
I still need to set up travis but the code is all there and the tests have almost 100% coverage when run locally.

@mstemm @davidvgalbraith Please take a look at the code.
@dmehra @rlgomes Please take a look at the README and travis setup. Also try it for yourself :)

I'd still like to go through the install process myself but ran into a problem I'll work out with demmer tomorrow. I think somebody else with more familiarity with optimizers should look at the optimizer, but the rest looked good.
Also I found the name 'ga' confusing--I was wondering what the general availability adapter was. I know 'googleanalytics' is too long. How about 'ganalytics'?

The problem I ran into was that I didn't have permissions to add the service account to the google analytics account. Once @demmer gave me permissions, the rest worked.

Agree with @mstemm that read ga sounds like "goo goo ga ga". I'd do the full read google_analytics, as typing is less of a burden than we sometimes make it out to be.

I'm having trouble with -from/-to/-last when doing read ga. For example, read ga -last :52 weeks: | reduce -every :day: gives me data for 2015-03-24 to 2015-04-27, and read ga -last :year: | reduce -every :week: gave me data for just one day of 2015-01-08.

@dmehra @davidvgalbraith @mstemm I pushed a bunch of commits that include fixes for the time bounds, adding support for paging, and renaming the adapter to googleanalytics. Please look again but then I'll publish.

+1


replace React.PropTypes with prop-types

React.PropTypes has moved into a different package since React v15.5. Please use the prop-types library instead.

When can we expect this merge?

@JackPu help us, please)

@JackPu help us, please)


Could not request results from Google Speech API - audio file processing

Hi guys,
I'm trying some things and I want to compare and analyze speech recognition services performance and their effectiveness. I'm trying to compare precision between English and Serbian. It works fine when I use my microphone as source, but when I try using wav file as source, I'm getting Could not request results from Google Speech API.... Bad Gateway. I read article about limits of Google Service - my wav file props:
15s length, less than 100 words surely. Code is fine - it is example from library - only replaced some details - language, chunk size... nothing special.
I read in source code sample rate characteristics: it is mentioned about 8kHz - 16kHz. I downloaded my file from YT - convert it and trim it. My sample rate is 48kHz and Channel is Mono. I think there might be problem with sampling and frequency characteristics. Any hint?

@ NenadPantelic - Take a look at their API limits: https://cloud.google.com/speech/limits
I have a small .WAV file, length is 1 minute and I keep getting

Could not request results from Google Speech Recognition service; recognition connection failed: [Errno 32] Broken pipe

The script I was using has the Google Speech Recognition code in it.

Hi @NenadPantelic,
If you can post a sample audio file, I'd be happy to take a look at what's going wrong. The sample rate will be automatically converted if it's outside of the accepted range, so that shouldn't be an issue.

Hi @Uberi,
sorry for my late response, I had some things to do. Well, I solved the problem, I cannot actually remeber how :) - I think I used sox tool in Linux, instead of the ffmpeg which I used at first
and with which I made the first file that made problems.
Thank you for your help and interest,
Nenad

Great to hear!


no main manifest attribute, in AES_Killer.jar

Sir,
I have error massage as following: no main manifest attribute, in AES_Killer.jar. Please advice....

Hi @ashraful1980 ,
you need to use this as Burp Plugin because burp plugins can't run standalone. Please the step as shown in the GIF

OR can refer to this section https://github.com/Ebryx/AES-Killer#how-to-install


feat: Add support for 'model/step' MIME type

Was missing from this allowlist. Does this require additional work?

Good catch, ty for adding! no additional work, we'll get this in for next release 🙏


Feature: Add new game "Memory Game"

🎮 New Game: Memory Card Game 🃏
Overview
This pull request adds a new Memory Card Game to the Awesome Open Source Python Games and Tools repository. The game provides a fun and interactive way for beginners to practice Python programming skills.
Game Features

4x4 grid memory matching game
Emoji-based card symbols
Move and pair tracking
Command-line interface
Simple and intuitive gameplay

Implemented Functionality

Dynamic board generation
Card revealing and matching mechanics
Input validation
Clear screen for better user experience
Performance tracking (moves and pairs found)

Screenshots

Checklist

[x] Implemented core game mechanics
[x] Added README documentation
[x] Ensured cross-platform compatibility

Addresses #2

Great work! Thank you for your contribution.


documentation error: changed from nodechat.docpad to canvas.docpad

hey,
It looks good. It is running find but not able to run the chat part :)
in console it is always printing asd .. I will check.

Dam you're quick!!! The chat part isn't done yet :-)
Will be done hopefully tomorrow :-)

just saw your commit on my stream.. thought to look at it.. I am very much interested in the technology u used.. I want to hack on it little bit :) let me know, if I can do some small thing here...

Updated and documented a bit. Give it a go now :-)


request: add a context menu to the deployed canisters to obtain the dfx canister log

Sometimes there is crucial information in the log which the user wants to see. One cannot use the dfx canister log command, as there is no controller identity around.
CAVEAT: the log may contain garbled unicode characters.

May be better to do this in Candid UI, so that it supports canisters outside of playground as well.

Done in #260


SDK is deprecated. Docs are misleading.

See title.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ede7d8cc-3e4e-8901-62c6-795eabd9cc82
Version Independent ID: 15342293-48a0-9aa2-7b00-2f3ccb55f44e
Content: Engage customers with the Microsoft Store Services SDK - Windows UWP applications
Content Source: windows-apps-src/monetize/microsoft-store-services-sdk.md
Product: uwp
Technology: publish
GitHub Login: @hickeys
Microsoft Alias: hickeys

Seems to be working. Please see: https://github.com/MicrosoftDocs/windows-uwp/issues/2012#issuecomment-543833348


Use .container instead of .container-fluid for Last Updated date

On https://juxt.pro/edge/docs/lifecycle.html the Last Updated section is wrapped in a .container-fluid div. I think if that was .container then it would line up with the rest of the documentation?

I've been trying to decide where to put this. I did briefly put it into the footer. You're right that a .container would be a quick win for better alignment.
Changed, Thanks!


zshell path with Gems quick fix

/home/sanoy/.gem/ruby/2.2.0/bin
I need help setting up my path in for Gems with shell.
I have the line below in my .zshrc file but no cigar.
export PATH=$PATH:/home/sanoy/.gem/ruby/2.2.0/bin

You need to put that line at the top of the zshrc file

Did that work?


Cannot build in CLion

Hi,
I was trying to build the code in CLion. However, it gives me the following errors:
/opt/clion-2018.3.2/bin/cmake/linux/bin/cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo -G "CodeBlocks - Unix Makefiles" /home/wd-ai/catkin_ws_debug/src/cartographer_superbuild
-- Build type: RelWithDebInfo
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   iostreams
--   regex
-- Found installed version of Eigen: /usr/local/share/eigen3/cmake
-- Found required Ceres dependency: Eigen version 3.3.5 in /usr/local/include/eigen3
-- Found required Ceres dependency: glog
-- Found required Ceres dependency: gflags
-- Found Ceres version: 2.0.0 installed in: /usr/local with components: [EigenSparse, SparseLinearAlgebraLibrary, LAPACK, SuiteSparse, CXSparse, SchurSpecializations]
-- Using CATKIN_DEVEL_PREFIX: /home/wd-ai/catkin_ws_debug/src/cartographer_superbuild/cmake-build-relwithdebinfo/devel
-- Using CMAKE_PREFIX_PATH: /home/wd-ai/catkin_ws/devel;/opt/ros/kinetic
-- This workspace overlays: /home/wd-ai/catkin_ws/devel;/opt/ros/kinetic
-- Using PYTHON_EXECUTABLE: /usr/bin/python
-- Using Debian Python package layout
-- Using empy: /usr/bin/empy
-- Using CATKIN_ENABLE_TESTING: ON
-- Call enable_testing()
-- Using CATKIN_TEST_RESULTS_DIR: /home/wd-ai/catkin_ws_debug/src/cartographer_superbuild/cmake-build-relwithdebinfo/test_results
-- Using Python nosetests: /usr/bin/nosetests-2.7
-- catkin 0.7.14
-- Using these message generators: gencpp;geneus;genlisp;gennodejs;genpy
-- cartographer_ros_msgs: 15 messages, 6 services
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   system
--   iostreams
--   regex
-- Eigen found (include: /usr/local/include/eigen3, version: 3.3.5)
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   system
--   filesystem
--   thread
--   date_time
--   iostreams
--   serialization
--   chrono
--   atomic
--   regex
-- looking for PCL_COMMON
-- Build type: RelWithDebInfo
-- Using CATKIN_DEVEL_PREFIX: /home/wd-ai/catkin_ws_debug/src/cartographer_superbuild/cmake-build-relwithdebinfo/devel
-- Using CMAKE_PREFIX_PATH: /home/wd-ai/catkin_ws/devel;/opt/ros/kinetic
-- This workspace overlays: /home/wd-ai/catkin_ws/devel;/opt/ros/kinetic
-- Using PYTHON_EXECUTABLE: /usr/bin/python
-- Using Debian Python package layout
-- Using empy: /usr/bin/empy
-- Using CATKIN_ENABLE_TESTING: ON
-- Call enable_testing()
-- Using CATKIN_TEST_RESULTS_DIR: /home/wd-ai/catkin_ws_debug/src/cartographer_superbuild/cmake-build-relwithdebinfo/test_results
-- Using Python nosetests: /usr/bin/nosetests-2.7
-- catkin 0.7.14
-- Using these message generators: gencpp;geneus;genlisp;gennodejs;genpy
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   system
--   iostreams
--   regex
-- Build type: RelWithDebInfo
-- Using CATKIN_DEVEL_PREFIX: /home/wd-ai/catkin_ws_debug/src/cartographer_superbuild/cmake-build-relwithdebinfo/devel
-- Using CMAKE_PREFIX_PATH: /home/wd-ai/catkin_ws/devel;/opt/ros/kinetic
-- This workspace overlays: /home/wd-ai/catkin_ws/devel;/opt/ros/kinetic
-- Using PYTHON_EXECUTABLE: /usr/bin/python
-- Using Debian Python package layout
-- Using empy: /usr/bin/empy
-- Using CATKIN_ENABLE_TESTING: ON
-- Call enable_testing()
-- Using CATKIN_TEST_RESULTS_DIR: /home/wd-ai/catkin_ws_debug/src/cartographer_superbuild/cmake-build-relwithdebinfo/test_results
-- Using Python nosetests: /usr/bin/nosetests-2.7
-- catkin 0.7.14
CMake Error at cmake-build-relwithdebinfo/devel/share/cartographer_ros/cmake/cartographer_rosConfig.cmake:148 (message):
  Project 'cartographer_rviz' tried to find library '-lpthread'.  The library
  is neither a target nor built/installed properly.  Did you compile project
  'cartographer_ros'? Did you find_package() it before the subdirectory
  containing its code is included?
Call Stack (most recent call first):
  /opt/ros/kinetic/share/catkin/cmake/catkinConfig.cmake:76 (find_package)
  cartographer_ros/cartographer_rviz/CMakeLists.txt:39 (find_package)

-- Configuring incomplete, errors occurred!
See also "/home/wd-ai/catkin_ws_debug/src/cartographer_superbuild/cmake-build-relwithdebinfo/CMakeFiles/CMakeOutput.log".
See also "/home/wd-ai/catkin_ws_debug/src/cartographer_superbuild/cmake-build-relwithdebinfo/CMakeFiles/CMakeError.log".

[Finished]

The build works in the command line but not in CLion. Can you help me?
Thank you!

Much thanks! I've met the same problem and setting CLion to use a lower version system CMake works for me.

Much thanks! I've met the same problem and setting CLion to use a lower version system CMake works for me. I'm using ubuntu 16.04, and the system CMake is 3.5.1, whose executable file is located in /usr/bin.
The detailed operation for reference is:
in CLion, File --- Settings --- Build, Execution, Deployment --- Toolchains, firstly push the "+" button to create a new toolchain setting called "System", alongside the origin "Default". And in this new "System" toolchain, set the "CMake:" as "Custom CMake Executable" and select the cmake binary from /usr/bin.
The make and following C/C++ compilers should be automatically detected.
Save this new toolchain, and come to File --- Settings --- Build, Execution, Deployment ---CMake.
After the Build type (which should be RelWithDebInfo as instructed), select the toolchain as newly created "System".
Apply the above operations and the whole system works fine.
Besides, I've also noticed that, if CLion is installed in /opt, this project may not be properly compiled because of the permission issue. But after reinstalling CLion to /home and configure the CMake toolchain to a lower version, everything works fine.

I fixed the problem above, but i got another ( i have same environment with yours):
CMake Error in cartographer/CMakeLists.txt:
  Specified unknown feature "cxx_std_11" for target
  "cartographer.transform.transform_interpolation_buffer_test".

CMake Error in cartographer/CMakeLists.txt:
  Specified unknown feature "cxx_std_11" for target
  "cartographer.transform.timestamped_transform_test".
....

that means i have too older version of cmake?

It‘s a bug with CMake 3.13 bundled with CLion and Catkin, see https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=915148.
Try setting CLion to use the lower version system CMake, or apply this to your Catkin install: https://github.com/ros/catkin/pull/975/files

Hi, thanks for the link you provided. May I ask how to apply those changes to my catkin install? I have already had catkin installed.

Dear
I was trying to run cmake ../ to generate build, I get this problem.
CMake Error in cartographer/CMakeLists.txt:
Target "cartographer" INTERFACE_INCLUDE_DIRECTORIES property contains path:
"/home/zpw/cartographer_superbuild/cartographer/"

which is prefixed in the source directory.
can you find some solution?

Dear
I was trying to run cmake ../ to generate build, I get this problem.
CMake Error in cartographer/CMakeLists.txt:
Target "cartographer" INTERFACE_INCLUDE_DIRECTORIES property contains path:
"/home/zpw/cartographer_superbuild/cartographer/"

which is prefixed in the source directory.
can you find some solution?

remove the following statement
#install(
EXPORT CartographerExport
DESTINATION share/cartographer/cmake/
FILE CartographerTargets.cmake
#)


Language bindings

Ordered by priority:

[x] C++ (natively supported)
[x] Go
[x] Rust
[ ] Python
[x] Node.js
[x] Swift
[ ] Java (not yet mature)
[ ] Haskell (not yet mature)

Thank you for creating this wonderful library. I wanted to play around with Shamir secret sharing in Nim so I made a very simple wrapper.

Awesome! Would you like for it to be added to the README.md file?

Awesome! Would you like for it to be added to the README.md file?

Yes, that would be great.

I've put together a quick NIF binding for Erlang/Elixir at https://github.com/arekinath/esss in case it's interesting. NIF bindings require that the computation will complete "quickly" (in practice, approximately sub-10ms), but it seems to work fine for this. Happy for it to be added to the README.md.

@arekinath Thank you!

Concerning Python, see https://gist.github.com/Huge/d8a3057a5523c0413215e8bd8aa088da showing the same functionality with what I believe is a default package, pycryptodome.


pinvh deprecated in scikit-learn 0.19 removed in 0.21

utils.extmath.pinvh was deprecated in scikit-learn version 0.19 and removed in version 0.21. The following error therefore occurs when using scikit-lean >= 0.21:
~\AppData\Local\Continuum\anaconda3\envs\chemo2\lib\site-packages\skbayes\rvm_ard_models\fast_rvm.py in <module>
      5 from sklearn.utils import check_X_y,check_array,as_float_array
      6 from sklearn.utils.multiclass import check_classification_targets
----> 7 from sklearn.utils.extmath import pinvh,log_logistic,safe_sparse_dot
      8 from sklearn.metrics.pairwise import pairwise_kernels
      9 from sklearn.utils.validation import check_is_fitted

ImportError: cannot import name 'pinvh'

Sorry, I was not actively working on this package for some time, so it needs work. I will have a look at this on the weekend, should be able to fix that.


feat: support allocating IP with index from node chunk

This feature enables us to allocate the IP that is offset by index from the start of the range allocated for a particular node using CNI args. It is useful when a particular workload that runs as DaemonSet wants to allocate always the same IP, even if it's recreated.

coverage: 70.337% (+0.2%) from 70.122%
when pulling 35250f066253b8a51a18e88b4980befac0dc3c7f on vasrem:feature/support-request-for-specific-ip-index-in-chunk
into 6649bfc810caa75889b7c4592cd90e67f64aaa78 on Mellanox:main.


vNext - Seg Canvas Fill + Outline Rendering, Seg Demo + Store Module Refactor Wombo Combo

Segmentations can now be rendered as just an outline, just fill, or both.
The segmentation renderer no longer creates image bitmaps using createImageBitmap. Instead only the regions occupied by a label are rendered to the canvas using canvas draw methods. This is huge speed up for platforms like Safari with poor createImageBitmap support. There are still room for optimisations in the future.
Can toggle visibility of segments through the Labelmap3D's visibileSegments property, which is a boolean array indexed by segmentIndex, where true is visible and false is hidden. If an element is undefined for a segment, it defaults to visible.
More configuration options for segmentations:

const configuration = {
  renderOutline: true,
  renderFill: true,
  renderInactiveLabelmaps: true,
  radius: 10,
  minRadius: 1,
  maxRadius: 50,
  segmentsPerLabelmap: 65535, // Max is 65535 due to using 16-bit Unsigned ints.
  fillAlpha: 0.2,
  fillAlphaInactive: 0.1,
  outlineAlpha: 0.7,
  outlineAlphaInactive: 0.35,
  outlineWidth: 1,
};

The state object of a cornerstoneTools module has been split into two objects to have a seperation of concerns:
state: internal, only modified by cornerstoneTools and cornerstoneTools plugins.
configuration: Configuration settings intended to be modified at run time by the host application.

The live seg demo demonstrates most of these configurables.
TLDR:

Just fixing tests.


Re: 'Support'

Problem with 'Support' (https://tdt-documentation.london.cloudapps.digital//support/)
This page mentions the CDDO tech writing team - not sure about the current status of this.

Let's change 'Central Digital and Data Office (CDDO) technical writing team' to 'tech docs template working group'

Raised a PR to fix this https://github.com/alphagov/tdt-documentation/pull/193, sorry it took so long to get around to.


Installation Guidance

Installation Guidance Missing
No guidance on whether installation requires a reboot, or restart of services. No guidance on silent installation.
General
Trying to determine how to deploy this software using a configuration management tool, if there are command line options and whether restart of server/services is required.
Please provide a code sample for your issue if it is relevant, either inline, link to a gist (or similar) or add it as a zipped attachment.
For some issues, you will get a quicker and better response if you file it at a more specific .NET repo. For example, if the problem is with ASP.NET Core, you are encouraged to use the aspnet/home repo.

Reboot should only be needed if the installer is replacing an in-use file, in which case you should get a prompt. There is a /quiet flag you can use to avoid displaying any prompts.
CC @dagood

Heads up that if you try to update the .NET Core Runtime, it will shut down FDE applications locking the old runtime without prompting. For FDD apps, it will raise a prompt due to the lock on dotnet.exe, but shut them down anyway due to the lock on the old runtime. More info at https://github.com/dotnet/core-setup/issues/8546.
The best way to handle it is to shut down running apps yourself, then run the update. It then won't require a reboot and won't unexpectedly shut anything down.

For initial install, a reboot is not required.
The install does add to PATH, so apps that are already running won't be able to find dotnet.exe using their old value of PATH. This is why you need to restart command lines before they can use dotnet commands. I don't think you will run into this in a deployment scenario.

You can run the exe with /? to see the full list of options (including /quiet).

@leecow, @MichaelSimons, @dleeapho, a doc does seem useful for update scenarios on service hosts. I see https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/iis/?view=aspnetcore-3.0#install-the-net-core-hosting-bundle but it doesn't touch on details like this.

The install does add to PATH, so apps that are already running won't be able to find dotnet.exe using their old value of PATH. This is why you need to restart command lines before they can use dotnet commands. I don't think you will run into this in a deployment scenario.

Could one potential issue here be the Windows Task Scheduler? I am not aware of a "simple" way to restart that and it would require the PATH variable to schedule any dotnet commands after initial install.

Could one potential issue here be the Windows Task Scheduler? I am not aware of a "simple" way to restart that

Apparently terminating Taskeng.exe will make it refresh the env vars it uses next time a scheduled task runs: https://superuser.com/a/527727. (Or even simpler, reboot the machine. 😄) A scenario where this is a problem isn't clear to me though, do you have one in mind?

I recall in the past after initially installing net core on a machine that never had it, we didn't reboot the system as it wasn't required. Shortly after, we setup a scheduled task to run using dotnet ... and it failed. It took us a while to figure out that it had the environment variables cached out. We ended up fully qualifying the path to dotnet until we were able to reboot the system, at which point we switched it back to dotnet...
I wouldn't call this a "problem", but if any installation guidance documentation would be created, I figured I would mention this as a possible candidate to include in a Windows specific section.

Ah, yeah, that makes sense. Thanks for the detail. (Removed my "I don't think" note from the comment.)


Code related updates for Observability packs to quickstarts rename

Description
Various code updates for the rename to quickstarts.
Acceptance Criteria

[ ] Notify the Partner Team that we'll be changing the repo name and structure so they aware. - @jpvajda
[ ] This repository is renamed to newrelic-quickstarts
[x] The /packs folder is renamed to /quickstarts
[x] @jbeveland27  is updated on repository rename and folder rename, we could also submit a PR!
[x] Workflows and related files are updated, including those under the /utils directory
[x] Add a blurb to the top of the readme to let contributors know about the rename

at the end of this, we may want to confirm things still work in the io site if this isn't verified a long the way. we can still view packs, confirm that they link to the correct place in github, etc.

I did 2 separate PR's for this, the first is our code, documentation and filenames. The second is changes to the importer which will need to be tested apres merge

#283
#284
#287

@rudouglas The newrelic-cli is now decoupled from o11y packs, so we're good to move forward with renaming the repo.

Notify the Partner Team that we'll be changing the repo name and structure so they aware.
@jpvajda One minor thing to be aware of (in case you weren't already) - once the repo is renamed, I believe existing forks stop working (i.e. they can't PR back to the original repo). Those partners will need to re-fork. So if there's any in-progress work, they should just be aware that they might need to copy it over to a new fork.


JMS support

We need to detect when the JMS connection is closed an reconnect automatically.
    @Override
    public boolean started() {
        return true;
    }

One work around for this (at least with ActiveMQ) is to use the failover support.
            /** Use the failover support by default. */
            final JmsServiceBuilder jmsServiceBuilder = getJmsServiceBuilder()
                    .setProviderURLPattern("failover:tcp://#host#:#port#");

I made this the default.
http://activemq.apache.org/how-can-i-support-auto-reconnection.html


Small fixes

From Jachym

I noticed a typo at:

http://standard.open-contracting.org/1.1-dev/en/extensions/lots/
lotGroups -	Tender - ere the buyer reserves the right to combine lots, or wishes to specify the total value for a group of lots, a lot group is used to capture this information.	array

Also, is it intentional that in the table, the column which contains "sections" is labeled "definition"?

You also use relatedLot, not relatedLots quite frequently on the page.

Updates to make

Seems closed in referenced commit. The column headers for those tables may be confusing, but they are shared across all extensions. Can open an issue on standard if we want to change them.


Added bootstrap theme and logo.

The index.html file will override the existing express/jade setup.
Will look into splitting up the index.html file more in the near future. Might also comment out or remove the "Links" on the navbar if we don't use them soon.

lgtm


Add support for ngrok in the build pipeline

gulp ngrok-serve

We can also design it to update messaging endpoint of bot channel registration via. gulp process and auto side loading of teams package in teams.
With current implementation are the following steps supported:

run ngrok
get url of ngrok tunnel
extract host name from url
set host name to process.env.HOSTNAME
generate manifest file
------> FEATURE: as next we can side load it in teams - we should design the teams access)
------> FEATURE: register the package in a teams channel
serve the express application
FEATURE: Change the bot channel registration  in azure via. Azure AD App

Then we have no manual processes for debugging the teams application.

Included in version 2.7.1 (7c45f921ef44af81a909295724a4dd387f4bc1e1)


Update neopixel.js

Fixed connection from non-local host

Oh this is actually really elegant, nice work


Allow SBERT to load other transformer models

Models which are not specifically trained sentence transformer models such as SciBERT, BioBERT or longformer can be loaded from Huggingface

I merged #1439. Could you update your branch to include those changes? And could you remove the type casting from line 109, so it just becomes X = np.array(model.encode(texts))? Then I think this is ready to merge.

Thank you @PeterLombaers, was about to push after removing the import.. will check the linter before committing from next time :)

Thanks you @rohitgarud !


No 404's in UI

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/541
Originally created by @jschatz1 on 2019-03-30 11:39:38

What is the current bug behavior?
(What is happening now)
localhost:8080/filez goes to a blank page
What is the expected correct behavior?
(What should be happening)
localhost:8080/filez goes to a 404
Steps to reproduce
(How one can reproduce the issue)
Go to localhost:8080/filez
Relevant logs and/or screenshots
(Paste any relevant logs - please use code blocks (```) to format console output)
None
Possible fixes
(If you can, link to the line of code that might be responsible for the problem or suggest a fix)
Fix on Flask side. Approve all URL's that are correct. 404 the rest.
Further regression test
(Ensure we automatically catch similar issues in the future)

[ ] Write additional adequate test cases and submit test results
[ ] Test results should be reviewed by a person from the team

View 6 previous comments from the original issue on GitLab


Open Edit Connection Points dialog via right click on bond

Background
Currently if the user wants to change connection points that participate in a bond, they would have to delete that bond, and establish a new one.
This feature would allow the user to change connection points more easily and provide bond details in a user-friendly way.
.
.
Requirements

When the user right clicks on a bond, the context menu with "Edit Connection Points..." option should appear.
.
.
Clicking on the "Edit Connection Points..." option opens the "Edit Connection Points" dialog.
.....2.1. That dialog is identical to the "Select Connection points" dialog, just with a different title, and a different button label ("Reconnect" instead of "Connect")
..........2.1.1. The dialog contains appropriately marked connection points: gray if they participate in other bonds, white if they participate in no bonds, and teal if they participate in this bond (just like "Select Connection points" dialog).
(Image for better understanding)

..........2.1.2. The user is able to interact teal and white attachment points (just like "Select Connection points" dialog).
.....2.2. If there are no available (white) connection points on both monomers, button "Reconnect" should be disabled.
.
.
The user can change the attachment points that participate in the selected bond.
.....3.1. Clicking "Reconnect" in the dialog (if different attachment points were chosen) results in deletion of the previous bond and establishment of the new one (with new attachment points).
.....3.2. Clicking "Reconnect" in the dialog (if different attachment points were not chosen) results in no change of connection points of the bond.
.....3.3. Clicking "Cancel" in the dialog results in no change of connection points of the bond.

Tests added to the automation table.

Tested. We have 5 bugs connected to ticket.

Desktop:

OS: Windows 11
Browser Chrome
Version 124.0.6367.79 (Official Build) (64-bit)

Ketcher version
[Version 2.24.0-rc.1]
Indigo version
[Version 1.23.0-rc.1]


Fixes to threading concurrency issues

This PR moves all lazy initialization into object initializer (eg. __init__) for the purpose to avoiding potential collisions when running in a multithreaded environment. The backside is that module loading takes more cycles.
The alternative approach would be to introduce explicit locking, though that seems unnecessary expensive at runtime.

Codecov Report

Merging #58 into master will increase coverage by 0.01%.
The diff coverage is 93.67%.

@@            Coverage Diff             @@
##           master      #58      +/-   ##
==========================================
+ Coverage    83.1%   83.11%   +0.01%
==========================================
  Files          28       28
  Lines        3729     3726       -3
==========================================
- Hits         3099     3097       -2
+ Misses        630      629       -1

Impacted Files
Coverage Δ

pyasn1/type/constraint.py
91.42% <100%> (-0.13%)
:arrow_down:

pyasn1/type/tag.py
87.39% <66.66%> (-0.21%)
:arrow_down:

pyasn1/type/namedtype.py
84.5% <94.59%> (+0.58%)
:arrow_up:

Continue to review full report at Codecov.

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


Feature Request: Support Pulsechain (Ethereum fork)

Pulsechain is a fork of Ethereum and the 11th biggest chain by TVL and 24h volume (S1). Meaning it is bigger than already supported chains like Cardano, KAVA and Kujira.
It also has a growing community of 125k followers on Twitter (S2).
Sources:

S1: https://defillama.com/chains
S2: https://x.com/pulsechaincom

Yes. We have plans to add this network in the very near future.

@tureck1y Thanks for your feedback. Any specific date for "in the very near future"?

Within 1-2 months I hope

@tureck1y
Oh wow. I thought it would take less time as all needed code is already there.
It would make sense to provide your users with a "add network" interface like MetaMask. The user enters RPC url, chain id, symbol, explorer and is good to go. Just a recommendation so that users like me don't have to bother you ;)
Thanks again!


Wrong padding in widgets list

Context
It seems like the left padding is too big in the widget list
Steps to Reproduce

Visit the widget list

Current Result

Expected result
It has the same padding as the other sides
Additional info
Seems to be related to .BlockList-dragIcon and opacity, changing it to display: none fixes it.

Already deployed to production! 🎉

cc @piensaenpixel


Issue 288 open first dict

Addresses #288
Can be merged once #289 is ok -- this branch builds off of that.

Launched in 3.2.4.


Add sandbox mode

Adding sandbox just enableing a internal variable _sandbox which is set by the config object passed by the constructor.
This Variable is also passed when signing the request (so the url matches the requested one)
Also catching error when sandbox mode is on and error.code is 'InternalFailure' this means sandbox parameters are not correct.

Hi!
Just merged your PRs. I changed it a bit (renamed var to use_sandbox and moved it the options inside config object) and also added a small "sandbox mode" information in Readme. Would be great if you could take a look at the readme just to make sure nothing is wrong.

Hi!
Just merged your PRs. I changed it a bit (renamed var to use_sandbox and moved it the options inside config object) and also added a small "sandbox mode" information in Readme. Would be great if you could take a look at the readme just to make sure nothing is wrong.


Webview2 with Excel-Dna gives System.NullReferenceException

Attempting to use Webview2 with an Excel-Dna addin gives the following error
System.NullReferenceException
  HResult=0x80004003
  Message=Object reference not set to an instance of an object.
  Source=<Cannot evaluate the exception source>
  StackTrace:
<Cannot evaluate the exception stack trace>

StackTrace:
   at Microsoft.Web.WebView2.WinForms.WebView2.OnVisibleChanged(EventArgs e)
   at System.Windows.Forms.Control.AssignParent(Control value)
   at System.Windows.Forms.Control.ControlCollection.Add(Control value)
   at System.Windows.Forms.Form.ControlCollection.Add(Control value)
   at ClassLibrary1.Form1.InitializeComponent()

It fails on line this.Controls.Add(this.webView2); in Form1.Designer.cs
Attached is sample project where bug occurs Sample repo
There is no error if Webview2 is removed entirely. Also Webview2 appears to work okay in a 'pure' winform app without Excel-Dna.

<?xml version="1.0" encoding="utf-8"?>
<packages>
  <package id="Excel-DNA" version="1.1.1" targetFramework="net472" />
  <package id="ExcelDna.AddIn" version="1.1.1" targetFramework="net472" developmentDependency="true" />
  <package id="ExcelDna.Integration" version="1.1.0" targetFramework="net472" />
  <package id="Microsoft.Web.WebView2" version="0.9.579-prerelease" targetFramework="net472" />
</packages>

Hi @cirrusone, this repo is intended for documentation and related issues. To get support from the Engineering team, can you open this issue on the WebView repo please? Here's the link: https://github.com/MicrosoftEdge/WebViewFeedback
I'll go ahead and close this issue now.


[1.15.2]Chinese localization

Fixed the wrong string
Supplementary Chinese localization
1.15工业先锋zh_cn.zip

Don't know why Chinese localization doesn't work

Why doesn't Chinese localization take effect?
There is no problem with the file check.
When I redownload it also still shows English

When I manually renamed zh_cn.json to en_us.json
And keep only one of them
It will show like this

Has been resolved


Deploy on Ubuntu with Gunicorn

The doc is only for uWSGI but Gunicorn is the bigger player. We have to add it.

Gui is working on the use of the embedded flask-socketio server, which will remove the use of Gunicorn.


Make all dependencies part of the standard install except for known-dangerous packages

Closes #1668
Closes #1667
Closes #1676
Closes #1677
Code Changes

[x] update the fides CI check to also import the webserver
[x] move the fides install check to the static checks file, since no Docker build is required
[x] more the optional-requirements.txt file, moving them all into requirements.txt
[x] remove the setup.py logic for the various optional requirements, except for the dangerous mssql one
[x] update all documentation that references this feature
[x] update documentation to encourage users to use best practices, i.e. pipx or venv for installing fides
[x] Remove code related to conditionally checking for packages

Steps to Confirm

[ ] list any manual steps taken to confirm the changes

Pre-Merge Checklist

[x] All CI Pipelines Succeeded
Documentation Updated:

[x] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[x] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md

Description Of Changes
While importing and using fides in other repos we've discovered that some of the optional dependencies weren't actually optional for running the webserver. This PR includes a test for this in CI checks as well as moves all optional dependencies, except for the dangerous one (mssql) into the main requirements file.
But why get rid of optional dependencies altogether?
It is a feature no one has asked for (@ThomasLaPiana believed it was a good idea, as some other projects follow this pattern) but in the end without a single cohesive framework for actually handling the careful importation of potentially non-existent packages, the engineering burden is too high.
This should get revisited with a design for a proper plugin framework (like those other projects have!) when there becomes a need for it

current failing checks are all expected

I think these failures are just colliding with the runs on main (besides the external ctl ones), will rerun to confirm when both are done

@sanders41 those 3 ctl failures are known-bad, I'm rerunning the one failed job here to confirm it passes

yep, all expected are passing

This is definitely the simplest solution to the bug issues that I opened, but there's a problem: We currently have a valid requirement for some dependencies remaining optional. See: ethyca/fidesops-plus#99 (review).
I would rather have seen a PR that restores our guarantee that dependencies defined as optional actually are optional, and then work to remove dependency optionality as individual feature enhancement PRs. A workflow like that would provide the opportunity to discuss the ramifications of each currently optional dependency becoming required. Per the above linked comment, the fides repo does not currently support extension from fidesops-plus, and unfortunately these changes wouldn't enable that.

@PSalant726 I'm not fully understanding how these issues are related. That issue is specifically referring to the inability to run the application without redis and the database spun up, which has its own ticket. That product also gets packaged as a standalone docker image only, so the extra dependencies aren't a huge concern to me.
What is a concern to me is the complexity of testing and verifying each optional dependency against a specific feature-set. As stated above, the additional complexity is not worth the value we are getting, and I'd rather ship this change to increase stability and revisit optional dependencies later when we can engineer a proper solution from the ground up.
We built checks into ctl as we knew this feature was something we wanted to support, but ops has a lot of code paths that are non-trivial to rewrite. I feel that this is the best solution for where we are now

this looks like it is hitting Shopify rate-limits during normal test runs, this should be addressed

I created a bug issue for these failures: #1714
merging as unrelated


[analyzer] Adjust computation of flatten

This is the analyzer specific issue for https://github.com/dart-lang/sdk/issues/47053, which has the details.

https://dart-review.googlesource.com/c/sdk/+/212802


tplimpl: Add gtag.js template that is new google analytics tracking code.

This PR add new internal template gtag.js template that is new google analytics tracking code.
Currently, Google Analytics show only gtag.js for tracking code.
I think that should replace google_analytics.html implementation in the future. But Google will support both analytics.js and gtag.js tracking code for the while, so I add new template at the moment.
FYI

https://developers.google.com/analytics/devguides/collection/gtagjs/

I just want to note that if this implemented, there can be no options in Hugo's Privacy Config for the gtag.js

In short, if the admin is using GTM as a one-stop-shop for events and setup of the Universal Analytics Code, the compliance with GDPR falls on his shoulders. He needs to configure it on the GTM workspace. The default code for GTM does not break GDRP, as far as I know.

The quote is from @brunoamaral 's comment over here

@onedrawingperday -its true that gtag.js is designed so that it can be configured using google tag manager site - but it's also possible to add configuration options to the implementation script.
So if this was implemented to Hugo, it'd be possible still to have an anonymizeIP: true option in the privacy config for example.
REF: https://developers.google.com/analytics/devguides/collection/gtagjs/ip-anonymization

@coliff Pull Requests are always welcome.
Personally I'm done with the Hugo simple variants for internal templates, except for the internal shortcodes Speaker Deck and possibly YouTube.

Sorry for coming late to the party. So this PR in relation to the "privacy settings" we have for GA (as in: is it possible to add for gtag, too):

[ ] UseSessionStorage  -- not sure if we can disable cookies in gtag ?
[x] RespectDoNotTrack
[x] AnonymizeIP

Google is sending me email to upgrade analytics.js to gtag.js, but I am using the _internal/google_analytics.html template.  So, is the plan to revise the template?

With the merge of #8287 is this PR still necessary?

With the merge of #8287 is this PR still necessary?

I think this can be closed now


Fix leading zero being displayed

This is related to https://github.com/kanflo/opendps/pull/185
In certain cases, unintentional leading zeros were being displayed after the fix. Certain cases would render values such as 00.123 or 01.234 even when not selected. The expected result should be 0.123 and 1.234, respectively.
I spent some time trying to understand the logic but ended up rewriting the decimal portion in order to fix the issue. The original loop seems to have worked when i overflowed which almost seems like a bug and this has been rewritten and should be somewhat clearer as to the intention of the logic.
This PR should fix these issues and result in numbers being rendered as expected.

Thanks!


[FEATURE] Import Europass CV

Is your feature request related to a problem? Please describe.

Unfortunately, Europass CV since a recent update no longer allows downloading an XML file, manipulating it locally, and importing it to the web tool. Instead, changes must be made on the website. It's a move I don't like, and would be interested in moving to Reactive Resume.
Describe the solution you'd like

I would like to be able to import my Europass CV, which is exported in PDF with embedded XML, into Reactive Resume (for editing & re-exporting).
Describe alternatives you've considered

Additional context

There is a page on interoperability, however there is no information yet (there was a stupid placeholder which they seem to have removed after my enquiry). I have asked if & when interoperability information would become available - but based on my experience with EU product development I doubt it'll be any time soon.
However, based on the existing XML file a reliable import might already be produced. It would be great if it could be supported.

I'm not familiar with Europass CV's embedded XML in PDF format. Would it be possible for you to share a sample PDF with some information that can be parsed?

Thanks @AmruthPillai for the quick reply. I created a CV with all possible sections, and at least one item for each section. Some have structured data (like the driving license or job title), but most items consist of a title, start (and end) date, description and link.
TestCV to see structured data.zip
Let me know if you have any questions or comments.


add public method afterSpawnProcess to Linter class

Gives subclasses a chance to read or change the child
process, after creating new BufferedProcess while lintFile.
Useful for passing data via stdin (process.process.stdin).
This is a backward compatible improvement.

This PR was submitted as part of a fix to https://github.com/AtomLinter/linter-jshint/issues/25. Subclasses like linter-jshint need access to stdin on the spawned child class.
I'm hoping @iam4x can review this PR, and the related linter-jshint issue.
And sorry for failing the CI build. It's such a lightweight PR, I don't know where it went wrong...


Magnetdl block

Bro magnetdl block in your demo link plz host your demo link where it unblock plz

@Rajveer0916 it is because of the ddos protection will try to fix it

Bro plzz i lovee this project plz dp something to fix it

@Rajveer0916 i have already fixed it wait till tomorrow i will upload a new repo

@Rajveer0916 https://github.com/Ryuk-me/Torrent-Api-py


MAINT: Build wheels against NumPy 2.0

Opening just for completeness in case 2.0-compatible wheels want to be cut before 2.0 actually comes out. Probably no point in merging if new wheels won't be released before NumPy 2.0.0 is released. So feel free to close!
Related to #325

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

This PR is not necessary with this pin https://github.com/Unidata/cftime/blob/d549c09638c627f6fcec3d34b0ae38c92056667c/pyproject.toml#L7
Added in #319

Closing since it seems like this won't be useful given the release plan!


Add Preact error boundaries across the UI

Category
Other
What are you missing?
UI errors shouldn't break the whole mod.
Code of Conduct

[X] I agree to follow the code of conduct
[X] My request fits the contribution guidelines
[X] I have searched for similar issues and nothing similar exists

Reopening - added changes to the workflow editor, but not the dashboard


Spaces inside array literal brackets vs spaces inside array/object access brackets

I'd really want to be able (e.g. enforce) my code like this:
var a = [ 1 ];
console.log(a[0]);

I know there are requireSpacesInsideArrayBrackets, disallowSpacesInsideArrayBrackets, allowSpacesInsideBrackets and disallowSpacesInsideBrackets rules, but i couldn't find any combination to achieve what i want.
I can enforce spaces inside of array literal brakcets with requireSpacesInsideArrayBrackets, but i can't disallow spaces inside array/object access brackets with disallowSpacesInsideBrackets without creating a conflict between those 2 rules.
Pull requests for #923 didn't seem to provide a solution for this.

Yeah, currently there is no way to enforce this particular style, it seems you need something more specific than SpacesInsideBrackets an opposite analogue of SpacesInsideArrayBrackets for getters only.
We could accept that rule if you choose to send us a pr? Would you like to do that?

We released 3.0 recently! We also wrote a post about how 3.0 will be our last major release (since we are deprecating the project) and that we are merging with ESLint!
Thus we won't be working on any new features moving forward which includes new rules/options. We will continue to fix bugs and work on CST/better autofixing for current rules in the upcoming months both to help JSCS users and to gain experience that we can hopefully use in ESLint. But otherwise we both teams will be working on getting ESLint feature parity with JSCS and just a nice transition for our users.
I'm putting an orphaned label to note this issue isn't planned to be worked on, but might be a good contribution as a custom rule (not in JSCS core itself).
I would encourage anyone who wants the rule to try writing it themselves: I would either write a custom rule or submit an issue to ESLint and if accepted a PR. Otherwise a custom rule that won't be in core repo itself.
JSCS rules are also very similar to ESLint rules: here are some resources to get started:

Project-specific Lint Rules with ESLint
Creating Custom Rules for ESLint
Maintaining JavaScript Code Quality with ESLint


proxy reverse nginx subfolder or subdomain

Hello, is there any method or project for future implementation of reverse proxy nginx subdomain (jellystat.mydomain.com) or reverse proxy nginx subfolder? (mydomain.com/jellystat).
thanks.

i currently use the NGINX Proxy Manager to have jellystat as a subdomain of my actual domain(jellystat.domain.extension) and have no issues with it.

i currently use the NGINX Proxy Manager to have jellystat as a subdomain of my actual domain(jellystat.domain.extension) and have no issues with it.

Can you post your config?

i currently use the NGINX Proxy Manager to have jellystat as a subdomain of my actual domain(jellystat.domain.extension) and have no issues with it.

Can you post your config?

i am using docker to host nginx, and i use this image: jc21/nginx-proxy-manager
it has an easy to use GUI, and i massively prefer using GUI's whenever possible.
but this is what i have in the GUI.

the advanced and custom locations is not used.
all my pages are setup like this, with small changes, such as the subdomain being different, and the forward hostname and port being changed pr program.
if i had jellyseerr on one of my other raspberry pis, instead of hostname, i would use the ip of the pi, but since jellyseerr and nginx proxy manager is on the same pi(same network in docker) i just use hostname, and dont forward the port out of docker.

Thank you for your post. Unfortunately, that does not really help, as I use config files and don't see anything out of the ordinary in your pictures. I hope the maintainer can look into this.

Thank you for your post. Unfortunately, that does not really help, as I use config files and don't see anything out of the ordinary in your pictures. I hope the maintainer can look into this.

managed to dig around, and found the file. i have removed a ton of blank spaces, and removed the actual domain, but this is the file i have that handles my reverse proxy.
`# ------------------------------------------------------------
jellystat.DOMAIN.EXTENSION
------------------------------------------------------------
map $scheme $hsts_header {
https   "max-age=63072000; preload";
}
server {
set $forward_scheme http;
set $server         "<IP_ADDRESS>";
set $port           8096;
listen 80;
listen [::]:80;
listen 443 ssl;
listen [::]:443 ssl;
server_name jellystat.DOMAIN.EXTENSION;
Let's Encrypt SSL
include conf.d/include/letsencrypt-acme-challenge.conf;
include conf.d/include/ssl-ciphers.conf;
ssl_certificate /etc/letsencrypt/live/npm-38/fullchain.pem;
ssl_certificate_key /etc/letsencrypt/live/npm-38/privkey.pem;
Block Exploits
include conf.d/include/block-exploits.conf;
# Force SSL
include conf.d/include/force-ssl.conf;
access_log /data/logs/proxy-host-59_access.log proxy;
error_log /data/logs/proxy-host-59_error.log warn;
location / {
# Proxy!
include conf.d/include/proxy.conf;
}
Custom
include /data/nginx/custom/server_proxy[.]conf;
}`
if you need anything else, i can now find it :)

Thank you for your reply. I thought that it might have something to do with you using a subdomain instead of a directory. However, right after the user creation, it displays "Error: Unable to connect to Jellystat Backend".
Which is more than I ever got with a subdirectory.


Apply entity mods alphabetically for more consistent pasteing results

Before I merge, I'll test and see if lua table order actually does get randomized based on string/insert-order or not.

Just confirmed insert order does matter. I'll merge later after testing.


Fix jumpiness in home page icon bar

Not all icons have the same hight, and this causes the home page to
jump.
Force all icons to fit in a 80x80 box.

Pull Request Test Coverage Report for Build 215

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 54.461%

Totals

Change from base Build 209:
0.0%

Covered Lines:
4182

Relevant Lines:
7117

💛  - Coveralls


Remove Amazon.Corretto.8 <IP_ADDRESS>2 to Amazon.Corretto.8.JDK <IP_ADDRESS>2

Microsoft Reviewers: Open in CodeFlow

URL: https://corretto.aws/downloads/resources/8.412.08.1/amazon-corretto-8.412.08.1-windows-x86-jdk.msi
OK
(Automated message - build 910)

Hi @SpecterShell,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 886.)


Problems installing cvxpy with latest Anaconda 3

I just created a fresh Anaconda 3 installation, and tried to install cvxpy using the instructions on the Web site:
conda config --add channels oxfordcontrol
conda install -c conda-forge lapack
conda install -c cvxgrp cvxpy=1.0.24
I get lots of errors on the third command:
(base) RS_MacBookPro:~ stanton$ conda install -c cvxgrp cvxpy=1.0.24
WARNING conda.base.context:use_only_tar_bz2(632): Conda is constrained to only using the old .tar.bz2 file format because you have conda-build installed, and it is <3.18.3.  Update or remove conda-build to get smaller downloads and faster extractions.
Collecting package metadata (repodata.json): done
Solving environment: failed
UnsatisfiableError: The following specifications were found to be incompatible with each other:

anaconda==2019.03=py37_0 -> importlib_metadata==0.8=py37_0 -> zipp[version='>=0.3.2']
anaconda==2019.03=py37_0 -> mkl-service==1.1.2=py37hfbe908c_5
importlib_metadata
mkl-service
zipp

Any suggestions for what I should do next?
Thanks very much.
Richard Stanton

I forgot to mention - this is under OS X.

The only dependency there that's associated with cvxpy is MKL. Also I've only seen errors like this when you used conda's "pinned" file to keep dependencies at certain versions. Are you using a pinned file? (You would know if you are.)
If you want to use conda, you could try again but run conda update conda -y before the various cvxpy statements. However we recommend using pip.
You can still use pip within a conda environment. Just run conda install pip and pip install cvxpy.

This was with a brand new Anaconda installation, and no pinned files. I had previously tried using pip, and that failed too. Installing under Python 3.6 rather than 3.7 did the trick.

On Jul 14, 2019, at 8:56 AM, rileyjmurray<EMAIL_ADDRESS>wrote:
The only dependency there that's associated with cvxpy is MKL. Also I've only seen errors like this when you used conda's "pinned" file to keep dependencies at certain versions. Are you using a pinned file? (You would know if you are.)
If you want to use conda, you could try again but run conda update conda -y before the various cvxpy statements. However we recommend using pip.
You can still use pip within a conda environment. Just run conda install pip and pip install cvxpy.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cvxgrp/cvxpy/issues/761?email_source=notifications&email_token=AAQNZDTL6UQ43HUHHDSKBMTP7NEDZA5CNFSM4H6TNE32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ4IFPQ#issuecomment-511214270, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQNZDVOUNTCSIY4COXMGKLP7NEDZANCNFSM4H6TNE3Q.

Could you post the pip error?

I would, but of course this time it worked fine…!

On Jul 15, 2019, at 1:17 PM, Steven Diamond<EMAIL_ADDRESS>wrote:
Could you post the pip error?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cvxgrp/cvxpy/issues/761?email_source=notifications&email_token=AAQNZDRVLFKKM2FCLLOVFC3P7TLNXA5CNFSM4H6TNE32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ626NA#issuecomment-511553332, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQNZDUDPTQE7GK73KKM4D3P7TLNXANCNFSM4H6TNE3Q.

I have the exact same problem with conda, I can share my error log for pip:
Collecting cvxpy
Downloading https://files.pythonhosted.org/packages/31/1d/3a40c544de8cddbd943e955758e58bf12b5ce871ec989fdbff54fc97af43/cvxpy-1.0.24-cp37-cp37m-macosx_10_7_x86_64.whl (670kB)
100% |████████████████████████████████| 675kB 2.4MB/s
Requirement already satisfied: multiprocess in /anaconda3/lib/python3.7/site-packages (from cvxpy) (0.70.8)
Requirement already satisfied: six in /anaconda3/lib/python3.7/site-packages (from cvxpy) (1.12.0)
Requirement already satisfied: scipy>=1.1.0 in /anaconda3/lib/python3.7/site-packages (from cvxpy) (1.2.1)
Collecting ecos>=2 (from cvxpy)
Downloading https://files.pythonhosted.org/packages/b9/3a/59aa93b573a22fda44402383aeddcc2a081c31e61080af3da9d11855c77a/ecos-2.0.7.post1.tar.gz (126kB)
100% |████████████████████████████████| 133kB 3.8MB/s
Requirement already satisfied: numpy>=1.15 in /anaconda3/lib/python3.7/site-packages (from cvxpy) (1.16.2)
Requirement already satisfied: osqp>=0.4.1 in /anaconda3/lib/python3.7/site-packages (from cvxpy) (0.5.0)
Collecting scs>=1.1.3 (from cvxpy)
Downloading https://files.pythonhosted.org/packages/f2/6e/dbdd778c64c1920ae357a2013ea655d65a1f8b60f397d6e5549e4aafe8ec/scs-2.1.1-2.tar.gz (157kB)
100% |████████████████████████████████| 163kB 3.6MB/s
Requirement already satisfied: dill>=0.3.0 in /anaconda3/lib/python3.7/site-packages (from multiprocess->cvxpy) (0.3.0)
Requirement already satisfied: future in /anaconda3/lib/python3.7/site-packages (from osqp>=0.4.1->cvxpy) (0.17.1)
Installing collected packages: ecos, scs, cvxpy
Running setup.py install for ecos ... error
Complete output from command /anaconda3/bin/python -u -c "import setuptools, tokenize;file='/private/tmp/pip-install-70a30ppa/ecos/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /private/tmp/pip-record-yny2mh52/install-record.txt --single-version-externally-managed --compile:
running install
running build
running build_py
creating build
creating build/lib.macosx-10.7-x86_64-3.7
creating build/lib.macosx-10.7-x86_64-3.7/ecos
copying src/ecos/version.py -> build/lib.macosx-10.7-x86_64-3.7/ecos
copying src/ecos/init.py -> build/lib.macosx-10.7-x86_64-3.7/ecos
copying src/ecos/ecos.py -> build/lib.macosx-10.7-x86_64-3.7/ecos
running build_ext
building '_ecos' extension
creating build/temp.macosx-10.7-x86_64-3.7
creating build/temp.macosx-10.7-x86_64-3.7/src
creating build/temp.macosx-10.7-x86_64-3.7/ecos
creating build/temp.macosx-10.7-x86_64-3.7/ecos/external
creating build/temp.macosx-10.7-x86_64-3.7/ecos/external/ldl
creating build/temp.macosx-10.7-x86_64-3.7/ecos/external/ldl/src
creating build/temp.macosx-10.7-x86_64-3.7/ecos/src
creating build/temp.macosx-10.7-x86_64-3.7/ecos/external/amd
creating build/temp.macosx-10.7-x86_64-3.7/ecos/external/amd/src
creating build/temp.macosx-10.7-x86_64-3.7/ecos/ecos_bb
gcc -Wno-unused-result -Wsign-compare -Wunreachable-code -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -I/anaconda3/include -arch x86_64 -I/anaconda3/include -arch x86_64 -DPYTHON -DDLONG -DLDL_LONG -DCTRLC=1 -Iecos/include -Iecos/external/amd/include -Iecos/external/ldl/include -Iecos/external/SuiteSparse_config -I/anaconda3/include/python3.7m -I/anaconda3/lib/python3.7/site-packages/numpy/core/include -c src/ecosmodule.c -o build/temp.macosx-10.7-x86_64-3.7/src/ecosmodule.o
In file included from /usr/local/lib/gcc/x86_64-apple-darwin14.0.0/4.9.2/include-fixed/syslimits.h:7:0,
from /usr/local/lib/gcc/x86_64-apple-darwin14.0.0/4.9.2/include-fixed/limits.h:34,
from /anaconda3/include/python3.7m/Python.h:11,
from src/ecosmodule.c:4:
/usr/local/lib/gcc/x86_64-apple-darwin14.0.0/4.9.2/include-fixed/limits.h:168:61: fatal error: limits.h: No such file or directory
#include_next <limits.h>  /* recurse down to the real one */
^
compilation terminated.
error: command 'gcc' failed with exit status 1
----------------------------------------

Command "/anaconda3/bin/python -u -c "import setuptools, tokenize;file='/private/tmp/pip-install-70a30ppa/ecos/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /private/tmp/pip-record-yny2mh52/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /private/tmp/pip-install-70a30ppa/ecos/

The current recommended way to install cvxpy is via pip. There are also up-to-date cvxpy versions (1.0.27) on conda-forge. In the next few months cvxpy developers will shift conda deployment efforts to conda-forge, and that should be more reliable.
In view of the points above (and no activity on this issue in 8 months), I'm closing this issue.


Extract lake morphometry

Using R package rLakeAnalyser, we will extract the morphometry of every lake in our data sets.
We think that this issue should be completed as of June 25th. 🕥

This will finally not be done since we have the lakes' bathymetry.


Add checkout fees, update min/max amounts, ..

Added checkout_fee_rate (0.1%)
Increased min & max checkout amounts
Changed eth token bridge configs for testnet-* from eth mainnet to ropsten
Added a blockchain_params.json placeholder for the mainnet

Thanks for the review! Now merging~


vllm+KubeRay deployment, streaming is very slow

Thank you for the great repo. I followed the instruction to deploy Mistral7B-AWQ model using vLLM and KubeRay to GCP k8s clusters. What I find is that for the exact same request (temperature=0 for reproducibility), streaming takes much longer compared to regular request, especially when the decoding is lengthy.
I can't reproduce this in my local deployment, so I suspect it's an issue with Ray cluster? Any help would be much appreciated. Thank you.

Hi @AIApprentice101 .Seems you should place the issue in vlm/ray's official repo and provide more details like :

how you create your ray cluster
how you use vllm with ray.

@AIApprentice101 http://kubeagi.k8s.com.cn/docs/Performance/distributed-inference, not sure if you're using distributed inference with multiple GPUs across nodes, then the performance might be bad.

@bjwswang @nkwangleiGIT Thank you for your prompt response. I'm doing very basic stuff, using 1 L4 GPU to serve the model. Here're the vllm configs I'm using.
llm_model_name = "Mistral-7B-Instruct-v0.2-AWQ"
tensor_parallel_size = "1"
gpu_memory_utilization = "0.9"
quantization = "awq"
worker_use_ray = "false"
max_model_len = "19456"
enable_prefix_caching = "true"
max_num_seqs = "64"
enforce_eager = "false"

I closely follow the instruction in this link: http://kubeagi.k8s.com.cn/docs/Configuration/DistributedInference/deploy-using-rary-serve. The only modification I did is to use the vllm/vllm-openai:v0.4.2 image rather than the 0.4.1, since the patch you guys did has been fixed in v0.4.2 (https://github.com/vllm-project/vllm/issues/2683).
In terms of the Ray cluster environment, we have it setup by the SRE on GCP, so I don't have much to say about that.
What I observe is that non-streaming response (stream=False) in the Ray cluster performs very close to my local deployment using serve run. But the streaming is abnormally slow in the Ray cluster deployment. This is very obvious for long-decoding tasks (e.g. Write me a long essay with at least 20 paragraphs).
Any directions or suggestions would be appreciated. Thank you.

It turns out to be the issue of the setup on our end. Thank you for your help.


Update the samples using /edits api in the ai.text:Client

Description:
Currently there are two samples implemented to showcase the usage of the /edits api endpoint in OpenAI. But this API have been deprecated and shutdown its operation. So the given samples are not working. Below are those samples.

Write robust API powered AI applications.
Bring text alive with openai chat (Sample 2).

Both the samples are pointed to a single sample ballerina code. We need to update this sample once issue is fixed from the library team.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Closing as per https://github.com/ballerina-guides/ai-samples/pull/71


Add MockSpecifiedPersonForm

Although SpecifiedPersonForm doesn't require GraphQL mocking, this does add our Storybook & GitHub button links.


chore: Cleanup --report-unused-disable-directives

PR Checklist

[x] Addresses an existing open issue: fixes #1089
[x] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview
Removes all references to --report-unused-disable-directives

Created this one as draft. I was getting some errors when running pnpm run test. My guess is that the snapshots need to be updated, probably related to the changes going on in #1048
I'd wait for that one to be merged so CI tests run successfully on this one.

getting some errors when running pnpm run test

Huh, weird, CI is passing here. I don't think you should need to wait for any other PR. What are the errors? The changes here look perfect to me. :smile:

Ok let's try it


Updated API version to v1beta1 for the Kubernetes Operator docs

Description of the change
Updated the CRD API versions to v1beta1 to match the versions of the Kubernetes Operator's latest version.
Checklist
Please make sure that the proposed change checks all the boxes below before requesting a review:

[x] I have reviewed the guidelines for contributing to this repository.
[x] The preview looks fine.
[x] The tests pass.
[x] The commit history is clean and meaningful.
[x] The pull request is opened against the main branch.
[x] The pull request is no longer marked as a draft.
[x] You agree to license your contribution under the MIT license to Spacelift (not required for Spacelift employees).
You have updated the navigation files correctly:

[x] No new pages have been added, or;
[ ] Only nav.yaml has been updated because the changes only apply to SaaS, or;
[ ] Only nav.self-hosted.yaml has been updated because the changes only apply to Self-Hosted, or;
[ ] Both nav.yaml and nav.self-hosted.yaml have been updated.

If the proposed change is ready to be merged, please request a review from @spacelift-io/solutions-engineering. Someone will review and merge the pull request.
Spacelift employees should request reviews from the relevant engineers and are allowed to merge pull requests after they got at least one approval.
Thank you for your contribution! 🙇

Good catch, thanks a lot for that @eminalemdar ❤


Ignore unavailable deployments and handle kubeconfig secret rotation

What this PR does / why we need it:
This PR handles skipping the scaling operation for deployment when they are not available in the cluster.
The logs are also refined to avoid noise and clutter.
It also handled the bug introduced with reloading of shoot kubeconfig as reported with #36.
Which issue(s) this PR fixes:
Fixes #40 #36
Special notes for your reviewer:

The PR upgrades GO dependency to 1.17
There is also a delay introduced before checking dependents for desire availability. This is introduced as checking for availability immediately after scaling operation always failed as it takes some time for the deployment to become ready.
Currently it is a hacky way with 2 sec delay as per what was observed from the tests.
Without this delay also the scaling operation happens but it is honored in the next reconciliation cycle. This creates an overall delay of 30s scaling delay per deployment requiring scaling.
In order to handle the kubeconfig rotation race condition where for some edge case the probe fails to access the latest kubeconfig and continues to fail unless restated. A retry mechanism is introduced which shall update the secret. The retry only acts if the error api errors of
Refactors the error handling for probe.
Updated alpine image to 3.15

Release note:

A bug is fixed which allowed dependency-watchdog to not ignore scaling operations on deployment which are not enabled/deployed in a given cluster
A bug with uploading of a rotated dependency-watchdog-probe secrets is now fixed with introduction of refresh of the clients with updated secrets.

Thanks Madhav, I agree we can make a total separation of concerns.
The changes are made with the commit https://github.com/gardener/dependency-watchdog/pull/41/commits/15f476af3a05b03b7922b3866cf2a46df1687c51 and it also tested on a local seed/shoot setup.

Fixed test errors with missing arguments https://github.com/gardener/dependency-watchdog/compare/15f476af3a05b03b7922b3866cf2a46df1687c51..1d5b24b617cefdf54bef6493eddbec22aecf9108

/lgtm


Hide generated dummy const in rustdoc

When creating a binary crate with a src/main.rs and #[derive(Deserialize, Serialize)], running cargo doc on it will result in new doc items for generated constants:
use serde::{Deserialize, Serialize};

#[derive(Deserialize, Serialize)]
struct Dummy {
}

fn main() {
    println!("Hello, world!");
}

This is because Cargo runs with --document-private-items internally for src/main.rs. If you put the same code in a src/lib.rs the private items will not be documented by default, so you won't see them there for default cargo doc.
The items will be visible both for src/lib.rs and src/main.rs if the struct itself is pub.
I believe the generated _IMPL_{}_FOR_{} constants should not be visible in rustdoc and this merge request seems to fix the issue for me.

Published in 1.0.106.


Design changes

I'm pretty happy with this! I didn't make many changes content wise- I basically took the existing components and made the interface look nicer. I really like the dynamic tabbed nav bar, which eliminates the need for URL redirects when navigating the tabs. I also spent many hours looking into how to preserve this Single Page Application nature for displaying the scanned lines, i.e. rewriting the current web page with the result.html contents without the page having to reload. Unfortunately, I wasn't able to find a solution to this, so my temporary solution was to display the scanned lines within the box section (which I did via template inheritance). It would be ideal if the form submission could occur without page reload, so it would be great if you could help tackle that (if the feat is accomplishable in the first place).
I think my next goal is to improve the formatting of the scanned lines. I'm still thinking about what the most readable, clean  way would be to indicate the syllables and feet, along with the original/modified pronunciations. For instance, do we want to give users the option to select which pronunciations they want to display, or do we just want to choose one for standardization purposes? I thought of a few ideas for the structure. One is to display the original text on the left half, and the variable pronunciation on the right half with macrons/breves to indicate weight (and either D, S, T for the feet or color coding). Another idea is to display the original text and the variable pronunciation on the left half, and pure syllable structures on the right (e.g. ¯ ˘ ˘ ˈ ¯ ˘ ˘  ˈ ¯ ¯ ˈ ¯ ¯ ˈ ¯ ˘ ˘ ˈ ¯ ˚).
Let me know what your thoughts are.

https://user-images.githubusercontent.com/57687959/143527270-013bbe43-03b9-4c85-b62d-35dd2beec84c.mov
Video demo of what it looks like

Could you add some information to the README about how you minify? Or rather, how one would minify: what commands to run etc.

I just used a website online that minifies (?)

I just used a website online that minifies (?)

ha nevermind then ;)

What’s your suggestion on what to say?

On Nov 26, 2021, at 2:01 PM, Kyle Gorman @.***> wrote:
@kylebgorman commented on this pull request.
Looks to me like you just need to make sure everyone is using the minified CSS and we're good to go.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/CUNY-CL/latin_scansion_webapp/pull/1#pullrequestreview-817015687, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANYD7F7VOESSYWT3OKXTKN3UN7KRVANCNFSM5IZZTFKQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

What’s your suggestion on what to say?

Re: minification? Nothing I guess, we can come back to that later.

What’s your suggestion on what to say?

Re: minification? Nothing I guess, we can come back to that later.

Okay, should we merge?

The one issue I see before merging is that the static pages need to be
pointed at the style.min.css too, right?
On Fri, Nov 26, 2021 at 8:52 PM jillianchang @.***>
wrote:

What’s your suggestion on what to say?
Re: minification? Nothing I guess, we can come back to that later.
Okay, should we merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/latin_scansion_webapp/pull/1#issuecomment-980484849,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4OIMM2F36NYLOR5AVOLUOA2VNANCNFSM5IZZTFKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

About.html and input.html? Oh actually, I can remove those files because they're not in use. JS allows index.html to change div content upon clicking the navigation tabs.

Yea, remove those and we should be good to go.
On Fri, Nov 26, 2021 at 8:58 PM jillianchang @.***>
wrote:

About.html and input.html? Oh actually, I can remove those files because
they're not in use. JS allows index.html to change div content upon
clicking the navigation tabs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/latin_scansion_webapp/pull/1#issuecomment-980485448,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4OOAWBXJURC3O5GUOQTUOA3MJANCNFSM5IZZTFKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

It says I don't have write access?

Ah, I'll merge and then grant that to you.
On Fri, Nov 26, 2021 at 9:03 PM jillianchang @.***>
wrote:

It says I don't have write access?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/latin_scansion_webapp/pull/1#issuecomment-980486088,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4OKKVF65CUDIPJKSEW3UOA4ALANCNFSM5IZZTFKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I just granted you write access but please use the fork-and-pull approach
all the same ;)
On Fri, Nov 26, 2021 at 9:04 PM Kyle Gorman @.***> wrote:

Ah, I'll merge and then grant that to you.
On Fri, Nov 26, 2021 at 9:03 PM jillianchang @.***>
wrote:

It says I don't have write access?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/latin_scansion_webapp/pull/1#issuecomment-980486088,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4OKKVF65CUDIPJKSEW3UOA4ALANCNFSM5IZZTFKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Yes, of course. It looks like it didn't automatically deploy to Heroku, guess it'll have to manually be done.

Never mind, I see it now :)

Yeah, it takes a few minutes because it basically restarts the app from
scratch. I was actually logged into Heroku and it was sending me the logs
more or less live so I knew just went to press "refresh". Looks great in
person.
On Fri, Nov 26, 2021 at 9:09 PM jillianchang @.***>
wrote:

Never mind, I see it now :)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/latin_scansion_webapp/pull/1#issuecomment-980486846,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4OOSOHXJPEK666AX2R3UOA4UTANCNFSM5IZZTFKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[Feature Request] Decline pull request

You would benefit greatly from being able to decline a pull request in a CICD environment.
In an automated system, you may want to perform various tests upon receiving a new pull request, and then decline it if it fails. 👍

Hello thanks for writing in. I would like a few clarifications?

Is this about GitHub (the website/platform) or hub (the command-line tool)?
What is a CICD environment and how is it relevant here?
How does "declining" a pull request manifest itself? Does the PR get closed, or something else?

Hello thanks for writing in. I would like a few clarifications?

Is this about GitHub (the website/platform) or hub (the command-line tool)?
What is a CICD environment and how is it relevant here?
How does "declining" a pull request manifest itself? Does the PR get closed, or something else?

This is about using the hub command-line tool to remotely decline pull requests
Github webhook -> Jenkins -> Build, Test, Deploy. Jenkins should be able to use the hub command-line tool to decline the pull request if it fails during testing.
Good question. It would be very beneficial to be able to decide whether or not you want to close PR, ask for change, or accept the change. For my own use, I would probably only use it to request a change instead of closing the PR completely.

Thank you for the clarifications. It sounds like you want to run some CI checks for every Pull Request (or subsequent push to a PR), and "decline" (i.e. mark as "failed") the PR if any of the checks failed. GitHub has features precisely for that; see https://help.github.com/en/articles/about-status-checks
If you need to interface with the GitHub API directly from the command-line (for example, to write to the Statuses API), you can use the hub api command. For example, in your CI job you could do:
GITHUB_TOKEN=<your-oauth-token> hub api repos/{owner}/{repo}/statuses/<SHA> \
  -f state=failure \
  -f context=my-check

However, doing any API integration like this is left as an exercise to the reader; at the current time we will not create features for updating the Statuses or Checks API in hub since that falls outside of the scope of the project.


call to os_perso_derive_node_with_seed_key hangs

I need to implement key pair derivation for Curve25519 using SLIP10 (rather than Bip44). When I'm trying to call os_perso_derive_node_with_seed_key() function, it just hangs the device. Below is the minimalistic code which reproduces the issue:
pub fn derive_private_key_curve25519() -> Result<[u8; 32], AppErrors> {
    unsafe {
        let mut seed = [0u8; 32];
        // m/44'/1022'/365'
        let path: [u32; 3] = [
            44u32 | 0x80000000u32,
            1022u32 | 0x80000000u32,
            365u32 | 0x80000000u32,
        ];

        os_perso_derive_node_with_seed_key(
            HDW_ED25519_SLIP10,
            CX_CURVE_Curve25519,
            path.as_ptr(),
            path.len() as c_uint,
            seed.as_mut_ptr() as *mut c_uchar,
            null_mut(),
            null_mut(),
            0,
        );

        return Ok(seed);
    }
}

The corresponding version in C looks like this:
uint8_t seed[32];
uint32_t path[] = {44 | 0x80000000, 1022 | 0x80000000, 365 | 0x80000000};

os_perso_derive_node_bip32_seed_key(HDW_ED25519_SLIP10, CX_CURVE_Ed25519, path, 3, seed, NULL, NULL, 0);

The os_perso_derive_node_bip32_seed_key is just an alias to os_perso_derive_node_with_seed_key defined in os_seed.h.
Any ideas why code above hangs?

Hi, did you try this code with speculos? Trying to know if you're actually getting a segfault.
Does the C code you show hangs also or only the Rust one?

Also, tried C version, and it seems is working flawlessly in speculos.

I made a small modification to the code and speculos started producing different result.
Changed code (added seed):
pub fn derive_private_key_curve25519__() -> Result<[u8; 32], AppErrors> {
    unsafe {
        let mut seed = [0u8; 32];
        // m/44'/1022'/365'
        let path: [u32; 3] = [
            44u32 | 0x80000000u32,
            1022u32 | 0x80000000u32,
            365u32 | 0x80000000u32,
        ];
        let mut ed25519_seed: [u8; 12] = [
            b'R', b'a', b'd', b'i', b'x', b'B', b'a', b'b', b'y', b'l', b'o', b'n',
        ];

        os_perso_derive_node_with_seed_key(
            HDW_ED25519_SLIP10,
            CX_CURVE_Curve25519,
            path.as_ptr(),
            path.len() as c_uint,
            seed.as_mut_ptr() as *mut c_uchar,
            null_mut(),
            ed25519_seed.as_mut_ptr(),
            ed25519_seed.len() as c_uint,
        );

        return Ok(seed);
    }
}

Speculos output:
siy@bobryzen:~/CLionProjects/Ledger/speculos$ ./speculos.py ../../Radix/babylon-ledger-app/target/nanos/release/babylon-ledger-app
Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway.
[*] speculos launcher revision: d94210d
[*] using SDK version 2.1 on nanos
[*] loading CXLIB from "/home/siy/CLionProjects/Ledger/speculos/speculos/cxlib/nanos-cx-2.1.elf"
[*] patching svc instruction at 0x126358
[*] patching svc instruction at 0x400017d8
 * Serving Flask app 'speculos.api.api' (lazy loading)
 * Environment: development
 * Debug mode: off
17:52:53.462:werkzeug:  * Running on all addresses.
   WARNING: This is a development server. Do not use it in a production deployment.
17:52:53.462:werkzeug:  * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
QPixmap::scaled: Pixmap is a null pixmap
17:53:01.449:apdu: > aa12000000

Unhandled exception 2 at /home/siy/CLionProjects/Ledger/speculos/src/bolos/cx_ec.c:1433
Exiting.

Thanks for all the info!
I had trouble noticing but I think this the difference between the Rust code and the C example:

C uses CX_CURVE_Ed25519
Rust uses CX_CURVE_Curve25519

The second one is incorrect to use for SLIP10 derivation, and is not supported in speculos indeed :)

Was about to write the same :) Thanks a lot!

After the fix, the code shown above works in speculos, but hangs on the real device. Any ideas?

Check that the SDK version used with speculos (2.1 for Nano S given your example above) matches the version of your device, as a first check.

Device (Nano S) has latest available firmware:
Secure Element: 2.1.0
Microcontroller: 1.11
Bootloader: 0.11
Is it OK?

It is, and actually I'm not sure what would cause a hang on Nano S but not in speculos at this point :/

By the way: I'm trying to setup speculos so it will use same seed phrase (12/18/24 words) as test device in order to ensure that tests will perform identically. Unfortunately after diggigng documentation and skimming sources I've not found way to set seed phrase in speculos. Any hints?

I think the --seed argument should do what you need, https://github.com/LedgerHQ/speculos/blob/master/speculos/main.py#L140-L144

Thanks! From the examples it looked like this argument is used for RNG init, rather than mnemonic setup.

Question: might the issue be caused by compiler options with which C part of the SDK is compiled?

OK, the issue was caused by invalid curve declaration in metadata. Simulator ignores metadata and supports all curves (not just ones declared for app) and works fine.
During investigation, I've found that following metadata declaration does not work:
[package.metadata.nanos]
curve = "secp256k1"
curve = "ed25519"
flags = "0x240"
icon = "icons/nanos_app_radix.gif"
icon_small = "icons/nanos_app_radix.gif"
path = "44'/1022'"
name = "Radix Babylon"

Attempt to build binary fails with
Duplicate key `curve` in table `package.metadata.nanos`

I need both curves to be present in the metadata. Any ideas?

OK I see, thanks for the explanation.
There's a PR available here https://github.com/LedgerHQ/cargo-ledger/pull/16 that I will merge today that handles curves as a list
[package.metadata.nanos]
curve = ["secp256k1", "ed25519"]

Issue is fixed. Thanks for help!


Improved Android platform support

!WARNING! This is a "work in progrees".  Don't merge befor I post a message "Done! Can be merged."
List of improvements:

[x] improved build time on AppVeyor CI;
[ ] fixed git version detection;
[ ] ...

Cmake behavior is slightly weird on Windows. It properly detects cache for dependencies (like Boost), but for some reasons decides to ignore it.

One of the commits in this PR fixed cmake warnings whentrying to read project version from git tag.

I am waiting and looking forward to your commits =)

Thanks @qwertycoin-org 🙂 I prepared a lot of things for this PR.
But at the moment I'm a bit confused with the way Boost is installed on AppVeyor CI and can't figure out what is causing cmake errors when find_package(Boost ...) is called.

Replaced "DIAVIK" with "@VERSION@" in src/version.h.in.

@qwertycoin-org FYI, CMake (from MSVC toolchain) is often one step behind Boost.
It is possible to work around this with simple hack from Boost.GIL:
# Latest FindBoost.cmake has likely been updated to detect Boost version not yet released
if (NOT EXISTS "${CMAKE_BINARY_DIR}/cmake/FindBoost.cmake")
  message(STATUS "Downloading FindBoost.cmake from https://gitlab.kitware.com/cmake/ master branch")
  file(DOWNLOAD
    "https://gitlab.kitware.com/cmake/cmake/raw/master/Modules/FindBoost.cmake"
    "${CMAKE_BINARY_DIR}/cmake/FindBoost.cmake")
endif()
list(INSERT CMAKE_MODULE_PATH 0 ${CMAKE_BINARY_DIR}/cmake)

I decided to simplify this "hack" and directly add latest FindBoost.cmake version into cmake/find folder.

After a few updates/tweaks build time on AppVeyor CI went down from 45-60 minutes to 8-10 minutes.

Next up is Android build on Travis CI... 🙂

Done! Can be merged 🙂
cc @qwertycoin-org

Perfect! Good work
I think the Android platform will bring us many benefits.

I think the Android platform will bring us many benefits.

Me too 🙂


Dashboard A11y: Settings View - publisher logos should be an identifiable group

Bug Description
When using a keyboard on the settings view to tab through the page, you have to tab through every publisher logo available.
Expected Behaviour
This group of publisher logos should be a semantic list with a clear indicator of what it's of so that content can be skipped if desired.
Steps to Reproduce
Tab through the settings page of the dashboard, see that you must tab through all publisher logos, no identifiable group.
Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Fix should include karma test

Implementation Brief

Verified in QA


Deprecate old-Mongo-connector-implementation and switch to new-Mongo-connector

Based on QScript etc.

Since this is a mega-mega project, @jdegoes suggests breaking this down into more fine-grained tasks - @sellout can you please do that?

These apply upstream of the connector, but fix issues that make it difficult to generate plans that are competitive with pre-QScript Mongo.

[ ] apply PruneArrays to branches
[ ] fix provenance on autojoins (#1929, #1951)

These are Mongo-specific.

[ ] fix remaining failing integration tests (unless they also fail on old Mongo)
[ ] compare timing of planning+execution between old and new Mongo, improving the latter as necessary

I expect the switchover may happen before the last step is complete. At which point, we can identify queries that still underperform and make individual issues for them.

Tickets for PruneArrays work are #1794 and #1793.

Everything here is covered in other issues so I am closing this.
We need to have a separate discussion about what performance testing needs to be done on new mongo and when that should be done. Connector performance is affected by qscript compilation performance, so I expect most performance work (at least for new mongo) will be on qscript.


Remove so file and use Maven plugin to trigger make

The PR includes:

add maven antrun plugin to trigger make command automatically
untrack the so file from Git

This fixes  #22

+1


XML and UI element support - inclusive gateway

Support for:
<inclusiveGateway id="inclusiveGatewayId" name="Inclusive gateway name" flowable:async="true">
            <documentation>Inclusive gateway docs</documentation>
</inclusiveGateway>

Fixed with #61


Add @sftim as website maintainer

Add @sftim (me) as a website maintainer.
I'm a technical lead for SIG Docs; see OWNERS_ALIASES@a4a1d2f in k/community.

Relates to https://github.com/kubernetes/website/pull/20109

Relates to https://github.com/kubernetes/org/pull/1771

@sftim, What is the role of a website maintainer (what are the privileges)?

I want this access so that I can manage branches for localizations. There might be other access that this lets you do. I'm not sure that all of this is documented—it might well be—or that I know all the things that this role allows.

I'd like to know more about this role (hope there is documentation 😄 ).
/lgtm

@sftim You should already have write access to the website repo. @kbhawkey You should be in the website-maintainers team as well, as well as website-milestone-maintainers. Will you please open a PR and assign me or @jimangel or @kbarnard10?
After investigating and dredging through memory, the sig-docs-maintainers team is a historical artifact that was replaced by @kubernetes/sig-docs-en-owners and no longer has any permissions associated with it. The only place it's mentioned anymore is in this repo's netlify.toml, which is also out of date.
On that basis, I'm closing this PR and will open separate PRs in k/website to remove the website-maintainers team entirely and clean up netlify.toml, and a separate PR in k/org to update website-admins.
/close