pass encoding to napalm_cli for eos

In regards to Issue #50, need to ability to toggle between 'text' and 'json' encoding for EOS platform.

You can just push a commit fixing it. One comment regarding:
encoding: Optional[str] = "text"

Given you are setting a default value I think that encoding: str = "text" should suffice. Optional is mostly to indicate that something is allowed to be None which I don't think it's the case.

Thanks for the wisdom.  And yes, completely concur.  Latest commit reflects your suggestions.

@ktbyers what are your thoughts on this? I am not sure how I feel about napalm_cli taking explicit argument that are platform dependant. Maybe just allow for **kargs and push the responsibility to the user?

Are you thinking something like this?
def napalm_cli(task: Task, commands: List[str], **kwargs) -> Result:
    """
    Run commands on remote devices using napalm

    Arguments:
      commands: commands to execute
      encoding: pass 'text' or 'json' to napalm cli
    Returns:
      Result object with the following attributes set:
        * result (``dict``): result of the commands execution
    """
    device = task.host.get_connection(CONNECTION_NAME, task.nornir.config)

    try:
      result = device.cli(commands, **kwargs)
    except NotImplementedError:
      result = device.cli(commands)

I agree with your approach.  Seeing as how the goal of napalm is to be as agnostic as possible.  I can push another commit, but I'll wait to see what Kirk says.

Yeah, let's definitely use **kwargs.
Format would be:
https://github.com/nornir-automation/nornir_napalm/blob/master/nornir_napalm/plugins/tasks/napalm_get.py#L15

Use the type hints Any (which you will probably need to import as well).
Also make sure your linting is passing.

I decided to not put in error handling (i alluded to this earlier in a comment) as if it was me running this, I would want it to choke so I could correct the problem.  Also, fixed my linting problems (i think :-/).

LGTM, @ktbyers any comments? Otherwise, feel free to merge.


Transform font weight keywords to numeric values

CSS is more compatible with numeric values (100,200,300,...) for font weight than for keywords. However, it is more common in design to use keywords (light, medium, bold)
Add a transformer for font weights to transform these keywords into numeric values:

Value
Common weight name

100
Thin, Hairline

200
extralight, ultralight

300
Light

400
Normal, Regular

500
Medium

600
semibold, demibold

700
Bold

800
extrabold, ultrabold

900
Black (Heavy)

950
extrablack, ultrablack

https://developer.mozilla.org/en-US/docs/Web/CSS/font-weight#common_weight_name_mapping

Done in #52, copying code from sd-transforms, although in current explorations it seems to not be necessary for swift output, so it should not be on that transform-group


Add testamonial

Describe the feature
Add a testamonial we got for this project via our announcement tweet: https://twitter.com/i/timeline

"Beautiful work on sidebase!" by https://github.com/Atinux, CEO of Nuxt

He send this to me via DM, I asked him if we can use it as a testamonial and he said yes ❤
Additional information
No response

@zoey-kaiser do you want to have a look if you can add it to the website quickly? avatar + quote would be nice.

@zoey-kaiser do you want to have a look if you can add it to the website quickly? avatar + quote would be nice.

Sure! The component is already finished but i did not add it as only showing one testimonial looked a bit sad. Do you have a few more for me? I always need:

Name
Avatar
Screenshot of their comment

for name + avatar: see https://github.com/Atinux github profile

Screenshot of their comment

it was a twitter DM, I don't think that it would be super pleasent to put that into the website (:
Here's a second testamonial: https://twitter.com/elbrujolegal/status/1571525564632113152
and I bet if we go through the feed of https://twitter.com/sidebase_io a bit more, we can find more

Okay sounds good! Ill try and get at least a total of 5 and then I will add them. Do you want me to list them here beforehand so you can check them as well?

