automatically generate merge constructors for structures

Suppose a structure S extends R1, R2, R3 with new axioms (x1 : X1) (x2 : X2).
The structure command should prove a "merge constructor" that builds an instance of S from (s1 : R1) (s2 : R2) (s3 : R3) (x1 : X1) (x2 : X2), with preconditions that the overlapping components of s1, s2, and s3 agree.
There is a discussion here: https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/lean-discuss/K5I7UtwZ7nE/FuUdlG6Xr6EJ
Here is an example for group -> comm_monoid -> comm_group.
definition merge {A : Type} (g : group A) (m : comm_monoid A) : (comm_monoid.mul = group.mul) → comm_group A :=
group.cases_on g
(λ mul₁ mul_assoc₁ one₁ lid₁ rid₁ inv₁ is_inv₁,
comm_monoid.cases_on m
(λ mul₂ mul_assoc₂ one₂ lid₂ rid₂ mul_comm₂,
λ Heq,
have Heq' : mul₂ = mul₁, from Heq,
comm_group.mk mul₁ mul_assoc₁ one₁ lid₁ rid₁ inv₁ is_inv₁ (eq.rec_on Heq' mul_comm₂)))

I implemented syntax sugar for creating structures. It addresses this issue.
Here are some examples:
   {| point, x := 10, y := 20 |}   -- create a point setting s.t. field x contains 10, and field y 20
   {| point, x := 10, using p |}  -- similar to previous one, but gets unspecified fields from p

More examples:
https://github.com/leanprover/lean/blob/f9d7480f5ce5986cff0708f4add3620082c900ba/tests/lean/run/struct_inst_exprs.lean
I used the new construct to cleanup many definitions in algebra.
Example:
https://github.com/leanprover/lean/blob/master/library/algebra/ordered_ring.lean#L262

This was a great idea. It depends on the fields having the same name, but
it covers all the examples I can think of. It will also make it easier to
declare concrete instances too, e.g. in nat and int. I will do these and
make sure you got all the others when I am back in Pittsburgh.
The structure mechanism is getting close to optimal. The only remaining
desiderata that I can think of are these:
(1) performance issues (short-circuiting paths)
(2) having a convenient way to instantiate all the theorems for a structure
in a concrete instance.
Thanks again for making Lean so great.
On Fri, Jan 16, 2015 at 9:14 PM, Leonardo de Moura<EMAIL_ADDRESS>
wrote:

Closed #350 https://github.com/leanprover/lean/issues/350.
—
Reply to this email directly or view it on GitHub
https://github.com/leanprover/lean/issues/350#event-220446143.

Can we use more than one structure, as in "using s1, s2"?
Jeremy
P.S. We have notation "dec_trivial" for "of_is_true trivial"
On Sat, Jan 17, 2015 at 8:19 PM, Jeremy Avigad<EMAIL_ADDRESS>wrote:

This was a great idea. It depends on the fields having the same name, but
it covers all the examples I can think of. It will also make it easier to
declare concrete instances too, e.g. in nat and int. I will do these and
make sure you got all the others when I am back in Pittsburgh.
The structure mechanism is getting close to optimal. The only remaining
desiderata that I can think of are these:
(1) performance issues (short-circuiting paths)
(2) having a convenient way to instantiate all the theorems for a
structure in a concrete instance.
Thanks again for making Lean so great.
On Fri, Jan 16, 2015 at 9:14 PM, Leonardo de Moura <
<EMAIL_ADDRESS>wrote:

Closed #350 https://github.com/leanprover/lean/issues/350.
—
Reply to this email directly or view it on GitHub
https://github.com/leanprover/lean/issues/350#event-220446143.

Yes, we can more than one. Here is an example:
https://github.com/leanprover/lean/blob/edcc92d9c79a9f8f7b2d04fe3a724e370d2249f1/tests/lean/run/struct_inst_exprs2.lean
BTW, I removed the using.
I also described the new notation in the tutorial.
http://leanprover.github.io/tutorial/10_Structures_Records.html

Beautiful.
On Sat, Jan 17, 2015 at 9:58 PM, Leonardo de Moura<EMAIL_ADDRESS>
wrote:

Yes, we can more than one. Here is an example:
https://github.com/leanprover/lean/blob/edcc92d9c79a9f8f7b2d04fe3a724e370d2249f1/tests/lean/run/struct_inst_exprs2.lean
BTW, I removed the using.
I also described the new notation in the tutorial.
http://leanprover.github.io/tutorial/10_Structures_Records.html
—
Reply to this email directly or view it on GitHub
https://github.com/leanprover/lean/issues/350#issuecomment-70394526.


Include Past AMA sessions in Archive

Description
This adds a common "event" tag for the AMA and Webinar sessions which makes them accessible into a single collection. That is now the collection we loop on for the "Past Webinars" section, instead of just the webinars.
In addition to this, I've also include the video recording for the first AMA session we did, this is done by adding:
video: <youtube-video-id>

into the top of the relevant .md file (fyi @iskerrett) - this wasn't in the handbook, so have added that too.
Related Issue(s)
Closes #511
Checklist

[x] I have read the contribution guidelines
[x] I have considered the performance impact of these changes
[-] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

The diff is so small, it's like you'd seen this request coming!

it's like you'd seen this request coming!

I did get the request: https://github.com/flowforge/website/issues/511


Integrate Original HUNTRESS code

https://github.com/PASSIONLab/HUNTRESS/tree/v0.1.1

It appears the code creates many different temp files. - This is very hacky and not compatible with the parallelised experiments we do. - Abandon issue.


How to work with DebugServer and DebugServerArgs

Hi,
My first post here: let me congratulate you on this truly awesome tool!
I work on embedded systems and being able to debug from Visual Studio is a dream come true. :smile:
Now my issue.
I want to be able to debug an ELF using gbd and OpenOCD.
Right now I can start the debug session but I need to launch OpenOCD first from a command prompt.
Tried using DebugServer and DebugServerArgs to start it but it's not working.

I'm understanding the correct purpose for these two settings?
Follows my LocalLaunchOptions for you to review

Thanks!

You will need a few more parameters to set up the server

  The ServerStarted parameter tells the debugger what to look for in the server output stream to signify that the server is listening on its port. For OpenOCD we usually use:

ServerStarted= "Info\ :\ [\w\d\.]:\<file://w/d/.%5d:/> hardware"

  OpenOCD also writes its status messages to stderr, not stdout, so you need to tell the debugger to look there:

FilterStderr=true
If there are any custom commands that you need to issue to your board (through OpenOCD) in order to initialize debugging then you may need other parameters too. For example on an Arduino Zero I use these:

target remote localhost:3333
file blink.elf
monitor reset halt

monitor reset init

 exec-continue
From: José Simões<EMAIL_ADDRESS>Sent: Saturday, February 20, 2016 1:01 AM
To: Microsoft/MIEngine<EMAIL_ADDRESS>Subject: [MIEngine] How to work with DebugServer and DebugServerArgs (#191)
Hi,
My first post here: let me congratulate you on this truly awesome tool!
I work on embedded systems and being able to debug from Visual Studio is a dream come true. [:smile:]
Now my issue.
I want to be able to debug an ELF using gbd and OpenOCD.
Right now I can start the debug session but I need to launch OpenOCD first from a command prompt.
Tried using DebugServer and DebugServerArgs to start it but it's not working.

I'm understanding the correct purpose for these two settings?
Follows my LocalLaunchOptions for you to review

MIDebuggerPath="arm-none-eabi-gdb.exe"
MIDebuggerServerAddress="localhost:3333"
ExePath="blink.elf"
TargetArchitecture="arm"
MIMode="gdb"
DebugServer="C:\Program Files\GNU ARM Eclipse\OpenOCD\0.10.0-201601101000-dev\bin\openocd.exe"
DebugServerArgs="stlink-v2-1.cfg -f board\stm32f4discovery.cfg"
/>
Thanks!
—
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/MIEngine/issues/191.

@paulmaybee thank you for your explanations. I did manage to get further, but openOCD still fails to load properly....
Here is my current LocalLaunchOptions file:

target remote localhost:3333
monitor reset halt

monitor reset init
break main

exec-continue

Follows the output of OpenOCD when started from a cmd prompt
GNU ARM Eclipse 64-bits Open On-Chip Debugger 0.10.0-dev-00287-g85cec24-dirty (2016-01-10-10:13)
Licensed under GNU GPL v2
For bug reports, read
http://openocd.org/doc/doxygen/bugs.html
Info : The selected transport took over low-level target control. The results might differ compared to plain JTAG/SWD
adapter speed: 2000 kHz
adapter_nsrst_delay: 100
none separate
srst_only separate srst_nogate srst_open_drain connect_deassert_srst
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
Info : clock speed 1800 kHz
Info : STLINK v2 JTAG v23 API v2 SWIM v0 VID 0x0483 PID 0x3748
Info : using stlink api v2
Info : Target voltage: 2.897285
Info : stm32f4x.cpu: hardware has 6 breakpoints, 4 watchpoints
I've set a breakpoint in line 48 @ ServerTransport and I get up to the line:
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
After that a exception throws and the message saying unable to start debug. no process associated with this object.
Any ideas?

Looks like there was a cut-paste error above in my original message, ServerStarted line should read:
ServerStarted= "Info\ :\ [\w\d\.]*:\ hardware"

Yikes. And trying again ...
ServerStarted= "Info\\ :\\ [\\w\\d\\.]*:\\ hardware"

Got it working!
I add to remove the double \ as these don't seem to be needed in the XML file.
I also had to add a line to load the elf file "manually" otherwise the symbols and code weren't loaded.
Still getting some exceptions with the breakpoint set and on pausing but I'll get to that latter.
Thanks so much for your help! :smile:


Release 0.8.0

There have been quite a number of changes, and I think together they would make a nice new release.
There are still 5 outstanding issues in 3 categories:

More flexible help formatting:

https://github.com/tatsuhiro-t/argparse4j/issues/48 - Implement help formatter
https://github.com/tatsuhiro-t/argparse4j/issues/81 - Positional arguments should appear before optional arguments in help output

Deprecation/migration support:

https://github.com/tatsuhiro-t/argparse4j/issues/28 - Check required options based off the destination, not the presence of the option name?
https://github.com/tatsuhiro-t/argparse4j/issues/44 - Provide way of transitioning from -argument to --argument

Supporting a default sub parser:

https://github.com/tatsuhiro-t/argparse4j/issues/49 - Default subparser?

Most of these issues are pretty old, and the first 2 categories require substantial changes and testing. So I think it is better to push these issues to a later release.
We do have a number of source incompatible changes, which also may lead to run time errors if argparse4j is extended in very specific ways: 3 methods added to 2 interfaces. I think the chances of run time problems are very slim. It needs to be documented in NEWS though.
As far as I can tell the only remaining task before it is release ready, is updating NEWS.

@tatsuhiro-t Do you agree 0.8.0 is ready for release? If so, when do you have time to create and upload it?

Agreed.  I can spare time for making a release.  It's been a while since the last release, so I have to re-learn about maven and nexus stuff.  How about the next weekend?

Next weekend sounds great.

I uploaded snapshots upload to OSSRH successfully.  All jars were there, so it looks good.
To make a release, I just remove "-SNAPSHOT" from all pom files, and then run deploy, and updates web pages.
@jstuyts are you OK for me to make a release as a current form of this repository?

@tatsuhiro-t  Yes, I think everything is ready. There are quite a number of usability improvements that users will appreciate. Documentation and news files have been updated.
The remaining open issues have a major impact, so are better postponed to a future release.

Thank you.  I do release work from now.

All release works have done.  Thank you!  Looking forward to 0.9.0 release.

It looks like your forget to publish the argparse4j-root POM. My build fails because it cannot find this POM in Maven Central.

I confirmed that argparse4j-root is missing as well as argparse4j-extensions.
When I deployed 0.8.0-SNAPSHOT staging build, they were uploaded, but they weren't with 0.8.0.
I suspect that we need org.sonatype.plugins configured for root and extensions as well.

I'll release 0.8.1 because mvn deploy changes signature, and nexus refused to update the existing ones.

0.8.1 was released.  release-0.8 branch contains the fixes I made to mvn deploy work for argparse4j-root and argparse4j-extensions.  It looks like <name> is required for argparse4j-root and argparse4j-extensions.  argparse4j-root also requires <description> and <url>.  I also added them in argparse4j-extensions, but I'm not sure they are really required.  Perhaps, only root package requires them?

It is working now. Thanks a lot.
I don't mind having the description and URL duplicated in all POM files. If somebody creates an issue for it, we will fix it then.


RTL implementation

the Arabic admin panel version isn't working well at all since there isn't support for rtl

For RTL integration you can contact us here https://unfoldadmin.com/


feat: support useServerInsertedHTML (wip)

closes https://github.com/hi-ogawa/vite-plugins/issues/533

#536  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hi-ogawa and the rest of your teammates on  Graphite


Add missing --volumes-from argument.

This proxy companion did not function correctly until I added this argument.

Changes was already made.
Closed by b002a6bfa56db3698722907a23e1a9dbc19eb46d


The <style> element is not supported yet. Support for <style> will require implementing CSS selector logic.

first need to add support for it to svgdom

waiting for its support too )

This is related to https://github.com/igagis/svgdom/issues/32

@JaimeIvanCervantes @Papirosnik I'm currently working on implementing <style> element support. If you have any SVG files which use <style> element would be nice if you share those here, I need sample files for testing.

hey @igagis this is great! I just realized that you started a project called cssdom. I am working on a CSS parser myself, and was actually thinking about adding a CSStyleable and support for ElementTypes and classes  to your library, but it looks like you already got started on your own. If you give me 3 more days, I can open a repo with what I have so far, and you can take a look. Perhaps you don't have to reinvent the wheel after all ;).
As for examples, I think the most difficult part is getting complex selectors and animations right. I could come up with a few examples for you.

Thanks @JaimeIvanCervantes , I have the CSS parser designed already, at some extent, but would be interesting to see your approach to CSS as well.
I did not plan to add animations yet, as there is no animation support in svgren anyway and I want to start from simple things.
Regarding the examples, I meant that if you have any real SVG you would expect to work, I did not mean to invent test samples, as this takes efforts.

First <style> element support added to svgren version 0.4.69. All dependency libs have to be update to their latest versions as well.

@igagis thanks! This is good news, I'll check it out

There probably will be some bugs related to CSS... I only tested on very simple SVGs


bear: generated compile_commands.json is polluted by compiler wrapper machinery

Issue description
The bear tool is supposed to wrap a build script invocation (eg. bear make) to record the compile commands into a compile_commands.json, which is then consumed by tools.
However, when the compiler used is one of Nix's gcc/clang compiler wrappers, we end up with multiple results in the compile_commands.json. This file was created from compiling a single executable:
[
    {
        "command": "c++ -c -o CMakeFiles/bear_test.dir/test.cpp.o /run/user/1000/tmp.bsY0QLdSyj/test.cpp",
        "directory": "/run/user/1000/tmp.bsY0QLdSyj",
        "file": "/run/user/1000/tmp.bsY0QLdSyj/test.cpp"
    },
    {
        "command": "c++ -c -o CMakeFiles/bear_test.dir/test.cpp.o -B/nix/store/9nw5abnvdw91vrgv1z9x06560nxhyka8-gcc-5.4.0-lib/lib -B/nix/store/1jwkjzrivp541v5ibgzb4wc2avla5zx9-glibc-2.24/lib/ -idirafter /nix/store/s222vly64qrr3l4mqm6n3fsrkprhrnmm-glibc-2.24-dev/include -idirafter /nix/store/psmdlfqys1031hhyjhky4qphgyscmgdg-gcc-5.4.0/lib/gcc/x86_64-unknown-linux-gnu/5.4.0/include-fixed -B/nix/store/9k8fjm8zs3qnbri9fij7l36rkfks6kd0-gcc-wrapper-5.4.0/bin/ -O2 -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param ssp-buffer-size=4 -fPIC -fno-strict-overflow -Wformat -Wformat-security -Werror=format-security /run/user/1000/tmp.bsY0QLdSyj/test.cpp",
        "directory": "/run/user/1000/tmp.bsY0QLdSyj",
        "file": "/run/user/1000/tmp.bsY0QLdSyj/test.cpp"
    }
]

Steps to reproduce
nix-shell -p cmake -p gcc -p bear
cd "$(mktemp -d)"
cat > CMakeLists.txt <<-EOF
    cmake_minimum_required(VERSION 3.0)
    project(bear_test)
    add_executable(bear_test test.cpp)
EOF
cat > test.cpp <<-EOF
    int main() {
        return 0;
    }
EOF
cmake . && bear make
cat compile_commands.json

Technical details

System: Description: Ubuntu 16.04.1 LTS
Nix version: nix-env (Nix) 1.11.4
Nixpkgs version: "17.03pre94694.fa4167c"

cc @vcunat (maintainer of the bear package)

The issue here is understandable: bear notices that the wrapper is being called and records it, and then notices that the wrapper is calling the actual compiler and records that. bear -vv confirms it:
$ bear -vv make
...
bear: format this command: ['/nix/store/9k8fjm8zs3qnbri9fij7l36rkfks6kd0-gcc-wrapper-5.4.0/bin/g++', 'CMakeFiles/bear_test.dir/test.cpp.o', '-o', 'bear_test', '-rdynamic']
bear: format this command: ['/nix/store/psmdlfqys1031hhyjhky4qphgyscmgdg-gcc-5.4.0/bin/g++', '-Wl,-dynamic-linker', '-Wl,/nix/store/1jwkjzrivp541v5ibgzb4wc2avla5zx9-glibc-2.24/lib/ld-linux-x86-64.so.2', 'CMakeFiles/bear_test.dir/test.cpp.o', '-o', 'bear_test', '-rdynamic', '-B/nix/store/9nw5abnvdw91vrgv1z9x06560nxhyka8-gcc-5.4.0-lib/lib', '-B/nix/store/1jwkjzrivp541v5ibgzb4wc2avla5zx9-glibc-2.24/lib/', '-idirafter', '/nix/store/s222vly64qrr3l4mqm6n3fsrkprhrnmm-glibc-2.24-dev/include', '-idirafter', '/nix/store/psmdlfqys1031hhyjhky4qphgyscmgdg-gcc-5.4.0/lib/gcc/x86_64-unknown-linux-gnu/5.4.0/include-fixed', '-B/nix/store/9k8fjm8zs3qnbri9fij7l36rkfks6kd0-gcc-wrapper-5.4.0/bin/', '-O2', '-D_FORTIFY_SOURCE=2', '-fstack-protector-strong', '--param', 'ssp-buffer-size=4', '-fPIC', '-fno-strict-overflow', '-Wformat', '-Wformat-security', '-Werror=format-security', '-z', 'relro', '-z', 'now', '-Wl,-rpath', '-Wl,/nix/store/vhhygpc95apy9s3ng6xz37n2kv1czi8q-shell/lib64', '-Wl,-rpath', '-Wl,/nix/store/vhhygpc95apy9s3ng6xz37n2kv1czi8q-shell/lib', '-L/nix/store/lcnm0g7i7wq8zi525lhrxgwbqvd28zy9-bear-2.2.0/lib', '-L/nix/store/1jwkjzrivp541v5ibgzb4wc2avla5zx9-glibc-2.24/lib', '-L/nix/store/9nw5abnvdw91vrgv1z9x06560nxhyka8-gcc-5.4.0-lib/lib']
...

My feeling is that it should only output the actual compiler call, rather than the only the wrapper compiler call, as that's the only one that actually captures all the flags (even if it is a little weird).

I didn't notice that happening; it might explain some minor issues I was having with it. I share your feeling about what should happen, though I don't see how to achieve it (yet).


Deprecations

Describe your changes!

Reviewer checklist

Read PR description: a summary about the changes is required
Changelog updated
Documentation: docs/{Reference, Cli, ...}, Docker and cli help/usage
Pulled branch, manually tested
Verified requirements are met
Reviewed the code
Reviewed the related tests

Please revert the commit https://github.com/avenga/couper/pull/734/commits/5b924b7e7d762072d6b6cad68ce7725b0641bb7c since its changes almost 20 func calls to use an empty argument to get rid of 3 lines of code.


Fix Non-Contiguous Tensor Issue in Checkpoint Consolidation

What does this PR do?
This PR fixes an issue in the consolidate_tensor_parallel_checkpoints function where non-contiguous tensors caused a ValueError when saving consolidated model checkpoints, particularly with the Safetensors format. The fix ensures that tensors are made contiguous before and after concatenation, resolving memory layout issues during saving.
Error Example:
When running the following command:
consolidate_model_parallel_checkpoints_to_unified_checkpoint(
    Path("./model_sharded/model/shards"),
    "./consolidated",
    "safetensors"
)

The following error occurs:
ValueError: You are trying to save a non contiguous tensor: model.layers.0.self_attn.o_proj.weight which is not allowed.
This PR solves the issue by applying the .contiguous() method to tensors at key points in the consolidation process, ensuring the consolidated model can be saved without errors.
Motivation and Context:
When consolidating distributed model checkpoints (e.g., from model shards), tensors may not be contiguous in memory. Safetensors, which is a memory-efficient way to store models, does not allow saving non-contiguous tensors. This PR modifies the consolidate_tensor_parallel_checkpoints function to apply .contiguous() to tensors to ensure that the checkpoints can be properly consolidated and saved.
This change impacts the overall consolidation pipeline, as consolidate_model_parallel_checkpoints_to_unified_checkpoint depends on the lower-level functions to process tensors correctly.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@michaelbenayoun gentle reminder ...

Could you just run themake style command to fix check_code_quality please.

@michaelbenayoun where can I see instructions on how to do that?

The steps to follow are:

Put yourself inside the optimum-neuron directory
Run pip install -e ".[quality]"
Run make style

@michaelbenayoun done

Something went wrong when you ran the command, because it changed many files.
Rebasing and merging as #736


Set Up Integration Suite Trial

Hi,
Tutorials: https://developers.sap.com/tutorials/cp-starter-isuite-onboard-subscribe.html
Issue: Could not create service instances/key
Here, observation: In STEP 3, in the Integration Suite home page When i tried to Add the Capabilities, API Management Capability Activation is failing with the following error.

When i Reactivated again by that time it got activated.

And when i proceeded with the further step, in STEP 4 , Assigning Roles and create service instances using Booster step, From the Booster i Enabled Integration suite by clicking  the Start and  configured the subaccount and Selected Activated Capabilities and when i clicked on finish button then "Assigning Role Collection" and "Assigning Service Quotas" were Activated but Creating Service Instances got failed with the following error:
Error APIMGT100: Activation of API Management capability has failed. Please try again later. If the issue persists, please send your application URL and details of the error message<EMAIL_ADDRESS>Error Details:
Could not create service instance/key as the capability is still not activated.
Creating a service instance for the entitlements below failed:
it-rt, Plan : integration-flow
it-rt, Plan : api
Creating service key(s) for the service instances(s) below failed:
it-rt, Plan : integration-flow
it-rt, Plan : api
Solution:
Please wait for the capability to be activated and then execute the booster.

Labels:
Landscape Labels - trail2
Sprint Label: SCPTest-2309A
Scenario Label: SCPTest-cloudin
Priority: High
Thanks & Regards,
Soumya Gyanapnor

Issue is resolved. Hence closing this ticket.


Can't execute CLUSTER_NODES,which version solved?

@mrniko hello,i had view some close issues,but i dont know which version  really solved this problem.
redis on port 7001 is master
redisson version:2.8.2
the following is my error log
18-12-22.00:10:55.177 [redisson-netty-4-5] ERROR ClusterConnectionManager  - Can't execute CLUSTER_NODES with /xxx.xxx.xxx.xxx:7001
java.lang.IllegalArgumentException: java.net.MalformedURLException: unknown protocol: redis
	at org.redisson.misc.URLBuilder.create(URLBuilder.java:125) ~[redisson-2.8.2.jar:na]

@hisenyuan Please can I ask you to try it again with a later version of Redisson?

ok,thank you for your reply. Which version do you recommend?

@hisenyuan Please can I ask you to try it again with a later version of Redisson?

ok,thank you for your reply. Which version do you recommend?

I would always suggestion the latest version but you should read the release note and make sure you have done all the necessary tasks to ensure your application is compatible.

I am using the latest version of redisson and still have this problem. 3.11.1 and 3.11.2 are the latest

fixed


Add small improvements to documentation

Hey :)
if you want you could add that presets are stored in ~/.config/easyeffects/output and that they can be loaded with the preset button in the upper left corner in EasyEffects and combine multiple effects and are more than simply an equalizer.
Those were the things that were unclear to me at first because I have never used EasyEffects before and maybe that would also be helpful for others!

Technically it's written here https://github.com/wwmm/easyeffects/wiki/Community-Presets

Thanks, that should be sufficient


Fixed codemirror SQL syntax highlighting

Just a few minor changes for SQL syntax highlighting to work out-of-the box.

cool, thanks!


Inputs - Add SAP AO number to side sheet and main table. Plus a Filter

Expected outcome

Side page input for SAP AO  -> should be free text

There should also be a SAP AO column in the main sheet:

Acceptance Criteria

[ ] Side page input for SAP AO number
[ ] Column for SAP AO
[ ] Filter for SAP AO number

For the time being we will put this in the description - moved to low priority

see issue: https://github.com/equinor/lighthouse-client/issues/1546


New homepage

What does this pull request do? Explain your changes. (required)

This PR updates the homepage, and some theme tokens (mainly colors). It also adds a prettier config that i tooked from seeing the codebase a bit. I added this because my internal prettier config was messing with the code a lot.
Specific updates (required)

I tweaked a bit the Button primitives
Added Link primitive. Very useful to normalize the whole external vs internal links and theme-ui vs native links...
Updated the footer
Created Logo component

How did you test each of these updates (required)

I tested visually, on chrome, safari, firefox and brave. In Safari and Firefox I'm not showing the gradients because they really struggle to render them.
Does this pull request close any open issues?

Nope
Screenshots (optional):

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

@julianbenegas looks incredible! I'm going to squash and merge this into a new branch I just created called rebrand and we can create PRs into that branch for the inner pages you're starting this upcoming week. Then once the rebrand is complete next week we can squash and merge rebrand into master.


MGDOBR-537: Code coverage using jacoco

MGDOBR-537
PR owner checklist
Please make sure that your PR meets the following requirements:

[x] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[x] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[x] The layers in the kustomize folder have been updated accordingly.
[x] All new functionality is tested
[x] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

Thank you for PR @Aakanksha0311 . Can be effetc of your changes seen somewhere? Or it is just an 'attempt' to fix?

Yes, I tried to check for the coverage report using a draft PR - [(https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/1149)]. Here you can see the coverage percentage.


How do I configure this work with SystemJS?

I don't see an index.js file or any other file for SystemJS to point to.

its included on npm when installed from there. https://github.com/jkuri/ng2-datepicker/blob/master/package.json#L34.

@avoliva Include the .umd.js in the map: attribute of your systemjs.config.js :
'ng2-datepicker':    'npm:ng2-datepicker/bundles/ng2-datepicker.umd.js',


wip

Codecov Report

Merging #188 (288b9c0) into master (1950eaf) will decrease coverage by 0.54%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
- Coverage   92.45%   91.91%   -0.53%
==========================================
  Files          19       20       +1
  Lines         172      173       +1
  Branches        1        3       +2
==========================================
  Hits          159      159
- Misses         13       14       +1

Impacted Files
Coverage Δ

...e/src/main/scala/zio/spark/UnavailableMethod.scala
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

We decided to inform the user about this instead: https://univalence.github.io/zio-spark/faq#why-method-is-returning-a-nosuchmethoderror-exception-


Support other design systems

A team from the NHS working on COVID19 response projects reached out, they would like to use this form builder to aid their work.
This is an exploratory branch testing out the feasibility of the form builder supporting other compatible design systems, namely that used by the NHS.
By compatible, I'm referring to design systems where the form elements match the GOV.UK version in all but the prefixes of the class names.
In the NHS version, it appears that govuk has just been replaced with nhsuk, so a label would look like this:
<label class="nhsuk-label">First name</label>

How it works
There is a new configuration option called brand that defaults to govuk.
All hard-coded references to govuk throughout the form builder have been replaced to calls to the #brand method which in turn pulls the value from the config.
What's left to do

[x] Await feedback from the NHS team to ensure everything's working as they expect
[x] Tests, tests, tests
[x] Version bump, to 1.2.0 (beta pushed, 1.2.0b1)
[x] Updating the guide and docs where appropriate
[x] Explore the possibility of aliasing the form builder helper methods if a non-default brand is used, eg = form.nhsuk_text_field(...)
[ ] Update the README accordingly

Preview

@peteryates are you able to get in contact with either myself (Mark Sutton) or Nuno Costa who are both working on the PHE project who you have adapted this for?
<EMAIL_ADDRESS>

It appears that there are some deeper differences between the NHS Design System and the GOV.UK Design System than initially thought. Some JavaScript functionality in the NHS DS isn't properly triggered because of slightly-differing class names/attributes.
The NHS team using the form builder have had to monkey patch some methods to bring them in line.
This gem will not claim that it supports the NHS DS, but the work in this PR is still valid and could be used as the basis for a companion gem that applies refinements/monkey patches where required to add the necessary markup needed by the NHS DS; it would also need its own set of tests.

[ ] Make it obvious that the NHS Design System isn't officially supported
[ ] Set the branding guide page to draft but leave the API documentation in place


Adding an implementation of Denoised Smoothing

This PR adds Denoised Smoothing under the research folder.
The success of Randomized Smoothing is proven and it works for many different scenarios. But it also operates under the assumption that the underlying classifier is able to perform well under Gaussian perturbations. Won't it be better if we could just take our standard pre-trained image classifiers (including the Cloud APIs) and have the benefits of Randomized Smoothing inside of them in an easy manner?
That is preciously what Denoised Smoothing does by prepending a Denoiser to an image classifier and still maintains the theoretical guarantees of robustness against L2 attacks.
Besides, the implementation includes a suite of utilities that may be helpful to generate robustness certificates. I think that gives us a unique opportunity to actually design an API inside NSL that would allow easy generation of robustness certificates. To the best of my knowledge, there does not exist any such framework as we speak.

@csferng,

Please run pylint to format the code into TensorFlow style (instruction here).

I have followed most of the instructions I got from the linter barring a few exceptions:

scripts/analyze.py directly comes from here without any change (with citation of course). This is why I have kept it intact.
models/resnet20.py comes from here with the license attached.

Apart from these, I could not resolve the following:
scripts/certification_utils.py:57:4: R1705: Unnecessary "else" after "return" (no-else-return)
scripts/certification_utils.py:82:4: R1705: Unnecessary "else" after "return" (no-else-return)

scripts/certify.py:14:0: W9008: Redundant returns documentation (redundant-returns-doc)
scripts/certify.py:30:2: R1732: Consider using 'with' for resource-allocating operations (consider-using-with)
scripts/certify.py:37:2: C0200: Consider using enumerate instead of iterating with range and len (consider-using-enumerate)
scripts/certify.py:57:4: R1732: Consider using 'with' for resource-allocating operations (consider-using-with)

models/dncnn.py:27:13: R1719: The if expression can be replaced with 'bool(test)' (simplifiable-if-expression)

Also, I was checking some of the modules under research and I felt they do not strictly follow tensorflow/tools/ci_build/pylintrc (such as type annotation, trailing newline, etc.).

@csferng I am happy to change it Apache. Please advice.

@sayakpaul, sorry that I didn't make my point clear. Since Google does not own the copyright to those code, please move the directory to research/third_party/denoised_smoothing. Either MIT or Apache 2.0 license is fine, and you can choose one you'd like.

@csferng done.

@csferng could you provide an update on the progress of the PR?

@csferng just wanted to see if an update is available for this PR.

@sayakpaul Sorry for the delay. The PR is merged now.

Thank you so much.
Is it possible to include a note on the third_party projects here?


instance_seg_image has 4 channels

When I ran test_lanenet.py, I found one of the network output, instance_seg_image, has 4 channels. Why? I thought it should have 3 channels.  How can I convert the 4 channel tensor to an image?

@ArtlyStyles It is the visualization result of the pix embedding result:)


Add support for Wood Stuff

Wood Stuff adds variants of various chests utilizing wood from other mods. You might want to update the recipes to use oredictionary definitions of chests.

I've added support for some recipes. But I don't know how to add support for reinforced chests.

@Adaptivity Here's where the recipe is registered. You'd have to change getReinforcedRecipe to take an object instead of a block.

Chests are part of ore dictionary since 1.8. I've just added private static void registerOreDictionary() in my PR and registered them. I know very little about java programming, so I'm not sure what I should change. Better someone else should take a look.

The chests in Wood Stuff are registered as ore:chest according to Minetweaker.


Splitting the window selected by number

Independently of #313: what do you think about window selection by optional prepending window number to split? This is more precise than "wincmd p", which not distinguish buffer window from another window type.
For example: 5s would vertically split window 5 (like :5wincmd w|vsplit /file/under/cursor/in/nerdtree) and so on (2i, 9o).
Showing winnr() in status bar may be helpful in jumping between windows.

This is a good recommendation for a lot of NERDTree code in general. However, it is especially good since window numbers became unique in Vim 8.0.

This was a neat idea, but there isn't really an issue to be resolved here.  You may want to explore this further independently.  Thanks!


Firebase version not getting updated to latest v8

I have no options to install firebase v8, it's installing 7.3.1 in Xcode 13 beta I tried the same way in Xcode 12 it install v8 as expected.

I'll be closing this issue since it's a duplicate of #8313. Thanks.


PackageInfo becomes a global variable

PackageInfo becomes a global variable and can be called anywhere in the code through "Utils.packageInfo".

Codecov Report

Merging #156 (4a0cb94) into master (7cc496a) will increase coverage by 0.01%.
The diff coverage is 55.26%.

@@            Coverage Diff             @@
##           master     #156      +/-   ##
==========================================
+ Coverage   69.56%   69.57%   +0.01%
==========================================
  Files          80       80
  Lines        6348     6351       +3
==========================================
+ Hits         4416     4419       +3
  Misses       1932     1932

Impacted Files
Coverage Δ

lib/generated/intl/messages_ro.dart
95.23% <10.00%> (-0.40%)
:arrow_down:

lib/generated/intl/messages_en.dart
96.42% <40.00%> (ø)

lib/main.dart
53.01% <50.00%> (-0.08%)
:arrow_down:

lib/generated/l10n.dart
64.10% <90.00%> (+0.05%)
:arrow_up:

lib/pages/settings/view/request_permissions.dart
95.94% <100.00%> (+0.05%)
:arrow_up:

lib/pages/settings/view/settings_page.dart
97.89% <100.00%> (+2.89%)
:arrow_up:

lib/resources/utils.dart
60.00% <100.00%> (+2.85%)
:arrow_up:

Continue to review full report at Codecov.

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


Get Address using Latitude and Longitude

It will be nice if the feature implemented in this plugin. i.e conversion of latLng to address. So that we can avoid using so many plugins.
Example:

LatLng(13.00064608385679, 77.66582395881414) to 114, Nagavaara, Bangalore- 5600XX

Something like this.
Thanks :)

