Implement clarification of MAX_REQUEST_BLOB_SIDECARS

PR Description
Implement https://github.com/ethereum/consensus-specs/pull/3269
Fixed Issue(s)
N/A
Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[X] I thought about adding a changelog entry, and added one if I deemed necessary.

LGTM But I'd rework BlobSidecarsByRangeMessageHandler to use calculation of maxRequestSizesimilar to BlobSidecarsByRootMessageHandler. It could be done in separate PR of course

Doing it as part of this PR.


SparsePauliOp casting of coeffs causes errors for dtype=real

Environment

Qiskit Terra version: 0.22
Python version: All
Operating system: All

What is happening?
The current init for SparsePauliOp has handling for storing coefficient arrays using their input dtype, but this is used to cast the coeffs to that type after moving the Pauli phase to the coeffs. This can cause errors if the dtype is not complex.
How can we reproduce the issue?
Here is an example which gives an incorrect answer:
# Initialize ops, with coeffs with dtype double
a = SparsePauliOp(['X'], np.array([1]))
b = SparsePauliOp(['Y'], np.array([1]))
a.dot(b)

This raises a warning
# Raises waring
.../sparse_pauli_op.py:131: ComplexWarning: Casting complex values to real discards the imaginary part
  self._coeffs = np.asarray((-1j) ** (phase - count_y) * coeffs, dtype=coeffs.dtype)

and returns the incorrect value
SparsePauliOp(['Z'],
              coeffs=[0])

What should happen?
For the above example the correct answer should be
SparsePauliOp(['Z'],
              coeffs=[0.+1.j])

Any suggestions?
Removing the explicit dype in the line from the warning should fix this, which would allow double coeffs to be converted to complex if required.
I suspect this support was added in the first place to handle object arrays for parameters, and I think that should still be supported if the dtype arg is not used since the asarray should default to an object array in that case.

Yeah, seems like the allowed dtypes here should only be np.complex128 and object - any other numeric dtype should perhaps be raised to complex to avoid this error.
@ikkoham: did you have more dtypes in mind when you wrote that part of the handling?

Ever wanted to use a float to save data? Well, no.

OK, I'll restrict it to complex and object.


Panel editor: Improve error reporting when API calls fail

In case an API call fails it reports back a structure like this:
{"result": "Check_MK exception: Currently not supported with this Check_MK Edition", "result_code": 1}

Can we show the error messages of the result_code=1 responses to the use in the panel editor?

Implemented in 83ba322f7b515b27d335e06ded590e5dabb3eb34.


perf(html): apply preTransformRequest for html scripts

Description

When handling URLs for <script type="module">, also pretransform their requests. This should speed up startup time a little.
Additional context

An interesting side effect with this is that, if someone calls server.transformIndexHtml manually (custom server), this would start pre-transforming things. Maybe that's a good thing 🤔

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

/ecosystem-ci run

When using transformIndexHtml during Dev SSR, this will kick start the client processing. I think this makes sense also for this case. If ecosystem-ci is happy, I think we should try this during the beta


Dask config value setting question

Is dask.config.set(**dict_mapping) a supported way to set configuration values?
Previously, using with dask.config.set(**mapping): would set the config options in mapping inside the with context. For example (on commit d92f4015a1da3da10c04c682ed2acae8469e9576):
In [1]: import dask

In [2]: dask.config.get('array.svg')
Out[2]: {'size': 120}

In [3]: with dask.config.set(**{'array.svg': 2}):
   ...:     print(dask.config.get('array.svg'))
   ...:
2

In [4]: dask.config.get('array.svg')
Out[4]: {'size': 120}

with the changes in #5226, this is no longer the case. On the current master, we have:
In [1]: import dask

In [2]: dask.config.get('array.svg')
Out[2]: {'size': 120}

In [3]: with dask.config.set(**{'array.svg': 2}):
   ...:     print(dask.config.get('array.svg'))
   ...:
{'size': 120}

In [4]: dask.config.get('array.svg')
Out[4]: {'size': 120}

However, removing the ** (i.e. with dask.config.set({'array.svg': 2}):) does work on master.
The config docs only make reference to dask.config.set(mapping) for setting config values, not passing as keyword argument like dask.config.set(**mapping). Is the descrepency here a backwards compatibility issue, or was dask.config.set(**mapping) previously not a supported way to set config values?

cc @jcrist @djhoese

In [3]: with dask.config.set(**{'array.svg': 2}):
   ...:     print(dask.config.get('array.svg'))
   ...:

Using kwargs in config.set (as you're doing above), is valid, but splits on __ instead of . for nested parameters (since array__svg is a valid parameter name, but array.svg isn't). However, due to how **kwargs works in python, this just happened to work before. If you already have a mapping, you shouldn't splat it into the function call, you should just pass it directly.
Unless you think otherwise, I don't think this is a bug, just a misuse.

To be more clear, both of these work as intended:
In [3]: with dask.config.set(array__svg__size=140):
   ...:     print(dask.config.get("array.svg"))
   ...:
{'size': 140}

In [4]: with dask.config.set({"array.svg.size": 140}):
   ...:     print(dask.config.get("array.svg"))
   ...:
   ...:
{'size': 140}

Is it easy to support anyway?  Are there problems with supporting it?
On Thu, Aug 15, 2019 at 2:38 PM Jim Crist<EMAIL_ADDRESS>wrote:

To be more clear, both of these work as intended:
In [3]: with dask.config.set(array__svg__size=140):
...:     print(dask.config.get("array.svg"))
...:
{'size': 140}
In [4]: with dask.config.set({"array.svg.size": 140}):
...:     print(dask.config.get("array.svg"))
...:
...:
{'size': 140}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/5282?email_source=notifications&email_token=AACKZTD4BP5E2UKI67QCSGDQEWPEHA5CNFSM4IMAJFHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4MUK5I#issuecomment-521749877,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACKZTD6Q66MJQG7NKS6ZQDQEWPEHANCNFSM4IMAJFHA
.

It's possible, but imo a weird thing to do. Functions shouldn't expect keyword arguments that aren't valid python parameters. I'd rather not, and declare any (unlikely) past uses of this user error.

This also was completely undocumented behavior before (docstring from before this change):
https://github.com/dask/dask/blob/d92f4015a1da3da10c04c682ed2acae8469e9576/dask/config.py#L263-L273

OK, no objection from my end
On Thu, Aug 15, 2019 at 2:46 PM Jim Crist<EMAIL_ADDRESS>wrote:

This also was completely undocumented behavior before (docstring from
before this change):
https://github.com/dask/dask/blob/d92f4015a1da3da10c04c682ed2acae8469e9576/dask/config.py#L263-L273
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/5282?email_source=notifications&email_token=AACKZTFWBZ7EHEEQ5YQENNLQEWQAFA5CNFSM4IMAJFHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4MU7KI#issuecomment-521752489,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACKZTCGFXW4YWXSRRW4E7TQEWQAFANCNFSM4IMAJFHA
.

Functions shouldn't expect keyword arguments that aren't valid python parameters

This seems reasonable to me
Thanks for the responses all! Closing.


fancyzones stopped working for Inkscape floating dialogs. v0.53 + v0.55, It is also not working for GIMP tool pallets, but this could have been the case previously, as I hadn't tried FancyZones with GIMP until now.

Microsoft PowerToys version
0.53.3
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Inkscape v1.1.1
inkscape preferences; Interface, Windows, Dialogue behavior "floating", restart inkscape
✔ Expected Behavior
zones should be visible on shift + click + drag of floating dialogs. dialogs should snap to zones.
❌ Actual Behavior
no zones visible on shift + click drag, not able to snap dialogs to zones
Other Software
Inkscape v1.1.1

Duplicate of #15426

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software. 15426


Add extra step to Makefile config target to force CMake rescan installed packages

Something for consideration as I'm not sure what would be the best way to handle this.
Building with -DOpenImageIO_BUILD_MISSING_DEPS=all fails on Linux with the following error:
In file included from /home/kuba/SRC/OpenImageIO/__build2/include/OpenImageIO/Imath.h:11,
                 from /home/kuba/SRC/OpenImageIO/src/libutil/strutil_test.cpp:8:
/home/kuba/SRC/OpenImageIO/__build2/include/OpenImageIO/half.h:8:10: fatal error: Imath/half.h: No such file or directory
    8 | #include <Imath/half.h>

Steps to reproduce:
make config MY_CMAKE_FLAGS="-DOpenImageIO_BUILD_MISSING_DEPS=all"
make build

Initially I thought this is related with OpenImageIO/Imath.h expecting system includes: include_directories(BEFORE ${IMATH_INCLUDES} ${OPENEXR_INCLUDES})
As it turns out this was redherring and the problem could be with CMake "state?" (Maybe a specific CMake version 3.27.7 to blame).
A potential fix - after the initial configuration completes - is to reconfigure the project one more time: cmake .. which resolves the error.
The second reconfiguration command is also protected by ${build_dir}/${BUILDSENTINEL} hence runs only once - during the initial configuration. If anyone is able to reproduce this or have any thoughts what the root problem may be - please chime in.

I'm going to close this one and continue in https://github.com/AcademySoftwareFoundation/OpenImageIO/pull/4306


Empty _dl folder after download completed

Hi,
Is there a setting to empty the _dl folder automatically after the download(s) have finished?

What's the _dl folder?
If you want to delete any file from the server, use remote tools, aria2 doesn't and would never provide such interface.

I would suppose you are using some self-modified version of this project, and the maintainer of that fork didn't change the link.
Search in All code shows nothing about "_dl", and so does the aria2.


unexpected 22 empty columns in addition to 24 known columns