I don't think we have to wait for 5, even if it's just 2 that'll be a sufficient start.
Just open a PR and ping and then we review them there (:


Add Arthera Testnet for 1.3.0 and 1.4.1

Deployed from:

https://github.com/safe-global/safe-contracts/releases/tag/v1.3.0-libs.0
https://github.com/safe-global/safe-contracts/releases/tag/v1.4.1-build.0

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1205602968885000

@Uxio0 could you help to unblock merging of https://github.com/safe-global/safe-eth-py/pull/682 by reviewing and merging this PR ?
This PR is the result of the PR you reviewed and merged for the singleton deployed on Arthera testnet.


[protobuf] Build error on x64-windows

issue_body.md
patch-x64-windows-2-err.log
patch-x64-windows-0-err.log
copy-tool-dependencies-0-err.log

Sorry, I cannot reproduce this issue locally.

Not sure how to generate a repro. I tried on 2 different computers with a clean install with the same results:
Microsoft Windows [Version 10.0.22631.3155]
(c) Microsoft Corporation. All rights reserved.
M:____tracey_vcpkg>git clone https://github.com/microsoft/vcpkg.git
Cloning into 'vcpkg'...
remote: Enumerating objects: 223875, done.
remote: Counting objects: 100% (7916/7916), done.
remote: Compressing objects: 100% (569/569), done.
remote: Total 223875 (delta 7572), reused 7506 (delta 7347), pack-reused 215959
Receiving objects: 100% (223875/223875), 66.49 MiB | 48.60 MiB/s, done.
Resolving deltas: 100% (147812/147812), done.
Updating files: 100% (11150/11150), done.
M:____tracey_vcpkg>cd vcpkg
M:____tracey_vcpkg\vcpkg>bootstrap-vcpkg.bat
Downloading https://github.com/microsoft/vcpkg-tool/releases/download/2024-02-07/vcpkg.exe -> M:____tracey_vcpkg\vcpkg\vcpkg.exe... done.
Validating signature... done.
vcpkg package management program version 2024-02-07-8a83681f921b10d86ae626fd833c253f4f8c355b
See LICENSE.txt for license information.
Telemetry
vcpkg collects usage data in order to help us improve your experience.
The data collected by Microsoft is anonymous.
You can opt-out of telemetry by re-running the bootstrap-vcpkg script with -disableMetrics,
passing --disable-metrics to vcpkg on the command line,
or by setting the VCPKG_DISABLE_METRICS environment variable.
Read more about vcpkg telemetry at docs/about/privacy.md
M:____tracey_vcpkg\vcpkg>vcpkg install protobuf
warning: In the September 2023 release, the default triplet for vcpkg libraries changed from x86-windows to the detected host triplet (x64-windows). For the old behavior, add --triplet x86-windows . To suppress this message, add --triplet x64-windows .
Computing installation plan...
The following packages will be built and installed:
<EMAIL_ADDRESS>
vcpkg-cmake:x64-windows@2023-05-04
vcpkg-cmake-config:x64-windows@2022-02-06#1
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
A suitable version of 7zip was not found (required v23.1.0) Downloading portable 7zip 23.1.0...
Downloading 7zip...
https://github.com/ip7z/7zip/releases/download/23.01/7z2301-extra.7z->M:____tracey_vcpkg\vcpkg\downloads\7z2301-extra.7z
Downloading https://github.com/ip7z/7zip/releases/download/23.01/7z2301-extra.7z
Extracting 7zip...
Restored 0 package(s) from C:\Users\thank\AppData\Local\vcpkg\archives in 8.94 ms. Use --debug to see more details.
Installing 1/3<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Installing: M:/____tracey/_vcpkg/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake
-- Installing: M:/____tracey/_vcpkg/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake
-- Installing: M:/____tracey/_vcpkg/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake
-- Installing: M:/____tracey/_vcpkg/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg-port-config.cmake
-- Installing: M:/____tracey/_vcpkg/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/copyright
-- Performing post-build validation
Stored binaries in 1 destinations in 43.3 ms.
Elapsed time to handle vcpkg-cmake:x64-windows: 84.1 ms
vcpkg-cmake:x64-windows package ABI: 3b723cf8364ba8901efea4ccd4abc897cbb20221ea1e32cd760377858d0e9503
Installing 2/3<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Installing: M:/____tracey/_vcpkg/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/vcpkg_cmake_config_fixup.cmake
-- Installing: M:/____tracey/_vcpkg/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/vcpkg-port-config.cmake
-- Installing: M:/____tracey/_vcpkg/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/copyright
-- Performing post-build validation
Stored binaries in 1 destinations in 23.5 ms.
Elapsed time to handle vcpkg-cmake-config:x64-windows: 51.2 ms
vcpkg-cmake-config:x64-windows package ABI: c9fc8cfaf5b1a45d0986c49fba97e11b15ddcedc532861a5d3db87e2b80800b5
Installing 3/3<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Downloading https://github.com/protocolbuffers/protobuf/archive/v3.21.12.tar.gz -> protocolbuffers-protobuf-v3.21.12.tar.gz...
-- Extracting source M:/____tracey/_vcpkg/vcpkg/downloads/protocolbuffers-protobuf-v3.21.12.tar.gz
-- Applying patch fix-static-build.patch
-- Applying patch fix-default-proto-file-path.patch
-- Applying patch compile_options.patch
-- Using source at M:/____tracey/_vcpkg/vcpkg/buildtrees/protobuf/src/v3.21.12-fdb7676342.clean
-- Found external ninja('1.11.0').
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
Command failed: C:/Users/thank/.dotnet/tools/pwsh.exe -noprofile -executionpolicy Bypass -nologo -file M:/____tracey/_vcpkg/vcpkg/scripts/buildsystems/msbuild/applocal.ps1 -targetBinary M:/____tracey/_vcpkg/vcpkg/packages/protobuf_x64-windows/tools/protobuf/protoc.exe -installedDir M:/____tracey/_vcpkg/vcpkg/packages/protobuf_x64-windows/bin -verbose
Working Directory: M:/____tracey/_vcpkg/vcpkg
Error code: -532462766
See logs for more information:
M:____tracey_vcpkg\vcpkg\buildtrees\protobuf\copy-tool-dependencies-0-err.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_copy_tool_dependencies.cmake:9 (vcpkg_execute_required_process)
scripts/cmake/vcpkg_copy_tool_dependencies.cmake:41 (z_vcpkg_copy_tool_dependencies_search)
scripts/cmake/vcpkg_copy_tools.cmake:48 (vcpkg_copy_tool_dependencies)
ports/protobuf/portfile.cmake:92 (vcpkg_copy_tools)
scripts/ports.cmake:172 (include)
error: building protobuf:x64-windows failed with: BUILD_FAILED
Elapsed time to handle protobuf:x64-windows: 27 s
Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+protobuf
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?title=[protobuf]+Build+error+on+x64-windows&body=Copy+issue+body+from+M%3A%2F____tracey%2F_vcpkg%2Fvcpkg%2Finstalled%2Fvcpkg%2Fissue_body.md
M:____tracey_vcpkg\vcpkg>

Please provide M:____tracey_vcpkg\vcpkg\buildtrees\protobuf\copy-tool-dependencies-0-err.log log.

Here it is
copy-tool-dependencies-0-err.log

You remove the special characters from the path, use only alphabetic file paths, and then reinstall.

I don't understand what special characters to remove - if you mean the _ (underscore), I tried installing on M:\vcpkg with the same results:
M:>git clone https://github.com/microsoft/vcpkg.git
Cloning into 'vcpkg'...
remote: Enumerating objects: 223909, done.
remote: Counting objects: 100% (11241/11241), done.
remote: Compressing objects: 100% (606/606), done.
remote: Total 223909 (delta 10874), reused 10803 (delta 10635), pack-reused 212668
Receiving objects: 100% (223909/223909), 68.38 MiB | 49.38 MiB/s, done.
Resolving deltas: 100% (148222/148222), done.
Updating files: 100% (11147/11147), done.
M:>cd vcpkg
M:\vcpkg>vcpkg install protobuf
'vcpkg' is not recognized as an internal or external command,
operable program or batch file.
M:\vcpkg>bootstrap-vcpkg.bat
Downloading https://github.com/microsoft/vcpkg-tool/releases/download/2024-02-07/vcpkg.exe -> M:\vcpkg\vcpkg.exe... done.
Validating signature... done.
vcpkg package management program version 2024-02-07-8a83681f921b10d86ae626fd833c253f4f8c355b
See LICENSE.txt for license information.
Telemetry
vcpkg collects usage data in order to help us improve your experience.
The data collected by Microsoft is anonymous.
You can opt-out of telemetry by re-running the bootstrap-vcpkg script with -disableMetrics,
passing --disable-metrics to vcpkg on the command line,
or by setting the VCPKG_DISABLE_METRICS environment variable.
Read more about vcpkg telemetry at docs/about/privacy.md
M:\vcpkg>vcpkg install protobuf
warning: In the September 2023 release, the default triplet for vcpkg libraries changed from x86-windows to the detected host triplet (x64-windows). For the old behavior, add --triplet x86-windows . To suppress this message, add --triplet x64-windows .
Computing installation plan...
The following packages will be built and installed:
<EMAIL_ADDRESS>
vcpkg-cmake:x64-windows@2023-05-04
vcpkg-cmake-config:x64-windows@2022-02-06#1
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
A suitable version of 7zip was not found (required v23.1.0) Downloading portable 7zip 23.1.0...
Downloading 7zip...
https://github.com/ip7z/7zip/releases/download/23.01/7z2301-extra.7z->M:\vcpkg\downloads\7z2301-extra.7z
Downloading https://github.com/ip7z/7zip/releases/download/23.01/7z2301-extra.7z
Extracting 7zip...
Restored 0 package(s) from C:\Users\thank\AppData\Local\vcpkg\archives in 9.5 ms. Use --debug to see more details.
Installing 1/3<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Installing: M:/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake
-- Installing: M:/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake
-- Installing: M:/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake
-- Installing: M:/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg-port-config.cmake
-- Installing: M:/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/copyright
-- Performing post-build validation
Stored binaries in 1 destinations in 44 ms.
Elapsed time to handle vcpkg-cmake:x64-windows: 84.8 ms
vcpkg-cmake:x64-windows package ABI: 53b2f2e5c54b4a3691b5fcae9da6474ed8537e3e3cec93a701b71da8056da5de
Installing 2/3<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Installing: M:/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/vcpkg_cmake_config_fixup.cmake
-- Installing: M:/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/vcpkg-port-config.cmake
-- Installing: M:/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/copyright
-- Performing post-build validation
Stored binaries in 1 destinations in 23.2 ms.
Elapsed time to handle vcpkg-cmake-config:x64-windows: 50.7 ms
vcpkg-cmake-config:x64-windows package ABI:<PHONE_NUMBER>ecc555470e3ced3820e332f4e98810638796f99c0703164b155906
Installing 3/3<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Downloading https://github.com/protocolbuffers/protobuf/archive/v3.21.12.tar.gz -> protocolbuffers-protobuf-v3.21.12.tar.gz...
-- Extracting source M:/vcpkg/downloads/protocolbuffers-protobuf-v3.21.12.tar.gz
-- Applying patch fix-static-build.patch
-- Applying patch fix-default-proto-file-path.patch
-- Applying patch compile_options.patch
-- Using source at M:/vcpkg/buildtrees/protobuf/src/v3.21.12-fdb7676342.clean
-- Found external ninja('1.11.0').
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
Command failed: C:/Users/thank/.dotnet/tools/pwsh.exe -noprofile -executionpolicy Bypass -nologo -file M:/vcpkg/scripts/buildsystems/msbuild/applocal.ps1 -targetBinary M:/vcpkg/packages/protobuf_x64-windows/tools/protobuf/protoc.exe -installedDir M:/vcpkg/packages/protobuf_x64-windows/bin -verbose
Working Directory: M:/vcpkg
Error code: -532462766
See logs for more information:
M:\vcpkg\buildtrees\protobuf\copy-tool-dependencies-0-err.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_copy_tool_dependencies.cmake:9 (vcpkg_execute_required_process)
scripts/cmake/vcpkg_copy_tool_dependencies.cmake:41 (z_vcpkg_copy_tool_dependencies_search)
scripts/cmake/vcpkg_copy_tools.cmake:48 (vcpkg_copy_tool_dependencies)
ports/protobuf/portfile.cmake:92 (vcpkg_copy_tools)
scripts/ports.cmake:172 (include)
error: building protobuf:x64-windows failed with: BUILD_FAILED
Elapsed time to handle protobuf:x64-windows: 28 s
Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+protobuf
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?title=[protobuf]+Build+error+o
n+x64-windows&body=Copy+issue+body+from+M%3A%2Fvcpkg%2Finstalled%2Fvcpkg%2Fissue_body.md
copy-tool-dependencies-0-err.log
error-logs-x64-windows.txt

Unhandled exception. System.ComponentModel.Win32Exception (2): An error occurred trying to start process 'dotnet' with working directory 'M:\vcpkg'. The system cannot find the file specified.
   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
   at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
   at Microsoft.PowerShell.GlobalTool.Shim.EntryPoint.Main(String[] args)

Judging from the error code: -532462766 and the error log, this should be a compiler error, not a vcpkg error.

We are using the latest release of MSVC (17.9.2). Any suggestions? Maybe you can file a bug report for MSVC. Do you not test with MSVC?

I downloaded a win11 virtual machine, added the C++ package to VS2022, and got protobuf working for the latest version of the compiler (17.9.3). So this is not a compiler problem. It is not a protobuf problem either, since this also happens with other packages.

Any chance to use a different powershell executable?
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
    Command failed: C:/Users/thank/.dotnet/tools/pwsh.exe -noprofile -executionpolicy Bypass -nologo -file M:/____tracey/_vcpkg/vcpkg/scripts/buildsystems/msbuild/applocal.ps1 -targetBinary M:/____tracey/_vcpkg/vcpkg/packages/protobuf_x64-windows/tools/protobuf/protoc.exe -installedDir M:/____tracey/_vcpkg/vcpkg/packages/protobuf_x64-windows/bin -verbose

This is not the location where I would expect find pwsh.exe on my computer.
Unhandled exception. System.ComponentModel.Win32Exception (2): An error occurred trying to start process 'dotnet' with working directory 'M:\____tracey\_vcpkg\vcpkg'. The system cannot find the file specified.
   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
   at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
   at Microsoft.PowerShell.GlobalTool.Shim.EntryPoint.Main(String[] args)

'dotnet' seems to match your location.
But this is not the kind of stack trace which I would expect from pwsh.exe.

Thanks for the help. Removing the folder 'C:/Users/thank/.dotnet' seems to have fixed the problem. I have no idea how this was installed on my computer (actually 2 computers).

Thank you for your reply. If your problem is solved, I will close this issue. If you have new questions, please submit a new issue.


Implement line simplification

I am not seeing significant performance increase out of it. I am also not sure it is working for the large width roads. I know it is working with the small roads, paths, etc.. But the large roads are not really changing even if I have it simplify at like 5000. It turns everything into single lines (as expected) except for the roads we are actually trying to fix. Maybe my implementation is wrong. Uses simplify_dart plugin for simplification.
You can check it out on my fork of your project. This does not need to be merged (or should be merged in a separate branch) just a proof of concept.

Thanks, thats interesting. Hopefully I'll get a chance to play with it over
the next day or two.
On Tue, May 11, 2021 at 11:15 PM Moo @.***> wrote:

[image: Uploading Screenshot_20210511-171229.jpg…]
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ibrierley/flutter_map_vector_tiles/pull/19#issuecomment-839226999,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5YN5KK6F3NAL5F4AW5CXLTNGUADANCNFSM44WXWFTA
.

Was thinking a bit more about this. I feel like the simplifier as a concept would actually sit really well in flutter_map as well, itself as an accessible method to convert polylines etc. One of the things we may find, is people want to add their own custom lines etc which also impacts performance, and this could be really useful there as well and not just vector tiles.
Do you think it could sit there and this plugin access it there, or is it too far removed/separate and would be more optimal as it is (even if it doesn't do much atm, we may find other vector tile providers its more useful)? I think it would be good as an option for sure, but I've needed it in the past for polylines also (basically simplifying gpx routes/points), and not sure if it could work for both (paths in the vector plugin, and polylines drawn in a separate layer)?

Revisited this, and created a branch at https://github.com/ibrierley/flutter_map_vector_tiles/tree/line_simplify with the changes, as there's been quite a few other changes since.
I think I'm with your original comments, I can't really see the performance improvements, and it makes the lines look clunkier, so going to keep this on the backburner for the time. Feels like it could be useful somehow, but not just at the moment.
I do really appreciate the effort you put into it though, if nothing else it's given a possible option some testing which is really useful.
I'm going to close for the moment, but may revisit again :D


Some copy editing.

E.g. ref. https://en.wiktionary.org/wiki/mind-blowing#Adjective and http://en.wikipedia.org/wiki/Internet.

Can this be accepted?


Customizing the stat command

I am new to yosys. However, I want to find out how to customize the stat command to print only the gate counts to a specified file. Or if there is already a way to do that, can be directed on how to achieve this.

You can use tee to write the output from stat to a file. It doesn't support printing just gate counts.

@daveshah1 I tried stat tee -a info.txt, and it printed something other than the stat information.

tee's help is:
yosys> help tee

    tee [-q] [-o logfile|-a logfile] cmd

Execute the specified command, optionally writing the commands output to the
specified logfile(s).

    -q
        Do not print output to the normal destination (console and/or log file).

    -o logfile
        Write output to this file, truncate if exists.

    -a logfile
        Write output to this file, append if exists.

    +INT, -INT
        Add/subtract INT from the -v setting for this command.

Thus you would want: tee -a info.txt stat

@eddiehung Thanks


onAuthStateChange fires twice. Is this normal?

I've read through the docs and it's not clear.
For some reason, onAuthStateChange fires twice! This is problematic because once a user is authenticated, I want to fetch their profile (based on their user id) with the SIGNED_IN action. Since it fires twice, it will make a fetch twice and update my Pinia store twice.
Is this normal? I suspected it may be firing on SSR and then client but both appear to be on the client side, so I'm not sure why this happens.
I'm looking for some clarification on the matter, whether this is expected and what can be done about a situation like this?

I'm using the composables in the most simple way possible:
const supabaseAuthClient = useSupabaseAuthClient()

supabaseAuthClient.auth.onAuthStateChange(async (action, session) => {
  console.log(action, session) // logs twice!
})

@michaelpumo I have tested this in my project but it fires only once, Maybe if you share more details, we can help.


BINANCE - SMART SWITCH?

great work and communication, thanks for that.
forgot to ask - is binance supporting a smart switching?
doesnt seem to be indicated in the documentation whether yes or no, just absent from config...
thanks

smart switching is working from 1h to 15m in Binance also

Yes, all the features for Coinbase Pro are replicated for Binance.


[VS] Guests can't add/delete files in "Solution View"

With the newest updates, there are no options in the right click menu for live share participants in solution explorer.

This has been resolved in the latest Visual Studio extension update 👍


Add ndt7-client

This change adds the ndt7-client to the minimal script-exporter support image.
This makes it possible to have monitoring and automation targeting the autonode servers in sandbox and staging (and possibly production).

This change is

The build succeeds and the basic functionality is working in sandbox.


Redshift: fix query to retrieve table schemas

also removed a noisy console warning
and fixed all the lint warnings

💃  Looks great, thanks for doing this!


[Feature] Git Based Resolver

The way we use VCL and manage dependencies at $company is by having them versioned in a separate git repository and I would like to build a git file resolver that allows you to resolve paths based on path:version and then use git show $REV:$FILE to get the file rather than just reading it from disk
So the proposed changes are:

GitFileResolver that implements the Resolver interface
func NewGitFileResolvers(main string, c *Config) ([]Resolver, error) and switch between the two implementations based on the command line flag

Addition of two command line flags:

-repo              : Path to the git repo that contains the modules
-git               : Use git to read versioned files that in the form of path/to/file/:version

WDYT?

I have this implemented just wanted to push it the upstream rather than having this patch on our fork.
I believe this might be useful for other people who version their VCL dependencies


Terminal accessibility help should have top/bottom padding

Instead of setting the height explicitly, try something like this?
max-width: 80%;
overflow: scroll;
transform: translate(-50%, -50%);
left: 50%;
top: 50%;

/duplicate https://github.com/microsoft/vscode/issues/172344


Simplify writing to host filesystem + export container to OCI archive

This is a proposal for improving the Graphql API, following up on this discussion. It only modifies graphql schema files without consideration for implementation.
Write to host filesystem

Proposal for a simpler and more dev-friendly API
Also allow writing a file to the host filesystem

Export container to OCI archive

Short-term implementation built on simpler host filesystem API
Longer-term implementation to make things even simpler

@vito do you like the writing part though? :)

I believe this PR has served its purpose as a conversation starter, closing in favor of @vito 's actual implementation work.


[DOCFIX] Update cn version of Upgrade doc

What changes are proposed in this pull request?
Update cn version of Upgrade doc.
Why are the changes needed?
There is no corresponding Chinese documentation for upgrade.
Does this PR introduce any user facing changes?
More Chinese users can access Alluxio documentation more easily.

@voddle thanks for updating the doc

alluxio-bot, merge this please


TP: Generate ISO view won't resolve

I'm submitting a ...

[x] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[x] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:
Clicking on the Generate ISO menu item does nothing.

Expected / new behavior:
Clicking on the Generate ISO menu item should display the form required to generate an ISO.
Minimal reproduction of the problem with instructions:

Click on the Generate ISO menu item

This is due to a leftover restangular dependency. Easy fix. I'll take it.


"Cannot read property 'getId' of undefined" when clicking on a pod

Describe the bug
When clicking on a pod of our production cluster, the left pane starts to show up and it's immediately replaced by the error screen showing the error above. It does not happen with pods of our dev cluster
To Reproduce
Steps to reproduce the behavior:

Go to Pods
Click on a pod
See error

Expected behavior
I expected to see the pod details
Environment (please complete the following information):

Lens Version: 4.1.3
OS: ubuntu
Installation method (e.g. snap or AppImage in Linux): DEB

Logs:
When you run the application executable from command line you will see some logging output. Please paste them here:
This what I get from the terminal
warn: PrometheusLens: failed to list services: services "prometheus" not found
warn: PrometheusLens: failed to list services: services "prometheus" not found
warn: PrometheusLens: failed to list services: services "prometheus-server" not found
warn: PrometheusLens: failed to list services: services "prometheus-server" not found
warn: PrometheusLens: failed to list services: services "prometheus" not found
warn: PrometheusLens: failed to list services: services "prometheus" not found
warn: PrometheusLens: failed to list services: services "prometheus-server" not found
warn: PrometheusLens: failed to list services: services "prometheus-server" not found

This is from the error page
Component stack:

    at QI (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:8013519)
    at span
    at div
    at jd (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:7756151)
    at div
    at kI (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:8016142)
    at Details
    at div
    at div
    at div
    at Bp (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:7728188)
    at Qd (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:7753647)
    at sy (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:7874585)
    at pa (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:7560311)
    at t (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:191498)
    at eT (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:8261853)

Error stack:

TypeError: Cannot read property 'getId' of undefined
    at http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:8013840
    at http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:42956
    at Array.map (<anonymous>)
    at Proxy.forEach.Array.<computed>.pr.<computed> (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:42931)
    at QI.render (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:8013805)
    at Ce (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:13973)
    at http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:7444538
    at ce (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:12079)
    at e.track (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:23443)
    at QI.s [as render] (http://249ff66b-b091-40f1-804b-a670a6c70fd8.localhost:40019/build/Lens.js:2:7444486)

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: DATA
    server: https://IP
  name: dev
- cluster:
    certificate-authority-data: DATA
    server: https://IP
  name: prod
contexts:
- context:
    cluster: dev
    user: dev
  name: dev
- context:
    cluster: prod
    user: prod
  name: prod
current-context: prod
kind: Config
preferences: {}
users:
- name: dev
  user:
    auth-provider:
      config:
        access-token: TOKEN
        cmd-args: config config-helper --format=json
        cmd-path: /usr/lib/google-cloud-sdk/bin/gcloud
        expiry: "2021-02-25T09:12:24Z"
        expiry-key: '{.credential.token_expiry}'
        token-key: '{.credential.access_token}'
      name: gcp
- name: prod
  user:
    auth-provider:
      config:
        access-token: TOKEN
        cmd-args: config config-helper --format=json
        cmd-path: /usr/lib/google-cloud-sdk/bin/gcloud
        expiry: "2021-02-25T09:12:24Z"
        expiry-key: '{.credential.token_expiry}'
        token-key: '{.credential.access_token}'
      name: gcp

Additional context
Add any other context about the problem here.

I'm facing the exact same issue, also 4.1.3 on ubuntu
Some things to note:

I am testing with a read-only role (no issues as admin)
I do seem to see the pod details open before the error appears
No issues getting pod details with kubectl describe pod ....

I wanted to start via cli but when I do that I seem to have 4.1.0, anyway the issue doesn't occur there.

+1
4.1.3 on ubuntu

I'm facing this same issue, Lens 4.1.3 / MacOS Big Sur 11.2.2.
This is under a limited read-only access only - no issue with expanded access to the namespace.

Facing the same issue on Kali Linux 2020.4
Haven't tested on full access account.


Having trouble from a subdomain

I've been fighting with this for days and have tried a ton of combinations of plugins without success. Currently I only have the ActivityPub plugin installed. When I try a WebFinger request I get:
21:25:50 Looking up WebFinger data for<EMAIL_ADDRESS>21:25:50 GET https://blog.bluestarcreations.net/.well-known/webfinger?resource=acct%3Abluestarultor%40blog.bluestarcreations.net
21:25:51 Error getting JRD: 400 Bad Request
Same when I try my author page. It's weird because everything seems to know the site exists, but nothing can actually talk to it. When I search in Mastodon I get a simple "0 results" where searching for my base domain returns an error as it would for any missing domain.
I'm on an Apache server with AwardSpace and while I can't get this thing working, I think over time other plugins have worked somehow because I can see traffic spikes over the past few days. Darned if I know from WHAT, but it doesn't seem to be from this plugin. Also, installing the standalone WebFinger plugin gives a critical site error that ActivityPub can't talk to it. I feel like there's some permissions issue somewhere, but I'm not smart enough with *nix to figure out what.
I'm hoping that there's just something really simple I'm missing with configuration or some magic plugin I need to fix everything.  Help a Windows user out?

I think there is something broken with your rewrite rule, if you call the endpoint directly, it seems to return the proper result: https://blog.bluestarcreations.net/wp-json/activitypub/1.0/webfinger?resource=acct%3Abluestarultor%40blog.bluestarcreations.net
Maybe regenerate the rewrite rules, by pressing the save button on the permalinks page.

I also tested your profile page, this seem to work properly.

This is all I have in my .htaccess after re-saving:
BEGIN WordPress
The directives (lines) between "BEGIN WordPress" and "END WordPress" are
dynamically generated, and should only be modified via WordPress filters.
Any changes to the directives between these markers will be overwritten.

RewriteEngine On
RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}]
RewriteBase /
RewriteRule ^index\.php$ - [L]
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule . /index.php [L]