Hi @PrakashNaidu2662
Please explain in detail which plugin you referring to.
Thank you

@PrakashNaidu2662
Closing since long without an answer.
Please explain more precisely your proposal and I will reopen it.
Thank you


Question regarding NFS about performance and security

I have been a long time Dinghy user (4+ years) but the past 5-6 months, I've used Docker for Mac (Experimental) release, which had support for Mutagen and it actually worked pretty well, just up until they decided to drop it and go with gRPC-Fuse which performed almost as slow as the original OSXFS. Now I am having to install a very specific version of the experimental release just to have D4M be usable and that is clearly not an ideal solution.
So I am most likely going to switch back over to Dinghy. But my issue w/ Dinghy was that the containers were outside of the localhost network and that there were lingering question about security. Other than that, Dinghy has been perfectly fine. But at the same time, I was not entirely sure what the downside of Dinghy were and want to clarify.
I've read some security issue and it was brought up again over at D4M when they recently tried exploring NFS as an alternative solution. There is some interesting chat going on https://github.com/docker/roadmap/issues/7#issuecomment-736694100, with many things over my head. Recently what was interesting was that, when they tested, they found NFS w/ inotify to be even slower than OSXFS, which is something I did not expect to hear. They did mention it was for write-heavy workload, which I guess isn't for my case or any of our cases? I'm still not really sure who are the main target audience for D4M.
Also they mentioned about security implication and I know you mentioned this too. But to clarify, are every Dinghy user atm when on a LAN, are they opening up their homedir to everyone on the network to see?

