Error: Got unexpected extra arguments (list of files)

Hi @Luttik ,
I am trying to use it in an existing project and I am adding next config to my .pre-commit-config.yaml file.
The idea that I want to specify path to command so it checks src/ folder only
Here is an overview structure of project:
...
├── requirements-dev.in
├── requirements-dev.txt
├── requirements.in
├── requirements.txt
├── src
├── static
├── templates
├── tmp
...

This is config for auto-optional which I integrate into .pre-commit-config.yaml file
          ......
        - id: auto-optional
          name: auto-optional
          description: Adds the Optional type-hint to arguments where the default value is None
          entry: auto-optional src/
          language: python
          types: [python]
          ....

And I am getting next errors:

Error: Got unexpected extra arguments (src/asgi.py src/accounts/models/bill_blacklist_models.py src/hooks/models.py .... (continue list of files which should be proceeded)

How do I resolve it:
Added pass_filenames: false so in the end the config should be:
        - id: auto-optional
          name: auto-optional
          description: Adds the Optional type-hint to arguments where the default value is None
          entry: auto-optional src/
          language: python
          types: [python]
          pass_filenames: false

It takes longer time on the whole for verifying but this is a temp solution for that.

@danygielow do you have an quick answer to this? Otherwise I'll try to make time this weekend.

@Luttik , @danygielow , updated the request with section "How do I resolve it".
Unfortunately, it takes a longer time because it should verify the whole src/ folder and not only modified files.

It does look like expected behavior then rather than a bug (don't you agree?).
It is expected that it takes longer if you verify the entire project rather than just the changed files.
What I would advise is applying auto-optional once manually to all files and then using the pre-commit only for changed files, that seems like the most optimal solution.

@Luttik , I also simulated that behavior and changed 2 files locally with next config:
        - id: auto-optional
          name: auto-optional
          description: Adds the Optional type-hint to arguments where the default value is None
          entry: auto-optional
          language: python
          types: [python]

@DmytroLitvinov @Luttik
pre-commit runs by default only over staged files or files you want to commit. If you want to let it run over all files, you can run pre-commit run [-a|--all-files].
(see https://pre-commit.com/#top_level-files and https://pre-commit.com/#top_level-exclude for include or exclude config options)
You seem to have added the config from .pre-commit-hooks.yaml to your  .pre-commit-config.yaml.
You should use the following configuration.
see https://github.com/Luttik/auto-optional#pre-commit
  - repo: https://github.com/luttik/auto-optional
    rev: v0.2.3 # the current version as of this writing, use 'pre-commit autoupdate' to update to latest version
    hooks:
    - id: auto-optional

You seem to have added the config from .pre-commit-hooks.yaml to your .pre-commit-config.yaml.

@danygielow , it is because I am running it from local install package. I use repo: local for that case.
Simple configuration as Dany proposed:
default_language_version:
  python: python3.8

repos:
  - repo: https://github.com/luttik/auto-optional
    rev: v0.2.3 # the current version as of this writing, use 'pre-commit autoupdate' to update to latest version
    hooks:
    - id: auto-optional

Modified 2 files:

src/core/utils/send_emails.py
src/email_handling/mail.py

Error at the console:
auto-optional............................................................Failed
- hook id: auto-optional
- exit code: 2

Usage: auto-optional [OPTIONS] [PATH]
Try 'auto-optional --help' for help.

Error: Got unexpected extra argument (src/email_handling/mail.py)

Did you test/run it on your projects?
Because you will receive the same error if you modify 2+ files.
The problem still remains.

I see you modified your response so you can confirm the problem.

Yes, see my Edits in my comment above. I have now re-created the problem. And maybe https://pre-commit.com/#hooks-require_serial in the hook config is the solution.

@Luttik @DmytroLitvinov
I opened a PR with a bugfix. The PR has a breaking change though, because giving no arguments are not allowed anymore.


Minor docs improvements

Mostly changes for consistency but added a new section on how to test non-Python tools.

Codecov Report

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


[SMALLFIX] Do not apply umask for loaded file/dir from UFS

Also added integration tests, which failed before this change and passes after this fix.

@apc999 @calvinjia

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12178/
Test PASSed.

LGTM


lazyQuote option for stringSlice

possible fix for https://github.com/spf13/pflag/issues/370

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Actually ship the extension files

Also pin code-server at >=3.6.0, which is when we began pinning nodejs at 12.*.
Closes #3

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi!
Thanks for opening this PR. Some thoughts:

With the changes made in https://github.com/conda-forge/code-server-feedstock/pull/40, specifying the --extensions-dir should not be necessary anymore.
Regarding the current code-server pin: Thanks for bringing that up! vscode-icons requires vscode>=1.40.2 and apparently back then I still thought that code-server and vscode have the same versioning (which is not the case). I feel like we could probably go lower than 3.6 (the latest v2 release already runs run vscode>1.40.2) but it also seems like a reasonable requirement, considering the release is already 3 months old.

So, moving forward I'd suggest to remove all the explicit --extensions-dir code again and focus on the pin change for code-server. What do you think? @jtilly

With the changes made in conda-forge/code-server-feedstock#40, specifying the --extensions-dir should not be necessary anymore.

Sure, but then you should be pinning code-server to the version that specifies the --extension-dir. When I was building the package locally (on osx), I ended up with nodejs 14.* and code-server 3.5 in my build environment and nothing ended up in the package.

Fixed by #5

@jtilly Please never push to the main conda-forge repo ;)


Template name as variable

How to keep template name as variable
<template x-for="c in components">       <x-component-wrapper         x-component         x-template="c.name"         x-data="{ item: c }"></x-component-wrapper>     </template>

Oh, I'm not even sure that's possible but it's template not x-template. Hopefully that works 🤞

I've done some testing and it doesn't work as when you pass in :template it will read the value as component.template, or whatever variable you have passed in. However, I think I have it working but not when passing a URL.
As this is the first time it's been asked about I believe just supporting template as an ID will be OK for now. What do you think?


Requests from remote client return 404

Description
Application runs fine when client is on same computer and access is done using "localhost" on the URL. When client is on a remote computer OR if the client is on the same computer but access is done using "<IP_ADDRESS>" the application returns the 404 message below for the previously running url paths (including /ui)
{
  "detail": "The requested URL was not found on the server.  If you entered the URL manually please check your spelling and try again.",
  "status": 404,
  "title": "Not Found",
  "type": "about:blank"
}

I've tested using both app.run(host='<IP_ADDRESS>') and app.run(host='<IP_ADDRESS>').
I'm using flask.
I'm running using "python app.py".
Any help would be appreciated.
Thanks
Output of the commands:

python --version
Python 3.6.1

pip show connexion
Name: connexion
Version: 1.1.9
Summary: Connexion - API first applications with OpenAPI/Swagger and Flask
Home-page: https://github.com/zalando/connexion
Author: Zalando SE
Author-email: UNKNOWN
License: Apache License Version 2.0
Requires: swagger-spec-validator, inflection, flask, clickclick, PyYAML, requests, jsonschema, six

Could you provide some more information/details? The described issue sounds a bit weird to me and I have no good clue on how to help you. Maybe try to reproduce the problem with a minimal set of code/swagger and put this somewhere public (so we can reproduce the problem too).

Thanks @hjacobs. I've created a minimal app to demonstrate this.
The source code is published at https://github.com/remintz/connexion-test.git
You may check it behavior at this URL: http://<IP_ADDRESS>:8080/ui
No matter what request I send it always returns the same error.
The application console shows
INFO:werkzeug: * Running on http://<IP_ADDRESS>:8080/ (Press CTRL+C to quit)
INFO:werkzeug:<IP_ADDRESS> - - [23/Jun/2017 20:33:58] "GET /ui HTTP/1.1" 404 -
Tested with both .../ui and .../ui/
Thanks

Sorry... closed by mistake

@remintz see https://github.com/remintz/connexion-test/pull/1 . You are overwriting the SERVER_NAME and this causes the problem. I'm not sure why you do it (also other configs are not necessary IMHO).

It worked! Thanks a lot @hjacobs ! This code came from some sample I've found on internet. I'll be cleaning it up.


Show partial matches

That seems like a pretty nice feature to me - always print the longest partial match found. Yes? No?

Additional logic for this will likely degrade performance. I'm thinking that no.

it depends on whether it's using intfilter or default binfilter. with intfilter, it uses integer sized masking and checking operations wholesale, while with binfilter it does byte-by-byte masking and checking.

currently it discards tens of millions of keys if not more for few successful key finds, and adding any sort of "longest partial match" state logic in that path would seriously mess with perf. maybe not in very large way but it definitely would be measurable

so yeah i won't be implementing this.


Effects will be rendered out of the window.

See https://github.com/leanflutter/window_manager/issues/108.
I also tried not to use any window manager, this also appear. I think this maybe a bug.

Since there's been a discussion at the issue you linked. I will answer it there.


Backgroundtasks

Hey guys,
Is your library able to run and receive ble notifications while the app is in the background?
If not are you planning in implementing that feature?
Kind Regards

Hm strange.
I wanted to know why 'xabre' added "needs investigation" label.
As I know, this library is working in the background.
To be clear, it's not the library but native library is created to this way.

This needs investigation, because we haven't tested it and we have no implementation in the sample app that proves it ^^ And we haven't read a lot about the restrictions and requirements on each platform regarding the background ble.

OK. I see. Thanks for reply!

Well yeah, every native platform has its own background implementation. Windows has the background tasks with a GattCharacteristicNotificationTrigger and Android has its Services which can run in the background, dunno what iOS can do ;) what I know so far is, that apps wont run in the background for long, if they dont have any permission to do so. This permission can be achieved by register "backgroundtasks" for bluetooth or other external sources.  Please correct me if i got something wrong, but i dont understand why this library should run in the background without any registration for it.

@DumDumin @allbdeveloper This is exactly why I flagged it as 'needs investigation'.

@xabre -> add to docs


Point to the new provisioner version in KEB

Description
Changes proposed in this pull request:

Updated the KEB to use the latest provisioner version from the merged PR https://github.com/kyma-project/control-plane/pull/1462

Related issue(s)

/retest-required

/retest

/test

/test all


Linux alignment

I think the "Linux" heading of the IaaS table is too far to the left? I'm assuming the first "Non-Azure" heading on the next line is for Windows and Linux starts from Azure VMs again.

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

ID: 525c1c48-3355-bc91-b8d0-6104c1f5f098
Version Independent ID: 0d1c6361-aeac-b0c9-016d-38d21f483396
Content: Features and platforms supported by Azure Security Center
Content Source: articles/security-center/security-center-os-coverage.md
Service: security-center
GitHub Login: @monhaber
Microsoft Alias: monhaber

@dcbrown16 Thanks a lot for taking the time to report this. We really appreciate it.
You are correct, I have updated the doc with the correct alignment. It will be published shortly.

@dcbrown16 The doc has been updated. Thanks again for reporting this.


User/nick palmar/207/aruco pose detection

Refer to previous open PR on comments on why opencv dependencies are breaking. Need to fix prior to merging,

Going to keep this open for the next person who needs to deal with ARUCO tags... already exists. TBH should test and merge this..


"Poiana Blenchii" is listed in the drop-down menu\

Description:
"Poiana Blenchii" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Poiana Blenchii" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Poiana Blenchii, SĂLAJ".


cython and gcc

I used cython to convert the .py file to a .c file, and then used gcc to convert the .c file to a .so file. The following problem occurred in the second process.
gcc -fPIC -c test.c
In file included from Python.h:68:0,
from test.c:4:
pytime.h:122:12: warning: 'struct timeval' declared inside parameter list will not be visible outside of this definition or declaration
struct timeval *tv,
^~~~~~~
pytime.h:127:12: warning: 'struct timeval' declared inside parameter list will not be visible outside of this definition or declaration
struct timeval *tv,
^~~~~~~
test.c:1711:30: fatal error: structmember.h: No such file or directory
#include <structmember.h>
^
compilation terminated.

You should probably be using distutils to orchestrate the second step, as it will set up the flags and paths correctly for your Python. If this still doesn't work, please don't hesitate to ask on the mailing lists.

@robertwb  Thank you, I will give it a try, then I have any questions in the mail to ask you。

@robertwb I will try the method you said, what is the problem in the mail to ask you


Removed unused radioplayer.js

Removed radioplayer.js as I couldn't see it referenced.
@alexesprit could you review this please :+1:

We can always bring it back in a better form when people ask :+1:


✨ Close tabs on cmd (⌘) + W

Description
Pressing cmd (⌘) + W to close the currently open tab / file instead of the edit window similar similar to how it's done in Safari.
Alternatives Considered
Currently, you need to hover over the tab with the opened file name to see the close tab (x) icon button appear. Clicking on it closes the tab.
Additional Context
No response
Screenshots
https://user-images.githubusercontent.com/16290753/230755559-9fa1e2e5-8a6d-45d1-b23a-c1a90ce729b2.mp4

Duplicate of #1097. Closing this

How did I not find it? My bad!
Appreciate the work you're doing here 🔥


No build for 2.4.6

https://github.com/vvanholl/elasticsearch-prometheus-exporter/releases and the readme file in the 2.x branch both list released for ES versions up to 2.4.5, but there appears to be no build for 2.4.6. Could we have one?

There's an old PR for it: https://github.com/vvanholl/elasticsearch-prometheus-exporter/pull/52

https://github.com/vvanholl/elasticsearch-prometheus-exporter/releases/tag/<IP_ADDRESS> was released on 26th of October.


what is the maximal token number for InternVL-Chat-V1-5?

what is the maximal token number for InternVL-Chat-V1-5?
Thanks.

Thank you very much.


Conversation: sort messages

Thank you for contributing to the Cohere Toolkit!

[ ] PR title: "area: description"

Where "area" is whichever of interface, frontend, model, tools, backend, etc. is being modified. Use "docs: ..."  for CI changes.
Example: "deployment: add Azure model option"

[ ] PR message: Delete this entire checklist and replace with

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change

[ ] Add tests and docs: Please include testing and documentation for your changes

[ ] Lint and test: Run make lint and make run-tests

AI Description

The messages property in the Conversation class now sorts messages based on their created_at timestamp instead of their position.
File Changes

src/backend/database_models/conversation.py

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@5e8db60). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #333   +/-   ##
=======================================
  Coverage        ?   86.08%