END WordPress
#RewriteCond %{HTTPS} off
#RewriteRule (.*) https://%{HTTP_HOST}%{REQUEST_URI}

Maybe try to install the WebFinger Plugin: https://wordpress.org/plugins/webfinger/

Now I get a critical site issue:
WebFinger endpoint is not accessible | ActivityPub
Your WebFinger endpoint<EMAIL_ADDRESS>does not return valid JSON for application/jrd+json.

No, the endpoint is working, but something seems to block the resource paramater at all, but only when using the .well-known URL.

This URL<EMAIL_ADDRESS>is a rewrite to that URL https://blog.bluestarcreations.net/wp-json/activitypub/1.0/webfinger?resource=acct%3Abluestarultor%40blog.bluestarcreations.net
I have no idea, why the first one does not forward the resource param.

Is there a rule I can manually enter for that?  Or do you think it's a file or folder permissions issue?  I can check to see if maybe Execute is disabled on the files if you think that might help.

@mediaformat do you have an idea?

I tried creating one early on in troubleshooting, but deleted it after seeing that would cause issues in other threads here.  There is not one on the server at this time.

@bluestarultor could you try to comment out this line in the .htaccess RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}]

Sorry for the delay.  I tried it and it didn't work.  Worse yet, that gets restored any time I re-save my Permalinks.

