Open Tabs Next to Current is causing problems with Tab Session Manager

@seniori (Tab Session Manager developer):

The cause is probably that the tab specification of "Open Tabs Next to Current" is incorrect.
Please use other add-on.
https://addons.mozilla.org/ja/firefox/addon/always-right/

discussion/source: https://github.com/sienori/Tab-Session-Manager/issues/369

I have no idea what that means "the tab specification is incorrect"...
Seems to me like always-right uses this API: https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/browserSettings/newTabPosition which if I understand it correctly is the same as setting browser.tabs.insertAfterCurrent in about:config to true. So what's the point of having an extension for that at all? (see the README for this extension)

I have just wanted to let you know that there is a problem with Open Tabs Next to Current running together with Tab Session Manager. The description of problems I experienced can be found here: https://github.com/sienori/Tab-Session-Manager/issues/369 (approx. Apr 6, 2019). I hope that this will help you find the reason why this is happening.

It's probably not a problem on my side, I have some special handling to avoid re-ordering for restored tabs. And that works well for tabs restored by Firefox directly. So I assume it the way tabs are restored by tab-session-manager.

Maybe this will give you hint:

Open Tabs Next to Current + Firefox Session Restore + no new tabs (empty tabs) => no issue
Open Tabs Next to Current + Firefox Session Restore + new tabs (empty tabs) => no issue
Open Tabs Next to Current + Tab Session Manager + no new tabs (empty tabs) => no issue
Open Tabs Next to Current + Tab Session Manager + new tabs (empty tabs) => issue

I have problem with order of tabs in Firefox window only in the last case, when both extensions are present and there are new tab (empty tabs) present.
Please, check  sienori/Tab-Session-Manager#369 (approx. Apr 6, 2019) for details.


clob_bugfix...

Thank you for contributing to LangChain!

[ ] PR title: "package: description"

Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..."  for template changes, "infra: ..." for CI changes.
Example: "community: add foobar LLM"

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

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change
Twitter handle: if your PR gets announced, and you'd like a mention, we'll gladly shout you out!

[ ] Add tests and docs: If you're adding a new integration, please include

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/docs/integrations directory.

[ ] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Additional guidelines:

Make sure optional dependencies are imported within a function.
Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
Most PRs should not touch more than one package.
Changes should be backwards compatible.
If you are adding something to community, do not re-import it in langchain.

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17.

@baskaryan, @rohanaggarwal7997  I have fixed a bug relating to CLOB issues in oraclevs.py. I have published my changes to a branch clob_bugfix. Can you please approve the workflow.

Thanks Shailendra