That discussion doesn't give any details about how they did the benchmarks, so it's hard for me to comment -- all I can really say is the dinghy implementation of NFS + filesystem events was definitely much much faster at reads and writes than Docker for Mac's solution back when we did the benchmarks. There's some numbers here: https://github.com/codekitchen/dinghy/issues/166
As mentioned in the dinghy README, the NFS daemon only binds on the private interface between the host MacOS and the virtual machine. So no, your files aren't even exposed on the local network.
To be clear, I don't use dinghy myself anymore. I find Docker for Mac plenty performant enough for my current use cases, and it just makes sense to use the standard solution if at all possible. But I've tried to keep the project alive as much as I can, since a lot of people still seem to rely on it.

@codekitchen it is interesting to hear D4m is performant enough for your use case! Currently D4M doesn't seem to be suitable for any of the web applications such as Rails, Nest.js, etc that have a lot of package files. Even just starting it up takes over 5 - 10 minutes (Dinghy was near bare-metal speed, so it takes just a few seconds). This is on an i9 2.3ghz 16gb memory on an SSD drive. If you do similar work but don't have this problem, I'm curious to know what you may be doing that makes it work for you.
Regarding the NFS security, that is re-assuring. It makes me wonder if the D4M team have explored the solution you have put in, and if so, what the problem were.
I'd like to keep using Dinghy moving forward as it was the easiest to setup compared to some other solutions. As for D4M, it would be the easiest way but I don't see any performance breakthrough happening in the near future. I hold a class w/ many students and it is quite a painful process of getting them to install Docker in a specific way to be performant enough to do Rails/Node projects. It literally scares them away from using Docker.
 From what I can tell, Dinghy is the most 'usable' out-of-the-box solution for me and many developers. It's just was not mainstream for whatever reasons.

