Theoretical problem

Thank you for sharing, is there any relevant essay or document to help understand the corresponding methodology?

Here are three of our papers that describe the building of the model and some of the processes that we use.
General process:
Swirski, L., & Dodgson, N. (2013). A fully-automatic, temporal approach to single camera, glint-free 3d eye model fitting. Proc. PETMEI.
Eye model for calibration:
Itoh, Y., & Klinker, G. (2014, March). Interaction-free calibration for optical see-through head-mounted displays based on 3d eye localization. In 3D User Interfaces (3DUI), 2014 IEEE Symposium on (pp. 75-82). IEEE.
Pupil fitter on P4:
Orlosky, J., Itoh, Y., Ranchet, M., Kiyokawa, K., Morgan, J., & Devos, H. (2017). Emulation of physician tasks in eye-tracked virtual reality for remote diagnosis of neurodegenerative disease. IEEE transactions on visualization and computer graphics, 23(4), 1302-1311.


Update openlayers_interop.js to fix checking for ScaleLineUnit None

If "None" is passed as a unit, OL fails with invalid unit error.  This fix correctly checks if user disables the scale line.

This will fix #74

thanks!


Implement AshGraphql.Error for Ash.Error.Query.ReadActionRequiresActor

Contributor checklist

[ ] Bug fixes include regression tests
[ ] Features include unit/acceptance tests

LGTM, let me know when it's no longer a draft.

LGTM, let me know when it's no longer a draft.

I was going to try to create a test for it but maybe it's not required in this case.

Yeah, I guess a test would be warranted. Easiest way would be to add an action with the require actor change and assert an error response.

🚀 Thank you for your contribution! 🚀


lang.int:fromHexString does not work  properly for hexdecimal strings of negative numbers

Description:
$title
Steps to reproduce:
import ballerina/io;

public function main() {
    string a1 = "ffffffffffffffff";
    int|error b1 = int:fromHexString(a1);
    io:println(b1); // prints error("{ballerina/lang.int}NumberParsingError",message="For input string: "ffffffffffffffff"")
                    // should be -1

    int|error b2 = int:fromHexString("ffffffff80000000");
    io:println(b2); // prints error("{ballerina/lang.int}NumberParsingError",message="For input string: "ffffffff80000000"")
                    // should be  -2147483648
}

Affected Versions:
slbeta3-rc1

In the above example, the decimal equivalent of ffffffffffffffff is<PHONE_NUMBER>3709551615 and ffffffff80000000 is<PHONE_NUMBER>1562067968. These both values are larger than the Ballerina int range .
And as the spec does not mention this behavior specifically, it is expected to get an error for the string value.

In the above example, the decimal equivalent of ffffffffffffffff is<PHONE_NUMBER>3709551615 and ffffffff80000000 is<PHONE_NUMBER>1562067968. These both values are larger than the Ballerina int range . And as the spec does not mention this behavior specifically, it is expected to get an error for the string value.

Thanks for the clarification.
Since the given behavior is expected, closing the issue.


Change Rating Professional in Rating tab

Change rating words to be more precise
| 2 | 🙀 Weak | Insufficient | -> | 2 | 🙀 Weak | Weak |
| 1 | 😾 Bogus | Insufficient | ->  | 1 | 😾 Bogus | Critical |

Aaah okay ! Then we just let's write it like that :)


Layer 2: impossible to use MetalLB on a subset of Kubernetes nodes.

Is this a bug report or a feature request?:
Feature request (possibly a regression since v0.6)
What happened:
If you run a service with externalTrafficPolicy: Cluster MetalLB Speaker will run announcer only on nodes with at least one ready endpoint. Which makes perfect sense for externalTrafficPolicy: Local, but not required for services with externalTrafficPolicy: Cluster.
Due to this it's impossible to use MetalLB with layer2 protocol on a subset of Kubernetes workers. It will work only if Speaker is running on all the nodes.
What you expected to happen:
It's possible to run MetalLB on a subset of k8s nodes and it runs Layer2 announcer on a leader node for services with externalTrafficPolicy: Cluster.
How to reproduce it (as minimally and precisely as possible):

On a 2+ node cluster run MetalLB speaker with nodeSelector set to a part of nodes.
Run the Nginx deployment and service from https://raw.githubusercontent.com/google/metallb/v0.7.2/manifests/tutorial-2.yaml with nodeSelector set to the other part of nodes. I.e. make sure Nginx pod is not running on the same node as MetalLB speaker.
Speaker is not responding to ARP who-has requests for external IP.

Anything else we need to know?:
The typical use case is border LB/ingress nodes. Which are supposed to terminate external traffic and load balance / ingress into the k8s cluster. Such nodes usually do not run any workloads.
Environment:

MetalLB version: v0.7.3
Kubernetes version: 1.11.2
BGP router type/version: layer2
OS (e.g. from /etc/os-release): CentOS Linux release 7.5
Kernel (e.g. uname -a): 3.10.0-862.3.2.el7.x86_64 #1 SMP Mon May 21 23:36:36 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

It looks like there's no easy fix for this with the current (0.7.x) design. In order to implement this feature properly Speaker needs to either have leader election (per service or global like in 0.6.x) or some other external control mechanism (from Controller?).
So basically, it's not possible to use MetalLB with layer2 protocol on a subset of Kubernetes workers. It will work only if Speaker is running on all the nodes.

bummer. I was hoping to do this too.

I actually don't need externalTrafficPolicy: Local. would it be simple to patch (maybe with a command line flag) to change it to work with Cluster only by running it on all nodes always. Just until a better fix can go in?

As you can tell, I haven't had time to dig into this more lately. Yes, the way 0.7 refactored things to enable the local traffic policy makes it hard to support running MetalLB on only part of the cluster. To be fair, I don't think that's a configuration I ever explicitly said I supported. All of MetalLB assumes that any node in the cluster could serve traffic, under the right conditions.
That said, partial node coverage is a reasonable configuration, and we should support it. For now the quickest thing I can suggest to make this work is to use MetalLB 0.6. Really the only change in 0.7 is this changed handling of layer2 mode, so if you just revert to 0.6 your configuration will work while I figure out how to support partial node coverage for 0.8.
Right now, it's looking like I will have to do some kind of active probing between speakers anyway to get nicer failover properties in Local mode. So, given that, partial node coverage would just show up as fewer nodes eligible to carry traffic, and the current deterministic selection logic would take it from there.

This won't be in 0.8, because I'm releasing 0.8 with "whatever is in master right now", and this isn't fixed in there. Sorry. Still want to fix it, but unknown timeline.

This feature is indeed useful as we too will only expose a portion of the nodes to internet.  Will try to use the 0.6.2 version till this is incorporated.

+1 for this feature


DNSmonster ASIM DNS Parser

This request adds an ASIM DNS Parser for the data created by the DNSmonster passive DNS monitoring framework which provides Sentinel output as a feature.
Change(s):

Added ASIM DNS Parser for DNSmonster

Reason for Change(s):

New functionality

Testing Completed:

Yes.  Passed tests with only Info: extra unnormalized columns

Checked that the validations are passing and have addressed any issues that are present:

Yes

As this is my first contribution, I'm unclear if I should execute the KqlFuncYaml2Arm.py in my repository and include the resulting ARM files and changes to FullDeploymentDns.json as part of this pull request or if that is done elsewhere after the merge.

Did you run the ASIM tests on the parser? Based on my review, there will be errors. A valid parser is one that has no errors, and document exceptions for warnings.

I did not.  I have updated the PR and will conduct those tests.

Can you provide sample data so I can do a better job of reviewing?

I added a set of sample data in aae59ee

Do you want it to part of the ASIM package, so deployed by default to every customer. I would love it, but it requires more work on your side. No worries, I will tell you what's needed.

I would be OK with that but really, I'm open to whatever works best.  I'm not sure how widespread the use of DnsMonster would be but since it implements dnstap in conjunction with Sentinel output, it solves a problem that we encountered by having DNSdist load balancing in front of our Microsoft DNS which resulted in the masking of the true client IP.  Instead of loading our Microsoft DNS into Sentinel, we are tapping DNSdist through DnsMonster and getting workable DNS data in Sentinel that we were missing before.

@fhp-kkuphal : We observed this PR is in Draft state since long time. Could you please move this PR from Draft to Open ?

@fhp-kkuphal : We observed this PR is in Draft state since long time. Could you please move this PR from Draft to Open ?

@oshezaf : Could you please review and provide your signoff.Thanks!!

Hi @fhp-kkuphal, happy to get back to your contribution. One question that comes to mind is what is the connector you are using to get the events to Sentinel. Would it be this one?

Hi @oshezaf,
Author has addressed your question. Can you please review and provide your sign off.
Thanks

Hi @oshezaf,
Sorry for multiple follow ups, can you please review and provide your approval.
Thanks

Hi @oshezaf,
Can you please review and provide your approval.
Thanks

Hi @fhp-kkuphal  and @v-mchatla : the parser needs additional work. To not overburden with comments I have cloned @fhp-kkuphal's repo and will PR to that, so the you can merge and push up. It will take me some time to finish though.

@fhp-kkuphal Wanted to check if you are still interested in making this contribution. If so, please follow the updated guidelines recently added here - https://github.com/Azure/Azure-Sentinel/blob/master/Parsers/Readme.md

We wanted to check on the status of PR #4544. PR is pending for more than 30 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation.

Hi @fhp-kkuphal, since we have not received a response in the last 7 days, we are closing your PR #4544 per our standard operating procedures. If you still need support for this issue, you can re-open the PR at any time.
If you do re-open, we simply request that you ensure the PR has response to the last request. Thank you for your cooperation


404 page on docs renders without any styling

NUC flashing instructions dashboard->application->add device link to getting started guide using this URL:
http://docs.resin.io/nuc/nodejs/getting-started/#adding-your-first-device , this gets redirected to
https://www.balena.io/docs/learn/getting-started/docs/intel-nuc/nodejs/#adding-your-first-device
The page loads as pure text, with no styles in both Firefox and Chromium under Linux

If you scroll all the way to the bottom you can see it says
Page not found
We think we don't have a page you're looking for.

@cyplo this appears to be fixed now, or at least I cannot reproduce it. Reopen if still an issue 👍

@chrisys Looks like any non-existent page still displays as completely unstyled, e.g. https://www.balena.io/docs/learn/page-not-here

@lurch thanks for pointing that out, have reopened and renamed to keep all the discussion here.

I just had a quick look at it, and the issue seems to be that the not-found page looks for the style and js at /dist/<styles.css|main.js>, but the other pages fetch it from /docs/dist/<styles.css|main.js>. I am assuming the reason is doxx adds the prefix to all static pages, but the not-found pages is not handled by doxx, therefore the error.


Formatting and localize dx-date-box

Hi,
I try to find out how to set the dx-date-box to a display format like "dd-MM-YYYY" and switch to locale de.
Some settings are working:
<dx-date-box [pickerType]="native" [value]="currentDate" [showClearButton]="true" [displayFormat]='{type: "longDate"}'></dx-date-box>
but if I use for instance  [displayFormat]='{skeleton: "dd-MM-YYYY"}' only a text box without anything appears.
Where can I find the new rules to customize the widgets?
Please give me an excample how to switch the locale to "de-DE" and set the behavior of the dx-date-box.
Thanks, Michael