Previously, 24 columns were present in the output.csv of NHM Interaction Bank. According the recent travis check (see https://travis-ci.org/globalbioticinteractions/natural-history-museum-london-interactions-bank/builds/271807937, see log snip below), 46 columns are now present. At first glance, none of the new columns have headers or values.
@edwbaker Are the new column additions expected?
19:27:23,857  INFO ResourceUtil:39 - caching of [http://data.nhm.ac.uk/api/3/action/package_show?id=nhm-ib] complete.
19:27:23,916  INFO ResourceUtil:37 - caching of [http://data.nhm.ac.uk/dataset/82e807f0-6273-4f19-be0a-7f7558442a25/resource/9f49865f-ca81-488e-8738-82c569fb562e/download/output.csv] started...
19:28:34,576  INFO ResourceUtil:39 - caching of [http://data.nhm.ac.uk/dataset/82e807f0-6273-4f19-be0a-7f7558442a25/resource/9f49865f-ca81-488e-8738-82c569fb562e/download/output.csv] complete.
19:28:34,582 ERROR StudyImporterForGitHubData:63 - failed to import data from repo [globalbioticinteractions/natural-history-museum-london-interactions-bank]
org.eol.globi.data.StudyImporterException: read [46] columns, but found [24] column definitions.
at org.eol.globi.data.StudyImporterForMetaTable.importAll(StudyImporterForMetaTable.java:267)
at org.eol.globi.data.StudyImporterForMetaTable.importTable(StudyImporterForMetaTable.java:127)
at org.eol.globi.data.StudyImporterForMetaTable.importStudy(StudyImporterForMetaTable.java:67)
at org.eol.globi.data.StudyImporterForGitHubData.importData(StudyImporterForGitHubData.java:86)
at org.eol.globi.data.StudyImporterForGitHubData.importData(StudyImporterForGitHubData.java:59)
at org.eol.globi.tool.GitHubRepoCheck.main(GitHubRepoCheck.java:86)

Shouldn't be any different - any idea what line of the file the issue arises on?

First line has a bunch of extra commas in it.

I have fixed some issues - it would be helpful to know what line it is failing on now, is that possible to get?

Managed to track them down. A number of other potential issues also fixed.

@edwbaker glad to hear that the issues were fixed. And yes, line numbers would be nice to include in the report. I've opened a separate issue on that here: https://github.com/jhpoelen/eol-globi-data/issues/313 .


Release v1.14.1

This is a draft of changelog for the v1.14.1 release.
Here you can find all related translations changes: v1.14.0...master.

⚠ Do not merge this pull request manually ⚠
You should approve this pull request instead. After approving, I will automatically update it (release date needs to be adjusted), merge, and tag a new release. You can read more about the process on forum.

Show release notes preview
v1.14.1

更新翻譯:

fof/polls

全部變化：v1.14.0...v1.14.1

Success! Now you can announce new release on forum:
## 版本 [`v1.14.1`](https://github.com/flarum-lang/chinese-traditional/releases/tag/v1.14.1)

**更新翻譯**:

* [`fof/polls`](https://github.com/FriendsOfFlarum/polls)

全部變化：[v1.14.0...v1.14.1](https://github.com/flarum-lang/chinese-traditional/compare/v1.14.0...v1.14.1)

更新命令：

```console
composer update flarum-lang/chinese-traditional
php flarum cache:clear
```


Peer-review Backend

Peer Review: Backend of Group 17
Backend code included in the app.R  file of the group repo is very clear and well written.  Detailed comments make it easy to read and understand the underlying functionality.
9. Are all the options that you can select in the widgets working correctly? Have you found some options/combinations where you got errors?
9.1. After reviewing carefully all options and combinations I found no mistakes.  A few minor notes are:

You could format the data better to fix some of the year values in the plots.  For example the combination Art Type: Memorial or Monument + Neighborhood: Mount Pleasant has values in the Year Installed plot of 2009.75, 2010.00.  Because there is only one element, the plot axis is set as continuous and shows up with fractions of the year.  Same thing happens when I select the Neighborhood.  If you add something like this to your plot, it seems to fix the problem: scale_x_continuous(breaks = seq(1950, 2020, 10))

One other comment: when the filters result in an empty output e.g. Memorial or Monument + Mount Pleasant and  set the sliders for the Year installed for example at 2011 and 2022 return an empty map.  Similarly when you slide the right slider down to 2009 the Everything disappears for the above combination.  This might seem like a but to the user.  It would probably be better to show empty map or show a message which saying something like: “No Art for this period with the selected criteria.”

9.2. Could be added reactive expressions to avoid code duplication?
The reactive elements are well structured and mutually exclusive, which makes the code very efficient already.  Each element of the app has it’s own reactive element, which works as expected.
9.3. Could be included dynamic UI elements or actions buttons? Or what other element - widget would you include?
The app is really well structured without being cluttered.  I like the leafletOutput for the menus which make a very efficient use of the screen.
One additional element that could add value is to have a search box for searching monuments by name.
It could also be helpful to have the popup descriptions of each element come out when the user hovers over them instead of having to click.
These are of course, design choices, which are up to the authors and do not take away from the excellence of the backend implementation.
Overall backend is really impressive, especially considering the complicated dataset the team is using.  Great Job!


static logger initialization and subsequent static initializations using logging.get is undefined behavior

We initialize villas::logging as a static member in log.cpp. At the same time several other objects are created as static objects. Some of them (e.g., MemoryManager, DirectedGraph, HostRamAllocator) use the static object to create their own logging instances, i.e., villas::logging.get().
The initialization order of these objects is undefined and thus what we are doing is undefined behavior.
If I activate link time optimization (-lto) I get segmentation faults because the initialization order is different.
I would suggest to move the initialization of these objects to an explicit initialization function, where we can make the initialization order explicit and call this function for example from a library constructor or via a single static initialization.
I only researched this briefly so I may have misunderstood some of this.

Nitpick: This would not have happened if we were using static macro loggers :P


Feature Request: Do not load on excluded directories

I routinely work on remote git repositories, e.g.
sntentos@home [2018-03-16 14:47:39]:/run/user/1000/gvfs$ ls -lah
total 28K
dr-x------  9 sntentos sntentos    0 Μάρ  16 11:33 .
drwx------ 12 sntentos sntentos  320 Μάρ  16 13:59 ..
drwxr-xr-x  1 sntentos sntentos 4,0K Μάρ  14 06:52 sftp:host=<IP_ADDRESS>
drwxr-xr-x  1 sntentos sntentos 4,0K Φεβ  22 06:17 sftp:host=<IP_ADDRESS>

Where git calls are too slow. I want to keep my extensions etc as-is, but simply 'not access' the repositories for information.
or, change something in Sublime? I don't know if it is possible that way

ok... What are you asking for?

I think what he is requesting is to disable GitSavvy for certain directories.

@randy3k :+1:
@stoivo I tried really hard to make it understandable on the first try ... thank you for proving me wrong 😂

There are three places to specify such settings.

use git config

git config --add GitSavvy.disable true
git config --get GitSavvy.disable

use Sublime project settings in foobar.subime-projects

{
    "settings": {
        "GitSavvy": {
           "disable": true
        }
    }
}

use GitSavvy.sublime-settings with a setting such as

{
      "directories_to_exclude" : []
}

I recently started to use projects and likes it alt but I would implement 1. and 2.

Disclaimer; I will use my personal scope. Feel free to ignore me 😃
@randy3k @stoivo

"I am not opposed" to having a git add/get config. That's manageable.

I find it complicated to fit this in my usecase - since the file would be required to exist in the repository. Problem is not "the repository itself", rather it's location (on the network)

That's what I've been thinking. I'd see me write one of the following versions:

"/run/user/1000/gvfs/", provided "it applies to any git repository found underneath,
"/run/user/1000/gvfs/**" / "/run/user/1000/gvfs/*", as a glob-pattern, or
"(gvfs|sftp|nfs)", as a ReGex pattern

I am happy you are taking up the task 😄


Feature: Fixed search results header (jatkinson)

Can one of the admins verify this patch?

[Current coverage][cc-pull] is 79.64%

Merging [#35][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #35   diff @@
==========================================
  Files           112        112
  Lines          7123       7123
  Methods        1535       1535
  Messages          0          0
  Branches       1041       1041
==========================================
  Hits           5673       5673
  Misses         1450       1450
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [da5fb1a...e213a5a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/voyagersearch/navigo/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/voyagersearch/navigo/compare/da5fb1a8a3aa92f638abaa3d267f34d0e445ffbe...e213a5a0f300e30da508390218e9f2ba76f5fa59
[cc-pull]: https://codecov.io/gh/voyagersearch/navigo/pull/35?src=pr


testChromePrintPageRange and testLocalPrintPageRange issue

Hi Kelly/GCPTeam,
We are having this weird issue under testChromePrintPageRange and testLocalPrintPageRange. This test will try to set the page range to 2-3 but as you can see from the attached screenshot, it failed to do that.
These two tests are frequently failing using Windows 10. We are not sure if this is OS related. We've tried running this test using Windows 7 and everything works fine, there are no failures. We've also tried setting the page range manually and it works fine. We also suspect that this is a network connection related issue. Our internet connection is sometimes slow so maybe that's the cause of this failure or maybe this is a webdriver/selenium issue?.
Windows 10 - Fail

Windows 10 - Fail

Windows 7 - Success

Thanks,
Marc

Marc,
We'll be pushing out a whole new branch soon that will by-pass using
ChromeDriver, and will speak directly to the GCP Services and the printer.
You can try it now by downloading the branch called logocert_api. Try that
and see how it works for you.
It changes some of the way of doing things, like now you actually set the
suites inside the _config.py file.
Kelly
kdLucas
On Tue, Dec 6, 2016 at 11:45 PM, Marc<EMAIL_ADDRESS>wrote:

Hi Kelly/GCPTeam,
We are having this weird issue under testChromePrintPageRange and
testLocalPrintPageRange. This test will try to set the page range to 2-3
but as you can see from the attached screenshot, it failed to do that.
These two tests are frequently failing using Windows 10. We are not sure
if this is OS related. We've tried running this test using Windows 7 and
everything works fine, there are no failures. We also suspect that this is
a network connection related issue. Our internet connection is sometimes
slow so maybe that's the cause of this failure.
Windows 10 - Fail
[image: testlocalprintpagerange failure - win10]
https://cloud.githubusercontent.com/assets/19194749/20959008/c9e66340-bc93-11e6-9982-e5ae56741cc5.png
Windows 10 - Fail
[image: testchromeprintpagerange failure - win10]
https://cloud.githubusercontent.com/assets/19194749/20959033/d9215a40-bc93-11e6-8450-5840eb778e86.png
Windows 7 - Success
[image: testchromeprintpagerange - success - windows 7]
https://cloud.githubusercontent.com/assets/19194749/20959004/c0b18bb0-bc93-11e6-9adf-b9d95cf22543.png
Thanks,
Marc
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/cloudprint_logocert/issues/95, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ADk_fWbMTuNReay_qpCKejsbWTywjqYWks5rFmQwgaJpZM4LGSgP
.

Hi Kelly,
This change is great and more robust compared to the older tool. I will
test this as soon as I can. I am currently waiting for the machines I need
to be available.
Regards,
Marc
On Wed, Dec 7, 2016 at 4:47 PM, Kelly Lucas<EMAIL_ADDRESS>wrote:

Marc,
We'll be pushing out a whole new branch soon that will by-pass using
ChromeDriver, and will speak directly to the GCP Services and the printer.
You can try it now by downloading the branch called logocert_api. Try that
and see how it works for you.
It changes some of the way of doing things, like now you actually set the
suites inside the _config.py file.
Kelly
kdLucas
On Tue, Dec 6, 2016 at 11:45 PM, Marc<EMAIL_ADDRESS>wrote:

Hi Kelly/GCPTeam,
We are having this weird issue under testChromePrintPageRange and
testLocalPrintPageRange. This test will try to set the page range to 2-3
but as you can see from the attached screenshot, it failed to do that.
These two tests are frequently failing using Windows 10. We are not sure
if this is OS related. We've tried running this test using Windows 7 and
everything works fine, there are no failures. We also suspect that this
is
a network connection related issue. Our internet connection is sometimes
slow so maybe that's the cause of this failure.
Windows 10 - Fail
[image: testlocalprintpagerange failure - win10]
<https://cloud.githubusercontent.com/assets/19194749/20959008/c9e66340-
bc93-11e6-9982-e5ae56741cc5.png>
Windows 10 - Fail
[image: testchromeprintpagerange failure - win10]
<https://cloud.githubusercontent.com/assets/19194749/20959033/d9215a40-
bc93-11e6-8450-5840eb778e86.png>
Windows 7 - Success
[image: testchromeprintpagerange - success - windows 7]
<https://cloud.githubusercontent.com/assets/19194749/20959004/c0b18bb0-
bc93-11e6-9adf-b9d95cf22543.png>
Thanks,
Marc
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/cloudprint_logocert/issues/95, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/ADk_fWbMTuNReay_
qpCKejsbWTywjqYWks5rFmQwgaJpZM4LGSgP>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/cloudprint_logocert/issues/95#issuecomment-265389861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASTjffvq3VDxZpwl_hEjfy9TMfYUykppks5rFnKNgaJpZM4LGSgP
.

Great, we'd like any feedback you can give us on version 2.0 of the tool.

Hi Kelly,
Here's what we've found while running the test tool 2.0:
During startup of the tool we've encountered this error:

And there's also another error which will only show if the printer model/name has a Space (' ') in config.py

Thanks,
Marc

Hi Marc,
I am the developer of the logocert_api branch.
This branch is not quite production ready yet. All existing test coverage should be there, but it's still undergoing code reviews and will need to be tested thoroughly after. This branch will continue to see quite a bit of changes until it is fully ready. Also, I've not started testing logocert_api on Windows yet. The  arrow and random character strings are used for colorizing text in Unix. Please bear with me while I push through testing in the coming weeks.
However, It is awesome to see this branch being Alpha-tested and I am happy to hear feedback from you and help along the way.
Addressing the issues you encountered:
Startup error:

The setup code has been replaced in the past week (among many other updates) and the error should not occur anymore. Please pull from the latest logocert_api branch.

Registration error:

From looking at your Privet services, I'm guessing that printer name you have in _config.py is 'ModelName' and the error occurs when 'ModelName ' is used. What is likely happening is that you already have registered printers under your test account that start with 'ModelName '.

For this test specifcally, the GCP /search interface is used to find all registered printers under your test account that starts with the _config.py name you've set. The test expects this list to be empty because the device should not be registered yet. Since there are other similarly named printers registered under your test account, it's returning an non-empty list and thus failing.

If this indeed is the case, all you have to do is to keep your test account clean of all printers except the one you are currently testing. In other words, only the printer you are testing should be registered under your account during the time of testing (only for the suites that require the printer to be registered however). I am open to adding an update to strictly match the specified printer name also so other printers can stay registered.

Hopefully this solves your problems!
Cheers,
Kevin

Marc, please grab the latest source and try again, as we want to get as much feedback in version 2.0 of the tool on as many platforms as possible. I think most of our development has been on Linux, so it would be interesting to get OS X or Windows users to provide some feedback.

Note: once we merge the logocert_api branch back into the main branch, we'll close this issue.

I apologize I wasn't able to perform this test on the new tool. Was really
busy with other works. I've already introduced this new tool with our test
engineers and I think they're already planning to perform it.
Btw, you may close this issue now since this specific test already passed
the last time I've tried it (old tool).
Thanks!
On Fri, Feb 17, 2017 at 4:58 AM, Kelly Lucas<EMAIL_ADDRESS>wrote:

Note: once we merge the logocert_api branch back into the main branch,
we'll close this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/cloudprint_logocert/issues/95#issuecomment-280458266,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASTjfYNxGNtJlqbt1fHzV5CT7aYxEMINks5rdLh3gaJpZM4LGSgP
.

Sounds good, and thanks for the update.


Angular 12 / 13 Support

The following error occurs on a fresh Angular 13 project:
ng add ngx-deploy-ftp

ℹ Using package manager: npm
⚠ Unable to find compatible package.  Using 'latest' tag.
⚠ Package has unmet peer dependencies. Adding the package may not succeed.

The package<EMAIL_ADDRESS>will be installed and executed.
Would you like to proceed? Yes
✔ Package successfully installed.
An unhandled exception occurred: NOT SUPPORTED: keyword "id", use "$id" for schema ID
See "/private/var/folders/65/0k8dncr10pj8lqmh192hjqgc0000gn/T/ng-5Dkb7c/angular-errors.log" for further details.

This is well-know breaking change of Angular 12, please see this commit:
https://github.com/angular-schule/ngx-deploy-starter/commit/55b7c98dd7e8094ec68db232aa1abd4a2b4650a8
We also added support for Angular 13 to the npx-deploy-starter demo here:
https://github.com/angular-schule/ngx-deploy-starter/commit/337e3c6d9edc22746528f2b2e9a33da7dbabd3d3
Greetings,
Johannes

It would be great if there will be a new release with Angular 12 / 13 support.

@JohannesHoppe @twiedow I'll make the required changes as soon as I find the time to.
PRs welcome

@JohannesHoppe @twiedow support updated via 1.0.0-next.2 which was published to npm just now

Such a version will be ignored by yarn, and it will still use the old version. Please bump to 1.0.0. See https://github.com/angular-schule/angular-cli-ghpages/issues/140

@JohannesHoppe I'm sorry - I didn't know about such an issue with yarn. 1.0.0 is up now


Improve definition GetLeadingNonServiceArgs to keep parameter names

This is the current definition of GetLeadingNonServiceArgs:
type GetLeadingNonServiceArgs<Args> =
	Args extends [...BrandedService[]] ? []
	: Args extends [infer A, ...BrandedService[]] ? [A]
	: Args extends [infer A, ...infer R] ? [A, ...GetLeadingNonServiceArgs<R>]
	: never;

Unfortunately, it loses the names of the parameters:

However, if we change the definition to this:
type GetLeadingNonServiceArgs<TArgs extends any[]> =
	TArgs extends [] ? []
	: TArgs extends [...infer TFirst, infer TLast] ? TLast extends BrandedService ? GetLeadingNonServiceArgs2<TFirst> : TArgs
	: never;

The parameter names are kept:

I don't know how this affects type-checking performance though.

@hediet I think you should just submit this as  a PR :)


Remove old dummy recycler

And rename NewDummyRecycler, if there is no old one it does not make sense to only have a new one ;)

see https://github.com/peq/WurstScript/pull/435


Maintaining the contribution guide

Description

Updates to contribution guide to keep it current.

Maybe the "Contributing Analyzer Rules" should be set as a ##-level title? I feel it's an important section that gets lost under "Development"


Add to LinkedIn Profile removed Credential URL and retains Credential ID

This may have been wrong in the PR, but it's definitely wrong in the build.
Add to LinkedIn Profile should have the VerifierPlus public share URL and no Credential ID.

Dmitri will test after publishing LinkedIn date fix.

@dmitrizagidulin - please check to see if this was included in recent build.

Ah ok, I think I may have misunderstood the issue originally. The Credential ID /is/ still being included. I'll remove it and republish.

@dmitrizagidulin - can this be closed?

Looks good @dmitrizagidulin

@dmitrizagidulin For LCW v2.0.11-Build 66, both iOS and Android, I'm still seeing our urn:uuid... in the Credential  ID field when Adding to LinkedIn.

I've confirmed that credential id is empty when sharing a credential to LinkedIn in both iOS and Android.


CORS errors in blockstack:react authentication

See: https://forum.blockstack.org/t/dealing-with-cors-errors-in-blockstack-auth-and-react/2592
fix is simple - in package.json start command for webpack dev-server should be runned at same origin as blockstack browser:
"start": "webpack-dev-server --host <IP_ADDRESS> --port 3000",

Is your native Blockstack Browser opening on <IP_ADDRESS>? Mine opens on localhost, and that's the behavior I've seen for everyone else, too.

No, it's runned at localhost. <IP_ADDRESS> is localhost and in this configuration it works with the browser runned at localhost. Anyway if I was the only one after using this generator eccountering this error just mark the issue as invalid.

Ah, I see. Yeah, since both the app and the browser are on localhost, and I haven't been able to reproduce this issue, I'm going to close. Thanks for reaching out!


FR: Add effective fee (or fee data) to getAssetTransfers

Hello.
Is your feature request related to a problem? Please describe.
(effective) Fee data is vital for any serious implementation of transaction history. Is there any chance it could be added to the alchemy_getAssetTransfers method?
Otherwise, for each transaction, getTransactionReceipt has to be called, which basically re-fetches already known data with the addition of fees and would massively bloat necessary request count.
Describe the solution you'd like
Along with other fields, effective fee amount would be returned.
Thanks

@xdzurman Since the the transaction receipt payload is quite large, it'd take a nontrivial amount of eng work to support. We'll look into making batch requests on top of the existing payload.

Updating with blocked ticket on #159.


Parallelization opportunities

Part of the gap observed in #119 is due to webp-animation using multi-threading. libwebp also has a multi-threading option for still images. But right now image-webp is single-threaded.
I think there are some opportunities for easy wins from parallelization.
Decoding frames in parallel
Each frame is contained in its own ANMF chunk which specifies the length, so we should be able to read the raw bytes and spawn the decoding on a thread pool quite easily.
Complication: this may dramatically increase memory usage on machines with lots of cores, or if the decoding of the frames performs faster than compositing them and uncomposited frames pile up. We will need to introduce some sort of backpressure mechanism so that it doesn't happen. For example, decoding is only allowed to have at most X frames or at most Y megapixels in flight. We know all the frame dimensions so it's not too hard to enforce, but still adds some complexity to the code as opposed to just slapping .par_iter() on it.
Compositing frames in parallel
In the general case compositing is inherently sequential: it requires the previous frame to be fully composited before the next one can start. However, each row in the image can be processed independently, so we should be able to just use .par_chunks_mut() on the buffer and composite each row separately.
The profile in #119 shows that we're spending a lot of time just doing alpha blending, so this could translate to significant performance gains.
Still images?
libwebp also has optional multi-threading support for still images. I've benchmarked dwebp compared to dwebp -mt on a lossy image and only saw a 20% improvement in performance. The single-threaded bottleneck in #71 also makes it seem like we won't get much out of single-image parallelism. I don't think it's worth the trouble for static images.

I think I've overcomplicated parallelizing animations. Having just two threads - one for decoding, one for compositing - is going to be almost as good as it's going to get.
Since compositing is inherently sequential, decoding many frames at the same time wouldn't get us improved performance. They will be consumed one by one anyway, so if we cap the amount of frames held in memory at the same time the pipeline is going to fill up to that amount and then process one frame at a time anyway.
And compositing in parallel isn't super useful because it only needs to go as fast as decoding, and beyond that we won't get any gains because the decoding thread becomes the bottleneck.

if we cap the amount of frames held in memory at the same time the pipeline is going to fill up to that amount and then process one frame at a time anyway.

This isn't true. If decoding is 10x slower than compositing then once the pipeline fills up, we'll start by pulling one frame to composite and starting one frame to decode. But nine additional frames will be pulled from the pipeline for compositing before the first additional frame decode finishes.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing 51319db...9b1f09c:
  > 9b1f09c upgrade to latest dependencies (# 5681)
bumping knative.dev/pkg 2bd91f7...82f3a9f:
  > 82f3a9f Add triggers to the downstream test (# 2236)
  > e957ee5 Drop several deprecated APIs. (# 2228)

/lgtm
/approve


Implement GK sketching on GPU.

The whole sketching pipeline is available on GPU.  Sorted entries are no longer copied back to host.
Now GPU is able to load extremely wide and sparse datasets like URL.
Bottleneck on Device Quantile DMatrix is now on Ellpack construction instead of sketching.  But overall performance is quite good now.
Unifying some functions between sketching on DMatrix and Adapter.
Now we need to track memory usage on the whole pipeline, which is tricky but still manageable.  Before having an inplace sketching algorithm, I couldn't think of an alternative.  Memory usage calculation is unified between test and real algorithm.
Tests for GK are added.  The distributed test requires dask to setup the environment then run gtest.  This way we have a finer test to compare the result.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #5846   +/-   ##
=======================================
  Coverage   82.61%   82.61%
=======================================
  Files          12       12
  Lines        2727     2727
=======================================
  Hits         2253     2253
  Misses        474      474

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 dcff96e...79eb6af. Read the comment docs.

It seems you accidentally changed the permissions for some files.

I corrected them.  It's very distracting during development ...

Added stronger tests on distributed environment.


I added a TODO lol


Investigate adding a generic, stylable, configurable login control that people can just drop into their game, configure, and automatically have full login capabilities.

Fixed via the FirebaseUI controls


feat(avatar): update avatar color contrast

WIP: contrasting Avatar gradient

:hammer_and_wrench: Type Of Change

[ ] Fix
[ ] Feature
[ ] Refactoring
[ ] Documentation

:book: Description

:bulb: Context

:pencil: Checklist

[ ] I have reviewed my changes
[ ] I have added tests
[ ] I have added all relevant documentation
[ ] I have validated components with a screen reader
[ ] I have validated components keyboard navigation
[ ] I have considered the performance impact of my change
[ ] I have checked that my change did not significantly increase bundle size
[ ] I am exporting any new components or constants in the index.js in the component directory
[ ] I am exporting any new components or constants in the index.js in the root

:crystal_ball: Next Steps

:camera: Screenshots / GIFs

:link: Sources

Abandoning, in favor of https://dialpad.atlassian.net/browse/DT-1176


Where is the implementation of error estimation？

See the error estimation based on variational sampling in the “verdict..” paper, but did not find the implementation and corresponding API interface in the code.  Why?
Thanks for your time.

Please find the error estimation logic here.
Since then, we have made much change to this repository to test different ideas.


Separate IO from txBuild and txBuildRaw functions

First step towards condensing modules responsible for reading and validating the things required for transaction construction. The end goal is to expose these modules via cardano-api so users can implement their own transaction construction and/or validation functions.

bors r+


feat: finish role_store implementation

Feature Request
role_store has been partially done but some functions are missing. The issue consist of finishing it's implementation.
Following this PR: https://github.com/keep-starknet-strange/satoru/pull/420
And this issue: https://github.com/keep-starknet-strange/satoru/issues/241
Also add tests.

Happy to take this

Assigned !


Stop deduplication across products

When using a parser that has a dedupe algorithm set to DEDUPE_ALGO_UNIQUE_ID_FROM_TOOL_OR_HASH_CODE, such as Veracode, deduplication would occur between products. This can be seen by importing the same report into two different products

nice find, could be something to add to the unit tests as well, i can take a look.


As a user, I want to be able to add new patrons to my library.

As a user, I want to be able to add new patrons to my library.

Done as of b6d1e952080d12768a90fc22b453ab34fe26daac


 There were about 200-300 Bots, and our servers weren't able to endure that.
If you provide such tools to some bad kids on the internet, do you have any kind of prevention method for that? How can we protect ourselves from this?

How do you know it's actually this tool? The tool is used to stress test my plugins, but its too simple that it properly won't used in any real attack.
To prevent the attacks, you could use firewall tools and anti bot plugins. There is wide variety of software and services that can help you there. Antibot plugins often handle the request too late and suffer from a bad BungeeCord event architecture. Firewalls or other reverse proxies could handle the easier, because they don't have any Minecraft logic to process and they handle the packet on much higher layer. There are also services from server hosters or CDNs to handle attacks on the IP layer. Furthermore even CloudFlare has a Minecraft related service.
However before picking the service with the highest price, you should analyze the bottlenecks on your server. This is what this tool is all about. Create a copy of your server. Switch it to offline mode, to allow fake accounts and then generate syntactic load. Now you could attach a profiler to it and check where your bottleneck is. Is a specific plugin generating too much CPU usage. Is your server bottlenecked by I/O (-> maybe load it async). Are there any blocking calls on the main thread?
Your solution should depend be specific to your problem. You could also integrate something dependent on the incoming load. For example: GEO-Block the IP ranges, switch onlinemode, only allow existing players, disallow any new join attempts, drop connections on IP Layer.


Rewrite structs using pure cffi

Right now we have dependencies on :cl-autowrap & :cl-plus-c because, at the time we wrote struct support, we didn't understand expand-to-foreign & co and used those libs to 'get around' limitations we thought we were under.
It would be good to get a grasp on whether data should pull to a struct type or a list (as it does now) it may be nice to have every gpu-struct have a lisp struct. We would then have a pour-into function to take the lists and populate the structs.

Fixed on feature/move-away-from-autowrap


Easy pick codelenses with keyboard shortcut

Signed-off-by: Gustavo Sampaio<EMAIL_ADDRESS>
This PR closes #41579
This adds a new command that when invoked, shows in a quickpick all the codelenses available in the current line. This makes it easier to "activate"/run a codelens with keyboard. One can set a keybinding for that command and easily switch between the available codelenses with the keyboard.

This is my first pr! I appreciate any suggestions or comments on where to improve this.

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

I tried debugging the failure in the monaco job but without success. Can anyone point me to the right direction of how I can solve this?

Thanks


WIP: serialization and validation middleware

WIP
Potential ways of breaking it up into pieces: 2 dimensions

Request
Response

Serialization

Validation

Closing in favor of #1588.


Option to disabled or modify HTTP headers

Please describe the feature you would like to see implemented.
Currently, Shopware 6 is setting HTTP headers like referrer-policy, x-content-type-options and x-frame-options.
It would be nice to have an option to disable or modify these headers, since I'd like to set these headers via my Nginx configuration instead of letting it be handled by the web application (Shopware).
Can be found in src/Core/Framework/Routing/CoreSubscriber.php
c4d2a49fff2f4a60230b37a157127808d99c668d

@pweyck I'd do this myself, however I'm more of an administrator than a coder and I don't know how to implement a function that checks for a config option before applying the headers.
Is this a lot of work or rather easy to implement?
I don't think we'd need a GUI function here, a simple config true / false would do the trick in my opinion.

Check: https://symfony.com/doc/current/event_dispatcher/before_after_filters.html
KernelEvents::RESPONSE

Any chance we'll see this implemented in a Shopware 6.5.x version?


Display toast message when no orders found after clicking on Load more Purchase orders

Current behavior

When clicking on Load More Purchase Order button and if no more orders found then there is no message displayed to the users

Expected behavior

A message (Purchase Orders not found) should be displayed to the user

Steps to reproduce the issue

Go to PO list page
Click on Load More Purchase Orders button
See that there is no toast being displayed

Can you handle fixing this bug by yourself?

[x] YES
[ ] NO

Environment details

Browser:
OS:
Code Version:

Additional information

I'm Interested in this issue , kindly Assign this to me !!

Sure @Shubhamjain287

I'm interested, kindly assign this to me.


Milestone I Report

To Do: As discussed in Meeting-10, it's time to prepare our first milestone report that gathers the major deliverables related to the requirements, design and plan of our project.
According to our meeting, task allocation is as follows:

[x] Executive Summary (Project Description and Project Status): @serdarakol
[x] List and Status of Deliverables & Evaluation of the status of deliverables: @KarahanS
[x] A summary of work done by each member: everyone (refer to related wikipage)
[x] Evaluation of tools and processes that you have used to manage your team project: @kostanya
[x] RAM (responsibility assignment matrix): @simsekhebun, @ooodogodogodogo, @serdarakol, @mumcusena
[x] Project Plan: @sinemKocoglu, @kostanya, @KarahanS, @BElifb
[x] Combining deliverables (project repository, requirements, software design documents, project plan, RAM, communication plan etc.): @mustafa-cihan

We will be working in Google Docs to prepare the necessary items. Refer to our Discord channel for the link.
Please refer to this issue in the related sub-issues.
Deadline:  15.04.2022 @19.00

Milestone I report can be found here.


change go version from 1.9 to 1.8

 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.

#24 3.156 /go/pkg/mod/golang.org/x/exp@v0.0.0-20221204150635-6dcec336b2bb/slog/level.go:95:13: undefined: atomic.Int64

atmic.Int64 was added in go version 1.19


KAFKA-9305: Add version 2.4 to Streams system tests

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Streams' system test job to verify this PR can be found under
https://jenkins.confluent.io/job/system-test-kafka-branch-builder/3650/

Call for review: @mjsax @ableegoldman

Started a new run of system tests: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/3656/

System tests passed: http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2019-12-18--001.1576691765--cadonna--update-systests-24--df644b1/report.html


feat: add tree_exclude sites so that Colombian sequences don't need to be excluded

Description of proposed changes
There are 2 Colombian sequences with 4 reversions to A.1.
These mess up tree topology. So we've excluded them for now.
A better way is to mask these 4 sites (they are in ITR so not surprising there can be issues there) for tree build only (soft mask). That way the tree topology is correct but the sequences show still on the tree.
Test run here: should appear on staging in 30-45 min  for checking
https://github.com/nextstrain/monkeypox/actions/runs/3007463882

I'm curious, why add a new config/tree_mask.tsv instead of adding the sites to mask.bed and mask_overview.bed?

Never mind, now I see this part of the description:

That way the tree topology is correct but the sequences show still on the tree.

Yes indeed - if we masked in mask.bed, it would end up being totally masked hence shortening the branches and reducing the clock rate.

Yes sorry for not answering @AngieHinrichs! Using a separate file, we can exclude sequences only from the tree topology inference, but still use them for other steps (branch length, ancestral reconstruction). This is a nice solution for sites that are right in most sequences (they will still be informative for branch length in most of the tree & will be reconstructed appropriately most places) but may be messing up the topology somewhere (like false reversions). If it site itself is a problem (in a repeat, repeated primer error, area of generally bad coverage like start & end) then instead, we'd just exclude it from ever being used for anything.


Current needs to vary between rest/action for plunger motors

overview
This issue is for:

Bug/feature
API

expected or desired behavior
The plunger motors should be set to low current (~100 mA) when not in use. Before issuing move commands to the plunger, current should be increased to high current (~500 mA) for actions. Return to low current after actions. Issue a dwell command after issuing current commands.

Fixed in #396


Add changelog and update version of miden-objects.

This PR adds CHANGELOG file and updates the version of miden-objects to v0.1.1 in preparation for v0.1.1 release.

miden-objects v0.1.1 is now on crates.io


Some discussion

Hi bro @dvbui

Great progress!
Do you need a sub-domain at vietspeak don't you? Please tell me.
I can work with front-end, so please tag me if you have no much time to handle it.

Good afternoon,
My current plan is to make sure that the new VietSpeak Bot becomes available ASAP for beta testing, so a front-end or a sub-domain is not needed at the moment. The current front-end is for testing purposes only and nor for production.

Sure! But no need to hurry.


enhance: TBIExtensions with NavigationToFirstPage

What would you like to be added:

NavigationToFirstPage that pops entire backstack
option to specify behaviors/actions are for on-click(when already selected; current-impl) or on-selection (new; to add)

Why is this needed: n/a
For which Platform: all-of-them
Anything else we need to know

current impl:

.OnClickBehaviors=None|BackNavigation|ScrollToTop|(Auto=BackNavigation or ScrollToTop)
.OnClickBehaviorsTarget=(optional: UIElement as root for visual-tree discovery, or if omitted the parent of TabBar)

implemented in #293


[STARTRAIL-682] Custom history review action for subgraph

custom history type event added and createCustomHistory event name changes

Sorry by mistake while resolving conflict merged, reverted into next PR,


Lock icon disappear when deploying

Current Behavior
When I lock a flow from the information sidebar and I click on deploy (it doesn't matter which one) the lock icon of the flow disappear and comes back when I fully reload the site.
Expected Behavior
Flow locked Icon should still be visible when clicking on Deploy.
Steps To Reproduce

Click on the lock icon of a flow from the information tab.
Click on Deploy (doesn't matter which one).
Now lock icon disappeared from the flow that needs to be locked.

Example flow
No response
Environment

Node-RED version: 4.0.2
Node.js version: 20.17.0
npm version: 20 I guess
Platform/OS: FlowFuse
Browser: Version 1.69.168 Chromium: 128.0.6613.138 (Official Build) (64-bit)

The ensureUnlocked function creates a conflict because when the event is triggered the locked property is false and then the loop below locks the workspace.
https://github.com/node-red/node-red/blob/a3c5b75368efcbdac690e4d3e2d0b54ece117c74/packages/node_modules/%40node-red/editor-client/src/js/ui/deploy.js#L651-L661

I'm preparing a fix 😉


Why are these YUL recs dying in pipeline?

attribution assignments on the ids on this work:
https://lux-front-sbx.collections.yale.edu/data/text/55d93cc2-7a93-42d7-aed1-16a0b9452431
are not in YUL recordcache2, but made it past re-ider in testing.

They don't reidentify properly in the referring records for some reason. By the end of the process, they're fixed, so running reidentify on the recordcache records now works just fine. OCLC doesn't have a name (made a Yale issue for it) but the YUL record does.

another example: creation > influenced_by
https://lux-front-tst.collections.yale.edu/data/concept/232e1dd6-0d83-4ab6-a723-df79e1a91065


Improve the documentation, pull in README

This makes the docs a bit nicer, pulling in all the great information in the README

Thanks!!

Pushed the updated docs, looking much better now :+1:


Apply query order change to 2.2 docs

Follow-up to https://github.com/cockroachdb/docs/pull/4154.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/01d7c5e93fd574cc75fa3d555b47ca7eefcf76b5/


Add new poudriere audit command to check repositories for vulnerable …

…packages

@bdrewery Do you have any thoughts on this?

I like it!

I forgot about the pkg side of the changes.. see freebsd/pkg#2247

Hmm I still see ${PKG_BIN} in latest commit.

 Done now

Let's get the pkg change in first before merging this?

the proposed patch for pkg would have never worked however in git pkg now has support for audit -d
https://github.com/freebsd/pkg/commit/5860bc22ede55eebe37769205ae8ca7014c89a24

Is this ready to land now? or do we need to refresh it?


Add a Github action to update the GraphQL schema nightly

configure introspection
update schema

OoOOOpps, this was fired a bit early while testing the update-schema.main.kts script 🙈 .
The update-schema.main.kts does:

fetch the latest schema
check if it has changed
if changed, opens a PR (this is where this PR got created...)

Anyways, now that the PR is here, I guess it's a good time to start testing it. This is setup to run everyday at midnight "Github Action Time", whatever that is (did I already say time is hard 😅 ?)

Nice!


Search works incorrect when using synonyms

i have current synonyms {"ä":"ae","ö":"oe","ü":"ue","ß":"ss"};
if i have text "oeoe" and search term "oeoe" it only highlights "oeo"
there's a code pen https://codepen.io/kot-pilot/pen/GBjzEL
maybe i just make something wrong?

Hello @kot-pilot,
You don't need these synonyms, as this is what the diacritics option is for.


Page header cleanup

Fixes #1586 (expanded), #1972, and #1716.
Replaces #1687 for this effort.

Markup is all updated, and the components have all been structured better.
Support for icons & breadcrumbs has been added (Setup example),
Anew prop to be used with object-home called nameSwitcherDropdown has been added to better reflect the SLDS structure.
onRenderActions and onRenderContent now consistently refers to the same items across variants.
Also added a SLDSPageHeaderControl component to handle the .slds-page-header__control wrapper as discussed.

CONTRIBUTOR checklist (do not remove)
Please complete for every pull request

[x] First-time contributors should sign the Contributor License Agreement.  If you haven't before, wait a few minutes and a bot will comment on this pull request with instructions.
[x] npm run lint:fix has been run and linting passes.
[x] Mocha, Jest (Storyshots), and components/component-docs.json CI tests pass (npm test).
[x] Tests have been added for new props to prevent regressions in the future. See readme.
[x] Review the appropriate Storybook stories. Open http://localhost:9001/.
[x] Review tests are passing in the browser. Open http://localhost:8001/.
[x] Review markup conforms to SLDS by looking at DOM snapshot strings.

REVIEWER checklist (do not remove)

[ ] TravisCI tests pass. This includes linting, Mocha, Jest, Storyshots, and components/component-docs.json tests.
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

Required only if there are markup / UX changes

[ ] Add year-first date and commit SHA to last-slds-markup-review in package.json and push.
[ ] Request a review of the deployed Heroku app by the Salesforce UX Accessibility Team.
[ ] Add year-first review date, and commit SHA, last-accessibility-review, to package.json and push.
[ ] While the contributor's branch is checked out, run npm run local-update within locally cloned site repo to confirm the site will function correctly at the next release.

Please assign the PR to me when it is ready for review @kevinparkerson

To much spacing on https://design-system-react-co-pr-2011.herokuapp.com/?selectedKind=SLDSPageHeader&selectedStory=Base with controls&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

Why are these gray? https://design-system-react-co-pr-2011.herokuapp.com/?selectedKind=SLDSPageHeader&selectedStory=Object Home&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel
I think they are the wrong variant.

It looks like Breadcrumb no longer has bold text in it. https://lightningdesignsystem.com/components/breadcrumbs/#Base
That's probably the issue here:
https://design-system-react-co-pr-2011.herokuapp.com/?selectedKind=SLDSPageHeader&selectedStory=Related List&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

https://lightningdesignsystem.com/components/page-headers/#Related-List

https://lightningdesignsystem.com/components/page-headers/#Related-List

meta info section font size looks too small:

https://design-system-react-co-pr-2011.herokuapp.com/?selectedKind=SLDSPageHeader&selectedStory=Related List&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

@kwilloughby This has the Setup PageHeader that you requested in it and is viewable.

@interactivellama updated everything you commented on thus far

@kevinparkerson
http://localhost:9001/?selectedKind=SLDSSplitView&selectedStory=Base%3A Open&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

See
http://design-system-react-components.herokuapp.com/?selectedKind=SLDSSplitView&selectedStory=Base%3A Open&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel for comparison.
This might mean that the old API navRight and contentRight aren't working.

@interactivellama this is ready for re-review now!

@kevinparkerson & @interactivellama, looking good. Don't forget about this issue.
https://github.com/salesforce/design-system-react/issues/2035

@kwilloughby-sf thanks, almost forgot about that one! I'll add it in a separate PR


WRR-6697: Fix node_js to use lts version only on travis

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
npm intall not working on node v23 because of node-canvas
https://github.com/enactjs/sandstone/pull/1723/checks?check_run_id=31623413345
So we decided to change the node version to lts/*
Resolution
Fix node_js to use lts version only on travis
Additional Considerations
Links
WRR-6697
Comments
Enact-DCO-1.0-Signed-off-by: Taeyoung Hong<EMAIL_ADDRESS>

Note that lts version is 22 as of now. Due to unstable dependencies whenever a major version of node, we'd like to use lts for Travis CI.


Modified password in documentation to match app

Modified password in documentation to match app

Yep, nice catch - thanks!


Remove responsive layout

Hi,
I need some help with the responsive layout.
I set the responsive to false so that the items don't stretch, but they keep stretching at higher resolutions.
I would like the GridItem to be the same width as it was created, regardless of the resolution.
Thank you

Wouldn't be easier to apply CSS styling or use something like bootstrap?

Thank you for the answer, but how?
I need to lock some GridItems to not scretch when resolution change.

Hi, checkout this if you're using or planning to use Bootsrap 👉🏼  https://getbootstrap.com/docs/4.5/layout/grid/ otherwise, you can "lock" pixels using CSS rules.
Just inspect the element > find CSS class > add it to your .css file and override the parameters. Use !important if needed.
That's what I would do.

Thank you for your help!
That will solve my problem, I can set specifc class for the elements that I want to lock the size and use css to set max-width for them.
Thank you again!


Cate-core build fails on windows with several errors

Expected behavior
Cate-core builds without errors
Actual behavior
CondaVerificationError: The package for pytest located at D:\Miniconda3\pkgs\pytest-3.2.2-py_0
appears to be corrupted. The path 'resources/cli-64.exe'
specified in the package manifest cannot be found.
... (NEXT ERROR ) ...
ERROR conda.core.link:execute_actions(330): An error occurred while installing package 'conda-forge::pyshp-1.2.12-py_0'.
CondaError: pyc file failed to compile successfully
python_exe_full_path: D:\Miniconda3\envs\cate-core\python.exe
py_full_path: D:\Miniconda3\envs\cate-core\Lib\site-packages\shapefile.py
pyc_full_path: D:\Miniconda3\envs\cate-core\Lib\site-packages_pycache\shapefile.cpython-36.pyc
Attempting to roll back.
... (NEXT ERROR ) ...
ERROR conda.core.link:execute_actions(330): An error occurred while installing package 'conda-forge::h5netcdf-0.4.1-py_0'.
CondaError: pyc file failed to compile successfully
python_exe_full_path: D:\Miniconda3\envs\cate-core\python.exe
py_full_path: D:\Miniconda3\envs\cate-core\Lib\site-packages\h5netcdf_init.py
pyc_full_path: D:\Miniconda3\envs\cate-core\Lib\site-packages\h5netcdf_pycache__init_.cpython-36.pyc
Attempting to roll back.
Steps to reproduce the problem
conda env create -n cate-core --file environment.yml
Specifications
cate 0.9.0.dev6 on Windows 10 x64

All thee libraries has been updated last week.
A quick workaround is to freeze pyshp, h4netcdf, pytest versions and use previous release.
hotfixes

pyshp = 1.2.11
h5netcdf = 0.4.0
pytest = 3.2.1

Success with

pytest >=3.1,<3.2
pytest-cov >=2.5.1,<2.6


Does not detect group messages correctly?

I am wondering why this is not reported yet.
I am using the nodejs sdk. The personal messages with the bot is working fine and the bot is recognizing the messages send.
But in group messages, when we mention "@Bot name command" The message is not recognised since the "@Botname" gets added to the message. I am not sure if its the desired feature. But my messages are not recognized in the group conversations due to this.

https://github.com/Microsoft/BotBuilder/issues/2196

@aakashvit Its in C#.
Is there nodejs code for it?

@aakashvit BTW I think this is not my issue.

related: https://github.com/Microsoft/BotBuilder/issues/2419

@Nisthar - What channel are you using for the group messages?

@Nisthar are you still experiencing the issue using<EMAIL_ADDRESS>

@nwhitmont I am using skype
I don't think its fixed. But i managed to temporarily fix it.
It might be still broken in the core library. I mentioned the error in my first post of this issue,

function public_channel_ensure(session){
var obj={"isGroup":false,"isAllow":false}
var address=session.message.address;
var message=session.message;
var botname=address.bot.name;
if(address.conversation.isGroup&&message.text){
var msgtoken=message.text.toLowerCase().trim().split(/\s+/);
var attachments=message.attachments||[]
if(attachments.length==0&&botname&&msgtoken.indexOf("@"+botname.toLowerCase())>-1){
obj["isGroup"]=true;obj["isAllow"]=true;
return obj;
}
else{obj["isGroup"]=true;obj["isAllow"]=false; return obj;}
}
else{return obj;}
}


fix(namespace): show yaml in namespaces without resource quota EE-6705

As discussed closing this PR


about attention map

Hi, @umyta ,
Thanks for releasing your package. I'm interested in how to make Fig.6 and Fig.7 in your paper. Could you release the corresponding code? Or could you list the main steps or hints on how to capture and visualize the attention weights?
THX!

Hi @amiltonwong , thanks for your interest! If I remember this correctly, you can save these weights and visualize the value of each corresponding point against the rest in terms of a color range for both figures. It is possible that you may not learn exactly the same set of weights, hence your image may not be exactly the same as what is in the paper. Hope this helps.


if the browser has no web3 wallet，such as metamask, the user click the connect button will no respone

if the browser has no web3 wallet，such as metamask, the user click the connect button will no respone respond, how to deal with it?

Thank you @vegetax for the report. We already merged the fix and it'll be added in v0.1.158 coming days


Object group rotation

Currently it's possible to select and copy multiple objects at the same time.
My use case for that is making small serene forests of beautiful calming trees, and then splatting them here and there, but it's not possible to rotate those forests as wholes as far as i can see: doing the ctrl+drag thing rotates each individual feature in the group, rather than the group as a whole.
It would be cool if it were possible (or if i knew how) to rotate the entire selection around some single point.

Glorious victory awaits.


PyTorch state_dict to RTNeural JSON

I'm looking for an effective way to export a PyTorch Lightning model to the JSON format that RTNeural accepts. I've seen the previous post about using PyTorch models exported with torchscript, but can't seem to get this to work.
Instead, the PyTorch state_dict dictionary appears to contain all the information required to build a JSON file in the format that RTNeural expects. I understand that this will involve writing some kind of parser, but I'm not sure how to go about this. Any help would be massively appreciated. An example of the dictionary returned after executing the state_dict() method on a model is attached.
Thanks!
state_dict_export.txt

Hello!
For using RTNeural with a specific model like this one, it should be possible to implement the model "by hand":

Convert the state_dict object to a regular JSON file (see here).
Create your model with RTNeural in C++.
Write some C++ code to load the weights into the model directly from the JSON file (here's an example).

Now it would be nicer (though more difficult) to automate this process, so that we can "transform" the state_dict into an RTNeural model file like we're able to get from TensorFlow. The RTNeural model file needs to have the following things:

The input shape of the network
A list of layers in sequential order
Each layer must have:

The layer type
The layer activation type
An output shape
The layer's weights
Convolutional layers should also provide a kernel size and dilation

With that in mind there's a few things about the state_dict that you shared which bring about a few questions:

The state_dict doesn't seem to show the "type" of each layer? Some layers are labelled "hidden" or "residual", but I'm not sure if those can be translated into things like "Dense" or "Conv1D". Maybe the layer type could be inferred from the shape of the weights?
I don't see any activation functions mentioned in the state_dict. Maybe this network does not use any activations?
I'm not sure I understand the order of the layers in the state_dict. For example, it seem like the input layer is second to last?

It's also worth mentioning that RTNeural currently only supports automated loading for sequential networks. More complex network architectures will require a "by hand" approach like the one mentioned above. If you have suggestions for improving the RTNeural model format, that would be welcome as well!
Hope this information is helpful!

Thank you so much for your reply Jatin, this is really helpful!
Would you be able to explain a little more about the structure of the RTNeural model JSON? How exactly should it be laid out?
Thanks again!

No problem!
For the RTNeural format, I would definitely suggest checkout out this example model file, but the basic format goes:
{
    "in_shape": [
        null,
        null,
        1
    ],
    "layers": [
        {
            "type": "dense",       // layer type goes here
            "activation": "tanh",  // activation type goes here
            "shape": [
                null,
                null,
                8                  // layer output shape goes here
            ],
            "weights": [
                ... // layer weights (including biases) go here
            ]
        },
        ... // the rest of the layers continue on here
    ]
}

No problem!
For the RTNeural format, I would definitely suggest checkout out this example model file, but the basic format goes:
{
    "in_shape": [
        null,
        null,
        1
    ],
    "layers": [
        {
            "type": "dense",       // layer type goes here
            "activation": "tanh",  // activation type goes here
            "shape": [
                null,
                null,
                8                  // layer output shape goes here
            ],
            "weights": [
                ... // layer weights (including biases) go here
            ]
        },
        ... // the rest of the layers continue on here
    ]
}

That's perfect, thank you for your help.
In the example model file, I noticed that the final array of weights for each layer is separated from the rest. Is there a reason for this? Are these the bias values rather than the weights?
Thanks again!

In the example model file, I noticed that the final array of weights for each layer is separated from the rest. Is there a reason for this? Are these the bias values rather than the weights?

The order of the weights for each layer is whatever the TensorFlow getWeights() method returns for a given layer, but yes, I think for most layers that ends up being the layer biases.

What should the JSON activation function be for layers with a gated activation function, as seen in WaveNet, for example?

RTNeural doesn't currently support gated activations, but it would be great to add that support. Are you familiar with how some existing framework implements gated activations? (For example, most of the layers currently implemented are based on the Keras implementation)

Hmm, I'm not familiar. Do you have an email where we could speak further? Thanks! :)

Sure thing, feel free to email at<EMAIL_ADDRESS>


Channel improvements

Adds type-safety to the Message in handle action
Renames SocketInfo to ClientInfo
Adds the whole bunch of documentation
Mark SocketHub and SocketMiddleware classes as internal

CC: @baronfel @robertpi @isaacabraham

I like these changes a lot. Obviously docs are great, but the typed message handlers are nice as well.  One thing that I see being challenging is people wanting to model channel messages as DUs and then running into deserialization errors.

channel messages as DU

Well, it’s problem everywhere - in controllers, in normal http handlers. And it depends - channels are using same IJSonSerializer as any other feature so the behavior should be consistent

Looks good to me, all I was doing was using the json serialize to deszeialize the message, so will save me a few lines.


Big Sur: You do not have permission to open the application “Crypto Price Widget”.

"Contact your computer or network administrator for assistance."

Same issue


Offer downloading via GET parameter

e.g. http://kinolien.github.io/gitzip/?download=myrepo/folder

Currently, i don't have plan to setup server to implement it.
And this site is pure client, so if i do this, It will have some "features", like:
you cannot use command curl (or similar) to request, (only get html, not file)
you cannot use XMLHttpRequest to request in javascript, (only get html, not file)
you can use this url in the browser address bar,
you can use <iframe> to request this url.
So, is this what you need? or any ideas? please let me know, thanks.

@KinoLien You can also determine the specified parameters with JS and then deliver the ZIP


Min-height adjustment

Now the min-height is being set to 46px (font-size: 18px). Is there a way to decrease that? I want something like 18px (just one line).
Thanks,
Glauber

Set rows count and set heioght in css
e.g.
<textarea rows="1"></textarea>

    textarea {
      height: 18px;
      resize: none;
    }


add the feature-request template

Issue Number
fixes #19

Describe the changes you've made
Added the feature-request template to follow while proposing a feature

Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't)
NA

A good test plan should give instructions that someone else can easily follow.

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

@KeenWarrior Could you please review this?


Feature/example app 5.0

Preliminary implementation of an example app. Still needs more comments to be added. Tests not implemented yet.

Please remove .DS_Store files from the repo

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: paweljaneczek:x: kkosovskyYou have signed the CLA already but the status is still pending? Let us recheck it.


Make primenet.py work per-directory

Synopsis: to allow multiple job types with one script run.
Hypothetical steps:

Allow multiple --dir options
Require two parameters for each --dir option: 1) work directory; 2) work type
Typical usage should look like, for example: --dir /home/user/work1 PRP --dir /home/user/work2 PRP_DC

I see you are requesting the same --dirs option as is in the PrimeNet script included with GpuOwl, although with an additional worktype parameter.
Porting that --dirs option is actually my top priority after I finish testing the PRP proof uploading (#12). Although, getting different work types on different CPU cores or GPUs is a separate issue: https://github.com/tdulcet/Distributed-Computing-Scripts/blob/66de4840e271e482188fa9c92b72bc436505b9a9/README.md#L208-L209
In Python with the optparse module, one can only have a single parameter per option, so unfortunately, your proposal would not work. In addition, our PrimeNet script currently supports slightly different worktype mnemonics: https://github.com/tdulcet/Distributed-Computing-Scripts/blob/66de4840e271e482188fa9c92b72bc436505b9a9/primenet.py#L698-L699
I would be open to supporting the GpuOwl ones if there were interest in that, although this would also be a separate issue.
Anyway, using your example, I think the usage would be something like this:  --workdir /home/user/ --dir work1 --dir work2, which users could shorten to  -w ~ -D work1 -D work2.

One possible solution to the optparse module limitation is to couple options positionally. So in this case it would be:
-w ~ -D work1 -t 150 -D work2 -t 151
which obviously means: dir work1 worktype 150, dir work2 worktype 151.

Yes, that would be a solution. Although as I said, updating the existing -T/--worktype option would be a separate issue, as it would require first adding support for setting per thread/directory options and I am also not sure how much interest there would be in it. Pull requests are of course welcome! For this issue, I will likely add a -D/--dir option, similar to the --dirs option in the script included with GpuOwl.
For now, it may be easier for you to just run a separate instance of the script for each directory, particularly if you are using it with different GPUs.

@selroc - I added a -D/--dir option in https://github.com/tdulcet/Distributed-Computing-Scripts/commit/b4ae3ca31a09638530eb46233021b658902cb212. The output still needs to be improved so that it more clear which directory/worker/job it is referring to, but I will handle that in a future update. Feel free to create a new issue for your requested changes to the existing -T/--worktype option.


Add Email Config Endpoints

Acceptance Criteria

All tests should pass.
Add endpoints for emailConfig creation, updates, and information retrieval.
Understandable endpoint documentation within the API Docs.

Closing in favor for #10.


Insert mode incredibly laggy and unusable despite higher timeout values

Hey,
I'm just about to switch to this plugin for my setup. I tested it on some new random test files. All fine, works good. Then I switched to a bigger and (for for the language server) quite complex code-base. Now the insert mode is incredibly laggy. So basically makes it impossible to write. I randomly set throttle_time to 300 and source_timeout to 600. I don't know if these time values are helpful. I also set them once to 3s and 6s. Nothing helps. What do I need to do to make my editor usable again?

I think the test file should be added.

Can you post your compe configuration?

Can you post your compe configuration?

Simply this:
set completeopt=menu,menuone,noselect

lua require('compe').setup({
      \   throttle_time = 3000;
      \   source_timeout = 6000;
      \   source = {
      \     path = true,
      \     calc = true,
      \     nvim_lsp = true,
      \     nvim_lua = true,
      \     ultisnips = true,
      \     treesitter = true,
      \     tags = false,
      \     vsnip = false,
      \     spell = false,
      \     buffer = false,
      \     snippets_nvim = false,
      \   },
      \ })

I think the test file should be added.

I assume a single would not help here. My exact example is this repo and then just edit raiden-ts/src/raiden.ts. I'm using the tsserver with nvim-lspconfig. The efm language server is always running aside, but has completion capabilty disabled.

Try disabling all sources and re-enable them one by one. Maybe that will tell us what's causing the lag.

Okay yeah, it is clearly the treesitter source. I can re-enable all my former enabled sources. But when I toggle the treesitter source it switches from super fast to super slow.

That definitely needs some optimisation. In the meantime, I think what you could do is only activate TS source if there's no server running (acts as a fallback for simple completion) and then turn it off when the LSP is attached.

Hmm. My general intention is to use Treesitter only as a faster completion and when the slower LSP hits in, due to the higher priority I only see these candidates. But apparently this assumption is wrong, as the LSP is much faster then treesitter. xD
So as this is just the slow treesitter source, I'll just close the issue. Thanks for the help!

I will look into it. thank you!

@weilbith Hi, I'm improed a bit on the treesitter source. Thank you.

Sorry. I noticed it is still heavy... maybe we should learn treesitter queries if improve it...

Maybe some of the smart guys from the treesitter plugins could provide some useful hints and tips?

We are already using queries, ts_locals.get_definitions uses the locals query under the hood. I think the issue is that treesitter queries are just slow and are not asynchronous, that is why it is freezing the editor. I wrote a pr for nvim-treesitter-context to use locals queries for the context and also accept nested context. I had the same issue, on medium sized files it would completely block the editor, even when the function was throttled. I think the solution is to make the queries asynchronous, although that is hard.


Plugin Not Found

I must be missing something about how zsh plugins work.
I have the following plugins enabled:
plugins=(aws, git, gradle, npm, pip, yarn)
All of them show up in the ~/.oh-my-zsh/plugins folder.
All of them are part of the bundled plugins but only yarn isn't throwing a not found warning.
https://github.com/robbyrussell/oh-my-zsh/wiki/Plugins
This is what I see when I open up the terminal:
Warning: plugin aws, not found
Warning: plugin git, not found
Warning: plugin gradle, not found
Warning: plugin npm, not found
Warning: plugin pip, not found

Any insight would be great! Thanks!

The zsh arrays aren't use commas to separate entries, just use spaces.

Thanks, @mcornella I see what's happening now!

That worked! Thanks @mcornella
That being said, this is quite strange... my ~/.zshrc was working perfectly until last update. 😱

It wasn't working correctly, you just didn't know it. We only added the warning so that you can know it from now on.


require all cores on the same node

see https://www.ibm.com/support/knowledgecenter/en/SSETD4_9.1.2/lsf_admin/span_string.html
fixes #172 and replaces #176

Thanks @willirath for your involvement too!


Pass 0 as translocoParams

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior

If we passing 0 as dynamic value parameter it is will not be showed
Expected behavior

Showing 0 as param value
Minimal reproduction of the problem with instructions
//translation file
"cart": {
    "count": "Cart items: {{value}}"
  }

template
{{t.cart.count | translocoParams: { value: cartItems} }}
What is the motivation / use case for changing the behavior?

Environment

Angular version: 8

Browser:
- [x] Chrome (desktop) version 76
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

It is fixed in v1.7.3. Thanks for the report.

Wow

I can't stand bug :)


Use versionless GRASS GIS (8) instead of hardcoded grass7x

Related to https://github.com/mundialis/actinia_core/issues/331

All checks have now passed.


[BUG]: Awards on Profile page jitter and teleport

Description
I'm using Brave browser with the link on the github repo page. I believe Brave uses parts of Chromium under the hood. Anyways, here's a video to show what I see.
https://github.com/webdevcody/code-racer/assets/66073333/66c76964-8caa-4eeb-a36b-630ab1398e70
(optional) What browsers are you seeing the problem on?
Chrome
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I can work on this

Although not as severe, the issue persists. This time in between animation states. For example on hover, the element will teleport to the animation start location. If you put the cursor somewhere between the animation rest (the default) position and under the drawn frame's current position the animation gets stuck. Also, I'd recommend starting the animation from it's rest position and translating it through the animation to the end to eliminate the initial jump.


Added Eclipse project files and some minor changes.

Why should eclipse project files be part of the project? That makes no sense. There is a deployment descriptor there, which Eclipse should read just fine.

Hi Edvin. For eclipse conversion files, of cource it is up to you. No problem.Others ara some minor corrections. Plus, I will be adding Localication support for Turkish translation. Yours sincerley. Yüksel Penekli Windows 10 için Posta ile gönderildi Kimden: Edvin SyseGönderilme: 25 Haziran 2020 Perşembe 13:37Kime: edvin/fxlauncherBilgi: ypenekli; AuthorKonu: Re: [edvin/fxlauncher] Added Eclipse project files and some minor changes. (#189) Why should eclipse project files be part of the project? That makes no sense. There is a deployment descriptor there, which Eclipse should read just fine.—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

A new full pull request is waiting..


Update to Create React App v3.2.0

I usually give them a few days to make sure there are no show stopping bugs in new releases of create-react-app. So the plan is to do this next week sometime...

Done.


Do we really need notFoundAction?

The question is simple: do we really need notFoundAction?
https://github.com/zendframework/zend-mvc/blob/master/src/Controller/AbstractActionController.php#L43
I don't think so...
My problems:

It's not triggering a dispatch error, while all other and similar (!) errors do. If we can't locate the controller, we trigger a dispatch error, if we can't locate the action inside that controller, we do not trigger any error. What is the reason for handling similar errors in different ways? If we don't have notFoundController, then why have we notFoundAction?
Because it doesn't trigger any error, it's hard to bypass it. If a custom error and exception handling is needed, we can do it easily by attaching custom listeners to the dispatch.error event, but this doesn't work for notFoundAction becasue the missing event. We cannot use the dispatch event neither: with priority 1 or lower it's too late, notFoundAction already returned a ViewModel with 404 status code, with priority 2 or higher it's too early, we don't have the controller yet.

What should be done?

We should remove the notFoundAction and modify onDispatch (https://github.com/zendframework/zend-mvc/blob/master/src/Controller/AbstractActionController.php#L60).
We should check the existance of the method in the DispatchListener (https://github.com/zendframework/zend-mvc/blob/master/src/DispatchListener.php#L77). If the method doesn't exists, we should trigger a dispatch error and set the name of the error to the following:

const ERROR_CONTROLLER_METHOD_NOT_FOUND = 'error-controller-method-not-found';

(I think it would be better than the old ERROR_CONTROLLER_CANNOT_DISPATCH, which means nothing for me, it's too general.)
After that we could do small simplifications in RouteNotFoundReason, and remove weird lines (!!!) similar to this one: https://github.com/zendframework/zend-mvc/blob/master/src/View/Http/RouteNotFoundStrategy.php#L231
By the way, that line and this switch case (https://github.com/zendframework/zend-mvc/blob/master/src/View/Http/RouteNotFoundStrategy.php#L135) perfectly shows how illogical the error handling in it's current form is. There are errors (retrieved via getError from MvcEvent) and there is ERROR_CONTROLLER_CANNOT_DISPATCH.
(The impossibility of retrieving the error type ERROR_CONTROLLER_CANNOT_DISPATCH from the MvcEvent via the getError method can cause further problems too. Let's say we would like to log them...)
Opinions?

One small remark: removing notFoundAction would allow us to remove Zend\Mvc\View\Http\RouteNotFoundStrategy listener (prepareNotFoundViewModel) from the dispatch event listeners.
Currently we are checking the status code of the response for each and every request (https://github.com/zendframework/zend-mvc/blob/master/src/View/Http/RouteNotFoundStrategy.php#L174), which, in my opinion, is a bad practice: 404 status code isn't a general thing in an application. Listeners of the dispatch event should do general things, unless we don't want bad performance.

+1 I'm having the same issue with the notFoundAction in Zend 3. I want to catch dispatch errors but they are never thrown when an action that doesn't exist is called on a controller.
This issue has been opened since Oct. 2016 but it hasn't been assigned to anyone and it hasn't been labelled?

@burrellramone

This issue has been opened since Oct. 2016 but it hasn't been assigned to anyone and it hasn't been labelled?

The zend-mvc component is subject to some changes which are under development.
An assessment of the current problem did not occur previously. So the issue itself is still open and relevant.

This repository has been closed and moved to laminas/laminas-mvc; a new issue has been opened at https://github.com/laminas/laminas-mvc/issues/21.


Incorrect symbol names

Just after you fixed the incbin tool problem, I updated and re-compiled my source using the script - the tool itself works again, thanks for that :).
But now I have a more awkward problem:
src/IceTea.cpp:154:29: error: use of undeclared identifier 'gSTDData'
        {"std.os",          INCBIN_DATA(STD),           INCBIN_LEN(STD)},
                            ^
src/IceTea.cpp:20:27: note: expanded from macro 'INCBIN_DATA'
#define INCBIN_DATA(name) g ## name ## Data
                          ^
<scratch space>:356:1: note: expanded from here
gSTDData
^

I was so confused that it took me a while untill I actually realized how I could best test this. I took the file containing the INCBIN statements and wrote a teensy tiny C++ program:
#include "scripts.rc"
int main() {
  return 0;
}

Nothing special going on - except script.rc, which is the header in which I have my incbin statements. the .rc is probably misleading, but it makes me think of resource container. So that works for me - besides, it has worked for a logn time now :).
 Now, that does explain why my tiny helper macro did no longer work. :)
Looking into incbin.h, this also does not seem to be the intended behavior either. Could you look into this please? This basically broke my app for now. ^^"

That is odd, is there a better forum for us to communicate on. IRC?

I have skype, and there is also Gitter.

Think I fixed it

Try now

Nice!
 Thanks. :)


very noisy start-storybook CLI output

I'm using custom config as suggested in https://storybook.js.org/basics/writing-stories/#loading-stories-dynamically to dynamically load stories.
is there a way to tame the CLI outputs of start-storybook a little bit? cause currently it's too noisy to me

Was just about to ask about this myself. It clashes with my default dev task CLI output. :)

oh ya, another issue I get is the building modules never really get to 100%,
always stuck in 69% but my stories are loaded just fine in the browser

Hey, Guys!
I'm not sure what the reason for the new CLI behavior but in my case, I just need to make the shell window wide enough to collapse all this output into the only one line:

I'm for silencing the webpack progress by default. The output makes configuring compilation on IDEs like WebStorm a nightmare.
Has anyone tried progress-bar-webpack-plugin yet?

This was introduced in version 3.1.2.
Reverting to 3.0.1 removes the progress output. Looking at the release notes, there doesn't seem to be anything related to build progress output.

It was introduced in this commit, that add adds the ProgressPlugin:  https://github.com/storybooks/storybook/commit/7c707f6de2ca392ee6b0e00ce9133059412d1591

Temporary fix is to remove the plugin from webpack config you create:
const webpack = require('webpack');

module.exports = (config, env) => {
  config.plugins = config.plugins.filter(plugin => !(plugin instanceof webpack.ProgressPlugin))

  return config;
};

@thebuilder the release notes are a concatenation of every closed PR that is not labeled with the  cleanup (reserved for minor fixes like typos).
@ndelangen we should be more disciplined about not lumping too many separate issues into PR's

@thebuilder yeah but unfortunately it's not that easy to alter the setting in a project created by CRA or CRNA.
@shilman well, a small & focus PR is always better, right?

@luftywiranda13 @ndelangen did some seriously heavy lifting on the project, e.g. enabling strict linting (something for which we should all be grateful!) and it would have been impractical to split it up into many PR's. the downside is that when we get sloppy, stuff like this doesn't make it into the changelog.

I'm sorry if this inconvenienced you! I can try write a ProgressPlugin myself that doesn't have this issue. but of course the best option would be for me or someone else to actually fix this in webpack.
We should actually file a bug for this on webpack ProgressPlugin.
I have been looking for a good entry point to start contributing to webpack, but I fear that it would mean I'd get too much on my plate. There's only so much I can do.
I'm trying to be disciplined and only do 1 thing in 1 PR, but of course the fact is also there's a lot going on.

@danielduan I have tried the progress-bar-webpack-plugin, it's not as informative as the regular webpack ProgressPlugin.

@luftywiranda13 It's not actually stuck, it does reach 100%, but the terminal doesn't print the output right. It's probably that the ProgressPlugin doesn't account for the max-length (screen-width) of the terminal correctly, causing line that are supposed to be on 1 line wrap to 2 lines.

@thebuilder Thanks for sharing your solution!

@ndelangen I think it's fine with a ProgressPlugin - Just think it should be possible to disable and/or configure it.
A --no-progress command in the CLI options would be a great start. 👍

@ndelangen to have it be configurable from storybook sounds good to me, I agree with @thebuilder

Toggleable sounds OK to me.

So who's interested in turning this issue into a PR?
@luftywiranda13 @thebuilder @danielduan ?

So what hapened here?
Storybook 5.3 still fills up my console with epic spam:

Storybook: Let me tell you a story of high adventure
Me:

You can disable this with:
https://github.com/storybookjs/storybook/blob/c604b2626b70bbd0ddbd38fe61242a9ff79690c1/lib/core/src/server/cli/dev.js#L29
from the docs:

For start-storybook
Usage: start-storybook [options]

Options:

--help                        output usage information
-V, --version                 output the version number
-p, --port [number]           Port to run Storybook
-h, --host [string]           Host to run Storybook
-s, --static-dir <dir-names>  Directory where to load static files from, comma-separated list
-c, --config-dir [dir-name]   Directory where to load Storybook configurations from
--https                       Serve Storybook over HTTPS. Note: You must provide your own certificate > information.
--ssl-ca <ca>                 Provide an SSL certificate authority. (Optional with --https, required if using a self-signed certificate)
--ssl-cert <cert>             Provide an SSL certificate. (Required with --https)
--ssl-key <key>               Provide an SSL key. (Required with --https)
--smoke-test                  Exit after successful start
--ci                          CI mode (skip interactive prompts, don't open browser)
--quiet                       Suppress verbose build output
--no-dll                      Do not use dll reference
--debug-webpack               Display final webpack configurations for debugging purposes

The --quiet option works great, but it also removes the "Storybook...started" message which contains the local URL, and I'm running a mono repo with many storybooks with Nx workspace. Each configured to have a different port, so I want to run multiple storybooks with a single command, but the webpack process messages spam the output. Using --quiet does work but I can't remember what all the different port numbers are.
I found this approach works best for me with Nx. Here is how I run 3 storybooks from projects named project## in Nx.
nx run-many -t storybook -p projectA projectB projectC --parallel=12 --output-style=stream 2>&1 | grep -v "webpack.Progress"