Yeah most of my projects these days are very small so I just haven't run into any performance blockers. I originally started the dinghy project back when I was working on a very large Rails monolith and we desperately needed a better solution to those same startup time issues.
As for why Dinghy never really got super wide usage, I'm sure that's mostly my fault. I created the project to be used internally by my company, because somebody had to do it, but it's not something I was ever particularly passionate about so I never really tried to get the word out about it, or pushed the idea as far as I could have.

@codekitchen i must say, that was quite an impressive effort you have made back then. and yes, there could've been a bit more PR and possibly Youtube videos demonstrating the usage explaining the why of combining both nginx proxy, dnsmasq, and nfs. perhaps that could've fueled the fire a bit. but regardless, i appreciate you for coming up w/ this years back when there was no decent solution, i and many others have benefited greatly.
i really hope you can continue to work on this project though, i think it solves a problem for many web developers who are not just dealing w/ performance issues but also those dealing w/ port conflicts running multiple docker containers and wanting a simple 1-click installer for dnsmasq and proxy.

@codekitchen While we've moved our large Rails monolith (and ancillary phoenix services) to Docker Desktop, we couldn't have gotten here without dinghy. We were avid users across our organization until just a couple months ago. Thank you so much!!!

Also we continue to use dinghy-http-proxy with Docker Desktop!

@ryansch I'm guessing the performance was the same or close enough after the switch as compared to with using dinghy? Was there any special configuration on top of a standard setup to achieve good results? Were NFS mounts still used for the containers?


How to use authorization token to login instead of using username+password?

Hi, I couldn't seem to find any documentation specifiying how to login with the access token instead of regular auth scheme?

@JohnnyRacer This authorization is meant to work as a simple placeholder for something that's going to be overwritten in a production application. Auth is a large and mostly complex topic. FastAPI doc has some pointers on how you can implement token-based auth.
https://fastapi.tiangolo.com/tutorial/security/simple-oauth2/?h=

@rednafi  Thank you so much for pointing me in the right direction! Much appreciated.


Can't update to Autoform 5

Just wanted to point out that installing this package prevents you from being able to update to Autoform 5.  (currently restricted to 4.0.0)

I second this request.
Any chance on updating it to autoform 5?
If you can just update the api.use section I can test it and confirm everything is working.
Thanks!

+1

Any update on when this will be update?

+1

I already fixed this.  Pull request is waiting to be merged.
https://github.com/comerc/meteor-autoform-selectize/pull/11

+1

+1

