enable HDF5 support of CGNS, but 'cgio_set_label:H5Aopen:open of node attribute failed'

Platform: Windows 10 17134.885
IDE: Microsoft Visual Studio 2015 version 14.0.25421.03
Compiler: Visual Studio 2015 (v140)/ Intel C++ Compiler 19.0
TK/TCL: ActiveTcl <IP_ADDRESS>09.2
nsis: 3.04
HDF5: 1.10.3 (build with the instruction of https://cgns.github.io/FAQs.html)
Cmake: 3.14.2

The build of HDF5 1.10.3
I followed the instructions from CGNS disabled slib and zlib building except fellow variables:
...
####      alternate toolsets       ####
# Use Microsoft C++ Compiler will cause lots errors.
set(CMAKE_GENERATOR_TOOLSET "Intel C++ Compiler 19.0")
...
####      fortran enabled      ####
# Just for some test, I know that CGNS have never used that.
set (ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_BUILD_FORTRAN:BOOL=ON")
### enable Fortran 2003 depends on HDF5_BUILD_FORTRAN
set (ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_ENABLE_F2003:BOOL=ON")
####      fortran disabled      ####
#set (ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_BUILD_FORTRAN:BOOL=OFF")
### enable Fortran 2003 depends on HDF5_BUILD_FORTRAN
#set (ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_ENABLE_F2003:BOOL=OFF")
...

I made a batch file include ctest -S HDF5config.cmake,BUILD_GENERATOR=VS201564 -C Release -VV -O hdf5.log, then build it successfully
It can be installed with the build result, the hdf5.log shows that the target platform version is 10.0.10586.0

Configure CGNS3.2.1:

Open cgns.sln with Visual Studio 2015 then modify all c++ project configurations like:

The build result shows that almost all project has been successfully built but failed a test named f2c_test
...
5>------ Build started: Project: f2c_test, Configuration: Release x64 ------
...
5>MSVCRT.lib(exe_main.obj) : error LNK2019: unresolved external symbol main referenced in function "int __cdecl __scrt_common_main_seh(void)" (?__scrt_common_main_seh@@YAHXZ)
5>D:\Programing\CGNS-3.2.1\build64\src\tests\Release\f2c_test.exe : fatal error LNK1120: 1 unresolved externals
...
...
========== Build: 25 succeeded, 1 failed, 4 up-to-date, 4 skipped ==========

Seems everything is good. Although one test case has failed, the others should not be affect.

But when I run all test case, something strange happens:
PS D:\Programing\CGNS-3.2.1\build64> ctest .
Test project D:/Programing/CGNS-3.2.1/build64
      Start  1: cgioc
 1/18 Test  #1: cgioc ............................   Passed    0.04 sec
      Start  2: elemtest
 2/18 Test  #2: elemtest .........................***Failed    0.05 sec
      Start  3: test_exts
 3/18 Test  #3: test_exts ........................***Failed    0.04 sec
      Start  4: test_goto
 4/18 Test  #4: test_goto ........................***Failed    0.04 sec
      Start  5: test_partial
 5/18 Test  #5: test_partial .....................***Failed    0.05 sec
      Start  6: test_ver31
 6/18 Test  #6: test_ver31 .......................***Failed    0.05 sec
      Start  7: test_multifam
 7/18 Test  #7: test_multifam ....................***Failed    0.05 sec
      Start  8: write_array
 8/18 Test  #8: write_array ......................***Failed    0.04 sec
      Start  9: write_bcdata
 9/18 Test  #9: write_bcdata .....................***Failed    0.05 sec
      Start 10: write_links
10/18 Test #10: write_links ......................***Failed    0.05 sec
      Start 11: write_rind
11/18 Test #11: write_rind .......................***Failed    0.05 sec
      Start 12: write_test
12/18 Test #12: write_test .......................***Failed    0.05 sec
      Start 13: write_zones
13/18 Test #13: write_zones ......................***Failed    0.04 sec
      Start 14: f2c_test
Could not find executable f2c_test
Looked in the following places:
f2c_test
f2c_test.exe
Release/f2c_test
Release/f2c_test.exe
Debug/f2c_test
Debug/f2c_test.exe
MinSizeRel/f2c_test
MinSizeRel/f2c_test.exe
RelWithDebInfo/f2c_test
RelWithDebInfo/f2c_test.exe
Deployment/f2c_test
Deployment/f2c_test.exe
Development/f2c_test
Development/f2c_test.exe
Unable to find executable: f2c_test
14/18 Test #14: f2c_test .........................***Not Run   0.00 sec
      Start 15: cgwrite
15/18 Test #15: cgwrite ..........................***Failed    0.05 sec
      Start 16: cgread
16/18 Test #16: cgread ...........................***Failed    0.05 sec
      Start 17: cgzconn
17/18 Test #17: cgzconn ..........................***Failed    0.05 sec
      Start 18: cgsubreg
18/18 Test #18: cgsubreg .........................***Failed    0.05 sec

6% tests passed, 17 tests failed out of 18

Total Test time (real) =   0.96 sec

The following tests FAILED:
          2 - elemtest (Failed)
          3 - test_exts (Failed)
          4 - test_goto (Failed)
          5 - test_partial (Failed)
          6 - test_ver31 (Failed)
          7 - test_multifam (Failed)
          8 - write_array (Failed)
          9 - write_bcdata (Failed)
         10 - write_links (Failed)
         11 - write_rind (Failed)
         12 - write_test (Failed)
         13 - write_zones (Failed)
         14 - f2c_test (Not Run)
         15 - cgwrite (Failed)
         16 - cgread (Failed)
         17 - cgzconn (Failed)
         18 - cgsubreg (Failed)
Errors while running CTest

When I tried run some test manualy:
PS D:\Programing\CGNS-3.2.1\build64\src\tests\Release> .\cgwrite.exe
cgio_set_label:H5Aopen:open of node attribute failed
PS D:\Programing\CGNS-3.2.1\build64\src\tests\Release> .\cgread.exe
cgio_get_label:H5Gopen:open of a node group failed
PS D:\Programing\CGNS-3.2.1\build64\src\tests\Release>

They can't write any data into cgns files:

To be continued...

I use Microsoft C++ Compiler recompiled HDF5 and CGNS again, but it has no difference between then (same error)
...
####      alternate toolsets       ####
# Use Microsoft C++ Compiler will cause lots errors.
#set(CMAKE_GENERATOR_TOOLSET "Intel C++ Compiler 19.0")
...
####      fortran enabled      ####
#set (ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_BUILD_FORTRAN:BOOL=ON")
### enable Fortran 2003 depends on HDF5_BUILD_FORTRAN
#set (ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_ENABLE_F2003:BOOL=ON")
####      fortran disabled      ####
set (ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_BUILD_FORTRAN:BOOL=OFF")
### enable Fortran 2003 depends on HDF5_BUILD_FORTRAN
set (ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_ENABLE_F2003:BOOL=OFF")
...


Per Package statistics some Client version appear blank.

For example:
http://nuget-preprod-0-v2gallery.cloudapp.net/stats/packages/Newtonsoft.Json?groupby=ClientVersion
the top client type appears blank. Shouldn't it at least say "unknown" if we don't know what it is?

In some other cases the unknown client version does actually appear as (unknown) for example:
http://nuget-preprod-0-v2gallery.cloudapp.net/stats/packages/dotnetrdf?groupby=ClientName

@xavierdecoster Do we have a dimension named "" ?

@maartenba no, there's no client dimension with clientname ''
This should say 'unknown' or 'other' or something.

Deployed a fix - version was set to blank in these cases, changed it to read "(unknown)" now.


Right-aligned QLineEdits have different borders enabled vs disabled

From https://github.com/UN-GCPDS/qt-material/issues/85#issuecomment-1432194497, the right border is not aligned correctly when the QLineEdit is enabled vs disabled.
See the two bottom controls:

This behaviour is defined here:
https://github.com/UN-GCPDS/qt-material/blob/5a99be3d6914cd5570282b29968aa39a7f8d1408/qt_material/material.css.template#L180
Removing the padding aligns the behaviour with disabled and non-disabled states. But the code to add padding based on the density value suggests me that maybe there was an issue with differents density scales... so again Yeison is the best person to apply the proper fix here.


Can't disable font ligatures

When I set the editor font to one that supports ligatures, I can't seem disable them via the editor.fontLigatures setting. Not even after a reload.

Yes, since we updated to the newer electron version, Chromium always shows ligatures, even without our enabling them.


⚙ Build side menu for 0xhabitat.org

Scope

build menu for 0xhabitat.org

https://www.figma.com/file/vYIiWTuvQsAyb821lkmu7J/Habitat?node-id=90%3A266
links to (0xhabitat.org + 0xhabitat.org/evolution + 0xhabitat.org/token
Deliverables

deployment when token bootstrap page is pushed

Funding Circle

deora/habitat

Bounty Owner/Gardener

@MaxKuck
Roles
bounty gardener: @MaxKuck (-)
bounty worker: @troggy (... DAI)
bounty reviewer: name / share
Gardener checklist

[ ] I have attached one of the size- labels to this bounty
[ ] I have submitted this bounty via http://bounty.leapdao.org/viewform
[ ] I have added this bounty to bounty board here https://github.com/orgs/leapdao/projects/6

[x] build draft in figma board

love it @troggy nice work! let's talk on the daily about compensation :)

Peeps, requesting a 200 DAI worth of HBT for this work :—)


Return a set of well defined errors from validateCovidPassport

Currently Will is just returning false if a check fails.
For example in the spec it says
Check if the payload received from the QR Code begins with the prefix NZCP:/, if it does not then fail.
We should do something like
return { success: false, code: 100, message: "Payload must begin with \"NZCP:/\""}

Hey, I'd like to work on this

We're happy with the structure we landed on and now are just working though the spec.


Application - Long Form Question Word Counter

Description
Lauren would like to have @yonikid15's word counter added to the long form questions on the job application so that applicants have an idea of how long their answers should be. For now, she'd like it set to 250 words for each question.
I'm not sure what the status is on the Word Counter component right now - but it would be great to get this implemented when it's finished.
Tagging @laurenoncoffee just in case she wants to add context.

I'm gonna put this in the Icebox as I think it's something we should build into Jerbo's redesign of the application process (timeline). Happy to re-prioritise at your suggestion @joshdrink @laurenoncoffee


Description is losing focus on entry

Description input is losing focus, when you try to enter anything in it.
Only enters single character and then drops the focus...

Apparently this is the issue with placeholder option on SimpleMDE component.
Removing it, brings back editing to normal.
Replicated here - https://codesandbox.io/s/simplemde-placeholder-problem-sts9rd - to ensure it's not Playground issue, but rather something on component side.

@kerrywei can we add this issue to current sprint?

@MaxStalker yeah I agree this is an issue that severely impacts user experience, deserving a fix asap
any chance we can preserve the placeholder, or at least hint the user in some way to indicate the markdown syntax is supported in the description text field?

@MaxStalker to provide status update

@MaxStalker  reminder: we want to mark a task closed when the changes are released

@kerrywei understood.
I will keep the issue open until we confirm it's properly deployed to production server. Sorry for this mishap :bow:

@MaxStalker I tried production playground, the losing focus issue is still there. we should plan a release asap to address this

Fix for this one is currently building and will be deployed on staging in next 5 minutes.
Once confirmed we would push it to production asap.


Migrate apache tomcat importer

@johnmhoran please provide importer and improver logs for this importer.

@TG1999 What are importer and improver logs?  I don't think I did this for the archlinux, apache_httpd, apache_kafka or postgresql importers.

@johnmhoran please run the importer you have migrated and also run the improver over the data you migrated, and show the logs that you got by running the importer and the improver.

Thanks @TG1999 .  I'll do a rebase/--force first.  Shall I paste the terminal results of running the importer/improver as a comment here?

@johnmhoran you can attach a txt file in comments here

Excellent -- will do.  👍

@TG1999 After running the import command I received the following error:
(venv) Fri Jan 20, 2023 11:34 AM  /home/jmh/dev/nexb/vulnerablecode jmh (970-migrate-apache-tomcat-importer)
$ ./manage.py import vulnerabilities.importers.apache_tomcat.ApacheTomcatImporter
Importing data using vulnerabilities.importers.apache_tomcat.ApacheTomcatImporter
Traceback (most recent call last):
  File "/home/jmh/dev/nexb/vulnerablecode/vulnerabilities/management/commands/import.py", line 60, in import_data
    ImportRunner(importer).run()
  File "/home/jmh/dev/nexb/vulnerablecode/vulnerabilities/import_runner.py", line 44, in run
    count = process_advisories(advisory_datas=advisory_datas, importer_name=importer_name)
  File "/home/jmh/dev/nexb/vulnerablecode/vulnerabilities/import_runner.py", line 57, in process_advisories
    data.summary = data.summary.replace("\x00", "\uFFFD")
AttributeError: 'generator' object has no attribute 'summary'
Failed to run importer vulnerabilities.importers.apache_tomcat.ApacheTomcatImporter. Continuing...
CommandError: 1 failed!: vulnerabilities.importers.apache_tomcat.ApacheTomcatImporter

(venv) Fri Jan 20, 2023 11:39 AM  /home/jmh/dev/nexb/vulnerablecode jmh (970-migrate-apache-tomcat-importer)
$

@TG1999 I'm attaching vcio-apache_tomcat-import-and-improve-2023-01-20.txt containing the terminal output from
running the import and improve steps.  Please note that while both were successful, the import step output included this item:
'RC1' is not a valid SemverVersion InvalidVersion("'RC1' is not a valid <class 'univers.versions.SemverVersion'>")

vcio-apache_tomcat-import-and-improve-2023-01-20.txt

LGTM!


fixed debian installation

howdy,
I fixed and cleaned up the debian aspects of installing with.namespace.
cheers,
Evan

Thanks for the review.
re: k5start
I think there was the notion that krb5 and afs would not be in the core of with.namespace, but could be added by some sort of extension system (or simply invoked by the user with whatever specific incantations they need).   It was removed from the main 'with', but still existing in the 'with_exec.lua' library.   Perhaps the change should include the deletion of that function.
re: debian/rules
This package uses debhelper to get itself built.  debhelper does a bunch of things based on the files in your debian folder.  You can inject behavior with 'override_dh_buildstepname'
In this case, we are telling debhelper to chmod exec_with_namespace with setuid.  Also telling dh_installinit to install the with upstart script.
This was all stuff that with's old package 'xxx-utils' did, but under a different debian build system.  It was lost in the extraction.


Can i use TPM with this config?

hey, first THANKS for the great config.
Issue:
I am looking for a way to use some tmux plug ins, I would love to mange them via TPM.
What is the best practice while using this config?
especially looking for a way to use https://github.com/tmux-plugins/tmux-resurrect
regards

Hello @fwarmuth 👋
I'm not a TPM user myself but as far as I remember people got it working with TPM. Search the issues queue.
I'll think about adding instructions but as I don't use TPM myself, I don't want this configuration make people believe I'm the good person to answer TPM related questions.

One issue that discusses tpm and oh-my-tmux is #228.  I followed that approach and it works.
I am very new to tpm.  I had assumed it would just work with oh-my-tmux, but eventually figured out it didn't and found this issue  I agree that something in the README may be useful to others.

Sourced from an issue on TPM page:

TPM has a known issue when using a custom config file with the -f option. The solution is to use alternative plugin definition syntax. Here are the steps to make it work:
remove all set -g @plugin lines from tmux config file
in the config file define the plugins in the following way:
 # List of plugins
 set -g @tpm_plugins '          \
   tmux-plugins/tpm             \
   tmux-plugins/tmux-sensible   \
   tmux-plugins/tmux-resurrect  \
 '

 # Initialize TMUX plugin manager (keep this line at the very bottom of tmux.conf)
 run '~/.tmux/plugins/tpm/tpm'

Reload TMUX environment so TPM is sourced: $ tmux source /path/to/my_tmux.conf
The plugins should now be working.

💁♂ There is now a tpm branch up for testing that integrates Oh My Tmux! with TPM, see .tmux.conf.local for instructions

Closing as duplicate of #61


Breakpoint in collapsed code is visible, but not usable

VSCode Version: 1.11.0-insider
OS Version: Windows 10 Enterprise

Steps to Reproduce:
Given:
function f() {
    return 0;
}

Add a breakpoint at the return statement.
Collapse the function.
The breakpoint is still visible.
Right-click on it.
This gives me an option to "Add breakpoint". But I want to remove it.

Duplicate of https://github.com/Microsoft/vscode/issues/4812


NIFI-8113 Adding persistent status history repository backed by embed…

This would be a proposal for having persistent status history for both component and node level metrics. During the implementation I tried to balance between introducing some flexibility in usage and supporting the existing behaviour. As an end result, I did split the component status repository (now it is StatusRepository) into one which is responsible for component metrics and an other responsible for node level metrics. They might be configured independently on some level (like it is possible to store data into a different storage or for a different amount of time window), but in order to support the previous configuration, there is a facade for them which provides the same composite service as it did before (component and node).
As for code organisational part I worked with three concepts: repository is the top level entity, provides the service for the clients (FlowController, etc.). In general, this is the same as before, only split into two parts: node and component. The storage classes are part of the repositories and are responsible for manage the details of a given type, like processor status, node status, etc. Finally the WriterTemplates and ReaderTemplates are merely helpers exist to deal with QuestDB API calls.
Some remarks on design decisions:

I kept the name for VolatileComponentStatusRepository, but the actual repositories are using the prefix InMemory to show more contrast with QuestDB
The PR contains a small fix on ProcessGroupStatusDescriptor#calculateTaskMillis(): previously every subsequent level makes a nano->milli conversion, which accumulates, reducing the task time in children groups into 0. Now this should be fixed (The QuestDB tests are implicitly proves this as well)
Configuration contains the new parameters but they are commented out. At this point I think, the VolatileComponentStatusRepository should be kept as default
The implementation depends on the latest of the 4.X version of QuestDB. Currently QuestDB is at 5.0.5, but the 5.X line depends on Java 11. There are some non vital features in 5.X (all_tables request, dropping partition using WHERE closure), but these are not unavoidable for us.

Thank you for investing your time to my PR!
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[x] Have you verified that the full build is successful on JDK 8?
[x] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[x] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

I do abandon this review. Based on @markap14 's great comments, I simplified the configuration which resulted a more clean codebase as well. This comes with some changes I reverted and in order to keep the review readable I decided to create a new. All comments in this review are answered or aimed here.

Please find the predecessor PR here: PR 4839


The skills for ++ / -- don't compile

I'm using MSVC, and it complains it can't convert the underlying type to the named type. Looking at the definition, I see it really is doing that, which seems wrong.

Not sure if the same, but on g++, I can't use these because of ambiguities:
using U = NamedType<unsigned, struct UTag, Arithmetic>;
for (U i = U(0); i < U(10); i++)
    ;

In function 'int main()':
error: request for member 'operator++' is ambiguous
|     for (U i = U(0); i < U(10); i++)
|                                  ^~
In file included from ..../NamedType/include/NamedType/named_type.hpp:5,
..../NamedType/include/NamedType/underlying_functionalities.hpp:43:26: note: candidates are: 'constexpr T fluent::PostIncrementable<T>::operator++(int) [with T = fluent::NamedType<unsigned int, main()::UTag, fluent::Arithmetic>]'
|     FLUENT_CONSTEXPR17 T operator++(int)
|                          ^~~~~~~~
..../NamedType/include/NamedType/underlying_functionalities.hpp:29:27: note:                 'constexpr T& fluent::PreIncrementable<T>::operator++() [with T = fluent::NamedType<unsigned int, main()::UTag, fluent::Arithmetic>]'
|     FLUENT_CONSTEXPR17 T& operator++()
|                           ^~~~~~~~

It appears for some reason that it is struggling to determine whether it should call the pre-increment or post-increment version. I'd have to guess that it can resvole to a base class through either route, and that it must find one specific class to try and call the increment on before then picking between the pre- and post-increment version, asI guess the extraneous int parameter is a little but of a hack in the language, that may need normal overload rules to be carried out first. But it's just a guess.
For now, I can work around using i += U(1)


Dashboard refinements:

Moved title and hint into main column instead of above both columns
Added a block for messages

Before:

After:


Drop support for EOL Python 2.4-2.6 and 3.2-3.3

Python 2.4-2.6 and 3.2-3.3 are EOL and no longer receiving security updates (or any updates) from the core Python team.

Version
Release date
Supported until

2.4
2004-11-30
2008-12-19

2.5
2006-09-19
2011-05-26

2.6
2008-10-01
2013-10-29

3.2
2011-02-20
2016-02-27

3.3
2012-09-29
2017-09-29

Source: https://en.wikipedia.org/wiki/CPython#Version_history
They're also little used. Here's the pip installs for pyasn1-modules from PyPI for November 2018:

category
percent
downloads

2.7
62.12%
4,102,324

3.6
22.14%
1,461,950

3.5
9.71%
641,265

3.7
3.67%
242,367

3.4
2.02%
133,543

null
0.30%
19,961

2.6
0.04%
2,477

3.3
0.00%
281

3.8
0.00%
31

Total

6,604,199

Source: pypistats python_minor --last-month pyasn1-modules

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   98.87%   98.87%
=======================================
  Files          19       19
  Lines        2845     2845
=======================================
  Hits         2813     2813
  Misses         32       32

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 c4f600a...46ca910. Read the comment docs.

Conflicts resolved.

Thank you!


z3 translation bug

iv = intvar(1,9, shape=2)
Model((iv[0] == iv[1])).solve("z3") # works
Model(~(iv[0] != iv[1])).solve("z3") # should be identical, but triggers z3 exception

ah, I think it literally tries to post (iv[0] != iv[1]) == 0, which is our way of expressing negation. Should catch the ==0 and post it as negation instead of an equality?

That seems to work, I made a pull request for it (#189).
Had to make sure that we still keep some constraints as lhs == 0 if lhs is an intvar for example.


Filter search Functinality

Hi team,
I went through this application Its quite good but Its will more advanced and better working and more adaptable if it will come with log search filter with multple index or entries . One issue i faced that most of the time screen went ot 'loading' mode while scrolling or putting filter.
Feature added which which will provide advantages and hoping for improvement specially from frontend side on console level:

Client Search individual logs
Logs must be expand in each line entries instead of whole console log 'wrap'
Logs entries must be dyanmically fit in browser row of console instead of overalapping each other . Please check attached document.

There is still lot of scope of improvement is there as far devOps cloud prespective I can see potentials because I remember similar application I want to develop quite 1 years ago.
If team want to help I will definately help with respect to improvement and testing application for live world
I Hope ans wish may be next release will come up with improvement.

Thanks for the feedback!

Client Search individual logs

We're actively working on search and we hope to have a version with search enabled soonish (~2 months). It would be great to get your feedback when it's ready.

Logs must be expand in each line entries instead of whole console log 'wrap'

Did you try toggling the "Wrap" option in the settings drop down (click on the gear icon in the upper-right corner). Let me know if that addresses your issue.

Logs entries must be dyanmically fit in browser row of console instead of overalapping each other . Please check attached document.

Based on your screenshot, it looks like there's a bug related to your setup that I haven't seen myself. This is what the console should look like with "Wrap" enabled:

Here are some questions for you:

Which browser are you using?
Do you see the same problems when you view the demo at https://www.kubetail.com/demo?
With regards to the "loading" screen, what is the ping time to your remote cluster?
Also how large are the log files you're viewing and what is the event rate?

Here's an update related to search: https://github.com/kubetail-org/kubetail/issues/13#issuecomment-2370690081


bug: log class properties in JS/TS

Did you check docs and existing issues?

[X] I have read all the plugin docs
[X] I have searched the existing issues
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.10.2
Operating system/version
MacOS 15.1.1
Describe the bug
On JS/TS code, I can't insert a log variable for class properties.
Steps To Reproduce
taking this code, with | meaning the cursor position:
class Foobar {
  foo = 1;

  bar() {
    this.f|oo = 2;
  }
}

if I hit glv, it doesn't insert any log
Expected Behavior
it should insert a log variable like:
class Foobar {
  foo = 1;

  bar() {
    this.f|oo = 2;
    console.log("this.foo", this.foo)
  }
}

Please check if this works

works great, cheers!


Problem in using my own dataset

Description
I was unable to use my own dataset. I was trying to use a dataset having english-hindi translation data. I updated translation_ende.py , text_problems.py and generate_utils.py. I tried to hardcode these programs in order to download my own dataset . Yet, when I executed it , it was downloading the previously existing dataset.
I have also written a python script in data_generators. No change in the output was encountered even though I created a class and registered it as a subclass of Problem. I have already read the previous issues on this topic. However, I was unable to get a solution for this kind of problem.
...
Environment information
OS: <your answer here>

$ pip freeze | grep tensor
# your output here

$ python -V
# your output here

For bugs: reproduction and error logs
# Steps to reproduce:
...

# Error logs:
...

In my case, my work is english-to-chinese translation, my registered problem:

ENZH_RAW_DATASETS = {
    "TRAIN": "raw-train.zh-en",
    "DEV": "raw-dev.zh-en"
}

def get_enzh_raw_dataset(directory, filename):
    train_path = os.path.join(directory, filename)
    if not (tf.gfile.Exists(train_path + ".en") and
            tf.gfile.Exists(train_path + ".zh")):
        raise Exception("there should be some training/dev data in the tmp dir.")
    return train_path

@registry.register_problem
class TranslateEnzhSub50k(translate.TranslateProblem):

    @property
    def approx_vocab_size(self):
        return 50000

    @property
    def source_vocab_name(self):
        return "vocab.enzh-sub-en.%d" % self.approx_vocab_size

    @property
    def target_vocab_name(self):
        return "vocab.enzh-sub-zh.%d" % self.approx_vocab_size

    def get_vocab(self, data_dir, is_target=False):
        vocab_filename = os.path.join(data_dir, self.target_vocab_name if is_target else self.source_vocab_name)
        if not tf.gfile.Exists(vocab_filename):
            raise ValueError("Vocab %s not found" % vocab_filename)
        return text_encoder.TokenTextEncoder(vocab_filename, replace_oov="UNK")

    def generate_samples(self, data_dir, tmp_dir, dataset_split):
        train = dataset_split == problem.DatasetSplit.TRAIN
        dataset_path = (ENZH_RAW_DATASETS["TRAIN"] if train else ENZH_RAW_DATASETS["DEV"])
        train_path = get_enzh_raw_dataset(tmp_dir, dataset_path)
        return text_problems.text2text_txt_iterator(train_path + ".en",
                                                    train_path + ".zh")

    def generate_encoded_samples(self, data_dir, tmp_dir, dataset_split):
        generator = self.generate_samples(data_dir, tmp_dir, dataset_split)
        encoder = self.get_vocab(data_dir)
        target_encoder = self.get_vocab(data_dir, is_target=True)
        return text_problems.text2text_generate_encoded(generator, encoder, target_encoder,
                                                        has_inputs=self.has_inputs)

    def feature_encoders(self, data_dir):
        source_token = self.get_vocab(data_dir)
        target_token = self.get_vocab(data_dir, is_target=True)
        return {
            "inputs": source_token,
            "targets": target_token,
        }

my trainsets are separated by space:
# raw-train.zh-en.en
For greater sharpness, but with a slight increase in graininess, you can use a 1:1 dilution of this developer.
# raw-train.zh-en.zh
为 了 更 好 的 锐 度 ， 但 是 附 带 的 会 多 一 些 颗 粒 度 ， 可 以 使 用 这 个 显 影 剂 的 1 ： 1 稀 释 液 。


Be sure to call youtube-dl with the --verbose flag and include its complete output.

Checklist

[ ] I'm reporting a broken site support issue
[x ] I've verified that I'm running youtube-dl version 2021.12.17
[ ] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[x ] I've read bugs section in FAQ

Verbose log

nextcord.ext.commands.errors.CommandInvokeError: Command raised an exception: DownloadError: ERROR: Unable to extract yt initial data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

WRITE DESCRIPTION HERE
My youtube-dl now working, I reinstalled it and my version is 2021.12.17

I guess "...not working..."?
Review #30839 and #32499.

Same issue
WARNING: [youtube:tab] unable to extract yt initial data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

with version
yt-dlp -U
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>


ImportError: cannot import name 'get_chrome_web_driver' from 'newsfetch.helpers'

`Python 3.9.1 (default, Dec 13 2020, 11:55:53)
[GCC 10.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

from newsfetch.google import google_search
Traceback (most recent call last):
File "", line 1, in
File "/home/zeka/ZEKA/pythonProjects/news-fetch/newsfetch/google.py", line 1, in
from newsfetch.helpers import (get_chrome_web_driver, get_web_driver_options,
ImportError: cannot import name 'get_chrome_web_driver' from 'newsfetch.helpers' (/home/zeka/ZEKA/pythonProjects/news-fetch/newsfetch/helpers.py)
from newsfetch.news import newspaper
news = newspaper('https://www.bbc.co.uk/news/world-48810070')
File "", line 1
news = newspaper('https://www.bbc.co.uk/news/world-48810070')
IndentationError: unexpected indent
from newsfetch.google import google_search
Traceback (most recent call last):
File "", line 1, in
File "/home/zeka/ZEKA/pythonProjects/news-fetch/newsfetch/google.py", line 1, in
from newsfetch.helpers import (get_chrome_web_driver, get_web_driver_options,
ImportError: cannot import name 'get_chrome_web_driver' from 'newsfetch.helpers' (/home/zeka/ZEKA/pythonProjects/news-fetch/newsfetch/helpers.py)
`

I could not find get_chrome_web_driver, get_web_driver_options, set_automation_as_head_less, set_browser_as_incognito, set_ignore_certificate_error in the newsfetch.helpers

I have updated the package please check once. I am using this package for chrome driver selenium purpose https://pypi.org/project/chromedriver-binary-auto/
new version - https://pypi.org/project/news-fetch/

I have updated the package please check once. I am using this package for chrome driver selenium purpose https://pypi.org/project/chromedriver-binary-auto/
new version - https://pypi.org/project/news-fetch/

google.py
from newsfetch.helpers import (get_chrome_web_driver, get_web_driver_options,                                set_automation_as_head_less,                                set_browser_as_incognito,                                set_ignore_certificate_error)
I can't see where get_chrome_web_driver and other function are defined
helpers.py doesn't contains them

google.py
from newsfetch.helpers import (get_chrome_web_driver, get_web_driver_options,                                set_automation_as_head_less,                                set_browser_as_incognito,                                set_ignore_certificate_error)
I can't see where get_chrome_web_driver and other function are defined
helpers.py doesn't contains them

Thanks for pointing it out. I have now added the function in helpers.
Remember I have just a simple version of the google search page scraper if more requests noted. Google will throw a captcha.
The only thing we can do to solve the captcha uses VPN.
Install the latest version - https://pypi.org/project/news-fetch/
This package includes chromedriver-binary, please try to install chromedriver-binary that matches your current chrome browser version

Thanks for pointing it out. I have now added the function in helpers.
Remember I have just a simple version of the google search page scraper if more requests noted. Google will throw a captcha.
The only thing we can do to solve the captcha uses VPN.
Install the latest version - https://pypi.org/project/news-fetch/
This package includes chromedriver-binary, please try to install chromedriver-binary that matches your current chrome browser version


Add comment about types

Description
This adds a comment about available types for jest-plugin-context.

Great, thank you! Also thanks for creating the types 😄


Update disclaimer

The wording on the disclaimer is out of date. Please change to the following:
"This software has been approved for release by the U.S. Geological Survey (USGS). Although the software has been subjected to rigorous review, the USGS reserves the right to update the software as needed pursuant to further analysis and review. No warranty, expressed or implied, is made by the USGS or the U.S. Government as to the functionality of the software and related material nor shall the fact of release constitute any such warranty. Furthermore, the software is released on condition that neither the USGS nor the U.S. Government shall be held liable for any damages resulting from its authorized or unauthorized use."
From: https://www2.usgs.gov/fsp/fsp_disclaimers.asp#5
On code.usgs.gov, this is being required for public repositories in a "DISCLAIMER.md" file.

Thanks!


Doesn't build on OS/X

/Applications/Xcode.app/Contents/Developer/usr/bin/make depend && /Applications/Xcode.app/Contents/Developer/usr/bin/make _all
make[1]: *** No rule to make target crypto/include/internal/cryptlib.h', needed by crypto/aes/aes_ige.o'.  Stop.
make: *** [all] Error 2
hornenj@home:~/src/openssl$ find . -name cryptlib.h
./include/internal/cryptlib.h
hornenj@home:~/src/openssl$ ls crypto/include/internal/
__DECC_INCLUDE_EPILOGUE.H	engine.h
__DECC_INCLUDE_PROLOGUE.H	err_int.h
aria.h				evp_int.h
asn1_int.h			md32_common.h
async.h				objects.h
bn_conf.h			poly1305.h
bn_conf.h.in			rand_int.h
bn_dh.h				sha.h
bn_int.h			siphash.h
bn_srp.h			sm3.h
chacha.h			sm4.h
cryptlib_int.h			store.h
ctype.h				store_int.h
dso_conf.h			x509_int.h
dso_conf.h.in

I ran this command, then 'make' and 'make test' both worked.
ln include/internal/cryptlib.h crypto/include/internal/

Yes,you have to follow the instructions.  Closing this issue.

Also you're not even calling the correct make. Don't access it from within the Xcode.app.

I typed 'make' at the command line, I did not "access from within the Xcode.app" as you suggest.

Please read the readme before filing issues.
https://wiki.openssl.org/index.php/Compilation_and_Installation

Sorry my ESP chip didn't work.  There is no mention of this important stage on the README at https://github.com/openssl/openssl.

Oh!  Could it be that you placed new source in a spot where you had files from an older build remaining?  See, cryptlib.h moved in august last year (commit 67dc995eaf538ea309c6292a1a5073465201f55b), so if you still have dependency files from that time lying around, the error you get makes sense, although unfortunate.
Please try make clean before make, and if that doesn't help, try this:
find /src/openssl -name '*.d' | xargs rm


Add embark theme

Hey @skbolton, here's the PR!

Thanks @kirontoo I'll update the theme homepage with a link to this repo shortly


Fix sample count on OpenXR composition layers

Fixes https://github.com/godotengine/godot/issues/91935
I've tested this with SteamVR and the Meta Quest 3 via Quest Link.

Thanks!


Tests: introduce hax::expect_failure and hax::expect_success attributes

Currently, a test is exactly a package.
It's not nice, a test should be any rust item in any rust package. (possibly a module)
Proposal
introduce hax::expect_failure and hax::expect_success attributes, which takes the same arguments as https://github.com/hacspec/hacspec-v2/tree/54f84b9ad593053135d2d1d3d22d7925a8a225b0/test-harness#test-specifications, that is a (possibly empty) issue id, an error code, a broken flag, a list of backends / linter.
Roadmap
~Requires attributes on items #49.~ (Hax now exposes attributes on items :heavy_check_mark:)
The infrastructure for attributes is there, adding those attributes should be easy. Some documentation about adding attributes exists there: https://github.com/hacspec/book/blob/main/src/contributing/libraries_macros.md
A subsequent task would then be to update the test harness to make use of those attributes.

Still something I want to do


what format is this<PHONE_NUMBER>80505152505700

why It was changed in EPC tag E2801191A50300061A148A908 as<PHONE_NUMBER>80505152505700. and what format is this<PHONE_NUMBER>80505152505700

Hello  everyone,
I want to let you know that I currently do not have access to a Zebra scanner device. However, I've developed a Zebra scanner integration and made it available publicly here.
I encourage you to explore the project, contribute, and help enhance its functionality. Your contributions will not only help improve the project but also benefit other developers in the community.
Thank you for your support and collaboration!
Best regards,
Sanket Warule


Please clarify how can i run the file that is included in package contents

Previously i used the run file command without path and included file into the package. After some update, when custom update interface changed, this seems to have stopped working, if i just use the file name.

Please clarify how can i run the file that is included in package contents.

Seems this is working, i just failed to check correctly

Seems this is working, i just failed to check correctly


Failure to find com.github.eirslett:TeacherInfo-maven-plugin:jar:1.6 in https://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not   be reattempted until the update interval of central has elapsed or updates are forced

com.github.eirslett:TeacherInfo-maven-plugin:jar:1.6 that looks like sth that might have been taken from your local settings.xml


Any command freeze in WSL2 and Windows 10

Which version and edition of Flyway are you using?
8.3.0 and 8.4.1
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Yes, same problem in 8.4.1
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using? (Type & version)
H2
Which operating system are you using?
WSL2 (ubuntu 20.04) on Windows 10
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
I'm trying to do the "First steps" tutorial, over a local h2 database.
The file ./conf/flyway.conf:
flyway.url=jdbc:h2:file:./foobardb
flyway.user=SA
flyway.password=

And in ./sql directory, the first migrate file called 'V1__Create_person_table.sql'. Like the tutorial.
Then, i try to do first migration:
./flyway migrate
What did you expect to see?
Something :)... a simple migration.
What did you see instead?
Nothing.
The command is launched, but nothing is show in the console. No error, or log in the screen.  Only the process wait without end. The "-X" flag doesn't help.
But i use a wrong command, the error is show. Or if i use the help, it shows in the screen. But command like "migrate", "info", not works.
In powershell (with the windows version), no problem at all, migration works.

We've been unable to reproduce this. Could you provide a minimal reproduction project?

My scenario:

Windows 10 (not admin user) with WSL2 with root rights.
Java local version (i think that flyway use its local jre)
I have followed the download instructions: wget -qO- https://repo1.maven.org/maven2/org/flywaydb/flyway-commandline/8.4.1/flyway-commandline-8.4.1-linux-x64.tar.gz | tar xvz
I have modified the file ./conf/flyway.conf adding in the last lines:

[... original file ...]
flyway.url=jdbc:h2:file:./foobardb
flyway.user=SA
flyway.password=

Then add in ./sql directory, the file V1__Create_person_table.sql, with the sql:

create table PERSON (
    ID int not null,
    NAME varchar(100) not null
);

Then i use in the shell: ./flyway info -X. Black screen, nothing happens:

But i only use: ./flyway, i had the following output:

I dont know which other info you need, but ask me for anything.
I could generate more debug log, but i dont know how.
Thank you very much!!!

Hi @Tab3r,
Are you running flyway on a machine which has internet access? Additionally, how long have you been leaving flyway before cancelling the command?

The machine has internet access. The wildfly files are outside de wsl "virtual" storage (/mnt/...).
I have waited more than 1 hour...
The point is if i launch in powershell (even the linux version), it works ok!...
How can debug this?...

Could you try running flyway again, but with the -community and -skipCheckForUpdate flags?
So something like:
./flyway info -X -community -skipCheckForUpdate

Closing this issue until further activity

Using the parameter: "-skipCheckForUpdate", all commands works as expected!.
But, why it's happen? WSL is blocked by windows?
Thank you very much for your help!


Chore/integ 1426 adjust landing and ref content

this merges some changes that were in Tony's branch with some content/recommendations that Rob supplied that is discussed in INTEG-1426 (internal link)

Do we want to add the scheme to all the examples? Doesnt have to be in this PR. Or maybe just "Replace RequestUrl with https://

Do we want to add the scheme to all the examples? Doesnt have to be in this PR. Or maybe just "Replace RequestUrl with https://


template uses non-existing app.name when app image is used

When an app image is used via proxy.specs.logo-url shinyproxy throws a (really long) exception. Among the many lines one finds:
Caused by: org.thymeleaf.exceptions.TemplateProcessingException: Exception evaluating SpringEL expression: "app.name" (template: "index" - line 57, col 10)

Indeed, at
https://github.com/openanalytics/shinyproxy/blob/19b65556af99823497855c056d2754335bb9961f/src/main/resources/templates/index.html#L57
app.name is used, while without an image app.id is used.

Good catch, thanks for bringing this up.
A fix is committed in develop:
https://github.com/openanalytics/shinyproxy/commit/2cf408eae855427af1736c20e459d0ef7cfdf618


QUESTION: unclosed <div>?

Hello,
I was wondering whether this <div> should actually be </div>. It renders properly, but I was just curious.
https://github.com/poldracklab/fmriprep/blob/ce01c27e1fd26d7550468e0180ae65fbb2b3b8ed/fmriprep/viz/report.tpl#L58

Looks like it. Probably renders properly because the </nav> implicitly closes both <div>s. Mind submitting a quick fix?


Fix resetting IPython custom exception hook

Closes gh-2395
Ping @DSLituiev - this should fix the problem you showed me yesterday.

Looking at the docstring for set_custom_exc, I wonder why None was there in the first place. Was it required in older versions of IPython?

It's possible, but I don't think that code has changed for some time.

@takluyver, thanks a lot for taking a look at this one :-)
@blink1073, please take a look at this one, given that it was you who added this code.

Looks good to me.


Implement CLI client

The CLI client connects to the API (provided by the api plugin) and issues requests based on the arguments it receives.
In order to be useful, the API needs to provide a "directory" of actions that the client can query and display to the user with a brief description. It might also be wise to have only specific actions exposed via the API using another property. (The "ui" property should probably be renamed to "menu" or something.

https://github.com/OAI/OpenAPI-Specification ?

That's definitely a possibility. I'll see how the time goes as we approach the release (which is not even close to being ready yet).

Unfortunately, there is no tray icon in Fedora. :(
I have no idea how to interact with nitroshare running in background. Making it both GUI and CLI would help a lot, especially with debugging discovery problems.


Move files into (sub)folders

Some folders where pretty large.
Moved a lot into (sub) folders. Only moves (+ Resharper config), 0 code changes
for example:

and

and

Oh so awesome.  When I was working on the LogManager change I noted in my head it would be nice to do this.


Let Link capture state variables

Currently, state variables in a link (e.g. c and h in LSTM) is defined by the concrete link class, and there's no way for base Link class to know about them.
I propose adding a way to let Link know about its state variables.
Benefits:

Allow serialization of link (See https://github.com/pfnet/chainer/issues/2743#issuecomment-301985525)
For hierarchical graph visualization (See #2748), this is needed to visualize RNN links  (like LSTM), because otherwise there would be no way to determine subgraph boundary.

Proposal:

Add Link.get_state() method in which the concrete class returns the current state variables.

bump

In an internal discussion, we discussed that it may not be appropriate to introduce a new concept in general link. We should begin with  a specific feature of LSTM or some other adhoc way.

Some details about the above comment:

In order to settle in better interface, it's better to start with ad-hoc implementation first and generalize only when it's found useful.
In the ad-hoc implementation, specific applications (such as graph visualization, static graph optimization, link hooks, etc.) can test for the type of subject links with e.g. isinstance and change the way they handle the link according to the result.

Currently we have no plan to introduce a general mechanism to support this, as stated above.


FIxes #64 - wires Rest, Http, Metric modules through MacWire.

@aquamatthias I did it the classical way, meaning with classes and arguments, instead of traits. I also have the cake pattern trait version. I would upload it in a separate branch, so we could compare it directly. I am still leaning towards cake, but that is just a personal matter.

@gkleiman @meichstedt also interested in your opinion

@gkleiman @meichstedt I'll provide the cake pattern version in a separate branch today.


Fix README.md

Hi.
I think this is great component.
Modified the contents of README.md a little.
Please merge if you like.

Duplicate syncHeaderScroll
Indents were mixed with space and tab

I recommend using editorconfig for this project.

Thanks!


Update app.ini.sample

Give further information on hyperlink rendering in Markdown
Partially addresses #8478

Codecov Report

Merging #8498 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #8498      +/-   ##
=========================================
+ Coverage   41.89%   41.9%   +<.01%
=========================================
  Files         507     507
  Lines       66506   66506
=========================================
+ Hits        27865   27868       +3
+ Misses      35063   35060       -3
  Partials     3578    3578

Impacted Files
Coverage Δ

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

models/repo_indexer.go
68.21% <0%> (+1.93%)
: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 ba201aa...324943b. Read the comment docs.

Make lg-tm work


How to reverse the order of the events

How to reverse the order of the events?

Sorry, it's not supported yet, but I will add function to reverse the order. Move into the backlog.

Hello @MohamSayed , the plugin has been updated to include sort event support. Please upgrade to version 0.0.3 and review the new functions.


Warning: owner-based and parent-based contexts differ (values: [object Object] vs [object Object]) for key (store) while mounting Connector (see: http://fb.me/react-context-by-parent)

I'm getting the error since the router was implemented. Any thoughts?
Warning: owner-based and parent-based contexts differ (values: [object Object]vs[object Object]) for key (store) while mounting Connector (see: http://fb.me/react-context-by-parent)

In my understanding, in the following example, the parent of the connecter (of myComponent) is reduxRouteComponent and the owner of the connecter is Router.  Both Provider and reduxRouteComponent provide store as context, which are different instances.
Thus with React <=0.13, we can not use store extended by reduxRouteComponent, which enables us to use transition_to action. (Actually, it seems there are some ways to use transiton to action.)
<Router>
  <Route component={reduxRouteComponent(store)}>
    <Route component={myComopnent}>
    </Route>
  </Route>
</Router>

getting the same thing. is this just going to be fixed by bumping to react 0.14?

Same issue here. Does this mean we are forced to use 0.14 beta?

Same issue here. I removed the line which adds the middleware to the dispatcher, in order to remove the warning. Im now making transitions like normal react router.

This will be fixed in the next version: https://github.com/acdlite/redux-react-router/tree/next


Support all auto inject strategies

Allow Dry::AutoInject options to be passed to Dry::Component::Container#injector. This allows the underlying auto_inject object to be configured as the user requires, while still retaining dry-component's desired behaviour of lazily loading dependencies.
The immediate benefit of this is that a user can now set up an injector with their own custom strategies container. However, it does mean that any options added to dry-auto_inject in the future will automatically be available for dry-component's injector as well.
This requires a breaking behavioural change, though. The Inject constant that we were setting up automatically on Dry::Component::Container subclasses can't be set up in the same way anymore, since there's no way we can infer by that time if the user wants any particular options to be passed. The recommended behaviour, instead, would be to have the user explicitly define their own constant to store their own preferred injector (e.g. MyApp::Inject = MyApp::Container.injector(my_options)). I think this is a reasonable requirement of users, though, and it was actually what we required of them until the last 0.3.0 release of dry-component, anyway.
Sound OK?

I'll merge this in now since all the code tweaking is done and there was no issue raised with the API adjustment :)


make this a Julia package?

The repo needs to be renamed, to NDSparseArrays perhaps?
We could then retain NDSparse as the exported type, which seems nice.

How about NDSparseData? Since we do want multiple data columns.

Yes, sounds good.

Renamed.


Speed up unit tests and add elapsed time report

This should make life easier

:+1: My comments are minor nitpicks.  Nice cleanup.

:+1:


Adding py.typed file for distributing type information?

As a user of accelerate, this would be nice in enabling typechecking for usages of accelerate.
It's usually not too much trouble to add: https://peps.python.org/pep-0561/#packaging-type-information

add a py.typed file to the package root
add this file to the package_data dictionary in the setup.py file

If this is something that the maintainers would be amenable to, I'm more than happy to open a PR for it!

We generally don't do this for our repos. See more here: https://github.com/huggingface/datasets/issues/3841


Create Medical Record Feature

As a main feature, this is need to be delivered first before the others.

Finished by #19.


yarn install is slow and gets stuck after a while

This is where its stuck for hours.
MacBook-Pro:angular-seed-advanced ishandutta2007$ yarn install

yarn install v0.20.3
[1/4] 🔍  Resolving packages...
warning<EMAIL_ADDRESS>electron-prebuilt has been renamed to electron. For more details, see http://electron.atom.io/blog/2016/08/16/npm-install-electron
warning gulp-atom-electron > github-releases ><EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
warning gulp-atom-electron > github-releases > grunt ><EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
warning gulp-atom-electron > github-releases > request ><EMAIL_ADDRESS>Use uuid module instead
warning electron-prebuilt > electron-download > nugget > progress-stream > through2 > xtend ><EMAIL_ADDRESS>⠂<EMAIL_ADDRESS>

I won't be able to support yarn issues here however in meantime you might just use npm i instead.


Error on using pretrained Yolov4-320

Thank you for your great work and contribution. :+1:  I am facing some problems while running your code.
I saw that you have used "Yolo-v4-320" as the pretrained model, but I also tried to download the pretrained weights in the form of ".pb" files and I get the following error:
    While executing: features,yolodata = mdl_yolo(img),      ValueError: not enough values to unpack (expected 2, got 1)
So, can you please upload the pretrained yolo model you used? or help me solve this problem?
Thank you for your time and help.

Hello,
Thank you for your interest in OG-SGG. I have just uploaded the pretrained YOLOv4 model I used in this project: link. I'm hoping that helps.

OK, Thank you!!


'RecognizerResult' object has no attribute 'recognition_metadata'

Describe the bug
A clear and concise description of what the bug is.
When i try to add new recognizer as mentioned in the documentation it throws error.
Throwing an error 'RecognizerResult' object has no attribute 'recognition_metadata'.
To Reproduce
Steps to reproduce the behavior:
class MoneyRecognizer(EntityRecognizer):
expected_confidence_level = 0.7 # expected confidence level for this recognizer

def load(self) -> None:
    """No loading is required."""
    pass

def analyze(
    self, text: str, entities: List[str], nlp_artifacts: NlpArtifacts
) -> List[RecognizerResult]:
    """
    Analyzes test to find tokens which represent currency/money etc.
    """
    results = []

    # iterate over the spaCy tokens, and call `token.like_num`
    for ent,token in zip(nlp_artifacts.entities,nlp_artifacts.tokens):
        if (ent.label_ == 'MONEY'):
            result = RecognizerResult(
                entity_type="MONEY",
                start=token.idx,
                end=token.idx + len(token),
                score=self.expected_confidence_level
            )
            results.append(result)
    return results

money_recognizer = MoneyRecognizer(supported_entities=["MONEY"])
analyzer = AnalyzerEngine()
analyzer.registry.add_recognizer(money_recognizer)
print('check2')
results = analyzer.analyze(text=text,
entities=entities,
language='en')
Error
Error thrown in the last step analyzer.analyze
'RecognizerResult' object has no attribute 'recognition_metadata'

Hi, thanks for raising this issue. Could you please provide the full stacktrace with the error?

Closing for now, please re-open if needed.


[Github] fix: Added default node structure for the system_config node

1. Why is this change necessary?
https://issues.shopware.com/issues/NEXT-35831 issue introduced possibility to define system config through yaml shopware.system_config node. However - current implementation does not allow to merge system_config node settings from multiple, environment dependent yaml files.
2. What does this change do, exactly?
Allows to merge shopware.system_config yaml nodes from multiple yaml config files.
3. Describe each step to reproduce the issue or behaviour.
Create new yaml file for system config e.g: config/packages/system_config.yaml with content like:
shopware:
    system_config:
        default:
            core.mailerSettings.host: 'some-host'
            core.mailerSettings.port: 465
            core.mailerSettings.emailAgent: 'smtp'

Create another yaml file specific for dev environment e.g config/packages/dev/system_config.yaml with content:
shopware:
    system_config:
        default:
            core.mailerSetting.disableDelivery: true

Notice now that vendor/shopware/core/System/SystemConfig/SymfonySystemConfigService.php receives only configuration from config/packages/dev/system_config.yaml, all configurations from base file are compeletelly ommited.
4. Please link to the relevant issues (if any).
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Imported from Github. Please see: https://github.com/shopware/shopware/pull/3808

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-37373
[created from NEXT-37373, comment 514399]


Example build failed

I want my first experience of ferretDB,and I follow the steps of the quickStart,I get the error:

I updated an example to remove --silent flag; please pull and try again – the real error should be visible now. Probably, that was a temporary NPM registry problem; it works for me now.

@wangagui Does it work for you now?


YouTube violations

So I am using plyr to show youtube videos and to increase my quota I had to go through a review by their team.
They noted that the design of the video player breaks their terms of service.
I could not get the player stop/start to work without the plry.css loaded. Is it possilbe to disable decorating the youtube player and still use plyr?

Hello,
Thank you for working with us on the YouTube API Services Compliance Review. We acknowledge your response.
In order to proceed with the quota allocation, please provide us with the following information :
A ( API Client Terms of Use and Privacy Policies) (link)
Policy #: III.A.1,2 (link)
Violation:
No reference to YouTube Terms of Service (ToS) in API Client’s ToS.
No reference/link to Google Privacy Policy.
Proposed Solution: Kindly update  YouTube Terms of Service (ToS) in your website terms of service page and Google Privacy Policy in your Website Privacy Policy Page.
C (Implementing YouTube Features) (link)
Policy #: III.C 1(link)
Violation: API Client is not adhering to YouTube’s RMF features. As per YouTube Developer policies, API Clients are not allowed to play multiple videos playing simultaneously in a webpage.
Proposed Solution: Please make the necessary changes so that a page or screen must not have more than one YouTube player that plays  content simultaneously.
Please fix the violations in the next Three (3) business days and feel free to reach out to us if you have any questions.

We're not an API client. This sounds like something you need to change in your own application that's consuming Plyr. The last issue I've provided an example for.

Hi!
We received the dreaded Youtube violation email as well.

F (User Experience) (link) Policy #: III.F.1 (YouTube Look and Feel) (link)
Violation: As per this developer policy, API Client is not allowed to modify (or) change the user interfaces/functionality in YouTube Applications unless you have obtained YouTube's prior written approval.
We didn't find any play bar in the below screenshot that alters the functionality of YouTube.

We have not modified the plyr css.
Plyr doesn't claim to be complying with YouTube ToS, so this is by no means criticism. It would be interesting to hear in case some people have managed to resolve the violations or whether the only way out is to use the official YouTube player.
Would it be possible for plyr to render the official youtube player instead?

We could certainly add an option in for this, yeah. Another option is to render the controls beneath the player like we had in v2. The reason I changed is it means the players are no longer 4:3 or 16:9.

Thanks for your quick reply!
We would 100% use this option as we need to resolve the YouTube violations. We're an edtech company - and we offer our own videos as well as YouTube videos.
If you could point me towards the right direction in the code I could have a look myself.

Hey @sampotts! Any further thoughts on this? Thanks for your two cents.

Vime has official youtube support. I'm switching to that.
On Thu, May 21, 2020 at 12:24 AM Benjamin Gröhbiel<EMAIL_ADDRESS>wrote:

Hey @sampotts https://github.com/sampotts! Any further thoughts on
this? Thanks for your two cents.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sampotts/plyr/issues/1738#issuecomment-631930042, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAGV5JRWKI4I7WUCDRKOBLRSTJKFANCNFSM4LSAOEPA
.

--
Anthony Ettinger
http://anthony.ettinger.name
<EMAIL_ADDRESS>+1<PHONE_NUMBER>

Vime has official youtube support. I'm switching to that.

It will be no more official than our support but good luck with it 👍 looking at the UI, it also overlays the player...

In their demo it shows youtube how they require it. Nothing interfering.
Maybe I don't know. It looked good.
On Thu, May 21, 2020 at 3:13 PM Sam Potts<EMAIL_ADDRESS>wrote:

Vime has official youtube support. I'm switching to that.
It will be no more official than our support but good luck with it 👍
looking at the UI, it also overlays the player...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sampotts/plyr/issues/1738#issuecomment-632373310, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAGV5PI2JO6XQ4SYL6GJ2TRSWRRJANCNFSM4LSAOEPA
.

--
Anthony Ettinger
http://anthony.ettinger.name
<EMAIL_ADDRESS>+1<PHONE_NUMBER>

I guess it depends which version you're looking at. There's a few different incarnations. Thanks for highlighting it though. I have had a few similar ideas for v4 👍

Thanks @chovy for sharing this, we will check it out.
@sampotts what's your thinking around adding support for this? Is this something that could come rather soon for plyr, or not?


worker: don't use CUDA as base image

The CUDA image isn't necessary in the worker anymore.
Use the python image to improve cache reuse across generated docker images (substra-backend / celeryworker / celerybeat)

e2e :heavy_check_mark:

Looks like celeryworker and celerybeat have the same dockerfile. It may be better to merge them and use the same image for both services :)

@Kelvin-M Good idea, but looks like not the purpose of this pull request

@Kelvin-M Awesome! Merging this for now.


refactor(cdc): refactor query plan for table-on-source cdc backfill

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Refactor the query plan for table-on-source cdc backfill to: source -no_shuffle-> filter -> backfill -> mview

Rewrite the backfill plan in the StreamCdcTableScan node
Add a new CdcFilter executor, since we need additional upstream source information
Remove the CdcTablename dispatcher introduced before

close: https://github.com/risingwavelabs/risingwave/issues/13147
Checklist

[ ] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Continue with the https://github.com/risingwavelabs/risingwave/pull/13553#discussion_r1400218037 because it's on the wrong place.

Does the mean that all rows between backfill -> mview will possibly transferred via network?

According to the image in the PR description:

CdcScan (Fragment #3) operator is connected to Materialize (Fragment #3) via a hash dispatcher. This means an CDC change event will go through 2 times of remote shuffle before being materialized into the table:
source --noshuffle--> filter --simple--> backfill --hash--> materialize

I understand this is because:

Backfill should be run in multiple nodes to avoid single-node bottleneck, so a simple dispatcher is required
Table should be distributed by PK, so an additional hash dispatcher is required

I am thinking about a question that: is 2 really worth? Since backfilling is already single-node, it doesn't improve a lot to distribute the table. Is it better to make it a singleton table, so that the second hash shuffle can be omitted.
This is kind of off-topic. Can create a separated issue for discussion.

Since backfilling is already single-node, it doesn't improve a lot to distribute the table. Is it better to make it a singleton table, so that the second hash shuffle can be omitted.

This could be a good point. However, it's quite simple to do the change so either LGTM.


AC: support MO transformations config option

--tf_custom_operation_config option in MO is deprecated in this release.  (now in the latest versions DLDT, it raises deprecation warning). MO team recommend to replace it with --transformations_config
P.S. Probably it also should affect Model Downloader configs for some TF models.

pls see error message:
15:00:10 accuracy_checker/launcher/model_conversion.py:186: [C0326(bad-whitespace), ] Exactly one space required after comma
15:00:10 mo_params, framework, tf_custom_op_config_dir, transformations_config_dir, mo_path

@vladimir-dudnik, thank you, I already fix it.


Example version_settings constraints are incorrect

The documentation for version_settings (https://github.com/tilt-dev/tilt.build/blob/master/api/api.py#L927) specify two examples:
- `<0.13.0` - less than 0.13.0
- `>=0.12.0` - at least 0.12.0

However, the constraints feature was only introduced from version 0.13.2 (I think), so this example would not when running these tilt versions.
I propose updating the examples to:
- `<0.17.0` - less than 0.17.0
- `>=0.13.2` - at least 0.13.2

Although it doesn't quite solve the problem (as <0.17.0 could still reference a version before 0.13.2), at least the examples given don't directly state versions where this parameter would not work.

thanks for the report! fixed


Show last added games?

Hello! I was thinking that when new games are detected, could be a skin that shows what are those detected games.
I'm thinking of something like this:

The way I think is to create a table with new games when those are detected to avoid write addition dates on games.json, so this will show added games just in the actual "cycle" of the skin. When the skin is closed or refreshed, they're gone. This will avoid increase the size of the games.json file.
The other option is to do exactly that, write the addition dates on games.json, and then you're be able to order games also with their addition date. However, like I said, this will increase the size of games.json file.

One could probably just add a temporary table to games.json with each new game's gameID, which are only valid during the session that the IDs were generated anyway. That table would then be removed (or replaced with a new one if more games are detected) the next time that the skin is loaded. Were you thinking of a scrollable list of games? What do you think about e.g. an overlay (name, platform, installed or uninstalled, etc.) when you hover the mouse over a game? Perhaps the number of new games should be shown somewhere? The menu should probably be optional.

After your questions I think that this could have two parts:
The menu as showed, just with the images to have a initial perspective of what is added (if you enable that), with the option to do scroll to show all added games. But nothing except that.
And then add a new filter that will show all last added games (if any), the number (as you said), and when selecting that filter you will have the option to show their overlay, play, etc.
So if you have it enabled on the startup you'll have a visual information of what been added, but in any case you will be able to filter the last games if exits, interact with them, etc.


FAQ Documentation

Have a FAQ which list common error / exception message with a small explanation and links to other documentations related

Example:

No message factory nor discovery in Guzzle5Adapter
Virtual package
"Factory not found" when Puli fails.


[BUGFIX] fix typo in install instruction output for bash completion

The Problem/Issue/Bug:
instruction for installing bash completion shows wrong filename "ddev_bash_completions.sh"
How this PR Solves The Problem:
change filename in output to "ddev_bash_completion.sh"
Manual Testing Instructions:
Automated Testing Overview:

Related Issue Link(s):
Release/Deployment notes:

 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.

@rfay done ;)


Node Exporter 1.16.0 introducing braking changes

Latest release of node-exporter is introducing breaking changes in metrics name. I think that it would be nice to add some note to README when you will bump node_exporter version. That way if someone does not follow node-exporter repository at least will have chance to not be surprised when everything  stops working(alerts, dashboards).
Link to rc: https://github.com/prometheus/node_exporter/releases/tag/v0.16.0-rc.0

Thank you for the issue, I usually mention breaking changes in releases https://github.com/ernestas-poskus/ansible-prometheus/releases. Do you think it is worth mentioning in README as well? In case of breaking changes for node exporter 0.16.0 when the 0.17.0 version will come out should 0.16.0 notes stay?

I`m not sure how to do it but maybe section in README about upgrading between versions could be nice. It is also duplication of informations available in node-exporter or prometheus so you need to decide if you want to provide that info.
The problem is that when you bump node-exporter version or some other exporter to version with breaking changes you are introducing breaking change for you library. Probably it would be nice to inform users about that in README also as not everyone is looking into releases.
Maybe providing some compatibility matrix would be nice but it might be unnecessary and hard to maintain.
For example with the latest Prometheus team wants to provide sample metric_relabel_config for backward compatibility(https://github.com/prometheus/node_exporter/issues/830).

Related https://github.com/ernestas-poskus/ansible-prometheus/pull/73
No backward compatibility as it will complicate maintenance of playbook it up to the user to migrate to the new format of /metrics.

bumped


F-score expresion is wrong

The F-score expresion in compute_f1() in expr/nnet.py is given by:
f1 = (2. * precision * recall /
T.maximum(1, precision + recall))
As far as I understand, T.maximum() is used to avoid the denominator from being zero. While this protection is fine in compute_recall() and compute_precision() in the same file, in compute_f1() this is wrong because precision+recall can legitimally be lower than 1, and in those cases, the reported f-score would be wrong (lower than it should be).
My suggestion is to implement this formula instead:
f1 = 2_hits / maximum(1,2_hits + false_alarms + misses)

I think using
f1= 2_hits/max(1,2_hits+false_alarms+misses)
would be a great idea


Support OSTree-based systems

Summary of the new feature / enhancement
This is the same as #16679, which was closed for housekeeping, yet this is still a problem.
Proposed technical implementation details (optional)
No response

https://github.com/PowerShell/PowerShell/issues/19217


Unpin jupyterlab and jupyterlab_latex

Resolves https://github.com/binder-examples/latex/issues/10
Fixed with jupyterlab-latex 2.0.1 release

I've locally tested to be OK w/ repo2docker and here's the (still pending) mybinder check: https://mybinder.org/v2/gh/trybik/latex/iss10?urlpath=lab

excellent! thanks for following up on this @tribik :-)

Nota bene: I believe in pinned versions, for once to avoid same issues in future (so for the record, the working version nowis  jupyterlab==2.2.7, jupyterlab_latex==2.0.0, @jupyterlab/latex@2.0.1). I know it's a bit of more maintenance, but otherwise reproducibility is questionable w/ such fragile dependencies system (cf. https://github.com/jupyterlab/jupyterlab-latex/pull/137/files ).


Printing via hplip fails because cupsext can't be found

Issue description
Every time I try to print something it fails and I see the following error message in my journal:
cupsd[19016]: : ERROR: Error importing HPLIP modules:20913 No module named cupsext

This looks related to https://github.com/NixOS/nixpkgs/issues/21084.
I'm using the following printer:
Driver: HP LaserJet 700 color MFP M775 Postscript (recommended) (color, 2-sided printing)
Connection: dnssd://HP%20LaserJet%20700%20color%20MFP%20M775%20%5B3121DA%5D._ipp._tcp.local/?uuid=6bbc56fd-a278-11e4-b8cb-d1c7746da18e

And this is my NixOS printing setup:
{
  services = {
    printing = {
      enable = true;
      drivers = [ pkgs.hplip ];
    };
    avahi = {
      enable = true;
    };
  };
}

Steps to reproduce
Print something like a test page and observe that nothing is being printed and an error message is logged to the journal.
Technical details

System:

$ nixos-version
17.09pre-git (Hummingbird)

Nix version:

$ nix-env --version
nix-env (Nix) 1.11.15

Nixpkgs version:

$ nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion
"17.09pre-git"

I'm using nixpkgs at 1a6a7b0f33f0789734c10643da3396f12953b58d.

Sandboxing enabled:

$ grep build-use-sandbox /etc/nix/nix.conf
build-use-sandbox = false

Although I'm not at my office right now which means I can't connect to the printer I tried upgrading hplip from 3.16.11 to 3.17.10 and printing a test page. The process looks like to go further than before right to the point where it tries to connect to the printer and then obviously fails.
I will try this again tomorrow at the office.
@ttuegel  if it works I believe we just have to cherry pick your ec62d5462dce070f9a1587255e6d418e77a8612e and 721e07ea99573cb242a0df6c00f260471e12388b on release-17.09.

I can confirm that the upgrade fixes the problem. I created a PR at: https://github.com/NixOS/nixpkgs/pull/31601.

PR  #31601 has been merged.


fix(view-controller): fix memory leak when destroying ViewController

There's a memory leak when opening modals and popovers (and every other classes that inherits from ViewController) which causes the application to crash.
Short description of what this resolves:
This fixes a memory leak that was caused by an event listener that was added in ViewController's constructor but wasn't properly removed when ViewController was destroyed.
Related : https://github.com/ionic-team/ionic/issues/12574#issuecomment-368843850
Ionic Version: 3.9.2

Does this happen to the inonic V4? I have a similar problem. Thank you

Hi there,
We have moved all V3 related issues to a separate repo. I am going to close this PR in favor of https://github.com/ionic-team/ionic-v3/pull/973 but will also note your contribution there.
Thanks for the PR!


feat: Support object type of splitChunks.minSize

Summary

fixes #4331
This code snippet attempts to add the splitChunks.minSize functionality. If the code is correct, I will proceed to add support for the same functionality to other sizes and remove the currently apparent "temporary" code.
Test Plan

Require Documentation?

[ ] No
[x] Yes, the corresponding rspack-website PR is TBD

 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.

close for outdated, @JSerFeng does splitchunks.minSize supports object type now？


Bug with BPU update in fetch module

In situations, when we want to provide BPU with information from decode and branch modules simultaneously, information from decode would be ignored. So I've fixed it.

Yes, but that is possible only if branch target array has two ports.

Yes, but that is possible only if branch target array has two write ports,

I can't understand why branch target array should have two write ports.
void update( const BPInterface& bp_upd) final
    {
        const auto set = tags.set( bp_upd.pc);
        auto[ is_hit, way] = tags.read( bp_upd.pc);

        if ( !is_hit) { // miss
            way = tags.write( bp_upd.pc); // add new entry to cache
            auto& entry = directions[ way][ set];
            entry.reset();
        }

        directions[ way][ set].update( bp_upd.is_taken);
        targets[ way][ set] = bp_upd.target;
    }

I don't see use of ports here.

I don't mean simulation, I mean the real hardware. If you want to write two cache entries in the same cycles, you need a lot of additional hardware.

I don't mean simulation, I mean the real hardware. If you want to write two cache entries in the same cycles, you need a lot of additional hardware.

Understood. Thanks.


Enable / disable textarea spellchecking via config

Is your feature request related to a problem? Please describe.
The v7 ui app is generating textarea form elements with explicit spellcheck=false. This
disables browser based spellchecking functionality (which I'm informed worked fine in
v6), without additional clicks to make it happen.
Describe the solution you'd like
Ideally there would be at least a config option for generating textarea form elements with
spellcheck=true so browser spellchecking can work at least with parity to v6.
Describe solutions you've considered
This branch makes it so that browser spellchecking can be enabled / disabled via the config
file by passing through the config value to DynamicTextAreaModel /
DsDynamicTextAreaModelConfig:
https://github.com/DSpace/dspace-angular/compare/main...mark-cooper:dspace-angular:textarea-spellcheck
Happy to make a PR if this looks generally useful.
Before:

After (form.spellCheck: true):

I was going to say that spellcheck-or-not is a personal issue and should be a user profile option rather than a global one, but then realized that browsers probably make this personally configurable, and on checking found that at least Firefox does.  So maybe the best thing to do would be to just remove the parameter from these forms and leave it to browser settings.

"So maybe the best thing to do would be to just ... leave it to browser settings"
I think spellcheck=true effectively does this. If the browser settings are set to not spellcheck (for example, Firefox "Check your spelling as you type" disabled), then the attribute value (spellcheck=true) is ignored. But if you do want spellcheck ("Check your spelling as you type" enabled ) you're good.
With the current situation you can't "passively" opt-in to spellcheck. You have to force it (Right click -> Check Spelling [Firefox]), because of the generated spellcheck=false. The feedback I've received is that this isn't desirable, and different from v6.
Not generating the attribute at all would be more or less equivalent as spellcheck=true, although leaving it a bit more to browser "choice" on whether spellchecking is applied in the absence of a spellcheck attribute.

We rely heavily of the default spell checking to catch metadata errors. We favor the current behavour in DSpace 6.

Just re-discovering this ticket, and I agree that it seems reasonable to attempt to replicate DSpace 6 behavior with regards to spellcheck.  So, I'd be OK with an option to make this configurable, or remove spellcheck=false altogether (I haven't had a chance to check yet which option better aligns with DSpace 6)

@tdonohue I think I'll have time to do it over the next couple of weeks 👍

Oof, I got well behind on this one.
@tdonohue I rebased the original branch and retested it:
https://github.com/DSpace/dspace-angular/compare/main...mark-cooper:dspace-angular:textarea-spellcheck
Checks out.
Alternatively it's much simpler (less boilerplate) if we can assume spellcheck is generally desirable by default. It can always be disabled by the user in browser:
https://github.com/DSpace/dspace-angular/compare/main...mark-cooper:dspace-angular:textarea-spellcheck-enabled
So perhaps the simpler (in line with v6) approach is better?
I don't believe it's currently possible to not generate a spellcheck attribute:
https://github.com/udos86/ng-dynamic-forms/blob/master/projects/ng-dynamic-forms/core/src/lib/model/input/dynamic-input.model.spec.ts#L89-L91
But I haven't really pushed against it.

@mark-cooper :  Thanks for getting back to this.  Giving the two options you propose, I have a slight preference for making this configurable (the first branch you linked) & defaulting it to true.
While I understand that hardcoding it to true is easier & doesn't force the browser to do a spellcheck, I think there's a chance some users may want it to be false because of the "Security and privacy concerns" listed in the docs for this attribute: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/spellcheck
So, if you could create a PR based on your first branch (https://github.com/DSpace/dspace-angular/compare/main...mark-cooper:dspace-angular:textarea-spellcheck), I think that'd be wonderful!


ARROW-2332: Feather Reader option to return Table

@xhochy It makes sense to remove that from FeatherReader. Does it make sense to keep the as_pandas argument for the read_feather function?

@xhochy done! Whats your commit policy? I can squash these changes into a single commit if needed

@dmadeka no problem with the commits. Our merge script will squash them automatically into a single single commit. The only important policy we have is that the PR title needs to start with ARROW-XXX:

@xhochy Thank you! Once this is merged, ill do the FeatherDataset


Updating multiple plot object

In b3a7e65: see mwe_plot. Multiple line object cannot be updated with the axes. This is curable with a simple loop, however there is a deeper problem how info of multiple line objects are stored under the plot. Now it assumes that plot can have only one line data stored to ; this needs to be changed also. So far done for line and scatter, need to still update for

histogram
bar

We could probably merge different plot object update functions (e.g. _UpdateScatterParams) into one.

Done in v0.1.6


transform; initial value is mutable

> transform(map(square), [])([1,2,3])
[ 1, 4, 9 ]
> const squareAll = transform(map(square), [])
undefined
> squareAll([1,2,3])
[ 1, 4, 9 ]
> squareAll([1,2,3])
[ 1, 4, 9, 1, 4, 9 ]

Only occurs on reduce/transform function reuse
Copying initival value will fix. This one ties in pretty closely with https://github.com/a-synchronous/rubico/issues/60

reduce won't enforce immutability, that is up to you. Transform will, however.

actually, this is a feature. I don't want rubico to implicitly deep copy anything. https://github.com/a-synchronous/rubico/issues/60 would resolve the example like so
> transform(map(square), () => [])([1,2,3])
[ 1, 4, 9 ]
> const squareAll = transform(map(square), [])
undefined
> squareAll([1,2,3])
[ 1, 4, 9 ]
> squareAll([1,2,3])
[ 1, 4, 9 ]


What does A/P OSQP tuning means?

Good afternoon. In the basic version of the problem formulation of optimizing a polynomial using a quadratic formulation, the matrix P explains the integrated analytical description of the polynomial function (the 6x6 matrix, resulting from the product of the transposed and ordinary vectors of the first derivative coefficients of the polynomial with respect to s), and in the matrix A - the factors at the coefficients of the polynomial. Each line in A is a spline equation in S. In the Apollo code, I did not find the calculation of integrals and derivatives, but I saw the formulas for filling the matrices P and A using strange weight functions.
weight_x_ + weight_x_ref_vec_[i]) /(scale_factor_[0] * scale_factor_[0])
weight_x_ + weight_x_ref_vec_[n - 1] + weight_end_state_[0]) /(scale_factor_[0] * scale_factor_[0])
weight_dx_ / (scale_factor_[1] * scale_factor_[1])
(weight_dx_ + weight_end_state_[1]) /(scale_factor_[1] * scale_factor_[1])
(weight_ddx_ + weight_dddx_ / delta_s_square) /(scale_factor_[2] * scale_factor_[2])
(weight_ddx_ + 2.0 * weight_dddx_ / delta_s_square) /(scale_factor_[2] * scale_factor_[2])
(weight_ddx_ + weight_dddx_ / delta_s_square + weight_end_state_[2]) /(scale_factor_[2] * scale_factor_[2])
(-2.0 * weight_dddx_ / delta_s_square) /(scale_factor_[2] * scale_factor_[2])

Could you explain in more detail why this was done?
Based on the code, I was able to understand that in general, A and P have the following structure: (description of 0 derivative | description of 1 derivative (graph growth rate) | description of 2 and 3 derivatives (acceleration of graph growth and jerk)) do I understand correctly what have you done?

The code is from PiecewiseJerkProblem which optimizing a piecewise-jerk trajectory (not polynomial).
See paper Optimal Trajectory Generation for Autonomous Vehicles Under Centripetal Acceleration Constraints for In-lane Driving Scenarios


[webapi] Automate 15 case for text

Impacted TCs num(approved): New 0, Update 15, Delete 0
Unit test Platform: Andriod
Unit test result summary: Pass 15, Fail 0, Blocked 0

please remove the tmp file in PR:  webapi/tct-text-css3-tests/css3-text-app/testscripts/.feature
Besides, 15 cases, why 16 feature files

Our bdd won't do automation for design cases, please remove the relate code


Automated cherry pick of #6669: Add docker credentials to windows CI to avoid rate limit

Cherry pick of #6669 on release-1.15.
#6669: Add docker credentials to windows CI to avoid rate limit
For details on the cherry pick process, see the cherry pick requests page.

/test-windows-all

/test-windows-containerd-e2e

/test-windows-containerd-e2e

/test-windows-all

/test-windows-all

/test-windows-all

@antoninbas Windows CI passed, previous failure is caused by busybox image error bacause antrea has remove this image in its dockerhub. I have backported the upgrade PR to release-1.15 in https://github.com/antrea-io/antrea/pull/6701.


Chart do not permit use of specific docker credentials

The server and client templates does not permit the use of specific docker credentials.
It should embark this directive :
imagePullSecrets:
  - name: name_of_my_dockercreds

I am preparing a PR regarding this issue, will push it tomorrow.

Hi @bgiarrizzo :) We really appreciate your willingness to contribute :)
cc me when you are done, and i'll review your PR :)

Hello @asafc, thank you for your reply.
I pushed my modification this afternoon, you can see the work in #7 .


Select: fix a bug that makes the browser jitter in zoom mode

Please make sure these boxes are checked before submitting your PR, thank you!
fix: #21142
fix: #21178
fix: #21089
fix: #19499
close: #21143

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

@iamkun  I'm getting a bunch of errors everywhere - "Cannot read property 'getBoundingClientRect' of undefined"
Seems like this line is causing the problems var tagsHeight = Math.round(tags.getBoundingClientRect().height);

@cs1707 @iamkun I'm getting a bunch of errors everywhere - "Cannot read property 'getBoundingClientRect' of undefined"
Seems like this line is causing the problems var tagsHeight = Math.round(tags.getBoundingClientRect().height);

Thanks for your feedback,  this has been fixed in version 2.15.5.

大佬，还有个就是Select选择器开启搜索后，ios手机弹不出来输入法的bug