=======================================
  Files           ?      173
  Lines           ?     6441
  Branches        ?        0
=======================================
  Hits            ?     5545
  Misses          ?      896
  Partials        ?        0

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


Lower tx fees 10x

Reusing DIP0001 activation once again

utACK

Even though this is not a hard fork change, activating at the hard fork makes sense.  I like this.


Cannot parse milli/microseconds with %s or %Q

It is impossible to parse timestamps like this<PHONE_NUMBER>.535  - one would expect %s.%L to do that, as that is the output of format() for that specifier.
I've addressed this on our fork and is part of this PR: https://github.com/d3/d3-time-format/pull/44

That said, if you have timestamps of the form %s.%L, I’d probably avoid d3-time-format entirely and just say new Date(timestamp * 1000).
Agreed, however we need a way for users to specify the format of the timestamps in their data, which sometimes happens to be %s.%L

Thanks for the fix!


Retry az login to fix dev release failures

Az login has a expiry of 3 hours, on of the job for dev release is taking longer than that. Hence, it fails to push the image.
This attempts to fix it!
https://github.com/devcontainers/images/actions/runs/4044601960

It takes couple of seconds, so should be fine. I'll get back to improving the logic if I find it degrading release perf.


Return type not set for a lambda assigned to a script variable

Fixes #14445.
Also refactor the code slightly.

thanks!


How to add custom header in postal

How to add custom header in postal ? I want to add unsubscribe link  in the header section.
Please guide me on this