Since this was preventing Autoform updates and I use it in several apps, I went ahead and published my own package to fix the version constraint.  For anyone interested...
meteor add jeremy:autoform-selectize

https://github.com/jshimko/meteor-autoform-selectize
https://atmospherejs.com/jeremy/autoform-selectize
Note that this now depends on my jeremy:selectize package and will install it automatically.
Sorry @comerc, I certainly don't mean to steal your thunder here.  I just really needed this update to happen.  :)  Thank you for your work on this!

Thanks @jshimko for the fork/update.

@jshimko thanks for the fork/update!

@jshimko thanks mate

@jshimko thank you very much :+1:

Thanks @jshimko for the fork

@jshimko, it is not fork, but copy. I dont see changes for merge. Very bad. Bugs also duplicated. :)


Redefine definition in collection constructor

The PR https://github.com/pzuraq/ember-classy-page-object/pull/13 fixes an issue when collection definition is a PageObject. However, it does not completely fix all problems as the definition of collection is still a PageObject and hence definition.scope is defined incorrectly.
This PR fixes that redefining this.definition in the constructor if it detects if it's a PageObject.

@pzuraq

Hold on merging this. I want to test it in app to verify that it actually works

It works. You can merge now @pzuraq


Pages with 2+ password fields must use different programmatic labels for 'show' and 'hide' buttons on those password fields

This issue is from May 2023 external accessibility audit report.
WCAG Reference:
2.4.6 Headings and Labels (Level AA)
Understanding Headings and Labels | How to Meet Headings and Labels
Issue ID:
DAC_Non-descriptive_Labels_01
URLs:
URL: https://signin.account.gov.uk/create-password
Screen shots

Both of the ‘Show’ buttons presented for users have been provided with an ‘aria-label’ to offer users of screen reading assistive technologies additional context that they are to show the password. However, both buttons have been provided the same name of ‘Show password’, meaning that screen reader users are not able to distinguish which button relates to which field.
This same occurs when the button is activated, and the label is changed to ‘hide’.
Current code ref(s)
#main-content > div > div > form > div:nth-child(3) > div > div > button
<button class="govuk-show-password__toggle" aria-controls="password" type="button" aria-label="Show password">Show</button> […] <button class="govuk-show-password__toggle" aria-controls="confirm-password" type="button" aria-label="Show password">Show</button>
Screen-reader comments

“I found that two buttons announcing as ‘Show password’ were present on the page which when selected updated to read as ‘Hide password’. The label of each button was duplicated but related to different password fields. It was not immediately clear when situated out of context which button related to which password field potentially leading to the user making an incorrect selection. Although I was not personally affected in this instance some screen reader users may become confused. Giving each button label unique content will enable all users to make an informed selection both in and out of context.”

Solution
Ensure that labels are descriptive enough for users to identify their purpose and to which content they relate. The label for each ‘show’ button should inform users of screen reading assistive technologies to which field they relate.
The second button should have an ‘aria-label’ value of ‘Show re-type password’ for example.

When there are two or more password fields on a page, the 'show' and 'hide' labels for each password input need to be different.
For example, for an input labelled "Password" the label could be "show first password". And for a second input labelled "Re-enter password" the label could be "show second re-entered password".

Yes, this seems like a good plan to me. A couple of minor tweaks suggested below:
When there are two or more password fields on a page, the 'show' and 'hide' labels for each password input must be different.
For example, you can label the input "Password" as "show first password" and label the second input "Re-enter password" as "show second re-entered password".

Works for me, yep! Do we want to go ahead and just make a one-off PR for this piece?
I'm unsure how to approach, because it's not technically related to WCAG 2.2, but it is a guidance update for accessibility. It also seems like something we could easily merge and check off as done rather quickly.

Yes, it's a simple change for now. We can do further reviews and changes if we amend the component or make an addition. I'd like @calvin-lau-sig7 to have final sign off as the site's content owner.

I've opened a PR for this (which i think is the next step), and noticed that some more contextual info is needed. So I've also added a draft of some info on 'what exactly is a show/hide password button?'

https://github.com/alphagov/govuk-design-system/pull/2861

Thanks both! I've put in an edit suggestion and comment. Overall, I'm hoping we can find a quicker/simpler/lazier way to setup the context for these messages.


docs(oauth2) give more details to header X-Credential-Identifier

Summary
The original docs confuses customers. Upstream header
"X-Credential-Identifier" actually evaluates to client_id.
Reason
Customer reporting issue at FTI-3320.

@outsinre can you please rebase your PR? We've renamed all plugin index.md files to _index.md in a recent change.


How does websockets handle disconnects?

I couldn't find this in the docs or SC, could you help me out?
A server has a pingTimeout of 20 seconds.
A user connects, authenticates, and subscribes to foo at time = 0.
That user loses internet at time = 5.
A message is published to foo at time = 10.
If the user reconnects at time = 15, will they get the message? is it guaranteed? If so, where is the queue? If not, are they aware they missed a message?

@mattkrick Sorry for the massive delay in reponse. You probably already figured this out but for the sake of closing this issue:
In this situation, when you do a publish while disconnected, the SC client will try to reconnect. If the internet is still down, SC won't be able to reconnect. This will cause the publish action to fail - If a callback was provided to the publish method, an error will be passed to it which will indicate the timeout.

@jondubois I'm not sure if your response answers the question (as I read it). It sounds like you are saying that the user who publishes the message will have the error passed to the callback if the publish fails, but what happens if the publish succeeds by one user and a second user's internet is the problem? Here's my understanding of the question (since I'm looking for the answer myself):
User A and B both subscribe to channel "foo" at time = 0
At time = 5, user A loses internet
At time = 10, user B publishes a message to "foo"
At time = 15, user A reconnects
Will user A get the message that was published to the "foo" channel?

As I understand it, no. The message was pushed to A, we didn't know A was disconnected because it was just a hiccup between heartbeats, and there's no callback to ensure durability. To ensure A got it, the server would need to wait for a callback from A for every message and a queue (redis?) would have to persist the message until every connected user received the message. Not impossible, but at scale, I can think of very few sites that use durable messaging without employing something like rabbitMQ.

@agilethomas As @mattkrick mentioned, in the case were the connection is lost for a short amount of time - This is not long enough for the heartbeat/ping timeout to occur (but this depends on the value of your pingTimeout option), as a result, the same underlying WebSocket/TCP connection will be maintained by SC (not terminated).
Because TCP guarantees in-order message delivery (over a single session/connection), client A would still receive the message in your scenario. If multiple messages were 'missed', they will all arrive to client A in quick succession once the TCP session is recovered (once internet connectivity comes back).
If however, user A's connection is lost for a longer amount of time (whatever your pingTimeout option is), the underlying WebSocket/TCP connection would have timed out and been closed down. In this scenario, when the internet comes back for user A, the SC client will reconnect using a new underlying WebSocket connection - So in this case client A would have missed all messages published while it was offline.
So basically, SC guarantees in-order, exactly-once message delivery for minor (short-duration) network failures. However, If a client times out for a long time, then you may have to recover missed messages by getting a new snapshot of your data/message log from a database/datastore.
Note that SC exposes MIDDLEWRE_PUBLISH_IN and MIDDLEWARE_PUBLISH_OUT middleware lines which allow you to define custom functions to capture inbound (coming from a single client and going to the server) and outbound messages (coming from the server and going out to multiple clients) so you can add logic to persist messages yourself.

Thank you very much for the detailed explanation. That's good to know and I think will serve our needs well. We'll just have to code our app so that any time it has to reconnect, we'll request a snapshot of the current state. We're currently using socket.io in our application, and it appears that we are losing messages when the network connection is poor, but I can't be sure if it's a socket.io, TCP, or application issue. But in any case, once I stumbled upon this framework, I realized it will be so much easier to work with than socket.io, since the nature of our app is a pub/sub system. That means very minimal code to route messages between clients.
Another quick question (not sure where else to ask this): if a client subscribes to a channel, and then publishes to that same channel, will the client that published the message also receive the same message that it sent? I wouldn't expect that to be the behavior but that's how it is working for me, so I wasn't sure if I was doing something wrong or not.

@agilethomas Yes, if the client is subscribed to a channel and publishes to it, that client will in fact receive its own message. See https://github.com/SocketCluster/socketcluster/issues/54


Make colors used configurable

A recent message on the mailing list asked how to modify the colors used by beets when "color: True" is set in the config. It is not currently possible, would it be worth the work? And then how would it be implemented?
An enum with COLOR1, COLOR2, etc.? Or a description of when it is used? e.g. DIFF, DIFF_MINOR, SHORTCUT, SHORTCUT_DEFAULT, PENALTY, INFO, ...

Yes, I suppose this would work by making a new configurable dict mapping roles to color names. It seems doable, but it's not quite clear how much of a problem this is (or, where it is a problem, when we should just make our default colors better :smiley:). Opinions either way would be welcome.

It doesn't really add any overhead, and it's easy to add, so I don't see why not. The following dict can map all the color requirements that I could find (and would allow the distinction between error messages and highlights, both of which are currently red).
COLORS = {
    'text-success': 'green',
    'text-warning': 'yellow',
    'text-error': 'red',
    'text-highlight': 'red',
    'text-highlight-minor': 'lightgray',
    'action-primary': 'turquoise',
    'action-secondary': 'blue',
}

I could make a PR for this, if the color names are alright

Also see https://github.com/sampsyo/beets/issues/363, so it's not the first time this has been asked.

Good catch on the previous issue. Names seem fine and it seems to cover all current usages (in beets.ui.commands beets.ui beetsplug.fetchart and beetsplug.play).
action_* colors name could be improved, maybe action_default and action_others (or just action)?
Finally I prefer the named which dashes (as you initially wrote them) but that 100% personal taste.

The whole codebase (and the config file) uses underscores, that's why I changed them from the dashes.
I agree that action_default fits better than primary. Not too sure about action_others though.
I would also propose to nest the colors option under the ui config. Would be nice to have color there as well, but suppose that would break compatibility.
color: yes
ui:
    terminal_width: 80
    length_diff_thresh: 10.0
    colors:
        text_success: green
        text_warning: yellow
        text_error: red
        text_highlight: red
        text_highlight_minor: lightgray
        action_default: turquoise
        action: blue

LGTM. Yes, color would be better under ui. Maybe we can offer
both, and write a deprecation message when people use the top-level
color setting. That's a similar situation to the one hinted in PR
#1269 with list_format_*.