Hello,
You need to use the globalize library.
Read here for details on how to localize DevExtreme widgets.
This example illustrates how to assign a format value to the displayFormat option:
<dx-date-box displayFormat="dd-MM-YYYY"></dx-date-box>

Hi,
implementing all of globalization in the proper order, localization works and also displayFormat="dd-MM-YYYY".
Thanks a lot, Michael

@LoganFitz : Could you provide an example what you did exactly to make this work?
The link provided by dxvladislavvolkov is for Knockout which Angular obviously isn't...


Get the most rated games of current year

This PR is intended to:

Get the highest rated games of the current year

Cool, thanks again @albertmir !


Readme: Fix lottie sample code

tvg::cast(picture) -> tvg::casttvg::Picture(picture)
This change should also be applied on our tutorial site.(https://www.thorvg.org/tutorial)

Thanks for notifying this. We can replace the usage with https://github.com/thorvg/thorvg/pull/1711


Make #1084 also covers 1.10.2

Apparently I forget the fact that RFTools supports 1.10.2 and 1.11.2 at the same time, so here is the very simple fix.

See #1097 for picture.

Any updates?


Please update mining stats

since 2 month we did not receive an update for https://github.com/Grinnode-live/grinnode.live/blob/master/documentation/mining-pools-list.md

updated doc

closing


Can not find fc-layer in SE blocks

Hi Hujie,
First of all thank you open your code,In your introduction,you use two fc-layer in SE block,but in prototxt,I find that you use two conv-layer with 1*1 kernel size. Is this a improvement? If yes ,is this improved performance to compared with the fc-layer.
Please help.
Thanks,
Totoro

This is a duplicate issue, please refer to #16.


[WIP] feat: add CatalogSource polling interval for new catalog sources

Description of the change:
Add a Poll Interval to the Spec of Catalog Sources in order to enable the catalog operator to automatically check for new catalog source versions. Part of https://github.com/openshift/enhancements/blob/master/enhancements/olm/operator-registry.md
Motivation for the change:
Have the catalog operator be responsible keeping catalog sources up to date via the container runtime.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

You probably need to run some code-gen to ensure this PR passing CI but we can deal with it after the code is completed.

@Denton24646 @dinhxuanvu Where can I see the manual test scripts / images you were using?

@Denton24646 @dinhxuanvu Where can I see the manual test scripts / images you were using?

@ecordell https://github.com/dinhxuanvu/catsrc-test

I'm going to close this in favor of #1109 since that is the same implementation just further ahead and QE is asking about this.
I think we may want to revisit the overall design of this feature and have it be goroutine based, with a timer.Tick set by the interval and a new channel hanging off the top level operator struct. But that can be discussed at a later point.


SolverProblemError when installing aiohttp

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: Lubuntu 18.04
Poetry version: 0.11.1
Link of a Gist with the contents of your pyproject.toml file: https://gist.github.com/Victor-Savu/e26e28cffd506bc399b5e0fae19dee62

Issue
Poetry incorrectly identifies idna-ssl to be a dependency of aiohttp when the required python is 3.7.
Here is the result of poetry lock -vvv.
Here is a Dockerfile which can be used together with the linked pyproject.toml file to reproduce the error:
FROM python:3.7.0b5-alpine
RUN apk add --no-cache curl \
 && curl -sSL https://raw.githubusercontent.com/sdispater/poetry/master/get-poetry.py | python3
ADD pyproject.toml .
RUN poetry lock -vvv

The setup.py of aiohttp only includes the dependency on idna-ssl for versions lower than 3.7:
if sys.version_info < (3, 7):
    install_requires.append('idna-ssl>=1.0')

install_requires is later on passed to setuptools.setup(...).
idna-ssl indeed only supports python versions <3.7 but it should not be included as a dependency to begin with.

aiohttp should use environment markers to tell that idna-ssl is only needed for Python > 3.7 (see PEP 508.
Poetry never executes setup.py to get dependency information for obvious security reasons and rather inspect either wheels or sdist tarballs. In the sdist, there is a file (requires.txt) which content is:
attrs>=17.3.0
chardet<4.0,>=2.0
multidict<5.0,>=4.0
async_timeout<4.0,>=3.0
yarl<2.0,>=1.0
idna-ssl>=1.0

There is no environment marker so Poetry tries to resolve idna-ssl which is only compatible with Python <3.7, hence the conflict you encounter.
There is nothing Poetry can do and should do for packages like this. Poetry tries to enforce good practices and defining dependencies only compatible with a subset of python versions programmatically is not one.
The problem lies upstream so you should report it there.

@sdispater Thanks for the quick response! I was wondering if that was going to be the case :) I'll report (and submit PR) upstream.
Thanks for being awesome! :1st_place_medal:

@sdispater Your explanation helped me find a workaround at least until the problem is fixed upstream. You got me wondering where poetry gets the requirements from if there is no sdist ready to go (such as when you install from git). Spoiler: installing from git works :)
Dockerfile:
FROM python:3.7.0b5-alpine
RUN apk add --no-cache curl git \
 && curl -sSL https://raw.githubusercontent.com/sdispater/poetry/master/get-poetry.py | python3
ADD pyproject.toml .
RUN poetry lock -vvv