As mentioned previously (in #1323 and originally discussed in #1292) you will need to include this header when you give the message to Postal.
This is entirely down to the software you are using so you should have a look at the support for that (or include the header if you are coding it yourself).

Thank you @willpower232  i will have a look into it.


show window info when did tap cluster

I want to show window info when click cluster like when click maker, but how i don't know implement. Please, help me.

This is actually a great request and should be on top of our list. Thanks.

FWIW: you can make markers that were added via the cluster library show their infoWindow. But it seems like you need to set the necessary marker properties before the marker gets rendered.
I was able to make cluster markers show the default info window, when I set their title or snippet properties in the willRenderMarker method of the  GMUClusterRendererDelegate. E.g.:
- (void)renderer:(id<GMUClusterRenderer>)renderer willRenderMarker:(GMSMarker *)marker {
    if ([[[marker.userData class] description] isEqualToString:@"POIItem"]) {
        POIItem *poiItem = marker.userData;
        marker.title = poiItem.name;
        marker.icon = [UIImage imageNamed:@"custom_marker_icon"]; // it’s also handy to specify a custom marker icon for all markers
    }
}

Please note that this code might not work well for copy-pasting. I only wanted to give a hint on how to achieve showing default infoWindows. Adjust accordingly.
Of course you’d need to conform to the GMUClusterRendererDelegate protocol and specify the delegate when initialising the renderer. For example like so:
id<GMUClusterRenderer> renderer = [[GMUDefaultClusterRenderer alloc] initWithMapView:_mapView clusterIconGenerator:iconGenerator];

((GMUDefaultClusterRenderer *)renderer).delegate = self;

Hi all,
we have committed a solution to this issue here https://github.com/googlemaps/google-maps-ios-utils/commit/cf739384397f3453ed667bbd8153f0cb3f231e2b but since it's a breaking change we would like to release it in a 2.x.x version (pending other big features we wanted to get in). For now please see the solution in the commit and see if you can work around using it.

Any update about this issue?
Is there any release date of version 2.x.x?
I am unable to do it using the approach explained before.

We are in the process of pushing out 2.0.0 so stay tuned. Probably sometime next week if everything goes smoothly!

@mountainvat  any update on the issue as of now ?

Hi @hiteshwork , 2.0.0 is out so please update your pod. The clusterManagerDelegate events (https://github.com/googlemaps/google-maps-ios-utils/blob/master/src/Clustering/GMUClusterManager.h#L40) now return a BOOL. Return NO to pass the event to the default handler (i.e showing info window).
Thanks.

@mountainvat returning NO still doesn't show the info window. when tapping on the marker in the cluster. What is the proper way to show the info window then?

Ok, in combination with willRenderMarker solution above and returning NO, I managed to show the info window.

In case anyone is looking for the swift version of the renderer:
func renderer(_ renderer: GMUClusterRenderer, willRenderMarker marker: GMSMarker) {
        if let poiItem = marker.userData as? MapClusterItem {
            marker.title = poiItem.title
            marker.snippet = poiItem.snippet
            marker.icon = GMSMarker.markerImage(with: colorAccent)
     }
}

MapClusterItem is your class (in examples the name is POIItem) and colorAccent is a UIColor of your choice.


feat(nexus): register an NVMe reservation on adding a child

Implement NexusChild::resv_register() which sends an NVMe
Reservation Register command as an NVMe IO Passthru command via
BlockDeviceHandle::{nvme_resv_register,io_passthru} when adding a
new child or when creating a nexus with children, though only to NVMe
child devices.
Add a Rust integration test, tacked on to nexus_multipath as that
already sets up a remote child, to verify that the key has been
registered.
Use a hard-coded key for now as this just lays the groundwork for
subsequent commits to acquire the reservation.
Fixes CAS-849

bors try

bors merge


Fix the Examples

Also, add the following cases:

Antenna pattern export
Interference analysis

see the kungfoo-multi branch.


Exception in clearPendingFlushCalls()

Test 'NATSUnitTests.TestReconnect.TestIsReconnectingAndStatus' failed: Test method NATSUnitTests.TestReconnect.TestIsReconnectingAndStatus threw exception:
System.InvalidOperationException: Collection was modified after the enumerator was instantiated.
at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource)
at System.Collections.Generic.Queue`1.Enumerator.MoveNext()
Conn.cs(2190,0): at NATS.Client.Connection.clearPendingFlushCalls()
Conn.cs(2222,0): at NATS.Client.Connection.close(ConnState closeState, Boolean invokeDelegates)
Conn.cs(2271,0): at NATS.Client.Connection.Close()
UnitTestReconnect.cs(488,0): at NATSUnitTests.TestReconnect.TestIsReconnectingAndStatus()
7 passed, 1 failed, 0 skipped, took 17,00 seconds (MSTest 10.0).

Great catch, thank you for contributing!  The current tests never picked that one up.  The fix looks good, but I suggest instead moving clearPendingFlushCalls() in close(), to after the lock before stopPingTimer();  No functional change, but it matches the go client more closely then.  Do you think this could be easily reproducible in a unit test?

I can reproduce on my workstation : just launching this test a few times make it fail.

Which test?

TestReconnect .TestIsReconnectingAndStatus

OK - Thanks @stephenjannin.  Would you mind making the adjustment and resubmitting?

done

LGTM.  Thanks for the contribution!


is this NAS compatible with trident ?

hello
i would like to know if the nas listed below can work with trident.
I have a kube cluster and i am looking for storage
https://www.hetzner.de/storage-box/bx60

Hi @dimthe,
Trident is currently only compatible with NetApp storage devices. For NAS this is limited to the NetApp ONTAP platform.

ok i understand @adkerr , no trident for me :(

@dimthe You may be able to use https://github.com/kubernetes-incubator/external-storage. I believe the NAS provisioner creates directories/shares as "volumes" under an existing NFS share.


fix: make add_mfa_indexes re-runnable

What kind of change does this PR introduce?
Noticed while doing support that amr_id_pk does not need currently check if the index already exists. As such, if the migration is re-run it will lead to an error.
Additional context
See Internal support ticket

lgtm! thanks for putting this together :)


Remove external dependencies from Scyther

Hey, awesome Project you made here - switched from netfox to scyther and have big hopes for it.
SnapKit and SDWebImage is cool and all, but i think it's better if a debug-tool does not introduce new dependencies to a project.
This all happened in a quick late night session and a hope i did not f**k things up ^^

i just noticed something is broken on sub iOS13 devices, even before my changes. I'm on it.

i just noticed something is broken on sub iOS13 devices, even before my changes. I'm on it.

It looks as though you don't have an app icon set in your Assets folder. Is that the case?

I will review this PR in detail and provide feedback. I am all for removing SDWebImage given its extremely minimal usage, however remain unconvinced on the value of removing SnapKit.
Regarding introducing new package dependencies, I feel as though this is quite common place and is exactly what SPM was designed to handle. There is definitely something to be said about introducing as few dependencies as possible but I will need to undertake extensive testing before we merge this in and remove SnapKit.

@stfnhdr Are you able to please separate this PR into two different ones.
One that removes SDWebImage and one that removes SnapKit. I ask as I do not want to completely negate your contribution however I do feel as though there are not compelling enough arguments right now that convince me that SnapKit should be removed. Would like to merge your SDWebImage stuff and then continue the conversation over on a separate PR for the SnapKit stuff.

Sounds good, yeah let's close this PR and make two separate PRs. I'll fix some error and make it a not late night action ;)

Sounds good, yeah let's close this PR and make two separate PRs. I'll fix some error and make it a not late night action ;)

Awesome. Thanks for understanding 😄


GeoJSON stay loaded at the end of a game

At the end of a game with a suggested map, I don't want the GeoJSON file to stay loaded.

Unfortunately, we don't have a cache system for maps or replay function
So, for me, it's useful.
But, It seems essential to have a reset button on the home page to remove directly custom map.


See the issue #1862

Adding an event when ManagedMqttClient actually subscribe/unsubscribe to topics. This event has to be used in parallel with connected event to know the exact state.

@dotnet-policy-service agree


Add version control info to release artifact pom

This will allow clojars and cljdoc to link back to project sources.
Also: Added description (grabbed from GitHub repo description) and url. These can be interesting to services.
Closes #71
Please answer the following questions and leave the below in as part of your PR.

[x] I have read the developer documentation.

[x] This PR corresponds to an issue with a clear problem statement.

[ ] This PR contains a test to prevent against future regressions

[x] I have updated the CHANGELOG.md file with a description of the addressed issue.

Thanks!


if should be an expression

There is no utility to conditionally evaluate an expression. A pragmatic example would be calculating a URL for the project when git.remote_provider contextual value exists. If if command acted like an expression instead of a command, then it would satisfy this requirement as well:
prompt: What is the project URL?
default:
  if: git.remote_provider
  eval: 'https://{{ git.remote_provider }}/{{ git.remote_owner }}/{{ git.remote_name }}'

this is also resolved via a078ad0


Pressing space in osu! Direct search does not move cursor.

When searching for a map in the osu direct menu, pressing space does not make the cursor move making it appear like space was not pressed.
Video: https://drive.google.com/open?id=1uH2S9KPPFDoh2vlsNVSwQCfACDH_0IPC
osu!lazer version: 2019.608.0 iOS

This is done with the stock iOS keyboard, I just have a skin tweak which doesn’t effect the usability.

Not important (since @swoolcock has confirmed) but that is not the stock keyboard.


Some README fixes

It's either misspelled (US) or misspelt (UK), I went for "misspelled" as it was already used in the README.
Also punctuation.

Thanks!


feat: add bodaay/HuggingFaceModelDownloader

bodaay/HuggingFaceModelDownloader: Simple go utility to
download HuggingFace Models and Datasets
$ aqua g -i bodaay/HuggingFaceModelDownloader

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ hfdownloader --help
a Simple HuggingFace Models Downloader Utility
Version: 1.2.9
Running on: /root/aquaproj-aqua/pkgs/github_release/github.com/bodaay/HuggingFaceModelDownloader/1.2.9/hfdownloader_linux_arm64_1.2.9/hfdownloader_linux_arm64_1.2.9

Usage:
  hfdowloader [flags]

Flags:
  -m, --model string         Model/Dataset name (required if dataset not set)
                             You can supply filters for required LFS model files
                             ex:  ModelName:q4_0,q8_1
                             ex:  TheBloke/WizardLM-Uncensored-Falcon-7B-GGML:fp16
  -d, --dataset string       Model/Dataset name (required if model not set)
  -b, --branch string        ModModel/Datasetel branch (optional) (default "main")
  -s, --storage string       Storage path (optional) (default "Storage")
  -k, --skipSHA              Skip SHA256 Hash Check, sometimes you just need to download missing files without wasting time waiting (optional)
  -f, --appendFilterFolder   This will append the filter name to the folder, use it for GGML qunatizatized filterd download only (optional)
  -c, --concurrent int       Number of LFS concurrent connections (optional) (default 5)
  -t, --token string         HuggingFace Access Token, this can be automatically supplied by env variable 'HUGGING_FACE_HUB_TOKEN' or .env file, required for some Models/Datasets, you still need to manually accept agreement if model requires it (optional)
  -i, --install              Install the binary to the OS default bin folder, Unix-like operating systems only
  -p, --installPath string   install Path (optional) (default "/usr/local/bin/")
      --maxRetries int       Max number of retries (optional) (default 3)
      --retryInterval int    Retry interval in seconds (optional) (default 5)
  -h, --help                 help for hfdowloader

Thank you!

v4.129.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.129.0


Run "go test " on save.

Hello there,
at first, thank you very much for that great plugin. I am not firm with Sublime Text yet nor the GoSublime plugin so if I can accomplish this task myself just leave me a hint. I would like to to have a shortcut to run "go test" with the package of my current go file. So one would have to read the "package" statement from the go file and run "go test" with that package name. The second feature I would love is to bind that functionality to the "Save" process. So my normal workflow would be editing a random go file, pressing ctrl+s (save) and automatically get a test run for the whole package. Coupled with the speed of go's compilation process, this is a perfect feature for test driven development.
Thanks,
Tobi

Is it possible to do this for all packages in the topmost open directory in Sublime Text, instead of just the package of the current file?


go 1.15 x509 common name deprecation

https://golang.org/doc/go1.15#commonname
I believe there will be updates necessary here to address this deprecation.

I ran the tests against RC1 without any issues, do you have any particular problems?

I wow, I did not leave very much information earlier.  Sorry about that.
The error I'm running into is this:
ERRO[2020-08-16T21:33:43-07:00] Get "https://vault.example.com8200/v1/auth/token/lookup-self": x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0

This appears to come from here: https://github.com/johanbrandhorst/certify/blob/master/issuers/vault/types.go#L67
But now after looking again, I see that the vault cert itself does not have a SAN.  I'll replace the certificate and see if I still have the issue.
The other thing that I was thinking needed an update was the Certify object that seems to require CommonName.  Omitting that portion yields an error, which I can get you if its useful.  I'm currently doing the following, but perhaps the CommonName is not effected by the deprecation?  Any pointers you have here would be appreciated.
https://github.com/xaque208/znet/blob/4ff74994197a37016af3eed3fd1cbe251be05ce8/znet/tls.go#L95

CommonName in the Certify object is required because there's otherwise no way to know what SAN to use when requesting a client certificate. I'd set that explicitly rather than inferring it from some unrelated config. The server side always uses the name used to connect to it when requesting a certificate from the issuer. Does that help? If you're only using it for server certificates I don't know if you need the Common Name.

That does help, thank you.  I do wonder if perhaps the need to pass the SAN could be handled with a NewCertify() type pattern that I've used and seen.  I'm using client auth as well, and that struct literal I pointed at is used by both client and server, but perhaps I should split that out if its just the server side of the cert that has deprecated the CommonName.

All good here, thank you for the pointers.


Simple tweak to make sure output from git release is right

release takes an explicit branch name and does all the right things except it was still outputting current_branch.name even if that were overridden. So if you were checked out to master but said git release foo-branch it would say 'Release master to master (y/n)'.

Coverage remained the same at 99.607% when pulling 2a6a11dd597fa65f6472053a13723896d94ef42d on mshappe:fix-release-with-explicit-branch into 8f61f2dee4749ecb2b42fba7f12facfe1c98b0ed on wireframe:master.


[kube-up][scale-out] event-exporter pod stuck in CrashLoopBackOff state

What happened:
after kube-up scale-out cluster started, from tp-1 master node,
kubectl get pod -AT -l k8s-app=event-exporter
TENANT   NAMESPACE     NAME                                     HASHKEY               READY   STATUS             RESTARTS   AGE
system   kube-system   event-exporter-v0.2.5-868dff6494-q4xs5<PHONE_NUMBER>252183928   0/1     CrashLoopBackOff   6          21m

What you expected to happen:
pod in Running state
How to reproduce it (as minimally and precisely as possible):
start 1 tp, 1 rp w/ 1 worker node scale-out cluster via kube-up.sh (code from poc-2022-01-30, with pr1320 cherry-picked)
Anything else we need to know?:
Environment:

Arktos version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools: kube-up scale-out (1 tp +1 rp x1 worker)
Network plugin and version (if this is a network-related bug):
Others:

pod log
{"log":"F0204 20:51:24.842859       1 main.go:75] Failed to initialize sink: failed to build sink config: not running on GCE, which is not supported for Stackdriver sink\n","stream":"stderr","time":"2022-02-04T20:51:24.843226811Z"}
root@hwc-kubeup-test-rp-1-minion-group-gdvq:/var/log/pods/system_kube-system_event-exporter-v0.2.5-868dff6494-q4xs5_67c6840e-001a-42df-bead-2bcaaf33cbaa/event-exporter#

Check whether this is service related.

@h-w-chen Did you solve that?


How to test blockingIntent with new test framework?

Based on the documentation of org.orbitmvi.orbit.syntax.simple.blockingIntent it's meant for updating the state immediate like this:
	@OptIn(OrbitExperimental::class)
	fun update(value: Int) {
		blockingIntent {
			reduce {
				state.copy(value = value)
			}
		}
	}

which means I need to call
@Test fun test() = runTest {
	createHost().test(this) {
		expectInitialState()
		invokeIntent { update(42) }
	}
}

but invokeIntent requires update to return a Job, which blockingIntent doesn't.
I've also considered simply skipping invokeIntent, but I have no access to the containerHost inside the test(this) { } lambda.

Thanks for the report, this is an oversight.
I'll fix it shortly - In the meantime you can return Job() in the invokeIntent lambda to make it work.

https://github.com/orbit-mvi/orbit-mvi/releases/tag/6.1.0


add topic tag mixed-precision

I suggest adding the topic mixed-precision in the About section

Good suggestion. Added mixed-precision in about section.


[TPCDS][TEST] Fix TPCDS suite issue and introduce tpcds queries(ten plus)

Why are the changes needed?

Fix tpcds schema issue and covert all tests, whose file is named like q14a.sql.
Introduce tpcds queries, whose index between ten and twenty.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #590 (d58c4e9) into master (746736f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #590   +/-   ##
=======================================
  Coverage   79.86%   79.86%
=======================================
  Files         118      118
  Lines        4545     4545
  Branches      549      549
=======================================
  Hits         3630     3630
  Misses        608      608
  Partials      307      307

Continue to review full report at Codecov.

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

Thanks, merged to master for v1.2.0


Lazy downloads

Downloading all the cmake binaries was too slow for me and they occupy about 20G on my hard disk. So I implemented a feature so that they're only downloaded when they're required.

Sorry for the late response. I will check your branch now.

I like the idea, but the current approach is mixing a lot of things: downloading, keeping a version list, and the actual checks. I am currently not entirely sure if I would merge it like this, or if first the two scripts need to be joined and a version overview can be expected in all cases.


Tigre core capabilities supported

W3C has a table with default capabilities: https://w3c.github.io/webdriver/#capabilities
Selenium docs have another opinion about that: https://www.selenium.dev/documentation/en/driver_idiosyncrasies/shared_capabilities/
IMHO we need support in version 1.0 only this, proxy is a bonus, but it is very important. I think that proxy can be supported only in version >1.0

Capability
tool key
Tigre equivalent
type
function

Browser name
"browserName"
import defitinion
string
Identifies the user agent.

Browser version
"browserVersion"
.version(int) import defitinion
string or int
Identifies the version of the user agent.

Accept insecure TLS certificates
"acceptInsecureCerts"
.insecure(bool)
boolean
Indicates whether untrusted and self-signed TLS certificates are implicitly trusted on navigation for the duration of the session.

Page load strategy
"pageLoadStrategy"
.strategy(str)
str
Defines the current session’s page load strategy.

Proxy configuration
"proxy"
.proxy(ProxyObject)
ProxyObject
Defines the current session’s proxy configuration.

Related to #25


[docs] Fix layout regression

Fix https://github.com/mui-org/material-ui/pull/27138#issuecomment-878951342

Next up, we might want to look into https://github.com/mui-org/material-ui/pull/27043#issuecomment-877776720 before the release of v5.0.0-beta.1, it's a regression, and seems to have a large impact (my assumption is that we would quickly get a bug report if we release it). The transition is now technically strict mode compatible 👌  but does no longer serves its original product purpose. I will see what I can do to improve it tomorrow. The simplest might be to remove the transition, it would already be better, but I hope that with more effort we can get the best of the two worlds, a simple, strict mode transition, with a great UX.

@eps1lon Are you able to reproduce the issue? So far we have the following information:

A URL for the reproduction: https://next--material-ui.netlify.app/guides/migration-v4/
A screenshot that shows the issue:

What else would you like to see? I don't get what you mean by providing a proper bug report. It doesn't seem to be about opening a GitHub issue, what you seem to ask for is about providing more information?
Maybe an extra piece of information that could make it clearer: the issue triggers when the viewport is larger than lg (when the side nav triggers), meaning for screen width that are larger than 1200px. But this can already be inferred from the screenshot and for which 83.3% of the G.A. sessions are above

What else would you like to see? I don't get what you mean by providing a proper bug report. It doesn't seem to be about opening a GitHub issue, what you seem to ask for is about providing more information?

You don't have to fill out an issue. The template is for people who don't know how to file a proper bug report. It doesn't have to be follow rigidly but the content needs to be the same. And you seem to willfully ignore this. I've told you this before. It's up to you to show that you are willing to improve. I probably had by now half a dozen of instances where I told you what you're missing. I was patient. I explained it. This credit is now gone.

Reverting accidental merge in https://github.com/mui-org/material-ui/pull/27269

I have merged to unlock the situation. I think that #27269 helps move the discussion forward, thanks for opening it 👍. I can't really act on these two feedback:

Please write a proper bug report.

the content needs to be the same

For me, it seems to suggest that you can't reproduce the issue, but I don't know why, nor what you have tried. My incentive is to provide enough information so that anyone can reproduce the issue.


chore: update simple-service and simple-job example

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Success of test-correctness depends on having new tagged version of catalog because fields interval, timeout and scheme have been removed from the schema after v0.1.1 which is the version the examples are using.

Closing and re-open targeting develop-0.10.0 branch in #145


New PyPI release with updated Python versions?

First of all, thanks for this great package, it saved me a lot of time when developing mocap support for Crazyflie drones in Skybrush (see here for the Skybrush server extension where we use libmotioncapture).
Is there any chance for a new release on PyPI with the recent commits, targeted at newer Python versions (3.10 and 3.11)? Version 1.0a1 has support only up to Python 3.9. If you don't have time for this but you are willing to accept a PR, let me know and I'll look into this. Ultimately, I'd like to make sure that motioncapture is pip-installable on Python 3.10 and 3.11 as we intend to move on to newer Python versions.

I agree, yes please make it compatible with later python versions :-)
I was looking a bit at using it in the python lib (https://github.com/bitcraze/crazyflie-lib-python/issues/411) but also bumped into this problem .

In principle this should be easy (see https://github.com/IMRCLab/libmotioncapture/pull/13), but currently the Mac doesn't work in CI (simply times out). Have you had this problem before @krichardsson ?

Nevermind Kristoffer, I found it. The code here is actually almost identically to the one at https://github.com/bitcraze/crazyflie-link-cpp, which was updated earlier. Good idea with having a weekly cron-job, too.

There is a new release now, including for Python 3.10 and 3.11.


Add end to end pipeline

What this PR does / why we need it:
This adds an end-to-end pipeline running on kubespray.
Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes https://github.com/elastisys/ck8s-pipelines/issues/49
Public facing documentation PR (if applicable)

Special notes for reviewer:
You may want to compare this pipeline to the ck8s-pipeline repo. Most of it should be the same.
Green run: https://github.com/elastisys/compliantkubernetes-apps/actions/runs/613901502
Checklist:

[ ] Added relevant notes to WIP-CHANGELOG.md
[x] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[x] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Pipeline config PR (if applicable)

If you change some config options (e.g. rename variable or change the default value) we may need to update the pipeline.

One more thing. Not sure if kubespray.yml is still the best name for the workflow file. Maybe change it to end-to-end.yml or something else that's more generic.


Rate Limit Response - I'd like to modify the body

https://github.com/spring-cloud/spring-cloud-gateway/blob/223165882707f2b9242e27c2d3851a9d76b04b6f/spring-cloud-gateway-core/src/main/java/org/springframework/cloud/gateway/filter/factory/RequestRateLimiterGatewayFilterFactory.java#L129
I'd like to customise the body when Rate Limit filter returns 429.
I can add the headers I want, but body is empty and I have a requirement to customise it.
I've tried other filter combinations, they work ok for other responses but it seems after this response is set to complete none of the filter / modifications are applied
Is there any workaround I could apply? Or any chance custom body is added any time soon to
org.springframework.cloud.gateway.filter.ratelimit.RateLimiter.Response ?
Thanks!

Not currently. Not quite sure how that would work right now.

me too

u just need to write a new GateWayFilterFactory,maybe code like that:
DataBuffer buffer = exchange.getResponse().bufferFactory().wrap(bytes);
return exchange.getResponse().writeWith(Flux.just(buffer));

Not currently. Not quite sure how that would work right now.

My GateWay version is 2.2.5.RELEASE,I had writed a new GatewayFilterFactory to limit request,which response that rateLimiter returned was not allowed.Filter would throw the exception,but sometime Url don't match any requestMapping,the response cause to throw exception.that is weird


small bug : in "missing update" tab "update date" colonne is order by month instead of year

small bug : in "missing update" tab "update date" colonne is order by month instead of year

@larg000 This was already fixed in a commit two weeks ago. You can see this in the closed issues section under issue #29 . Please check both open and closed issues prior to reporting to make sure it's not already reported and/or fixed. As instructed there, you need to download the latest build from the Actions section.
Please mark as closed. :)

sorry
but i don't know and don't undertstand how to download lastest build

@larg000 you can download the latest files under the "Action" tab here on GitHub, currently this is the link to the latest build: https://github.com/trembon/switch-library-manager/actions/runs/9628416371
and closing as duplicate of #29 as already meantioned


Switch test runner to project-fuzzball/test

Not sure if I'll switch to this yet since it's not "production ready" but just wanted to see what it'd look like. You can checkout branch project-fuzzball to try it out.

🎉 elm-test 2.0.0 has been released!


Drop Edge (render engine) support

Since Edge is switching to Chromium next month, we could/should drop support for the older layout engine in v5. Thoughts @twbs/team?

Will the switch roll out to all existing Edge users, or are we going to see another significant fracture in browsers?

Could be, because some systems stay behind. Like Microsoft supports older Windows 10 versions.

By default, Edge will replace their old engine unless some option is explicitly enabled: https://docs.microsoft.com/en-us/deployedge/microsoft-edge-update-policies#allowsxs

The browser is being replaced, the render engine within Windows however is not. Apps from the Microsoft Store, for now, will continue to rely on EdgeHTML (including PWAs in the Store like Twitter) and even once Microsoft provides WebView2 (the Blink-based webview) here, developers will have to actively enable it.
Regardless, Edge is a default update, but it isn't mandatory. People will be able to uninstall it even if it is applied to their system by default and many will have legitimate reasons to remain on the old Edge, or run them side-by-side.
Edge will be supported as a fully functional browser until at least May 2022, and that's assuming Microsoft will replace Edge by default with no way back in the Spring update for 2021 (as the EdgeHTML-based browser is still the default in version 2004 and won't be replaced in the second release for 2020).
Sure, Bootstrap doesn't need to provide support for that long, but killing support before normal users reasonably would no longer come into contact with software that do use EdgeHTML is just premature. Given that with Bootstrap 4 you ("you" as in "Bootstrap") were to afraid to kill of Internet Explorer 10, suddenly killing an entire browser seems extreme.

Thanks for the info @Studio384. Where did you get that from?

It's common knowledge, TBH.
Let's drop this patch for the time being.


Export named graphs from Catlab.Graphs

CC @slwu89

Export all the things! Thanks. After you merge this I'll update my #850 before requesting review.

Now that vertex_name and edge_name are part of the official API, we should update GraphvizGraphs and GraphCategories to use those by default, which will work for named graphs while preserving the old behavior unnamed graphs.

Ok, happy to do that, as it'll help figure out how to adjust #850.

@epatters I'm actually not sure what's the best way forward on updating GraphvizGraphs to use vertex_name and edge_name. Currently there's a lot of flexibility, as the argument node_labels can be boolean or a symbol referring to a subpart. For example, if we pass a NamedGraph to to_graphviz we have the option of printing the vertex ID, a subpart associated to vertices, or nothing. It would seem that using vertex_name in place of the method node_label for example, would mean we are forced to use the vname subpart.


Update Ember CLI

For upgrading to ember 2.x integrate the following commit: https://github.com/James1x0/ember-cli-build-info/commit/93b14c6b5c8adece34711d2e7baf9ae2d8f96dc9

Yes, but if also like to bring this addon up to date with Ember CLI version (currently 1.13.14 as of).


Tests

[x] Start adding unit tests for hooks.
[x] What are the best practices for testing React Hooks?

Pending tests to be added

State block

[x] createMemo
[x] createReducer
[x] useBoolean
[x] useCounter
[x] useNumber
[x] useGetSet
[x] useGetSetState
[x] useList
[x] useMap
[x] useObservable
[x] usePrevious
[x] useSetState
[x] useToggle
[x] useDefault

Animation block

[x] useRaf
[x] useSpring
[x] useTimeout
[x] useTween
[x] useUpdate
[x] useTimeoutFn
[x] useInterval

Lifecycle block

[x] useDeepCompareEffect
[x] useEffectOnce
[ ] useIsomorphicLayoutEffect
[ ] useLifecycles
[x] useLogger
[x] useMount
[x] useMountedState
[ ] usePromise
[ ] useRefMounted
[x] useUnmount
[x] useUpdateEffect

I've been using react-hooks-testing-library which seems pretty good so far.

When I was testing some async hooks I wasn't able to adequately get around the act() error messages from generating. I know with CI=true on create-react-app that these messages can prevent a build pipeline from succeeding, even if the tests themselves passed.

pick the test tools and make a test branch start it.

Please see pull request #221 :) let me know what you guys think

Can I help with this? I love this collection of hooks and I like adding tests so I'm more than happy to help with this.
I've written tests for hooks before just with react-testing-library by creating a fake component within the test itself, but react-hooks-testing-library is another good option (which basically do the same: wrap the hook into a fake component and return the results).
I've seen some tests already added by @ankithkonda so the set up and the structure is already defined, I can add more tests following same pattern.

@Belco90 that is very very welcome!

Ok so my idea is to add tests for all hooks section by section on storybook (animation, lifecycle, state, etc) and create a PR for each of those sections. Does that make sense for you? I'll start from animation section since it already has some tests and are simple to be tested.

Does that make sense for you?

@Belco90 Yes, it does.

It took me longer as expected as I was on holidays, but PR for first block of tests ready! https://github.com/streamich/react-use/pull/500

Second block of tests ready! https://github.com/streamich/react-use/pull/554
From now on I'll keep adding tests by blocks but I'll create a PR for each hook test, so 1) I avoid re-doing tests done by someone else while I create my PR, 2) I have my branches up to date and I don't have to keep it updated with master, and 3) I can merge bug fixes asap if I found any issue while testing.

@streamich I've edited your description to use it as pending hooks and blocks I need to test.

I don't know if you plan to keep the conversation about tests in this issue but just in case, just created a PR adding tests for useWindowScroll: https://github.com/streamich/react-use/pull/1183 following the same style @ankithkonda used for useWindowSize.

@gerardkabre Sorry but I can't continue adding tests here since I've been really busy lately. However, I think I added a nice bunch of tests here to get a solid base to keep testing other hooks.


Exotel Support

Hi ,
I can see exotel as a provider as code, but I don't see that option available.
So you plan to add it and if yes, Is there a timeline?

Do you mean on the dashboard?

Yes , is it available on dashboard , and If not can i integrate with it without dashboard , do you have any guides on it ?


graph drawio: Pull strategy and Interactive mode support

This PR introduces two major improvements to the clab-io-draw functionality:

Automatic Image Updates

When using the latest tag, containerlab now automatically checks and pulls
the newest version of clab-io-draw
Users are informed about the update status through clear log messages
Ensures users always have the most recent version when using latest tag

Native Interactive Mode Support

Adds native support for the -I (interactive) mode directly in containerlab
Users can now utilize the interactive

# Start interactive cli ui
containerlab graph -t st.clab.yml --drawio --drawio-args="-I --theme nokia_modern_dark"

@steiler I think I tried it when I originally added it but I was failing to get it work. If you want to move it to the docker lib, feel free to do it ;)


react-router wants<EMAIL_ADDRESS>

Just wanted to document it here, we can close it but if someone searches for the issue, maybe it'll help. If you get an error like this on npm install and you have history as a dependency in your project root:
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
Recommended course of action (for now?) is to change dependencies in package.json:
"dependencies": {
  "react-router": "1.0.2",
  "history": "1.13.x"
}

cc @mjackson @ryanflorence

Thanks, @chaseadamsio.
Yes, this workaround is only for right now. We thought that it would be an ok solution to just do a patch release of RR that doesn't show all warnings from the deprecations in the history package, and in a clean npm install it works well. In fact, people who are still using npm 2 will even get that peer dep automatically.
What we didn't think about is that people who already had<EMAIL_ADDRESS>installed would have to intentionally downgrade to get the warnings to go away.
@ryanflorence is working on a blog post right now to help people understand what we're thinking we'll do from here on out, but it basically boils down to:

bump major version numbers when we deprecate stuff
establish a regular release cycle to give people time to upgrade

I'll link to the blog post once he posts.

Yup, travis stopped working of us because of this and had to downgrade our dep on history module to fix it.

@mjackson What do you think of pushing out an interim history release that just drops the depreciation warnings? It might be the easiest all around.

I think the current workaround is fine. We've documented it and told everyone about it. Next major release will be better.

@BerkeleyTrue Why did your CI break?

@mjackson my travis build has also broke (without the workaround). https://travis-ci.org/mike-engel/bkmrkd/jobs/95724906, with my package.json here: https://github.com/mike-engel/bkmrkd/blob/redux-tests-43/package.json#L45

oops, me too
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

@mike-engel The key lines from your Travis log are:
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
But it's strange that you had history 1.15 installed when you specified 1.13 in your package.json... was it cached from a previous npm install?

@mjackson It was trying to install 1.15, because it matched the semver tag for ^1.13.1, which is how npm adds packages to package.json by default. More info
So, while I specified (and most people will this way if they run npm install history@1.13.x), and it got added as ^1.13.1`, 1.15.0 was treated as compatible by semver with the caret prefix.

@mjackson Same story here. We had history: ^1.9.0 in our package.json. npm installs 1.15 and then tries to install react-router which breaks the install.

Great, thanks for letting us know. We'll definitely fix this in the next release. Please use the workaround for now.

Just an update here: we've shipped history v1.16 which silences all warnings that were introduced into the history package since v1.13. Please give it a shot and let us know how it goes. If it goes well, we'll update the peer dep on the router.
Next time we ship those warnings will be in v2 of history, so you don't have to worry about it breaking your build until you intentionally upgrade.

I am unable to install ReactRouter due to this issue. I specify a dependency on (what I think is the right version) 1.13.0 but still I get this error. My dependencies from package.json:
  "dependencies": {
    ...
    "history": "^1.13.0",
    "react": "^0.14.3",
    "react-dom": "^0.14.3",
    "react-router": "^1.0.0"
  },

Error:
>npm install
npm WARN peerDependencies The peer dependency<EMAIL_ADDRESS>included from react-router will no
npm WARN peerDependencies longer be automatically installed to fulfill the peerDependency
npm WARN peerDependencies in npm 3+. Your application will need to depend on it explicitly.
\
><EMAIL_ADDRESS>postinstall c:\ws\bridalappnode\node_modules\history
> node ./npm-scripts/postinstall.js

Active code page: 65001
-

><EMAIL_ADDRESS>postinstall c:\ws\bridalappnode\node_modules\react-router
> node ./npm-scripts/postinstall.js

Active code page: 65001
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\\Program Files\\nodejs\\\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! code EPEERINVALID

npm ERR! peerinvalid The package history does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
npm ERR! Please include the following file with any support request:
npm ERR!     c:\ws\bridalappnode\npm-debug.log

It looks like it is trying to install 1.16.0 anyways... How to prevent it?

Update: changing deps to this seems to fix it:
"dependencies": {
    ...
  "history": "1.13.x",              <---- changed this
  "react": "^0.14.3",
  "react-dom": "^0.14.3",
  "react-router": "^1.0.0"
},

@Download :+1:

@Download fwiw, that was the recommendation in the opening issue comment of this thread.

Note that history 1.17.0 is out and doesn't have any deprecation warnings. Use it instead of 1.13.0 for now.

@chaseadamsio Yes, later I realized that as well... I completely misread the opening post and tried to set the dependency to 1.13.0
For me, using a letter instead of number in the version is just so unnatural! I unconsciously assumed that when it said 1.13.x it meant any micro version of 1.13...

Thanks, updating history in the dependencies did the trick for me.
"dependencies": {
"history": "1.17.0",                <== updated this line here
"react-router": "^1.0.0-rc4",
}

There is now also issue with router 3.0.0 and React + React-Dom ^15....
CircleCi builds also complaining:
<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^15.0.0 but none was installed.
Been holding us up for a while,

@stevematdavies It sounds like your build on Circle is using npm 2.x (I believe this is the default). I haven't used them in a bit (still waiting on an invite to their 2.0 beta. Hint hint to any @circleci folks! 😉 ), but you might try bumping your node version to v6.1.0 (which bundles npm 3) or adding a pre: - npm install -g npm to your circleci.yml
This is the same format as react-redux. Dan has some more info on what's going on here: https://github.com/reactjs/react-redux/pull/342#issuecomment-207647767


Move all regression tests to pytests

Is your feature request related to a problem? Please describe.
Several of the existing regression tests simply do output diff checking. We should migrate them to python tests so we can do better assertions and print more clear error messages when tests fail
Describe the solution you'd like
Move all regression tests to pytests

Thinking to take this one if no one else starts the work yet.

Thinking to take this one if no one else starts the work yet.

@collado-mike here is the PR for this: https://github.com/apache/polaris/pull/545


Library support

resolves https://github.com/hercules-ci/elm2nix/issues/28 but with mentioned limitations.

closing this. this doesn't seem that important and probably it would be better to get there in smaller steps anyway.


Clippy job in the pipeline can use sqlx Offline mode

The clippy job, unless I missed something or did not get there yet, (currently chapter 7) doesn't require to use a live database. So it can be shortend (importantly also in execution time and resource usage) by removing everything but the checkout, the cache and then the linting task.
Its generally somewhat annoying to me how many tasks are repeated between the different jobs, but admittedly I also cant find an approach that does this better without tradeoffs (such as doing them only as steps in one task, or committing some image to an image repository, which requires more buy in into some container image repository such as github, and might leave behind a lot of garbage).

here is the job for clippy, worked up to chapter 7:
  # `clippy` container job
  clippy:
    name: Clippy
    runs-on: ubuntu-latest
    container: ghcr.io/catthehacker/ubuntu:act-latest
    env:
      SQLX_OFFLINE: true
    steps:
      - uses: actions/checkout@v3
      - uses: dtolnay/rust-toolchain@stable
        with:
          components: clippy
      - uses: Swatinem/rust-cache@v2
        with:
          key: sqlx-${{ env.SQLX_VERSION }}
      - name: Linting
        run: cargo clippy -- -D warnings

Fixed!


Issue (or possible bug?) using the underscore _ after curly brackets {}.

Hi, just want to say thank you so much for the plugin! It makes using obsidian for maths not only bearable but very flowing.
I have a slight issue which is after using curly brackets {} (for example $\vec{x}$), then using the underscore character _ creates a pair of underscores __ as I would expect in the normal obsidian editor but within the latex delimiters, I would expect just one _ to be inserted as it is when inserted after for example $x_$. This also creates an issue with the automatic enclosing of things following the underscore since where $x_2$ goes to $x_{2}$ and $\vec{x}2$ goes to $\vec{x}{2}$, with the double insertion and blindly typing onwards, $\vec{x}__$ goes to $\vec{x}2$ which goes to $\vec{x}2}{$ which is definitely not desired behaviour (and means one has to go back to fix the resulting mess breaking the flow of typing) nor can I see a ruleset which would cause this.
This would be completely solved by not pairing _ characters within the latex delimiters I believe!

hi @mayurankv !
sorry, I don't seem to be able to replicate the issue you are facing.
on my machine, typing \vec{x} followed by the underscore creates only one underscore.

Ah, it turns out that I had automatic pairing of markdown syntax (so _ and *) on in obsidian as a whole. I realised I never really use this feature outside math blocks anyway since I use Cmd I and Cmd B for italics and bold anyway so I've just turned it off and that now works for me! Thanks!


Test-sign an ethereum address

Digicert's cloud-based document signer works on PDFs only. They tell me that they have an application which will allow other data to be signed, but we should test that this works by signing (in our case) an ethereum address.

Not able to complete


I'm looking to improve the web interface

I wanted to throw some ideas around
so far i have added a second input selection box for "console"
this filters the list by console which i have added to the map JSON files
eg:
{
"name":"N64 Tony Hawk 1, 2 & 3",
"desc":"A & C-Down and B & C-Left share the same source.",
"console":"N64",
"map":[mappings]
}
this allows for any new type of console and game to be added without having a massively cluttered list

Merged thanks, no need to open issue, PR is enough.


First pass at shared gradle plugin to share code with ios

@hanningni please review

:+1:

@aleffert Just wanted to suggest on thing, Can we have custom build names for both Android and iOS, so that the build name will include build number ?
We always have been renaming builds manually and I believe it's possible to customize the build name through gradle script. I would suggest build name should include version name, version code, the config type (staging, prod etc.)  and the build type (debug or release).
cc @hanningni


Tooltips to show help on assembly

Several requests to have help about instructions from those (like me!) who are rusty in assembler.

https://github.com/zneak/x86doc looks good, it's used to generate http://www.felixcloutier.com/x86/ from the Intel manuals.
I use http://ref.x86asm.net/ in https://github.com/AdoptOpenJDK/jitwatch
Will let you know if I find any others (ARM would be next on my list).

I'm 5 years late, but I'm on it!

Thanks @RabsRincon - I'm very excited!


workbox-cli pipe __WB_MANIFEST to another process?

Welcome! Please use this template for reporting bugs or requesting features. For questions about using Workbox, the best place to ask is Stack Overflow, tagged with [workbox]: https://stackoverflow.com/questions/ask?tags=workbox
Library Affected:
workbox-cli
Browser & Platform:
Node
Issue or Feature Request Description:
If I understood these #2383, #2401  correctly it appears the webpack plugin allows for a compileSrc which in turn provides the __WB_MANIFEST as a file?
I have a similar use case but outside webpack.
For context, I am utilizing workbox InjectManifest from the CLI "workbox injectManifest workbox-config.js && workbox copyLibraries public".
Obtaining the results which are injected into the symbol would consolidate a build process I have for a server rendered application which utilizes hashid and file names. The object would be a drop in replacement to push assets to storage and in another step, map the file names to server side templates generation.
Perhaps the config file could include a new optional key? Here is what I was thinking:
const path = require('path')
const mode = process.env.NODE_ENV || 'production'
const assetPath = new RegExp(/fonts|images|css|js/)

const cdnRewrite = async (manifestEntries) => {
  const manifest = manifestEntries.map((entry) => {
    const cdnOrigin = '/static'
    if (assetPath.test(entry.url)) {
      entry.url = `${cdnOrigin}/${entry.url}`
    }
    return entry
  })
  return { manifest, warnings: [] }
}

module.exports = {
  mode,
  manifestDest: 'assetManifest.json', // pun very intended
  swSrc: 'service-worker.js',
  swDest: 'public/sw.js',
  globDirectory: './public',
  globPatterns: ['**/*.{js,css,png,ttf}'],
  globIgnores: ['js/workbox-v*/*', 'js/sw.*'],
  manifestTransforms: [cdnRewrite],
}

If there is a way to do this with existing APIs please let me know - I just could not find anything that did not focus on webpack.

👋 @rhodee
I think your best bet is to use the getManifest() function from workbox-build directly from a Node script. (workbox-cli is a wrapper on top of workbox-build, but it doesn't expose the getManifest() functionality.)
That provides a clean way to get the manifest info without doing any other steps, and you can then write that info to a file, use it as part of a larger build process, etc.
Let me know if you need further assistance!

Did exactly what I need - thank you for the guidance @jeffposnick


chrome://history redirects to default new tab page

This is not a problem with the extension. I'm just hoping that someone here could direct me to the solution. thx! The default new tab page is chrome-search://local-ntp/local-ntp.html. I added this question to Super User - so you can also answer this question there if you so choose: https://superuser.com/q/1304032/315947

How about you open issues related to the project and not clutter the issue tracker with crap?

@robbyoconnor please don't be rude. I openly allow people to post questions.

@quipquopro sorry I never saw your message. I changed my Gmail inbox to auto-sort into categories, and it screwed up some of my rules that tagged messages containing "New Tab Redirect".
If you're still having this issue, it's possible that it's another extension that's overriding "chrome://history". I know there's a way to install "hidden extensions", or ones that don't show in the extensions list that a user sees in the browser. I unfortunately don't know that process; it may be policies, registry entries in Windows, or even a custom service in your workplace which makes a one-time modification.
If it's another extension, you can search through your installed extensions directory, which should be C:\Users\**username**\AppData\Local\Google\Chrome\User Data\Default\Extensions\. If you find a history override, it would look like this:
"chrome_url_overrides": {
    "history": "something.html"
  },

If you do find something, I would suggest notifying the security team in your company. If there's a malicious extension, the company should be made aware of it. If there's an extension which is installed by your company, I would press them to consider removing it. Disallowing users to view the history on their own machines (even if it is company owned) could be a security risk.

@QUIPQUOPRO -- I apologize :heart:

@QUIPQUOPRO -- I apologize ❤

No problem, I never saw your comment, so my fragile ego lost no hit points.


is Posible to set webhook in config via POST Request

Hello.
that my request for update i hope
is Posible to set webhook in config via POST Request?
thanks

I'm expecting include this options on library, but not enough time yet. ASAP when implemented i will let you know.

🎉 Implemented, Check it on Change Webhook


Deployment docu: Confusion about authentication token

I have some confusion about this wiki page:
https://github.com/OfficeDev/microsoft-teams-hrsupport-app/wiki/Deployment-Guide
In step 1, number 9, there is this sentence:

Now go to your app registrations, then go to Authentication and check the checkbox ID tokens and Access tokens for Bot App and Only ID tokens for Configurator App.

But without adding a platform "Web" with URI, there is no implicit grant and so no checkboxes.
Otherwise, later in the deployment guide, there is the step 3 which correctly describes the process of configuring authentication.

Thank you for the feedback. Please refer step 3 for the process of configuring authentication.


Test Cases Following International Standards

Convert base currency to USD and align test cases with international (read: US centric) accounting conventions.

Codecov Report
Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

pyledger/ledger_engine.py
0.00%
1 Missing :warning:

Flag
Coverage Δ

unittests
69.44% <50.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines
Coverage Δ

pyledger/memory_ledger.py
94.23% <100.00%> (+0.05%)
:arrow_up:

pyledger/ledger_engine.py
59.83% <0.00%> (-0.21%)
:arrow_down:

... and 1 file with indirect coverage changes


feature/cplp 1616  reg/company role agreement data refactor

https://jira.catena-x.net/browse/CPLP-1616

corresponding devenv-PR https://github.com/tractusx-team-portal-onboarding/devenv/pull/160


Duplicated distribution information

In some cases, the information on native range, introduction pathway and degree of establishment in the description extension is duplicated  for a single species. The reason for this is that a single species can have two or three records in the main checklist: one for Flanders, one for Wallonia and/or one for Brussels Capital Region.
Example:

scientific name
locality
degree of establishment
introduction pathway

Boa constrictor (Linnaeus, 1758)
Flanders
casual
escape_pet / release_other

Boa constrictor (Linnaeus, 1758)
Wallonia
casual
escape_pet / release_other

For each of these records, the native range information must be the same. I suppose that introduction pathway and degree of establishment could vary between these records, but this is not the case for this specific checklist. This results in a lot of duplicated information in the description extension, see example below.

taxonID
description
type

taxonID_aabbcc
introduction pathway
escape_pet / release_other

taxonID_aabbcc
introduction pathway
escape_pet / release_other

taxonID_aabbcc
degree of establishment
casual

taxonID_aabbcc
degree of establishment
casual

I have two suggestions to solve this:
1: Remove all duplicated information, the information is general for the species in Belgium

taxonID
description
type

taxonID_aabbcc
introduction pathway
escape_pet / release_other

taxonID_aabbcc
degree of establishment
casual

2: Keep all duplicated information, and mention the specific locality:

taxonID
description
type

taxonID_aabbcc
introduction pathway (Flanders)
escape_pet / release_other

taxonID_aabbcc
introduction pathway (Wallonia)
escape_pet / release_other

taxonID_aabbcc
degree of establishment (Flanders)
casual

taxonID_aabbcc
degree of establishment (Wallonia)
casual

For obvious reasons I prefer option 1
@timadriaens
@LoicVanDoornINBO

we should probably deal with this the same way as for other checklists. How is that done there e.g. in the manual of alien plants or any of the others?

For the manual of plants, we don't have this problem as distribution information is contained in a single line.
For the ad hoc checklist, we ignore the multiple distribution lines, same as my first suggestion.

OK, yes, makes sense. Let's go for option 1.


Populate recordingURL when source playback is ready (#1847)

This is to re-roll #1847 which was reverted previously in 6336f953b66035f7c280dd428c13544056b87a3e

LGTM but looks like tests are failing


update the powertoys tray graphic to a a single color

Summary of the new feature/enhancement
Please change the powertoys icon to something that uses a single color. Most applications follow this rule which makes is easy to organize their icons in the tray in a way that lets you quickly find the icon, but a multicolor icon like the current one doesn't fit anywhere. Alternatively, just make it a white outline icon without any color, like most other win10 icons, so it can be grouped with those.
Small thing, surprisingly important to get right.

Duplicate of #3523

I wouldn't use the word "important", feels like a personal preference to me.

accessibility is never personal preference.


Defer switch exhaustiveness checks

Fixes the issue reported here: https://github.com/microsoft/TypeScript/pull/34840#issuecomment-551283936.

@RyanCavanaugh @weswigham We might consider cherry picking this for 3.7.3.

@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt

@ahejlsberg does this also fix #34960?

Based on this exchange in the other PR, I'm guessing yes?

Yes, this fixes #34960.


fix: if endpoint does not support Version CC, still query CC versions for the endpoint, not the node

fixes: https://github.com/zwave-js/node-zwave-js/issues/5546

@zwave-js-bot automerge


Null producer_block_id

In some cases, the producer_block_id field in the response of cleos get transaction is null.
After pushing a transaction into the block number n, then I send the transaction query (cleos get transaction) to the active BP of the block number n. The producer_block_id field is always null anytime. But the non-active BPs of the block number n will response the correct producer_block_id.

Version Test: v1.6.6 / v1.7.4 / v1.8.1
BP Num: 1 BP / 2 BPs / 3 BPs / 5 BPs

Minimum PoC:
$ nodeos --http-server-address=<IP_ADDRESS>:8888 --p2p-listen-endpoint=<IP_ADDRESS>:9000 \
    --producer-name="eosio" \
    --agent-name="eosio" \
    --enable-stale-production \
    --signature-provider "EOS6MRyAjQq8ud7hVNYcfnVPJqcVpscN5So8BhtHuGYqET5GDW5CV=KEY:5KQwrPbwdL6PhXujxW37FSSQZ1JiwsST4cqQzDeyXtP79zkvFD3" \
    --plugin eosio::chain_api_plugin \
    --plugin eosio::http_plugin \
    --plugin eosio::history_plugin \
    --plugin eosio::history_api_plugin \
    --filter-on="*" \
    --delete-all-blocks \
    --http-validate-host false &

$ cleos create account eosio eosio.token EOS6MRyAjQq8ud7hVNYcfnVPJqcVpscN5So8BhtHuGYqET5GDW5CV
create account eosio eosio.token EOS6MRyAjQq8ud7hVNYcfnVPJqcVpscN5So8BhtHuGYqET5GDW5CV
executed transaction: b6ebb386ec0bc66d7ab4d3e6a924b8c83ad57130a24b95f12f720ef4032e8cc8  200 bytes  375 us
#         eosio <= eosio::newaccount            {"creator":"eosio","name":"eosio.token","owner":{"threshold":1,"keys":[{"key":"EOS6MRyAjQq8ud7hVNYcf...
warn  2019-08-18T14:14:14.713 cleos     main.cpp:495                  print_result    warning: transaction executed locally, but may not be confirmed by the network yet

$ cleos get transaction b6ebb386ec0bc66d7ab4d3e6a924b8c83ad57130a24b95f12f720ef4032e8cc8
...
      "producer_block_id": null,
...

Thanks.

Hopefully this helps clarify: https://github.com/EOSIO/eos/blob/master/plugins/mongo_db_plugin/mongo_db_plugin.cpp#L330

Thank you for sharing.
But in the PoC, there is only one producer producing the block. The pending_block_mode should be in the producing mode, not the speculating mode. How does an incomplete block produce in this way? Or how to make an incomplete block finalized?
Thanks.

When you are the producing a block the block_id is not known until the block is complete. Therefore, as a single producer your traces for transactions placed in the block have a null producer_block_id.


Have dashboard handle requestId parameter in login endpoint requests

Is your feature request related to a problem? Please describe.
This is related to rancher/dashboard#8964. Currently, when using the rancher token command with a SAML provider (such as OKTA) the rancher CLI produces a URL for the user to initiate the login process inside of rancher. This uses the /login endpoint, which produces a login page for the old ember UI.
Going forward, the rancher CLI should not use this endpoint, and instead should use an equivalent page from the dashboard.
Describe the solution you'd like
The URL produced by the CLI should refer to a dashboard endpoint, like /dashboard/auth/login.
Describe alternatives you've considered
Other endpoints could be used, as long as they are not from the ember UI.
Additional context
Initial testing showed some issues with the /dashboard/auth/login endpoint. More testing will be needed to verify if this endpoint is suitable (or needs some modification).
Update from https://github.com/rancher/rancher/issues/41651
@eliyamlevy spoke with @mantis-toboggan-md and after some investigation, they came to the conclusion that the solution is some updates to the Dashboard code to handle the case that the /dashboard/auth/login/ endpoint receives a request with the requestId parameter. Currently the dashboard does not handle this parameter.
The solution is to add this case to the dashboard code, have it forward the request to the norman api (mirroring the ember flow) and then updating the login url to the dashboard one.
There shouldn't need to be any major changes to the CLI code.

The relevant Ember code we need to replicate appears to be this: https://github.com/rancher/ui/blob/master/lib/shared/addon/saml/service.js#L12-L30

The original issue requesting this functionality in the Ember UI is https://github.com/rancherlabs/rancher-security/issues/444

@torchiaf I think there are 3 params that need to be passed through:
requestId
publicKey
responseType

I think in here - https://github.com/rancher/dashboard/blob/master/shell/components/auth/login/saml.vue - where it passes the body, the body needs to include the above parameters from the query string, if they are set there.


cmd: rename otk externals

Externals being used by otk are going to try and follow a naming schema. This commit adjusts the current names to that schema.
Needs PR in otk as well.

Do we want some separate directory for this?

The discussion in https://github.com/osbuild/otk/pull/196#discussion_r1756580990 has been resolved by following the original idiom. I've updated this PR accordingly. Will need a PR in otk as well.

Alright.  Let's get this merged and I'll rebase #960 to match.


Keep bumping warning logs for reading process_agent_config.host_ips

Output of the info page (if this is a bug)
2020-03-05 05:45:40 UTC | PROCESS | WARN | (pkg/config/viper.go:168 in GetStringSlice) | failed to get configuration value for key "process_agent_config.host_ips": unable to cast <nil> of type <nil> to []string

Describe what happened:
It keep bumping this message for a period of time.
Describe what you expected:
Do not be so noisy
Steps to reproduce the issue:
Run the agent, and observe the logs.
Additional environment details (Operating System, Cloud provider, etc):
AWS ECS  using  datadog/agent:7-jmx
Additional infomation:
Search the word process_agent_config.host_ips, can find it only shows in the code,
don't see it in any config files.  Maybe can clean up, or at least document it somewhere.
https://github.com/search?q=org%3ADataDog+process_agent_config&type=Code

I'm seeing this, too. Running on AWS EKS, seems to occur once during Helm deployments.  I assume this is just noise due to deployments disrupting things?

I'm seeing this, too. Running on AWS EKS, seems to occur once during Helm deployments.  I assume this is just noise due to deployments disrupting things?

Same, I started noticing it as well, although its a warning, I was hoping if there is some config change needed on the DD agent deployment.

Same, I started noticing it as well when I redeployed the agent, similar to what @mgrecar  mentioned. Although it's a warning, I was hoping if there is some config change needed on the DD agent deployment. For me, it's happening on EKS, not ECS


Cannot run Network Architecture Search, ends up in error

Unable to run notebooks 2,3 which have network architecture search.
For instance, for notebook 3, on Deephyper version 0.4.2, it ends up with error:
/usr/local/lib/python3.9/dist-packages/deephyper/evaluator/_evaluator.py:101: UserWarning: Applying nest-asyncio patch for IPython Shell!
  warnings.warn(
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
[<ipython-input-17-10be92266985>](https://localhost:8080/#) in <module>
      3
      4
----> 5 agebo_search = AgEBO(
      6     problem,
      7     run_base_trainer,

2 frames
[/usr/local/lib/python3.9/dist-packages/deephyper/skopt/learning/forest.py](https://localhost:8080/#) in __init__(self, n_estimators, criterion, max_depth, min_samples_split, min_samples_leaf, min_weight_fraction_leaf, max_features, max_leaf_nodes, min_impurity_decrease, bootstrap, oob_score, n_jobs, random_state, verbose, warm_start, ccp_alpha, max_samples, min_variance, splitter)
    232         splitter="random"
    233     ):
--> 234         super().__init__(
    235             base_estimator=DecisionTreeRegressor(splitter=splitter),
    236             n_estimators=n_estimators,

TypeError: __init__() missing 1 required positional argument: 'estimator'

for notebook 3, on Deephyper version 0.5.0, it ends up with error:
/usr/local/lib/python3.9/dist-packages/deephyper/evaluator/_evaluator.py:126: UserWarning: Applying nest-asyncio patch for IPython Shell!
  warnings.warn(
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
[<ipython-input-17-10be92266985>](https://localhost:8080/#) in <module>
     10     random_state=42,
     11 )
---> 12 results = agebo_search.search(max_evals=30)

11 frames
[/usr/local/lib/python3.9/dist-packages/keras/optimizers/optimizer_experimental/optimizer.py](https://localhost:8080/#) in tf___update_step_xla(self, gradient, variable, key)
     26                 try:
     27                     do_return = True
---> 28                     retval_ = ag__.converted_call(ag__.ld(self)._update_step, (ag__.ld(gradient), ag__.ld(variable)), None, fscope)
     29                 except:
     30                     do_return = False

ValueError: in user code:

    File "/usr/local/lib/python3.9/dist-packages/keras/engine/training.py", line 1249, in train_function  *
        return step_function(self, iterator)
    File "/usr/local/lib/python3.9/dist-packages/keras/optimizers/optimizer_experimental/optimizer.py", line 208, in _update_step_xla  *
        return self._update_step(gradient, variable)
    File "/usr/local/lib/python3.9/dist-packages/keras/optimizers/optimizer_experimental/optimizer.py", line 224, in _update_step  **
        self.update_step(gradient, variable)
    File "/usr/local/lib/python3.9/dist-packages/keras/optimizers/optimizer_experimental/adam.py", line 200, in update_step
        variable.assign_sub((m * alpha) / (tf.sqrt(v) + self.epsilon))

    ValueError: None values not supported.

Found similar issue: https://github.com/deephyper/deephyper/issues/172
In short adding the epsilon=1e-7 to the problem.hyperparameters fixed the issue.


add HEAD method for dry-run

As of turbo 1.5.2, the --dry-run option checks the cache for the artefacts.
This PR lets the worker respond to the HEAD http method

Can you clean up your fomatter's changes?

can you provide a prettier/eslint config for me to run against?

I've added a prettierrc file, and run it against the code.
I've got another branch where I'm using R2 since I have some artefacts larger than 1mb.
Should I open another PR to your main branch, or do you think it's best to have this on another branch?

No just please change only relevant. you can use "Save without formatting" if you are on VSCode.

Consider this as two features then. I’ve added the HEAD method, and initialised prettier. This will avoid this exact issue should anyone else make a PR


Migrate USAF metforcing processing (AGRMET) code into NASA-LIS/LISF

Code compiles without issue with -3 strict checks.  I approve.


theme-obsidian2: 2.6 -> 2.7

Motivation for this change

Update to version 2.7

Changes:

added support for the new nautilus pathbar in GNOME 3.30
button details

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: theme-obsidian2
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: theme-obsidian2
Partial log (click to expand)
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/2pg3knyn713m82c5749991n56svzjhfw-theme-obsidian2-2.7
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/2pg3knyn713m82c5749991n56svzjhfw-theme-obsidian2-2.7
checking for references to /build in /nix/store/2pg3knyn713m82c5749991n56svzjhfw-theme-obsidian2-2.7...
/nix/store/2pg3knyn713m82c5749991n56svzjhfw-theme-obsidian2-2.7

Success on x86_64-linux (full log)
Attempted: theme-obsidian2
Partial log (click to expand)
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/an4i231grdf6bj50gfpwp3s1f5lbladf-theme-obsidian2-2.7
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/an4i231grdf6bj50gfpwp3s1f5lbladf-theme-obsidian2-2.7
checking for references to /build in /nix/store/an4i231grdf6bj50gfpwp3s1f5lbladf-theme-obsidian2-2.7...
/nix/store/an4i231grdf6bj50gfpwp3s1f5lbladf-theme-obsidian2-2.7