Looking good already.

How about changing ui.colorize so it takes the name of the action as the first parameter (e.g. 'text_success') and it does color = COLORS[color_name] inside? that would be a better abstraction.

Agreed; that would be awesome.


is anybody actively supporting this project ?

is anybody actively supporting this project ? Does it support versioning if used to migrate the sites and projects?

I guess that answers the question


NineAnime - Manga doesn't appear

Source information
NineAnime 1.4.4
Source language
English
Steps to reproduce
1.Browse NineAnime in the app
2.tap on any manga
3. Tap the read button
4. It will Kick back
5. Some error appear "www.glanceoflife.com:SERVFAIL"
Expected behavior
It should appear the manga image so can read it without any problem
Actual behavior
It kick out back when want to read so it interrupted reading session
Mihon/Tachiyomi version
0.15.3
Android version
Android 12
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Hi Is it this problem can't be solved?


Create a base model and generic serialization.

@vincentsarago  This is what I ended up doing based on #143
I realized all of the classes included BBox and the GeoInterfaceMixin, so combined it all in one new base.
I added a class variable __exclude_if_none__ that is used to determine which fields are excluded. Not particularly invested in the name, was just what I came up with. Also considered starting it with __geojson_pydantic to be a bit closer to some of the __pydantic ones.
Its getting late and my brain has quit on me. I think there were more tests I wanted to add but I will have to take another look in the morning.

I added a class variable exclude_if_none that is used to determine which fields are excluded. Not particularly invested in the name, was just what I came up with. Also considered starting it with __geojson_pydantic to be a bit closer to some of the __pydantic ones.

it might be safer to put something more specific, we can just go with __geojson__exclude_if_none__ but I'll be ok if we don't change 👍


Text-Formatter not Quoting non-string Values

Hi,
I've noticed that all values that aren't strings or errors are written to the output buffer without any quoting. This causes issues when a value is e.g. a time.Time struct:
msg="..." updated_at=2017-07-12 14:24:59 +0000 UTC foo=bar

From such a log line, logfmt parsers extract fields like "14:24:59": true and "+0000": true and so on.
Currently, these values are written like this:
fmt.Fprint(b, value)

My suggestion is to first string-encode the value using fmt.Sprint into a variable and then apply the regular and optional quoting. Will send a PR soon.

Good idea! It would also probably be a good opportunity to use b.WriteString and be consistent in that method.


Several fixes

Thanks!
(N.B. This prompted me to look up "javascript trailing commas." The degenerate behavior still present in IE9 makes me sad.)


Argument mismatch, evaluation error

When I try to evaluate (import ./default.nix) { silent = false; } it fails with an evaluation error. At first this was confusing me since the function clearly has such an argument. However the fail happens later in the call of ./support/nix/eval-with-configuration.nix with the same args, which indeed does not have a silent argument.
IME, nothing good has ever come out of @args argument capturing, and some alternative mechanism should be used here instead.

This was likely leftover from early futzing about with the repo.


Image transport compressed format can't be changed

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, feature request, or anything other than a bug report directly related to this project.

Description
I am using Zed2 on ROS2 using the latest SDK 3.8, CUDA 11.4 on a Jetson NX2.
Launching they zed2.launch.py everything works including the compressed, compressedDepth transport plugins. i can echo both of the topics but now i would like to change the format of the plugin from 'jpegtopng. As this a configurable parameter of image_transport` package i should be able to do so.
Unfortunately, i can't  because as far as i can see from ros2 run rqt_reconfigure rqt_reconfigure. there is no parameter for the plugin options, only for the original topic ie zed2/zed_node rgb.image_rect_color.format. Even this though is not configurable with a message like this from terminal running zed launch file:
 zed_node.rgb.image_rect_color.format is not a dynamic parameter
Steps to Reproduce

Launch zed2 launch file
Run rqt_reconfigure node
Try changing image format to png but ends with error
Image format of compressed, compressedDepth and other plugins can't be seen in rqt under zed2 node

Expected Result
I expect i can change the format of the image transport plugin or any other parameter of it
Actual Result
Can't chagne due to  zed_node.rgb.image_rect_color.format is not a dynamic parameter error
ZED Camera model
ZED2
Environment
Jetpack LT35.1
ARM
Nvidia Jetson NX2 module
ROS Galactic

Anything else?
No response

Hi @xkaraman
ROS2 Galactic is not LTS, so it's not officially supported

Should i expect any difference in humble? Is there any differnce on how you handle image transport plugins and their parameters?
Moreover Jetson's Jetpack that is LTS only supports (officialy) galactic due to being based on Ubuntu 20.04.

The official LTS for Ubuntu 20.04 is Foxy.
Galactic is an intermediate release that will reach the EOL in the next month.
You can use ROS2 Humble on Jetson in a Docker container.

There is a known problem with image_transport in Foxy:
https://github.com/stereolabs/zed-ros2-wrapper#known-issues
It works correctly instead in Humble.

Ah OK, you can fix this behavior by upgrading to the latest version of the master branch that we released a few minutes ago.

Ok i will check it out and report back if the problem persists!
Thanks for the quick respondence! :)

I get this error when trying to built the latest master branch!
/home/ssd-nvidia/ros2_workspaces/sensors_ws/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:41:2: error: #error Unsupported ROS2 distro    41 | #error Unsupported ROS2 distro
So, i can't built this at all at ROS2 Galactic?

Hi @xkaraman
As I told you, we do not officially support Galactic. In the new release, we added CMake commands and C++ macros to avoid the compilation for unsupported distributions.
You can remove them manually to force the compilation and then fix the eventual compilation issues.
However, by reading the Galactic and Humble changelogs, I noticed that the fixes that you need have been introduced in Humble and not in Galactic.