Hey, sorry.  Since I couldn't get this working I just went with a different extension that auto-posts to an account I otherwise wasn't using for much.  I would have liked this to work instead, but I have a feeling my host is probably just blocking something and I'm just not equipped to figure out what if anything I can do to verify or get around it, but thank you for your efforts and for making this to begin with.

I don't suppose there's a way we can open this back up, can we?  I still would prefer to get this working over using the auto-poster, if only because I'm getting a reminder of why I originally abandoned the server that's attached to.

I'm currently sitting at this:
<IfModule mod_rewrite.c>
RewriteEngine On
RewriteBase /
RewriteRule ^\/\.well-known.*$ \/profile\/myProfile.json
</IfModule>

# BEGIN WordPress
# The directives (lines) between "BEGIN WordPress" and "END WordPress" are
# dynamically generated, and should only be modified via WordPress filters.
# Any changes to the directives between these markers will be overwritten.
<IfModule mod_rewrite.c>
RewriteEngine On
RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}]
RewriteBase /
RewriteRule ^index\.php$ - [L]
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule . /index.php [L]
</IfModule>

# END WordPress

#RewriteCond %{HTTPS} off
#RewriteRule (.*) https://%{HTTP_HOST}%{REQUEST_URI}

Is there any chance this is just because my Wordpress is on too high a version?  It updated itself to 6.0, but even before that it was on an untested version.  If that's the case, any idea what the timeline for 6.0 support looks like?

@bluestarultor My guess is that the shared hosting provider messed with the .well-known directory in the main config (which you are unable to edit) in order to support their https renewal infrastructure.
Unfortunately, you probably can't fix this without changing providers.


Feature Request: Support URL

Invalid prop: type check failed for prop "source". Expected Object, String, Uint8Array, got URL

Hi @MartinX3,
This was addressed in 8060f0379ce22b4813bfdfb18fa7be86320a1698, and will be included in the next release.
Thanks for noticing!


[WIP] XGBoost Titanic tutorial

This branch also contains (most) changes from #132, so the diff is larger than it's really is.
The notebook with the tutorial itself is here: https://github.com/TeamHG-Memex/eli5/blob/xgboost-tutorial-2/notebooks/xgboost-titanic.ipynb.
I also added the csv that I downloaded from kaggle (train set) - there is no clear indication whether it's ok to re-distribute it (although there are plenty of copies of it on github), but it's also available here https://vincentarelbundock.github.io/Rdatasets/datasets.html, and the data itself comes from public sources https://vincentarelbundock.github.io/Rdatasets/doc/datasets/Titanic.html so it seems ok to include it for convenience.
The tutorial is rather light, the goal was just to show how to use xgboost, sklearn and eli5 together. @kmike do you have ideas how it could be extended or changed?

The tutorial looks great!
It seems there are still unrelated commits and changes from #132 (it is merged now).

I think that's fine to bundle csv file. It won't be obvious that the file is bundled when reading the tutorial from the docs though - we don't say that users should clone the repo and run the notebook from the checkout. Maybe it worths adding a note ("we've downloaded the data file from ... ").

Could you please also add a link to this tutorial to http://eli5.readthedocs.io/en/latest/libraries/xgboost.html?

Current coverage is 97.34% (diff: 100%)

Merging #136 into master will increase coverage by <.01%

@@             master       #136   diff @@
==========================================
  Files            37         37
  Lines          2217       2219     +2
  Methods           0          0
  Messages          0          0
  Branches        430        430
==========================================
+ Hits           2158       2160     +2
  Misses           29         29
  Partials         30         30

Diff Coverage
File Path

•••••••••• 100%
eli5/formatters/text.py

•••••••••• 100%
eli5/formatters/utils.py

•••••••••• 100%
eli5/formatters/html.py

Powered by Codecov. Last update 0f3ef2d...ab66a13

I added the link to tutorial from the xgboost docs and removed extra diff. Apart from tutorial itself, it contains the change to how values are formatted: they don't have a mandatory "+", but weights still do.
Now the only remaining issue is treating 0 as missing values - I'll think how to best solve it here.

Found that I misunderstood what missing attribute of XBGClassifier means: it turns out it's only used when converting dense input into internal XGBoost sparse matrix, but not if input is already sparse: https://github.com/dmlc/xgboost/issues/341#issuecomment-141755530
And also it turns out I don't understand yet how to combine sparse features with dense features that can have zeros using scipy sparse matrixes and sklearn.

I wrote a separate issue for missing values in xgboost: #137 - I think it's a blocker for the tutorial in it's current form, but not a 0.3 blocker I think.

@kmike I merged #142 into this branch and updated tutorial with correct missing handling: https://github.com/TeamHG-Memex/eli5/blob/xgboost-tutorial-2/notebooks/xgboost-titanic.ipynb

@kmike so hopefully now it's complete and correct, can you please have another look at it? :)

This looks great @lopuhin!


What does 'ref_from_car' mean?

code:
https://github.com/open-mmlab/OpenPCDet/blob/master/pcdet/datasets/nuscenes/nuscenes_utils.py#L275

I'm not sure if it means converting from the ego to the lidar coordinate system?

It is where the ego pose is in the lidar coordinate system. It is used here: https://github.com/open-mmlab/OpenPCDet/blob/9c8e583e28e45582565175f848b7a2538eb6acf6/pcdet/datasets/nuscenes/nuscenes_utils.py#L327
where we go from the lidar_frame to the car frame to the global frame then go from global frame to car frame to key lidar frame(ref_frame). I think it is to accumulate sweeps in the key lidar frame.


[Material] fix audio player on smaller phones

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #android-bugs.

It gets smushed

Completed in full rewrite


add -partition option for unhfs

Hey Tessa, would you be willing to consider adding the -partition option for unhfs so that a user could extract files from a specific partition, similar to the change you made to allow a user to specify the sector offset with tsk_recover? Currently, w/ the hfs option, brunnhilde only extracts files from the first partition. Thanks for considering it! --Brian

Hi Brian! Sure, I'd be happy to add the -partion option for unhfs to Brunnhilde. I should be able to carve out a bit of time on Friday to take a look at this. Thanks for your patience!

Hi @bdietz,  just a quick note to say that I just released Brunnhilde 1.9.0, with new --hfs_partition and --hfs_fsroot options for unhfs. More details are available here: https://github.com/tw4l/brunnhilde/releases/tag/v1.9.0. You should be able to update with sudo pip install --upgrade brunnhilde.
Thanks again for the suggestion, and for your patience! This release ended up being a bit bigger in scope than I had planned at first and was definitely delayed by COVID malaise, so it took me a bit longer than I anticipated. I hope you find the new features useful!


Add mechanism to supply additional options to node.js process

Some Node.js command line options such as --experimental-permission cannot be set via the NODE_OPTIONS environment variable. I would like to be able to set such options when running bytenode.
For example, by using an environment variable:
export BYTENODE_NODE_OPTIONS="--experimental-permission --allow-fs-write=/foo --allow-fs-read=/bar"
bytenode foo.jsc