@ccurme: The essential change is that in the INSERT, the CLOB columns (in this case metadata, text need to be at the end, or else one will get ORA24816 for large documents. The DDL does define a table structure in the order you outlined however we explicitly specify the order in which we need the INSERT to work hence that's the order that is taken.


Obfuscating code

Is there a way to obfuscate the code? thnx

You can obfuscate manually in
https://javascriptobfuscator.com

JPBuenaventura! yes! it works thanks


blivet-gui: init at 2.4.2

Description of changes
This adds:

blivet, a Python library for managing storage
blivet-gui, a GUI using said library (something like gparted, but also supports things like LVM)

Fixes #214073 and #257955
Obsoletes #260572

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

[ ]  Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

This gives me 1 package failed to build: libbytesize but does not provide any reason or details. Since I can't reproduce this locally any way I try, and the tool refuses to elaborate, I am forced to conclude that the tool is faulty. https://github.com/Mic92/nixpkgs-review/issues/200 seems to be the relevant bug report.

CC @icyrockcom (#214073), @siryoussef (#257955), @jfvillablanca (#260572), @aszlig (b0f4ae64c29289d024d22767b9ca0d582cf1db3e) 🙂

ping

ping

Yes, how to move forward here?

@AndersonTorres This is ready for review, if you'd lie to have a look.

i tried to nix run github:CyberShadow/nixpkgs/pull-20231224-104834#blivet-gui but i get error: nose-1.3.7 not supported for interpreter python3.12 did i run it wrong ?

@theotheroracle Looks like an instance of https://github.com/NixOS/nixpkgs/issues/325657. I've rebased and it seems to be working now.

@Aleksanaa Thank you for the review, I applied all suggestions except the adwaita-icon-theme one, as that breaks running the software for me.

Ah, never mind, looks like it was a problem with Nix on my PC.

@Aleksanaa Thank you for the suggestions, I think this is ready for another look when you have a moment.


strip api key

The api key loaded during initialization causes it to include a new line character. As a result the new line is passed onto the HTTP request which results in an error during any HTTP request.
ValueError: Invalid header value b'[redacted_key]\n'

This problem is solved by stripping the key during an initial loading of white space present before or after the API text.
Strip is safe to use here because it will never strip data inside the key, only leading and trailing white space will be removed.

Hi there,
We recently merged https://github.com/Nasdaq/data-link-python/pull/11 which tries much harder to strip whitespace and find the api key. Apparently there's still an edge-case that wasn't covered in one of the introduced tests?

1.0.1 has been pushed to pypi.  If you can re-try your use-case and let us know if your issue has been resolved.
I've opened issue #14 to collect any further issues users have, as it is recognized cleaning whitespace isn't solved for env-var use-cases.

Good Morning,
The API key stripping refereed to in the file is fixed now.


ENH: Modernizing vcl_ to std::

In all supported compilers, the need for vcl_ specialized
functions has been removed.  there is no longer a need
to have these overrides, and code is easier to read
and easier to maintain without these specializations.
The vcl_* definitions can now be greatly simplified.
After removing specializations for early non-conformant
c++ compilers the end result was that only the
std:: version of the functions could ever be
used by the compiler.
ITK_SCRIPT=ITK/Utilities/Maintenance/VCL_ModernizeNaming.py
SRC_BASE_DIR=$(pwd)
for ext in ".h" ".cxx" ".cpp" ".hxx" ".hpp" ".txx"; do
find ${SRC_BASE_DIR} -type f -name "*${ext}" \         -exec python ${ITK_SCRIPT} {} ;
done

Thanks for doing this, Hans.  I'll merge as soon as all the travis builds finish.

Hey Hans, do you delete these branches or do you prefer to keep them?

I delete them once merged.
Hans
From:  Nick Tustison<EMAIL_ADDRESS>Reply-To:  stnava/ANTs<EMAIL_ADDRESS>Date:  Monday, March 14, 2016 at 12:29 PM
To:  stnava/ANTs<EMAIL_ADDRESS>Cc:  Hans Johnson<EMAIL_ADDRESS>Subject:  Re: [ANTs] ENH: Modernizing vcl_ to std:: (#290)
Hey Hans, do you delete these branches or do you prefer to keep them?
—
Reply to this email directly or view it on GitHub.


ESA 2018 - logistics

Logistics to do's

[ ] KM confirm details with AB on travel
[ ] KM register, secure hotel and lodging
[ ] bulk up on swag (issue #32)
[ ] coordinate shipping of materials down to ESA / hotel

6 orgs across 4 booths.

3 booths have paired orgs (cafe / shared space / ), fourth booth is a "stand alone" for workshops, demos and talks. There will be a schedule for talks that include outside organizations.

messaging will be really important.

AB to check on full registrations vs booth registrations

Early bird deadline: June 14

Kathryn registered for ESA

Kathryn booked hotel for Sat 8/5 - Fri 8/10 in NOLA


First Login

Skip the password screen when initial onboarding.

The message on the home page will be that the wallet is not backed up. User can always go and do the backup from there.

Verified this issue in dev app v2.0.70(439).Password screen is removed but the loader modal is very quick it is visible for less than 1 second.Do we need to remove it as well or what?

Verified this issue on dev app v2.0.70(440)

Android Device:It is taking 3/5 seconds to enter into the wallet(Tested on One plus 9R)
ios device:It is taking 8/10seconds to enter into the wallet(Tested on iphone 7)

Unable to reproduce it on other devices so closing it for now.


32bit Crystal segfaults while compiling anything

Okay, we had this one coming I guess... :P
So, cross-compiling is fixed :tada: and actually produced fully working binaries:
64bit-machine $ bin/crystal build --cross-compile "Linux i686" --mcpu i686 --target i686-pc-linux-gnu -o crystal32 src/compiler/crystal.cr
64bit-machine $ scp crystal32.o 32bit-machine:
64bit-machine $ ssh 32bit-machine
32bit-machine $ cc crystal32.o -o crystal32   `(llvm-config-3.5 --libs --system-libs --ldflags 2> /dev/null) || (llvm-config --libs --system-libs --ldflags 2>/dev/null)` -lstdc++ -lpcre -lm -lgc -lpthread -lunwind
32bit-machine $ CRYSTAL_PATH="src" ./crystal32 eval 'puts "hi"'
hi

So, what to do with that shiny new compiler? Right, let's build a compiler
32bit-machine $ cp crystal32 .build/crystal
32bit-machine $ make

Okay, cool, yet another new compiler! Does it work though?

32bit-machine $ bin/crystal
[..]  shows the help menu, great!
32bit-machine $ bin/crystal eval 'puts "hi"'
Segmentation fault (core dumped)

Oh well. Let's add some debug output:
32bit-machine $ CRYSTAL_PATH="src" valgrind .build/crystal eval 'puts "hi"'
[...]
==9570== Process terminating with default action of signal 11 (SIGSEGV)
==9570==  Access not within mapped region at address 0x0
==9570==    at 0x86E59B1: *Crystal::TypeLookup#visit<Crystal::TypeLookup, Crystal::Path>:Crystal::Type+ (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x837ED76: *Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::ASTNode+, Crystal::TypeLookup>:Nil (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x81D2B57: *Crystal::GenericClassType+@Crystal::GenericClassType#initialize_instance<Crystal::GenericClassType+, Crystal::GenericClassInstanceType+>:Hash(String, Crystal::ASTNode+)? (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x81D28B6: *Crystal::GenericClassType+@Crystal::GenericType#instantiate<Crystal::GenericClassType+, Array((Crystal::Type+ | Crystal::ASTNode+))>:Crystal::GenericClassInstanceType+ (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x858F10A: *Crystal::Generic#update<Crystal::Generic, Nil>:(Crystal::Type+ | Nil) (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x858EA2F: *Crystal::Generic#update<Crystal::Generic>:(Crystal::Type+ | Nil) (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x869E11D: *Crystal::TypeVisitor#visit<Crystal::TypeVisitor, Crystal::Generic>:Bool (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x8387363: *Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::ASTNode+, Crystal::TypeVisitor>:Nil (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x85EA289: *Crystal::TypeDef#accept_children<Crystal::TypeDef, Crystal::TypeVisitor>:Nil (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x8388737: *Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::ASTNode+, Crystal::TypeVisitor>:Nil (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x842FB68: *Crystal::Expressions#accept_children<Crystal::Expressions, Crystal::TypeVisitor>:Array(Crystal::ASTNode+) (in /home/jhass/crystal/git/.build/crystal)
==9570==    by 0x838909C: *Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::ASTNode+, Crystal::TypeVisitor>:Nil (in /home/jhass/crystal/git/.build/crystal
[...]

If you need/want any of the binaries, just let me know.

Hmm... I think the reason might be that:

There's something in the codegen that is kind of hardcoded to work for 64-bits but not for 32
Some of the C bindings might not have an ifdef for the 32 bits case

Or maybe another reason.
I'll try to try this in a VM.
In the meantime you can try to compile the std specs (bin/crystal spec/all_spec.cr, but first comment the line related to the compiler specs in that file). Maybe one of those specs fail and we can have an idea of where the issue is.

Those were my first thoughts too, but then the cross compiled version should have the same bug, no? Currently I'm thinking that maybe we pass the wrong target triple if not explicitly given or something like that.
Spec compilation actually fails with
Error in /home/jhass/crystal/git/spec/std/oauth/consumer_spec.cr:4: instantiating 'describe(OAuth::Consumer:Class)'
describe OAuth::Consumer do
^~~~~~~~
in /home/jhass/crystal/git/src/spec/spec.cr:80: instantiating 'Spec::RootContext:Class#describe(String, String, Int32)'
  Spec::RootContext.describe(description.to_s, file, line) do |context|
                    ^~~~~~~~
in /home/jhass/crystal/git/src/spec/spec.cr:80: instantiating 'Spec::RootContext:Class#describe(String, String, Int32)'
  Spec::RootContext.describe(description.to_s, file, line) do |context|
                    ^~~~~~~~
in /home/jhass/crystal/git/spec/std/oauth/consumer_spec.cr:4: instantiating 'describe(OAuth::Consumer:Class)'
describe OAuth::Consumer do
^~~~~~~~
in /home/jhass/crystal/git/spec/std/oauth/consumer_spec.cr:30: instantiating 'OAuth::Consumer#get_request_token()'
    consumer.get_request_token(oauth_callback: "foo.bar.baz")
             ^~~~~~~~~~~~~~~~~
in /home/jhass/crystal/git/src/oauth/consumer.cr:12: instantiating 'with_new_http_client(Nil, Nil, Hash(String, String))'
    with_new_http_client(nil, nil, {"oauth_callback": oauth_callback}) do |client|
    ^~~~~~~~~~~~~~~~~~~~

in /home/jhass/crystal/git/src/oauth/consumer.cr:48: instantiating 'authenticate(HTTP::Client, Nil, Nil, Hash(String, String))'

    authenticate client, oauth_token, token_shared_secret, extra_params
    ^~~~~~~~~~~~

in /home/jhass/crystal/git/src/oauth/consumer.cr:58: instantiating 'authenticate(HTTP::Client, HTTP::Request, Nil, Nil, Hash(String, String))'

      authenticate client, request, oauth_token, token_shared_secret, extra_params
      ^~~~~~~~~~~~

in /home/jhass/crystal/git/src/oauth/consumer.cr:63: instantiating 'oauth_header(HTTP::Client, HTTP::Request, Nil, Nil, Hash(String, String))'

    request.headers["Authorization"] = oauth_header(client, request, oauth_token, token_shared_secret, extra_params)
                                       ^~~~~~~~~~~~

in /home/jhass/crystal/git/src/oauth/consumer.cr:71: instantiating 'OAuth::Signature#authorization_header(HTTP::Request, Bool, String, String)'

    signature.authorization_header request, client.ssl?, ts, nonce
              ^~~~~~~~~~~~~~~~~~~~

in /home/jhass/crystal/git/src/oauth/signature.cr:25: instantiating 'compute(HTTP::Request, Bool, String, String)'

    oauth_signature = compute request, ssl, ts, nonce
                      ^~~~~~~

in /home/jhass/crystal/git/src/oauth/signature.cr:21: instantiating 'OpenSSL::HMAC:Class#digest(Symbol, String, String)'

    Base64.strict_encode64(OpenSSL::HMAC.digest :sha1, key, base_string)
                                         ^~~~~~

in /home/jhass/crystal/git/src/openssl/hmac.cr:22: argument 'n' of 'LibCrypto#hmac' must be UInt32, not UInt64

    LibCrypto.hmac(evp, key_slice, key_slice.length, data_slice, data_slice.length.to_u64, buffer, out buffer_len)

I went on to just write a script that runs each spec file individually. All specs requiring the HMAC stuff (that's all oauth specs I guess), fail to compile with above, the following provide a segfaulting binary, As that takes a while, I'll update the issue once the results are there.

Alright, finished the test runs, see above ;)

Good! I just made two commits that fix some of the specs.

I changed the sizeof specs. The comments and the changes are self-explanatory.
I moved an annoying (and sometimes incorrect) cast error from compile-time to runtime.

Number 2 happens when you do this:
class Foo
end

class Bar < Foo
end

class Baz < Foo
end

foo = Foo.new || Bar.new # compiler treats foo as Foo+: Foo or any subclass of it
foo as Baz

You clearly can't cast foo to Baz because Baz was never instantiated. However, that cast might be in a code path that is only invoked if Baz is indeed instantiated, so issuing the error at compile time was wrong. Now you get that error at runtime.
For example, this program didn't compile:
require "./src/compiler/crystal/**"

Crystal::Program.new

The can't cast to Crystal::GenericClassInstanceType because it was never instatiated was that error, but with the new compiler it should be gone. So you can try to run the specs again and see what fails.
However, you are right that the 32-bits compiler will be already faulty, but the failing specs might give us a clue as to where the problem is.

Alright, after 5941a46a074fd079c4e0833ca0b65ede1903c053 and ab4fea60b7023d72399377feaa9b6baf5f6449c3 we have the failure in HMAC left and the following failure:
Failures:

  1) Complex sqrt
     Failure/Error: Complex.new(-2.2, 6.22).sqrt.should eq(Complex.new(1.4828360708935342, 2.0973323087062226))

       expected<EMAIL_ADDRESS>@imag=2.09733)
            got<EMAIL_ADDRESS>@imag=2.09733)

     # ./spec/std/complex_spec.cr:79

The following files fail because of above issues or segfaults:
spec/compiler/codegen/alias_spec.cr
spec/compiler/codegen/block_spec.cr
spec/compiler/codegen/c_struct_spec.cr
spec/compiler/codegen/c_union_spec.cr
spec/compiler/codegen/cast_spec.cr
spec/compiler/codegen/class_spec.cr
spec/compiler/codegen/closure_spec.cr
spec/compiler/codegen/const_spec.cr
spec/compiler/codegen/declare_var_spec.cr
spec/compiler/codegen/def_spec.cr
spec/compiler/codegen/exception_spec.cr
spec/compiler/codegen/fun_spec.cr
spec/compiler/codegen/generic_class_spec.cr
spec/compiler/codegen/if_spec.cr
spec/compiler/codegen/is_a_spec.cr
spec/compiler/codegen/lib_spec.cr
spec/compiler/codegen/macro_spec.cr
spec/compiler/codegen/method_missing_spec.cr
spec/compiler/codegen/module_spec.cr
spec/compiler/codegen/no_return_spec.cr
spec/compiler/codegen/previous_def_spec.cr
spec/compiler/codegen/super_spec.cr
spec/compiler/codegen/tuple_spec.cr
spec/compiler/codegen/union_type_spec.cr
spec/compiler/codegen/untyped_expression_spec.cr
spec/compiler/codegen/virtual_spec.cr
spec/compiler/codegen/void_spec.cr
spec/compiler/codegen/yield_with_scope_spec.cr
spec/compiler/compiler_spec.cr
spec/compiler/parser/parser_spec.cr
spec/compiler/type_inference/array_spec.cr
spec/compiler/type_inference/block_spec.cr
spec/compiler/type_inference/c_struct_spec.cr
spec/compiler/type_inference/c_type_spec.cr
spec/compiler/type_inference/c_union_spec.cr
spec/compiler/type_inference/cast_spec.cr
spec/compiler/type_inference/class_spec.cr
spec/compiler/type_inference/closure_spec.cr
spec/compiler/type_inference/declare_var_spec.cr
spec/compiler/type_inference/def_overload_spec.cr
spec/compiler/type_inference/def_spec.cr
spec/compiler/type_inference/did_you_mean_spec.cr
spec/compiler/type_inference/exception_spec.cr
spec/compiler/type_inference/fun_spec.cr
spec/compiler/type_inference/generic_class_spec.cr
spec/compiler/type_inference/initialize_spec.cr
spec/compiler/type_inference/is_a_spec.cr
spec/compiler/type_inference/lib_spec.cr
spec/compiler/type_inference/macro_spec.cr
spec/compiler/type_inference/method_missing_spec.cr
spec/compiler/type_inference/module_spec.cr
spec/compiler/type_inference/nil_spec.cr
spec/compiler/type_inference/no_return_spec.cr
spec/compiler/type_inference/previous_def_spec.cr
spec/compiler/type_inference/primitives_spec.cr
spec/compiler/type_inference/static_array_spec.cr
spec/compiler/type_inference/super_spec.cr
spec/compiler/type_inference/virtual_metaclass_spec.cr
spec/compiler/type_inference/virtual_spec.cr
spec/std/array_spec.cr
spec/std/big_int_spec.cr
spec/std/cgi_spec.cr
spec/std/complex_spec.cr
spec/std/enumerable_spec.cr
spec/std/hash_spec.cr
spec/std/inifile_spec.cr
spec/std/json/mapping_spec.cr
spec/std/json/parser_spec.cr
spec/std/json/pull_parser_spec.cr
spec/std/json/serialization_spec.cr
spec/std/oauth/consumer_spec.cr
spec/std/oauth/signature_spec.cr
spec/std/oauth2/access_token_spec.cr
spec/std/oauth2/session_spec.cr
spec/std/openssl/hmac_spec.cr
spec/std/set_spec.cr

This also fails (segfault), even though I fixed [some bindings[(https://github.com/manastech/crystal/commit/f445d124ebea357885d5785f1c0f941247167c3f):
PThread.create(out thread, nil, ->(data) {
  begin
    ret = Pointer(Void).malloc(1)
    PThread.exit(ret)
  rescue
  end
}, nil)

PThread.join(thread, nil)

Might be an issue with libunwind bindings or with LLVM exception handling (landing pad, etc.)

To continue, Dir.list is not working correctly in 32 bits. First, C::DirEntry must be defined like this:
      struct DirEntry
        d_ino : UInt32    # 4
        d_off : Int32     # 4
        reclen : UInt16   # 2
        type : UInt8      # 1
        name : UInt8[256] # 256
      end

And the function readdir must be that one, not readdir64 (both in https://github.com/manastech/crystal/blob/master/src/dir.cr).
However, it seems that dirent's d_type is always returning 0. And it seems that on some systems it's like that. So we would need another alternative for listing directories in linux 32 bits.

Seems fairly usable to me with last commits. Down to
spec/std/big_int_spec.cr
spec/std/complex_spec.cr
spec/std/oauth/consumer_spec.cr
spec/std/oauth/signature_spec.cr
spec/std/oauth2/access_token_spec.cr
spec/std/oauth2/session_spec.cr
spec/std/openssl/hmac_spec.cr

A couple of commits: 9eca7254260fc802f44ed10fd94e8fc7b550f44b and 165943551c048f8a8383e86005a8f3be9268a42f.
Now, on the VM I have (trusty 32 bits) complex_spec doesn't fail and Dir.list doesn't work well. Also, thread_spec (and the snippet related to threads) fails too. @jhass on which 32 bits linux are you?

".."
".git"
".editorconfig"
".gitignore"
".travis.yml"
"CHANGELOG.md"
"Dockerfile"
"Dockerfile.release"
"LICENSE"
"Makefile"
"README.md"
"Vagrantfile"
"bin"
"samples"
"spec"
"src"
".build"
".crystal"
"spec_test"

This is actually fixed from my side, do you still have issues on your side or did you fix them with the Dir refactor?

It's fixed, we can close it :-)
I just had it here as a reminder to make 32 bits releases, but I don't think we will forget to do that...


Plugin goes unresponsive after a couple days

Hi there -
I absolutely love this plugin. It works incredibly well with Homebridge/HomeKit, but I’ve found that it loses connection with the orbit servers after some time of inactivity.
Usually after a day or two, I need to restart the homebridge service to get updated status in HomeKit.
Is there any keep alive or retry logic built in? Not sure if it’s using web sockets, etc?
particularly, I have built some automation around detection of when the sprinklers turn on (even from the b-hyve app). Those automations stop working after a day or two until restart.

Thanks.
The updates are using a web socket connection which should auto reconnect. It will timeout in 30sec but the plugin send a ping on random interval between 20-25 sec which is set on start. My instance is not updating right now either. I take a look at this..

The connection can be interrupted all day long and auto reconnect, but it there is an error the connection would close. Unfortunately the ping used to keep the connection open would cause an error and hence close the connection.
Thanks for pointing this out, my network must be stable enough that I never noticed.
Fixed with a little better logging in the latest release.


Integration between Anserini and MatchZoo

The basic idea is to transform the run.* file into the mz format required by Matchzoo. The target is to run the ranking and reranking models in a shell script. I hope they are all written in python.
There are several points I am not clear:

How to get the raw documents from the document ID in the run.* file? One option is pyjnius + Anserini such as

import os
os.environ['CLASSPATH'] = "/home/larumuga/Anserini/target/anserini-0.0.1-SNAPSHOT.jar"

import jnius_config
from jnius import autoclass
JString = autoclass('java.lang.String')

index_test = autoclass('io.anserini.index.IndexUtils')
indexes = index_test(JString('/home/w85yang/Anserini/lucene-index-all.car18'))

print(indexes.getRawDocument(JString('7250e1b901bb59853deb38a452f9009999e790ae')

How to define train/test set? There should be no train/test splits in most TREC tracks since there are IR tasks but not ML tasks. My plan to allow user to DIY their own train/test split. For example, we can train on MB11 and test on MB13, or train on Robust04 and test on CORE17.

How to select negative samples? Some tracks provides negative samples like CORE17 but some only provide the relevant documents and all other documents are irrelevant like CAR17. We need to sample them. Can we use the query-doc pairs from run.* instead of qrels.* file as the training and test data for reranking? In CAR 17 I did both.

How to select sentences? For most tracks, the documents are long texts, which will cause a big efficiency problem in neural network reranking. One basic approach to solve this is to select some representative sentences for each document by the tfidf matching score.

Matchzoo is a toolkit for sentence pair modeling. (See https://github.com/faneshion/MatchZoo) It provides various NN models to predict the relevance scores of two sentences. Since NN becomes more and more popular in the IR area nowadays, I want to combine Anserini and Matchzoo to build a rank-reranking pipeline. Ranking in Anserini provides BM25, QL ... to retrieval documents from large corpus and reranking in Matchzoo accepts small and well labeled sentence pairs to predict new relevance scores.
Since the setup is different, there are some issues to be considered. For example, there is no training set in most TREC tracks. I am not familar with all tracks so I will start with CORE17 to see how it works.

Since you said you are going to use a shell script to run the process, I assume you are not targeting end-2-end?
Then I think what you need is probably a more power Python interface for Anserini?

Yes. It is only possible to Implement the end2end framework when we have a well-trained(offline) reranking model. My current plan is to generate the training data for reranking model training.
Yes. I'll try my best to do all of these in Python.

Result on the MB 2011 - 2014 corpus
I train the NN models using the golden labeled data from qrels files in MB2011, MB2012 and MB2013 and test the NN models on the run file generated by QL on MB2014. The results of the reranking are shown below:

Models
MAP
MRR
QP30

QL
0.4184
0.8408
0.6424

DSSM
0.2030
0.5977
0.3224

CDSSM
0.1599
0.3481
0.2182

DUET
0.2606
0.4551
0.3782

KNRM
0.2433
0.4702
0.3521

DRMM
0.3012
0.6317
0.4430

After preprocessing and fine tuning:

Models
MAP
MRR
P@30

QL
0.4184
0.8408
0.6424

DSSM
0.2258
0.5777
0.3533

CDSSM
0.1914
0.4192
0.2655

DUET
0.2606
0.4551
0.3782

KNRM
0.2530
0.5392
0.3527

DRMM
0.3778
0.7386
0.5588

So none of these reranking approaches improve over QL?

@arjenpdevries For my experiment on MB corpus, yes.
More papers on ad-hoc retrieval:
https://arxiv.org/abs/1606.04648 (Robust04)
https://arxiv.org/abs/1711.08611 (Robust04)
https://arxiv.org/abs/1805.05737 (MQ2007, MQ2008)
https://arxiv.org/abs/1706.06613 (search logs of Sougou.com)
http://aclweb.org/anthology/P18-1223 (search logs of Sougou.com)
http://www.cs.wayne.edu/kotov/docs/balaneshinkordan-cikm18.pdf (GOV2 DBpedia-v2 HomeDepot)

Result on the Robust04 corpus
I split the 250 topics in Robust04 into train/dev/test sets (200 for training, 25 for dev and 25 for test). Embedding is pretrained on Robust04 corpus using word2vec. For DRMM I use the same parameters described in the DRMM paper.
I train the NN models using the golden labeled data from qrels files and test the NN models on the run file generated by SimpleSearcher on the test topics. The results of the reranking are shown below:

Models
MAP
MRR
P@30

SimpleSearcher
0.2671
0.7699
0.3813

DSSM
training now, not good according to the dev performance

CDSSM
training now, not good according to the dev performance

DUET
0.0891
0.4103
0.1305

KNRM
0.1517
0.6439
0.2498

DRMM (the same parameters in the paper)
0.2458
0.7600
0.3453

DRMM (fine tuned)
0.2731
0.7717
0.3832

So the results basically match the numbers in the DRMM paper. I think NNs except DRMM fails to leverage the matching information of the whole passage (most of them do truncating at first 500 words).

After interpolation:

Models
MAP
MRR
P@30

QL
0.4184
0.8408
0.6424

DSSM+ (lambda=0)
0.4184
0.8408
0.6424

CDSSM+  (lambda=0.1)
0.4229
0.8531
0.6442

DUET+  (lambda=0.2)
0.4476
0.8364
0.6606

KNRM+  (lambda=0.1)
0.4389
0.8456
0.6564

DRMM+ (lambda=0.3)
0.4475
0.8708
0.6448

All results are higher than the baseline paper, because the NN reranking models accept the better QL(simple searcher) output.

Result on the Robust04 corpus

Models
MAP
MRR
P@30

SimpleSearcher
0.2671
0.7699
0.3813

DSSM+ (lambda = 0)
0.2671
0.7699
0.3813

CDSSM+ (lambda = 0)
0.2671
0.7699
0.3813

DUET+ (lambda = 0.05)
0.2735
0.7724
0.3954

KNRM+ (lambda = 0.15)
0.2713
0.7719
0.3908

DRMM+ (the same parameters in the paper) (lambda = 0.3)
0.2793
0.7784
0.3981

DRMM+ (fine tuned) (lambda = 0.35)
0.2914
0.7859
0.4015

I add the connection between Anserini and PACRR (https://github.com/khui/copacrr).

Models
MAP
MRR
P@30

QL
0.4184
0.8408
0.6424

PACRR
0.3980
0.7827
0.5576

Hi @Victor0118 and @lintool  thank you for providing the integration with MatchZoo. I am looking at documents describing this work: https://github.com/castorini/anserini/blob/master/docs/document-matchzoo.md However, I am confused regarding where respective training scripts can be found.
In particular, I cannot find the following scripts:
prepare_mz_data.py
matchzoo/main.py

I also checked the castor repository, but in vain. Is it some branch?
Could you guide me?
Many thanks!

Hi, @searchivarius. Thanks for your interest in this work.
This integration is between Anserini and Matchzoo v1.0. So you can find the script in the MatchZoo v1.0 here: https://github.com/NTMC-Community/MatchZoo/tree/1.0
I would suggest you use my repo since I have some updates based on the code above so that we can apply MatchZoo to Robust04 and Tweet datasets: https://github.com/Victor0118/MatchZoo/tree/rerank/data/robust04 and https://github.com/Victor0118/MatchZoo/tree/rerank/data/tweets

Hi @Victor0118 thanks a lot for the quick reply!


"{unkown}" type annotations could be more useful as "impl Trait"

Minimal example:
trait T {}

struct S;

impl T for S {}

fn f ()-> impl T {
    S {}
}

fn main() {
    let t = f();
}

In VSCode with the Rust Analyzer, if I hover over t (in main) I get {unknown}. I suppose it's too much to hope for the analyzer to look into the definition of f and compute that the type is S (in particular since in a more complete example the type might be unnameable). However, it would be quite useful if the Analyzer could tell me impl T instead of just {unknown}.

We just haven't implemented support impl Trait yet.

To expand on that a bit now that I'm at a keyboard: We're kind of waiting for support in Chalk for this, but we could actually already implement part of it:

add Ty::Opaque similar to the Chalk version
add an ImplTrait variant to TypeRef (the 'unresolved' type enum)
implement lowering from AST to TypeRef::ImplTrait and from that to Ty::Opaque
Then we would show impl T in your example, we just wouldn't actually know that that means it implements T ;)

@flodiebold Sounds fun, I will bug you for more details when I can find time to work on this.


Add type definitions

This commit add typescript type definitions. This allows Typescript
user to use this library out of the box. It also helps the editors
to provide auto-completion and quick documentation.
Closes substack/object-inspect#16

Typescript type definitions do not belong in this repo, see https://github.com/substack/object-inspect/issues/16#issuecomment-343682271

This project does not use TypeScript, and type definitions thus do not belong in this repo. The proper place to add type definitions is in the DefniitelyTyped repo.

Hi,
I know that this project is not written in Typescript itself, but it does not mean that it shouldn't host its own type definitions. DefinitelyTyped is more of a fallback solutions when there is nowhere else to put the definitions. Despite its bots, working with this huge repo is a pain because everything is tossed together. (It also has some other more fundamental flaws such as not supporting namespaced packages but that's another issue...)
Multiple libraries in pure JS that wish to support their users host their own definitions. This way they are closer to the source, and it's easier to ensure that they are up to date. This also a natural place for Typescript users to look at when they want to send improvements. In the case of object-inspect, there shouldn't be much change in the API, but in case you don't want to maintain the type definitions then someone will surely just send a PR when they need the types to be updated.
Here are some examples:

https://github.com/hiddentao/squel
https://github.com/vitaly-t/pg-promise
https://github.com/ForbesLindesay/throat

As mentioned in my first message, it also brings benefits for users only using Javascript. Smart editors can pull the typedefinition and use it for code completion.
Could you reconsider adding the type definitions?

A project that does not use TypeScript should not host its own type definitions. That some libs have chosen to do it doesn't mean it's the right way or it to work.
JavaScript has no types. This is a JavaScript project.

These project treat their hand-written type definitions as documentation that both humans and computers can understand. There is nothing wrong in this. Instead of Markdown it's .d.ts but there's not much difference.
Since you don't seem to agree I'll try the DefinitelyTyped route. If you change your mind some day, feel free to merge the PR. 😉


Local S3 for Development

the issue in the previous commit name was because of old config, it shouldn't affect new setups 🤞

@richardgoater When do you plan to merge this PR? I want to start working on my next task based on the changes in this PR.

@intsco I can merge it now


Dev 0.4

Codecov Report

Merging #183 (92df1fe) into main (2312251) will decrease coverage by 0.47%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
- Coverage   11.15%   10.68%   -0.47%
==========================================
  Files          35       36       +1
  Lines        1282     1338      +56
  Branches      221      225       +4
==========================================
  Hits          143      143
- Misses        925      977      +52
- Partials      214      218       +4

Flag
Coverage Δ

unittests
10.68% <0.00%> (-0.47%)
:arrow_down:

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

Impacted Files
Coverage Δ

packages/board/src/index.ts
0.00% <0.00%> (ø)

packages/board/src/lib/viewer.ts
0.00% <0.00%> (ø)

packages/core/src/index.ts
0.00% <ø> (ø)

packages/core/src/middleware/rule/index.ts
0.00% <0.00%> (ø)

packages/idraw/src/index.ts
0.00% <0.00%> (ø)

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


Updated news.md with the latest mkldnn submodule version

Description
As title
@aaronmarkham Please review.

@piyushghai Thank you for the change. Not sure if we also need have it on v1.4.x.

@piyushghai retrigger the CI?

@piyushghai retrigger the CI?

Done. @pengzhao-intel

@piyushghai Thank you for the change. Not sure if we also need have it on v1.4.x.

@TaoLv No we don't need it on the v1.4.x branch. The master branch contains the latest news info on the previous release.

Thank you. Merging now.


Minor fixes in the Translations

Sobretodo la falta de tildes.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Ready to merge @freeCodeCamp/translation-team

Thank you for your contribution.
Happy coding!


Roll Flutter Engine from 5702ec390eb5 to fc8aa9990c2f (1 revision)

https://github.com/flutter/engine/compare/5702ec390eb5...fc8aa9990c2f
2022-09-18<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from pNjLhqLsohBUBCSRa... to nIRDoJ-EZt7aynhoE... (flutter/engine#36237)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from pNjLhqLsohBU to nIRDoJ-EZt7a
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


[release-v1.15] Allow enabling sarama logging and disabling client pool and Upgrade knative.dev/eventing to latest 1.15

Backport of:

https://github.com/knative-extensions/eventing-kafka-broker/pull/4108
https://github.com/knative-extensions/eventing-kafka-broker/pull/4112

/retest-required


Insights fixed

when the score is between "bearish" and "bullish" the text in the signals should be "Neutral"
for example: if the "score":0.5, and "bearish":0.4,"bullish":0.6 -> Neutral

when toggle between the two selected tokens, the signals change for no reason.

Striked out items are done..

1. when the score is between "bearish" and "bullish" the text in the signals should be "Neutral"
for example: if the "score":0.5, and "bearish":0.4,"bullish":0.6 -> Neutral
2. when toggle between the two selected tokens, the signals change for no reason.
3. add "Powered by" before the into the block logo

You mean when the tokens are flipped the intoTheBlock tokens shouldn't move? How would this work when a different token is selected?


Enhancement: Some way of indicating the last time a Repository was (full?)index.

When i am troubleshooting a developers search that should return results but is not, i think it would be helpful to had some kind of indicator on the Repo to should the last time it was fully indexed or or even added/updated. I know i can look in the logs, but i think it also helps give developers confidence in the results..
I may look at it for something to down the road, but i wanted to throw it out there.. I have did a lot of parsing/indexing with Lucene indexes but not much with elasticSearch.

Good suggestion!  It wouldn't be hard to add an column to the config table that tracks the last time a complete index was performed and surface that to the user.

Would that be some where in src/main/java/com/palantir/stash/codesearch/updater/SearchUpdateJobImpl.java?
Or am i looking in the wrong place to add it


chore(main): release 0.4.8

:robot: I have created a release beep boop
0.4.8 (2024-10-31)
Bug Fixes

Bump xsmall CPU and fix preload (#359) (551f6d2)
File Uploader component (#361) (65950a2)
Home user ownership (#353) (fcb9e70)
Home user ownership (#354) (56b4efa)
Mkdir (56b4efa)
Remove tar after publish (#352) (da0b4b8)
Upgraded publish flow (#350) (2506246)
Use original working dir for all paths except file uploads (#356) (6399f34)
Warning messages (#357) (792bbff)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Zero-True/zero-true/releases/tag/v0.4.8 :sunflower:


Prompts make Chat GPT generate [LAST_PART]

Instead of showing "[CONTINUE]", [LAST_PART] starts to appear after 8-10 sections

This is improved in v1.4, and it should work much better.


Can't seem to trigger notification to a topic

Thanks for writing this SDK in Kotlin, it's amazing to see it supported as one of the first languages :heart:
I'm trying to figure out how to send a notification to a topic. In the docs it tells me to do this with the trigger endpoint but none of the constructor methods seems to allow for this input:
to: [{ type: 'Topic', topicKey: topicKey }],

Am I overlooking something or is this something that still needs to be added?

@nulllvoid @Crashiv Please check this


A second way to declare options

Currently (Before and for v1.0.0), options were/are declared like this:
entity A
entity B
entity C

paginate * with Something except A
paginate A with SomethingElse

The second way consists in copying the way the relationships are declared so that this syntax is added:
entity A
entity B
entity C

paginate {
    * with Something except A,
    A with SomethingElse
}

We wouldn't have to write the option's type every time we want to add an option to an entity.

Could you hold on with this. We need to discuss
On 18 Apr 2016 01:45, "Mathieu ABOU-AICHI"<EMAIL_ADDRESS>wrote:

Currently (Before and for v1.0.0), options were/are declared like this:
entity A
entity B
entity C
paginate * with Something except A
paginate A with SomethingElse
The second way consists in copying the way the relationships are declared
so that this syntax is added:
entity A
entity B
entity C
paginate {
* with Something except A,
A with SomethingElse
}
We wouldn't have to write the option's type every time we want to add an
option to an entity.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/jhipster/jhipster-domain-language/issues/10

Deepu, I'll postpone this for another time. After releasing v2.0.0 of JUML, I'll work on another huge project, so this can wait as it's not on my todo-urgently list.

Closing this, we don't need that now.


Password hash should use Argon and be usable without Nuxt context

The password hashing currently uses scrpyt, with no way to change this. But Argon2 is recommended nowadays for hashing of passwords, so I think this should be the default.
Moreover, it would be nice if the password methods could be exposed, say via nuxt-auth-utils/password so that one can use them eg in tests or other contexts where nuxt is not available (eg seeding in a db). For this, maybe extract the useRuntimeConfig call and pass the config as a parameter. Alternatively, use tryUseNuxtApp to check if a nuxt app is available and only then query the runtime config.

I decided to go with scrypt for a reason, it works on all JS runtime nowaday, once Argon2 will be supported, I will switch to it.
In the meantime, you can always use argon2 yourself


urg_c: 1.0.404-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository urg_c to 1.0.404-0:

upstream repository: git://github.com/ros-drivers/urg_c.git
release repository: https://github.com/ros-gbp/urg_c-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.19
previous version for package: null

urg_c
* Merge pull request #2 <https://github.com/ros-drivers/urg_c/issues/2> from dawonn/master
  Issue #1 <https://github.com/ros-drivers/urg_c/issues/1> - missing libmath link
* Ubuntu 14.04 Fix - missing libmath link
* Fix build for Android.
* Contributors: Chad Rockey, dawonn

Sorry, this seems to have collided with driver_common, please run the same command with -p added.

dies

:sob: It's ok it will be fast :rocket:


running the test failed

When I fired up the app, the backend sent an error saying file not found: "./models/funkSVD/2018-01-01 10:01:34.179393/model/". This directory does not exist anywhere. I run all the tests and any tests related to this file failed. Did I miss something here?

You didn't miss anything!
The error shouldn't be there, and I will see if I can fix it one of these days. The problem is that there should be a check to see whether there is a model before attempting to load it.
That said the site should work fine until you arrive at the part in the book that explains how to create the funk SVD model.

I can help you fix this error as well. I love your work on this recommender system - I really want to make it better. I'm thinking of adding a couple more models in addition to funkSVD and matrix factorization etc.


Latest nightly Adguard blocks QUIC connections

AdGuard version
7.15.0 nightly 26 (4434)
Browser version
Microsoft Edge Version 118.0.2088.76 (Official build) (64-bit)
OS version
Windows 11 Pro 23H2 (22631.2506)
What filters do you have enabled?
AdGuard Base filter, AdGuard Russian filter, AdGuard Tracking Protection filter, AdGuard Social Media filter, AdGuard DNS filter
What Stealth Mode options do you have enabled?
Hide your search queries, Strip URLs of tracking parameters, Disable cache for third-party requests, Block Flash, Turn off Advertising ID, Hide your Referrer from third-parties
Support ticket ID
No response
Issue Details

Enable Filter HTTP/3 in Adguard
Visit https://quic.nginx.org/quic.html
Make sure Adguard is filtering the website - check the website certificate
Run the test
Only HTTP connections are being established - dark gray pixels on the map.

Expected Behavior
QUIC connections are being established - green pixels on the map.
Screenshots

Additional Information
Tested with the latest stable/beta. Turning off traffic filtering and restarting the system helps. Switching traffic filtering from WFP to Proxy doesn't help. No third-party antivirus is installed, except Microsoft Defender.

@DavidOsipov Thank you for your report!
Edge is not supported now, it has the same problem as other Chrome-based browsers - certificate verifier does not allow User and Enterprise CAs for HTTP/3 connections.
Therefore, the only supported browsers for this feature are Firefox and Safari. There was a mistake in release notes.
The problem on quic.nginx.org is reproducable, we'll take a look.

@DavidOsipov The issue with https://quic.nginx.org/quic.html should be fixed in recently released version 7.16.

@sfionov
No, it's not. It's a pity, because time of my subscription is ticking.


NCP support in ot-br-posix

Use this issue to track PRs and todo tasks to support NCP in ot-br-posix.

[x] Add initial support for NCP in ot-br-posix

https://github.com/openthread/ot-br-posix/pull/2294
https://github.com/openthread/ot-br-posix/pull/2302
https://github.com/openthread/ot-br-posix/pull/2304
https://github.com/openthread/ot-br-posix/pull/2309
https://github.com/openthread/ot-br-posix/pull/2318
https://github.com/openthread/ot-br-posix/pull/2329
https://github.com/openthread/ot-br-posix/pull/2332
https://github.com/openthread/ot-br-posix/pull/2339
https://github.com/openthread/ot-br-posix/pull/2350

[x] Add basic control APIs (join, leave, scheduleMigration)

https://github.com/openthread/ot-br-posix/pull/2387
https://github.com/openthread/ot-br-posix/pull/2355

[x] Add scheduleMigration API

https://github.com/openthread/ot-br-posix/pull/2416
https://github.com/openthread/openthread/pull/10587
https://github.com/openthread/ot-br-posix/pull/2427
https://github.com/openthread/ot-br-posix/pull/2435

[ ] Add support of netif

[x] Unicast address update on wpan

https://github.com/openthread/ot-br-posix/pull/2410
https://github.com/openthread/ot-br-posix/pull/2419
https://github.com/openthread/ot-br-posix/pull/2431
https://github.com/openthread/ot-br-posix/pull/2437

[x] Multicast address update on wpan

https://github.com/openthread/ot-br-posix/pull/2447
https://github.com/openthread/ot-br-posix/pull/2460

[x] Netif state update

https://github.com/openthread/ot-br-posix/pull/2453
https://github.com/openthread/ot-br-posix/pull/2459

[x] Tx through wpan

https://github.com/openthread/ot-br-posix/pull/2452

[x] Rx through wpan

https://github.com/openthread/ot-br-posix/pull/2455

[x] TX/RX integration

https://github.com/openthread/ot-br-posix/pull/2467

[ ] subscribe/unsubscribe multicast addresses on wpan.

https://github.com/openthread/ot-br-posix/pull/2479
https://github.com/openthread/ot-br-posix/pull/2478

[ ] Support Border Routing

https://github.com/openthread/openthread/pull/10638

[ ] Support Service registration and publishing (Advertising Proxy)
[ ] Support Service discovery
[ ] Support Border Agent
[ ] Support Backbone Router
[ ] Support TREL

Hello @Irving-cl, I'm interested in this feature. Our product wants to support Matter over Thread on NCP arch. Hopefully, it will soon be completed.

Hi @Vanperdung, glad to hear that! However I cannot guarantee on the timeline. We will keep tracking the process on this issue.

Will start to submit the code for border routing. Since the code will be non-trivial, I will use this comment to give an abstraction of the implementation for reviewers to review more easily.
Implement NCP based version of otPlatInfraIf APIs
https://github.com/openthread/openthread/pull/10638 has added an empty version of these APIs. The remaining work is to implement them.

otPlatInfraIfDiscoverNat64Prefix will not be implemented temporarily because NAT64 will be implemented later.
otPlatInfraIfHasAddress: we will sync the IPv6 addresses of the infra if on the host to NCP and save them on NCP. When calling this API, we will directly search in the replica on NCP.
otPlatInfraIfSendIcmp6Nd: We will send a spinel property from NCP to the host and let host send the ICMP6 ND message.

New spinel commands

SPINEL_PROP_INFRA_IF_SETUP (host -> NCP)

This is used to initialize the "InfraIf" on NCP. The host will send InfraIfIndex, isInfraIfRunning and InfraIfAddresses to the NCP. The handler on NCP will save the addresses, call otBorderRoutingInit and otBorderRoutingSetEnabled.

SPINEL_PROP_INFRA_IF_STATE (host -> NCP)

This is used to notify NCP the change of infra if state or addresses. The host will send isInfraIfRunning and InfraIfAddresses to the NCP whenever there is any changes.

SPINEL_PROP_INFRA_IF_RECV_ICMP6_ND (host -> NCP)

When the host received an ICMP6 ND message, it forwards this message to the NCP using this spinel prop. And NCP handler will simply call otPlatInfraIfRecvIcmp6Nd to let OT core to handle it.

SPINEL_PROP_INFRA_IF_SEND_ICMP6_ND (NCP -> host)

When otPlatInfraIfSendIcmp6Nd is called on NCP, NCP will send this spinel property to the host. The spinel frame contains the InfraIfIndex, destAddress, Icmp6NdPacket.

Host InfraIf module
Like the Netif module, we will add a new InfraIf module to interact with the infraIf on the platform.

Initialization: The NcpHost will initialize the InfraIf if BackboneInterface is passed when starting the executable. Then theInfraIf will send SPINEL_PROP_INFRA_IF_SETUP to the NCP. We will also provide an API to initialize/Update the Infra after the executable already started (for Android use case).
InfraIf state update: A netlink socket will be created to receive netlink message about the state change (ON/OFF state and addresses) of the InfraIf. Whenever receiving a message, the module will send SPINEL_PROP_INFRA_IF_STATE to the NCP to update the state.
Receive ICMP6 ND messages (RA, RS, NA): A Icmp6 socket will be created to receive these messages from the platform. Then the InfraIf module will send a SPINEL_PROP_INFRA_IF_RECV_ICMP6_ND to the NCP to forward these messages and let NCP handle them.
Send ICMP6 ND messages: The InfraIf module will provide a method SendIcmp6Nd which sends the message though the Icmp6 socket as well. The method SendIcmp6Nd will be called in NcpSpinel when it receives a spinel frame of SPINEL_PROP_INFRA_IF_SEND_ICMP6_ND.

This comment gives an abstraction of the implementation of SRP Server & Advertising Proxy under NCP for reviewers to review more easily.
Implement NCP based version of otPlatDnssd APIs
Similar as NCP based version of otPlatInfraIf APIs, we need to have an NCP version of otPlatDnssd APIs to make Advertising Proxy work on NCP side. For SrpAdvertisingProxy, these otPlatDnssd APIs are required:

otPlatDnssdGetState

The ot::Dnssd module need to get the state from platform and then start/stop. We will sync the state of the mDNS Publisher on the host to the NCP and on NCP we directly return this state.

otPlatDnssdRegisterService / otPlatDnssdUnregisterService

The ot::Dnssd use the methods to register/unregister a serivce. We will implement them as sending a spinel frame to the host and the host will do the registration/unregistration by mDNS Publisher.

otPlatDnssdRegisterHost / otPlatDnssdUnregisterHost

Similar as above.

otPlatDnssdRegisterKey / otPlatDnssdUnregisterKey

Similar as above.

New Spinel Properties

SPINEL_PROP_SRP_SERVER_ENABLED (host -> NCP)

This property is used to set SrpServer enabled/disabled on NCP. When NCP receives it, it simply calls otSrpServerSetEnabled.

SPINEL_PROP_SRP_SERVER_AUTO_ENABLE_MODE (host -> NCP)

This property is used to set SrpServer auto enable mode on NCP. When NCP receives it, it simply calls otSrpServerSetAutoEnableMode.

SPINEL_PROP_DNSSD_STATE (host -> NCP)

This property is used to sync the mDNS Publisher's state to the NCP. When receiving it, the NCP simply stores this state and call otPlatDnssdStateHandleStateChange.

SPINEL_PROP_DNSSD_HOST (NCP -> host, command: Inserted/Removed)

This property is used to start request to the host to register/unregister a DNS host. We only define one property here and use command INSERTED/REMOVED to indicate whether this is a registration or unregistration.

SPINEL_PROP_DNSSD_SERVICE (NCP -> host, command: Inserted/Removed)

Similar as above. This property is for DNS services.

SPINEL_PROP_DNSSD_KEY_RECORD  (NCP -> host, command: Inserted/Removed)

Similar as above. This property is for DNS key records.

SPINEL_PROP_DNSSD_REQUEST_RESULT (host -> NCP)

This property is used to notify the NCP of the results for its DNSSD registration/unregistration requests. In the spinel frames of this property, there will be a request id to indicate the result is of which request.

Host side changes

The NcpHost will subscribe to mDNS publisher's state. Whenever there is a change, it will send SPINEL_PROP_DNSSD_STATE to the NCP.
When and how to start the Srp Server: Currently the host will either send SPINEL_PROP_SRP_SERVER_ENABLED or SPINEL_PROP_SRP_SERVER_AUTO_ENABLE_MODE in its initialization based on the build option. Later we may change this to starting SrpServer by an API.
Provide access of the mDNS publisher to NcpSpinel so that it can register Dnssd hosts/services/key records.


No license

Thanks for sharing this code.  Can you please add a license (e.g., MIT, BSD, etc.) so we know if we can use it for our applications?

Thank for pointing out my mistake.  I have just added a license file license.txt. This project is licensed under the terms of the MIT license.

Great!  Thanks for the fast response @akisato!


[Bug] Some characters considered invalid in embedded arguments

What happened?
There are still some issues regarding handling of embedded arguments. Some characters are denoted as invalid by W0301. The behaviour is very strange and depends on some things I consider insignificant. E.g. I have exactly this file:
*** Keywords ***
Perform A ${arg:\d+} ${arg2:\d+}
    Log To Console  ${arg}. ${arg2}

Perform ${arg:.*}
    Log To Console  ${arg}

Then in line 2, there both '$'s are denoted as invalid, and '{' in line 5 too. However, those keywords are written according to documentation and work perfectly when used in tests.
Besides, characters denoted as invalid depend on length of the keyword . E.g. when changing 'Perform A' to 'Perform ABCD', the first '+' is invalid instead of both '$'s. And when changing (the second) 'Perform' to 'Perform XY', '}' is considered invalid instead of '{'.
My configuration of the rule is -c invalid-char-in-name:invalid_chars:!.?/;+'"()[]{}#$%^&=<>|\
Operating System
Windows
Robocop version
1.10.0

invalid_chars:!.?/;+'"()[]{}#$%^&=<>|\ I see you're not playing around :D
I was not aware that regex could be used inside embedded variable - I used simpled matching system for this rule than the other that are is dealing with embedded vars. I will re-code it - thank you for test data as it will help me.


Various fixes for browser compatibility issues

Various fixes for browser compatibility issues seen in the examples.
In the library change Math.log2 to use ol/math/log2 and add a fallback for TypedArray.from()
Ensure examples which use the input event for range sliders also listen for change (as already done in Heatmap and WebGL filtering examples) as there are inconsistencies between browsers.

Are both input and change listeners needed?  If one will do, that would be preferable.  Also, the more explicit you can be about what issue you are trying to address (and in what browsers), the better.  When people revisit these changes in the future, they will appreciate the detail.

input usually fires while the slider is being dragged, so provides an immediate response.  change usually fires when only a move is complete.  This is quite an old answer https://stackoverflow.com/a/19067260/10118270 but IE11 still does not fire input (but does fire change when input would be expected) and Chrome now behaves as expected.  I have not tried Firefox or Safari, but their behaviour may affect https://openlayers.org/en/latest/examples/fractal.html which is the sole example which uses only the change event.


Added raw scanResult object to custom scanResult

so we can access things like isConnectable later on.
See issue #668

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

Hello @juliansteenbakker
Sorry for such a delay. Thanks for your contribution. Unfortunately I do not think that it is a good way to go. It would be better to backport the functionality to legacy scan results as well. With the current approach we would get back to fragmentation problem.

Hello @juliansteenbakker
Sorry for such a delay. Thanks for your contribution. Unfortunately I do not think that it is a good way to go. It would be better to backport the functionality to legacy scan results as well. With the current approach we would get back to fragmentation problem.

I agree with what you are saying, but the scanResult object is added in API 21. Do you have anything in mind on how we can backport this to older api's?

 All committers have signed the CLA.

Basically the same way it is implemented on Android side. ScanResult has a legacy constructor and shows legacy compatible behaviour.


Auto-regenerate LSP/JSON schema on pants.toml saving

No need to do it manually, that's just there while I sanity test a few things and to "force" it to happen.

Closed with #140


Manual: Explicitly mark commands that require to be run as root

This differentiates commands listed in the manual between requiring root and non-root. This means commands that require to be run as root are prefixed with # and the remaining commands are marked with $.
This documentation style has been observed in similar projects like the Archlinux Wiki and makes reading the documentation easier.
Implements #15073 (also see there for more discussion).

I just skimmed the commands, but the rest of changed ones does seem to be marked correctly. (I never used nixos-container, but when I try anything I get mkdir /etc/containers: Permission denied.)
Any comments from others?

Updated according to feedback.

I'd rather use sudo to be explicit about what needs root. # vs $ is very old school and requires common knowledge.

# vs $ is very old school and requires common knowledge.

Indeed, it isn't too obvious at first, that's why I included a note in the preface:
<note>Commands prefixed with <literal>#</literal> have to be run as
root, either requiring to login as root user or temporarily switching
to it using <literal>sudo</literal> for example.</note>

In its current state, the manual requires some basic knowledge about the *NIX environment and I think it can be assumed that the reader is already familiar with this convention (# vs. $), since it's used in a lot of documentation outside of the NixOS manual. Examples:

Archlinux documentation
Gentoo Handbook/Wiki
Fedora documentation
Debian installation manual/Wiki
(FreeBSD handbook)

(FWIW, the UNIX and Linux System Administration Handbook also uses this convention)
Places where I've seen sudo being used explicitly:

Ubuntu (help.ubuntu.com)

Interestingly, the Gentoo wiki explicitly prefixes commands with "root" and "user" in addition to # and $:

user $ emerge --search pdf
…
> ```
root # emerge --ask app-office/gnumeric

This looks pretty good. Is the sudo step a blocker, @domenkozar?

In my opinion, # vs $ is very opaque, I'd rather go for $ and $ sudo. Having said that, I think this PR is an improvement (once merge conflicts are resolved).

Rebased.

@domenkozar this PR doesn't have conflicts anymore.

Thank you!

Could it be that this breaks master?
i couldn't rebuild master until i reverted 5f8d14546...
but maybe my system is somehow screwed up
regards, iker

@ikervagyok "breaks"? Could you give some output?

sorry, edited my comment, to show what's going on - didn't expect this speedy response ;)

@ikervagyok Could it be that the note in the beginning is the problem?
@@ -26,6 +26,9 @@
     xlink:href="https://github.com/NixOS/nixpkgs/issues">NixOS’ GitHub
     issue tracker</link>.</para>

+    <note>Commands prefixed with <literal>#</literal> have to be run as
+    root, either requiring to login as root user or temporarily switching
+    to it using <literal>sudo</literal> for example.</note>
   </preface>

   <xi:include href="installation/installation.xml" />

Does it have to be wrapped in <para> tags? Sorry, I'm not too familiar with docbook...

@ikervagyok shouldn't you get an error because pandoc doesn't build?

pandoc is faulty too, atm - but i didn't follow the lastest haskell development, so i looked up this error first ;)

Does that fix it?
diff --git a/nixos/doc/manual/manual.xml b/nixos/doc/manual/manual.xml
index 42b2cf7..cd18b55 100644
--- a/nixos/doc/manual/manual.xml
+++ b/nixos/doc/manual/manual.xml
@@ -26,9 +26,9 @@
     xlink:href="https://github.com/NixOS/nixpkgs/issues">NixOS’ GitHub
     issue tracker</link>.</para>

-    <note>Commands prefixed with <literal>#</literal> have to be run as
+    <para><note>Commands prefixed with <literal>#</literal> have to be run as
     root, either requiring to login as root user or temporarily switching
-    to it using <literal>sudo</literal> for example.</note>
+    to it using <literal>sudo</literal> for example.</note></para>
   </preface>

   <xi:include href="installation/installation.xml" />

looks good to me, but something still breaks - maybe this causes it:
Invalid 8-Bit Character
In XML, the entire range of Unicode characters is available to you, but in SGML, the declaration indicates what characters are valid. The distributed DocBook declaration doesn't allow a bunch of fairly common 8-bit characters.
from: http://docbook.org/tdg/en/html/ch03.html

this is my error:
building Nix...
building the system configuration...
these derivations will be built:
/nix/store/pckvq4z0k27jcy02l29b516mxamm1m1p-nixos-manual.drv
/nix/store/iz7w91bnclfw24xa29jq9z8vndqbwrwd-nixos-help.drv
/nix/store/g0i2k68iv25q0rdmwijmbk76v1bjvcpc-system-path.drv
/nix/store/wlaiqi2wrgznf9qny6l1fcf5fr7qgw9p-dbus-conf.drv
/nix/store/9xqyppl6s1qzhavwx788vbpf0ij9i0in-unit-dbus.service.drv
/nix/store/jrvqnplw6ahfnfc9hzla4h36gnb4ln0w-unit-polkit.service.drv
/nix/store/7d47y7k1jlz5ipap2z1bybpczywxd062-system-units.drv
/nix/store/p3ah67vgbhyj1f79d6s1bwnlnvij857z-xsession.drv
/nix/store/lm2knq5l6yx28119z7p4fygw6nl2nprg-desktops.drv
/nix/store/ihbgkvn5bs9p718vwn3kcbmw16zszxmg-lightdm.conf.drv
/nix/store/rrl0w2lsgdl01ykgjfncnrr58bfpcpdc-etc.drv
/nix/store/zsabzhw5q5gd9vmx2w55sbq2i8wcr6aw-nixos-system-kobor-16.09.git.4ac191d.drv
building path(s) ‘/nix/store/9lgxl7ajb8rgzvgvgpp2fzlrcr0fx504-nixos-manual’
manual.xml:7: element info: Relax-NG validity error : Did not expect element info there
manual.xml:15: element para: Relax-NG validity error : Did not expect element para there
manual.xml:19: element para: Relax-NG validity error : Element preface has extra content: para
manual.xml:7: element info: Relax-NG validity error : Element book has extra content: info
manual.xml fails to validate
builder for ‘/nix/store/pckvq4z0k27jcy02l29b516mxamm1m1p-nixos-manual.drv’ failed with exit code 3
cannot build derivation ‘/nix/store/g0i2k68iv25q0rdmwijmbk76v1bjvcpc-system-path.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/zsabzhw5q5gd9vmx2w55sbq2i8wcr6aw-nixos-system-kobor-16.09.git.4ac191d.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/zsabzhw5q5gd9vmx2w55sbq2i8wcr6aw-nixos-system-kobor-16.09.git.4ac191d.drv’ failed
sudo nixos-rebuild boot -j 1 -I nixpkgs=. --show-trace  5.30s user 0.41s system 85% cpu 6.682 total
100 bali@kobor ~/github/nixpkgs (git)-[test] %

could be a pandoc follow-up error... just ignore me for now, looking at pandoc and the latest haskell-changes. maybe there's a low-hanging fix for this too

fixed in https://github.com/NixOS/nixpkgs/commit/81cdd0bf96f7a8932e079175366b8d679d615445

@domenkozar Thanks again. Sorry for the inconvenience.


How to upgrade 2.5 --> 3.5?

Hi,
I have an Apollo 2.5 vehicle and I want to run Apollo 3.5 on it. I was able to download and build the docker image, can launch DreamView, and play the demo Cyber record file. But it doesn't run with live sensors and it's not clear how to make it run nor does it seem to be documented anywhere.
My questions:

Is it possible to make Apollo 3.5 run on a 2.5 vehicle? Or is a 3.5 sensor set strictly required?
What configuration files do I need to edit to make it talk to the sensors?
In our 2.5 repo, is there any config I can simply copy/port over to over to my 3.5 repo?

Thanks!

@siqb thank you for your question and interest in Apollo!
Please refer to our Apollo 3.5 quickstart for details on both software and hardware upgrades. IN response to your specific questions:

It will not be possible, as Apollo 3.5 requires a different sensor setup and supports only Urban Road Scenarios where as Apollo 2.5 was specially developed for Highway driving
Apollo 3.5 works on Cyber RT whereas Apollo 2.5 was built on ROS. Please refer to documentation on Cyber RT to learn about it.
As mentioned, the versions serve different purposes.

Thank you for your reply, @natashadsouza. However, I'm not convinced about this part:

2. Apollo 3.5 works on Cyber RT whereas Apollo 2.5 was built on ROS. Please refer to documentation on [Cyber RT](https://github.com/ApolloAuto/apollo/tree/master/docs/cyber) to learn about it.

The docker container for 3.5 contains Cyber RT instead of ROS...after all, it's just user space middleware. As I mentioned in my original post, I can successfully play the demo Cyber record in the Docker container.

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


Fix XDM registry and npm test validation for user schema and its examples

Please link to the issue #724
Summary: user schema and its checked-in examples do not work for XDM registry which breaks at npm test schema validation stage. This PR marks user schema as deprecated. Since it is not known who uses user schema, suggestion from XDM team is to mark it deprecated instead of making backward incompatible changes to it. User schema is experimental. So, it should be okay to mark it to "deprecated".
This PR ensures that mandatory similar properties of principal do not become mandatory properties for new replacement schema - named - identity.schema.json. The replacement schema is in location that is covered by npm test which used to ignore user.schema.json and its example because of its location (/extensions).
Original issue -
I think the existing user.example.1.json example makes it seem that “principal” properties in user.schema.json were supposed to be optional. For example, @id is optional.
This is the current user.example.1.json
    {
      "xdm:identityProvider": "ims",
      "xdm:id": "C0B648DE57D701277F000101@AdobeID",
      "xdm:type": "https://ns.adobe.com/adobecloudplatform/ims/user"
    }

XDM registry rejects this example and expects a XDM object that matches user.schema.json to be like -
    {
      "xdm:identityProvider": "ims",
      "xdm:id": "C0B648DE57D701277F000101@AdobeID",
      "xdm:type": "https://ns.adobe.com/adobecloudplatform/ims/user"
      "xdm:provider": {
        "@id": "https://ims-na1.adobelogin.com/"
      },
      "@id": "C0B648DE57D701277F000101@AdobeID",
      "@type": "https://ns.adobe.com/adobecloudplatform/ims/user"
    }

This looks really awkward to have the similar values duplicated twice in xdm:id and @id. Similarly, @type and xdm:type. I think @type is not the preferred option compared to xdm:type as the latter has a namespace. This pull request drops import of principal.schema.json in identity.schema.json (user.schema.json's replacement). This would ensure that identity.example.1.json passes XDM registry test for identity.schema.json.

Please find a list of minor differences in identity schema compared to user schema -
1. "xdm:identityProvider" is a meta:enum with known value of "ims" instead of being a const. This lets the schema be used for other identity providers in future.
2. xdm:type is a meta:enum with a known value of ""https://ns.adobe.com/xdm/common/user" instead of a being a const "https://ns.adobe.com/adobecloudplatform/ims/user". The constant should not contain a specific identity provider name since a user is a user, and the value of this field should be a general user. This schema can be easily used for defining a group later on, if the property is an enum instead of being a const. So, it opens the door for that as well.
3. Dropping $ref to "https://ns.adobe.com/xdm/common/principal" based on issue raised above. The reference was causing issues with XDM registry validation for user.example.1.json and similar XDM objects that were built for now deprecated user.schema.json
4. The schema is marked extensible, so it can be used for defining other schemas if required.

This user schema is not used by AEP, so we can ignore the breaking change error as long as no other ACP service expects those fields.
Will let the ACP group review and merge this in

@cmathis The idea is that one AEP (collaboration) service would use it going forward.

I'm actually changing my mind on this one. We have no idea of knowing if this user schema is being used so removing principal would break existing users. Because of this we should mark the current definition as deprecated ("meta:status": "deprecated"), and create a new user schema file.

@cmathis Thanks for your feedback. Could you please review the new replacement schema. It additionally contains some minor changes. cc: @lrosenthol

We now have two "identity" schemas in the system. One with the $id of context/identity and this new common/identity.


OCM-12694 | fix: Pre-command validation for shared vpc flags

Adds validation for sharedvpc flags associated with HCP (account + op roles) before resources are created

/lgtm


How to RenewAccessToken using refresh token?

The initial RefreshToken that we get from User object is an empty string. So, it is not working while calling RenewAccessToken() to renew the access token. Is there any sample code which shows how to use it correctly?

Check out this thread

@ajinasokan Thanks for the link. Also, what is the time to expire of the access token after it is issued?

AccessToken expires every day morning and also when you log out.


Oauth authorization

I've added support for authorizing through OAuth, while maintaining the standard authorization method.

Hi,
Thanks for the pull request! It's mostly correct and can be merged as soon as some minor issues are tackled. I guess the biggest thing is that the oauth_authorized call can take app_key and/or redirect_url parameters.
Note that the app_key and redirect_url are only validated if passed due to requirements set by the oauth protocol and what data is available to the provider. Other than that I'd only remove the .rvmrc as this should work with any implementation of Ruby 1.8 or 1.9.
Do you want to these changes yourself or shall I?

Also, the new dependencies in the Gemfile should to go to the gemspec, the Gemfile (and lock) should be removed and replaced with:
source "http://rubygems.org"
gemspec

See http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/
Also preferrably, the Gemfile.lock should be added to .gitignore.

Oh, I thought the app_key would never be passed as a parameter in the case of OAuth.
Please do the changes yourself, I think we'll go faster this way.
It's OK to remove .rvmrc. You might also want to remove the travis.yml file as well, but I'm not sure it really works with 1.8.7... Travis complained about it.

ok, I'll take care of this!

OK, it's done. Gem version 2.2.8 contains your additions. Thanks for helping out :)
[In order to keep the history reasonably clean I squashed the commits]


Removing first column of the history.

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
Let's say we have a history of three commands:
> a
> b
> c

if you run history you will see:
 1  a
 2  b
 3  c

I would like to avoid seeing the first column when I do Ctrl+R. Basically doing:
history | awk '{$1="";$0=$0;$1=$1}1' FPAT='[^ ]+' OFS=' '
somewhere in __fzf_history__. But I didn't manage to get it right.
Any ideas?

For zsh, it is possible to use:

fc -ln 0

instead of history | awk ...

export FZF_CTRL_R_OPTS='--with-nth 2..'

Thanks, this helps! How much slower is fzf with --with-nth option?


JBPM-5050 and BZ1324245 - Consolidated common styles from workbench d…

…istributions

+1 to merge.

Thanks https://github.com/uberfire/uberfire/commit/41f841dbb65a6f42efa9893c16590b116434ded8


Scala3

👍

wow, really nice, are we going to migrate lila to Scala 3 as well?

btw, I think you have to regenerate github action with: sbt githubWorkflowGenerate

I did and it doesn't look right :thinking:

mm, let me take quick look.

This looks good. To make it more scala 3. You can use scalac option to rewite the wildcard import (from _ to *) or even crazier you can make it rewrite to braceless sytle (which is really nice imho).
So to rewrite wilcard import you can add -rewrite and -source:future to scalac options
To use braceless style you add one more option -indent
references:

https://docs.scala-lang.org/scala3/guides/migration/tooling-migration-mode.html
https://docs.scala-lang.org/scala3/guides/migration/tooling-syntax-rewriting.html

This looks good. To make it more scala 3. You can use scalac option to rewite the wildcard import (from _ to *) or even crazier you can make it rewrite to braceless sytle (which is really nice imho).
So to rewrite wilcard import you can add -rewrite and -source:future to scalac options
To use braceless style you add one more option -indent
references:

https://docs.scala-lang.org/scala3/guides/migration/tooling-migration-mode.html
https://docs.scala-lang.org/scala3/guides/migration/tooling-syntax-rewriting.html

Anything else block this PR? Except scalafmt of course.

Hello! Saw this has been left alone for a while so I've solved the merge conflicts with master locally, in order to get it going again. Do you want me to commit directly onto this branch or open a new PR onto the scala3 branch?
On another note, we could bump the scala version to 3.1.3. Or we wait a bit, since according to this forum post 3.2 should get released in a few days anyways.
Furthermore, I've noticed there were also some @ inline annotations, which could be replaced by the new inline keyword.

sounds good, please update the PR or make a new one, at your convenience.

To give you an update on my inactivity since, unfortunately I haven't been able to push. It's telling me I don't have write access to the repository. Tried making a new branch and pushing that, but no luck there either. Will keep trying to figure that out, whether it's an issue on my side.

You probably should fork the repo.


Querying at a workspace scope

HI,
I am running a query in the portal, the current scope is workspace.
When I want to change the scope, the portal warns me "You may only choose items from the same resource type."
This is not what this page says.

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

ID: d91b9858-da65-bfc9-429f-d5e2922cf6c2
Version Independent ID: 193e6f08-8b12-54c5-9eba-2a003eeb0b70
Content: Log query scope in Azure Monitor Log Analytics - Azure Monitor
Content Source: articles/azure-monitor/logs/scope.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@ciufudean Yes, as mentioned in the document , you can change the scope to log analytics workspace, application insights, resource group, subscription or other azure resources.
When you are trying to change the scope from log analytics workspace to any other resource, you might have to delete the current scope, which can be seen in bottom right page as highlighted in the image. Delete current scope and select as per your requirement.

@ciufudean As there are no further documentation changes, we will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


Updated to stop passing objects to celery (bug 1174219)

Closing for now as robhudson is no longer actively working on it.


20.0.13RC1, 21.0.5RC1, 22.2.0RC2

fail for hard-coded (not easily guessable) download urls of the release to upgrade from 🤔


Release v1.0.0

Initial release of component

@simu I fixed all your suggestions. Also changed the default fs from xfs to ext4, just to be more alligned with rook-ceph.


Tableish does not correctly parse some tables with colspans and rowspans

Tableish does not support many cases of tables that have rowspans and colspans. For example:

    a

    b
    c
    d

We've forked the project here: http://github.com/pivotal-benchmark/cucumber-rails and pushed our fix.
We've also test driven our fix, so check the tableish_spec.rb for all of the new coverage.
Thanks,
Jason LaCarrubba & Ian Zabel

Obsolete, since #tableish is deprecated (and removed in 1.2.0)


Requirements versions

Hi @peihaowang ,
I would like to experiment with your framework, but I am having some trouble with the dependencies mentioned in the repo's Readme
It would greatly help if you could provide the relevant requirements with their corresponding versions (for pip and/or conda)
Thank you!

Hi Thomas,
Thanks for your interest. I just had a quick trial. The following versions should work fine:
pytorch==1.12.1
torchvision==0.13.1
torchaudio==0.12.1
cudatoolkit=11.3
torch_geometric==2.2.0

I'd recommend to use conda to setup this environment especially for torch_geometric. Feel free to let me know any further questions.
Best,
Peihao


Hue Group features based on the bulbs in it

Computes the features of a hue group as the union of the features of the bulbs in the group.
Home assistant threats groups of bulbs create on the hue app as if all the bulbs in it had all possible features (brightness, color temperature and color, etc...), even if the group is only composed by simpler bulbs.

Breaking change

May break scripts that relay on information of bulb groups that might no longer exist. For instance, information about color of a group that does not have any bulb that supports color. This informations must be removed from the script.
Proposed change

Change the way the feature of a group is computed, by calculating the feature of a group as the union of the features of the bulbs in the group. As such, a group composed by bulbs that only have brightness and color temperature control (as the hue ambiance bulbs) will only display these options in the user interface. Simplifying the interface by remove unsupported features.
This change is specially important for the homekit integration. In the Home App, a group of bulbs containing only hue ambiance bulbs would had shown the color option as well, since the ambiance bulbs have no color option, it would ignore the color changes made through the Home App, making it not possible to change even the color temperature of the light (a feature that the hue ambiance bulbs do support).
Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml

Additional information

This might be related to issue #31784.
Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist
[x] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

Hi @guimaraes13,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Thanks, that's a great contribution !
Please make sure to add a test to verify that the change works.

Hello @balloob, thank you for the comments. I made the changes and added the test case.


Feature/ffs 72 multiplier selection option

In this request we add the option to enable an multiplier option inside the settings with a option to set a daily limit.
With this it is possible tho differ between sponsor and foundation based on if this option is set. (If set its a foundation)

Code is good, adjust the variable names and i'll merge it.


Node identification and operations

This issue concerns all issues related to:

node identification
operating on nodes (merging / adding /removing and the corresponding updates in the underlying datastructure of GeoGraph, such as updating ids, areas or the rtree for example).

With the new geograph object now ready, I've implemented a vanilla version of node identification with the new RTree now. We can now identify nodes based on their relative positions.
(C) Nodes are currently identified if:

they have the same class label
they intersect/touch in some way (with possibility to disallow touching at the corners only)

(L) Current limitations:

if nodes do not touch/intersect at all, they will not be identified. This is the desired behavior for most use cases on the dense graph as I understand. For sparse subgraphs we might want the option to search the nearest candidate of the same class and add that one if it's "reasonably similar" and "closer than X tolerance" (except for maybe what Toby wanted with his identification of trees)
If we check for corners (when disallowing corners only) we incur a slight slow-down (I'm currently doing a DE-9IM pattern match computation for that). Allowing corners is about 50% faster in the cases where corner-tests actually need to be performed (otherwise no notable difference).
The identification (allowing corners) takes about ~140 us on sci1 for one node in my test cases. This means we can identify about 3000 nodes per second
From my line-profiling investigations, it seems that the largest bottleneck is actually querying the RTree (takes up over half the time). Future speed-ups might be gained (in order of importance) by speeding up the rtree query, by directly leveraging GDAL implementations of the DE-9IM pattern computations and by vectorising the operations.

Images :
Facecolors = class labels
Blue edge = selected node for which to find identified nodes in other_graph
Green edge = nodes that are identified with the blue node

Closed by #28


ADD: Magento Log Viewer extension to the README

This pull request includes an update to the README.md file to add a new tool for Magento 2 developers. The change is straightforward and involves adding a new entry to the list of tools available for Magento 2.

README.md: Added a new entry for the "Magento Log Viewer - A Visual Studio Code Extension" which provides a convenient way to view, watch, and manage Magento log files and reports directly in your workspace.

Cool thing <3


chore(deps): Upgrade react native version

Description
This PR upgrades react native and expo versions to latest.

:tada: This PR is included in version 3.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Auto generate og-image for better SEO

I just fork to this repo https://github.com/denoland-id/og-image
I will working on it to suit denoland indonesia logo

Done, @grikomsn you can deploy to vercel maybe with domain og.denoland.id


AddColumns Not working in latest version of powerapps

Please fix documentation to show how to properly use it. Or fix it so it operates according to documentation.

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

ID: 64e1f6e9-01be-ec20-e777-643874d92c2a
Version Independent ID: 6b83ce58-9ea8-5f36-d518-6ef86c07d206
Content: AddColumns, DropColumns, RenameColumns, and ShowColumns functions - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/functions/function-table-shaping.md
Service: powerapps
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Here's a picture of the function. AddColumns doesn't work in adding the column I labeled TestColumn.
I use the ForAll loop as a workaround to fix this issue.

Hi John.  AddColumns does not add a column to a table or collection.  Tables in PowerApps are values just like a number or text string and can be modified as they flow through a function.  As the docs state "The functions that this topic describes don't modify the original table. Instead, they take that table as an argument and return a new table with a transform applied. See working with tables for more details."
The ForAll you used isn't bad.  You could also use AddColumns with Collect:
Collect( 'Sandbox Building', AddColumns( OpenIncidents, "Distance", (IncidentLat-Lat)+(IncidentLon-Lon) ) )

Tagging @JohnGGeo for visibility. If @gregli-msft answered your question, please let us know so that we can close the issue. Cheers.


Build Warning Reduction (Code Style Standardization)

Associated to #2648
Summary
Includes resolution of a number of specific types of build warnings

IDE0028
IDE0030
IDE0031
IDE0017
IDE0029
Reduces the number of build warnings from 667 to 469.

Review Notes
This is designed to help improve the standardization & readability.  With an eventual goal of being able to validate that no additional warnings occur in the future.

Those IDE build warnings are built-in style rules in Visual Studio.  They can be configured via EditorConfig, and we should probably attempt to create a config for the style to which we want to adhere.

@bdukes this is what I was thinking as well. We should really think about adding an EditorConfig

@bdukes @ahoefling Are you proposing we hold off on this PR until we have the correct config in the editorconfig? Or do we merge and worry about the config later?

I'm fine with merging as is, we can look at updating configuration in the future

Will be resubmitted in Dnn 10


docs: Table of Contents partially hidden (sometimes)

If the Table of Contents overflows, then that portion is unavailable (and invisible).

Closed by request. :)


 Who thought it was a good idea to write "Sanity check"? Who wrote this? Ableism is against the code of conduct. Replace it with a more neutral term, like "non-specific-normal change detect" or something, please.

Agreed. This kind of speech is hateful and unwarranted. Devs please correct this.

both @NaamanIskowitch and @benixlinux joined github today.
http://niceme.me/

That's one weird way to spam 😕

@NaamanIskowitch @benixlinux it gets worse! look at line 100! 😮
it has the word [l, e, g] in it! sickos!

With recent Redis and Pythin stuff I beg Crystal team to ignore such things.

Yes, just close this and lock it, don't give any room to spam

With recent Redis and Pythin stuff

@vladfaust That is...?

both @NaamanIskowitch and @benixlinux joined github today.
http://niceme.me/

"people who have just joined the community should be shunned"
Yep, this is a fash community

@Sija @vladfaust both @benixlinux and @NaamanIskowitch are the same guy. it's just a troll.
when the trolls arrive, you know you've made it big 👍


mysite.com -> example.com

As described in RFC 2606 and RFC 6761

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.
[ ] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

Test Plan

Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/
Related issues/PRs

Ran into this one just a moment ago while using examples from the docs and landed on the website as per the image here:

Better to use example.com as it is reserved for this (https://www.iana.org/help/example-domains)

closing because I do not have the correct commit message according to the guidelines


cw inserting a tab in the output

Goodmorning.
There is an unusual behaviour I'm seeing with cw 0.7.0 (from AUR).
Executing any command it returns the correct result but with a tab (or 4 to 6 spaces) in the stdin.
Example: cw vs wc

The last release was from 2019 so maybe you've already fixed this, but please update the release here and on AUR if so.
Thanks in advance.

It's just fixed field padding, which is common to most wc implementations.  GNU is a special snowflake in that it stats all its input files and sets a field width based on their combined size - and also special-cases the single-input case to disable padding.
e.g. on FreeBSD:
❯ cw -l ~/.zshrc
     401 /home/freaky/.zshrc
❯ wc -l ~/.zshrc
     401 /home/freaky/.zshrc
❯ gwc -l ~/.zshrc
401 /home/freaky/.zshrc

❯ wc -l lots-of-lines .zshrc
    1000 lots-of-lines
     401 .zshrc
    1401 total
❯ cw -l lots-of-lines .zshrc
    1000 lots-of-lines
     401 .zshrc
    1401 total
❯ gwc -l lots-of-lines .zshrc
 1000 lots-of-lines
  401 .zshrc
 1401 total

I don't particularly feel like there's a good argument for the added complexity of the GNU approach.

Well, thanks for the clarification then. I'll just remove the padding manually on output with another piped command then.

That's probably best, anything else is going to be non-portable.  To sum up:

Free/Net/Open/DragonFly/other BSD/Solaris/Illumos/SmartOS/etc: Static padding on all outputs
GNU: Dynamic padding if multiple inputs based on total stat() size
Busybox: Static padding except on single-input single-count.
uutils coreutils: Unconditional dynamic padding based on total bytes counted

So cw's following the most common approach.


chore(zappr): require approval of Norris+UX to merge

Closes #273

👍

Let's wait until we merge all the tasks today in order to speed it up and then we enable it.


Windows MSI doesn't install jdk where installation.html says it should

The Installation page says that the Windows MSI installer package by default installs Temurin to C:\Program Files\Temurin\

However, the installer actually installs, by default, to C:\Program Files\Eclipse Foundation\

Confirmed locally; there was no Temurin folder in my C:\Program Files; noticed after an uninstall & reinstall that it was instead in C:\Program Files\Eclipse Foundation\

So it nows installs to Eclipse Adoptium and has done for some time. I've fixed the path in the website-v2 docs (https://github.com/adoptium/website-v2/commit/ddc4c89e850dfc9d43cd4ba2dab8ec6b3187fc89)


Update _extra.tab

To supersede https://github.com/citation-style-language/styles/pull/2022

just double checked and it is in the _journals.tab. Don't know why I didn't find it the first time. Weird. Sorry for the false alarm.

just double checked and it is in the _journals.tab. Don't know why I didn't find it the first time. Weird.

Yeah, you couldn't really have known, but the journal metadata in this repo for T&F hasn't been synchronized the repository in quite a while. See the comment thread at https://github.com/citation-style-language/utilities/commit/2b53ca8deba7e63373aec3bca5265619daae280f. We were hoping to get fresh metadata from T&F before attempting this, but it's often not easy to get a hold of the right person.
For now I just generated this particular style from the metadata and added it by hand: https://github.com/citation-style-language/styles/commit/22a8adc436472ed8df255e6723764f31dc38f3a3

(it would also be good if we kept better track of when metadata was last comprehensively updated for each publisher, and make sure we're alerted when the "journals" repo is out of sync with the "styles" repo)

(I created a ticket for this at https://github.com/citation-style-language/journals/issues/30)


About the reported performances of DM

Your generously provided code helps me a lot for implementing new thang from existing methods.
Having said that, existing code do not show the reported performances of DM, even when i reproduced it with same hyper-parameters.
In detail, results for CIFAR10 with 50 samples per classes does not go over 55% even when i try bunch of hyper-parameter tunings.

Hi, I have run the code one more time on a new environment just now. It achieves the reported result on 50 ipc CIFAR10.


fix: should error for unhandled scheme

Summary

Test Plan

Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

bump, @ahabhgk this need updated

#6404


Ensuring that all snapshot tests return results.

Coverage decreased (-0.2%) to 94.594% when pulling 4b9ab85adb737272d9f0b999449c5f7e18786b87 on feat_snapshot_tests into ef77bf9d170f3f5d62b45b4c2c04617a31fe6f7c on master.