pyproject.toml:
name = "poet-fail"
version = "0.1.0"
description = ""
authors = ["The Author<EMAIL_ADDRESS>
[tool.poetry.dependencies]
python = "3.7"
aiohttp = { git = "https://github.com/aio-libs/aiohttp.git", branch = "v3.3.2" }

In the sdist, there is a file (requires.txt) which content is:

I am sorry, I couldn't find the file you are referring to. Could you please share a link? It would really help me with reporting the bug upstream to aiohttp.
I checked the sdist here last link
I only found a few requirements files in the requirements directory, and those do specify the correct PEP 518 markers, as can be seen in the requirements/ci-wheel.txt:
attrs==18.1.0
async-generator==1.9
async-timeout==3.0.0
brotlipy==0.7.0
cchardet==2.1.1
chardet==3.0.4
coverage==4.5.1
cython==0.28.3
flake8==3.5.0
gunicorn==19.8.1
isort==4.3.4
pyflakes==2.0.0
multidict==4.3.1
pytest==3.6.0
pytest-cov==2.5.1
pytest-mock==1.10.0
pytest-xdist==1.22.2
towncrier==18.5.0
tox==3.0.0
twine==1.11.0
yarl==1.2.4

# Using PEP 508 env markers to control dependency on runtimes:
aiodns==1.1.1; platform_system!="Windows"  # required c-ares will not build on windows
codecov==2.0.15; platform_system!="Windows"  # We only use it in Travis CI
uvloop==0.9.1; platform_system!="Windows" and implementation_name=="cpython" and python_version<"3.7" # MagicStack/uvloop#14
idna-ssl==1.0.1; python_version<"3.7"

Could it be that this file is not referenced correctly in the sdist?

Sorry I was not clear, the requires.txt file is in the *.dist-info directory. And if Poetry can't find it it will inspect the wheel(s) if necessary.
It's a file generated by the python setup.py sdist command I think. Poetry avoids using the requirements.txt (or equivalent) since those do not necessarily represent the library's dependency.
Also, the reason it works from the git repository is that in this case poetry executes python setup.py egg-info to get the information. But even then it's not prefect in the case of aiohttp: if you generate the lock file from Python 3.7 but your project is also compatible with Python < 3.7 idna-ssl will never appear. That's why environment markers are so important.

I see now, thanks! I looked at the wheel and indeed the METADATA file reads:
Requires-Python: >=3.5.3
Requires-Dist: attrs (>=17.3.0)
Requires-Dist: chardet (<4.0,>=2.0)
Requires-Dist: multidict (<5.0,>=4.0)
Requires-Dist: async-timeout (<4.0,>=3.0)
Requires-Dist: yarl (<2.0,>=1.0)
Requires-Dist: idna-ssl (>=1.0)

So the markers don't make it to the wheel. I'm now ready to open the issue upstream. Thanks for the help!

Hmm.. there still seems to be an issue: The wheel is not yet released for python 3.7 So poetry should not really take it into consideration. (I was hinted by this issue https://github.com/aio-libs/aiohttp/issues/3111).
In this case, should poetry ignore the wheel and instead run setup.py itself?

In this particular case, Poetry uses the sdist since there is not wheel matching and this sees idna-ssl (>=1.0).
There are still some improvements to be made in the logic Poetry uses when it needs to fallback on downloading the distributions (wheels and sdists). However, it should work with properly packaged and PEP-compliant packages.

And it seems someone already opened a PR: https://github.com/aio-libs/aiohttp/pull/3114

This particular problem was solved upstream. Thanks for the support and thanks for your great work with poetry!

Had a problem with multidict:
 $ poetry update
Updating dependencies
Resolving dependencies... (0.2s)

Package operations: 7 installs, 0 updates, 0 removals

Writing lock file

  - Installing multidict (4.5.2)

[EnvCommandError]
Command ['pip', 'install', '--no-deps', 'multidict==4.5.2'] errored with
the following output:
Collecting multidict==4.5.2
  Could not find a version that satisfies the requirement multidict==4.5.
2 (from versions: 1.0.0a0, 1.0.1, 1.0.2, 1.0.3, 1.1.0b1, 1.1.0b2, 1.1.0b4
, 1.1.0b5, 1.1.0b6, 1.1.0b7, 1.1.0b8, 1.1.0, 1.2.0, 1.2.1, 1.2.2, 2.0.0,
2.0.1, 2.1.0, 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.1.5, 2.1.6, 3.0.0, 3.1.0, 3.1
.1, 3.1.3a0, 3.1.3, 3.1.4a1, 3.1.4a2, 3.1.4a3, 3.2.0a3, 3.2.0, 3.2.1a0, 3
.3.0, 3.3.1)
No matching distribution found for multidict==4.5.2

After rm -rf ~/.cache/pypoetry/ the error is gone.
toml file is simple:
[tool.poetry]
name = "xxx"
version = "0.1.0"
description = ""

[tool.poetry.dependencies]
python = "^3.6"
peewee = "^3.8"
aiohttp = "~3.5"

[tool.poetry.dev-dependencies]
pytest = "^3.0"

[build-system]
requires = ["poetry>=0.12"]
build-backend = "poetry.masonry.api"


Make it possible to configure Turing UI dynamically after static build is done

Problem:
Currently, Turing UI configuration consists of a mix of hard-coded values (defined at config.js) and a set of REACT_APP_* environment variables, that are supplied during the build time. This approach implies that we need to re-build the entire application just to change some configuration keys. It also forces us to release separate builds of Turing for different environments (i.e. dev/staging/production) because the configuration of different environments is naturally expected to be different.
Solution:
This PR introduces necessary changes to support overriding Turing UI settings at the run time (after the application is bundled as an optimized static release). This is achieved by including app.config.js into index.html and merging the runtime config (defined at app.config.js) with build-time config (defined at config.js).
So, if the user wants to override Sentry DSN in Turing's static build, she can update app.config.js file in dist directory with a desired value like this:
var config = {
    sentryConfig: {
        dsn: "my-new-dsn"
    }
};

Changes:
The main change in the codebase is that config objects (i.e. appConfig, sentryConfig etc) should no longer be imported directly into components where they are used, but instead, required configuration should be retrieved with a custom useConfig React hook. This hook merges user-provided configuration with the configuration defined during the build.

So, if the user wants to override Sentry DSN in Turing's static build

Is that a good idea? It seems to me that this method makes it easy to expose sensitive information.
For our internal use case, perhaps the specific example of Sentry DSN can be set at build time because it's consistent across the different environments. But REACT_APP_OAUTH_CLIENT_ID varies and I'm not sure app.config.js is the place for it. How would they be handled?
Recently, you had also brought up fetching UI configs from the API. How does that work alongside this approach?

So, if the user wants to override Sentry DSN in Turing's static build

Is that a good idea? It seems to me that this method makes it easy to expose sensitive information.

It's not any different from the current approach. All the data from REACT_APP_* variables is embedded in the build (you can look for it in bundle js files) and is publicly accessible. There is no safe way of securely storing secrets on the client applications that are running on a browser.

For our internal use case, perhaps the specific example of Sentry DSN can be set at build time because it's consistent across the different environments. But REACT_APP_OAUTH_CLIENT_ID varies and I'm not sure app.config.js is the place for it. How would they be handled?

Sentry DSN is not a secret (it's just a sentry project identifier). Neither oauthClientId is.

Recently, you had also brought up fetching UI configs from the API. How does that work alongside this approach?

Fetching configs from API can still be implemented (then useConfig hooks needs to be updated to also incorporate configuration retrieved from the API), but now I'm less eager to introduce /settings API, because the primary motivation for it was the ability to configure the application at run-time (which is already addressed by this PR).


Rename repo to PyPeri or similar

PyPeri is taken, https://pyperi.readthedocs.io/en/latest/readme.html
but I don't think there will be much confusion if we name the repo to PeriPy, and if there is then the worst case is that the user has installed the wrong pacakage.

Renamed repo to PeriPy just now. I have not changed anything in master, waiting for changes from the PR in dir names, readme, etc.

All names have been changed now in the PR


Close #155 Units should not be forced to SI

Description
This makes the attributes unitSI, timeUnitSI and gridUnitSI recommended instead of required. This is important for some codes which do not have a well-defined conversion to SI.
Affected Components

base: These attributes are now recommended instead of required.
EXT-...: Any extension that relies on these attributes should now be aware of the fact that it may not exist.

Writer Changes
Write can optionally omit this attribute (although it is very strongly recommended to add it, unless there is a good reason).
Reader Changes

openPMD-validator: Should make these attributes recommended.

The readers below should now always check whether these attributes exist. If not, the readers should return the queried quantities without any conversion (i.e. as it is written in the openPMD file). This assumes that the user will understand the unit system. For readers that show the unit symbol (e.g. kg, m, V/m), these standard symbols should not be shown. It is up to each reader to choose how to display the non-converted data.

openPMD-viewer: Should check before reading
yt: https://github.com/yt-project/yt/...
VisIt: https://github.com/openPMD/openPMD-visit-plugin
libopenPMD (upcoming): https://github.com/ComputationalRadiationPhysics/libopenPMD/...
XDMF (upcoming): https://github.com/openPMD/openPMD-tools/...

Data Converter
How does this affect already existing files with previous versions of the standard?
No, existing file will have the attributes and do not need to change.

@beck-llr @mccoys @DavidSagan Let me know me know if you have any comment.

I took the time and implemented the change in the validator in https://github.com/openPMD/openPMD-validator/pull/37


[Monitor OpenTelemetry] Update Metric Export Example

Packages impacted by this PR
@azure/monitor-opentelemetry
Updated the metric export example to a sample app that properly exports metrics to Azure Monitor.
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[x] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


⚠ Avatar API Endpoint has degraded performance

In 39f77e6, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 5326 ms

Resolved: Avatar API Endpoint performance has improved in 864ff04 after 21 minutes.


Does bincode guarantee a 1:1 relationship between encodings and values?

I require a encoding system that guarantees a 1:1 relationship between encodings and values.
That is to say, given a particular value, only one sequence of bytes will deserialize to that value, all other sequences either represent some other value or are invalid and will throw an error.
From looking at the bincode format, it seems like if it's correctly implemented, and if there aren't any custom decoders in play that violate this assumption, it should fit this, but I'm not 100% sure (for instance, it could not unreasonably canonicalize floating-point NaNs when decoding, which would violate this assumption — I assume it doesn't, but I don't know).
Is this a guarantee that bincode intends to provide? If so, could that be documented?

Bincode tries to achieve this but like you mentioned cases like NaN floating point deserialization could violate this assumption. Specifically we use a lot of from_le_bytes and from_be_bytes calls which could result in situations where the encoded value is different than the in-memory value.
That being said, I think in reality all values from the standard library will be a 1:1 mapping.
The bigger thing where this guarantee could break is when 3rd parties implement bincode for their types. At this point they could implement it any way they want, even using a randomizer to populate the data.

Hmm, it seems like this isn't guaranteed for built in types, given that HashMap is serialized in a arbitrary order :/
I think for people who want this property Borsh or BCS would be a better choice.


feat(app): add liner progress bar

Overview
Add LinerProgress component for Software update.
The animation part will be update in the hi-fi design.
the design
https://www.figma.com/file/AoTLAYuWawlaWItB1umOjr/OT-3-On-Device-Display?node-id=1776%3A30238&t=npwIVDBH9ZRseGMG-0
I named this component since the spinner that we use for loading page/screen might be updated because pages/screens that fetch data will use Skelton so we might have circular progress bar in the future.
https://user-images.githubusercontent.com/474225/207183825-990ba897-723d-4819-a13e-13b22f1b6659.mov
Close RCORE-453

Changelog

add liner progress component
add stories for liner progress component
add tests for liner progress component

Review requests

Run make -C components dev and open LinerProgress

Risk assessment
low

good job! it looks really cool! I was wondering if we could rename it to something like ProgressBar or ProgressMeter? What do you think?

Will change the name to ProgressBar


Uncaught ReferenceError: callbackName is not defined

经常会出这个问题,偶现性

I had the same issue, specifying the timeout worked me. The callback is being removed from window before it is called. Specifying the timeout option wasn't necessary in the prior version I was using. For a baseline, the response times range from 60ms to 150ms in my use case.

I had the same issue, providing .use(jsonp({timeout: 10000})) solved the issue. Thanks.


Create Sketchers UI Clone

Is your feature request related to a problem? Please describe.
Yes, since this is a Website clone
Describe the solution you'd like.
Here we will use HTML + CSS + JS + Bootstrap Library
Describe alternatives you've considered.
No alternative
Add any other context or screenshots about the feature request here.

/assign


Refactor/95 value expression visitor

Closes #95
Closes #100

The combining of BooleanExpressionRenderer and ValueExpressionRenderer has the advantage that for complex structures still only one visitor instance is needed. That saves memmory in runtime.

I removed the TreeNode structure since it was not used consequently and could easily be replaced by a few logic in the ValueExpressionRenderer


[Feature]: Vision and Mission in footer is not working

Is your feature request related to a problem? Please describe.
Vision and Mission in footer is not working
Describe the solution you'd like
I will create a Userfriendly UI for this page
Additional context
I will make it to work.
I want to work on this issue.
Please assign me this issue. under gssoc and level label

Please assign this to me. I can make it work

@Priyanshu2743 it's working now


Ionic serve issues

Short description of the problem:
When editing and saving files - sometimes ionic serve are using file from an older save of the file. Looks like there are som caching that are used.
Also when adding pages with ionic g pageName i have to restart serve to make that page work.
What behavior are you expecting?
No cached files used and new pages are recognized after adding with ionic g pageName
Which @ionic/app-scripts version are you using?
0.0.37
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Ionic Framework Version: 2.0.0-rc.1
Ionic CLI Version: 2.1.1
Ionic App Lib Version: 2.1.1
Ionic App Scripts Version: 0.0.37
OS: Windows 10
Node Version: v6.7.0

@sveinl,
This is resolved now. Tomorrow (Friday) we are releasing 0.0.37 which will fix the issue. You could also install @ionic/app-scripts@beta which will fix it too. It is the second beta release that fixed this.
Thanks,
Dan


Add code owners table in README

Closes: https://github.com/MeltanoLabs/Meta/issues/7
Include a table with all repos, their current owner if it was obvious, the contributors role, and the connector status.
Since some of these were either migrated from the meltano gitlab namespace, I left some without an owner. I intended to do this to show which repos are looking for new owners.
@aaronsteers youre tagged on 4 repos as maintainer and 1 repo as co-maintainer, if you want less ownership (as you also are contributing to overall maintenance of the whole project as well) let me know and I can either leave them unassigned for a community member or I can take them if it makes sense.
Also I just did this manually for now but I was looking into ways to define these tables as yaml or maybe I write a quick script that generates it so theres less formatting to do.

@pnadolny13 I'm okay with this as written, especially since the SQL-based ones are still in flux pending the merge of the latest.
The yaml one could be passed on though at this point, since I'm not actively planning any additional features.

The yaml one could be passed on though at this point, since I'm not actively planning any additional features.

@aaronsteers I set target-yaml as "looking for maintainer" for now but I might end up assigning to myself later since we use it in the squared repo.


Same secret multiple times in the same file

Hello,
I'm trying to scan using detect-secrets --all-files and I noticed that the tool does not detect multiple instances of the same secret in a single file. It will simply flag the first instance found. The comment located here makes me believe that this is by design. Is it possible to change this behavior or is the tool not designed for this?

Hi @gustavotabares, @domanchi  and I have had some discussions around this. However we've kind of punted dealing with this particular issue in the recent past. Line numbers are purposely not part of the secret hash because when lines move around we don't want to re-alert, but I don't see why we couldn't have a count of the secret and keep track of it that way. It is kind of a nice-to-have IMO though.
Thanks for making this a GitHub issue 👍 In summary it is possible to change this behavior, it's just that we haven't addressed it yet.

This is heavily similar to https://github.com/Yelp/detect-secrets/issues/53, so closing this issue in favor of the older one.


Update README.md with typo and grammar correction

changed you to "your"
remove erroneous "to"

Thanks!


Fix installing autocompletion documentation

The written solution did not work for me with the error -bash: __git_ps1: command not found. Since the auto-completion is already installed you can simply add the Kubernetes autocompletion to it.

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.7 Features: set Milestone to 1.7 and Base Branch to release-1.7
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: Please check the “Allow edits from maintainers” box below to allow
reviewers fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.

This change is

@bklein1 what about the written solution didn't work? If you're using homebrew to install kubectl, this should work automatically. Additionally, line 182 states:

Follow the "caveats" section of brew's output to add the appropriate bash completion path to your local .bashrc.

The reason for this is that the syntax can be different depending on local setup.

Sorry I forgot that part. This command is not working: source <(kubectl completion bash). It returns -bash: __git_ps1: command not found, which means line 186 is not working.
I will add another commit for your second remark.


Change upload directories, add dashboard icon for 'Documentation' col…

…lection

uploads_dir shouldn't start or end with a slash (https://docs.cloudcannon.com/editing/options/#file-uploads).


Add assigns for init containers

It was wrong and Jim was right. There is a first thing for everything <3


Enum description is ommited

Hello.
I have an issue with EnumType. I want to add a description to each enum value:
public enum Number
{
    One,
    Two,
    Three
}

public class NumberType : EnumType<Number>
{
    protected override void Configure(IEnumTypeDescriptor<Number> descriptor)
    {
        descriptor.Value(Number.One).Description("One");
        descriptor.Value(Number.Two).Description("Two");
        descriptor.Value(Number.Three).Description("Three");
    }
}

services
    .AddGraphQLServer()
    .AddEnumType<NumberType>();

But in this case, a HotChocolate.SchemaException is thrown. 1. The enum type NumberType has no values. (HotChocolate.Types.EnumType<GraphQL.NumberType>)
Is it a bug, or I'm missing something? The version of HotChocolate.AspNetCore is 11.0.9.
Thanks.

Hi, What is the outcome of this? I am having the same issue. Not able to add any EnumType
.AddEnumType();
It always gives me an error.
Thanks.

It’s wrongly used. But the error message is bad

I will add more Info to this issue tomorrow

So,
.AddEnumType<NumberType>();

is the equivalent of
.AddType<EnumType<NumberType>>();

Meaning now your type is doubly wrapped.
You can either do
.AddEnumType< Number>(d =>
{
        descriptor.Value(Number.One).Description("One");
        descriptor.Value(Number.Two).Description("Two");
        descriptor.Value(Number.Three).Description("Three");
});

or
.AddType<NumberType>();

I can recommend joining us on slack since you get a quicker answer to questions there. Issues have a longer cycle time.
You can join here: https://bit.ly/joinchilli

Hi,
Thanks for your answer.
Sorry, my example was incorrect. What I meant, and is not working is like that:
I created an enumType similar to that:
public class NumberType : EnumType {}
And when I add that type like that:
.AddEnumType();
It raises and error on BananaCakePop. It doesn't even reach the app. It gives me a 500 Server Error. If I remove the AddEnumType it works properly again, but then I don't have the enum mapping.

@luizhennemann
.AddEnumType< Number >();
is the equivalent of
.AddType<EnumType<Number>>();
if you do
.AddEnumType<NumberType>();

You end up doing
.AddType<EnumType<NumberType>>();

Which will be
.AddType<EnumType< EnumType <Number>>>();

If you just do
.AddType<NumberType>()

you should be fine


add more log info about GPU usage

What would you like to be added:
Adding more log info about GPU usage could help debugging, and make users be aware of GPU usage.
Detail: add more log info when users does not enable useActiveGPU but GPUs are occupied by others.
Why is this needed: user friendly
Components that may involve changes: training service

@liuzhe-lz for awareness


video:miis_core_id details

Could you add some additional information for the video:miis_core_id? What is MIIS? What is MISB ST 0601.17.94? Maybe some links would help. As a non-video expert, I don't understand what this is all about.

Sure, I added a link to the MISB ST 0601.17 metadata spec landing page: https://nsgreg.nga.mil/doc/view?i=5093
I wouldn't say I'm a video expert either, but this ID seems to be well-known in the wild: https://github.com/stac-extensions/video/issues/17


How to rebuild the repository code index

Gitea Version
1.16.1
Git Version
<IP_ADDRESS>
Operating System
Windows Server 2019 Datacenter
How are you running Gitea?
Run gitea-1.16.1-windows-4.0-amd64.exe using Windows Service
Database
No response
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Description
Now, when I search for the code, most of them cannot be searched. I need to rebuild the es index or what should I do.
Screenshots

You can just delete the index files and restart Gitea

There is a button on repository setting to reindex the code index.


Fix travis build.

Let's give this another try.

Your Dockerfile is wrong; you need to use Docker commands. Please fix it.


fix(auth): handle undefined global role

:tada: This PR is included in version 2.6.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Large flow perftest

Hello, I am so impressed about your work, Justitia.
I think this is a interesting and important topic to think of when using RDMA.
Now I am trying to run Justitia in my servers.
I believe I did follow your instructions correctly but it does not work properly when running perftest for bandwidth test. (worked well in latency test)
I am using Connectx-3 Pro NICs and MLNX_OFED_LINUX-4.2-<IP_ADDRESS> driver.
Here is what I did. (*bandwidth test alone)
After setup_X3 script completed,
Server terminal 1:
./pacer 0 <IP_ADDRESS> 1 3
Client terminal 1:
./pacer 1 <IP_ADDRESS> 1 3
After I launched the Pacer, I ran 'ib_write_bw'.
Server terminal 2:
export LD_LIBRARY_PATH=/usr/lib64:$LD_LIBRARY_PATH
./ib_write_bw -F -s 10485760 -n 500000 -l 1 -t 1 -p 8888 -S 3
Client terminal 2:
export LD_LIBRARY_PATH=/usr/lib64:$LD_LIBRARY_PATH
./ib_write_bw -F -s 10485760 -n 500000 -l 1 -t 1 -p 8888 -S 3 <IP_ADDRESS>
The result from Server terminal 2 is:
************************************
* Waiting for client to connect... *
************************************
DEBUG mlx4_create_qp: split_qp->qpn = 000369
DEBUG mlx4_create_qp: split_qp[0]->qpn = 00036a
DEBUG mlx4_create_qp: orig_qp->qpn = 00036b
Contacting pacer...
SUN_PATH = /home/mingman/y306_rdma_socket
SOCK_PATH = /users/yiwenzhg/rdma_socket
Sending join message...
I'm a receiver.
My PID is 51344
Received slot number: 3
@@@At slot 3.
<<<<MODIFY SPLIT QP to INIT>>>>
---------------------------------------------------------------------------------------
                    RDMA_Write BW Test
 Dual-port       : OFF		Device         : mlx4_0
 Number of qps   : 1		Transport type : IB
 Connection type : RC		Using SRQ      : OFF
 CQ Moderation   : 1
 Mtu             : 1024[B]
 Link type       : Ethernet
 GID index       : 3
 Max inline data : 0[B]
 rdma_cm QPs	 : OFF
 Data ex. method : Ethernet
---------------------------------------------------------------------------------------
 local address: LID 0000 QPN 0x036b PSN 0xb854c6 RKey 0x5001040a VAddr 0x007f32776c6000
 GID: 00:00:00:00:00:00:00:00:00:00:255:255:10:00:36:02
 remote address: LID 0000 QPN 0x0269 PSN 0x9f808d RKey 0x60010401 VAddr 0x007f269aaf2000
 GID: 00:00:00:00:00:00:00:00:00:00:255:255:10:00:37:02
<<<<MODIFY SPLIT QP to RTR>>>>
<<<<MODIFY SPLIT QP to RTS>>>>
<<<<Request SPLIT CQ evnt notification
---------------------------------------------------------------------------------------
 #bytes     #iterations    BW peak[MB/sec]    BW average[MB/sec]   MsgRate[Mpps]
ethernet_read_keys: Couldn't read remote address
 Unable to read to socket/rdam_cm
 Failed to exchange data between server and clients
Contacting pacer...
SUN_PATH = /home/mingman/y306_rdma_socket
SOCK_PATH = /users/yiwenzhg/rdma_socket
DEBUG decrement SMALL counter by 0
libmlx4 exit

The result from Client terminal 2 is:
DEBUG mlx4_create_qp: split_qp->qpn = 000267
DEBUG mlx4_create_qp: split_qp[0]->qpn = 000268
DEBUG mlx4_create_qp: orig_qp->qpn = 000269
Contacting pacer...
SUN_PATH = /home/mingman/y307_rdma_socket
SOCK_PATH = /users/yiwenzhg/rdma_socket
Sending join message...
I'm a sender; vaddr_idx = 0
My PID is 24805
Received slot number: 3
@@@At slot 3.
<<<<MODIFY SPLIT QP to INIT>>>>
---------------------------------------------------------------------------------------
                    RDMA_Write BW Test
 Dual-port       : OFF		Device         : mlx4_0
 Number of qps   : 1		Transport type : IB
 Connection type : RC		Using SRQ      : OFF
 TX depth        : 1
 CQ Moderation   : 1
 Mtu             : 1024[B]
 Link type       : Ethernet
 GID index       : 3
 Max inline data : 0[B]
 rdma_cm QPs	 : OFF
 Data ex. method : Ethernet
---------------------------------------------------------------------------------------
 local address: LID 0000 QPN 0x0269 PSN 0x9f808d RKey 0x60010401 VAddr 0x007f269aaf2000
 GID: 00:00:00:00:00:00:00:00:00:00:255:255:10:00:37:02
 remote address: LID 0000 QPN 0x036b PSN 0xb854c6 RKey 0x5001040a VAddr 0x007f32776c6000
 GID: 00:00:00:00:00:00:00:00:00:00:255:255:10:00:36:02
<<<<MODIFY SPLIT QP to RTR>>>>
<<<<MODIFY SPLIT QP to RTS>>>>
<<<<Request SPLIT CQ evnt notification
---------------------------------------------------------------------------------------
 #bytes     #iterations    BW peak[MB/sec]    BW average[MB/sec]   MsgRate[Mpps]
Contacting pacer...
SUN_PATH = /home/mingman/y307_rdma_socket
SOCK_PATH = /users/yiwenzhg/rdma_socket
DEBUG POST SEND: INDEED increment BIG flow counter
FIND cqe syndrome: cqe->syndrome = 19
FIND ERROR STATUS: wc->status = 10
mlx4: local QP operation err (QPN 000269, WQE index 0, vendor syndrome 6f, opcode = 5e)
FIND cqe syndrome: cqe->syndrome = 2
FIND ERROR STATUS: wc->status = 2
 Completion with error at client
 Failed status 2: wr_id 0 syndrom 0x6f
scnt=1, ccnt=0
 Failed to complete run_iter_bw function successfully
DEBUG decrement BIG counter by 1
Contacting pacer...
SUN_PATH = /home/mingman/y307_rdma_socket
SOCK_PATH = /users/yiwenzhg/rdma_socket
libmlx4 exit

When I tried latency test, it showed a reasonable result.
On the contrary, it is bandwidth test that does not work properly.
I guess the problem might result from FIND ERROR STATUS: wc->status = 10 in the Client terminal 2. Still, I do not know what actually makes trouble in this situation. Did I miss any configs or settings? Any suggestions?

Thanks Mingman,
I verified that the perftest command you used ran just fine on my end. Could you double-check if you are using the most recent version of the code. A recent commit (ccba5a2) should have fixed that issue.
Let me know if that is not the cause of the issue.

You were right.
My codes were not the recent one. I did not imagine that there could be newer commits.
As you pointed out, pacer.h and pacer.c remained in an old state.
Now everything is working as expected.
Thank you for your help!


5.7 Release does not configure correct KMS permissions on Lambda Roles

I upgraded our deployment of the ServerLess Repo CTI Adapter to 5.7.3 but Lambdas are now failing with errors.
On investigation it seems code was added to salesforce.py to write an OAuth token to SecretsManager:
https://github.com/amazon-connect/amazon-connect-salesforce-lambda/blob/3a926720ee1ae550a2829f8976cdfea2bd985f69/sam-app/lambda_functions/salesforce.py#L88
However, the CloudFormation policy document for the Lambda functions has not been updated to include permissions to Encrypt with the SecretsManager KMS key as provided by the parameter:
SalesforceCredentialsKMSKeyARN
In the template the resource KMSManagedPolicy only attaches permissions for kms:Decrypt:
https://github.com/amazon-connect/amazon-connect-salesforce-lambda/blob/3a926720ee1ae550a2829f8976cdfea2bd985f69/sam-app/lambda_functions/template.yaml#L226
This causes the following error when the Lambda runs:

[DEBUG]	2020-11-16T16:44:43.856Z	3e02e379-48a5-4f10-8405-04c11d08f9ba	https://secretsmanager.eu-west-2.amazonaws.com:443 "POST / HTTP/1.1" 400 75

[DEBUG]	2020-11-16T16:44:43.857Z	3e02e379-48a5-4f10-8405-04c11d08f9ba	Response headers: {'Date': 'Mon, 16 Nov 2020 16:44:43 GMT', 'Content-Type': 'application/x-amz-json-1.1', 'Content-Length': '75', 'Connection': 'keep-alive', 'x-amzn-RequestId': 'ce54db28-100a-4fef-909f-3bc87b30fd6d'}

[DEBUG]	2020-11-16T16:44:43.857Z	3e02e379-48a5-4f10-8405-04c11d08f9ba	Response body:
b'
{
    "__type": "AccessDeniedException",
    "Message": "Access to KMS is not allowed"
}
'

Steps to reproduce
Launch AWS::Serverless::Application resource via Cloudformation passing SemanticVersion: 5.7.3. Fill in all required parameters as desired, they won't effect the behaviour.
cti:
    Type: AWS::Serverless::Application
    Properties:
      Location:
        ApplicationId: arn:aws:serverlessrepo:us-west-2:821825267871:applications/AmazonConnectSalesforceLambda
        SemanticVersion: 5.7.3

The issue is with 5.7.

Hi guys,
apologies for this issue. We've already merged the fix into our repo, but you can fix this yourself by adding the kms:generatedatakey action to the KMSManagedPolicy managed policy.


The data is wrong after decompression and decompression by zfp_field_1d, but it is correct when I use zfp_field_2d

Recently, I am working on a lossy compression tool for seismic data. My data is binary seismic data. It is about 1.6GB. When I use zfp_field_2d to compress and decompress the data, it is correct (because I put it into professional seismic processing software and compared it with the original data image), but the data obtained after decompressing and decompressing it with zfp_field_1d is wrong.
My data is one-dimensional. Initially, I converted it into a matrix to use zfp_field_2d, but I found that the time spent on matrix conversion alone is very long (but the compression time is very short). So I considered using zfp_field_1d, but his results were wrong.
I would appreciate it if you could assist me in reviewing my code to see if there are any issues. The file 'zfp_test_2d.txt' contains the correct output.
zfp_test_1d.txt
zfp_test_2d.txt

I continued to test the decompressed data and found that the data values were all 0。 I don't know how to solve it, I hope someone can give me some help

You've included a lot of code, e.g., with support for multiple compression modes.  It would help us isolate the issue if you could reduce the 1D example to only the essential code being executed.
After taking a quick look, it appears that you're compressing Samples * All_Trace ~ 420 million floats at a rate of only 2 bits/value.  In 1D, this means each zfp block of 41 = 4 values gets assigned 2 * 4 = 8 bits/block.  This is not sufficient to even represent the empty-block bit and common exponent in the block, and no mantissa bits will be encoded, which is presumably why you're getting all-zeros.  You would have to increase the rate to at least 4 bits/value, though ideally more.  That said, zfp was not designed for 1D arrays, so quality will likely suffer.
In 2D, a rate of 2 bits/value will translate to 2 * 42 = 32 bits/block, which at least is sufficient to represent the leading mantissa bits.
If your data truly is 1D, then you might still benefit from compressing it as 2D.  From the 2D example, I assume you're compressing many small data sets, each of which is 2001 samples.  2001 is not divisible by zfp's 2D blocks of 4 * 4 = 16 values, but you could compress the first 2000 values as a nx = 4, ny = 500 2D data set and then store the last value uncompressed.  Or you could pad the last block with copies of that last value and then compress the resulting 2016 values as nx = 4, ny = 504.

I believe this is not a problem with zfp per se and will go ahead and close the issue.


Update HTML guidance

After a short review we noted there are various aspects that are out of date and could go with an update especially around
aria-labelledby= guidance and
 use.
https://gds-way.cloudapps.digital/manuals/programming-languages/html.html

@chris-gds Thank for looking. Is there any more detail you can provide on what aspects need updating? Or even better, a PR...?

👋🏼  @timblair - Sure, more details below - initial thoughts centre around section use.  This most likely could be tested further across assistive tech.

Introduction

People have been catching fish for food since before recorded history...

Equipment

The first thing you'll need is a fishing rod or pole that you find comfortable
    and is strong enough for the kind of fish you're expecting to try to land...

I don't think this pattern is used commonly within GOV.UK nor is it present in the Design System
After testing it might be not needed
Or the reverse might be true and many parent containers that contain headings should be updated accordingly as well as the relevant publishing apps.

(I've also requested time to see if I can review / update in lot more detail).

@chris-gds Did you manage to carve out any more time to to a full review of the HTML guidance?

I noticed the ARIA in HTML spec' states that the technique of using aria-labelledby to assign an accessible name to
s gives them the role of region. Without it, they have no role:
https://w3c.github.io/html-aria/#el-section
Should be a consideration if this is added in this guidance.

Myself and @chris-gds had a very long discussion about this outside of here and decided that the current guidance on use of
 needs to change and that I'll create a PR to that end. When I do, I'll update this comment and close this issue.

I haven't found time to put together a PR for the proposed changes and, after discussing it with others, I intend to consult with the frontend community first. I'll update the status of that here when it happens.


Total number of files limit of FAT filesystem

In FAT filesystem, which is the default filesystem for external SD Card, we can only store around 15 ~ 20k files in a directory. In most cases, it shouldn't be a problem. But when I need to cache a lot of small images with DiskLruCache, it brings this issue to me.
Hence, I would like to know if it's possible to create the sub-directory within DiskLruCache's cahce directory so we can solve this issue for good.
FAT filesystem limit information: http://stackoverflow.com/questions/2651907/is-there-a-limit-for-the-number-of-files-in-a-directory-on-an-sd-card

A possible solution as part of #71 we could relax key restrictions and allow specifying hierarchical keys which would allow you to partition your keys based on some prefix (even if it's just the first two characters of an MD5 for example).


parallel_test fails

I have tried with PostgreSQL 11 as well as MariaDB 11, both seem to never pass this test case. Example logs attached.
I have tried to debug it with psql and to me it looks like it's more an issue with the test infrastructure than with the library itself. Apparently data created in one test case leaks into the next and / or the worker threads do not "see" records created in the main thread.
At this point I'd basically like to know if I am doing something wrong or if this is something that needs to be fixed.
postgres_11.log
mariadb_11.log

setting the DatabaseCleaner strategy to :truncation instead of :transaction in test/test_helper.rb seems to fix this. I added this to my PR #426

thank you, i merged the tests.


Install fails after successful cloning

I noticed on Friday that installation of NVM using install.sh was no longer succeeding in my VM image builds. I run NVM installation in a larger install script that uses set -e, and I started noticing that the install script would abort and break the build after the "Cloning" step. Investigating further, it looked like the script was exiting with status 1.
I believe I've traced it to this change – specifically, these lines:
command git clone "$(nvm_source)" "$INSTALL_DIR" \
       || echo >&2 "Failed to clone nvm repo. Please report this!" && exit 1

Bash behavior in this case, IIUC, is to run || before the &&, so that exit 1 is executed whether or not the cloning succeeds.
I am verifying a fix that I intend to submit in a follow-up PR. Thanks!

@ods94065 thanks for the report! I'm curious why you're using the master install script, instead of the latest tagged version? master is unstable, and only tagged releases are guaranteed to work.
I think your suspicion is correct, and a PR would be great!

I'm not sure... there's a lot of history in my install script. :-) I'll make that adjustment too. Thanks!


Add back shards as a separate package

I'm not sure that's a good solution. It seems shady being able to install shards in two ways and have it just be overriden. I think that combination of both approaches is worse than either single one.

Do you have another solution?


feat: make JSON serialization of referenced pydantic models possible

Change Summary
Allow to customize serialization of fields with BaseModel type
from typing import List, Optional

from pydantic import BaseModel

class Address(BaseModel):
    city: str
    country: str

class User(BaseModel):
    name: str
    address: Address
    friends: Optional[List['User']] = None

    class Config:
        json_encoders = {
            Address: lambda a: f'{a.city} ({a.country})',
            'User': lambda u: f'{u.name} in {u.address.city} '
                              f'({u.address.country[:2].upper()})',
        }

User.update_forward_refs()

wolfgang = User(
    name='Wolfgang',
    address=Address(city='Berlin', country='Deutschland'),
    friends=[
        User(name='Pierre', address=Address(city='Paris', country='France')),
        User(name='John', address=Address(city='London', country='UK')),
    ],
)
print(wolfgang.json())
# {"name": "Wolfgang", "address": "Berlin (Deutschland)", "friends": ["Pierre in Paris (FR)", "John in London (UK)"]}

Related issue number
https://github.com/samuelcolvin/pydantic/discussions/2622

Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

please update.

Thanks for the remarks. Please review the updated version

When is it merging?

thank you.


Selecting a schema on createClient does not work

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I am moving all contents from a Supabase server to another. I have exported the roles, schema and data, and inserted it again in the new server.
After that, I had to move the storage blobs from the bucket of the previous server to the new one, as indicated on this:
https://supabase.com/docs/guides/platform/migrating-and-upgrading-projects#migrate-storage-objects
But when I do, it fails on the step of obtaining the objects list from the previous DB. Inspecting the error returned I can see it fails because of this:
{
  code: '42P01',
  details: null,
  hint: null,
  message: 'relation "public.objects" does not exist'
}

and taking into account the script is selecting the schema storage for that oldSupabaseRestClient variable initialization, it does not make any sense.
I tried changing the name of the schema in the initialization in case it is just protecting storage but the error remains the same: always trying to access public.
To Reproduce
Have two instances and try to run the script on that docs page.
Expected behavior
Be able to get the list of files and continue with the migration
Screenshots

System information

OS: macOS
Browser (if applies): Does not apply
Version of supabase-js: 2.39.3
Version of Node.js: v18.16.1

Additional context

I have the same issue - please assist!

Figured out the problem, when accessing the schema, it needs to be:
const oldSupabaseRestClient = createClient(OLD_PROJECT_URL, OLD_PROJECT_SERVICE_KEY, {
    schema: 'storage',
})

@supabase team, can you update the docs on this?

same problem

same problem with TS project


Multiple events in addEventListener do not work

When I resized the window or switched the orientation the refresh did not occur.
Thats because you tried to bind two events in one window.addEventListener call which is not possible, so I splitted them into single calls.

Version 1.0.2 contains the fix. Thank you very much!

Could use
'''
['resize', 'orientationchange'].forEach(...
'''


Can't install

Can't install ghcr.io/yeonv/amd64-addon-ledfx:2.0.62: 500 Server Error for http+docker://localhost/v1.42/images/create?tag=2.0.62&fromImage=ghcr.io%2Fyeonv%2Famd64-addon-ledfx&platform=linux%2Famd64: Internal Server Error ("Get "https://ghcr.io/v2/": dial tcp: lookup ghcr.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:42914-><IP_ADDRESS>:53: read: connection refused")

Can't install ghcr.io/yeonv/amd64-addon-ledfx:v2.0.67: 500 Server Error for http+docker://localhost/v1.42/images/create?tag=v2.0.67&fromImage=ghcr.io%2Fyeonv%2Famd64-addon-ledfx&platform=linux%2Famd64: Internal Server Error ("manifest unknown")

Fixed need to work now

Can't install ghcr.io/yeonv/amd64-addon-ledfx:v2.0.67: 500 Server Error for http+docker://localhost/v1.42/images/create?tag=v2.0.67&fromImage=ghcr.io%2Fyeonv%2Famd64-addon-ledfx&platform=linux%2Famd64: Internal Server Error ("manifest unknown")

Was there a fix for this?

Can't install ghcr.io/yeonv/amd64-addon-ledfx:v2.0.67: 500 Server Error for http+docker://localhost/v1.42/images/create?tag=v2.0.67&fromImage=ghcr.io%2Fyeonv%2Famd64-addon-ledfx&platform=linux%2Famd64: Internal Server Error ("manifest unknown")

Was there a fix for this?

I do not recommend using this add-on.  There are a lot of problems with it and I never got it to work properly

Ah fair enough. You don't know of any alternatives currently do you?On 18 Jul 2023 08:42, Sp595 @.***> wrote:
Can't install ghcr.io/yeonv/amd64-addon-ledfx:v2.0.67: 500 Server Error for http+docker://localhost/v1.42/images/create?tag=v2.0.67&fromImage=ghcr.io%2Fyeonv%2Famd64-addon-ledfx&platform=linux%2Famd64: Internal Server Error ("manifest unknown")
Was there a fix for this?
I do not recommend using this add-on.  There are a lot of problems with it and I never got it to work properly
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Unfortunately there are no alternatives
I can't even run normal ledfx on my linux distro at all, the install
script is only suitable for ubuntu. Ledfx itself is mostly developed for
windows, although the go version is being developed.
ср, 19 июл. 2023 г., 17:08 Knottymister @.***>:

Ah fair enough. You don't know of any alternatives currently do you?On 18
Jul 2023 08:42, Sp595 @.***> wrote:
Can't install ghcr.io/yeonv/amd64-addon-ledfx:v2.0.67: 500 Server Error
for http+docker://localhost/v1.42/images/create?tag=v2.0.67&fromImage=
ghcr.io%2Fyeonv%2Famd64-addon-ledfx&platform=linux%2Famd64: Internal
Server Error ("manifest unknown")
Was there a fix for this?
I do not recommend using this add-on. There are a lot of problems with it
and I never got it to work properly
—Reply to this email directly, view it on GitHub, or unsubscribe.You are
receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/YeonV/home-assistant-addons/issues/25#issuecomment-1641967358,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY3O7HMMWLYYZ5HKIZHPXFTXQ7E4XANCNFSM6AAAAAAV3727GA
.
You are receiving this because you authored the thread.Message ID:
@.***>

No worries, thankyou for the reply. I'll stick to hyperion for my pi then!On 19 Jul 2023 13:31, Sp595 @.***> wrote:
Unfortunately there are no alternatives
I can't even run normal ledfx on my linux distro at all, the install
script is only suitable for ubuntu. Ledfx itself is mostly developed for
windows, although the go version is being developed.
ср, 19 июл. 2023 г., 17:08 Knottymister @.***>:

Ah fair enough. You don't know of any alternatives currently do you?On 18

Jul 2023 08:42, Sp595 @.***> wrote:

Can't install ghcr.io/yeonv/amd64-addon-ledfx:v2.0.67: 500 Server Error

for http+docker://localhost/v1.42/images/create?tag=v2.0.67&fromImage=

ghcr.io%2Fyeonv%2Famd64-addon-ledfx&platform=linux%2Famd64: Internal

Server Error ("manifest unknown")

Was there a fix for this?

I do not recommend using this add-on. There are a lot of problems with it

and I never got it to work properly

—Reply to this email directly, view it on GitHub, or unsubscribe.You are

receiving this because you commented.Message ID: @.***>

—

Reply to this email directly, view it on GitHub

https://github.com/YeonV/home-assistant-addons/issues/25#issuecomment-1641967358,

or unsubscribe

https://github.com/notifications/unsubscribe-auth/AY3O7HMMWLYYZ5HKIZHPXFTXQ7E4XANCNFSM6AAAAAAV3727GA

.

You are receiving this because you authored the thread.Message ID:

@.***>

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>


i cannot visit the project

iam a beginner for this project
i download the release1.2.0
Describe the question
i run the masterServer,workerServer,apiServerApplication successful,and install the dolphinscheduler-ui is ok, nginx is ok,but i cannot visit the views.  i thinks the apiServerApplicaiton's port is 8080.but the properties is 12345? so i visit the front is 404
Which version of DolphinScheduler:
-[release1.2.0]

i see the apiServer's port is 8080 ,i meants the proterties's server.port=12345 doestn't work?

like this ,try try

like this ,try try

oh,thinks,i try  try first

like this ,try try

thinks dalao,i can works

have a good time

like this ,try try


Navigation bar disappeared

I recently upgraded my modules on an app I am building, and I can't display a navbar on one container where I was able to before. Here's the routes:
<Router name="root">
  <Route
    name="launch"
    component={Launch}
    initial={true}
    hideNavBar={true}
    sceneConfig={Navigator.SceneConfigs.FloatFromLeft}/>
  <Route
    name="activityFeed"
    component={ActivityFeed}
    title="Activity Feed"
    hideNavBar={false}
    sceneConfig={Navigator.SceneConfigs.FloatFromLeft}/>
</Router>

The activity feed scene is not displaying the navbar at all. Before the upgrade, it did. (I upgrade today from v2.2.4 to current.)
Any ideas on where to start looking?

I saw something similar, when I was trying to use the example from @nyura123's PR  for react-native-tableview. hideNavBar={true} was not working in the editing routes "Example with editing mode" and "Custom Editing Example". It should hide the main navbar and allow the route-specific navbar to be seen but did not. This seems to be fixed in 2.3.5.
But now there is another regression: With 2.3.5 and clicking on any of the other (non-editing) routes, the Back button is missing from the navbar. This works fine in 2.1.8, and in 2.2.4, the back button is there but does not work.

@chetstone You are right, latest changes broke 'back' button functionality, didn't found the reason yet, reverted to old solution...

Well, good to know I am maybe not the only one. :)
Another thing I noticed in trying to superficially debug, the repo's readme.md example shows a nested router (<Router footer={TabBar} showNavigationBar={false}...) with a showNavigationBar attribute? How does it differ from the hideNavBar one? The configuration table above it does not document it.

It should be the same as hideNavBar, i believe

But, probably inversed? Meaning (showNavigationBar = true) === (hideNavBar = false)?
If so, that did not seem to alter the fact the the navbar is absent from my ActivityFeed screen.

Check latest version (2.3.6), if it doesn't work, please check demo, it hides/show navbar within tabbar.

Yes, it works now.

Is the back button fixed yet? I still see the back button not being rendered properly. Thanks

same!

just tested, the back button issue first appeared in 2.3.4

In the same case, NavigationBar disappeared in 2.2.7, and after upgrade RNRF to 2.3.6, NavigationBar appeared, but the back button disappeared again.
var React = require('react-native');
var RNRF = require('react-native-router-flux');
var {Text, View, TouchableOpacity} = React;
var {Router, Route, Actions} = RNRF;
var Home = React.createClass({
    render(){
        return (
            <View style={{flex:1, alignItems: 'center', justifyContent: 'center'}}>
                <TouchableOpacity onPress={()=>{Actions.search();}}><Text>Hello world</Text></TouchableOpacity>
            </View>
        );
    }
});
var Search = React.createClass({
    render(){
        return (
            <View style={{flex:1, alignItems: 'center', justifyContent: 'center'}}>
                <Text>Search world</Text>
            </View>
        );
    }
});
var Launcher = React.createClass({
    render(){
        return (
            <Router name="root">
                <Route name="home" component={Home} initial={true} hideNavBar={true}/>
                <Route name="search" component={Search} hideNavBar={false}/>
            </Router>
        );
    }
});
module.exports = Launcher;

Above is an example, I hope it can help some.

Maybe during routes switch, the NavBar doesn't change at all.
Check the code below, when route switch to search, **NavBar title is still 'Home Page', not 'Search **Page'.
var React = require('react-native');
var RNRF = require('react-native-router-flux');
var {Text, View, TouchableOpacity} = React;
var {Router, Route, Schema, Actions} = RNRF;
var Home = React.createClass({
    render(){
        return (
            <View style={{flex:1, alignItems: 'center', justifyContent: 'center'}}>
                <TouchableOpacity onPress={()=>{Actions.search();}}><Text>Hello world</Text></TouchableOpacity>
            </View>
        );
    }
});
var Search = React.createClass({
    render(){
        return (
            <View style={{flex:1, alignItems: 'center', justifyContent: 'center'}}>
                <Text>Search world</Text>
            </View>
        );
    }
});
var Launcher = React.createClass({
    render(){
        return (
            <Router name="root">
                <Schema name="default" sceneConfig={React.Navigator.SceneConfigs.FloatFromRight}/>
                <Route name="home" title="Home Page" component={Home} initial={true}/>
                <Route name="search" title="Search Page" component={Search}/>
            </Router>
        );
    }
});
module.exports = Launcher;

:+1: my navbar is gone with the latest update also

Please check 2.3.7

@aksonov This appears to be fixed. THank You!

Fixed, tks~

Added customization of nav bar again (2.3.7 is like 'reverted' version), please check 2.3.8, hope i didn't break it again ;)

Updated. Works for me.

2.3.8 works fine.


Questions about the proposed method

If I understand correctly, ELITE needs additionaly mask input compared to the related work, right? Although it has quicker editting, it seemed that ELITE should be compared to the baselines that also using mask. It is actually unfair to compare it with Textual Inversion, Dreambooth...
And then here comes another question: Now we have the mask, we do not directly use mask into the "global network training stage"?

We have tried to use masks directly in the "global network training stage", and it does improve the editability, but less than the current way. We will explore better ways to leverage the mask to learn a well-editable word embedding.


Update api-management-howto-create-or-invite-developers.md

Added line "Invitation link will be active for 2 days"

@Aditi-Nagarkar : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@dlepow
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

#sign-off


Update create-complete-managed-instance-indirectly-connected.md

@bweissman : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@bweissman - Because SQL Managed Instance is a product name, it shouldn't be preceded with "An". The line is correct as written, so I'll close this PR. Thanks!


[20.09] isync: 1.3.5 -> 1.3.6

Motivation for this change
Fixes: CVE-2021-3578
https://sourceforge.net/projects/isync/files/isync/1.3.6/
Master and release-21.05 are on 1.4.x, which we will not backport.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[x] Fits CONTRIBUTING.md.

Either way we provide security backports until the end of the month.

Yeah, I just meant "users can/should already switch to NixOS 21.05 anyway" as an "excuse" for not testing the binaries thoroughly (you didn't check "Tested execution of all binary files" and I only ran --version and --help and lack a proper test setup for NixOS 20.09 - normally I perform more extensive testing).


difference between imu_correct and imu_raw

What are the differences between those two IMU messages? I have the imu_raw from my imu, but how to get the corrected one? I have only one imu. Also, what do you mean by external imu? I guess it's the one not integrated with the lidar. In that case, I just have the external one. I have the tf among base link, lidar link and imu link. In the params.yaml file, the extrinsic matrices are of which imu? Internal or external.

Hi @zillur-av
I have same question. Did you find out?

Personally, I think it should be to simulate GPS data with IMU data.


[AUTOPATCHER-CORE] Upgrade rt-tests to 2.6 none

[AUTOPATCHER-CORE] Upgrade rt-tests to 2.6 none
Upgrade pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=483478&view=results
buddy build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=489323&view=results

shouldn't we pull the rest of the changes from fedora? https://src.fedoraproject.org/rpms/realtime-tests/tree/f39


AWS Lambda Powertools for Java: Idempotency (QUARKUS Support)

Is your feature request related to a problem? Please describe.
Use of Idempotency module without the aspectj-maven-plugin and annotations.
Just like the SqsUtils Utility API.
This way it's more compatible with Quarkus when in native mode. Or even better, support Quarkus!

Hi @argenstijn,
We'll have a think about separately the functionality from the annotation. What exactly do you mean, when you say "support Quarkus"? Do you mean include required native-image reachability configuration? or something else?

With Quarkus support i mean a quarkus extension for AWS powertools.

@msailes @argenstijn not sure what to do with this one?
This exists: https://quarkus.io/guides/amazon-lambda, what would you like to add on top of this, that cannot be done with powertools currently?

@msailes is it the case that using aspects and annotations is less quarkus-y? We have just added a builder-style interface for the new batch and large message handling functions - no annotations. We could do a similar thing here in V2 for idempotency too, if that made sense and feels helpful.


[Feature] "vela addon enable" can enforce to skip all the check items

vela addon enable --force

I want to work on this issue @wonderflow can you help me!

@kethavathsivanaik great. ping @wangyikewxgm ,please give some guides here.

@kethavathsivanaik the main logic is here. IMO you can parse the force flag in installer and skip this func

Hi Siva @kethavathsivanaik
Right now if we set version check in addon like:
system:
   vela: >=1.2.3
   kubernetes: <=1.22

Vela CLI will check the version, if the check fails, the addon can not be enabled.
This feature is added in this PR: oam-dev/kubevela#3304
But sometimes these check may need to be skipped, so we need vela addon enable <addon-name> --force to skip the version check. You can refer to these codes: https://github.com/oam-dev/kubevela/blob/master/pkg/addon/addon.go#L1081
If you want to test it locally, you can use make vela-cli to build local vela cli, and test it with files in catalog in local: https://github.com/oam-dev/catalog
eg.
vela addon enable ./catalog/addons/<addon-name>

@wonderflow I want to work on this issue. May I work on it?

Sure please go ahead

@wonderflow @wangyikewxgm @FogDong
Hi, I have one question: when I set the yes flag to vela addon enable, it seems to skip all checks, including the version check. Is this issue different from vela addon enable --yes?
Please reply when you have time


Update app-attach-faq.md

Added a FAQ about AAD DS not being supported

@TomHickling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@TomHickling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@TomHickling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@TomHickling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Get CI working again

Fix various bits and get CI working again.
Updated source is tested against test_integration.py test case and no regression observed so far.
In my run most sources were updated successfully despite cv32e40p which requires manual intervention as https://github.com/litex-hub/pythondata-cpu-cv32e40p/pull/3.
Module: cv32e40p
  download: ✓ Success
  update  : ✗ Failed
    Error: Command '['git', 'subtree', 'pull', '-P', 'pythondata_cpu_cv32e40p/system_verilog', '/home/runner/work/pythondata-auto/pythondata-auto/srcs/pythondata-cpu-cv32e40p', '360d272898d81806be[33771](https://github.com/FlyGoat/pythondata-auto/actions/runs/12528018962/job/34942183621#step:5:33829)93870dbf83a3ea79f', '-m', b'Bump pythondata_cpu_cv32e40p/system_verilog subtree to 360d272898d81806be3377193870dbf83a3ea79f\n\nUpdated using 0.0.post148 from https://github.com/litex-hub/litex-data-auto\n']' returned non-zero exit status 1.

Module: cv32e40x
  download: ✓ Success
  update  : ✓ Success

Module: cv32e40s
  download: ✓ Success
  update  : ✓ Success

Module: cv32e41p
  download: ✓ Success
  update  : ✓ Success

Module: cva5
  download: ✓ Success
  update  : ✓ Success

Module: cva6
  download: ✓ Success
  update  : ✓ Success

Module: ibex
  download: ✓ Success
  update  : ✓ Success

Module: lm32
  download: ✓ Success
  update  : ✓ Success

Module: marocchino
  download: ✓ Success
  update  : ✓ Success

Module: microwatt
  download: ✓ Success
  update  : ✓ Success

Module: minerva
  download: ✓ Success
  update  : ✓ Success

Module: mor1kx
  download: ✓ Success
  update  : ✓ Success

Module: opentitan
  download: ✓ Success
  update  : ✓ Success

Module: picorv32
  download: ✓ Success
  update  : ✓ Success

Module: rocket
  download: ✓ Success
  update  : ✓ Success

Module: serv
  download: ✓ Success
  update  : ✓ Success

Module: vexriscv
  download: ✓ Success
  update  : ✓ Success

Module: picolibc
  download: ✓ Success
  update  : ✓ Success

Module: tapcfg
  download: ✓ Success
  update  : ✓ Success

Statistics:
download: 19/19 successful
update  : 18/19 successful
push    : 0/0 successful

@enjoy-digital Do you mind merging this and re-enable CI job here: https://github.com/litex-hub/pythondata-auto/actions/workflows/ci.yml ?
I'm trying to get more cores working with CI and it seems like many of them require bumping from upstream.
Thanks

Thanks @FlyGoat, this is merged.

@enjoy-digital It seems like GitHub token for this bot is no longer valid, do you mind to refresh it or possibly create a new bot account for it?
Thanks

Thanks @FlyGoat for the feedback. @mithro: do you know more about the bot account / github token, should switch to a different account?


random time may be zero

What type of PR is this?
/kind bug
What this PR does / why we need it:
1、fix When i is 0, it will cause the following error
panic: invalid argument to Intn [recovered]         panic: invalid argument to Intn
2、add ut for the device/cambricon（setNodeLock、LockNode、ReleaseNodeLock）
Which issue(s) this PR fixes:
Fixes # https://github.com/Project-HAMi/HAMi/issues/569
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/lgtm
thanks your contribution


accepting number code at keyTap/keyToggle

As example chars like äöüß isn't working at keyTap/keyToggle.
Best solution would be if the keyTap/keyToggle would accept keycode/rawcode (like from iohook) too. So you can just enter integer keycode in keyTap instead string which its limited.
As example, if you want to enter "E", which code for "E" is 69, so you can enter "robot.keyTap(69);" to tap "E", This allow us more chars support.
For typeString -> maybe arrays of chars code so it can support special chars too?
More information about virtual key:
https://msdn.microsoft.com/de-de/library/windows/desktop/dd375731(v=vs.85).aspx
http://keycode.info/

I see this is already on PR: https://github.com/octalmage/robotjs/pull/357

Adding raw keycode support would be great for what I'd use it for.


Post and Replies need context

Context Usecase:
Suppose I have answered a question. Now I don't want to scroll and find the question -> I can quickly go to my profile and see my replies -> which I can right now but:

I can't see the question so I lose context of what was I even replying to.
I can't quick jump to the thread where I have this reply in place. (Something like what FB does when you see your friend commented on a post and then you click on it so FB scrolls automatically to the friend's comment)
I can't share my answer with somebody -> shareable links is the easiest way to have word of mouth marketing.

Imagine if I am sharing links like:

question
Answer by xyz name

^ xyz name if popular is click bait.

I can't label issues right? Label it as enhancement.

^ Something like this. This is click bait if somebody shares it on any form of social network.

@prastut Hyperlinking replies would mean adding a permalink option to each answers, yeah? If so, any thoughts on where to place that link glyph and how the UI would be?

@prastut

I can't see the question so I lose context of what was I even replying to.

Thoughts on this?


Azure Event hub check pointing on the account storage level

Service Package Request
Is there currently a way to subscribe to the whole storage account (multi-containers)?
I want to checkpoint my event hub consumer to multiple containers or the entire storage account, my use case is that I'm interacting with multiple containers, and events are dispatched through them, for now, what I've read through the documentation and examples is only possible to subscribe to one container.
this is my current implementation where listening to one container, when the metadata changes the event gets picked by the consumer.
sharedCredential, errAzCred := container.NewSharedKeyCredential(ae.AccountName, ae.AccountKey)
	if errAzCred != nil {
		log.WithError(errAzCred).Error("Creating sharedCredential failed")
		return errAzCred
	}
	checkClient, errAzClient := container.NewClientWithSharedKeyCredential(ae.ContanerUrl, sharedCredential, nil)
	if errAzClient != nil {
		log.WithError(errAzClient).Error("Creating container client failed")
		return errAzClient
	}

	checkpointStore, errCheckPt := checkpoints.NewBlobStore(checkClient, nil)
	if errCheckPt != nil {
		log.WithError(errCheckPt).Error("Creating checkpoint store failed")
		return errCheckPt
	}
	consumerClient, errClient := azeventhubs.NewConsumerClientFromConnectionString(ae.HubNamespaceConnectionString, ae.EventHubName, azeventhubs.DefaultConsumerGroup, nil)
	if errClient != nil {
		log.WithError(errClient).Error("Creating event hub consumer failed")
		return errClient
	}
	defer consumerClient.Close(context.TODO())
	processor, errProcessor := azeventhubs.NewProcessor(consumerClient, checkpointStore, nil)
	if errProcessor != nil {
		log.WithError(errProcessor).Error("Creating event hub processor failed")
		return errProcessor
	}

I tried the following, and it worked, but when the metadata of the file changes the event is not picked by the consumer.

properties, err := consumerClient.GetEventHubProperties(context.TODO(), nil)
	if err != nil {
		return err
	}
	var wg sync.WaitGroup
	for _, partitionID := range properties.PartitionIDs {
		log.Debugf("start consuming partition %s", partitionID)
		partitionClient, errPC := consumerClient.NewPartitionClient(partitionID, nil)
		if errPC != nil {
			log.WithError(errPC).Error("couldn't create a partition client")
			return errPC
		}
		wg.Add(1)
		go func() {
			defer wg.Done()
			if errProcess := ae.ProcessEvents(partitionClient); errProcess != nil {
				log.WithError(errProcess).Error("failed to process the event")
			}
		}()
	}

Thank you

Hey @LatVAlY, is this also your question on Stack Overflow?
https://stackoverflow.com/questions/78267880/azure-event-hub-check-pointing-on-multiple-containers/78270293#78270293
Just want to paste it here for completeness (if not, then by coincidence another person was asking me about this :) ):
The simple answer is 'no', with what we ship in the SDK.
However, we do provide a customization point if you want to build your own checkpoint store. The interface is pretty simple:
type CheckpointStore interface {
	// ClaimOwnership attempts to claim ownership of the partitions in partitionOwnership and returns
	// the actual partitions that were claimed.
	ClaimOwnership(ctx context.Context, partitionOwnership []Ownership, options *ClaimOwnershipOptions) ([]Ownership, error)

	// ListCheckpoints lists all the available checkpoints.
	ListCheckpoints(ctx context.Context, fullyQualifiedNamespace string, eventHubName string, consumerGroup string, options *ListCheckpointsOptions) ([]Checkpoint, error)

	// ListOwnership lists all ownerships.
	ListOwnership(ctx context.Context, fullyQualifiedNamespace string, eventHubName string, consumerGroup string, options *ListOwnershipOptions) ([]Ownership, error)

	// SetCheckpoint updates a specific checkpoint with a sequence and offset.
	SetCheckpoint(ctx context.Context, checkpoint Checkpoint, options *SetCheckpointOptions) error
}

The implementation we ship with, if you need inspiration, is here: https://github.com/Azure/azure-sdk-for-go/blob/main/sdk/messaging/azeventhubs/checkpoints/blob_store.go

Yes it was me 😄
Thanks for sharing


Bug fix

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

Coverage data is based on head (1215acf) compared to base (c0f3af7).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #362   +/-   ##
=========================================
  Coverage     70.47%   70.47%
  Complexity     1377     1377
=========================================
  Files           196      196
  Lines          4454     4454
  Branches        550      550
=========================================
  Hits           3139     3139
  Misses         1135     1135
  Partials        180      180

Impacted Files
Coverage Δ

...dress/logic/commands/editcommands/EditCommand.java
85.71% <ø> (ø)

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

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


Add additional approver for Japanese Localization team

Describe your changes
Add a new approver for Japanese Localization team

[A] Junya Okabe (@Okabe-Junya)

Related issue number or link (ex: resolves #issue-number)
Slack link: https://cloud-native.slack.com/archives/C057F81GFUG/p1705327504026509
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Hi @Okabe-Junya
Please add your handle to https://github.com/cncf/glossary/pull/2814/files#diff-caeaffdf9c6b18dce7055a1442dfa1a9c363c9695f3aedb18ad7079079f03f65R505
(FYI: the setting is not effective since cncf org uses another global tool recently, but let's update it anyway for this time.)

Thanks!!
updated setting.yml

PTAL @iamNoah1 @nate-double-u @jihoon-seo :)


Documentation for exception thrown within when expression

Type of issue
Missing information
Description
I could not find good documentation on the expected behavior if an exception is thrown within the when expression. I wanted to confirm whether any additional exception handling was necessary in this use case.
Based on a quick test, it appears that the exception is suppressed, and the associated exception handler does not execute. Can the behavior be documented?
Page URL
https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/when
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/language-reference/keywords/when.md
Document Version Independent Id
15502bbc-5355-2add-0fa8-ae31dc01c122
Article author
@BillWagner
Metadata

ID: 6ef10b29-fbf8-5af9-8f8b-a694a1416e33
Service: dotnet-csharp
Sub-service: lang-reference

@jskeet I propose adding this to the C# 8 milestone.


CLID-101: Fix graph image mirroring after TLS verification enabling

Description

Fix graph image mirroring during MirrorToDisk and MirrorToMirror:

The error that was encountered was because in the specific case of the graph image, src of the image is localhost:55000/graph:latest. Therefore we were attempting to mirror from the cache registry (tls-verify not forced to false) .
The graph image is already built and pushed to the destination registry during the collection phase (graph.go)

destination registry is the cache for the case of mirrorToDisk
destination registry is the remote mirror for the case of mirrorToMirror

The fix is to exclude the graph image from the batch mirroring for the case of MirrorToDisk and MirrorToMirror.

Fix the generation of UpdateService.yaml file for MirrorToMirror workflow.

The root cause was that the imageSetConfig (o.Opts.Config) was getting modified by the collector (more exactly cincinnati's GetReleaseReferenceImages which was adding minVersion and maxVersion for releases).  This made the executor fail while GetRelease to prepare for UpdateService.yaml generation. The modified filter did not match the one saved previously by the collector
The fix consisted of using a copy of the Config for GetReleaseReferenceImages, instead of using the original one.

Fix the way we count errors during the batch phase, by creating error counters for release, operator and additional images.
Example of what is displayed in logs in case of errors during mirroring:

2024/04/25 14:54:05  [INFO]   : === Results ===
2024/04/25 14:54:05  [INFO]   : Some release images failed to mirror: mirrored 185 / 185 (185 failures) ❌ - please check the logs
2024/04/25 14:54:05  [INFO]   : Some operator images failed to mirror: mirrored 11 / 11 (11 failures) ❌ - please check the logs
2024/04/25 14:54:05  [INFO]   : Some additional images failed to mirror: mirrored 1 / 1 (1 failures) ❌ - please check the logs

Fixes # CLID-101
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Successfully perform MirrorToDisk, DiskToMirror and MirrorToMirror with the following imageSetConfiguration:
kind: ImageSetConfiguration
apiVersion: mirror.openshift.io/v2alpha1
mirror:
  platform:
    channels:
    - name: stable-4.13
    graph: true
  operators:
  - catalog: registry.redhat.io/redhat/redhat-operator-index:v4.14
    packages:
    - name: aws-load-balancer-operator
    - name: external-dns-operator
  additionalImages:
  - name: registry.redhat.io/ubi8/ubi:latest

Expected Outcome
Please describe the outcome expected from the tests

done pre-merge test for mirror2disk+disk2mirror and mirror2mirror, the graph-data-tag-digest pod running well, no issue

/lgtm

/reopen


Using Custom JWT for API Authentication in ArangoDB Cloud Enterprise

In the ArangoDB Community version, we can access all Foxx microservices APIs without restrictions, as the APIs are open and do not require authentication. However, in the ArangoDB Cloud Enterprise version, API access requires a JWT token generated using database user credentials (username and password). This setup presents challenges, as we want to avoid exposing or using database user credentials for generating tokens.
Currently, to authenticate API calls, we need to first generate a database-specific JWT using the database user credentials and include it in the request. Additionally, we also append our custom JWT (used for our application’s internal authentication) with a different name in the same request. This approach adds complexity and creates redundancy, as we are forced to handle two separate JWTs.
Given these constraints, we want to explore the possibility of directly using our custom JWT for API authentication in the Cloud Enterprise setup. Is there a way to configure the system to accept our custom JWT for authenticating requests, without requiring the database JWT or exposing database user credentials?
We would appreciate any guidance on how to simplify this workflow or any alternative solutions that can better support this use case.

Hi,
This can be configured by the operations Team.
Please raise an ArangoGraph support ticket, so they know which installation to work with and so on.

Closing as solved.


You can not change the menu position with the vs-position property

try using vs-position = "left" and it does not work

did you found solution for this..

the actual prop is position="left" not vs-position="left", seems like the documentation is wrong and there's no proper examples.


NuGet package

And also separate samples into their own solutions based on NuGet.

NuGet package is finally out.


Save events tracked in beforeUnload

I'm trying to save custom metric - how long user was on mu page - using Application Insights. I wrote following code (it is a part):
 var start = new Date();

var recordMetrics = function () {
    var end = new Date();
    var timeSpentTotal = (end.getTime() - start.getTime()) / 1000;
    if (window.appInsights) {
        window.appInsights.trackMetric("timeSpentTotal", timeSpentTotal);
        window.appInsights.flush();
        console.log("Sent data with timeSpent");
    }
}
window.addEventListener("beforeunload",
    function (e) {
        recordMetrics();
    });

But every request after the flush function is canceled. I'm almost sure that this is because App Insights create asynchronous AJAX request. Can I somehow change this to synchronous?

beforeunload is a tricky event to work with, mainly because of the asynchronous problem you've identified. We have some code to try to send events for you during beforeunload, as a best effort. (See https://github.com/Microsoft/ApplicationInsights-JS/blob/cac7b363a7f59147adff510f698bc3f4943f2ab0/JavaScript/JavaScriptSDK/Initialization.ts#L120 for reference)
Using flush like this should be synchronous however. Are you finding that is not the case? Perhaps try appInsights.context._sender.triggerSend(false); instead.

Can I create a pull request with a boolean parameter in flush method?
 The documentation for that method says it should be synchronous. The workaround I suggested is of course not a permanent fix if this is the case, but something to unblock between SDK releases

Adding a bool parameter to flush method to force it to be synchronous is a good idea. Please feel free to send a PR.
By default, the SDK will not block the user from navigating to a new page (e.g. by calling sync flush) as this may impact the navigation experience. But of course, there is nothing stopping the developer to block the navigation. Please keep in mind that this is not guaranteed to work because the browser may close any pending requests and terminate any script execution.
Another option is to use a Beacon API, which the SDK will support in v1.0.9 (#394). We are planning to release this version on May 8th. You can enable it using the isBeaconApiDisabled (see config).
This is an experimental feature. It would help us a lot if you could share your feedback (negative and positive) if you decide to use it. Thank you!


Split different equalities into separate traits

This allows us to implement abs_diff_eq for integers as well.

cc. @Andlon

@Andlon I'm just wondering how useful the RelativeEq trait is in practice... thoughts from somebody more up to speed with this stuff?

Relative equality is useful when you don't know the size of your numbers (such as e.g. randomized tests). I know nalgebra uses relative equality extensively for its tests, at least.