Which might be implemented roughly like this in cli.js:
diff --git a/lib/cli.js b/lib/cli.js
index 1e556f8..f4b898a 100755
--- a/lib/cli.js
+++ b/lib/cli.js
@@ -191,10 +191,16 @@ if (program.flags.includes('--compile')) {
   }
 } else {
   try {
-    const result = spawnSync(program.nodeBin, [
+    let nodeArgs = []
+    if (process.env.BYTENODE_NODE_OPTIONS) {
+      nodeArgs = process.env.BYTENODE_NODE_OPTIONS.split(' ')
+    }
+    const allArgs = nodeArgs.concat([
       '-r',
       path.resolve(__dirname, 'index.js')
-    ].concat(args), {
+    ])
+    .concat(args)
+    const result = spawnSync(program.nodeBin, allArgs, {
       stdio: 'inherit'
     });
     process.exit(result.status);

I can open a more thorough PR if this approach seems reasonable.

Why not just set it directly from the bytenode cli? It should be possible, any flags unused by bytenode are delegated to node itself.

@OsamaAbbas that would be the ideal scenario. I was having issues with that but perhaps I was holding it wrong, will try again this weekend and report back. Thanks

@OsamaAbbas indeed this seems to work as you described:
bytenode [node args] [script] [script args]

e.g.
bytenode --experimental-permission --allow-fs-write=/foo --allow-fs-read=/bar index.jsc arg1 --foo

Sorry for the noise.


Improvement: Issue and PR cleanup action

Hi,
I have the feeling that the Issues and PRs are so chaotic that participation is getting artificially harder.
Because there are so many open issues, a user that comes here for an issue is unable to see if his issue is being worked on or not. Because there are so many open PRs, it's hard to understand if and when a requested bridge will get integrated at all. And in addition: Such a high amount of open topics signals to users that the application may not be up to date at all (2 year old open PRs..)
In an attempt to make support easier and PR more organized, I think we should go through all of the issues and PRs and decide on what to do. I have the feeling that there are many bridges that are still included in rss-bridge that dont work anymore and have been known for a long time. Either fix the bridge or throw it out. Having an open issue for 2 years without anyone doing anything on it doesn't help the tool, the user or the bridge itself. The deleted bridges will still be available and visible in the commit so if anyone wants to rebuild it, they have something to start from.
I would maybe tag a few of the regular contributors to ask them to go through the issues and PRs and comment with a note on their opinion (issue: close, request for help ; PRs: merge, fix or close)
The tool is great and I would love to have more people providing bridges to it but if a new contributor looks at the backlog, he might just go away with the idea that the application might be abandoned.
I dont want to link all the issues to this Issue, so I would do this:

Tag regular contributors in this issue to ask for help with the Issues and PRs
rss-bridge team either tests the comment themselves or trust in the comment from the contributor and acts on the comment.

We could define a time-frame to do a kind of hackathon where people say "ok, I will spend some time on the next weekend to help out with stuff on rss-bridge" so that multiple people can work on a bridge or so.
I'm open to suggestions, I would just like to have this done/do this in some capacity.

This is a great idea. I've gone through and found 20 issues and 2 pull requests that can be closed https://gist.github.com/VerifiedJoseph/538d693834cee8a24c136380d875b2b7

I have the feeling that there are many bridges that are still included in rss-bridge that dont work anymore and have been known for a long time.

Ideally, we need a tool that runs regularly to test each bridge and report broken ones so they can be fixed or removed. I've written a very basic script for checking each bridge's URI.

That's awesome!
I think if we do one big cleanup, it would make sense to prepare the whole testing scenario first and then make it mandatory (and rework all older bridges) to include a test section like in #2108 .
Then, this test could be run regularely and even open up new issues automatically if you do it right :)

In general, when I have time I spend it to review latest PRs and issues, and searching for outdated/invalid issues is less important task for me personally. I agree, that those issues should be closed.

Such a high amount of open topics signals to users that the application may not be up to date at all (2 year old open PRs..)

https://github.com/ytdl-org/youtube-dl has way more issues and open PRs. It is up to date.

This is a great idea. I've gone through and found 20 issues and 2 pull requests that can be closed https://gist.github.com/VerifiedJoseph/538d693834cee8a24c136380d875b2b7

Thank you for your work, Joseph! I visited that gist and closed some issues and PRs

We are basically in the process of it right now and will ping regulars in specific PRs and issues that might need their help


[DOC][ADDON] - Cluster Autoscaler

Under deploy a sample app:
Command to expose sample deployment is wrong; it's missing deployment. It should be kubectl expose deployment php-apache --port 80.

Under verifying section, the command to access cluster autoscaler logs should be:
kubectl -n kube-system logs -f deployment/blueprints-addon-cluster-autoscaler-aws-cluster-autoscaler


module import error

% go mod tidy
go: finding module for package github.com/crowdstrike/go-metrics-sliding-window
swm imports
        github.com/crowdstrike/go-metrics-sliding-window: module github.com/crowdstrike/go-metrics-sliding-window@latest found (v0.1.0), but does not contain package github.com/crowdstrike/go-metrics-sliding-window

I think you want to move the code to the main folder and not have it in slidingwindow directory for simpler usage as a library.

You know I had it that way originally then moved it into a directory because I thought it was the more canonical way, but it definitely makes the import path messy and this is another issue. Let me think on it.

Addressed in #7


Set default resource source based on environment

Currently, resources will alway default to being read from Google Cloud Public Datasets.
https://github.com/broadinstitute/gnomad_methods/blob/748c5e4c1d066bc71507ad9c8017454c43ce8d09/gnomad/resources/config.py#L14-L16
Preferably, the default would be based on the environment, such that jobs running on EMR automatically default to reading resources from the Registry of Open Data on AWS and jobs running on HDInsight default to reading from Azure Open Datasets.
Maybe Hail already has something for identifying the environment (at least in clusters started with hailctl)?

I can ask Dan if this is possible

If nothing else, we can read the default source from an environment variable that can be set by either the user or hailctl. I was curious if Hail had anything to "guess" which cloud platform code was running on.

Agreed, it would be nice if this could be automatic

We did not have such a function, but we do have some heuristics as well as some environment variables that we set in all hailctl invocations. I've codified this as a new function in hail.utils: https://github.com/hail-is/hail/pull/11230. If you have an EMR cluster running and can list the environment variables available there, I'm happy to update the heuristics function to include that too. I'm also happy to receive feedback on the returned strings (currently: "dataproc" and "hdinsight").

Added the option to configure the default source with an environment variable in #435. Once hail-is/hail#11230 is merged and that API finalized, we can update get_default_public_resource_source to also use Hail's guess for the cloud provider.


Issue with /docs/ (Installation & upgrade) for 1.6 and etcd3.

This is a...

[X] Feature Request
[ ] Bug Report

Problem:
Outline the upgrade and data migration process for etcd3 in 1.6.
Proposed Solution:
Write + review the docs.
Page to Update:
Installation and upgrades.
http://kubernetes.io/...
/cc @kubernetes/sig-scalability-misc @justinsb
xref: https://github.com/kubernetes/kubernetes/pull/36229

@mml @wojtek-t

/cc @mml @hongchaodeng @kubernetes/sig-cluster-ops-misc

@timothysc
We can own and update https://kubernetes.io/docs/admin/etcd/ together.
Sorry I missed the sig scale this morning. We probably need to also discuss deprecation of etcd2.
@xiang90 @calebamiles

This should have 1.6 milestone (though there doesn't seem to be 1.6 milestone in this repo).

@xiang90 Could you paste a link to your documentation here?

We are working on the generic etcdv3 specific doc here:
https://docs.google.com/document/d/16ES7N51Xj8r1P5ITan3gPRvwMzoavvX9TqoN8IpEOU8/edit?usp=sharing
Any feedback is welcomed. We hope to get this doc finalized early Feb.
Thanks.

ping @kubernetes/sig-docs-maintainers etcdv3 docs is waiting for feedback.

@wojtek-t @mml So I'm flummoxed on where to update docs here...
A bunch of docs need to be cleaned up, either they are owned by distros or are not directly relevant.
Then the main docs on cluster upgrade only really pertains towards the salt/GCE setup and don't even reference the details under the hood.
Any advise on where we should update?   New upgrade page?
/cc @hongchaodeng @justinsb

I think we need a separate/new page for the process of upgrade itself.
But if you are aware of any docs that might no longer be valid, can you post the links to them here?


Ray doesn't work in Lumen

Describe the bug
Ray doesn't work in Lumen the calls do not appear in the desktop app.
Versions
<IP_ADDRESS>

spatie/ray package version: 1.5.5
spatie/laravel-ray package version (if applicable):  1.3.4

PHP version: 8.0
Lumen version: 8
To Reproduce

Create a new lumen project
Register Ray service provide in app.php:
$app->register(\Spatie\LaravelRay\RayServiceProvider::class);
Add a ray log in web.php

$router->get('/', function () use ($router) {
    ray('My first ray call');

    return $router->app->version();
});

Expected behavior
A message in the ray desktop app.

OS: Windows with Docker
I've also published ray.php in the root of the project (using rays publish command).
And changed the host to: host.docker.internal

I don't think this is related to Lumen. Could you test this in Laravel app in the same environment?

I don't think this is related to Lumen. Could you test this in Laravel app in the same environment?

I also think that this is related to your environment. Works fine with my Lumen 8 installation under php 8 and latest Ray.

I also think that this is related to your environment. Works fine with my Lumen 8 installation under php 8 and latest Ray.

I'm going to close this one, as it seems to be working in Ray. If you are still have problems, please open up a new, specific issue.

I'm going to close this one, as it seems to be working in Ray. If you are still have problems, please open up a new, specific issue.

Hi there, seems to be a problem with my work laptop. Anyidea why?

Hi there, seems to be a problem with my work laptop. Anyidea why?

Hi @mennovanhout, without any information what settings you have configured in your ray.php config or Ray client, unfortunately no one can give you any advice.

Hi @mennovanhout, without any information what settings you have configured in your ray.php config or Ray client, unfortunately no one can give you any advice.

You wrote that you have published your ray.php in the root directory. That could be the problem. Try to put it into your /config directory. It's not needed to configure it in your app.php afterwards, apart from the RayServiceProvider.

You wrote that you have published your ray.php in the root directory. That could be the problem. Try to put it into your /config directory. It's not needed to configure it in your app.php afterwards, apart from the RayServiceProvider.

You wrote that you have published your ray.php in the root directory. That could be the problem. Try to put it into your /config directory. It's not needed to configure it in your app.php afterwards, apart from the RayServiceProvider.

Reading the documentation it needs to be in the root. Also there is no issue regarding the repository on my home pc it works well.
Something is blocking it on my work laptop. Probably something to do with Windows domain control. I do have an administrator account to the laptop. But the allow are laptops to use ray we need to know what kind permissions or windows settings are needed.

You wrote that you have published your ray.php in the root directory. That could be the problem. Try to put it into your /config directory. It's not needed to configure it in your app.php afterwards, apart from the RayServiceProvider.

Reading the documentation it needs to be in the root. Also there is no issue regarding the repository on my home pc it works well.
Something is blocking it on my work laptop. Probably something to do with Windows domain control. I do have an administrator account to the laptop. But the allow are laptops to use ray we need to know what kind permissions or windows settings are needed.

@mennovanhout Did you change your host in ray.php as shown in the docs?

@mennovanhout Did you change your host in ray.php as shown in the docs?

@mennovanhout Did you change your host in ray.php as shown in the docs?

Yes. Also the repository works on another windows machine. Just not the work laptop.

@mennovanhout Did you change your host in ray.php as shown in the docs?

Yes. Also the repository works on another windows machine. Just not the work laptop.

argh... just bought ray license today to debug a lumen projekt, but i get a strange error now:
Target [Illuminate\Contracts\Routing\ResponseFactory] is not instantiable. Is someone able to tell me how to resolve that?


Colab notebook breaking after TF1 support removed

Description
The iPython notebook breaks as mentioned in the Issue title
# Error logs:
ValueError: Tensorflow 1 is unsupported in Colab.

Your notebook should be updated to use Tensorflow 2.
See the guide at https://www.tensorflow.org/guide/migrate#migrate-from-tensorflow-1x-to-tensorflow-2.

did you find some solution on this? is there a fork that solves this?


Dedicated server page should contain more information

Added section about startup arguments
Added section about playlist overrides
Added section about ConVars
Added section about Gamemodes
Added section about Maps

Thanks :D
I noticed the map command is gone in the reworked version but that command is kinda broken anyway so I'm gonna add that to my TODO list for later ^^
Just gotta update SUMMARY.md again due to the removal of one of the files and then this should be set for merging.

hmm, gitbook seems to have broken manually added  links

Ye just noticed that as well. It seems to ignore HTML style <a> anchor points. It does however auto-create anchor points for headings so I updated the links where applicable in this commit<PHONE_NUMBER>b7b73f94972f5feda2fc04fb37d2c6
Just a thing to be aware of I guess :c


S2U-46 5.3.1 Microsoft: User and Group synchronization

If this has new tables or schema changes, we must provide conversion scripts for MySQL and Oracle.


Print service ID in cilium service list

cilium service list
<IP_ADDRESS>:53 =>
		1 => <IP_ADDRESS>:53 (1)
<IP_ADDRESS>:80 =>
		1 => <IP_ADDRESS>:80 (4)
		2 => <IP_ADDRESS>:80 (4)
<IP_ADDRESS>:443 =>
		1 => <IP_ADDRESS>:8443 (2)

[ ] print proper header to explain flag, ID, backend IPs
[ ] use text/tabwriter
[ ] consider a more compact format to print backend IPs

consider a more compact format to print backend IPs

Can you clarify this a bit?


Add SSL and remote debugger options.

Added option for SSL protocol type and remote debugger port specification.  Also added output of the generated command for verification of configuration during runtime.

Thanks for the submission @oblogic7! Just a few requests:

Once ready, remove any debugging output added in the course of setting up this PR.
Documentation for Readme file

Can you add the following to the Other section in the readme?

- `sslProtocol: [sslv3|sslv2|tlsv1|any]`
    - Sets the SSL protocol for secure connections (default is `SSLv3`).
- `remoteDebuggerPort: {port}`
    - Starts PhantomJS in a debug harness and listens on the specified port

I have updated the readme file as requested.
That is not debug output, but is an intentional addition to the output.  Many users, such as myself, already have phantomjs commands that are being used for our testing.  This output is helpful for users to verify that the phantoman configuration matches the required setup that was already in place.  It can also be helpful for cases where the phantomjs configuration needs to be adjusted when using different Codeception environments.

Ah alright. How about for this PR, just the new options get added and I'll setup a PR right now that has some "verbose" output for those that would like it. It'll include the command used at least and I'll see what else might be useful to have output.
I agree that having this output in some cases can be really useful, especially development. So maybe a debug mode is in order. Thoughts on that?

Thanks for the readme updates!

I like that solution.  I will remove the output from my branch so you can merge.


framework: have the gearControllerObject's ray use the correct shader

Fixes issue #1588

GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

Will merge on Wednesday


Example: codeigniter

Uploaded Codeigniter example

Hi could you please rebase it to latest master? Thanks.

Can you please rebase it?

Hey @f3l1x , just rebased, sorry about the delay here. Let me know if there is anything else you require... Maybe there is someone out there who needs to port his old codeigniter app to a serverless landscape...

Thank you


Register AWS DynamoDB state store

Signed-off-by: Shubham Sharma<EMAIL_ADDRESS>Description
Registers AWS DynamoDB state store from components-contrib
Issue reference
Please reference the issue this PR will close: #2469
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Thanks @shubham1172 and welcome to Dapr


Trunc sensitive trace logs

PR description
trunc sensitive data in trace logging, leaving enough for comparison
Fixed Issue(s)

addresses static analysis alerts 597 and 810

close


Materialize Alerts

Bootstrap has, why we can't?
Example in W3Schools:

Because Material Design by Google does not include this in their design specs and it utilises the toast notification bar and cards as a way to well notify the user of their actions. Can I ask what you need the alert messages for?

To show operations, like:

You were registered successfully!
Your password might be stronger
There's already and username registered like this one!
Thinks like that

Have you checked out http://t4t5.github.io/sweetalert/ for notifications, as the name says sweet alerts. I use it and it's really great, I have even had feedback from clients saying that their users love the alerts. Maybe give it a try and see if it works for you.

Ohh amazing, I'm gonna try this one!
Thanks for the help and your time!

It's an absolute pleasure. That's what we are all here for. Hope it works out for what you need it for?

I found this:
https://www.google.com/design/spec/components/dialogs.html#dialogs-alerts

You can accomplish this by using our modal.
On Sun, Nov 29, 2015 at 6:09 PM, Gabriel Caruso<EMAIL_ADDRESS>wrote:

I found this:
https://www.google.com/design/spec/components/dialogs.html#dialogs-alerts
—
Reply to this email directly or view it on GitHub
https://github.com/Dogfalo/materialize/issues/2340#issuecomment-160497842
.

--
Doggy sends his greetings from Mars.

I didn't see that the last time I checked the documents.

If you user Toast you can reach something similar to alerts, but if you want a confirm box, modals can be used or jquery-confirm can do this.
http://craftpip.github.io/jquery-confirm/
I'm using jquery-confirm in a project, no issues with materialize. Just need a workaround the ESC escape key when modal and jconfirm are being used at same time. :)

I agree with all of you, but for what do I need, I'll just copy the Boostrap CSS code, and chance some font and colour!
Thanks for all helps :smile:

@tmd-uk the dialogues mentioned in the Google specification are not mentioned in the materializecss.  Why don't you provide in your materializecss documentation? So that, It will be easy for the developer to use those snippets. You guys are working wonderfully. first, I started to use getmdl.io, but I found difficult to use. MaterializeCSS is more user-friendly and innovative.
We had to give time to develop these dialogues using model. If you provide these in the documentation it will be very helpful.
Thanks. once again.

For those, who got here from Google: you can use .card-panel with color combinations to achieve Bootstrap-like alerts styling.

Success! This alert box indicates a successful or positive action.

Info! This alert box indicates a neutral informative change or action.

Warning! This alert box indicates a warning that might need attention.

Danger! This alert box indicates a dangerous or potentially negative action.

Or we can simplify it with a pinch of SCSS:

The simplest alert.

Success! This alert box indicates a successful or positive action.

Info! This alert box indicates a neutral informative change or action.

Warning! This alert box indicates a warning that might need attention.

Danger! This alert box indicates a dangerous or potentially negative action.

.alert {
    @extend .card-panel, .lighten-4, .text-darken-4;

    &.alert-success {
        @extend .green, .green-text;
    }

    &.alert-info {
        @extend .blue, .blue-text;
    }

    &.alert-warning {
        @extend .yellow, .yellow-text;
    }

    &.alert-danger {
        @extend .red, .red-text;
    }

    .close {
        position: absolute;
        top: 50%;
        right: 0.5rem;

        color: inherit;
        margin-top: -18px;
    }
}

...and add a dismiss button with JS:
$('.alert').append('<button class="waves-effect btn-flat close">close</button>');

$('body').on('click', '.alert .close', function() {
    $(this).parent().fadeOut(300, function() {
        $(this).remove();
    });
});

how can i add sweetalert to my page without materialize interfer with the input created by swal?

I agree with @hsali in that the guidelines are only effective if some utilities are provided to accomplish them, if each developer has to write their own utilities then ....


wrong CI status

PR: https://github.com/nodejs/node/pull/44210

I think that's an issue that should be reported to GitHub, there's not much that can be done at the ncu level.

truly. This seems a longstanding issue. I doubt report would help much.


Integration with desktop doesnt work

Basic information

Your Linux distribution and its version: Ubuntu 16.04.5 LTS

Your desktop environment: Cinnamon

Nuvola Runtime version: 4.15.3

The web app you use and its version: Tried play music and youtube

Describe the bug
All works except the integration with the desktop, so I can't stop, play, skip etc using the buttons in the devices neither It appear over sound icon.
Steps to reproduce the behavior

Open Google Play Music in nuvola app runtime

Expected behavior
Sound icon changes to Music icon in the taskbar and controls to play, stop, etc appears there with the information of what is being played. When a new song starts it used to send a notification to the desktop, that doesn't work either.
Debugging log
Attach the debug log here: debug.txt
Screenshots
I think its not neccesary
Additional context
No comments

Hello @satonio, thanks for reporting the issue. Could you enable Media Player DBus Interface in Preferences and try again?

It don't let me do it. It says I need a license. It only works if you pay or should it work anyway?

When your 31-day free trial ends, you need to purchase Nuvola to use the features which are not available in the free tier. The Media Player DBus Interface feature requires a purchase. Alternatively, you can build Nuvola from the source code for free or try out third-party builds.

Hello. I'm cleaning stale tickets after the release of Nuvola 4.16. If this ticket still concerns you, feel free to reopen it.


MQTT join message not sending after Node JoinAccept

Is this a bug or a feature request?
Bug
What did you expect?
MQTT join mesage published after node join. The MQTT join topic message should be published when JoinAccept occurs without waiting for the first uplink.
What happened?
MQTT join message published after first uplink received from the node.
What version are your using?
v3.3.1
How can your issue be reproduced?
Node OTAA Join
Could you share your log output?

This is intentional, and this aligns with the flow as described in the LoRaWAN Backend Interface specification.
A (re)join can only be confirmed when a device sends its first uplink. Note that for a LoRaWAN 1.1 re-join request, the device will continue to use the old security context when it does not receive the join-accept.


Ticket N°5::2: Perform a hotfix release

This feature is copied from the previous project https://github.com/adorsys/release-scripts/?tab=readme-ov-file#perform-a-hotfix-release.

hello sir @stephane-segning please ticket  #12 and

Repetition with ticket #13


Form is sent multiple times

I have a very simple page that displays a list with an option to duplicate or delete every entry by a radio-button function. The form is sent by

and calls the site itself. PHP performs the deletion or the duplication in a MySQL-database.
That works fine, if I try to delete an entry, but if 'duplicate' is chosen, the entry is duplicated at the first time. If I try to duplicate a entry another time, two duplicates are stored in the database. Third time four duplicates, fourth time eight and so on. After reloading the page or waiting for 30 seconds, the error is 'reset' and it works for exactly one duplication.
If I remove all JQT-parts, all database operations are working fine.
The error occurs on the iPad and on Firefox at a Windows-PC.
With active JQT-Parts the response-time is much longer as with 'native' HTML-Code.
The only initialization-option I use is " statusBar: 'black' "
Thank you in advance for your help

Unlikely we will able to do a custom debugging session on it. (maybe try ask it on the mailing list?)
Please reopen if you can nail down a specific test case that doesn't requires server side setup.


CSSToggleEvent constructor is wrong in IDL

The CSSToggleEvent constructor is broken in two ways:
First, if I create an idlharness test that uses the IDL from the css-toggle spec, I get:
3	FAIL idl_test validation Validation error at line 43 in css-toggle, inside `interface CSSToggleEvent -> constructor -> argument eventInitDict`:
4	  constructor(CSSToggleEventInit eventInitDict);
5	                                 ^ Dictionary argument must be optional if it has no required fields

so I think the event init argument should be optional.
Second, I'm not aware of any other event constructors omitting the type of the event as the first argument (always DOMString type).
(I'm planning to fix the IDL that I commit to WPT and to Chromium on the basis of this comment.)

Yes, it should def be optional. That's my bad.

Second, I'm not aware of any other event constructors omitting the type of the event as the first argument

Hm, I guess usually a specific event class is reused for several related things that all want to expose the same data, so they take it to let you generate each type. I wasn't immediately planning on doing so for this, but I suppose it's a reasonable extension point, so sure. I'll have to figure out some names.
Looking at a few specs, it's not clear what the constructor does with the argument - does it just pass it thru regardless of its value? I don't see anything in DOM or other specs defining event subclasses that validates the value.

I thought the name was already defined as togglechange.

Right, but if I'm only doing the one type of event, letting the name be settable in the constructor args wasn't necessary.
(But yeah I might do more in the future, so I'll add it.)

Oh, and on an extremely similar topic:
CSSToggleEventInit should derive from EventInit, like every other event's init dict.


metadata: icon for nodes does not appear

oups forgot to fix this too, thanks

should be fix: https://github.com/skydive-project/skydive-ui/commit/ee86acb8952d94d4b844d920a91a5b872ec2de40

still exists:


fix symlinks action

A quick fix to the existing action that runs the symlinks file. It's currently disabled, but I figured I might as well fix it up in case it is desired at some point.
You can see it's working here. I deleted the spark scripts to check it's all working as expected.
Happy to rm the action if either of you feels it's unnecessary.
At the very least, it's a useful template for run a bash script and then committing and changes.

@ADBond, @RossKen on actions, does it make sense to combine our black and ruff checks into a singular action? They both perform the same purpose.
If Y, shall I implement an auto-lint using ruff so we can automatically capture and resolve as many issues as possible?

@ADBond, @RossKen on actions, does it make sense to combine our black and ruff checks into a singular action? They both perform the same purpose.
If Y, shall I implement an auto-lint using ruff so we can automatically capture and resolve as many issues as possible?

I think it seems reasonable to have them in the same workflow as there isn't really any meaningful difference in what they are doing for us, and using autofix for ruff also seems like a sensible choice (although I think that still leaves a few we need to manually do. But we're doing that anyway).
I think though it would be good if they would both run and make any changes before committing, but also allowing the separate outputs to be seen easily if there's stuff that needs to be sorted by the dev

https://github.com/moj-analytical-services/splink/pull/1324#pullrequestreview-1476623547
Ah, so the extra layer appears to have paid dividends already...
https://github.com/moj-analytical-services/splink/commit/24745cc14521b706f35ac3a6747e06c9d75179d6


How to introspect UISheetPresentationController

public func introspectSplitViewController(customize: @escaping (UISheetPresentationController) -> ()) -> some View {
        inject(UIKitIntrospectionViewController(
            selector: { introspectionViewController in

                // Search in ancestors
                if let sheetPresentationController = introspectionViewController.sheetPresentationController {
                    return sheetPresentationController
                }

                // Search in siblings
                return Introspect.previousSibling(containing: UISheetPresentationController.self, from: introspectionViewController)
            },
            customize: customize
        ))
    }

error Type of expression is ambiguous without more context

@y0zong sorry for the big delay. This is now implemented in master and will ship in the next version (0.7.0) of the new SwiftUIIntrospect module (more info in README).
Here's a usage example:
import SwiftUI
import SwiftUIIntrospect

struct ContentView: View {
    @State var isPresented = false

    var body: some View {
        Button("Sheet") {
            isPresented = true
        }
        .sheet(isPresented: $isPresented) {
            Button("Dismiss") {
                isPresented = false
            }
            .introspect(.sheet, on: .iOS(.v15, .v16, .v17)) { (controller: UISheetPresentationController) in
                // do something with controller
            }
        }
    }
}


Replace create sockaddr

This PR replaces net pkt parsing in net_context.c by net_pkt_get_src_addr() and net_pkt_get_dst_addr().

The addrlen parameter of net pkt get address function is still there. It might be superfluous but it's nothing wrong to check the size of sockaddr as it's the generic structure for sockaddr_in and sockaddr_in6.
In tcp_syn_rcvd(), the addrlen of accept_cb will always be the size of sockaddr rather than size of sockaddr_in or sockaddr_in6.

recheck

Update:

Follow the naming convention of the return value.
Keep the remote/local sockaddr inside the necessary block.
Pass the actual address length to acept callback.

Ok. I will add some test cases.

@pfalcon and @jukkar , does it make sense to you to add test cases of net_pkt_get_src_addr() and net_pkt_get_dst_addr() in tests/net/net_pkt/src/main.c?

@askawu , based on the functions prefix, yeah, that is suitable location. I'd say it grows too big, but splitting it is apparently a separate matter.

According to function prefix the tests would go to tests/net/net_pkt. But if you look the tests there they are more related to actual net_pkt and net_buf handling, and the tests you need to create are related to IP address management. I suggested a directory in tests/net/utils which has already some IP address parsing tests which are kind of similar what you have here. There is also tests/net/ip-addr directory that could have these tests too.
The actual directory is not that important, the tests are.

Update:

Rebase on the master branch
Add tests in tests/net/utils/src/main.c.


car docs: document parts of parts

Includes old and new code (why it's only a green diff)
This aims to make it easy to document and maintain the parts that come in the shop and the parts required to drive the car.

This allows you to specify sub-parts recursively for any part. So you can separate car harness, red panda kit, and comma three parts for the shop. You can also specify to only show the required parts on the docs and then have all parts in the shop.
It still may be a bit complicated, but I don't know if the alternative with the flat parts enum will be any easier to maintain. I tried to encapsulate all of the specifics of the part shop into one place, instead of having logic spread across openpilot, jinja website templates, and javascript which eventually will get out of hand.
@maxime-desroches what do you think about something like this? It's also possible we don't need this complexity just yet.

Yes this is pretty much exactly what I was about to write but with kits. It does add complexity + redundant/duplicate code, but at least it is in only one place and is in Python which make it much more manageable.

feel free to take this over and make changes as you wish then


GitHub toolkit improvements

Fixes an issue with the github tool where the API returned special objects but the tool was expecting dictionaries.
Also added proper docstrings to the GitHubAPIWraper methods and a (very basic) integration test.
Maintainer responsibilities:

Agents / Tools / Toolkits: @hinthornw

awesome, thanks @Gordon-BP!


Fix broken links

@mvelosop FYI on a broken link from the e-book

Thank you, @NextTurn, for making the additional change. I'll merge your PR now.

Thank you @mairaw


Add benchmarks

Closes #7

[ ] I have updated CHANGELOG.md
[x] I have added tests to demonstrate the contribution is correctly implemented
[x] No test failures were reported when running yarn test-all
[x] I did not do unrelated and/or undiscussed refactorings

/benchmark

/benchmark

/benchmark

/benchmark

/benchmark

/benchmark

/benchmark

/benchmark

test

test

/benchmark

/benchmark

/benchmark

/benchmark

It is not possible to interact with GitHub actions using comments. So I won't add running benchmarks in actions until https://github.com/orgs/community/discussions/59389 is not resolved


[FEATURE] Clarify/Improve DNS Propagation Timeouts

Hi,
So I've encountered this error while testing the latest lego for use within our infrastructure:
Time limit exceeded. Last error: NS ns5.dnsmadeeasy.com. returned NXDOMAIN for <domain>

I understand why it occurs, and have looked far enough into it to see how I can adjust the source code to increase the propagation timeout (by modifying the provider's timeout). In my particular case, the timeout being used is for the "Dyn" provider -- and I'm hitting the timeout when the DNS propagation check happens to choose one of our secondary DNS provider's (DNS Made Easy) servers. When it happens to choose a Dyn (primary) server, the 60s allowed by the Dyn provider timeout is enough; when it chooses a DNS Made Easy (secondary) server, the 60s allowed by the Dyn provider timeout is not enough.
I've also browsed through the various open issues around DNS timeouts, and see that this is a really pretty common problem.
That all said, I'm here to argue that fundamentally, timeouts can have per-provider defaults but that default should be overridable by a command-line argument. Providers having differing defaults makes sense (since generally they'll have different propagation latency characteristics), but really each individual's system is a little different (i.e., like mine; where the secondary provider servers aren't guaranteed to receive the propagation of the change within the necessary window). Allowing over-ride of this via a command-line argument in order to allow users to make adjustments for their specific situation makes sense to me.
What I would suggest is that an argument be implemented, and then the code in dns_challenge.go be updated to prioritize the argument over all other cases. i.e.,
// Insert code here that uses the timeout from the command-line, if set.
switch provider := s.provider.(type) {
case ChallengeProviderTimeout:
	timeout, interval = provider.Timeout()
default:
	timeout, interval = 60*time.Second, 2*time.Second
}

I'm not completely clear on what this command-line argument should be: --dns-timeout already exists and is used for something different (and is symmetrical with --http-timeout). Perhaps --dns-propagation-timeout should be implemented, and then the usage/help text for --dns-timeout should be updated as well so that the difference between the two of them is clear?
I'm willing to take this on and provide a PR for the change, but as per the contribution guidelines I wanted to check here first to see whether such a PR would be welcomed or not.

Running on kubernetes with Ingress records, I'm finding the googlecloud DNS-01 challenge to be very flaky.
I'm not sure if I'm experiencing this issue exactly, because the Google provider's timeouts seem pretty generous?
https://github.com/clio/lego/blob/dd74b99/providers/dns/googlecloud/googlecloud.go#L162-L166
Here's the relevant log snippet:
time="2018-04-06T21:05:49Z" level=warning msg="Endpoints not available for default/ui-ingress-ui-docs"
time="2018-04-06T21:05:57Z" level=info msg="Server configuration reloaded on :80"
time="2018-04-06T21:05:57Z" level=info msg="Server configuration reloaded on :443"
time="2018-04-06T21:05:57Z" level=info msg="Server configuration reloaded on :8080"
time="2018-04-06T21:06:59Z" level=error msg="map[ui-ingress.company.com:acme: Error 400 - urn:acme:error:connection - DNS problem: NXDOMAIN looking up TXT for _acme-challenge.ui-ingress.company.com
Error Detail:
]"
time="2018-04-06T21:06:59Z" level=error msg="Error getting ACME certificates [ui-ingress.company.com] : cannot obtain certificates map[ui-ingress.company.com:acme: Error 400 - urn:acme:error:connection - DNS problem: NXDOMAIN looking up TXT for _acme-challenge.ui-ingress.company.com
Error Detail:
]"
time="2018-04-06T21:07:59Z" level=error msg="map[ui-ingress.company.com:acme: Error 400 - urn:acme:error:connection - DNS problem: NXDOMAIN looking up TXT for _acme-challenge.ui-ingress.company.com
Error Detail:
]"
time="2018-04-06T21:07:59Z" level=error msg="Error getting ACME certificates [ui-ingress.company.com] : cannot obtain certificates map[ui-ingress.company.com:acme: Error 400 - urn:acme:error:connection - DNS problem: NXDOMAIN looking up TXT for _acme-challenge.ui-ingress.company.com
Error Detail:
]"
time="2018-04-06T21:14:40Z" level=warning msg="Endpoints not available for default/ui-ingress-ui-docs"
time="2018-04-06T21:14:40Z" level=info msg="Server configuration reloaded on :80"
time="2018-04-06T21:14:40Z" level=info msg="Server configuration reloaded on :443"
time="2018-04-06T21:14:40Z" level=info msg="Server configuration reloaded on :8080"

Closed by #634


[Bug]: After the SQL plan is deactivated, you can still receive messages

ODC version
ODC424
OB version
independent
What happened?
After the SQL plan is deactivated, you can still receive messages
What did you expect to happen?
After the SQL plan is deactivated, you can‘t still receive messages
How can we reproduce it (as minimally and precisely as possible)?
Disable sql schedule message notifications
Anything else we need to know?
No response
Cloud
No response

All events are or are related to other events


update sapm_docs returns statement

Minor revision to the "returns" statement in the docs for pvib.spectrum.spectral_factor_sapm() following this review

@cwhanse


adding Hexo

hexo is a a fast, simple & powerful blog framework

Sorry, this is a list of tools I personally use. I use Hugo and quite like it for static site generation. 🙂


Add an upgrade to allow items to be sorted into the network.

Am I missing an option for what was in previous mods the "slave controller" or a secondary input into the system.  Right now I'm having to pipe a lot of inputs into the hopper like mine cart unloading, automatic furnaces, and various other devices that output objects.
It would be really handy to just add a mod to a drawer that allows any items placed into it to be sorted into their respective drawer like putting them in the storage controller.

Added in latest version


Nightly builds appear to have stopped building again

TypeScript Version:  2.4.0 / nightly (2.5.0-dev.201xxxxx)
typescript@next
Code
$ date
Tue  3 Oct 09:00:18 BST 2017
$ cd `mktemp -d`
$ npm install typescript@next
$ ./node_modules/.bin/tsc --version
Version 2.6.0-dev.20170930

More critically:
$ npm show typescript | grep 201710

Expected behavior:
A list of the nightly builds from October 2017.
Actual behavior:
No builds listed.

We had a failing test during last night's publish (thus, we did not publish); should be fixed now, and we've released a new nightly. There was no publish on the 1st because it was a weekend with no changes to publish, though. ;)