Yes i have tried to remove the CMake checks and it seems to build alright!
I can now alter the format of the image transport. But a crucial point is that i can only do that for the base image topic (this was the case before too, only that i couldnt alter the parameter). There is no parameters for the compressed topic, compressedDepth or any of the plugins!
As you can see there is no zed_node.depth.depth_registered.compressed.format ( To be honest i am coming from ROS1 and i am sure there was a different parameter for base, compressed, compressedDepth, don't know if it stands true for ROS2. Correct me if i am wrong)

Yes, you are right.
I can confirm that they are all available in Humble

Ok then, so the best way forward for a Nvidia Jetson (latest version is based on 20.04 Ubuntu that humble is not available through official packaging) is through Docker Humble version i guess?


Feature request: zstandard compression

It would be nice to have zstandard compression, and from what I can tell it would be a simple addition.
I could make the PR?

Yes, please go ahead


[beta] Toggle Collapse Map

[ ] Add toggle collapse to the Map

I've forked FCC today... I'll try my first open source contribution and this issue seems a good point to start. Is there anyone already helping this one?

Hi @rossijonas thanks for looking into this, this might be taken care of until after beta is shipped. I recommend you should start with the issues labled as "help wanted" on the tracker.
You can hang out with us in the Contributors Chat room, for any assistance.
Thanks and Happy coding!

@QuincyLarson Thoughts on this. I remember at some point we decided to remove the buttons on the map feature.

@BerkeleyTrue Yes - they are all collapsed by default and you can drill into them.
@raisedadead We don't want to complicate the UI with buttons. I think https://devdocs.io does a great job of this - a clean, intuitive, unambiguous sidenav.
So I'm going to close this issue.


Request two models to display multiple columns.

Is your feature request related to a problem? Please describe.
A single model performs quite well, but with multiple models, switching between models via arrows in the dialogue box is very inconvenient.
Describe the solution you'd like
I hope to increase the displayed content in multiple columns, at least two columns side-by-side, making it easier to compare how different models handle issues.
Additional context

Related #2237


Revert "Ensemble copula coupling extension"

Reverts metoppv/improver#49

Reverting ECC merge as some tests failed.


Getting timeout trying to create totally new store / machine from b2d github releases

Try this with master:
$ docker-machine --storage-path /tmp/store create -d virtualbox dev
Creating CA: /tmp/store/certs/ca.pem
Creating client certificate: /tmp/store/certs/cert.pem
Image cache does not exist, creating it at /tmp/store/cache...
No default boot2docker iso found locally, downloading the latest release...
Error creating machine: Get https://api.github.com/repos/boot2docker/boot2docker/releases: dial tcp: i/o timeout
You will want to check the provider to make sure the machine and associated resources were properly removed.

Haven't dug into it too much yet, can anyone dupe?  cc @ehazlett @hairyhenderson

tl;dr - I can't reproduce...
With HEAD:
$ machine --storage-path /tmp/store create -d virtualbox testvbox
Creating CA: /tmp/store/certs/ca.pem
Creating client certificate: /tmp/store/certs/cert.pem
Image cache does not exist, creating it at /tmp/store/cache...
No default boot2docker iso found locally, downloading the latest release...
Downloading https://github.com/boot2docker/boot2docker/releases/download/v1.6.2/boot2docker.iso to /tmp/store/cache/boot2docker.iso...
Creating VirtualBox VM...
Creating SSH key...
Starting VirtualBox VM...
Starting VM...
To see how to connect Docker to this machine, run: machine env testvbox
$ machine --storage-path /tmp/store ssh testvbox uptime
00:23:25 up 1 min,  1 user,  load average: 1.04, 0.66, 0.26
$ # all good...

Mmm, yeah, I'm kind of suspicious that there's some funny business with the network at the office.

Closing as this looks network related.  Please re-open if you continue to have issues.

Same problem:(
docker-machine --storage-path /tmp/store create -d virtualbox dev
Creating CA: /tmp/store/certs/ca.pem
Creating client certificate: /tmp/store/certs/cert.pem
Image cache does not exist, creating it at /tmp/store/cache...
No default boot2docker iso found locally, downloading the latest release...
Error creating machine: Get https://api.github.com/repos/boot2docker/boot2docker/releases: dial tcp: i/o timeout
You will want to check the provider to make sure the machine and associated resources were properly removed.
If it's network related, it should still have something to do with docker machine. Everything else is just normal.

@chouclee that error is machine not being able to resolve github.com.  Are you behind a firewall or anything?

@ehazlett  Yes... I think you are right.  Now it's working.

I had the same problem because of the nameservers in my /etc/resolv.conf; the DNS resolver will pick them in the same order as they are listed and won't try the next one in the list unless the previous one times out.

I've had this happen to me since, but trying more creates seems to work eventually  :(

I got the same error:
docker-machine create --driver virtualbox dev
No default boot2docker iso found locally, downloading the latest release...
Error creating machine: Get https://api.github.com/repos/boot2docker/boot2docker/releases: dial tcp: i/o timeout
You will want to check the provider to make sure the machine and associated resources were properly removed.

I was able to resolve github.com:
curl https://api.github.com/repos/boot2docker/boot2docker/releases
...
    "url": "https://api.github.com/repos/boot2docker/boot2docker/releases/1537750",
    "assets_url": "https://api.github.com/repos/boot2docker/boot2docker/releases/1537750/assets",
    "upload_url": "https://uploads.github.com/repos/boot2docker/boot2docker/releases/1537750/assets{?name}",
    "html_url": "https://github.com/boot2docker/boot2docker/releases/tag/v1.7.1",
...

I have a local DNS server in my office. A TCP dump of port 53 from the two requests:
docker-machine create:
10:21:32.010850 IP <IP_ADDRESS>.54541 > <IP_ADDRESS>.53: 16666+ A? api.github.com. (32)
10:21:37.011731 IP <IP_ADDRESS>.54541 > <IP_ADDRESS>.53: 16666+ A? api.github.com. (32)

curl:
10:20:48.694514 IP <IP_ADDRESS>.58137 > <IP_ADDRESS>.53: 45930+ A? api.github.com. (32)
10:20:49.695032 IP <IP_ADDRESS>.58137 > <IP_ADDRESS>.53: 45930+ A? api.github.com. (32)
10:20:50.794949 IP <IP_ADDRESS>.58137 > <IP_ADDRESS>.53: 45930+ A? api.github.com. (32)
10:20:50.819443 IP <IP_ADDRESS>.53 > <IP_ADDRESS>.58137: 45930 1/0/0 A <IP_ADDRESS> (48)

The solution for me was to clear my DNS settings and re-connect to the network -- turns out the the first DNS server IP was dead (<IP_ADDRESS>), and after reconnecting it was replaced with the new IP. As @dnozay said it seems it will only try the first DNS server in the list, so you really need to make sure your DNS works perfectly.

For others who might've had the same issue, it turned out to be rate-limiting for me:
curl https://api.github.com/repos/boot2docker/boot2docker/releases
{
  "message": "API rate limit exceeded for <ip address>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)",
  "documentation_url": "https://developer.github.com/v3/#rate-limiting"
}

@mefellows If you're getting rate limited, there's now a --github-api-token flag to specify your own GH API token to dodge the rate lmit

Thanks @nathanleclaire, good tip!


♻(react) use react aria for select menu + modal updates

When there is not enough space to open the select menu, this one does not open 🤔
CleanShot.2024-05-14.at.16.55.15.mp4

I realized that Canvas in mdx are not inside iframe 👀, so I did some edit + avoid flipping for the overlay.


Cookie preferences button doesn't work

Current Behavior
The link to manage cookie preferences doesn't work in the page footer:

Expected Behavior
No response
Steps To Reproduce
No response
Environment

Date:
Platform/OS:
Browser:

Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

I also have this


It stopped to work

Got error message today:
In PackageDiff.php line 135:

  Could not open file ad66625dce6fd04da4b628f870a7230cce53fd10:apps/silverbac
  k-drupal/composer.lock or find it in git as ad66625dce6fd04da4b628f870a7230
  cce53fd10:apps/silverback-drupal/composer.lock

I tried to debug it by adding a new job step:
git show ad66625dce6fd04da4b628f870a7230cce53fd10:apps/silverback-drupal/composer.lock

This one worked without any issues.
Debugging here: https://github.com/AmazeeLabs/silverback-mono/pull/1021

Hmm, when I run the same code as in \IonBazan\ComposerDiff\PackageDiff::getFileContents, it works just fine.
Could it be that the action needs some kind of a GitHub permission to read from the git history?

I am experiencing the same issue. I've checked the workflow permissions on the repo I'm running the action on and it appears to have read permission to the repo.

Thanks for the reporting! I will have a closer look at this after Easter.
My first guess would be that it's related to actions/checkout v3 - I am still using v2 in my projects and it worked last time. @Leksat can you help to check if reverting to v2 helps?

Affecting me too. I'm pinned to actions/checkout@v2 and it isn't helping. Debug run at https://github.com/massgov/openmass/runs/6047519645?check_suite_focus=true

Confirming. Downgrading actions/checkout tov2 does not help.

we're also having this issue :/

@Leksat @oradwell I found that it's caused by https://github.com/actions/checkout/issues/760 - I will be fixing it now here.

The error is gone 🎉 Thank you!
Now there is a new little issue. When there are no composer changes, the action output is not empty, but is set to the following:
::set-output name=composer_diff_exit_code::0

I believe https://github.com/IonBazan/composer-diff-action/commit/f4eb9f4598cc7d5ac7df67920f8fd333120a696d should fix that - please try to use IonBazan/composer-diff-action@master to see if that works.

@IonBazan it works like a charm on master branch 🎉

@Leksat great! Closing this issue then 🚀

thanks both. I can confirm it works flawlessly


How to evaluate RDFConverter from BAM? Alternative for docker?

How to evaluate RDFConverter from BAM?
BAM does not allow to use of docker. Maybe other places also do not allow it.
We could increase the usability level of RDFConverter with an alternative for docker.
I got the following error using docker from a Linux Virtual Machine at BAM:
avaldest@VM8093:~/RDFConverter$ docker-compose up
ERROR: Couldn't connect to Docker daemon at http+docker://localhost - is it running?

If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable.

It is accessible now with the following URL:
http://rdfconverter.matolab.org/


Disconnects automatically after very short time

No error is thrown, but my localtunnel disconnects automatically after an incredibly short period of inactivity. It seems to disconnect in under 1 min of inactivity - which makes it very difficult to develop with. Is there any way to disable/increase this timeout?

create as bash script like try.sh with
#!/bin/bash
lt --subdomain testratesyapp --port 3000

and then run watch n30 ./try.sh so this runs the try.sh every 30 sec
and it worked for me with a constant subdomain

Duplicate of "the server is not running persistently #111"  https://github.com/localtunnel/localtunnel/issues/111

i did the following for my OSX with this issue
#!/bin/bash while :;    do    clear   date   /usr/local/bin/lt --subdomain (UR DOMAIN) --local-host <IP_ADDRESS> --port 80   sleep 30 done

I use a crontab to restart it every 15m, works for me.
*/15 * * * * cd /home/work/supervisor && supervisorctl restart localtunnel

Can some 1 explain me how to do this?

And author thinks this is better than ngrok???

And author thinks this is better than ngrok???

@sergeyganago relax, Where do you see me saying it is better than ngrok or forcing anyone to use this? I made this project because I had a random idea on how to do local server exposure with nodejs, that's it.

And author thinks this is better than ngrok???
For free users: Yes. I'm creating a messenger bot and it needs a persistent url.

@si458 I don't think your script will check for anything. It just simple do what watch do. Run lt every 30 secs


Implement PKCS#11 support for 'openss cms' commands

Currently, the openssl cms command does not support PKCS#11, which prevents using smart cards for signing or decrypting CMS files.

Currently, OpenSSL has no direct support for PKCS#11 whatsoever.  There is an external PKCS#11 engine provided by the OpenSC project's libp11, and the openssl cms command does support using engines with the -engine flag.

As per @levitte's response there does appear to be a way to do this, so closing.

Found a mailing list thread from 2016 about incorporating PKCS#11 support natively, but apparently the idea went nowhere. It's a pity, because I don't know of any other Linux tool to sign content with (when the privkey resides on a crypto token).

some people are giving a try on this , see #8200


Draw fonts with gradient fill

Make it possible to draw text with a gradient fill (top to bottom primarily).

This should be possible if you use masks and so forth.

I use ShaderMask do that.
new ShaderMask(
    shaderCallback: (Rect bounds) {
        return new LinearGradient(
        begin: Alignment.topCenter,
        end: Alignment.bottomCenter,
        colors: <Color>[Colors.red, Colors.yellow],
        tileMode: TileMode.clamp,
        )
            .createShader(bounds);
    },
    blendMode: BlendMode.modulate,
    child: new Center(
        child: new Text('Gradient Text'),
         style: Theme
            .of(context)
            .textTheme
            .display1
            .copyWith(color: Colors.white)),
    ),
),

this does not work for me, I also tried the example on the doc.: https://docs.flutter.io/flutter/widgets/ShaderMask-class.html ans it also does not do what it advertises, which is to color the text, instead, it paints a rectangle below the text
Here's my Doctor summary :
[✓] Flutter (Channel beta, v0.4.4, on Linux, locale en_GB.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] Android Studio (version 3.1)
[✓] Connected devices (1 available)

https://github.com/flutter/flutter/pull/18347 will fix this more thoroughly.

@Hixie thank you so much for the speedy and useful reply.
I'm still left with two questions, nonetheless:

The feature #18347 is still not available through flutter upgrade, right? And if so,when can we expect that?
Is the shaderMask deprecated? does the docs example work for you? or should the docs example be updated to a working one?

Thanks again


support sdk version wildcard format

add support for specifying a wildcard format for provisioning the latest patch release of an SDK (2.19.x, 3.1.x, ...)
close https://github.com/dart-lang/setup-dart/issues/56

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

@sortie - do you mind reviewing this?

I updated the PR to use 2.19 (and, rev'd the version of the action to 1.6.0 - this PR essentially adds new API to the action).

Thanks for the feedback and discussion! Much appreciated.

https://github.com/dart-lang/setup-dart/issues/107

\cc @kevmoo I really like this since it's can resolve the main issue I had with the pubspec SDK version specifier for package:mono_repo. If my lowerbound was ^3.0.0, the CI wouldn't run with later patch fixes until I changed the lower bound and regenerated.
Do you think a new pubspec-minor (or some similar name) choice would make sense after this is stabilized, that would use 3.0 instead of 3.0.0?


Add dynamically generated cookie secrets for oauth

Description
Follow up to: https://github.com/opendatahub-io/data-science-pipelines-operator/pull/34#discussion_r1129907202
How Has This Been Tested?
Tested locally and live on ocp. Simply run make deploy IMG=$PR_BUILD and test oauth proxy.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

lgtm after fixing the one bad name


Disallow assigning Optional, allow yielding void functions and nothing

Fixes #1544. First sample now shows an error "Cannot assign Optional<void> to a variable". Second sample prints "PASS".

Not stale

I'm not 100% on how we want to treat void, but we can fix (or break) that in the future if needed. LGTM.


Moved rabbit config into config.json

/cc @drnic as my pair

Hey guys, can I get a little explanation of what your trying to fix here.
It's been a while since I looked at this BOSH release need a little context.

Mate, the release didn't work for us (see recent issue) - but when we moved the files around it started working :)
On Mon, Apr 13, 2015 at 2:50 PM, trastle<EMAIL_ADDRESS>wrote:

Hey guys, can I get a little explanation of what your trying to fix here.
It's been a while since I looked at this BOSH release need a little context.
Reply to this email directly or view it on GitHub:
https://github.com/FreightTrain/sensu-client-boshrelease/pull/3#issuecomment-92511107

Cheers.
Having a read of the issue at the moment. I remember using the conf.d stuff
On Mon, Apr 13, 2015 at 11:00 PM, Dr Nic Williams<EMAIL_ADDRESS>wrote:

Mate, the release didn't work for us (see recent issue) - but when we
moved the files around it started working :)
On Mon, Apr 13, 2015 at 2:50 PM, trastle<EMAIL_ADDRESS>wrote:

Hey guys, can I get a little explanation of what your trying to fix here.
It's been a while since I looked at this BOSH release need a little
context.
Reply to this email directly or view it on GitHub:

https://github.com/FreightTrain/sensu-client-boshrelease/pull/3#issuecomment-92511107
—
Reply to this email directly or view it on GitHub
https://github.com/FreightTrain/sensu-client-boshrelease/pull/3#issuecomment-92512904
.

--
Troy.


[install] Super long install time on Apple MacBook Pro M1

Have you read and followed the installation tutorial at http://www.nokogiri.org/tutorials/installing_nokogiri.html?

[X] Yes!

What is the complete output of gem install or bundle install?
This does not complete (so far).
What are the complete contents of the mkmf.log file generated during the failed installation?
This has not yet generated.
Tell us about your system!
What is the output from ruby -v?
ruby 2.7.1p83 (2020-03-31 revision a0c7c23c9c) [x86_64-darwin19]
What is the output from gem -v?
3.1.2
What is the output from gem env?
RubyGems Environment:
  - RUBYGEMS VERSION: 3.1.2
  - RUBY VERSION: 2.7.1 (2020-03-31 patchlevel 83) [x86_64-darwin19]
  - INSTALLATION DIRECTORY: /Users/mzagaja/.rvm/gems/ruby-2.7.1
  - USER INSTALLATION DIRECTORY: /Users/mzagaja/.gem/ruby/2.7.0
  - RUBY EXECUTABLE: /Users/mzagaja/.rvm/rubies/ruby-2.7.1/bin/ruby
  - GIT EXECUTABLE: /usr/local/bin/git
  - EXECUTABLE DIRECTORY: /Users/mzagaja/.rvm/gems/ruby-2.7.1/bin
  - SPEC CACHE DIRECTORY: /Users/mzagaja/.gem/specs
  - SYSTEM CONFIGURATION DIRECTORY: /Users/mzagaja/.rvm/rubies/ruby-2.7.1/etc
  - RUBYGEMS PLATFORMS:
    - ruby
    - x86_64-darwin-19
  - GEM PATHS:
     - /Users/mzagaja/.rvm/gems/ruby-2.7.1
     - /Users/mzagaja/.rvm/rubies/ruby-2.7.1/lib/ruby/gems/2.7.0
  - GEM CONFIGURATION:
     - :update_sources => true
     - :verbose => true
     - :backtrace => false
     - :bulk_threshold => 1000
  - REMOTE SOURCES:
     - https://rubygems.org/
  - SHELL PATH:
     - /Users/mzagaja/.rvm/gems/ruby-2.7.1/bin
     - /Users/mzagaja/.rvm/gems/ruby-2.7.1@global/bin
     - /Users/mzagaja/.rvm/rubies/ruby-2.7.1/bin
     - /Users/mzagaja/.rvm/bin
     - /bin
     - /Users/mzagaja/.config/nvm/15.2.1/bin
     - /usr/local/bin
     - /usr/local/sbin
     - /usr/bin
     - /bin
     - /usr/sbin
     - /sbin
     - /Library/Apple/usr/bin
     - /usr/local/share/dotnet
     - ~/.dotnet/tools
     - /usr/local/MacGPG2/bin
     - /Library/Frameworks/Mono.framework/Versions/Current/Commands

If you're using Bundler:

what is the output from bundle version?
Bundler version 2.1.4 (2020-12-06 commit unknown)
what is the output from bundle config? (Take care to redact any credentials)

Settings are listed in order of priority. The top value will be used.
build.libv8
Set for the current user (/Users/mzagaja/.bundle/config): "--with-system-v8"

build.therubyracer
Set for the current user (/Users/mzagaja/.bundle/config<EMAIL_ADDRESS>
If you're on MacOS, please note:

the version of XCode you have installed (if you know)
the output of gcc -v or clang -v

Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin20.1.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin20.1.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

I have an Apple MacBook Pro M1 and am compiling Nokogiri native extensions for the first time on an application and it is taking 30+ minutes which seems, unexpected.

I ended up interrupting this after about an hour with no update in the UI.

@mzagaja Sorry you're having issues. You win some kind of prize, I think, for being the first person to try this on an M1.
Can you help me understand what you're seeing -- you didn't include any logs. Can you please tell us what you're seeing on stdout/stderr during gem installations (preferably gem install --verbose nokogiri) and -- even if you're interrupting it -- the mkmf.log file (which should be somewhere like /path/to/ruby-installation-dir/gems/ruby-2.7.2/gems/nokogiri-1.10.10/ext/nokogiri/mkmf.log).
Can you also please try installing a more-trivial C extension to make sure this isn't something wrong with the Ruby/compiler toolchain? Tell us what happens when you gem install --verbose bcrypt

Hi Mike,
Thanks for getting back to me. I figured someone has to go first 😉. So I didn't include any logs because none ended up generating from that attempt, I used CTRL+C to kill it. I got a signal interrupted message in the terminal but then had to manually kill the Ruby process in Activity Monitor as well.
At your advice I am now doing gem install --verbose nokogiri and it worked fine! I will note the version difference in this is 1.10.10 v. 1.10.8 in the bundle I'm trying.
This finally lead me to try bundle update nokogiri and predictably it worked fine.
For shits and giggles I also tried gem install --verbose bcrypt as suggested and it worked fine as well.
While the issue reproduced with 1.10.8, the fact it works with 1.10.10 makes me feel ok closing this and bumping my Gemfile up.

Thanks for closing the loop here. It's really strange that you're able to reproduce this with v1.10.8 but not v1.10.10, because the differences between those two versions are very small, and not obviously related to compilation/installation.

Agreed. https://github.com/Homebrew/brew/issues/7857 notes there was an issue with mkmf on the M1, and so it's possible this was related. I tried Googling and searching the repo for more details on this comment but wasn't able to surface the commit based on the reference they provided.

@mzagaja I have an experimental ARM64 Darwin native gem. If you want to help test in the next couple of days, ping me. If not, I'll ship it in the next RC and try to get feedback that way.

@flavorjones happy to give a try. Just let me know how to get connected and what might be useful to test. Otherwise I'll plug into existing app and run test suite.

Hey all - v1.11.0.rc4 has been released with pre-compiled native gem support for arm64-darwin. If you get a chance, please kick the tires and let me know here or at #2075 how it goes for you.
gem install nokogiri --prerelease should result in seeing something like:
$ gem install nokogiri --prerelease
Fetching nokogiri-1.11.0.rc4-x86_64-darwin.gem
Successfully installed nokogiri-1.11.0.rc4-x86_64-darwin
1 gem installed

except, you know, it should say arm64-darwin instead of x86_64-darwin.


YARN-6412 aux-services classpath not documented

YARN-6412
YARN-4577 introduced two new configuration entries yarn.nodemanager.aux-services.%s.classpath and yarn.nodemanager.aux-services.%s.system-classes. These are not documented in hadoop-yarn-common/.../yarn-default.xml

LGTM

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 40s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
38m 16s

trunk passed

+1 :green_heart:
compile
0m 50s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
compile
0m 43s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
mvnsite
0m 50s

trunk passed

+1 :green_heart:
javadoc
0m 55s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
0m 46s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
shadedclient
61m 38s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 38s

the patch passed

+1 :green_heart:
compile
0m 41s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javac
0m 41s

the patch passed

+1 :green_heart:
compile
0m 36s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
javac
0m 36s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 40s

the patch passed

+1 :green_heart:
javadoc
0m 37s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
0m 35s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
shadedclient
21m 44s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
5m 21s

hadoop-yarn-common in the patch passed.

+1 :green_heart:
asflicense
0m 36s

The patch does not generate ASF License warnings.

93m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5242/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5242

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint

uname
Linux f1582d25e56e 4.15.0-200-generic #211-Ubuntu SMP Thu Nov 24 18:16:04 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / da409611753e60f233e192f073b06129813bdf4f

Default Java
Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5242/1/testReport/

Max. process+thread count
557 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5242/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 37s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
39m  5s

trunk passed

+1 :green_heart:
compile
0m 49s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
compile
0m 45s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
mvnsite
0m 52s

trunk passed

+1 :green_heart:
javadoc
0m 57s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
0m 45s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
shadedclient
62m 53s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 39s

the patch passed

+1 :green_heart:
compile
0m 41s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javac
0m 41s

the patch passed

+1 :green_heart:
compile
0m 37s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
javac
0m 37s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 40s

the patch passed

+1 :green_heart:
javadoc
0m 37s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
0m 36s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
shadedclient
21m 49s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
5m 30s

hadoop-yarn-common in the patch passed.

+1 :green_heart:
asflicense
0m 39s

The patch does not generate ASF License warnings.

95m 13s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5242/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5242

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint

uname
Linux c2925d05135d 4.15.0-200-generic #211-Ubuntu SMP Thu Nov 24 18:16:04 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 3fb11400f645eae5224772a27f64c5a649381249

Default Java
Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5242/2/testReport/

Max. process+thread count
673 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5242/2/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Thanks @riyakhdl latest patch LGTM. Merged to trunk.