Add support for vector registers.

This PR also makes a small fixes:

in telamon-gen: universal quantifiers where not processed in the right order when
some quantified sets had parameters.
in variables.exh: some set definitions where wrong but not used before

bors retry

bors r-

bors r+

bors r+

bors r+


When using a custom clock and a transition with a delay of 0, the clock setTimeout function is called but xstate transitions to the next state despite the sent continue function not having been called.

Description
When interpreting a machine and using a custom clock, if you have an after with a delay: () => 0 then the transition will be taken even when the clock hasn't yet called for the statechart to continue.
Expected Result
The statechart should not continue until the function sent to clock.setTimeout is called.
Actual Result
The statechart transitions to the next state without the function argument that is sent to clock.setTimeout being called.
Reproduction
codesandbox
Additional context
Working in codesandbox with most recent xstate version.

I've just checked by forking your codesandbox and updating XState version there and this is working as expected by you. It had to be some bug in the older version of XState.


Recent update crashes a lot in windows at random times.

Just today using VS Code for 5 hours and it has already crashed 16 times. I do not think of any particular action that caused  VS Code to crash, it happens randomly.
Please Fix!

Need any details that you can provide. What does it do when it crashes? Which OS are you using?
Can you try running it with --disable-extensions to check whether an extension is involved?

It crashed a lot yesterday. Had to downgrade. I'm on windows 10 by the way (Also mentioned in the question :P :) ).

Ok but I need more details to understand the issue.

What exactly do you see when it crashes?
Can you try running it with --disable-extensions to check whether an extension is involved?

@roblourens I end up seeing this 👎  everytime my windows crashes.

And yaa running it with --disable-extensions did no good.

Here is a checklist to find out more about the freeze/slow/crash

[ ] it reproduces with extensions disabled (code --disable-extensions from the command line - NOT the integrated terminal of Code)

if it does not reproduce with extensions disabled, you can stop testing and report this!

[ ] it was not happening in the previous stable release
[ ] it reproduces in insiders builds (get it from here)
[ ] there is no suspicious output when running from the command line (code --verbose)
[ ] there is no suspicious output in the window itself (Help | Toggle Developer Tools | Console)
[ ] it reproduces on Windows, Linux and Mac
[ ] it reproduces after deleting <appdata>/User/keybindings.json and <appdata>/User/settings.json (backup first! see below for paths)
[ ] it reproduces when opening empty (code -n from the command line)
[ ] it reproduces opening any folder
[ ] it reproduces opening any file
[ ] it reproduces when only having a single window open
[ ] it reproduces by just using Code without external tools running on the same folder
[ ] it reproduces from a workspace that is not under Git version control
[ ] it reproduces independently if a file is opened or not after startup
[ ] it reproduces when running with GPU disabled (code --disable-gpu)
[ ] it reproduces when being offline
[ ] it reproduces when disabling the file watching (set files.watcherExclude to "**/**": true)
[ ] it reproduces after uninstalling and reinstalling
[ ] it reproduces after deleting the application data directory (backup first! see below for paths)
[ ] I have not played around with permissions or ran Code as administrator
[ ] I am seeing a single CPU process spin high (please share the full command line arguments)
[ ] I am seeing Code consuming lots of memory in a short time

Application Data Directory for VS Code:

Windows: C:\Users\<user>\AppData\Roaming\Code
Mac: /Users/<user>/Library/Application Support/Code
Linux: /home/<user>/.config/Code

Application Data Directory for VS Code Insiders:

Windows: C:\Users\<user>\AppData\Roaming\Code - Insiders
Mac: /Users/<user>/Library/Application Support/Code - Insiders
Linux: /home/<user>/.config/Code - Insiders

@roblourens

[x]  it reproduces with extensions disabled (code --disable-extensions from the command line - NOT the integrated terminal of Code)
- if it does not reproduce with extensions disabled, you can stop testing and report this!

[x]   it was not happening in the previous stable release

[ ]   it reproduces in insiders builds

[x]   there is no suspicious output when running from the command line (code --verbose)

[x]  there is no suspicious output in the window itself (Help | Toggle Developer Tools | Console)

[x]   it reproduces on Windows.

[x]  it reproduces after deleting /User/keybindings.json and /User/settings.json (backup first! see below for paths)

[ ]   it reproduces when opening empty (code -n from the command line)

[x]   it reproduces opening any folder

[x]   it reproduces opening any file

[ ]   it reproduces when only having a single window open

[ ]   it reproduces by just using Code without external tools running on the same folder

[ ]   it reproduces from a workspace that is not under Git version control

[ ]   it reproduces independently if a file is opened or not after startup

[x]  it reproduces when running with GPU disabled (code --disable-gpu)

[x]   it reproduces when being offline

[x]  it reproduces when disabling the file watching (set files.watcherExclude to "/": true)

[x]   it reproduces after uninstalling and reinstalling

[x]   it reproduces after deleting the application data directory (backup first! see below for paths)

[x]   I have not played around with permissions or ran Code as administrator

[ ]  I am seeing a single CPU process spin high (please share the full command line arguments)

[x]   I am seeing Code consuming lots of memory in a short time

Application Data Directory for VS Code:
Windows 10: C:\Users<user>\AppData\Roaming\Code
Application Data Directory for VS Code Insiders:
Windows 10: C:\Users<user>\AppData\Roaming\Code - Insiders

@Suzal3579 are you using the integrated terminal for work?

@bpasero yes, I do.

It might have something to do with the terminal. I occasionally crash when I save changes and then webpack has to recompile.

Yea, It might be the terminal. The crash only happened after the terminal actions. For me, it was nodemon.

Hmm, I too use nodemon.

Duplicate https://github.com/Microsoft/vscode/issues/34301


Update README.md

Closes #27

Thank you for 🌟 adding badges to our GitHub README file! Your contribution has 🚀 enhanced the visual appeal and informative value of our project. We appreciate your effort and look forward to your future contributions! Keep up the great work! 👏


Search variables in Run and Debug variables pane

During a study, a user was debugging their application and struggled to find specific variables they were looking for in the variables pane and rather found themselves scrolling endlessly through all of the other variables listed there. They noted that the variables panel was the "least helpful" due to this. It would be nice if there was a way to search for specific variables in this view. Some other ideas that came up were "smart ordering" to raise "important" variables to the top based on what is being examined in the application
cc/ @luabud @heejaechang

#230945 implies it is already possible.

It would be great if there was a 🔎icon at the top level rather that having to select a variable and then F3 which isn't very discoverable imo

I would like to help clarify the feature request and possibly later on also take the ticket for implementation.
Should this only apply to the Variables pane?
Except for the Breakpoints pane, all panes include the search feature (F3) in the Debug View which are the following:

Variables
Watch
Call stack
Loaded scripts
Event listener breakpoints

Should it reuse the existing search feature or is there another feature request for the way of searching?
As visible in the picture below, I added the "Search All" icon, left to the "Collapse All" icon, which can open the aforementioned search functionality. Is this the full expected behavior?

#230945 implies it is already possible.

It's in the docs
"Variable names and values can be filtered by typing while the focus is on the VARIABLES section."


JWT Decoder timestamp UI on hover

What feature or new tool do you think should be added to DevToys?
When decoding a JWT if there is a timestamp such as "exp":<PHONE_NUMBER>, then it would be nice to have that when hovering over the timestamp it would show a tooltip with the datetime in human readable format. Ideally converted the whatever timezone the PC timezone is using.
Why do you think this is needed?
I often want to know when a JWT expires. and end up having to google to get from a timestamp into a human readable form. Or I just end up using https://jwt.io/ which has this feature.
Solution/Idea

Comments
No response

Hello,
This has been added to DevToys 2.0, which will be released in a few weeks


Trouble with installing

After installing and running php artisan webloyer:install I got exception (see screenshot)

Please verify that your local MySQL db is accessible with these credentials.
In console write:
mysql -uroot -hlocalhost  webloyer
You should be connected to your local database named webloyer.
If database does not exist,  make it.
mysql -uroot -hlocalhost
CREATE database webloyer;

On 4 Dec 2017, at 15:20, Artem Rizhenkov<EMAIL_ADDRESS>wrote:
After installing and running php artisan webloyer:install I got exception (see screenshot)
https://user-images.githubusercontent.com/987517/33557093-4628da3c-d917-11e7-96b6-a1621b6c15f2.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/ngmy/webloyer/issues/81, or mute the thread https://github.com/notifications/unsubscribe-auth/AKAoMb1zvzSN0PR4oP07xMMEgVtyh3C5ks5s8_-ggaJpZM4Q0uRA.

@sdukaric
Thank you for answering instead of me!